Skip to content

Keep Active Agents visible in a second VS Code window#382

Merged
NagyVikt merged 1 commit intomainfrom
agent/codex/show-active-agents-in-second-vscode-wind-2026-04-23-16-50
Apr 23, 2026
Merged

Keep Active Agents visible in a second VS Code window#382
NagyVikt merged 1 commit intomainfrom
agent/codex/show-active-agents-in-second-vscode-wind-2026-04-23-16-50

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Automated by gx branch finish (PR flow).

The Active Agents repo scan treated the opened workspace path as the repo root. In a second VS Code window opened on a linked worktree or subfolder, that hid the owning repo's active-session files. Resolve the owning repo root from git metadata before reading sessions, keep template parity, and cover the linked-worktree window with a regression test.

Constraint: Second VS Code windows may open on linked worktrees or repo subfolders, not only the owning repo root
Rejected: Aggregate every parent-workspace daemon session | would leak unrelated repo agents into a repo-scoped window
Confidence: high
Scope-risk: narrow
Reversibility: clean
Directive: Resolve workspace folders to owning repo roots before reading Active Agents session state
Tested: NODE_PATH=/home/deadpool/Documents/recodee/gitguardex/node_modules node --test test/vscode-active-agents-session-state.test.js
Tested: openspec validate agent-codex-show-active-agents-in-second-vscode-wind-2026-04-23-16-50 --type change --strict
Tested: openspec validate --specs (no items found to validate)
Not-tested: Manual two-window VS Code clickthrough
@NagyVikt NagyVikt merged commit 7378ae8 into main Apr 23, 2026
@NagyVikt NagyVikt deleted the agent/codex/show-active-agents-in-second-vscode-wind-2026-04-23-16-50 branch April 23, 2026 15:03
NagyVikt added a commit that referenced this pull request Apr 23, 2026
The feature lane merged cleanly, but the cleanup boxes in the change record stayed unchecked because the finish flow stopped at local branch deletion while the /tmp worktree was still attached. Record the real PR, merge commit, and prune evidence so the merged change remains truthful on main.

Constraint: Cleanup evidence must match post-merge branch and worktree state, not pre-finish intent
Rejected: Leave the merged tasks file with unchecked cleanup boxes | makes the OpenSpec closeout inaccurate
Confidence: high
Scope-risk: narrow
Reversibility: clean
Directive: If finish exits after merge, capture PR state with gh and then backfill tasks.md from a follow-up lane
Tested: openspec validate agent-codex-show-active-agents-in-second-vscode-wind-2026-04-23-16-50 --type change --strict
Not-tested: Additional automated suites; docs-only bookkeeping change

Co-authored-by: NagyVikt <nagy.viktordp@gmail.com>
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