Skip to content

[codex] Consolidate shared prompts in codex-prompts#25151

Merged
anp-oai merged 1 commit into
mainfrom
codex/core-split-review-prompts
Jun 1, 2026
Merged

[codex] Consolidate shared prompts in codex-prompts#25151
anp-oai merged 1 commit into
mainfrom
codex/core-split-review-prompts

Conversation

@anp-oai
Copy link
Copy Markdown
Collaborator

@anp-oai anp-oai commented May 29, 2026

Why

codex_core is consistently a bottleneck for incremental builds during iteration. The simplest fix is to make the crate smaller.

Summary

codex-core owns several reusable prompt renderers and static prompt assets, which makes the crate harder to split apart.

Rename codex-review-prompts to codex-prompts and move shared review, goal, permissions, compaction, realtime, hierarchical AGENTS.md, and apply_patch prompts into it. Move prompt-only tests and update consumers and CODEOWNERS.

Validation

  • just test -p codex-prompts -p codex-apply-patch
  • just test -p codex-core prompt_caching
  • Bazel builds for the affected crates

@anp-oai anp-oai force-pushed the codex/core-split-review-prompts branch 2 times, most recently from c8215cd to 4064fc1 Compare May 29, 2026 20:58
@anp-oai anp-oai marked this pull request as ready for review May 29, 2026 21:38
@anp-oai anp-oai requested a review from a team as a code owner May 29, 2026 21:38
@anp-oai anp-oai force-pushed the codex/core-split-review-prompts branch from 4064fc1 to 388c807 Compare May 29, 2026 22:54
@anp-oai anp-oai changed the title [codex] Extract review prompts from codex-core [codex] Extract prompts from codex-core May 29, 2026
@anp-oai anp-oai force-pushed the codex/core-split-review-prompts branch from 388c807 to 2ce38ba Compare May 29, 2026 22:59
Comment thread codex-rs/core/BUILD.bazel Outdated
Comment thread codex-rs/prompts/src/lib.rs Outdated
@anp-oai anp-oai force-pushed the codex/core-split-review-prompts branch 3 times, most recently from c29c949 to c648d4f Compare June 1, 2026 17:32
@anp-oai anp-oai requested a review from aibrahim-oai June 1, 2026 17:45
@anp-oai anp-oai changed the title [codex] Extract prompts from codex-core [codex] Consolidate shared prompts in codex-prompts Jun 1, 2026
Copy link
Copy Markdown
Collaborator

@bolinfest bolinfest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the cleanup!

Comment thread codex-rs/prompts/src/goals.rs
@anp-oai anp-oai force-pushed the codex/core-split-review-prompts branch from c648d4f to e99b34a Compare June 1, 2026 18:26
@anp-oai anp-oai enabled auto-merge (squash) June 1, 2026 18:37
@anp-oai anp-oai merged commit ba2b67f into main Jun 1, 2026
46 checks passed
@anp-oai anp-oai deleted the codex/core-split-review-prompts branch June 1, 2026 18:45
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants