KB-7898

Codex Re-Review — Macro-9B2 Validator / Contract Remediation

11 min read Revision 1
codexre-reviewmacro9b2validatorcontractfail-closedpass-with-caveatsread-only2026-06-20

Codex Re-Review — Macro-9B2 Validator / Contract Remediation

STATUS: PASS_WITH_CAVEATS

Executive Summary

All seven findings from the prior Codex HOLD are closed at the engineering validator/contract layer.

The validator now rejects missing or blank channels, missing/blank/whitespace actors, trailing newline/carriage-return/tab/space/NUL targets, and malformed run IDs. It uses re.fullmatch rather than match(...$). The simulated real-run gate requires an actual boolean and opens only for exactly True; false-like strings, integer 1, and None reject. Guard 3 now consumes explicit before/after evidence and returns PASS/FAIL/UNKNOWN; both real-run modes require PASS before emitting any write-intent. Guard 4 no longer calls Guard 1 and instead shares a stateless target-validation helper.

The v2 evidence contains 52 matrix rows plus 12 real-run simulation rows, totaling 64, with eight meta-assertions. It reports 64/64 PASS, no fail-open, no assertion mismatch, and no no-write violation. The old 37/37 evidence has a prominent SUPERSEDED banner, while the admission record, artifact index, contracts, matrix, main report, and remediation report identify v2 as current bounded engineering evidence.

This clears the prior HOLD for the Macro-9B/Macro-9B1 package at the engineering layer only. The DOT remains unregistered and unwired; the runtime gate remains closed; no Owner authority exists; real-run and Macro-9A remain prohibited.

Files Reviewed

File Revision content_length Verdict
Macro-9B2 remediation report 1 12483 PASS_WITH_CAVEATS
Reference validator rev2 2 14415 PASS_WITH_CAVEATS
Validator evidence v2 1 10292 PASS_WITH_CAVEATS
Primary DOT contract rev2 2 12095 PASS_WITH_CAVEATS
Four guard contracts rev2 2 11333 PASS_WITH_CAVEATS
Bad-input matrix v2 2 8971 PASS_WITH_CAVEATS
Birth/admission record 9 19500 PASS_WITH_CAVEATS
Macro-9B artifact index 4 4739 PASS_WITH_CAVEATS
Macro-9B main report 6 21001 PASS_WITH_CAVEATS
Historical validator evidence rev1, supporting supersession check 2 4881 SUPERSEDED — not current proof

Review basis also included Operating Rules SSOT v7.58, Constitution v4.6.3, the prior Codex HOLD report revision 1/content_length 15099, and the mission's preserved authority/runtime boundaries.

Codex HOLD Closure Audit

Prior issue Verdict Evidence
Missing channel accepted CLOSED Router emits MISSING_CHANNEL for absent/blank channel; B38–B39 pass. Forbidden channels directus_generic, psql, manual_sql, docker_exec_psql, and host_exec still reject.
Missing/blank actor accepted CLOSED Router emits MISSING_ACTOR; B40–B42 cover missing, blank, and whitespace actor.
match(...$) accepted trailing newline CLOSED Both patterns use fullmatch; explicit whitespace/control-character guard precedes matching. B14–B19 and B23–B25 reject newline, CR, tab, spaces, and NUL. Static source check found no legacy allowlist/run-id .match( calls.
Truthy non-boolean gate opened simulation CLOSED Gate type checked with isinstance(..., bool), then is not True; S06–S09 reject string false/true, integer 1, and None with INVALID_GATE_TYPE; boolean False rejects REAL_RUN_GATE_CLOSED.
Guard 3 was plan-only/not enforced CLOSED production_untouched_verify returns PASS/FAIL/UNKNOWN over explicit evidence. Real-run router calls it and emits PROD_UNTOUCHED_FAIL or PROD_UNTOUCHED_UNKNOWN unless PASS.
Guard 4 directly called Guard 1 CLOSED Guard 4 calls shared pure _validate_target; it does not call allowlist_guard.
37/37 overclaimed zero fail-open CLOSED V2 provides 64 bounded cases + 8 structural assertions and explicitly disclaims universal/runtime proof. Rev1 evidence is marked SUPERSEDED.

Validator V2 Fail-Closed Audit

Check Verdict Evidence
Missing/blank channel PASS B38–B39; MISSING_CHANNEL.
Forbidden channels PASS B32–B36; FORBIDDEN_MANUAL_CHANNEL.
Missing/blank/whitespace actor PASS B40–B42; MISSING_ACTOR.
Strict target full-string matching PASS ALLOWLIST_RE.fullmatch(raw); malformed/control cases reject.
Strict run-id full-string matching PASS RUN_ID_RE.fullmatch(str(run_id)); control cases reject.
Protected schemas PASS B01–B06 reject.
SQL injection/non-allowlist PASS B07–B13 reject.
Strict gate type PASS S05–S09; no truthiness opening.
Guard 3 explicit evidence PASS_WITH_CAVEAT Missing/non-dict/incomplete becomes UNKNOWN; inequality becomes FAIL; equality becomes PASS. Evidence is caller-supplied and not live runtime proof.
Guard 3 required before real-run intent PASS Source invokes Guard 3 only after exact-True gate; S02–S04/S11 reject, zero writes.
Guard 4 separability PASS Shared stateless helper; no Guard4→Guard1 call.
No-write modes PASS A01–A05 emit zero writes.
Evidence count PASS 52 matrix + 12 simulation = 64; eight [OK] meta-assertions.
Bounded zero fail-open claim PASS Evidence says no invalid enumerated row accepted and correctly limits the claim to bounded local testing.

Real-Run Simulation Audit

Case Verdict Evidence
Gate exactly True + Guard 3 PASS, real_run PASS S01 emits eight write-intent strings only; no DB call.
Gate exactly True + Guard 3 missing/UNKNOWN PASS S02–S03 reject PROD_UNTOUCHED_UNKNOWN, zero writes.
Gate exactly True + Guard 3 FAIL PASS S04 rejects PROD_UNTOUCHED_FAIL, zero writes.
Boolean False PASS S05/S12 reject REAL_RUN_GATE_CLOSED.
String false/true, integer 1, None PASS S06–S09 reject INVALID_GATE_TYPE.
Teardown gate True + Guard 3 PASS PASS S10 emits one DROP write-intent string only.
Teardown Guard 3 FAIL PASS S11 rejects, zero writes.
Runtime interpretation PASS_WITH_CAVEATS Simulation proves decision logic only. Runtime gate is actually closed; no writer or runtime snapshot source is wired.

Evidence Supersession Audit

Check Verdict Evidence
Old 37/37 evidence clearly superseded PASS Rev1 file begins with a SUPERSEDED warning and says not to cite it as current proof.
V2 evidence cited by admission PASS Admission Artifact 5b and current evidence references point to v2.
V2 evidence cited by index PASS Index lists rev1 as superseded and v2 as superseding.
V2 evidence cited by main report PASS_WITH_CAVEAT Top update and addendum supersede historical 37/37 sections retained in the original report body. Readers must use the update/addendum and v2 artifact.
64 rows / 8 meta-assertions PASS Independently counted in the readback body.
Framed as local, not runtime proof PASS V2 evidence and remediation report state pure local validation only and disclaim universal/runtime authority.
Universal proof overclaim removed PASS Current evidence calls itself bounded adversarial coverage.

Non-Authorization Audit

Forbidden action Verdict Evidence
DB write / DDL / DML PASS Remediation report records zero runtime calls; local pure validator only.
dot_tools, dot_config, law_dot_enforcement, dot_agent_api_contract write PASS None recorded; registration remains held.
governance_object_ownership write PASS None; Owner authority remains absent.
Runtime registration PASS (NOT DONE) REGISTRATION_HOLD.
DOT execution / Directus mutation PASS None.
Schema/table/collection creation PASS None.
Runtime gate flip PASS None; gate remains closed.
Owner authority claim PASS (NO CLAIM) Engineering PASS remains distinct from authority PASS.
Macro-9A build / Macro-9C real-run PASS Neither launched.
Operator/future governance catalog creation PASS None; candidates remain paper-only.

This audit relies on the package's recorded non-mutation evidence and this Codex session's KB reads/static review. It is not independent live runtime forensics.

Caveats

  • The DOT remains unregistered, unwired, and unable to perform a runtime write.
  • No Owner authorization exists; real-run remains prohibited.
  • Runtime hardening gaps 2/3/4 remain open: generic directus schema-create privilege, no isolated DOT-executor role, and no policy block on generic Directus create.
  • Guard 3 validates supplied evidence; runtime collection and trust of true snapshots remain future work.
  • V2 is bounded adversarial evidence, not universal mathematical or runtime proof.
  • Contract §1 describes production_untouched_evidence as required for verify, while validator case A03 intentionally accepts evidence-free verify and returns UNKNOWN. This no-write behavior does not reopen a safety HOLD, but the wording should be aligned before registration.
  • Historical 37/37 passages remain in the main report as original record; explicit top-level and addendum supersession controls which evidence is current.

Three Declarations

  1. Permanent: required-field validation, strict lexical matching, strict gate typing, Guard 3 verdict enforcement, and shared-helper separation are now structural in the reference validator.
  2. Cannot be mistaken: no real-run write-intent appears unless input validation passes, gate is exactly boolean True, and Guard 3 verdict is PASS.
  3. 100% automatic: not yet achieved at runtime. The validator is automatic as a pure decision layer; registration, trusted snapshot collection, role isolation, policy blocking, and real-run remain separate Owner-gated work.

Final Decision

  • Is Macro-9B2 validator/contract remediation accepted at engineering layer? Yes, with caveats.
  • Does Macro-9B + Macro-9B1 package now clear the prior Codex HOLD at engineering layer? Yes.
  • Is runtime registration allowed now? No.
  • Is real_run allowed now? No.
  • Is Macro-9A build allowed now? No.
  • Is next step a separate Owner decision on DOT-registration path and manual-block runtime hardening? Yes.
  • Do not implement confirmation: Confirmed. This re-review performed no runtime mutation, registration, DOT execution, schema/table/collection creation, gate flip, Owner grant, Macro-9A build, or Macro-9C real-run.
Back to Knowledge Hub knowledge/dev/laws-new/reports/codex/codex-rereview-macro9b2-validator-contract-remediation-2026-06-20.md