Add DV MCP server#6
Merged
Merged
Conversation
jongio
approved these changes
Jun 5, 2025
hallipr
pushed a commit
that referenced
this pull request
Aug 20, 2025
* Add instructions for SSE * Remove numbering * Add one click install
hallipr
pushed a commit
that referenced
this pull request
Aug 21, 2025
* onboard code to cloud tools * fix source analysis errors * rename iac command (#6) * rename iac command * revert changes * update * update deployment plan template * fix comments * fix the build error and refine the plan template * refine the template * add quota group (#9) * add quota group * add unit test * update region checker return message * refactor * add unit test * update * revert * [GenerateArchitectureDiagramCommand] Fix and AKS support (#8) * init aks diagram support * fix and test * fix * prompt fix encoding issue * fix by replacing + with - * fix encoding * fix test * resolve conflicts * cspell and format * fix the iac-rules-get tool name in e2etestprompt.md * Refactor: extract prompt to md file (#10) * fix * extract prompt to md file * rename available-region-list * clean code (#11) * clean code * update * add deploy command in md and fix analyze error * Some fix according to e2e test (#12) * fix prompt for test * format * update * fix * update * update * add rules * Refactor deploy"a to project * fix test failure * Add sub command description to goup description * update description based on comment * fix live test (#13) * fix live test * update quota test * update * update cspell (#14) * force to save plan to file (#15) * update plan command description * Add PR guidance document * Add test scenarios * aot safe refactoring and update template md (#16) * aot safe refactoring * update template md * move app topology definition to resource file and update the command extension implementation * update test result - not completed * update test result * reorganize file folders (#17) * reorganize file folders * update test structure * update pipeline template * format * remove comments * fix * update the command names in md file * Add code review report for PR #626 on Deploy and Quota commands * fix the comments in code review report * remove code review doc * fix test failure * Add code review report for PR #626 addressing deploy and quota commands - Summarized findings and compliance with architectural guidelines - Documented command structure, integration issues, and AOT safety - Provided targeted recommendations and next steps for improvements - Included exhaustive merge-readiness checklist and quick quality gates snapshot - Suggested documentation deltas and compliance matrix against final recommendations * Add manual testing plan and update documentation checklist for PR #626 * update action plan and add invalid json test case * feat: Add comprehensive unit tests for quota commands (#19) - Add edge case tests for whitespace-only resource types - Add tests for mixed casing in resource types with proper case preservation - Add tests for very long resource types lists (50+ items) - Enhance test coverage for AvailabilityListCommand and CheckCommand - Ensure proper validation and error handling for edge cases * lint (#20) * Chentony/mermaid response refactor (#21) * Fix diagram prompting * Small change * Reminder * Fix * Remove Mermaid Encode/Decode --------- Co-authored-by: Tony Chen (DevDiv) <chentony@microsoft.com> * fix cspell errors * Mermaid generation: Fix </br> hallucination, fix copilotmd file target, fix extension installation reminder (#22) * Fix diagram prompting * Small change * Reminder * Fix * Remove Mermaid Encode/Decode * Fix </br> hallucination, fix copilotmd file target, fix extension installation reminder --------- Co-authored-by: Tony Chen (DevDiv) <chentony@microsoft.com> * feat: Add remaining work items for Deploy & Quota command areas * feat: Add test execution log section to Manual Testing Plan for Copilot prompts * Xf/updatecomm2 (#23) * add logger * Hard-coded endpoints * update GetQuotaByUrlAsync * update Error handling * update doc * HTTP usage pattern * doc * format * format * add test * update * aot check * update document * update the remaining work * fix parser (#24) * update remaining work * Adds document describing tool organization tasks * Updated organization task priorities * Add 32 character max for resource names (#25) Co-authored-by: Tony Chen (DevDiv) <chentony@microsoft.com> * fix build error and comments * remove image * Update Directory.Packages.props update ARM versions * Update cspell.json fix cspell json * Update sign-and-pack.yml add YamlDotNet.dll to allow list --------- Co-authored-by: xfz11 <81600993+xfz11@users.noreply.github.com> Co-authored-by: wchigit <129354560+wchigit@users.noreply.github.com> Co-authored-by: Vinay Gera <vigera@microsoft.com> Co-authored-by: Jon Gallant <2163001+jongio@users.noreply.github.com> Co-authored-by: Tonychen0227 <tony.chen@outlook.com> Co-authored-by: Tony Chen (DevDiv) <chentony@microsoft.com> Co-authored-by: Wallace Breza <wallace.breza@microsoft.com>
shrja-ms
added a commit
to shrja-ms/mcp
that referenced
this pull request
May 4, 2026
Telemetry report skill (SKILL.md + kql-queries.md): - Add case-sensitivity note for Kusto bag key access (microsoft#6) - Fix week-over-week query range to include current partial week (microsoft#7) - Add InvalidOperationException to MCP Tool Bug classification (microsoft#8) - Add missing KQL queries 11-13 for WorkloadType, DatasourceType, OperationScope custom dimensions (microsoft#9) - Fix live test count to use [Fact] only, avoiding double-count with [LiveTestOnly] on same method (microsoft#5) - Update decision tree to include InvalidOperationException Add-tool skill (SKILL.md): - Fix broken relative link to new-command.md (5 levels up, not 3) (microsoft#1) - Fix live test snippet to use [Fact] + CallToolAsync pattern, not non-existent [RecordedTest]/RunCommandAndAssert (microsoft#2, microsoft#11) - Add missing Update-AzCommandsMetadata.ps1 step after docs update (microsoft#3) - Add AOT/native build verification step (Build-Local.ps1 -BuildNative) (microsoft#4) - Fix vault/vaultType to reference AzureBackupOptionDefinitions, not OptionDefinitions.Common (microsoft#10) - Expand git add scope to include README.md and eng/vscode/README.md (microsoft#12) - Add commands metadata regeneration and AOT build to PR checklist
shrja-ms
added a commit
that referenced
this pull request
May 8, 2026
…workflow (#2570) * azurebackup: Add weekly telemetry report skill Add a Copilot skill under tools/Azure.Mcp.Tools.AzureBackup/skills/ that automates generation of weekly telemetry reports by: - Running KQL queries against the ddazureclients Kusto cluster - Classifying errors into Customer (4xx) / Azure Service / MCP Tool Bug - Correlating with merged PRs, releases, and known bug IDs - Producing an Outlook-compatible HTML report Includes: - SKILL.md with full procedure and error classification decision tree - references/kql-queries.md with all 10 production KQL queries - assets/report-template.html with Outlook-compatible HTML template * azurebackup: Add 'add-tool' skill for new command workflow Add a Copilot skill that guides the full lifecycle of adding a new Azure Backup MCP command: - Phase 1: Implementation (options, service, command, registration) - Phase 2: Input validation checklist - Phase 3: Unit tests with required test patterns - Phase 4: Live tests with Record/Playback workflow - Phase 5: CI validation (build, format, spell check) - Phase 6: ToolDescriptionEvaluator scoring - Phase 7: Documentation updates - Phase 8: PR submission checklist * azurebackup: Fix telemetry report skill - BUG-1 status and test count methodology - BUG-1 (backup_status ArgumentNullException) is fixed in #2518 (code included in merged PR, though PR title only lists BUG-3,4,5,6,8) - Fix unit test count methodology: count [Fact] + [InlineData] for runnable test cases, not [Fact] + [Theory] for method count - Add Release column to bug tracker table - Add note about verifying PR diff vs title for bug coverage * azurebackup: Address all 12 PR review comments on skills Telemetry report skill (SKILL.md + kql-queries.md): - Add case-sensitivity note for Kusto bag key access (#6) - Fix week-over-week query range to include current partial week (#7) - Add InvalidOperationException to MCP Tool Bug classification (#8) - Add missing KQL queries 11-13 for WorkloadType, DatasourceType, OperationScope custom dimensions (#9) - Fix live test count to use [Fact] only, avoiding double-count with [LiveTestOnly] on same method (#5) - Update decision tree to include InvalidOperationException Add-tool skill (SKILL.md): - Fix broken relative link to new-command.md (5 levels up, not 3) (#1) - Fix live test snippet to use [Fact] + CallToolAsync pattern, not non-existent [RecordedTest]/RunCommandAndAssert (#2, #11) - Add missing Update-AzCommandsMetadata.ps1 step after docs update (#3) - Add AOT/native build verification step (Build-Local.ps1 -BuildNative) (#4) - Fix vault/vaultType to reference AzureBackupOptionDefinitions, not OptionDefinitions.Common (#10) - Expand git add scope to include README.md and eng/vscode/README.md (#12) - Add commands metadata regeneration and AOT build to PR checklist * azurebackup: Convert relative links to absolute paths for CI link checker The link verification step requires absolute paths from repo root, not relative paths. Convert all ./references/ and ../../../ links to /tools/Azure.Mcp.Tools.AzureBackup/skills/... format. * azurebackup: Use full GitHub URLs for CI link checker Link checker requires https://github.com/microsoft/mcp/blob/main/... format, not repo-root-relative paths. * azurebackup: Add InvalidOperationException to Query 1, replace Power BI URLs with placeholder - Query 1 (primary per-tool health) was missing InvalidOperationException in the McpToolBug case, causing inconsistency with Queries 2 and 4 - Replace hardcoded Power BI dashboard URLs in report template with {{POWERBI_DASHBOARD_URL}} placeholder (public repo, internal GUIDs)
3 tasks
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
2 participants
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.