Skip to content

Add Codex Stop-hook plan review#577

Open
ivanov17andrey wants to merge 1 commit intobacknotprop:mainfrom
ivanov17andrey:feat/codex-plan
Open

Add Codex Stop-hook plan review#577
ivanov17andrey wants to merge 1 commit intobacknotprop:mainfrom
ivanov17andrey:feat/codex-plan

Conversation

@ivanov17andrey
Copy link
Copy Markdown

@ivanov17andrey ivanov17andrey commented Apr 17, 2026

Summary

  • add Codex Stop hook plan review support that extracts the latest plan from rollout transcripts, opens Plannotator for fresh plans, and returns continuation feedback on denial
  • cover the new Codex parsing and Stop flow with server tests and update Codex-facing installation and plan-review docs
  • add a reproducible local Codex E2E harness plus checked-in screenshots showing the initial review, revised review, versions sidebar, diff view, and archive sidebar
  • ignore the local .playwright-cli/ cache so Playwright-driven validation does not dirty the repo

Test plan

  • /bin/bun test apps/hook/server/*.test.ts
  • /bin/bun run --cwd apps/review build
  • /bin/bun run --cwd apps/hook build
  • PLANNOTATOR_BROWSER=/usr/bin/true ./tests/manual/local/test-codex-plan-review-e2e.sh --keep --skip-build --root-dir /tmp/plannotator-codex-stop-e2e-rerun3 with Playwright-driven review, denial, revision, versions, diff, archive, and approval flow

Screenshots

Initial plan review

Initial plan review

Feedback before denial

Feedback comment

Revised plan review

Revised plan review

Versions sidebar after denial

Versions sidebar

Diff view against v1

Diff view

Archive sidebar

Archive sidebar

@backnotprop
Copy link
Copy Markdown
Owner

Big one!

Give me some time to review but will priortize

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.

2 participants