From ffd003ce289c0522de1e16da4a5595423fbbb805 Mon Sep 17 00:00:00 2001 From: NagyVikt Date: Thu, 23 Apr 2026 17:05:47 +0200 Subject: [PATCH] Record merged cleanup evidence for Active Agents PR #382 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 --- .../tasks.md | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/openspec/changes/agent-codex-show-active-agents-in-second-vscode-wind-2026-04-23-16-50/tasks.md b/openspec/changes/agent-codex-show-active-agents-in-second-vscode-wind-2026-04-23-16-50/tasks.md index df42c56..8a7a7ee 100644 --- a/openspec/changes/agent-codex-show-active-agents-in-second-vscode-wind-2026-04-23-16-50/tasks.md +++ b/openspec/changes/agent-codex-show-active-agents-in-second-vscode-wind-2026-04-23-16-50/tasks.md @@ -9,6 +9,7 @@ This change is complete only when **all** of the following are true: ## Handoff - Handoff: change=`agent-codex-show-active-agents-in-second-vscode-wind-2026-04-23-16-50`; branch=`agent/codex/show-active-agents-in-second-vscode-wind-2026-04-23-16-50`; scope=`VS Code Active Agents repo-root resolution for second-window worktree/subdir views, template parity, focused regression`; action=`show owning gitguardex agents when another VS Code window opens on a linked worktree or nested repo path, verify, then finish via PR merge cleanup`. +- Completion: PR=`https://github.com/recodeee/gitguardex/pull/382`; state=`MERGED`; merge_commit=`7378ae883dbc4d4d31f2cff54e8f230d39acfb34`; cleanup_evidence=`source worktree removed from git worktree list; local branch deleted; git fetch --prune origin removed origin/agent/codex/show-active-agents-in-second-vscode-wind-2026-04-23-16-50`. ## 1. Specification @@ -30,6 +31,6 @@ This change is complete only when **all** of the following are true: ## 4. Cleanup (mandatory; run before claiming completion) -- [ ] 4.1 Run `gx branch finish --branch agent/codex/show-active-agents-in-second-vscode-wind-2026-04-23-16-50 --base main --via-pr --wait-for-merge --cleanup`. -- [ ] 4.2 Record the PR URL and final merge state (`MERGED`) in the completion handoff. -- [ ] 4.3 Confirm the sandbox worktree is gone (`git worktree list` no longer shows the agent path; `git branch -a` shows no surviving local/remote refs for the branch). +- [x] 4.1 Run `gx branch finish --branch agent/codex/show-active-agents-in-second-vscode-wind-2026-04-23-16-50 --base main --via-pr --wait-for-merge --cleanup`. +- [x] 4.2 Record the PR URL and final merge state (`MERGED`) in the completion handoff. +- [x] 4.3 Confirm the sandbox worktree is gone (`git worktree list` no longer shows the agent path; `git branch -a` shows no surviving local/remote refs for the branch).