[bug-005] Disable per-marker verifier by default; gate behind OLAVA_VERIFIER env

↗ view on GitHub · Nick Whitehouse · 2026-05-07 · b814d76a

Verifier (feat-007a) was awaiting all in-flight per-marker Olava calls
at end-of-turn before sending [DONE]. Per-call latency observed 12-17s
on olava-extract, and 3-of-4 calls came back empty in practice. This
added ~12s to time-to-[DONE] on every chat with citations.

Empirically the model emits a clean <CITATIONS> JSON block on its own
the vast majority of the time - citations land via the existing block-
parser path regardless. The verifier is mostly redundant work today.

Single env-gated flag: when OLAVA_VERIFIER is unset (or anything other
than "on"), the pre-extract is skipped, verifierDocId stays null, and
the existing fireVerifier early-return turns marker detection into a
no-op. verifierPromises stays empty so the end-of-turn await collapses.

All supporting code preserved (verifyCitation, marker detection,
streaming SSE emit) so re-enabling is just OLAVA_VERIFIER=on if/when
we observe the model regressing on the JSON tail.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Repository nwhitehouse/mike
Author Nick Whitehouse <nick.whitehouse@mccarthyfinch.com>
Authored
Parents 9c244fd2
Stats 1 file changed , +12 , -1
Part of Vision mode: PDF page images → Olava (feat-007a / 008 / 009 / 010 / bug-005)

Capture this commit into my fork

Download a Markdown prompt that tells Claude how to port this exact commit into your working tree. Run it via claude -p < capture-commit-b814d76a.md from inside the repo you want the change in.

⬇ Download capture-commit-b814d76a.md