repo-harness-cross-review

# 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 - Inside Claude Code -> direct Codex: `references/codex-mode.md`. - Inside Codex -> official `codex@openai-codex` plugin app-server: `references/codex-plugin-mode.md`. - Explicit Claude acceptance review -> `references/claude-mode.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 and the plugin app-server both use 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 ...