Elnor Repo Reader

INITIAL_REVIEW_PROMPT.md

Active Working and Red Team/DOC23 Working/DOC23 Red Teaming/RT-0001_DOC23_ADD_B_REVIEW_STUDIO_ADJ_CARD_V4_REVIEW/01_PROMPTS/INITIAL_REVIEW_PROMPT.md

Generated 2026-06-09T01:23:58.539Z from commit dbaa25962edc11ab30e8d4ca1715f9ae5bf77331. Worktree: clean.

Open text page · Open raw txt · Open path URL

You are an expert red-team reviewer and systems architect, reviewing an **adjudication card** for the ELNOR project. Be rigorous, specific, and direct.

## Read these directly from GitHub — repo `wbrody/Elnor-Specs`, branch `main`
You have access to this repository (Gemini: via the configured mirror). Open and read the files at these exact paths — do not ask for attachments.

**Artifact under review (the card):**
`Active Working and Red Team/DOC23 Working/DOC23 Red Teaming/RT-0001_DOC23_ADD_B_REVIEW_STUDIO_ADJ_CARD_V4_REVIEW/03_ADJUDICATION/CARD_v04/RT-0001_Adjudication_Card_v04.md`

**Original spec the card adjudicates (Review Studio D1):**
`Active Working and Red Team/DOC23 Working/DOC23 Non Operative Proposals/DOC23 Add B Review Studio D1.md`

**Optional context — owner-boundary checks only; do NOT re-review these:**
- Outcome Evaluator / Revisor V3.3.1 — `Current Specs/DOC23/DOC23 Addenda B/DOC23_ADDB_OUTCOME_EVALUATOR_REVISOR_V3_3_1.md`
- Evaluation Common Contracts — `Current Specs/DOC23/DOC23 Addenda B/DOC23_EVALUATION_COMMON_CONTRACTS_V1_1_1.md`
- Feedback Delivery — `Current Specs/DOC23/DOC23 Addenda B/DOC23_ADDB_SUBSYS_FEEDBACK_DELIVERY_V1_0_1.md`
- Addenda B Core — `Current Specs/DOC23/DOC23 Addenda B/DOC23_ADDENDA_B_CORE_R0_7_1.md`
- DOC20 R4.3 — `Current Specs/DOC20/DOC20_R4_3.md`

## Context
ELNOR is a local-first, single-user AI orchestration platform (OpenClaw / Node.js), built for maximum quality with **no shipping-deadline pressure and no feature phasing** (build-sequencing is fine; deferring capability to an "MVP" is not). **Review Studio** (DOC23 Addenda B) is a human review/edit/decision surface that sits between agent work and downstream consumers.

The artifact under review is **not the spec** — it is the **self-contained adjudication card** that rules on the red-team review of Review Studio D1. It consolidates two review rounds into **67 adjudicated rows (RS-000–RS-066)**, each with a disposition and a paste-ready fix, plus a corrected output/port model (§2), a contract/patch library (§3), a master decision index (§4), per-item rows (§5), lints/fixtures (§6), end-to-end traces (§7), reviewer coverage (§8), value tiers (§9), cross-artifact obligations (§10), and the verdict (§11–§12). Current verdict: **READY_AFTER_PATCH**. Read D1 for grounding, but **review the adjudication — do not re-review D1 from scratch.**

## Your job
Red-team the **adjudication itself** — the decisions and fixes in the card. A lot of the Round-2 material is new, so weight your attention toward the items below. Hunt hard for:

1. **Wrong adjudications.** Any row whose disposition or paste-ready fix is wrong, weak, or mis-scoped. State the corrected disposition + fix.
2. **The Round-2 net-new rows (RS-056–RS-066).** Are they sound, correctly scoped, and **traceable to a real finding or settled decision** (no invented "phantom" contracts)? Are the TypeScript contracts internally correct and consistent with the rest of the card? Flag any that are over-built, redundant, or under-specified. Look hard at: RS-056 (human-edit revalidation: shared `RevisionOperationKind` + §11.21 cascade + Evaluator-owned receipt), RS-064 (`ReviewAttestation`), RS-065 (`ReviewIterationBudget`), RS-066 (`ReviewExternalSurfaceBinding` vs RS-057 `ReviewInputFidelity` — is the binding/fidelity split clean or duplicative?).
3. **The five open-item resolutions (§1.1: RS-002, RS-017, RS-021→RS-056, RS-026, RS-027).** These are stated **compactly inline** rather than as full contracts. For each: is the resolution **correct and sufficient to implement**, or does it need a full contract first? Be explicit about which (if any) are under-specified — especially RS-002 (`ReviewableSubject` + `ReviewableArtifactKind` per-kind anchor model for non-document artifacts), RS-026 (DOCX version-spine reconciliation), RS-027 (`NativeRevisionSafetyGate`).
4. **The RS-062 REJECT (no auto-routing).** The architect's position is: *the gate is the gate* — no confidence-triggered auto-routing, **not even behind a toggle**. Argue the strongest case either way, then give your recommendation.
5. **Internal consistency (high priority — this card was merged from two rounds).** Check for contradictions across **§1.1, the §4 master index, the §5 rows, §9 value tiers, §10 cross-artifact, and §11 patch list**: a row RESOLVED in one place but OPEN in another; a row missing from the index; a fixture-name collision; a dangling `§`-anchor; a contract referenced but never defined.
6. **Coverage / misses.** Anything either round should have caught and didn't — across **both** the document-review / native-Word path **and** the structured-output (non-document artifact) path.
7. **Owner-boundary & cross-artifact correctness.** Do the §10 obligations land on the right owners (DOC20 / DOC27 / DOC12 / V3.3.1 / EC Core / DOC85)? Anything routed to the wrong spec, or missing an OP-A obligation? Note: privilege is **egress-only** in this system — it must not gate the internal review surface or cross-task use; flag any violation.
8. **Subtractive pass (required).** Identify **good-but-redundant** material that could be cut without losing capability — duplicate contracts, rows that restate each other, fixtures with no distinct assertion, over-engineered options. Be explicit about what to delete and why it's safe.

## Ground rules
- **No phantom features.** Every contract / field / lint / fixture you propose must trace to a real finding, a settled decision, or a concrete defect in the card. If you can't trace it, label it `OPEN_FOR_ARCHITECT_REVIEW`.
- **Cite precisely** — section (`§`), row id (`RS-0NN`), and line where possible.
- **Paste-ready over prose** — give the TypeScript / enum / lint / fixture text, not a description.
- **Type every finding** as one of: `BUG` · `GAP` · `INCONSISTENCY` · `SUGGESTION` · `BETTER_IDEA` · `CONFIRMED` · `ARCHITECT_STOP`.
- **Be direct.** No hedging, no praise padding. If the card is right, say `CONFIRMED` and move on. Disagreement with the architect's stated positions is welcome if you can support it.

## Output format
One Markdown document with:
1. **Verdict token** — one of `RATIFY` · `MINOR_FIXES_THEN_RATIFY` · `DESIGN_REVISION_NEEDED` · `ARCHITECT_STOP` — plus a 2–3 sentence rationale.
2. **Findings, value-tiered:** Critical / blocking · Substantive · Minor / polish · Considered and declined (with reasons). Each finding: your own `F#`, a type token, a `§ + RS-id + line` citation, the problem, and the paste-ready fix.
3. **Subtractive pass** — explicit list of good-but-redundant elements to cut.
4. **Coverage note** — anything missed by both rounds.

## Saving your output
If you can write to the repo (or the mirror), save your review to this exact path:
`Active Working and Red Team/DOC23 Working/DOC23 Red Teaming/RT-0001_DOC23_ADD_B_REVIEW_STUDIO_ADJ_CARD_V4_REVIEW/03_ADJUDICATION/CARD_v04/CARD_REVIEWS/RT-0001_<YOUR_MODEL>_Card_v04_Review.md`
(e.g. `RT-0001_ChatGPT_Card_v04_Review.md`.) If you cannot write files, do **not** claim you saved it — output the full review in chat and start it with this header so it can be filed automatically:

<!-- RT_SAVE
rt: RT-0001
model: [INSERT MODEL NAME]
artifact: Review Studio Adjudication Card v04 — card review
category: card_review
card_version: v04
include_for_adjudication: yes
-->

Replace [INSERT MODEL NAME] with your own name (Claude, ChatGPT, Grok, Gemini, or Codex).