Skip to content

Align Guardex finish-path regressions with the current workflow contract#223

Merged
NagyVikt merged 1 commit intomainfrom
agent/codex/fix-regression-finish-flows-2026-04-21-13-00
Apr 21, 2026
Merged

Align Guardex finish-path regressions with the current workflow contract#223
NagyVikt merged 1 commit intomainfrom
agent/codex/fix-regression-finish-flows-2026-04-21-13-00

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Automated by scripts/agent-branch-finish.sh (PR flow).

This change finishes the remaining workflow-regression cleanup around doctor sandbox repair sync, agent role naming, and codex-agent auto-finish gating. Doctor no longer treats its own .omx/agent-worktrees sandbox path as unrelated protected-branch drift, finish runs now allocate unique integration refs, and the focused install regressions were updated to the current protected-main and local-remote behavior. The OpenSpec change docs were also backfilled so the branch is reviewable instead of carrying placeholder scaffolding.

Constraint: Doctor and auto-finish can invoke multiple finish flows inside the same repo within one timestamp window
Rejected: Keep the stale install expectations and only patch runtime output | that would leave real doctor sync and integration-ref collisions hidden
Confidence: high
Scope-risk: moderate
Directive: Treat doctor sandbox worktrees as internal workflow state during protected-base repair sync, and keep temporary integration refs unique across consecutive finish runs
Tested: node --test test/install.test.js; bash scripts/test-agent-naming.sh; openspec validate fix-regression-finish-flows --type change --strict; openspec validate --specs
Not-tested: Live GitHub merge behavior outside the scripted harness
@NagyVikt NagyVikt merged commit 9dfa87f into main Apr 21, 2026
@NagyVikt NagyVikt deleted the agent/codex/fix-regression-finish-flows-2026-04-21-13-00 branch April 21, 2026 11:33
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