feat(vscode): add native SDD agents#708
Open
Snakeblack wants to merge 1 commit into
Open
Conversation
Author
|
Maintainer label request: please apply exactly one type label, |
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.
🔗 Linked Issue
Closes #504
Related context: #677
🏷️ PR Type
What kind of change does this PR introduce?
type:bug— Bug fix (non-breaking change that fixes an issue)type:feature— New feature (non-breaking change that adds functionality)type:docs— Documentation onlytype:refactor— Code refactoring (no functional changes)type:chore— Build, CI, or tooling changestype:breaking-change— Breaking change (fix or feature that changes existing behavior)📝 Summary
PR 1 of the VS Code Copilot SDD subagents stack. This adds the backend/native-agent foundation only: VS Code now supports native SDD
.agent.mdfiles in~/.copilot/agents, with embedded coordinator/phase assets, injection post-check support, idempotency coverage, and uninstall preservation for user-authored agents.Follow-up PRs will add VS Code model assignment foundation and TUI/docs. Those are intentionally out of scope here.
This PR requests
size:exception: it exceeds 400 changed lines because it ships substantive Markdown.agent.mdassets, not because the code surface is large.📂 Changes
internal/agents/vscode~/.copilot/agentsand added path testsinternal/assetsall:vscodeand added VS Code SDD.agent.mdassetsinternal/components/sdd.agent.mdin native sub-agent post-check and covered VS Code injection/idempotencyinternal/components/uninstall.agent.mdfiles🧪 Test Plan
Focused PR1 tests
Package smoke
go test ./internal/agents/vscode ./internal/assetsUnit Tests
go test ./...E2E Tests (Docker required)
go test ./...)cd e2e && ./docker-test.sh)Notes: focused PR1 tests passed locally. Full
go test ./...was not claimed here due known local Windows/CRLF/symlink/environment failures unrelated to this PR.🤖 Automated Checks
The following checks run automatically on this PR:
size:exception; most added lines are.agent.mdassetsCloses #504status:approvedstatus:approvedtype:*Labeltype:featurerequested (maintainer label needed)go test ./...✅ Contributor Checklist
status:approvedsize:exceptionwith rationale documented — maintainer label needed (size:exception)type:*label to this PR — maintainer label needed (type:feature)go test ./...)cd e2e && ./docker-test.sh)Co-Authored-Bytrailers💬 Notes for Reviewers
This is the backend/native-agent slice only. It deliberately does not include VS Code model assignment state, dynamic Copilot model catalog integration, TUI/profile UX, or final docs updates; those will come in follow-up PRs.