jev_review / jev_gate: optional per-file review mode

[Rex-Gao]

Implements #42.

What

Both jev_review and jev_gate now take files: [{ path, diff }] instead of diff (exactly one of the two, enforced with a fixed-string error). The tool still writes the rubric questions itself — they are asked once per file, all in the same request, each scoped with a Judge only files[i] ("path"), not the change as a whole sentence, so the vague-question failure mode from the TypeSafe docs stays fixed.

The action composes in code, per the issue's proposal:

• whole change is auto only when every file is auto (worstAction);
• composite is the file mean, safe_to_apply the file minimum;
• limiting names the file and rubrics that bound the decision (building on the Expose review score distributions and the rubric limiting automatic acceptance #20/Preserve review score distributions; reason codes for every action path #26 limiting-rubrics vocabulary);
• the full per-file review ships in a files array — nothing is averaged into invisibility, a weak file stays visible as itself.

Per-file projection reuses projectReviewHalf unchanged (slice the file_i_* answers back to bare rubric keys), so validation, thresholds, reason codes, and limiting rubrics behave exactly as in whole-change mode. jev_gate's claim questions name the file diffs instead of the diff in per-file mode; claims and verification are untouched.

Bounds mirror the gate's evidence discipline: up to 16 files, 200,000 characters in aggregate rejected before any model call; per-file diffs truncate at the same 50,000-character cap as diff, and truncated input still never returns auto.

Testing

npm run typecheck, npm run build, npm test: 245 tests (all 239 pre-existing pass unchanged — whole-change behavior is untouched). New coverage: one request with per-file keys only and the scoped instructions; auto composition; the weak file naming limiting with file and rubric; exactly-one-of diff/files; the aggregate budget rejecting without a request; per-file truncation demoting auto; and the gate composing per-file review with claims in one call.

Docs

README: bullet in jev_review (with the mode semantics) and one in jev_gate.

[jkudish]

Thank you for this. Merged with three amendments from review:

• questions scope by index only, so file paths can't carry directives into the rubric
• one combined budget (request + tests + all diffs), so a single call stays inside state limits
• truncation is per-file: an intact file is never demoted for a truncated sibling

Thanks again!

[jkudish]

Merged onto main (rebased; the amendments above are part of the merged commits).