Repository navigation
Retire embedded Aspire skills bundle - #20947
Ella Hathaway (ellahathaway) wants to merge 10 commits into
Conversation
This comment has been minimized.
This comment has been minimized.
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
ea92732 to
d07155a
Compare
|
🚀 Dogfood this PR with:
curl -fsSL https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.sh | bash -s -- 20947Or
iex "& { $(irm https://raw.githubusercontent.com/microsoft/aspire/main/eng/scripts/get-aspire-cli-pr.ps1) } 20947" |
## Description Users should choose their agents, not skill folders. `agent init`, `new`, and `init` now share client selection and project/global scope. - Use stable CLI IDs `copilot`, `claude`, and `opencode`, with localized GitHub Copilot, Claude Code, and OpenCode labels. - Prompt **agents → assets → scope**. Preselect detected agents; use Copilot only when none are detected. - Write only to selected clients and scope, deduplicating shared destinations. Selecting no agents/assets performs no agent writes. MCP remains a standalone opt-in. - Keep scanners read-only, client metadata focused on destinations, and configuration writes in shared helpers. This PR provides the foundation for `aspire-skills` plugin support by letting users choose their agent clients and installation scope rather than filesystem locations. Clients can share skill directories while requiring different native plugin registration, so client selection determines which installation behavior is needed, while detection supplies sensible defaults. This keeps the user-facing flow stable as native plugin support is added, while preserving copied-skill installation in this intermediate layer. This layer retains **copied skills and the existing bundle**. Native plugins [#20946](#20946) and bundle retirement [#20947](#20947) remain separate follow-ups. ```shell aspire agent init --agents claude --scope project --skills aspire --mcp=false ``` Related to #20247 --------- Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: c0f73663-c37c-4c5a-b83d-89f0b8d13c5f
c7277cb to
792f4eb
Compare
This comment has been minimized.
This comment has been minimized.
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
792f4eb to
673c82a
Compare
This comment has been minimized.
This comment has been minimized.
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Remove the cross-repository updater, verifier, provenance metadata, and protected synchronization branch while retaining the embedded runtime scripts. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
673c82a to
cd1ddee
Compare
Tests selectorSelects the full PR test matrix + all PR-gated jobs (ALL) — run-all fallback: 'eng/scripts/aspire-skills-bundle.common.ps1' is neither Layer-1-owned nor matched by a Layer 2 rule Selection computed for commit |
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
|
Retrying the failed CI jobs for this pull request from the CI run attempt. The rerun is being tracked in the rerun attempt. |
Description
Removes the embedded Aspire skills bundle and its acquisition/runtime machinery after #20946.
--aspire-skillsnow supportsnative|none, defaults to native client registration, and no longer copies a bundled archive into client skill directories. Existing copied Aspire skills are detected before native registration so the CLI does not silently create an ambiguous configuration.Telemetry hook scripts remain embedded because the Aspire CLI installs and executes them independently of the plugin. Aspire now owns those script sources directly: this removes the cross-repository updater/verifier workflows, provenance metadata, synchronization scripts, and protected synchronization-branch routing. Future script changes are ordinary Aspire code changes rather than bundle synchronization. The canonical
track-telemetry.shandtrack-telemetry.ps1bytes are unchanged.This is the final layer of stack #20948 (
#20932 → #20946 → #20947). Related to #20246 and epic #20245.Merge gates: #20946 and the corresponding
microsoft/aspire-skillscatalog release must be ready before this retirement layer lands. No formal security review has been completed for this PR.Validation:
dotnet build src\Aspire.Cli\Aspire.Cli.csproj --no-restore --nologo --verbosity quiet /p:SkipNativeBuild=true— passed.dotnet build tests\Aspire.Cli.Tests\Aspire.Cli.Tests.csproj --no-restore --nologo --verbosity quiet /p:SkipNativeBuild=true— passed.dotnet build tests\Aspire.Cli.EndToEnd.Tests\Aspire.Cli.EndToEnd.Tests.csproj --no-restore --nologo --verbosity quiet /p:SkipNativeBuild=true— passed.dotnet build /t:UpdateXlf src\Aspire.Cli\Aspire.Cli.csproj --no-restore --nologo --verbosity quiet /p:SkipNativeBuild=true— passed with no generated changes.a4f26ccb1925e48a26c1506fda20b636cfc27184(track-telemetry.sh) andff6c8b3f9a2f4b6762d0d890dfa57419e66f73e5(track-telemetry.ps1).The CLI builds emitted only the existing MessagePack 2.5.302
NU1902advisory. Validation was on Windows ARM64. Bash, quarantined, outerloop, E2E runtime, full-repository, Native AOT, downloaded-artifact, live catalog acquisition, post-push CI, and formal security-review validation were not run. No snapshot baselines changed.Checklist
<remarks />and<code />elements on your triple slash comments?