requesting-review: split the review job out of codex-companion, runtime fail-closed, engine script retired by SSFSKIM · Pull Request #55 · SSFSKIM/doperpowers

feat(reviewing-prs): route big diffs to the codex-companion review panel inside the engine

SSFSKIM

committed

5eb0a4b

chore: bump plugin version to 7.46.0 (review-engine panel routing)

SSFSKIM

committed

5a96d36

fix(reviewing-prs): keep the IN THE BACKGROUND phrase contract contiguous

SSFSKIM

committed

def7635

fix(reviewing-prs): review round — forward sandbox diagnostics through workflow leaves + doc/test hardening

SSFSKIM

committed

226b10b

fix(reviewing-prs): panel dogfood round — fail-closed renderer, throw-path sandbox forwarding, honest lane rendering

SSFSKIM

committed

87b71ca

docs: ExecPlan — requesting-review skill split, runtime fail-closed, engine retirement

SSFSKIM

committed

b656b9d

feat: requesting-review skill split + runtime fail-closed sandbox guard

SSFSKIM

committed

f461d0c

refactor: repoint review consumers at doperpowers:requesting-review

SSFSKIM

committed

214a496

docs: ExecPlan progress through milestone 5

SSFSKIM

committed

d58d85a

fix(reviewing-prs): restore CODEX_CODE_MODE_HOST_PATH fallback in the engine preamble

SSFSKIM

committed

82793c5

fix(reviewing-prs): RE-REVIEW names the redirection the engine commands actually use

SSFSKIM

committed

9c7a11b

fix(reviewing-prs): per-round engine home is removed, not leaked

SSFSKIM

committed

0ad5453

docs(requesting-review): scope target selection to the verbs; state the panel's base-only target

SSFSKIM

committed

548828a

docs(requesting-review): state the CODEX_SANDBOX exception to the fail-closed guarantee

SSFSKIM

committed

a831ba9

docs(requesting-review): carve setup out of the endpoint env contract

SSFSKIM

committed

034cb0a

docs(worker-host): verification gate step 2 points at the live review invocation

SSFSKIM

committed

4c7f190

fix(codex-companion): with-effort scans the unterminated stderr tail for the sandbox marker

SSFSKIM

committed

689fed9

fix(reviewing-prs): engine home lives inside <review-tmp>, so cleanup is executable

SSFSKIM

committed

4862ed8

fix(reviewing-prs): the rendered base ref is checked before it becomes a shell word

SSFSKIM

committed

19b416b

fix(reviewing-prs): a twice-interrupted panel goes straight to the outage path

SSFSKIM

committed

a6144f8

fix(review-bench): restore the isolated codex environment on the bench's codex path

SSFSKIM

committed

69aae20

test(implementing): pin the plan-review mechanism at its new owner

SSFSKIM

committed

68e59e7

fix(reviewing-prs): a park preserves wave boards, never the engine home

SSFSKIM

committed

675bd26

fix(reviewing-prs): state the retry budget's two terminal exclusions

SSFSKIM

committed

e8e12b7

docs(requesting-review): describe what a lost lane actually does to the verdict

SSFSKIM

committed

8b81b16

fix(engine): the throw path fails closed on sandbox markers too

SSFSKIM

committed

1b06ac6

docs(reviewing-prs): an interrupted attempt's confirmed findings survive the retry

SSFSKIM

committed

dad5343

docs(requesting-review): scope the lost-lane claim to lens lanes

SSFSKIM

committed

b4b75aa

docs(operation-manual): name the two failures that skip the retry budget

SSFSKIM

committed

27a585f

Revert "fix(engine): the throw path fails closed on sandbox markers too"

SSFSKIM

committed

79a69da

Merge remote-tracking branch 'origin/main' into worktree-review-engine-panel-routing

SSFSKIM

committed

f1dfba3

chore: bump version to 7.47.0

SSFSKIM

committed

2d9fdbc

docs: close the requesting-review ExecPlan — dogfood evidence, outcomes, follow-ups

SSFSKIM

committed

9fa7d11

Navigation Menu

requesting-review: split the review job out of codex-companion, runtime fail-closed, engine script retired - #55

requesting-review: split the review job out of codex-companion, runtime fail-closed, engine script retired#55
SSFSKIM wants to merge 33 commits into
mainfrom
worktree-review-engine-panel-routing

Commits on Aug 10, 2026

feat(reviewing-prs): route big diffs to the codex-companion review panel inside the engine

chore: bump plugin version to 7.46.0 (review-engine panel routing)

fix(reviewing-prs): keep the IN THE BACKGROUND phrase contract contiguous

fix(reviewing-prs): review round — forward sandbox diagnostics through workflow leaves + doc/test hardening

fix(reviewing-prs): panel dogfood round — fail-closed renderer, throw-path sandbox forwarding, honest lane rendering

docs: ExecPlan — requesting-review skill split, runtime fail-closed, engine retirement

feat: requesting-review skill split + runtime fail-closed sandbox guard

refactor: repoint review consumers at doperpowers:requesting-review

docs: ExecPlan progress through milestone 5

fix(reviewing-prs): restore CODEX_CODE_MODE_HOST_PATH fallback in the engine preamble

fix(reviewing-prs): RE-REVIEW names the redirection the engine commands actually use

fix(reviewing-prs): per-round engine home is removed, not leaked

docs(requesting-review): scope target selection to the verbs; state the panel's base-only target

docs(requesting-review): state the CODEX_SANDBOX exception to the fail-closed guarantee

docs(requesting-review): carve setup out of the endpoint env contract

docs(worker-host): verification gate step 2 points at the live review invocation

fix(codex-companion): with-effort scans the unterminated stderr tail for the sandbox marker

fix(reviewing-prs): engine home lives inside <review-tmp>, so cleanup is executable

fix(reviewing-prs): the rendered base ref is checked before it becomes a shell word

fix(reviewing-prs): a twice-interrupted panel goes straight to the outage path

fix(review-bench): restore the isolated codex environment on the bench's codex path

test(implementing): pin the plan-review mechanism at its new owner

fix(reviewing-prs): a park preserves wave boards, never the engine home

fix(reviewing-prs): state the retry budget's two terminal exclusions

docs(requesting-review): describe what a lost lane actually does to the verdict

fix(engine): the throw path fails closed on sandbox markers too

docs(reviewing-prs): an interrupted attempt's confirmed findings survive the retry

docs(requesting-review): scope the lost-lane claim to lens lanes

docs(operation-manual): name the two failures that skip the retry budget

Revert "fix(engine): the throw path fails closed on sandbox markers too"

Merge remote-tracking branch 'origin/main' into worktree-review-engine-panel-routing

chore: bump version to 7.47.0

docs: close the requesting-review ExecPlan — dogfood evidence, outcomes, follow-ups