Skip to content

fix: stop repeated review invariant rounds - #1579

Closed
mura0319 wants to merge 1 commit into
EveryInc:mainfrom
mura0319:fix/babysit-review-invariant-rounds
Closed

fix: stop repeated review invariant rounds#1579
mura0319 wants to merge 1 commit into
EveryInc:mainfrom
mura0319:fix/babysit-review-invariant-rounds

Conversation

@mura0319

Copy link
Copy Markdown

Summary

  • reviewerが返す安定したinvariant_keyをpush済みHEADごとに記録します
  • 同一invariantの3回目の修正前にneeds-humanへ切り替え、レビュー修正の非収束を止めます
  • approval:codexを追加し、現在HEADへのCodex承認を受けるまでready判定を保留します
  • HEADごとの@codex review依頼、承認待機中の無駄なwake防止、Codex投稿者の厳密判定を追加します

Closes #1575

User-visible behavior

  • 同じ根本原因のレビュー修正が2回続いた後、3回目を自動修正せず人へ方針判断を戻します
  • approval:codex指定時は、現在HEADに対するCodexの明示承認だけで完了します

Verification

  • bun test tests/ce-babysit-pr-contract.test.ts tests/ce-babysit-pr-snapshot.test.ts tests/skills/ce-babysit-pr-convergence-contract.test.ts: 218 passed
  • bun run test: 3,651 passed, 1 skipped, 0 failed
  • bun run release:validate: passed
  • python3 -m py_compile skills/ce-babysit-pr/scripts/pr-snapshot: passed
  • git diff --check: passed
  • fresh Codex skill eval: passed(read-only、own-eval only)

Limitations

  • approval:codexはinteractive mode専用で、mode:pipelineでは拒否します
  • Codex承認として受理する投稿者は既知のconnector loginに限定します

Security Disclosure

approval:codexの承認判定はfail-closedです。コメント本文から投稿者を推測せず、既知のCodex connector login、現在HEAD、依頼後のserver timestamp、明示承認をすべて検証します。

Agent Disclosure

Codex CLI · GPT-5が実装、テスト、レビュー対応の大部分を行いました。

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create an environment for this repo.

@mura0319 mura0319 closed this Aug 29, 2026
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.

ce-babysit-pr: persist review invariant rounds before non-convergence fixes

1 participant