* docs: make Fabric tool description evaluation optional
Keep Azure and other non-Fabric evaluation gates, including stricter toolset requirements, and all other validation unchanged.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
* docs: use absolute links for Fabric evaluation policy
Fix the three AGENTS.md policy links rejected by repository link-format guidance. Policy scope is unchanged.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
* docs: minimize Fabric evaluator exception wording
Keep only short Fabric qualifiers in existing guidance, preserving the original requirements and examples. Remove the expanded policy section and links as requested.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---------
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
* Ensure that Run tests (http) is actually testing http
* Update ClientToolTests to support both http and stdio
* Update readiness cancelation
* Update AzureMigrateCommandTests
* Update KeyVaultCommandTests and StorageCommandTests
* Add shared recorded test validation for local-only tools
* Use CancellationTokenSource.CreateLinkedTokenSource
* Ignore any vally-results
* Initial Commit.
* - Add .vscode gitignore
- Update agentic-workflow components
- Add initial template for pr-vally-eval
- Fix namespace in PromptDataStore
- Update launchSettings in VallyEvaluator
* Update AGENTS.md to understand vally testing.
* Add documentation
* Add instructions to CONTRIBUTING.md
* Add documentation for testing.
* Add "interaction" column to clarify if prompt is enough.
* Put extensions into their own folder.
* Update the instructions to be clearer.
* Update foundry descriptions to be more prescriptive in their offerings.
* Fix linting errors.
* Pin vally-cli to current version
* Update YamlSerializer to exclude default values.
* Update to use new test framework
* Add context-required for ones requireing files.
* Add words to spellcheck dictionary.
* Remove angle-bracket replacement logic
* Add description
* Add more spelling.
* Update e2e prompts that are ambiguous
* Fix Microsoft sln
* Update cspell words for vally.
* Add onboarding Copilot agent for new contributors
Create a custom GitHub Copilot agent at .github/copilot/agents/onboarding.md
that guides new contributors through environment setup, codebase understanding,
and their first contribution. Content sourced from CONTRIBUTING.md.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Update .github/copilot/agents/onboarding.md
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Update .github/copilot/agents/onboarding.md
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Update .github/copilot/agents/onboarding.md
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* update agent
* Refine onboarding agent: consolidate to .github/agents/, update for new skill
- Remove .github/copilot/agents/onboarding.md (not discoverable by GitHub)
- Update .github/agents/onboarding.agent.md with consolidated content
- Replace new-command.md references with /skills add-azure-mcp-tools
- Reference AGENTS.md for coding standards instead of duplicating
- Fix Build-Local.ps1 invalid -UsePaths flag
- Fix consolidated-tools.json path
- Update to two-generic pattern ([Option] attributes, SubscriptionCommandUnitTestsBase)
- Remove internal-only links (aka.ms/azmcp/intake, private project board)
- Add eng/scripts/Update-Solutions.ps1 -All guidance
- Use New-TestResources.ps1 for live test deployment
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Consolidate onboarding content from PR #2918
- Update onboarding.agent.md with external MCP server integration, NuGet feed, server modes, and namespace creation guidance
- Add docs/Onboarding.md as human-readable onboarding guide with correct patterns
- Add onboarding callout tip to CONTRIBUTING.md
- Fix typos in CONTRIBUTING.md (Substitue -> Substitute, whem -> when)
- Add cspell words: configfile, remotemcp, slnx, uniqueprefix
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Remove docs/Onboarding.md - handled by PR #2918
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Slim down onboarding.agent.md to agent-focused instructions
Remove duplicate detailed content (prerequisites, full workflows, server modes)
that belongs in docs/Onboarding.md (PR #2918). Keep only:
- Agent behavioral instructions (approach, tone, delegation)
- Key reference file table for the agent to read
- Essential facts and pitfalls to convey
- Quick commands lookup
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Update onboarding callout with agent usage steps
Replace docs/Onboarding.md link with concrete steps on how to
invoke the @onboarding agent in Copilot Chat.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Update AGENTS.md to use two-generic option pattern
Replace old one-generic RegisterOptions/BindOptions/OptionDefinitions guidance
with the new two-generic pattern using [Option] attributes on flat POCOs,
per docs/option-conversion.md.
Key changes:
- Do/Don't sections updated for [Option] attributes and ISubscriptionOption
- Option Handling Pattern section rewritten with two-generic example
- Replace new-command.md references with skill and option-conversion.md refs
- Replace Deploy-TestResources.ps1 with New-TestResources.ps1
- Remove -UsePaths from Build-Local.ps1 commands
- Update test base class guidance (SubscriptionCommandUnitTestsBase)
- Fix logging example to not log full options object
- Update Required Files to remove OptionDefinitions.cs
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Restore comprehensive content to onboarding.agent.md
Since docs/Onboarding.md is being removed, the agent file needs to
contain all the important onboarding info directly: prerequisites,
project structure, workflows, testing, external server integration,
coding standards, quality checklist, and common pitfalls.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Add VS Code debugging setup instructions to onboarding agent
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
---------
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Add git worktree safety guidance against git stash in AGENTS.md
Warn against using git stash in linked worktrees that share a common
.git directory, since stash entries live in the shared refs/stash and
collide across concurrent worktrees, risking clobbered edits.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* Clarify git worktree add arguments in AGENTS guidance
---------
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
* Begin investigation
* Merge Azure MCP Core Live and Unit tests
* Migrate test recordings and add attribute for live-only tests
* Fix checks in scripts
* Two small changes
* Merge in main and few review fixes
* Update to use --filter in Test-Code
* Add missing '+'
* Add base class for command unit tests
* Fix linting and add a few more examples
* Update design, add convenience methods for common workflows
* PR feedback
* PR feedback
* Extend ExecuteResourceQueryAsync to return a model type with more information
* Fix compilation
* Fix linting
* Add CHANGELOG entry
* Update based on feedback
* Update new tool
* Fix linting
* Update projects to target net10
- Update TargetFramework in directory.build.props to net10.0
- Update framework dependency package versions to the 10.0 series
* reader.EndOfStream isn't safe in async and unnecessary when using ReadLineAsync
* Update npgsql to 10.0.1
* Added development plan
* Added and updated docs
* Added scripts
* Removed old YAML file
* Added some QoL changes to the scripts
* Added a way to provide a specific version to compile entries into
* Updated dry run code
* Fixed the compile script to not delete entries with different indentation
* Added support for multi-line entries
* Added trimming for multi-line
* Added multi-line support to New-ChangelogEntry.ps1
* Added a script for syncing the main CHANGELOG with that one for VS Code
* Updated the changelog compilation script to remove empty sections
* Added support for any MCP server in this repo
* Moved a couple of docs
* Updated other docs
* Updated new-command.md
* Updated behavior to ask for a CHANGELOG path instead of defaulting to the Azure MCP Server. Added an optional filename parameter in New-ChangelogEntry.ps1. Updated schema. Updated documentation.
* Updated Compile-Changelog.ps1 to accept files any name and using either the .yml or .yaml extension
* Applied GitHub Copilot feedback for PowerShell scripts
* Applied PR feedback to PS scripts
* Added sample file
* Refactored indentation and trimming logic
* Refactored a couple of scripts to use logic found in `eng/common/scripts/ChangeLog-Operations.ps1`
* Updated test file and added examples for other servers
* Added logic for extracting PR number from the commit that added an entry file
* Fix Docker build instructions. Make ServerName mandatory.
* Update README and TROUBLESHOOTING.
* Move to single line.
* Update servers/Azure.Mcp.Server/TROUBLESHOOTING.md
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Add preamble and link to TROUBLESHOOTING
---------
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Add comprehensive AGENTS.md documentation for project overview, setup, and development workflow
* Fix formatting inconsistencies in AGENTS.md documentation
* Enhance AGENTS.md with detailed guidelines for coding practices, command usage, project structure, and development environment setup
* Refine AGENTS.md guidelines by removing specific references to Newtonsoft.Json, clarifying command submission practices, and updating examples for consistency in command naming and testing.