From 47d000052b0f6ab352147f903a3c91e0e78ac349 Mon Sep 17 00:00:00 2001 From: Rob Pugh Date: Mon, 11 May 2026 09:11:52 -0400 Subject: [PATCH 1/2] fix: widen /codex:review wait threshold to ~5 files / 200 lines --- plugins/codex/commands/review.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plugins/codex/commands/review.md b/plugins/codex/commands/review.md index fb70a487..f0782f1e 100644 --- a/plugins/codex/commands/review.md +++ b/plugins/codex/commands/review.md @@ -24,7 +24,7 @@ Execution mode rules: - For base-branch review, use `git diff --shortstat ...HEAD`. - Treat untracked files or directories as reviewable work even when `git diff --shortstat` is empty. - Only conclude there is nothing to review when the relevant working-tree status is empty or the explicit branch diff is empty. - - Recommend waiting only when the review is clearly tiny, roughly 1-2 files total and no sign of a broader directory-sized change. + - Recommend waiting when the review is small, roughly up to 5 files or 200 lines of diff, with no sign of a broader directory-sized change. Codex finishes within Claude's prompt cache window (5 minutes) for diffs of this size, so wait in the foreground and the result feeds the next step. - In every other case, including unclear size, recommend background. - When in doubt, run the review instead of declaring that there is nothing to review. - Then use `AskUserQuestion` exactly once with two options, putting the recommended option first and suffixing its label with `(Recommended)`: From 8b358db1c1a0affa91768e32449802a47e870211 Mon Sep 17 00:00:00 2001 From: Rob Pugh <52668747+robertbpugh@users.noreply.github.com> Date: Mon, 11 May 2026 09:57:30 -0400 Subject: [PATCH 2/2] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- plugins/codex/commands/review.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plugins/codex/commands/review.md b/plugins/codex/commands/review.md index f0782f1e..cb22e319 100644 --- a/plugins/codex/commands/review.md +++ b/plugins/codex/commands/review.md @@ -24,7 +24,7 @@ Execution mode rules: - For base-branch review, use `git diff --shortstat ...HEAD`. - Treat untracked files or directories as reviewable work even when `git diff --shortstat` is empty. - Only conclude there is nothing to review when the relevant working-tree status is empty or the explicit branch diff is empty. - - Recommend waiting when the review is small, roughly up to 5 files or 200 lines of diff, with no sign of a broader directory-sized change. Codex finishes within Claude's prompt cache window (5 minutes) for diffs of this size, so wait in the foreground and the result feeds the next step. + - Recommend waiting when the review is small, roughly up to 5 files or about 200 insertions+deletions as reported by `git diff --shortstat`, with no sign of a broader directory-sized change. Codex finishes within Claude's prompt cache window (5 minutes) for changes of this size, so wait in the foreground and the result feeds the next step. - In every other case, including unclear size, recommend background. - When in doubt, run the review instead of declaring that there is nothing to review. - Then use `AskUserQuestion` exactly once with two options, putting the recommended option first and suffixing its label with `(Recommended)`: