Skip to content

Keep Active Agents in its VS Code container#429

Merged
NagyVikt merged 1 commit intomainfrom
agent/codex/fix-active-agents-view-container-id-2026-04-25-15-48
Apr 25, 2026
Merged

Keep Active Agents in its VS Code container#429
NagyVikt merged 1 commit intomainfrom
agent/codex/fix-active-agents-view-container-id-2026-04-25-15-48

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Automated by gx branch finish (PR flow).

VS Code rejects activity bar container identifiers that contain dots, which caused the Active Agents contribution to fall back into Explorer and left the focus command targeting a container that was never registered. The live and template manifests now use a hyphenated container id while the existing view id stays unchanged.

Constraint: VS Code view container ids allow only alphanumeric characters, underscore, and hyphen

Rejected: Rename the gitguardex.activeAgents view id | wider command/menu/test churn was unnecessary

Confidence: high

Scope-risk: narrow

Directive: Keep live and template Active Agents manifests and focus commands in sync when changing VS Code contribution ids

Tested: node --test test/vscode-active-agents-session-state.test.js

Tested: npm test

Tested: node scripts/install-vscode-active-agents-extension.js
@NagyVikt NagyVikt merged commit 76dd505 into main Apr 25, 2026
@NagyVikt NagyVikt deleted the agent/codex/fix-active-agents-view-container-id-2026-04-25-15-48 branch April 25, 2026 13:56
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