Normalize CLI command descriptions for consistency#15204
Merged
JamesNK merged 3 commits intorelease/13.2from Mar 18, 2026
Merged
Normalize CLI command descriptions for consistency#15204JamesNK merged 3 commits intorelease/13.2from
JamesNK merged 3 commits intorelease/13.2from
Conversation
Contributor
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 15204Or
iex "& { $(irm https://raw.githubusercontent.com/dotnet/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 15204" |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR normalizes CLI command description text in the Aspire CLI for consistency. It lowercases "AppHost" to "apphost" in user-facing command descriptions, adds a missing trailing period to the aspire export description, and updates the aspire agent description wording. All corresponding .resx, .Designer.cs, and .xlf localization files are updated.
Changes:
- Changed "AppHost" to "apphost" (lowercase) across all CLI command descriptions in resx/Designer.cs/xlf files
- Added missing trailing period to
ExportCommandStrings.Description - Updated
AgentCommandStrings.Descriptionfrom "Manage AI agent specific setup." to "Manage AI agent environment configuration."
Reviewed changes
Copilot reviewed 168 out of 175 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
*/AgentCommandStrings.resx, .Designer.cs, *.xlf |
Updated agent command description wording |
*/ExportCommandStrings.resx, .Designer.cs, *.xlf |
Added trailing period to export description |
*/RunCommandStrings.resx, .Designer.cs, *.xlf |
Lowercased AppHost → apphost in descriptions |
*/PublishCommandStrings.resx, .Designer.cs, *.xlf |
Lowercased AppHost → apphost in project path description |
*/AddCommandStrings.resx, *.xlf |
Lowercased AppHost → apphost in project path description |
*/UpdateCommandStrings.resx, .Designer.cs, *.xlf |
Lowercased AppHost → apphost in project path description |
*/StopCommandStrings.resx, *.xlf |
Lowercased AppHost → apphost in descriptions |
*/SharedCommandStrings.resx, *.xlf |
Lowercased AppHost → apphost in shared descriptions |
*/SecretCommandStrings.resx, .Designer.cs, *.xlf |
Lowercased AppHost → apphost in secret command description |
*/NewCommandStrings.resx, *.xlf |
Lowercased AppHost → apphost in language option description |
*/ExecCommandStrings.resx, .Designer.cs, *.xlf |
Lowercased AppHost → apphost in exec command descriptions |
*/RestoreCommandStrings.resx, .Designer.cs, *.xlf |
Lowercased AppHost → apphost in restore description |
Files not reviewed (7)
- src/Aspire.Cli/Resources/AgentCommandStrings.Designer.cs: Language not supported
- src/Aspire.Cli/Resources/ExecCommandStrings.Designer.cs: Language not supported
- src/Aspire.Cli/Resources/ExportCommandStrings.Designer.cs: Language not supported
- src/Aspire.Cli/Resources/PublishCommandStrings.Designer.cs: Language not supported
- src/Aspire.Cli/Resources/RestoreCommandStrings.Designer.cs: Language not supported
- src/Aspire.Cli/Resources/SecretCommandStrings.Designer.cs: Language not supported
- src/Aspire.Cli/Resources/UpdateCommandStrings.Designer.cs: Language not supported
| </data> | ||
| <data name="NoInScopeAppHostsShowingAll" xml:space="preserve"> | ||
| <value>No running AppHosts found in the current directory. Select from all running AppHosts.</value> | ||
| <value>No running AppHosts found in the current directory. Select from all running apphosts.</value> |
Comment on lines
+26
to
+27
| <source>No running AppHosts found in the current directory. Select from all running apphosts.</source> | ||
| <target state="new">No running AppHosts found in the current directory. Select from all running apphosts.</target> |
Contributor
🎬 CLI E2E Test RecordingsThe following terminal recordings are available for commit
📹 Recordings uploaded automatically from CI run #23225102961 |
davidfowl
force-pushed
the
davidfowl/review-commands
branch
from
d28ce22 to
641e8db
Compare
IEvangelist
reviewed
Mar 13, 2026
Member
IEvangelist
left a comment
IEvangelist
left a comment
There was a problem hiding this comment.
It was already normalized as AppHost throughout, this is changing it to all lowercase. Would we need to update docs them too, or is this just a CLI concern?
davidfowl
force-pushed
the
davidfowl/review-commands
branch
from
641e8db to
d5e738c
Compare
Contributor
|
The transient CI rerun workflow requested reruns for the following jobs after analyzing the failed attempt.
|
Contributor
Author
WE can change these all to AppHost but there's mixed casing and AppHost feels very .NET esquee |
This was referenced Mar 14, 2026
maddymontaquila
approved these changes
Mar 17, 2026
JamesNK
approved these changes
Mar 18, 2026
- Use 'apphost' (lowercase) consistently instead of mixed 'AppHost'/'apphost' - Fix 'aspire export' missing trailing period - Update 'aspire agent' description to 'Manage AI agent environment configuration.' - Updated resx, Designer.cs, and all xlf localization files Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
JamesNK
force-pushed
the
davidfowl/review-commands
branch
from
5f48289 to
f102f67
Compare
Contributor
|
The transient CI rerun workflow requested reruns for the following jobs after analyzing the failed attempt.
|
Member
|
/deployment-test |
Contributor
|
🚀 Deployment tests starting on PR #15204... This will deploy to real Azure infrastructure. Results will be posted here when complete. |
Contributor
|
❌ Deployment E2E Tests failed — 25 passed, 2 failed, 0 cancelled View test results and recordings
|
Copilot AI
pushed a commit
that referenced
this pull request
Mar 18, 2026
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: James Newton-King <james@newtonking.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Normalize CLI command, option, and argument descriptions for consistency and CLI best practices.
Changes
Casing normalization
AppHost→apphost(lowercase, one word) in all user-facing CLI strings across 32+ resx filesAppHostcasing in code identifiers (e.g.IsCompatibleAppHost), package names (Aspire.AppHost.Sdk), and template display names ("Empty AppHost")Trailing period normalization
dotnet/git/kubectlCLI conventions)Move hardcoded strings to resx
McpCallCommand,McpToolsCommand,PipelineCommandBase,DeployCommand,InitCommand, andUpdateCommandinto their corresponding resx resource files for localization consistencyLocalization
Other
aspire exportdescriptionaspire agentdescription from "Manage AI agent specific setup" to "Manage AI agent environment configuration"Files changed
No behavioral changes — description text only.
Checklist