Conversation
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Configuration/RequirementsV1.json
Outdated
Show resolved
Hide resolved
|
The following pipelines have been queued for testing: |
|
The following pipelines have been queued for testing: |
|
The following pipelines have been queued for testing: |
timovv
left a comment
There was a problem hiding this comment.
Looks good to merge for scenario 1
l0lawrence
left a comment
There was a problem hiding this comment.
lgtm excited to try it in v1
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Models/Responses/VerifySetupResponse.cs
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Services/ILanguageSpecificResolver.cs
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Services/ILanguageSpecificResolver.cs
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Services/VerifySetup/DotNetRequirementsCheck.cs
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Services/VerifySetup/JavaRequirementsCheck.cs
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/Verify/VerifySetupTool.cs
Outdated
Show resolved
Hide resolved
|
The following pipelines have been queued for testing: |
richardpark-msft
left a comment
There was a problem hiding this comment.
I left some comments on there - it'd be good to think about the design, and whether it's possibly a leaky abstraction, but this is perfectly fine for v1.
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Services/VerifySetup/PythonRequirementsCheck.cs
Show resolved
Hide resolved
tools/azsdk-cli/Azure.Sdk.Tools.Cli/Services/VerifySetup/GoRequirementsCheck.cs
Show resolved
Hide resolved
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#12476 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: jennypng <63012604+JennyPng@users.noreply.github.com>
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#12476 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: jennypng <63012604+JennyPng@users.noreply.github.com>
Design spec: #12457
Adding an MCP tool that checks the environment for necessary requirements to run MCP tools.
To run in CLI for all languages:
dotnet run --project Azure.Sdk.Tools.Cli -- verify setup --allmanual testing results
These results require updating copilot-instructions in respective repos with the instructions linking to
verify-setup.instructions.mdazure-sdk-for-go
azure-sdk-for-python
example of agent asking for venv path

example of follow-up checking after installing missing reqs

azure-sdk-for-java
azure-sdk-for-javascript
example of asking for multiple langs

azure-sdk-for-dotnet
azure-rest-api-specs
example of checking all languages within azure-sdk-for-java
example of catching outdated version
checking python when there's a venv with python 3.8

focus areas for review
next steps after merging this