Fix dashboard app names with dash suffix by Copilot · Pull Request #19932 · microsoft/aspire

Copilot · 2026-09-04T14:15:55Z

Description

Source-file AppHosts can derive their dashboard application name from a directory such as aspire-apphost. The existing suffix normalization only recognized an optional . before AppHost, causing the dashboard to display aspire-.

This change recognizes both . and - separators before the AppHost suffix. The same handling is applied to DCP resource-name normalization, with regression coverage for both naming forms.

User-facing usage

Dashboard application names now normalize as follows:

TestShop.AppHost -> TestShop
aspire-apphost   -> aspire

Validation: regression tests were added to DashboardServiceTests and DcpExecutorTests. The test project could not be restored in the agent environment because the configured internal Arcade package feed returned HTTP 403.

Fixes # (issue)

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
  - If yes, did you have an API Review for it?
    - Yes
    - No
  - Did you add <remarks /> and <code /> elements on your triple slash comments?
    - Yes
    - No
- No
Does the change make any security assumptions or guarantees?
- Yes
  - If yes, have you done a threat model and had a security review?
    - Yes
    - No
- No

Created from a Microsoft Teams conversation.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

James Newton-King (JamesNK) · 2026-09-04T14:17:42Z

Copilot Share the ApplicationNameRegex. avoid duplication

Co-authored-by: JamesNK <303201+JamesNK@users.noreply.github.com>

Copilot · 2026-09-04T14:24:49Z

Copilot Share the ApplicationNameRegex. avoid duplication

Shared the regex through ApplicationNameHelper in 953fa88; both dashboard and DCP normalization now use it.

github-actions · 2026-09-04T14:47:34Z

🚀 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/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 19932

Or

Run remotely in PowerShell:

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

James Newton-King (JamesNK) · 2026-09-04T14:49:22Z

Copilot

What happens if the name is just .AppHost, -AppHost or AppHost?

What about tests for if there is no seperator, e.g. "TestShopAppHost".

Co-authored-by: JamesNK <303201+JamesNK@users.noreply.github.com>

Copilot · 2026-09-04T14:54:20Z

Copilot

What happens if the name is just .AppHost, -AppHost or AppHost?
...

Addressed in 5570cf8. .AppHost, -AppHost, and AppHost now remain unchanged; TestShopAppHost normalizes to TestShop. Added coverage for dashboard and DCP normalization.

github-actions · 2026-09-04T15:19:11Z

Tests selector

49 / 99 PR test projects · 4 PR jobs · 2 advisory-only targets, from 5 changed files.

Selected PR test projects (49 / 99)

Aspire.Hosting.Analyzers.Tests, Aspire.Hosting.Azure.Kubernetes.Tests, Aspire.Hosting.Azure.Kusto.Tests, Aspire.Hosting.Azure.Tests, Aspire.Hosting.Blazor.Tests, Aspire.Hosting.Browsers.Tests, Aspire.Hosting.CodeGeneration.Go.Tests, Aspire.Hosting.CodeGeneration.Java.Tests, Aspire.Hosting.CodeGeneration.Python.Tests, Aspire.Hosting.CodeGeneration.Rust.Tests, Aspire.Hosting.CodeGeneration.TypeScript.Tests, Aspire.Hosting.Containers.Tests, Aspire.Hosting.DevTunnels.Tests, Aspire.Hosting.Docker.Tests, Aspire.Hosting.Dotnet.Tests, Aspire.Hosting.DotnetTool.Tests, Aspire.Hosting.EntityFrameworkCore.Tests, Aspire.Hosting.Foundry.Tests, Aspire.Hosting.Garnet.Tests, Aspire.Hosting.GitHub.Models.Tests, Aspire.Hosting.Go.Tests, Aspire.Hosting.Java.Tests, Aspire.Hosting.JavaScript.Tests, Aspire.Hosting.Kafka.Tests, Aspire.Hosting.Keycloak.Tests, Aspire.Hosting.Kubernetes.Tests, Aspire.Hosting.Maui.Tests, Aspire.Hosting.Milvus.Tests, Aspire.Hosting.MongoDB.Tests, Aspire.Hosting.MySql.Tests, Aspire.Hosting.Nats.Tests, Aspire.Hosting.OpenAI.Tests, Aspire.Hosting.Oracle.Tests, Aspire.Hosting.Orleans.Tests, Aspire.Hosting.PostgreSQL.Tests, Aspire.Hosting.Python.Tests, Aspire.Hosting.Qdrant.Tests, Aspire.Hosting.RabbitMQ.Tests, Aspire.Hosting.Radius.Tests, Aspire.Hosting.Redis.Tests, Aspire.Hosting.RemoteHost.Tests, Aspire.Hosting.Rust.Tests, Aspire.Hosting.Seq.Tests, Aspire.Hosting.SqlServer.Tests, Aspire.Hosting.Testing.Tests, Aspire.Hosting.Tests, Aspire.Hosting.Valkey.Tests, Aspire.Hosting.Yarp.Tests, Aspire.Playground.Tests

Selected PR jobs (4)

cli-starter-validation, extension-e2e, polyglot, typescript-api-compat

Advisory workflow impact (2)

Aspire.EndToEnd.Tests (outerloop-only)
deployment-e2e (schedule/dispatch-only)

How these were chosen — grouped by what changed

⚠️ 45 of the 50 selected test projects come from a single change — src/Aspire.Hosting/Dashboard/DashboardService.cs.

🔧 src/Aspire.Hosting/Dashboard/DashboardService.cs (changed source)
→ 45 via the project graph

show 45

Aspire.Hosting.Analyzers.Tests (2 hops), Aspire.Hosting.Azure.Kubernetes.Tests (2 hops), Aspire.Hosting.Azure.Kusto.Tests (2 hops), Aspire.Hosting.Azure.Tests, Aspire.Hosting.Browsers.Tests (2 hops), Aspire.Hosting.CodeGeneration.Go.Tests, Aspire.Hosting.CodeGeneration.Java.Tests, Aspire.Hosting.CodeGeneration.Python.Tests, Aspire.Hosting.CodeGeneration.Rust.Tests, Aspire.Hosting.CodeGeneration.TypeScript.Tests, Aspire.Hosting.Containers.Tests (2 hops), Aspire.Hosting.DevTunnels.Tests (2 hops), Aspire.Hosting.Docker.Tests (2 hops), Aspire.Hosting.DotnetTool.Tests (2 hops), Aspire.Hosting.EntityFrameworkCore.Tests (2 hops), Aspire.Hosting.Foundry.Tests (2 hops), Aspire.Hosting.Garnet.Tests (2 hops), Aspire.Hosting.GitHub.Models.Tests (2 hops), Aspire.Hosting.Go.Tests (2 hops), Aspire.Hosting.Java.Tests (2 hops), Aspire.Hosting.JavaScript.Tests (2 hops), Aspire.Hosting.Kafka.Tests (2 hops), Aspire.Hosting.Keycloak.Tests (2 hops), Aspire.Hosting.Kubernetes.Tests (2 hops), Aspire.Hosting.Maui.Tests, Aspire.Hosting.Milvus.Tests (2 hops), Aspire.Hosting.MongoDB.Tests (2 hops), Aspire.Hosting.MySql.Tests (2 hops), Aspire.Hosting.Nats.Tests (2 hops), Aspire.Hosting.OpenAI.Tests (2 hops), Aspire.Hosting.Oracle.Tests (2 hops), Aspire.Hosting.Orleans.Tests (2 hops), Aspire.Hosting.PostgreSQL.Tests (2 hops), Aspire.Hosting.Python.Tests (2 hops), Aspire.Hosting.Qdrant.Tests (2 hops), Aspire.Hosting.RabbitMQ.Tests (2 hops), Aspire.Hosting.Redis.Tests (2 hops), Aspire.Hosting.RemoteHost.Tests, Aspire.Hosting.Rust.Tests (2 hops), Aspire.Hosting.Seq.Tests (2 hops), Aspire.Hosting.SqlServer.Tests (2 hops), Aspire.Hosting.Testing.Tests (2 hops), Aspire.Hosting.Valkey.Tests (2 hops), Aspire.Hosting.Yarp.Tests (2 hops), Aspire.Playground.Tests

🧪 tests/Aspire.Hosting.Tests/Dashboard/DashboardServiceTests.cs (changed test)
→ 1 directly: Aspire.Hosting.Tests
→ 3 via the project graph: Aspire.Hosting.Blazor.Tests, Aspire.Hosting.Dotnet.Tests, Aspire.Hosting.Radius.Tests

📦 affected project Aspire.Hosting
→ 1 test: Aspire.EndToEnd.Tests

🧪 tests/Aspire.Hosting.Tests/Dcp/DcpExecutorTests.cs (changed test)
→ 1 directly: Aspire.Hosting.Tests

Job reasons

Job	Triggered by
`cli-starter-validation`	affected project `Aspire.Hosting.JavaScript`
`deployment-e2e`	affected project `Aspire.Hosting.Azure`
`extension-e2e`	• `src/Aspire.Hosting/Dashboard/DashboardService.cs`, `src/Aspire.Hosting/Dcp/DcpExecutor.cs`, `src/Aspire.Hosting/Utils/ApplicationNameHelper.cs` • affected project `Aspire.Hosting`
`polyglot`	affected project `Aspire.Hosting.JavaScript`
`typescript-api-compat`	affected project `Aspire.Hosting`

Selection computed for commit 5570cf8.

Fix dashboard app names with dash suffix

29f9122

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot AI balanced review requested due to automatic review settings September 4, 2026 14:15

Copilot AI requested review from James Newton-King (JamesNK), David Negstad (danegsta), David Fowler (davidfowl), Karol Zadora-Przylecki (karolz-ms) and Mitch Denny (mitchdenny) as code owners September 4, 2026 14:15

Copilot AI removed the request for review from Copilot September 4, 2026 14:15

James Newton-King (JamesNK) added the area-app-model Issues pertaining to the APIs in Aspire.Hosting, e.g. DistributedApplication label Sep 4, 2026

Copilot started work on behalf of James Newton-King (JamesNK) September 4, 2026 14:18 View session

Share application name regex

953fa88

Co-authored-by: JamesNK <303201+JamesNK@users.noreply.github.com>

Copilot AI requested a balanced review from Copilot and removed request for Copilot September 4, 2026 14:24

Copilot finished work on behalf of James Newton-King (JamesNK) September 4, 2026 14:25

This comment has been minimized.

Sign in to view

Copilot started work on behalf of James Newton-King (JamesNK) September 4, 2026 14:50 View session

Cover AppHost suffix edge cases

5570cf8

Co-authored-by: JamesNK <303201+JamesNK@users.noreply.github.com>

Copilot AI requested a balanced review from Copilot and removed request for Copilot September 4, 2026 14:53

Copilot finished work on behalf of James Newton-King (JamesNK) September 4, 2026 14:54

github-actions Bot mentioned this pull request Sep 4, 2026

[repo-pulse] 📊 Repo Pulse — Daily Activity Dashboard #16404

Open

Navigation Menu

Uh oh!

Fix dashboard app names with dash suffix - #19932

Fix dashboard app names with dash suffix#19932
Copilot wants to merge 3 commits into
mainfrom
copilot/fix-dashboard-app-name

Copilot AI commented Sep 4, 2026

Uh oh!

James Newton-King (JamesNK) commented Sep 4, 2026

Uh oh!

Copilot AI commented Sep 4, 2026

Uh oh!

github-actions Bot commented Sep 4, 2026

Uh oh!

James Newton-King (JamesNK) commented Sep 4, 2026

Uh oh!

This comment has been minimized.

Copilot AI commented Sep 4, 2026

Uh oh!

github-actions Bot commented Sep 4, 2026

Job reasons

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

Copilot AI commented Sep 4, 2026

Description

User-facing usage

Checklist

Uh oh!

James Newton-King (JamesNK) commented Sep 4, 2026

Uh oh!

Copilot AI commented Sep 4, 2026

Uh oh!

github-actions Bot commented Sep 4, 2026

Uh oh!

James Newton-King (JamesNK) commented Sep 4, 2026

Uh oh!

This comment has been minimized.

Copilot AI commented Sep 4, 2026

Uh oh!

github-actions Bot commented Sep 4, 2026

Tests selector

Selected PR test projects (49 / 99)

Selected PR jobs (4)

Advisory workflow impact (2)

Job reasons

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants