[CI] Fix main build

[Copilot]

Description

PR #13963 updated BaseCommand constructor to require AspireCliTelemetry telemetry parameter. ResourcesCommand and LogsCommand were not updated, breaking main branch builds with CS7036 errors.

Changes:

• Added AspireCliTelemetry telemetry parameter to ResourcesCommand and LogsCommand constructors
• Added using Aspire.Cli.Telemetry; directive to both files
• Passed telemetry parameter to base constructor

public ResourcesCommand(
    IInteractionService interactionService,
    IAuxiliaryBackchannelMonitor backchannelMonitor,
    IFeatures features,
    ICliUpdateNotifier updateNotifier,
    CliExecutionContext executionContext,
    AspireCliTelemetry telemetry,  // Added
    ILogger<ResourcesCommand> logger)
    : base("resources", ResourcesCommandStrings.Description, features, updateNotifier, executionContext, interactionService, telemetry)  // Pass through

Checklist

• Is this feature complete?
  • Yes. Ready to ship.
  • No. Follow-up changes expected.
• Are you including unit tests for the changes and scenario tests if relevant?
  • Yes
  • No
• Did you add public API?
  • Yes
  • No
• Does the change make any security assumptions or guarantees?
  • Yes
  • No
• Does the change require an update in our Aspire docs?
  • Yes
  • No

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• centralus-2.in.applicationinsights.azure.com
  • Triggering command: /home/REDACTED/work/aspire/aspire/artifacts/bin/Aspire.Cli.Tests/Debug/net10.0/Aspire.Cli.Tests /home/REDACTED/work/aspire/aspire/artifacts/bin/Aspire.Cli.Tests/Debug/net10.0/Aspire.Cli.Tests --internal-msbuild-node /home/REDACTED/.local/share/136a9fd2fcba4471a93b9995a687af5e/.p --filter-not-trait category=failing --filter-not-trait quarantined=true --filter-not-trait outerloop=true (dns block)
  • Triggering command: /usr/share/dotnet/dotnet /usr/share/dotnet/dotnet exec --runtimeconfig /home/REDACTED/work/aspire/aspire/artifacts/bin/Aspire.Cli.Tests/Debug/net10.0/Aspire.Cli.Tests.runtimeconfig.json --depsfile /home/REDACTED/work/aspire/aspire/artifacts/bin/Aspire.Cli.Tests/Debug/net10.0/Aspire.Cli.Tests.deps.json /home/REDACTED/work/aspire/aspire/artifacts/bin/Aspire.Cli.Tests/Debug/net10.0/Microsoft.DotNet.RemoteExecutor.dll Aspire.Cli.Tests, Version=42.42.42.42, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51 Aspire.Cli.Tests.CliSmokeTests+<>c <LocaleOverrideReturnsExitCode>b__1_0 /tmp/ato44hlk.3t3 invalid-locale False ASPIRE_LOCALE_OVERRIDE (dns block)
  • Triggering command: /usr/share/dotnet/dotnet /usr/share/dotnet/dotnet exec --runtimeconfig /home/REDACTED/work/aspire/aspire/artifacts/bin/Aspire.Cli.Tests/Debug/net10.0/Aspire.Cli.Tests.runtimeconfig.json --depsfile /home/REDACTED/work/aspire/aspire/artifacts/bin/Aspire.Cli.Tests/Debug/net10.0/Aspire.Cli.Tests.deps.json /home/REDACTED/work/aspire/aspire/artifacts/bin/Aspire.Cli.Tests/Debug/net10.0/Microsoft.DotNet.RemoteExecutor.dll Aspire.Cli.Tests, Version=42.42.42.42, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51 Aspire.Cli.Tests.CliSmokeTests+<>c <LocaleOverrideReturnsExitCode>b__1_0 /tmp/kmc1wtxb.vc0 (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

Open an issue for this failure in https://github.com/dotnet/aspire/actions/runs/21406343091/job/61631188136 which is a rolling build for main:

/home/runner/work/aspire/aspire/src/Aspire.Cli/Commands/ResourcesCommand.cs(78,11): error CS7036: There is no argument given that corresponds to the required parameter 'telemetry' of 'BaseCommand.BaseCommand(string, string, IFeatures, ICliUpdateNotifier, CliExecutionContext, IInteractionService, AspireCliTelemetry)' [/home/runner/work/aspire/aspire/src/Aspire.Cli/Aspire.Cli.Tool.csproj]
/home/runner/work/aspire/aspire/src/Aspire.Cli/Commands/LogsCommand.cs(102,11): error CS7036: There is no argument given that corresponds to the required parameter 'telemetry' of 'BaseCommand.BaseCommand(string, string, IFeatures, ICliUpdateNotifier, CliExecutionContext, IInteractionService, AspireCliTelemetry)' [/home/runner/work/aspire/aspire/src/Aspire.Cli/Aspire.Cli.Tool.csproj]
/home/runner/work/aspire/aspire/src/Aspire.Cli/Commands/ResourcesCommand.cs(78,11): error CS7036: There is no argument given that corresponds to the required parameter 'telemetry' of 'BaseCommand.BaseCommand(string, string, IFeatures, ICliUpdateNotifier, CliExecutionContext, IInteractionService, AspireCliTelemetry)' [/home/runner/work/aspire/aspire/src/Aspire.Cli/Aspire.Cli.csproj]
/home/runner/work/aspire/aspire/src/Aspire.Cli/Commands/LogsCommand.cs(102,11): error CS7036: There is no argument given that corresponds to the required parameter 'telemetry' of 'BaseCommand.BaseCommand(string, string, IFeatures, ICliUpdateNotifier, CliExecutionContext, IInteractionService, AspireCliTelemetry)' [/home/runner/work/aspire/aspire/src/Aspire.Cli/Aspire.Cli.csproj]
/home/runner/work/aspire/aspire/src/Aspire.Cli/Commands/ResourcesCommand.cs(78,11): error CS7036: There is no argument given that corresponds to the required parameter 'telemetry' of 'BaseCommand.BaseCommand(string, string, IFeatures, ICliUpdateNotifier, CliExecutionContext, IInteractionService, AspireCliTelemetry)' [/home/runner/work/aspire/aspire/src/Aspire.Cli/Aspire.Cli.csproj]
/home/runner/work/aspire/aspire/src/Aspire.Cli/Commands/LogsCommand.cs(102,11): error CS7036: There is no argument given that corresponds to the required parameter 'telemetry' of 'BaseCommand.BaseCommand(string, string, IFeatures, ICliUpdateNotifier, CliExecutionContext, IInteractionService, AspireCliTelemetry)' [/home/runner/work/aspire/aspire/src/Aspire.Cli/Aspire.Cli.csproj]

Mention the PR the commit belonged to.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[github-actions]

🚀 Dogfood this PR with:

⚠️ WARNING: Do not do this without first carefully reviewing the code of this PR to satisfy yourself it is safe.

curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 14145

Or

• Run remotely in PowerShell:

iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 14145"

[Copilot]

Pull request overview

Fixes a build break in Aspire CLI commands after BaseCommand started requiring an AspireCliTelemetry constructor parameter (introduced in PR #13963).

Changes:

• Add AspireCliTelemetry parameter to ResourcesCommand and pass it through to BaseCommand.
• Add AspireCliTelemetry parameter to LogsCommand and pass it through to BaseCommand.

Reviewed changes

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

File	Description
src/Aspire.Cli/Commands/ResourcesCommand.cs	Injects and forwards AspireCliTelemetry to the BaseCommand constructor to fix missing-parameter build errors.
src/Aspire.Cli/Commands/LogsCommand.cs	Injects and forwards AspireCliTelemetry to the BaseCommand constructor to fix missing-parameter build errors.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.