5 Commits

  • chore(core) rm AskForApproval::OnFailure (#28418)
    ## Summary
    Deletes the OnFailure variant of the `AskForApproval` enum. This option
    has been deprecated since #11631.
    
    ## Testing
    - [x] Tests pass
  • [codex] Remove child AGENTS.md prompt experiment (#28993)
    ## Why
    
    `child_agents_md` is a disabled, under-development experiment that adds
    a second model-visible explanation of hierarchical `AGENTS.md` behavior.
    Keeping it leaves unused prompt, configuration, documentation, and test
    surface.
    
    ## What changed
    
    - remove the `ChildAgentsMd` feature and `child_agents_md` config schema
    entry
    - remove the hierarchical prompt asset, export, and instruction
    injection
    - remove feature-specific tests and documentation
    - keep the generic unstable-feature warning coverage using
    `apply_patch_streaming_events`
    
    Normal project `AGENTS.md` discovery and composition are unchanged.
    
    ## Testing
    
    - `just test -p codex-features`
    - `just test -p codex-prompts`
    - `just test -p codex-core agents_md`
    - `just test -p codex-core unstable_features_warning`
  • Make auto-review on-request prompt more proactive (#26496)
    ## Why
    
    `on-request` approval policy text is currently tuned for user-reviewed
    approvals. For auto-reviewed productivity runs, likely sandbox blocks
    should be escalated earlier so commands that need remote services,
    authentication, or other out-of-sandbox access do not first fail or hang
    inside the sandbox.
    
    ## What changed
    
    - Adds a separate `on_request_auto_review.md` permissions prompt
    selected for `AskForApproval::OnRequest` with
    `ApprovalsReviewer::AutoReview`.
    - Keeps the normal user-reviewed `on-request` wording unchanged.
    - Makes the `When to request escalation` bullets more explicit about
    likely sandbox blocks, network access, remote
    auth/cluster/cloud/database access, out-of-sandbox environment access,
    git operations that may write lock files, and short-timeout reruns after
    likely sandbox-blocked attempts.
    - Omits approved command prefix and `prefix_rule` guidance for the
    auto-review on-request prompt.
    - Adds prompt tests covering the auto-review path, normal on-request
    wording, and inline permission request behavior.
  • [codex] Consolidate shared prompts in codex-prompts (#25151)
    ## 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