Skip to content

Preserve full cleanup-evidence scaffolds for merged-proof takeovers#306

Merged
NagyVikt merged 1 commit intomainfrom
agent/codex/simple-record-merged-cleanup-evidence-fo-2026-04-22-12-32
Apr 22, 2026
Merged

Preserve full cleanup-evidence scaffolds for merged-proof takeovers#306
NagyVikt merged 1 commit intomainfrom
agent/codex/simple-record-merged-cleanup-evidence-fo-2026-04-22-12-32

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Automated by gx branch finish (PR flow).

Lightweight prefixes stay the default escape hatch for tiny work, but cleanup-evidence asks still need a real OpenSpec change workspace so takeover lanes can carry PR URL, MERGED state, and sandbox-prune proof honestly. This narrows the override to artifact-heavy cleanup wording, promotes those requests from T1 to T2, and adds a routing regression plus change docs for the exact failing shape.

Constraint: Ordinary simple-prefixed tasks must remain caveman-first and notes-only by default
Rejected: Promote all simple-prefixed workflow wording to T2 | would weaken the explicit lightweight escape hatch
Confidence: high
Scope-risk: narrow
Reversibility: clean
Directive: Keep lightweight-prefix overrides narrow; only artifact-heavy cleanup-proof requests should bypass T1
Tested: node --test test/sandbox.test.js test/metadata.test.js
Tested: openspec validate agent-codex-simple-record-merged-cleanup-evidence-fo-2026-04-22-12-32 --type change --strict
Tested: openspec validate --specs
Not-tested: End-to-end finish pipeline before this commit
@NagyVikt NagyVikt merged commit a40e42b into main Apr 22, 2026
@NagyVikt NagyVikt deleted the agent/codex/simple-record-merged-cleanup-evidence-fo-2026-04-22-12-32 branch April 22, 2026 12:58
NagyVikt added a commit that referenced this pull request Apr 22, 2026
The routing fix itself is already merged, but its change file still showed unchecked 4.x cleanup boxes because the finish command exited after GitHub had already deleted the source ref. This helper branch writes back the actual PR, merge, and prune evidence observed on main so the OpenSpec record matches reality.

Constraint: The original agent branch and worktree were already gone, so the evidence had to be recorded from a tiny helper branch on main
Rejected: Leave the unchecked cleanup boxes in place | violates the change definition of done
Confidence: high
Scope-risk: narrow
Reversibility: clean
Directive: When finish merges successfully but cleanup exits on an already-deleted ref, backfill the real proof instead of inventing a fresh narrative
Tested: gh pr view 306 --json number,url,state,mergedAt,mergeCommit
Tested: git worktree list
Tested: git branch -a
Not-tested: Automated suite (doc-only evidence backfill)

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