Skip to content

[azsdk-cli] Add compile-time analyzer to enforce Tool method return types #11565

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
benbp merged 11 commits into from
Aug 7, 2025
Merged

[azsdk-cli] Add compile-time analyzer to enforce Tool method return types #11565

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
c49b031
Initial plan
Copilot Aug 6, 2025
9a6637b
Implement analyzer to enforce Tool method return types
Copilot Aug 6, 2025
1d0d945
Add documentation for MCP003 analyzer
Copilot Aug 6, 2025
d7673a8
Apply suggestion from @benbp
benbp Aug 6, 2025
740d80e
Apply suggestion from @benbp
benbp Aug 6, 2025
c480f2e
Change analyzer severity to Warning and remove ValueTask support
Copilot Aug 7, 2025
2228641
Simplify type checking to only allow primitive types and string
Copilot Aug 7, 2025
0cc11a9
Fix code to work as intended
benbp Aug 7, 2025
50dcc81
Add readme for mcp analyzers
benbp Aug 7, 2025
8b54beb
Enforce if statement syntax
benbp Aug 7, 2025
d2b51bc
Enforce if statement syntax. Fix tools namespacing
benbp Aug 7, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Enforce if statement syntax
  • Loading branch information
@benbp
benbp committed Aug 7, 2025
commit 8b54beb99aa4ff65e1c0fab386d5ac214f30c747
3 changes: 2 additions & 1 deletion tools/azsdk-cli/.editorconfig
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,2 +1,3 @@
[*.cs]
# disallow single if statements with no curly braces
dotnet_diagnostic.SA1503.severity = error
dotnet_diagnostic.IDE0011.severity = error
1 change: 1 addition & 0 deletions tools/azsdk-cli/Azure.Sdk.Tools.Cli/Azure.Sdk.Tools.Cli.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@
<IncludeNativeLibrariesForSelfExtract>true</IncludeNativeLibrariesForSelfExtract>
<ToolCommandName>azsdk</ToolCommandName>
<NoWarn>ASP0000;CS8603;CS8618;CS8625;CS8604</NoWarn>
<EnforceCodeStyleInBuild>true</EnforceCodeStyleInBuild>
<AssemblyName>azsdk</AssemblyName>
</PropertyGroup>

Expand Down
1 change: 0 additions & 1 deletion tools/azsdk-cli/Azure.Sdk.Tools.Cli/Tools/HelloWorldTool/HelloWorldTool.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,6 @@ public DefaultCommandResponse EchoFail(string message)
}
}


[McpServerTool(Name = "hello-world"), Description("Echoes the message back to the client")]
public DefaultCommandResponse EchoSuccess(string message)
{
Expand Down