repo-harness-cross-review · git:20261002.cc1fc8e · 2026-10-02 · sha256 792eb1e9c70cccdc
repo-harness-cross-review git:20261002.cc1fc8eA
Immutable. This exact content never changes and is served at /api/v1/blob/792eb1e9c70cccdc.
--- name: repo-harness-cross-review description: Independent outside review of the current review scope (branch diff plus staged, unstaged, untracked changes). Uses the explicit Codex advisory provider; generic acceptance uses a persistent fleet deep-reasoner task-agent in Herdr. Use before merging, after a tricky change, or for a debug second opinion. when_to_use: "cross review, second opinion, outside voice, codex review, 让 codex 审, 找外部意见, 二审" --- # repo-harness-cross-review Canonical rule owner for independent outside review. Scope capture, provider invocation, timeout, structured-output validation, and error classification live in code (`src/core/review`, `src/effects/review`, `src/cli/commands/cross-review.ts`); this package owns only when to invoke, how to interpret findings, and the boundaries below. ## Mode Selection - Explicit Codex advisory review: `references/codex-mode.md`. - Acceptance review through task-agent -> `references/generic-review.md`. - An explicit provider request wins over the host default. ## When to use - Before merging an important diff (last gate). - After writing a spec/tests -- find ambiguity and weak assertions. - A hard bug whose root cause is unclear (independent diagnosis). ## Interpreting findings - Present the transcript verbatim -- never summarize or soften it. - Any `[P1]` finding -> **FAIL** (do not merge until addressed). Only `[P2]` or none -> **PASS**. - Agreement raises confidence; divergence is where to dig. A recommendation, not a decision -- you decide. ## Boundaries - Read-only: direct Codex uses a read-only sandbox. - Bounded: 2 provider attempts, then `SKIPPED` -- advisory, non-blocking (exit 0). Do not re-run it or narrow the diff to retry. `degraded_scope` and `stale_scope` block (exit 1). Never a synthesized pass. - Never produces or verifies a `merge-gate` receipt; that judge is separate.