Skip to content

Improve Aspire 13.5 AppHost discovery and pnpm recovery - #56

Draft
Ella Hathaway (ellahathaway) wants to merge 5 commits into
mainfrom
ellahathaway-aspire-13-5-skill-followups
Draft

Improve Aspire 13.5 AppHost discovery and pnpm recovery#56
Ella Hathaway (ellahathaway) wants to merge 5 commits into
mainfrom
ellahathaway-aspire-13-5-skill-followups

Conversation

@ellahathaway

@ellahathaway Ella Hathaway (ellahathaway) commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

What changed

Improve Aspire 13.5 skill guidance for TypeScript AppHosts and complex JavaScript repositories.

  • Use aspire.config.json to find the actual AppHost source file, including nested or custom paths.
  • Recognize apphost.run.json as launch metadata rather than an AppHost source file.
  • Recover safely when a pnpm workspace policy blocks esbuild during aspire init, without changing the root policy.
  • Add first-party Valkey guidance using Aspire.Hosting.Valkey and addValkey.
  • Ask users to choose the normal Compose command/profile when several local launch options exist.
  • Avoid duplicate resources when Compose and package manifests describe the same application, and exclude docs, tests, E2E, and tooling by default.
  • Add eval coverage for these scenarios and migrate changed specs from removed expect_skills constraints to skill-invocation graders.

Validation

  • Current @microsoft/vally-cli lint passes for the Aspire, Aspire Init, and Aspireify eval specs.
  • A real nested pnpm workspace ran esbuild's postinstall with the scoped allowBuilds policy while the root esbuild: false policy remained unchanged.
  • npm test is currently blocked by unrelated Windows/Node 25 telemetry-hook failures.

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

Copilot-Session: ad1ceffa-2889-4e8a-966c-5272b9643123
@ellahathaway Ella Hathaway (ellahathaway) changed the title Add Aspire 13.5 skill recovery guidance Improve Aspire 13.5 AppHost discovery and pnpm recovery Sep 2, 2026
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: ec6d1adc-bc3b-49b5-9d41-9dba40b73e07
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: ec6d1adc-bc3b-49b5-9d41-9dba40b73e07
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: ad1ceffa-2889-4e8a-966c-5272b9643123
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: ad1ceffa-2889-4e8a-966c-5272b9643123
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant