Merged
Conversation
hallipr
added a commit
that referenced
this pull request
Aug 20, 2025
17 tasks
Merged
g2vinay
added a commit
to g2vinay/mcp
that referenced
this pull request
Feb 11, 2026
- Enhanced monitor_webtests_createorupdate description with action-oriented language and clear use cases - Fixed documentation consistency in monitor_webtests_get (use --webtest-resource instead of webtest-name) - Tool evaluation results: createorupdate now ranks microsoft#1 for update scenarios (50% improvement) - monitor_webtests_get maintains 100% Rank microsoft#1 performance (score: 0.767)
g2vinay
added a commit
to g2vinay/mcp
that referenced
this pull request
Feb 11, 2026
- Enhanced monitor_webtests_createorupdate description with action-oriented language and clear use cases - Fixed documentation consistency in monitor_webtests_get (use --webtest-resource instead of webtest-name) - Tool evaluation results: createorupdate now ranks microsoft#1 for update scenarios (50% improvement) - monitor_webtests_get maintains 100% Rank microsoft#1 performance (score: 0.767)
g2vinay
added a commit
to g2vinay/mcp
that referenced
this pull request
Mar 12, 2026
Fixes microsoft#1994 - E2E test prompts in Azure Deploy do not trigger corresponding MCP tools in All and Namespace mode. Changes: - DiagramGenerateCommand: Fix concatenation bug (missing spaces between sentences), rewrite description to lead with user-facing purpose and use 'generate/create/visualize architecture diagram' keywords - RulesGetCommand: Replace 'guidelines' language with explicit 'rules and best practices' keywords that match natural user prompts; now scores above bicepschema_get (0.5077 vs 0.4591) - GuidanceGetCommand: Remove 'DO NOT call this tool' negative instruction that suppressed tool selection; lead with CI/CD pipeline creation purpose - GetCommand (Plan): Replace 'Entry point to help the agent' jargon with user-facing 'Creates a deployment plan' language; score improved from 0.5870 (rank microsoft#3) to 0.7173 (rank microsoft#1, tied with actual tool) - e2eTestPrompts.md: Update Azure Deploy test prompts to match the simpler natural-language prompts from the issue report Evaluator improvements (ToolDescriptionEvaluator scores): deploy_architecture_diagram_generate: 0.7260 -> 0.7486 (rank microsoft#2 -> microsoft#1 tie) deploy_iac_rules_get: 0.4312 -> 0.5077 (rank microsoft#2 -> microsoft#1 tie, beats bicepschema_get) deploy_pipeline_guidance_get: 0.6396 -> 0.6614 (rank microsoft#2 -> microsoft#1 tie) deploy_plan_get: 0.5870 -> 0.7173 (rank microsoft#3 -> microsoft#1 tie)
rajkumar-rangaraj
pushed a commit
to rajkumar-rangaraj/mcp
that referenced
this pull request
Mar 20, 2026
…Instrumentation_rest Switch ASP.NET Core greenfield onboarding to Application Insights 3.x
rajkumar-rangaraj
pushed a commit
to rajkumar-rangaraj/mcp
that referenced
this pull request
Mar 20, 2026
…Instrumentation_rest Switch ASP.NET Core greenfield onboarding to Application Insights 3.x
xiang17
added a commit
that referenced
this pull request
Mar 20, 2026
* Migrate remaining functions * Fix unit tests * Update missing changes * Sync code changes * Merge pull request #1 from rajkumar-rangaraj/xiang17/MonitorInstrumentation_rest Switch ASP.NET Core greenfield onboarding to Application Insights 3.x * Update according to instructions in new-command.md * Fix "error WHITESPACE: Fix whitespace formatting." with `dotnet format Microsoft.Mcp.slnx` * Remove "related:" sections * Convert internal markdown links * fix 404 link * Update changelog and remove unused line * ToolMetadata update * address Copilot feedbacks * merge with rest of Azure Monitor operations in e2eTstPrompts.md * Fixed OpenWorld ToolMetadata attributes and updated consolidated-tools.json file * Sync missing logic from standalone instrumentation MCP server - Add nuget-vs package manager case with Install-Package command and Package Manager Console instructions for classic ASP.NET scenarios - Add XML config file handling in AddConfig (ApplicationInsights.config, Web.config) instead of always emitting JSON instructions - Restore detailed BuildAnalysisInstruction with per-section guidance for brownfield analysis (AAD auth, telemetry types, Activity.Current) - Update AnalysisTemplate placeholders with comprehensive examples (all Track methods, IConfigureOptions, AAD auth mentions) - Restore BuildValidateInstallInstructions with FilesToExist and FileContentChecks support for validate-install actions - Add missing middleware telemetry access and manually-constructed telemetry object sections to migration doc * Fix nuget-vs to match standalone and add missing natural language sentence * update changelog to include the full list of supported languages and frameworks * reword changelog * Rename SendEnhancedSelection to SendEnhancementSelect --------- Co-authored-by: Rajkumar Rangaraj <rajrang@microsoft.com>
21 tasks
colbytimm
pushed a commit
to colbytimm/microsoft-mcp
that referenced
this pull request
Apr 20, 2026
…soft#2115) * Migrate remaining functions * Fix unit tests * Update missing changes * Sync code changes * Merge pull request microsoft#1 from rajkumar-rangaraj/xiang17/MonitorInstrumentation_rest Switch ASP.NET Core greenfield onboarding to Application Insights 3.x * Update according to instructions in new-command.md * Fix "error WHITESPACE: Fix whitespace formatting." with `dotnet format Microsoft.Mcp.slnx` * Remove "related:" sections * Convert internal markdown links * fix 404 link * Update changelog and remove unused line * ToolMetadata update * address Copilot feedbacks * merge with rest of Azure Monitor operations in e2eTstPrompts.md * Fixed OpenWorld ToolMetadata attributes and updated consolidated-tools.json file * Sync missing logic from standalone instrumentation MCP server - Add nuget-vs package manager case with Install-Package command and Package Manager Console instructions for classic ASP.NET scenarios - Add XML config file handling in AddConfig (ApplicationInsights.config, Web.config) instead of always emitting JSON instructions - Restore detailed BuildAnalysisInstruction with per-section guidance for brownfield analysis (AAD auth, telemetry types, Activity.Current) - Update AnalysisTemplate placeholders with comprehensive examples (all Track methods, IConfigureOptions, AAD auth mentions) - Restore BuildValidateInstallInstructions with FilesToExist and FileContentChecks support for validate-install actions - Add missing middleware telemetry access and manually-constructed telemetry object sections to migration doc * Fix nuget-vs to match standalone and add missing natural language sentence * update changelog to include the full list of supported languages and frameworks * reword changelog * Rename SendEnhancedSelection to SendEnhancementSelect --------- Co-authored-by: Rajkumar Rangaraj <rajrang@microsoft.com>
KarishmaGhiya
added a commit
that referenced
this pull request
Apr 22, 2026
…bility
- Fix typo in DetectorListCommand description ('detectors detector')
- Add keywords: troubleshoot, performance, availability, slow, failing, unhealthy
- Improve ranking in ToolDescriptionEvaluator from Fair/Poor to Excellent/Good
- DetectorListCommand: now ranks #1 for 'list diagnostic detectors' prompt
- DetectorDiagnoseCommand: now ranks #1 for 'check why my web app is slow' prompt
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
shrja-ms
added a commit
to shrja-ms/mcp
that referenced
this pull request
Apr 23, 2026
Three independent bug fixes for azurebackup tooling, surfaced via end-to-end testing against a live subscription. Bug microsoft#1 — `azurebackup vault create --vault-type dpp` created Data Protection backup vaults without a Managed Identity. Every subsequent `protecteditem protect` against the new vault then failed server-side with `VaultMSIUnauthorized`. Fixed by defaulting new DPP vaults to a System-Assigned Managed Identity, matching Portal and CLI behaviour. Bug microsoft#2 — `azurebackup protecteditem protect` returned a synthetic "Accepted" response with a base64-encoded job ID that callers could not resolve. Fixed to surface the true terminal outcome: - RSV: poll the underlying ConfigureBackup job (max 12 min) and return its real status (`Completed`, `CompletedWithWarnings`, `Failed`, `Cancelled`, or `InProgress` if polling times out) with the real job ID and error details (AOT-safe typed switch). - DPP: wait for the LRO to complete, read back the BackupInstance, and return the actual `protectionStatus`. DPP responses no longer carry a misleading `jobId` — DPP protection is not a backup job. - `ProtectResult` record extended with optional `ProtectionStatus` and `ErrorMessage` fields. Bug microsoft#3 — `azurebackup protecteditem protect` for IaaS VM performed a pre-emptive 180 s container-discovery poll that timed out on freshly created VMs. Removed — the Recovery Services backend registers the container as part of accepting the protect request, matching `az backup protection enable-for-vm`. Tests: - Unit: 334/334 pass; +5 new tests for terminal-status surfacing (DPP Succeeded/Failed, RSV Completed/Failed/InProgress). - Live: extended `VaultCreate_CreatesDppVault_Successfully` to assert System-Assigned MSI; added `ProtectedItemProtect_DppVault_DiskProtection_Succeeds_E2E` which creates a disk policy, protects a disk via MCP, and asserts `protectionStatus` on success or `errorMessage` on failure with no `jobId` for DPP. - Live infra: added `${baseName}-disk` (4 GiB Standard_LRS) and the Disk Backup Reader / Disk Snapshot Contributor role assignments needed by the DPP vault MSI. Manual E2E verification (live sub `f0c630e0-…`): - DPP vault create: identity = SystemAssigned, principalId populated. - RSV VM (fresh) protect: Completed in ~95 s (no 180 s pre-poll). - RSV SQL-in-VM (master) protect: Completed in 112 s. - RSV AFS protect: Failed surfaced with real job id + error message. - DPP Disk (fresh) protect: Succeeded → ProtectionConfigured. - DPP ADLS/ESAN: Failed surfaced with service-side error message (HNS unsupported, plugin error) — fix correctly reports the truth. Build: full solution 0 warnings, 0 errors. Spell: clean for all newly-added lines (pre-existing unrelated warnings on main remain untouched).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.