Skip to content

Report pending gx doctor PR state accurately#70

Merged
NagyVikt merged 1 commit intomainfrom
agent/codex/info-recodee-com-fix-gx-doctor-pending-autofinish
Apr 13, 2026
Merged

Report pending gx doctor PR state accurately#70
NagyVikt merged 1 commit intomainfrom
agent/codex/info-recodee-com-fix-gx-doctor-pending-autofinish

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

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

When doctor runs on protected main, sandbox auto-finish can return success while leaving the PR open due branch policy. The CLI previously labeled this as "Auto-finish flow completed", which is misleading and caused confusion. This change inspects agent-branch-finish output for pending markers, reports a pending status with PR URL, and adds a regression test for blocked merge policy behavior.

Constraint: Keep doctor exit behavior non-fatal when PR is created but merge is pending policy gates

Rejected: Force --wait-for-merge for all doctor runs | risks long hangs and timeouts on review-gated repositories

Confidence: high

Scope-risk: narrow

Reversibility: clean

Directive: Treat agent-branch-finish pending markers as non-complete states in higher-level status messaging

Tested: node --check bin/multiagent-safety.js

Tested: node --test test/install.test.js

Tested: npm test
@NagyVikt NagyVikt merged commit 35d217d into main Apr 13, 2026
7 checks passed
@NagyVikt NagyVikt deleted the agent/codex/info-recodee-com-fix-gx-doctor-pending-autofinish branch April 13, 2026 18:57
@NagyVikt NagyVikt restored the agent/codex/info-recodee-com-fix-gx-doctor-pending-autofinish branch April 13, 2026 19:08
@NagyVikt NagyVikt deleted the agent/codex/info-recodee-com-fix-gx-doctor-pending-autofinish branch April 13, 2026 19:22
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