← Back to Articles & Artefacts
artefactswest

What I expect the Opus review to say differently (written 2026-10-11 before reading it)

IAIP Research
ep76-proposal-scoring-with-jev-261010-261010-0d54154e-d8cd-4c8e-8eb5-2368d3a1032b

What I expect the Opus review to say differently (written 2026-10-11 before reading it)#

Commits reviewed: 71387fae..4ace34b8 on jgwill/Miadi main (#769). Reviewer: Opus 5.5, read-only.

Predicted findings, in the order I think it will rank them:

  1. packages/tracing-langfuse/src/index.ts listTraces: the wanted line is a no-op (tags are never applied), and limit * 5 is a heuristic with no basis; a trace with more than 100 observations is undercounted by getTrace.
  2. app/chronicle/components/EpisodeRoom.tsx: the Sessions card frame renders for every reader while its body returns null for anyone who may not write, so readers see an empty card.
  3. packages/episodic-memory-schema/src/episode-rispec.ts HOST_PATH_REPOSITORIES: host-specific paths (/a/src/Miadi-18, /etc/claude-code) inside a package that calls itself pure and host-neutral.
  4. app/chronicle/lib/artefactRef.ts: the wheel node's metadata is read for any viewer; it must be shown that editor_url / ui_url never reach a reader through artefactInfoOf.
  5. observeEpisodeSessions: team fallback and name handling when name is a string vs an object; order-independence is tested only for one session.
  6. rise-dsl.ts: riseLocalPath is only used server-side; a reviewer will ask whether /api/chronicle/open can be made to redirect anywhere but github.com (it cannot: host is fixed, owner/repo/path validated).
  7. Something I did not think of about the references.tsx a override: the rise branch rebuilds an ArtefactAddress by hand instead of a shared helper, so reference-ui and the room can drift.

I expect it NOT to find a defect in the manifest reach rule (editor_url on the node only), nor in the four-step pattern itself.