Rename .NET Aspire references to Aspire#1239
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
🟡 Impact Analysis — PR #1239Risk tier: 🟡 MEDIUM 📊 Summary
🎯 Risk Factors
📦 Modules Affecteddocs (6 files)
root (3 files)
squad-cli (3 files)
squad-state (2 files)
This report is generated automatically for every PR. See #733 for details. |
🏗️ Architectural Review
Automated architectural review — informational only. |
🛫 PR Readiness Check
PR Scope: 📦🔧 Mixed (product + infrastructure)
|
| Status | Check | Details |
|---|---|---|
| ❌ | Single commit | 2 commits — consider squashing before review |
| ✅ | Not in draft | Ready for review |
| ❌ | Branch up to date | dev is 1 commit(s) ahead — rebase recommended |
| ❌ | Copilot review | No Copilot review yet — it may still be processing |
| ✅ | Changeset present | Changeset file found |
| ✅ | Scope clean | |
| ✅ | No merge conflicts | No merge conflicts |
| ❌ | Copilot threads resolved | 3 unresolved Copilot thread(s) — fix and resolve before merging |
| ❌ | CI passing | 6 check(s) still running |
Files Changed (14 files, +22 −17)
| File | +/− |
|---|---|
.changeset/rename-dotnet-aspire-to-aspire.md |
+5 −0 |
.squad/agents/_alumni/saul/charter.md |
+1 −1 |
.squad/decisions-archive.md |
+1 −1 |
docs/src/content/blog/014-wave-1-otel-and-aspire.md |
+1 −1 |
docs/src/content/blog/022-welcome-to-the-new-squad.md |
+1 −1 |
docs/src/content/docs/features/mcp.md |
+1 −1 |
docs/src/content/docs/guide/build-autonomous-agent.md |
+2 −2 |
docs/src/content/docs/guide/sample-prompts.md |
+2 −2 |
docs/src/content/docs/sample-prompts.md |
+2 −2 |
packages/squad-cli/src/cli-entry.ts |
+1 −1 |
packages/squad-cli/src/cli/commands/aspire.ts |
+2 −2 |
packages/squad-cli/src/cli/core/command-help.ts |
+1 −1 |
samples/autonomous-pipeline/README.md |
+1 −1 |
samples/autonomous-pipeline/TEST-SCRIPT.md |
+1 −1 |
Total: +22 −17
This check runs automatically on every push. Fix any ❌ items and push again.
See CONTRIBUTING.md and PR Requirements for details.
|
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
This PR standardizes user-facing terminology by removing the “.NET” prefix from “.NET Aspire” across docs and the squad aspire CLI surface.
Changes:
- Updated CLI help text and console output to say “Aspire dashboard” instead of “.NET Aspire dashboard”.
- Updated docs/blog/sample materials to use “Aspire” branding consistently.
- Added a changeset for a patch release of
@bradygaster/squad-cli.
Reviewed changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| samples/autonomous-pipeline/TEST-SCRIPT.md | Adjusts demo script wording to “Aspire”. |
| samples/autonomous-pipeline/README.md | Updates observability instructions to “Aspire dashboard”. |
| packages/squad-cli/src/cli/core/command-help.ts | Updates squad aspire help header tagline wording. |
| packages/squad-cli/src/cli/commands/aspire.ts | Updates command doc comment and a user-facing error message. |
| packages/squad-cli/src/cli-entry.ts | Updates top-level CLI command list description for aspire. |
| docs/src/content/docs/sample-prompts.md | Updates sample prompt headings/text to “Aspire”. |
| docs/src/content/docs/guide/sample-prompts.md | Updates guide sample prompt headings/text to “Aspire”. |
| docs/src/content/docs/guide/build-autonomous-agent.md | Updates OTel/Aspire references to “Aspire dashboard”. |
| docs/src/content/docs/features/mcp.md | Updates MCP feature docs to “Aspire projects”. |
| docs/src/content/blog/022-welcome-to-the-new-squad.md | Updates blog wording to “Aspire Dashboard”. |
| docs/src/content/blog/014-wave-1-otel-and-aspire.md | Updates blog wording, but introduces a grammar issue (“a Aspire”). |
| .squad/decisions-archive.md | Updates internal decision wording to remove “.NET” prefix. |
| .squad/agents/_alumni/saul/charter.md | Updates internal charter wording to remove “.NET” prefix. |
| .changeset/rename-dotnet-aspire-to-aspire.md | Adds changeset documenting the terminology update for the CLI package. |
| ## Aspire Dashboard Integration | ||
|
|
||
| The `squad aspire` command (#265) wires Squad's OTLP exporter to a .NET Aspire dashboard. Aspire gives you: | ||
| The `squad aspire` command (#265) wires Squad's OTLP exporter to a Aspire dashboard. Aspire gives you: |
|
|
||
| ``` | ||
| Build a cloud-native app with .NET Aspire (read https://learn.microsoft.com/en-us/dotnet/aspire/): | ||
| Build a cloud-native app with Aspire (read https://learn.microsoft.com/en-us/dotnet/aspire/): |
| child = launchWithDocker(); | ||
| } else { | ||
| console.log(`${RED}✗${RESET} Neither Docker nor .NET Aspire workload found.`); | ||
| console.log(`${RED}✗${RESET} Neither Docker nor Aspire workload found.`); |
| ### Act 6: OTel / Aspire (30 seconds, if Aspire is running) | ||
|
|
||
| **Say:** "And all of this feeds into .NET Aspire." | ||
| **Say:** "And all of this feeds into Aspire." |
Co-authored-by: Meir Blachman <meirblachman@gmail.com>
| console.log(`${RED}✗${RESET} Neither Docker nor Aspire workload found.`); | ||
| console.log(`\n Install options:`); | ||
| console.log(` ${BOLD}Docker:${RESET} https://docker.com/get-started`); | ||
| console.log(` ${BOLD}Aspire:${RESET} dotnet workload install aspire\n`); |
There was a problem hiding this comment.
should this be removed completely?
Summary
Aspireacross CLI help, docs, samples, and historical Squad notesValidation
npm ci --no-audit --no-fundSKIP_BUILD_BUMP=1 npm run buildnode_modulesReview: FIDO approved the diff.