Add CTRL-C guidance message to aspire run command output #10203

Copilot · 2025-07-02T12:48:10Z

Summary

This PR adds a user guidance message to the aspire run command output that instructs users how to stop the AppHost. The message "Press Ctrl+C to stop the app host and exit." is displayed in the console grid after the dashboard links and before the logs information.

Changes

Modified src/Aspire.Cli/Commands/RunCommand.cs to add two new rows to the Spectre.Console grid:
1. A blank row for spacing
2. The CTRL-C guidance message in the rightmost column
Reused the existing localized resource string RunCommandStrings.PressCtrlCToStopAppHost which contains the properly formatted message with Spectre.Console markup

Output Format

The new output will display:

Dashboard:  [dashboard-url]
            [codespaces-url] (if applicable)

            Press Ctrl+C to stop the app host and exit.
Logs:       [log-file-path]

Implementation Details

The changes are minimal and surgical - only 2 lines added
No existing functionality is modified or removed
Uses existing localization infrastructure
Consistent with Spectre.Console formatting patterns used elsewhere in the codebase
All existing CLI tests continue to pass (99/99)

The positioning ensures the guidance message appears after all dashboard links but before the logs information, providing clear user instruction on how to terminate the running AppHost.

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

Co-authored-by: mitchdenny <513398+mitchdenny@users.noreply.github.com>

…1b3c

mitchdenny · 2025-07-06T05:40:26Z

@maddymontaquila

Copilot

Pull Request Overview

This PR updates the aspire run command to surface a “Press Ctrl+C to stop” message in the console output and highlights the updated markup in all localized resources.

Changed the default English resource to use [bold white on red]CTRL+C[/] markup and flagged translations for review
Updated each XLF translation file’s source markup and set targets to needs-review-translation
Inserted a blank line and the guidance message into the Spectre.Console grid in RunCommand.cs

Reviewed Changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 3 comments.

File	Description
src/Aspire.Cli/Resources/RunCommandStrings.resx	Changed markup to `[bold white on red]CTRL+C[/]` and updated comment
src/Aspire.Cli/Resources/xlf/RunCommandStrings.*.xlf	Updated source to `[bold white on red]CTRL+C[/]` and flagged targets for review
src/Aspire.Cli/Commands/RunCommand.cs	Added two rows (spacing + Ctrl+C guidance) to the console grid

Copilot · 2025-07-06T05:41:45Z

src/Aspire.Cli/Commands/RunCommand.cs

            topGrid.AddRow(new Align(new Markup($"[bold green]{logsLocalizedString}[/]:"), HorizontalAlignment.Right), new Text(logFile.FullName));
+            topGrid.AddRow(new Text(string.Empty), new Text(string.Empty));
+            topGrid.AddRow(new Text(string.Empty), new Markup(RunCommandStrings.PressCtrlCToStopAppHost));


The guidance message is added after the logs row—per the PR description it should appear before the logs. Move these two AddRow calls so the message displays between the dashboard links and the logs.

Suggested change

topGrid.AddRow(new Align(new Markup($"[bold green]{logsLocalizedString}[/]:"), HorizontalAlignment.Right), new Text(logFile.FullName));

topGrid.AddRow(new Text(string.Empty), new Text(string.Empty));

topGrid.AddRow(new Text(string.Empty), new Markup(RunCommandStrings.PressCtrlCToStopAppHost));

topGrid.AddRow(new Text(string.Empty), new Markup(RunCommandStrings.PressCtrlCToStopAppHost));

topGrid.AddRow(new Align(new Markup($"[bold green]{logsLocalizedString}[/]:"), HorizontalAlignment.Right), new Text(logFile.FullName));

topGrid.AddRow(new Text(string.Empty), new Text(string.Empty));

Copilot · 2025-07-06T05:41:45Z

src/Aspire.Cli/Commands/RunCommand.cs

            }
            topGrid.AddRow(Text.Empty, Text.Empty);
            topGrid.AddRow(new Align(new Markup($"[bold green]{logsLocalizedString}[/]:"), HorizontalAlignment.Right), new Text(logFile.FullName));
+            topGrid.AddRow(new Text(string.Empty), new Text(string.Empty));


[nitpick] Use the existing Text.Empty constant instead of new Text(string.Empty) for an empty cell to reduce allocations and improve readability.

Suggested change

topGrid.AddRow(new Text(string.Empty), new Text(string.Empty));

topGrid.AddRow(Text.Empty, Text.Empty);

Copilot · 2025-07-06T05:41:45Z

src/Aspire.Cli/Commands/RunCommand.cs

            topGrid.AddRow(Text.Empty, Text.Empty);
            topGrid.AddRow(new Align(new Markup($"[bold green]{logsLocalizedString}[/]:"), HorizontalAlignment.Right), new Text(logFile.FullName));
+            topGrid.AddRow(new Text(string.Empty), new Text(string.Empty));
+            topGrid.AddRow(new Text(string.Empty), new Markup(RunCommandStrings.PressCtrlCToStopAppHost));


[nitpick] Wrap the guidance message’s Markup in an Align with HorizontalAlignment.Right (like the logs row) to keep column alignment consistent.

Suggested change

topGrid.AddRow(new Text(string.Empty), new Markup(RunCommandStrings.PressCtrlCToStopAppHost));

topGrid.AddRow(new Align(new Text(string.Empty), HorizontalAlignment.Right), new Markup(RunCommandStrings.PressCtrlCToStopAppHost));

davidfowl · 2025-07-06T08:28:54Z

Why is CTRL+C pink?

mitchdenny · 2025-07-07T12:40:28Z

Was just trying to make it stand out a bit.

maddymontaquila · 2025-07-07T19:35:05Z

apphost should be one word but other than that LGTM

Co-authored-by: maddymontaquila <12660687+maddymontaquila@users.noreply.github.com>

Copilot · 2025-07-07T19:43:47Z

Why is CTRL+C pink?

Fixed in e399c96 - removed the pink/red coloring and changed it back to simple bold formatting: [bold]CTRL+C[/]

apphost should be one word but other than that LGTM

Fixed in e399c96 - changed "app host" to "apphost" (one word) as requested

maddymontaquila · 2025-07-07T20:06:19Z

The CTRL + C message should also appear below the endpoints listed when running in a codespace, not before.

Co-authored-by: maddymontaquila <12660687+maddymontaquila@users.noreply.github.com>

Copilot · 2025-07-07T20:19:10Z

The CTRL + C message should also appear below the endpoints listed when running in a codespace, not before.

<img width="819" alt="image" src="https://github.com/user-attachments/assets/cb25a9ee-386...

Fixed in 0e9ca8b - the CTRL+C message now appears after the endpoints when running in codespaces/remote containers, not before them.

mitchdenny · 2025-07-08T10:37:35Z

@copilot revert that last change

@maddymontaquila I put the CTRL-C there because if we don't we'll have to continually repaint it (which I was trying to avoid). However upon further reflection I think we should give it a go. I won't get a chance to code it myself for a day or so. The way that I'm going to approach it is that I'm going to have a method that prints out the CTRL-C message, but before I print each endpoint I'm going to have a method that clears the last two lines writes the endpoint, and then prints the CTRL-C message again.

The effect should be that the CTRL-C message flows down the page prepending the endpoints to the top if we are in an environment that displays them (codespaces/devcontainers).

… detection Co-authored-by: mitchdenny <513398+mitchdenny@users.noreply.github.com>

mitchdenny · 2025-07-10T12:30:27Z

mitchdenny · 2025-07-10T12:31:48Z

@maddymontaquila code updated.

mitchdenny · 2025-07-10T12:34:13Z

Without codespaces:

davidfowl · 2025-07-10T12:41:05Z

Delete the pink background from CTRL+C. It looks kinda tacky

maddymontaquila · 2025-07-10T15:19:11Z

agree delete the pink but other than that looks good!!!

mitchdenny · 2025-07-11T11:07:32Z

/backport to release/9.4

github-actions · 2025-07-11T11:07:42Z

Started backporting to release/9.4: https://github.com/dotnet/aspire/actions/runs/16218551607

github-actions · 2025-07-11T11:07:48Z

@mitchdenny backporting to "release/9.4" failed, the patch most likely resulted in conflicts:

$ git am --3way --empty=keep --ignore-whitespace --keep-non-patch changes.patch

Creating an empty commit: Initial plan
Applying: Add CTRL-C guidance message to aspire run command output
Using index info to reconstruct a base tree...
M	src/Aspire.Cli/Commands/RunCommand.cs
Falling back to patching base and 3-way merge...
Auto-merging src/Aspire.Cli/Commands/RunCommand.cs
CONFLICT (content): Merge conflict in src/Aspire.Cli/Commands/RunCommand.cs
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0002 Add CTRL-C guidance message to aspire run command output
Error: The process '/usr/bin/git' failed with exit code 128

Please backport manually!

mitchdenny · 2025-07-11T11:34:08Z

/backport to release/9.4

github-actions · 2025-07-11T11:34:16Z

Started backporting to release/9.4: https://github.com/dotnet/aspire/actions/runs/16219031974

github-actions · 2025-07-11T11:34:21Z

@mitchdenny backporting to "release/9.4" failed, the patch most likely resulted in conflicts:

$ git am --3way --empty=keep --ignore-whitespace --keep-non-patch changes.patch

Creating an empty commit: Initial plan
Applying: Add CTRL-C guidance message to aspire run command output
Using index info to reconstruct a base tree...
M	src/Aspire.Cli/Commands/RunCommand.cs
Falling back to patching base and 3-way merge...
Auto-merging src/Aspire.Cli/Commands/RunCommand.cs
CONFLICT (content): Merge conflict in src/Aspire.Cli/Commands/RunCommand.cs
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
hint: When you have resolved this problem, run "git am --continue".
hint: If you prefer to skip this patch, run "git am --skip" instead.
hint: To restore the original branch and stop patching, run "git am --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Patch failed at 0002 Add CTRL-C guidance message to aspire run command output
Error: The process '/usr/bin/git' failed with exit code 128

Please backport manually!

* Initial plan * Add CTRL-C guidance message to aspire run command output Co-authored-by: mitchdenny <513398+mitchdenny@users.noreply.github.com> * Update CTRL-C message * Fix CTRL-C message formatting and change "app host" to "apphost" Co-authored-by: maddymontaquila <12660687+maddymontaquila@users.noreply.github.com> * Move CTRL+C message to appear after endpoints in codespaces Co-authored-by: maddymontaquila <12660687+maddymontaquila@users.noreply.github.com> * Fix CTRL+C message positioning in codespaces by improving environment detection Co-authored-by: mitchdenny <513398+mitchdenny@users.noreply.github.com> * Fixes to implementation. * Fix formatting (bye bye pink) --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: mitchdenny <513398+mitchdenny@users.noreply.github.com> Co-authored-by: Mitch Denny <midenn@microsoft.com> Co-authored-by: maddymontaquila <12660687+maddymontaquila@users.noreply.github.com>

…0324) * Initial plan * Add CTRL-C guidance message to aspire run command output * Update CTRL-C message * Fix CTRL-C message formatting and change "app host" to "apphost" * Move CTRL+C message to appear after endpoints in codespaces * Fix CTRL+C message positioning in codespaces by improving environment detection * Fixes to implementation. * Fix formatting (bye bye pink) --------- Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com> Co-authored-by: mitchdenny <513398+mitchdenny@users.noreply.github.com> Co-authored-by: maddymontaquila <12660687+maddymontaquila@users.noreply.github.com>

Initial plan

ecd83ae

Copilot AI assigned Copilot and mitchdenny Jul 2, 2025

Copilot started work on behalf of mitchdenny July 2, 2025 12:48 View session

Add CTRL-C guidance message to aspire run command output

9681785

Co-authored-by: mitchdenny <513398+mitchdenny@users.noreply.github.com>

Copilot AI changed the title ~~[WIP] Add 'Press CTRL-C to stop AppHost.' message after dashboard links in aspire run output~~ Add CTRL-C guidance message to aspire run command output Jul 2, 2025

Copilot AI requested a review from mitchdenny July 2, 2025 12:59

Copilot finished work on behalf of mitchdenny July 2, 2025 12:59

mitchdenny added 2 commits July 6, 2025 15:27

Merge branch 'main' into copilot/fix-6fb67a15-0888-4c5f-92fa-da32b350…

81bd67d

…1b3c

Update CTRL-C message

d8ee041

mitchdenny requested a review from maddymontaquila July 6, 2025 05:40

mitchdenny added the area-cli label Jul 6, 2025

mitchdenny added this to the 9.4 milestone Jul 6, 2025

mitchdenny marked this pull request as ready for review July 6, 2025 05:40

Copilot AI review requested due to automatic review settings July 6, 2025 05:40

Copilot AI reviewed Jul 6, 2025

View reviewed changes

mitchdenny enabled auto-merge (squash) July 6, 2025 05:43

Copilot started work on behalf of maddymontaquila July 7, 2025 19:35 View session

Fix CTRL-C message formatting and change "app host" to "apphost"

e399c96

Co-authored-by: maddymontaquila <12660687+maddymontaquila@users.noreply.github.com>

auto-merge was automatically disabled July 7, 2025 19:43
Head branch was pushed to by a user without write access

Copilot finished work on behalf of maddymontaquila July 7, 2025 19:44

Copilot started work on behalf of maddymontaquila July 7, 2025 20:06 View session

Move CTRL+C message to appear after endpoints in codespaces

0e9ca8b

Co-authored-by: maddymontaquila <12660687+maddymontaquila@users.noreply.github.com>

Copilot started work on behalf of mitchdenny July 8, 2025 10:37 View session

Fix CTRL+C message positioning in codespaces by improving environment…

c6f09c2

… detection Co-authored-by: mitchdenny <513398+mitchdenny@users.noreply.github.com>

Copilot finished work on behalf of mitchdenny July 8, 2025 10:55

Copilot started work on behalf of mitchdenny July 10, 2025 12:30 View session

Fixes to implementation.

ff4994c

mitchdenny unassigned Copilot Jul 10, 2025

Copilot stopped work on behalf of mitchdenny due to an error July 10, 2025 12:34
Copilot has encountered an error. See logs for additional details.

davidfowl approved these changes Jul 10, 2025

View reviewed changes

maddymontaquila approved these changes Jul 10, 2025

View reviewed changes

davidfowl mentioned this pull request Jul 11, 2025

[AspireE2E] There is no prompt for 'Ctrl+C' to shut down the project when running an Aspire CLI project. #10321

Closed

Fix formatting (bye bye pink)

338a1ea

mitchdenny enabled auto-merge (squash) July 11, 2025 11:07

mitchdenny merged commit 0df566a into main Jul 11, 2025
275 of 276 checks passed

mitchdenny deleted the copilot/fix-6fb67a15-0888-4c5f-92fa-da32b3501b3c branch July 11, 2025 11:24

mitchdenny mentioned this pull request Jul 11, 2025

[release/9.4] Backport 10203: Add CTRL-C message back into aspire run command #10324

Merged

github-actions bot locked and limited conversation to collaborators Aug 11, 2025

	topGrid.AddRow(new Text(string.Empty), new Text(string.Empty));
	topGrid.AddRow(Text.Empty, Text.Empty);

	topGrid.AddRow(new Text(string.Empty), new Markup(RunCommandStrings.PressCtrlCToStopAppHost));
	topGrid.AddRow(new Align(new Text(string.Empty), HorizontalAlignment.Right), new Markup(RunCommandStrings.PressCtrlCToStopAppHost));

Add CTRL-C guidance message to aspire run command output #10203

Add CTRL-C guidance message to aspire run command output #10203

Uh oh!

Conversation

Copilot AI commented Jul 2, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Output Format

Implementation Details

Uh oh!

mitchdenny commented Jul 6, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Jul 6, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 6, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jul 6, 2025

Choose a reason for hiding this comment

Uh oh!

davidfowl commented Jul 6, 2025

Uh oh!

mitchdenny commented Jul 7, 2025

Uh oh!

maddymontaquila commented Jul 7, 2025

Uh oh!

Copilot AI commented Jul 7, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

maddymontaquila commented Jul 7, 2025

Uh oh!

Copilot AI commented Jul 7, 2025

Uh oh!

mitchdenny commented Jul 8, 2025

Uh oh!

mitchdenny commented Jul 10, 2025

Uh oh!

mitchdenny commented Jul 10, 2025

Uh oh!

mitchdenny commented Jul 10, 2025

Uh oh!

davidfowl commented Jul 10, 2025

Uh oh!

maddymontaquila commented Jul 10, 2025

Uh oh!

mitchdenny commented Jul 11, 2025

Uh oh!

github-actions bot commented Jul 11, 2025

Uh oh!

github-actions bot commented Jul 11, 2025

Uh oh!

Uh oh!

mitchdenny commented Jul 11, 2025

Uh oh!

github-actions bot commented Jul 11, 2025

Uh oh!

github-actions bot commented Jul 11, 2025

Uh oh!

Uh oh!

Copilot AI commented Jul 2, 2025 •

edited

Loading

Copilot AI commented Jul 7, 2025 •

edited

Loading