DI Refactor: Azure.Mcp.Tools.Marketplace

[Copilot]

Part of the broader DI refactor (#158): ProductGetCommand and ProductListCommand were resolving IMarketplaceService lazily via context.GetService<T>() inside ExecuteAsync instead of using constructor injection.

Changes

• ProductGetCommand / ProductListCommand: Added IMarketplaceService as a constructor parameter; removed runtime context.GetService<IMarketplaceService>() calls.

// Before
public sealed class ProductGetCommand(ILogger<ProductGetCommand> logger) : SubscriptionCommand<ProductGetOptions>
{
    public override async Task<CommandResponse> ExecuteAsync(...)
    {
        var marketplaceService = context.GetService<IMarketplaceService>(); // ❌
        ...
    }
}

// After
public sealed class ProductGetCommand(ILogger<ProductGetCommand> logger, IMarketplaceService marketplaceService) : SubscriptionCommand<ProductGetOptions>
{
    private readonly IMarketplaceService _marketplaceService = marketplaceService; // ✅
    ...
}

• Unit tests: Updated constructors to inject the mock service directly into the command rather than registering it in the service collection.

Original prompt

---

This section details on the original issue you should resolve

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

Refactor Azure.Mcp.Tools.Marketplace 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} = Marketplace.

An example PR is: #1815
</issue_description>

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

• Fixes DI Refactor: Azure.Mcp.Tools.Marketplace #1861

---

📍 Connect Copilot coding agent with Jira, Azure Boards or Linear to delegate work to Copilot in one click without leaving your project management tool.

[Copilot]

Pull request overview

Refactors the Marketplace tool commands to use constructor dependency injection for IMarketplaceService rather than resolving it from CommandContext during execution, improving explicitness and testability as part of the broader DI refactor effort.

Changes:

• Inject IMarketplaceService into ProductGetCommand and ProductListCommand via primary constructors.
• Remove context.GetService<IMarketplaceService>() usage from ExecuteAsync.
• Update unit tests to pass the mocked IMarketplaceService directly into command constructors.

Reviewed changes

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

File	Description
tools/Azure.Mcp.Tools.Marketplace/src/Commands/Product/ProductGetCommand.cs	Switches product “get” command to constructor-injected IMarketplaceService and uses it in ExecuteAsync.
tools/Azure.Mcp.Tools.Marketplace/src/Commands/Product/ProductListCommand.cs	Switches product “list” command to constructor-injected IMarketplaceService and uses it in ExecuteAsync.
tools/Azure.Mcp.Tools.Marketplace/tests/Azure.Mcp.Tools.Marketplace.UnitTests/Product/ProductGetCommandTests.cs	Updates tests to construct the command with a mocked IMarketplaceService instead of registering it in the service collection.
tools/Azure.Mcp.Tools.Marketplace/tests/Azure.Mcp.Tools.Marketplace.UnitTests/Product/ProductListCommandTests.cs	Updates tests to construct the command with a mocked IMarketplaceService instead of registering it in the service collection.

---

You can also share your feedback on Copilot code review. Take the survey.