Use single ActivitySource across CLI components

[Copilot]

Thanks for assigning this issue to me. I'm starting to work on it and will keep this PR's description up to date as I form a plan and make progress.

Original issue description:

We should consider using a single ActivitySource across all commands/dependencies in the Aspire CLI pipeline.

I know you are just following the existing pattern, but it is odd for each class to have its own ActivitySource. Typically there is a single ActivitySource for an entire assembly/app/etc.

https://learn.microsoft.com/en-us/dotnet/core/diagnostics/distributed-tracing-instrumentation-walkthroughs#best-practices-1

Create the ActivitySource once, store it in a static variable and use that instance as long as needed. Each library or library subcomponent can (and often should) create its own source. Consider creating a new source rather than reusing an existing one if you anticipate app developers would appreciate being able to enable and disable the Activity telemetry in the sources independently.

I don't see devs needing to enable and disable separate commands activities.

Originally posted by @eerhardt in #9792 (comment)

Fixes #9843.

---

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

[davidfowl]

Try again @copilot

[Copilot]

Try again @copilot

Done! I've consolidated all 11 individual ActivitySource instances across the CLI components into a single shared AspireCliActivitySource.Instance. The OpenTelemetry configuration has been updated to use the single "Aspire.Cli" source. Build and tests pass successfully.

Fixed in commit 019a27b.

[Copilot]

Try again @copilot

Done! I've successfully consolidated all 11 individual ActivitySource instances across the CLI components into a single shared telemetry service. The implementation uses dependency injection and all tests pass (except for one unrelated flaky test). Fixed in commit 742c530.

[davidfowl]

The activity itself should have the command name

[Copilot]

The activity itself should have the command name

Updated all CLI command activities to include the command name for better tracing context. Each command (run, add, new, publish, deploy) now passes its name when starting activities. Fixed in commit coming next.