Skip to content

feat: cross-model outside voice in plan reviews (v0.9.9.1)#326

Merged
garrytan merged 8 commits intomainfrom
garrytan/codex-in-ceo-review
Mar 23, 2026
Merged

feat: cross-model outside voice in plan reviews (v0.9.9.1)#326
garrytan merged 8 commits intomainfrom
garrytan/codex-in-ceo-review

Conversation

@garrytan
Copy link
Copy Markdown
Owner

Summary

  • Plan reviews now offer an independent second opinion. After all review sections complete in /plan-ceo-review or /plan-eng-review, you can get a "brutally honest outside voice" from a different AI model (Codex CLI, or a fresh Claude subagent if Codex isn't installed). Optional, recommended, never blocks shipping.
  • Cross-model tension detection. When the outside voice disagrees with the review findings, disagreements are surfaced automatically and offered as TODOs.
  • Outside Voice in Review Readiness Dashboard. /ship now shows whether an outside voice ran on the plan.
  • /plan-eng-review Codex integration upgraded. Replaced hardcoded Step 0.5 with the {{CODEX_PLAN_REVIEW}} resolver — adds Claude subagent fallback, review logging, dashboard visibility, xhigh reasoning effort.
  • Codex adversarial review caught a positioning bug (placeholder before review sections instead of after) — fixed.

Pre-Landing Review

No issues found.

Adversarial Review

Codex adversarial review ran (large tier). 6 findings:

Test plan

  • bun test passes (0 failures)
  • Generated SKILL.md files fresh for both Claude and Codex hosts
  • Outside Voice section present in plan-ceo-review/SKILL.md and plan-eng-review/SKILL.md
  • Section stripped from .agents/skills/gstack-plan-ceo-review/SKILL.md (codex host guard)
  • Tests re-run after adversarial fix — still passing

🤖 Generated with Claude Code

garrytan and others added 6 commits March 22, 2026 09:23
New resolver offers an optional Codex (or Claude subagent fallback) "outside
voice" after plan review sections complete. Includes cross-model tension
detection with auto-TODO proposals, review log persistence, and an Outside
Voice row in the Review Readiness Dashboard.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
CEO review: insert after Section 11 + add Outside Voice summary row.
Eng review: replace hardcoded Step 0.5 with resolver (adds fallback,
logging, dashboard, xhigh reasoning, cross-model tension tracking).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
ARCHITECTURE.md: added {{CODEX_PLAN_REVIEW}} to placeholder table.
CHANGELOG.md: added v0.9.9.1 entry for outside voice feature.
VERSION: bumped to 0.9.9.1.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Codex adversarial review caught that the placeholder was positioned
before the 4 review sections, so the "After all review sections are
complete" instruction could confuse the model. Moved it after Section
4's STOP directive where it belongs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…-review

# Conflicts:
#	.agents/skills/gstack-plan-ceo-review/SKILL.md
#	.agents/skills/gstack-plan-design-review/SKILL.md
#	.agents/skills/gstack-plan-eng-review/SKILL.md
#	.agents/skills/gstack-ship/SKILL.md
#	CHANGELOG.md
#	VERSION
…-review

# Conflicts:
#	CHANGELOG.md
#	VERSION
@garrytan garrytan merged commit 7fbf68b into main Mar 23, 2026
rapidstartup pushed a commit to rapidstartup/gstack that referenced this pull request Mar 29, 2026
)

* feat: add generateCodexPlanReview() resolver for cross-model plan review

New resolver offers an optional Codex (or Claude subagent fallback) "outside
voice" after plan review sections complete. Includes cross-model tension
detection with auto-TODO proposals, review log persistence, and an Outside
Voice row in the Review Readiness Dashboard.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* feat: integrate {{CODEX_PLAN_REVIEW}} into CEO and eng review templates

CEO review: insert after Section 11 + add Outside Voice summary row.
Eng review: replace hardcoded Step 0.5 with resolver (adds fallback,
logging, dashboard, xhigh reasoning, cross-model tension tracking).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: regenerate SKILL.md files from updated templates

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* docs: update project documentation for v0.9.9.1

ARCHITECTURE.md: added {{CODEX_PLAN_REVIEW}} to placeholder table.
CHANGELOG.md: added v0.9.9.1 entry for outside voice feature.
VERSION: bumped to 0.9.9.1.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: move {{CODEX_PLAN_REVIEW}} after review sections in eng review

Codex adversarial review caught that the placeholder was positioned
before the 4 review sections, so the "After all review sections are
complete" instruction could confuse the model. Moved it after Section
4's STOP directive where it belongs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: regenerate eng review SKILL.md files

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.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