DI Refactor: Azure.Mcp.Tools.AppService

[Copilot]

• Refactor AppService commands to use constructor DI
• Update unit tests to pass IAppServiceService directly to constructors
• Add AppServiceSetup to CommandFactoryHelpers.cs (Step 4)
• Add changelog entry (Step 5)
• Fix imports ordering in CommandFactoryHelpers.cs (AppLens before AppService)

Original prompt

---

This section details on the original issue you should resolve

<issue_title>DI Refactor: Azure.Mcp.Tools.AppService</issue_title>
<issue_description>## Summary

Refactor Azure.Mcp.Tools.AppService to use constructor dependency injection instead of resolving services via context.GetService<T>() in ExecuteAsync.

This is a sub-issue of #158.

Instructions

Follow the algorithm described in tools/di-refactor-plan.md with {ToolArea} = AppService.

An example PR is: #1815</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes DI Refactor: Azure.Mcp.Tools.AppService #1837

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[Copilot]

Pull request overview

Refactors the AppService tool’s commands to use constructor dependency injection for IAppServiceService (instead of resolving via CommandContext.GetService<T>()), improving testability and aligning with the broader DI migration tracked in #158.

Changes:

• Updated 4 AppService commands to accept IAppServiceService via constructor injection and use it during execution.
• Simplified AppService unit tests to pass IAppServiceService directly to command constructors (removing ServiceProvider wiring for GetService<T>()).
• Updated core unit test command factory helpers to include AppServiceSetup, and added a changelog entry documenting the change.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
tools/Azure.Mcp.Tools.AppService/src/Commands/Webapp/WebappGetCommand.cs	Injects IAppServiceService via constructor and uses it in ExecuteAsync.
tools/Azure.Mcp.Tools.AppService/src/Commands/Database/DatabaseAddCommand.cs	Injects IAppServiceService via constructor and uses it in ExecuteAsync.
tools/Azure.Mcp.Tools.AppService/src/Commands/Webapp/Settings/AppSettingsGetCommand.cs	Injects IAppServiceService via constructor and uses it in ExecuteAsync.
tools/Azure.Mcp.Tools.AppService/src/Commands/Webapp/Settings/AppSettingsUpdateCommand.cs	Injects IAppServiceService via constructor and uses it in ExecuteAsync.
tools/Azure.Mcp.Tools.AppService/tests/Azure.Mcp.Tools.AppService.UnitTests/Commands/Webapp/WebappGetCommandTests.cs	Updates tests to construct the command with IAppServiceService directly.
tools/Azure.Mcp.Tools.AppService/tests/Azure.Mcp.Tools.AppService.UnitTests/Commands/Database/DatabaseAddCommandTests.cs	Updates tests to use injected IAppServiceService and a minimal CommandContext.
tools/Azure.Mcp.Tools.AppService/tests/Azure.Mcp.Tools.AppService.UnitTests/Commands/Webapp/Settings/AppSettingsGetCommandTests.cs	Updates tests to construct the command with IAppServiceService directly.
tools/Azure.Mcp.Tools.AppService/tests/Azure.Mcp.Tools.AppService.UnitTests/Commands/Webapp/Settings/AppSettingsUpdateCommandTests.cs	Updates tests to construct the command with IAppServiceService directly.
core/Azure.Mcp.Core/tests/Azure.Mcp.Core.UnitTests/Areas/Server/CommandFactoryHelpers.cs	Adds AppServiceSetup so factory-based tests include AppService commands.
servers/Azure.Mcp.Server/changelog-entries/1772482309825.yaml	Documents the testability/DI improvement in the server changelog.

[conniey]

@copilot There is an build error. Please fix this by running dotnet format on Microsoft.Mcp.slnx

/mnt/vss/_work/1/s/core/Azure.Mcp.Core/tests/Azure.Mcp.Core.UnitTests/Areas/Server/CommandFactoryHelpers.cs(1,1): error IMPORTS: Fix imports ordering. [/mnt/vss/_work/1/s/core/Azure.Mcp.Core/tests/Azure.Mcp.Core.UnitTests/Azure.Mcp.Core.UnitTests.csproj]
❌ dotnet format detected formatting issues.
Please run 'dotnet format "/mnt/vss/_work/1/s/Microsoft.Mcp.slnx"' to fix the issues and then try committing again.

[Copilot]

@copilot There is an build error. Please fix this by running dotnet format on Microsoft.Mcp.slnx

/mnt/vss/_work/1/s/core/Azure.Mcp.Core/tests/Azure.Mcp.Core.UnitTests/Areas/Server/CommandFactory...

Fixed in e6ae5d6. Moved using Azure.Mcp.Tools.AppLens; before using Azure.Mcp.Tools.AppService; to restore alphabetical ordering in the imports.