Codex Re-Review — New IU Lego Problem Statement Rev2 Patch Closure
Codex Re-Review — New IU Lego Problem Statement Rev2 Patch Closure
0. Verdict
APPROVE_WITH_MINOR_PATCHES
Rev2 đã đóng các blocker kiến trúc/thẩm quyền chính của review trước. Authority taxonomy, ranh giới thin contract, reuse matrix, DOT anti-island, documentary evidence labels, Điều 39 status và invariant-vs-mechanism đều đã được sửa đúng hướng.
Còn ba mâu thuẫn cục bộ cần patch:
- §2.1 và §4 vẫn dùng cụm “universal_edges + Điều 39” như đích quan hệ, trong khi §7–§9 nói đúng rằng substrate chưa chọn.
- §11 tạo dependency cycle: Phase 3 yêu cầu Atomic Promote đã tồn tại, nhưng Phase 4 lại là phase build Atomic Promote.
- §14 gọi toàn bộ blockers là “quyết định governance của Owner, không phải kỹ thuật”, trái với §11 vốn xác định nhiều technical prerequisites không thể được Owner approval thay thế.
Các điểm này không tái mở kiến trúc và không tạo authorization trực tiếp vì Phase 3–5 vẫn NO-GO/Default=HOLD. Tuy nhiên chúng phải được sửa để patch closure hoàn toàn nhất quán.
3 câu Tuyên ngôn
- Vĩnh viễn: closure được đánh giá theo tám issue gốc và các invariant kiểm chứng được; không đóng issue chỉ vì patch report tự tuyên bố FIXED.
- Nhầm được không: authority class, current-pass verification, reuse verdict và NO-GO gate được kiểm ngay tại section/matrix liên quan; mâu thuẫn cục bộ vẫn bị ghi nhận thay vì bị self-review PASS che khuất.
- 100% tự động: mission là review read-only, không tạo runtime/manual step. Implementation tương lai vẫn fail-closed: Owner approval không thay checker, staging, rollback, Atomic Promote hay evidence kỹ thuật.
1. Sources Read
Ba tài liệu bắt buộc đã được đọc toàn văn qua Agent Data:
| Source | Revision/status | Coverage |
|---|---|---|
| de-bai-mieng-thong-tin-moi-lego.md | rev2, 42,291 chars, DRAFT/NON-AUTHORIZING | Đọc toàn văn offsets 0→42,291 |
| claude-patch-report-de-bai-mieng-thong-tin-moi-lego-after-codex.md | rev1, 10,984 chars, NON-AUTHORIZING | Đọc toàn văn offsets 0→10,984 |
| codex-review-de-bai-mieng-thong-tin-moi-lego.md | rev1, 19,724 chars | Đọc toàn văn offsets 0→19,724 |
Controlling sources được kiểm lại khi cần:
- knowledge/dev/ssot/operating-rules.md — v7.58 Active.
- knowledge/dev/laws/constitution.md — v4.6.3 BAN HÀNH.
- law-01 foundation principles — enacted source được xác nhận qua compatibility/index search.
- de-bai-cai-tien.md rev33; Matrix implementation plan rev5; quick rules rev8; stamp addendum rev14 — DRAFT/non-enacting direction.
- F2/F3/FX packets — non-authorizing; F2 giữ no-new-store và không mở pilot; F4/HOLD-2 giữ checker/Atomic Promote là unresolved technical obligations.
- Amendments Điều 33/36/38/L4 rev1 — DRAFT/NON-ENACTING.
- Điều 39 v2.3 — enacted; compatibility note rev1 — non-authorizing; runtime EMPTY/substrate unresolved.
- Prior documentary IU/KG evidence — chỉ dùng làm snapshot/reference, không current-pass live proof.
Không required source nào thiếu trong Agent Data KB. Không có current-pass production query.
2. Prior Issue Closure Matrix
| Issue | Status | Rev2 evidence | Reason |
|---|---|---|---|
| IU-REV-01 — DRAFT treated as enacted | CLOSED | §0.1; §1.3; §3; §8 Authority class | Taxonomy tách 5 lớp; Điều 33/36/38/L4 đều ghi DRAFT/NON-ENACTING; bỏ “luật đã chọn”. |
| IU-REV-02 — Thin contract boundary unclear | CLOSED | §3 option C; §4 Layers A/B/C + hard rule | IU chỉ sở hữu Subject Contract; IO/checker/promote/rollback/approval/DOT/event/registry thuộc shared layers và chỉ được reference. |
| IU-REV-03 — Over-optimistic REUSE_AS_IS | CLOSED | §7 AS_IS definition + full matrix | Pass không live-verify nên không asset nào AS_IS; các asset trọng yếu đã hạ về repair/wrapper/verify/reference/migrate/blocked. |
| IU-REV-04 — Writable DOT sub-registry | CLOSED | §7 DOT rows; §9.6; §12.2 | Default duy nhất migrate-to-dot_tools; chỉ read-only compatibility view; writable sub-registry bị bỏ. |
| IU-REV-05 — Documentary counts overstated | CLOSED | §0.1; §1 banner; §5–§7 snapshot labels | Mọi LIVE/count được gắn snapshot 2026-06-01, not re-verified; Current-pass verified = NO. |
| IU-REV-06 — Điều 39 overstated/misclassified | PARTIALLY_CLOSED | §7 universal_edges row; §8 Điều 39 row; §9.3 | Core fix đúng: enacted law, runtime-not-ready, substrate-unknown, projection-only. Nhưng §2.1 và §4 vẫn viết quan hệ “qua universal_edges + Điều 39”, tạo shorthand preselection trái với sections controlling. |
| IU-REV-07 — Phase 3 looked authorized | PARTIALLY_CLOSED | §11 Phase 3 NO-GO + prerequisites; §13 no macro opens pilot | Authorization risk đã đóng. Còn dependency cycle Phase 3↔Phase 4 và §14 phủ nhận tính kỹ thuật của blockers. |
| IU-REV-08 — Preserve legacy mechanisms | CLOSED | §10 invariant/outcome vs mechanism | Giữ property, không mặc định giữ table/function; mechanism phải reuse/wrap/repair/replace/deprecate. |
Net closure: 6 CLOSED, 2 PARTIALLY_CLOSED, 0 STILL_OPEN ở mức blocker, 1 minor new regression về phase dependency.
3. High-Level Assessment
Rev2 giữ đúng framing gốc và làm chặt đáng kể:
- Không phục hồi old IU standalone engine.
- Không rơi vào PG-only/dumb-row.
- New IU được giới hạn thành IU Subject Contract.
- Shared IO/checker/promote/approval/owner/DOT/event/registry systems không thuộc ownership của IU.
- Không asset cũ nào được AS_IS khi chưa current-pass verified.
- Điều 39 được tách đúng giữa enacted law và runtime/substrate readiness.
- Phase 3–5 đều NO-GO; fastest-safe path hiện chỉ Phase 0–2 documentary/read-only/design-only.
Ba lỗi còn lại là local consistency defects, không phải kiến trúc mới. Vì vậy rev2 đủ để GPT/Owner review hướng, nhưng chưa nên được coi là patch-closure hoàn tất trước khi áp dụng ba minor patches.
4. Remaining Issues
RR2-01 — Residual universal_edges preselection wording
- Severity: minor
- Section: §2.1; §4 đoạn “KHÔNG là KG riêng”.
- Problem: hai đoạn vẫn nói semantic relations đi qua universal_edges/Điều 39. Điều này không khớp §7, §8 và §9.3: universal_edges chỉ là candidate shared target, suitability/provenance/owner chưa verified; IU-native UUID edge + governed read-only projection vẫn là lựa chọn hợp lệ.
- Required correction: thay bằng “đi qua shared KG substrate được chọn sau Phase-1 verification; universal_edges là candidate/reference, chưa phải canonical target”.
RR2-02 — Phase 3/Phase 4 dependency cycle
- Severity: minor
- Section: §11.
- Problem: Phase 3 chỉ được mở khi Atomic Promote Contract đã tồn tại hoặc có equivalent approved; ngay sau đó Phase 4 lại được định nghĩa là phase build Atomic Promote Contract. Với roadmap đánh số 3→4→5, dependency này tự khóa hoặc ngụ ý một external track không được nêu.
- Required correction: chọn một cách diễn đạt duy nhất: hoặc (a) đưa checker/Atomic Promote readiness phase trước pilot; hoặc (b) ghi rõ Phase 3 chỉ là non-canonical dry-run không exercise promote, còn promote rehearsal/pilot chỉ xảy ra sau Phase 4. Không cần thiết kế mới trong patch này; chỉ sửa order/boundary.
RR2-03 — Final caveat misclassifies technical blockers as governance-only
- Severity: minor
- Section: §14 Caveat; patch report §9 Remaining Caveats.
- Problem: câu “các blocker … đều là quyết định governance của Owner, không phải kỹ thuật” gom HOLD-1, HOLD-2, RISK-BYPASS, staging/Atomic Promote cùng CONS/Owner decisions. Đây là mix của governance decisions và technical readiness evidence. Câu này trái trực tiếp với §11: Owner approval alone không thay technical prerequisites.
- Required correction: đổi thành “các blocker gồm cả governance decisions và technical readiness obligations; Owner quyết định authority/scope nhưng không được waive technical PASS/evidence”.
5. New Regressions
Có một minor new regression: dependency cycle Phase 3 requires Atomic Promote → Phase 4 builds Atomic Promote.
Không có regression mới về:
- authority leakage;
- reuse AS_IS;
- writable DOT island;
- documentary/live evidence;
- hidden KG SoT;
- direct build authorization.
6. Authority / DRAFT vs Enacted Check
PASS.
Rev2 tách rõ:
- ENACTED AUTHORITY.
- DRAFT-NON-ENACTING DIRECTION.
- NON-AUTHORIZING COMPATIBILITY/DECISION MATERIAL.
- DOCUMENTARY EVIDENCE.
- CURRENT-PASS LIVE PROOF.
Điều 33/36/38/L4 được ghi DRAFT amendment, NON-ENACTING trong §1.3 và từng row §8. Điều 39 được ghi enacted ở cấp law nhưng runtime thuộc documentary/not-ready. Không phát hiện câu còn dùng các amendment như enacted authority.
Một lưu ý không blocking: “ENACTED AUTHORITY” đang gom cả active OR/house rules. Ý nghĩa thực tế rõ trong table; không tạo authority leak cho New IU.
7. Thin Contract Boundary Check
PASS.
§4 tách đúng:
- Layer A: IU Subject Contract — ownership duy nhất của New IU.
- Layer B: shared Matrix/Lego IO Contract — reference only.
- Layer C: shared F4/One-Roof execution envelope — reference only.
Hard rule cấm IU sở hữu/nhân đôi registry, workflow, checker, promote và owner. §3, §6 và §13 đồng bộ với boundary này.
8. Reuse Matrix Check
PASS.
Các asset đặc biệt được xử lý đúng:
| Asset | Rev2 classification | Check |
|---|---|---|
| information_unit | REUSE_AFTER_REPAIR/WRAPPER | Correct |
| unit_version | REUSE_AFTER_VERIFY provisional | Correct |
| canonical_address | REUSE_AFTER_VERIFY provisional | Correct |
| lifecycle/write functions | REUSE_AFTER_REPAIR/WRAPPER | Correct |
| conformance_status | REUSE_AFTER_REPAIR | Correct |
| owner_ref | DEPRECATE_OR_AVOID | Correct |
| iu_relation | WRAPPER, projection-only | Correct |
| universal_edges | REPAIR / VERIFY_AS_SHARED_TARGET | Correct; residual prose cần minor patch RR2-01 |
| iu_three_axis_envelope | REFERENCE / MIGRATE_THEN_DEPRECATE | Correct |
| dot_iu_command_catalog/run | MIGRATE_TO_DOT_TOOLS + read-only view | Correct |
| governance_object_ownership | TARGET_REFERENCE_BLOCKED | Correct |
| vector assets | WRAPPER, gated boundary | Correct |
| Directus/Nuxt | DEFER | Correct |
| IU event types | REUSE_AFTER_REPAIR | Correct |
| old IU reports | REFERENCE_ONLY | Correct |
Không có old IU asset nào bị phân loại REUSE_AS_IS.
9. Anti-Island Check
PASS WITH RR2-01 MINOR WORDING PATCH.
- Governance: shared One-Roof only.
- Birth: shared birth/stamp direction, still authority/runtime gated.
- KG: no second SoT; projection-only option preserved.
- Registry: no IU-local writable registry.
- Approval: shared Điều 32/APR.
- DOT: migrate to dot_tools; read-only compatibility projection only.
- Axis: migrate-then-deprecate, no repair-in-place island.
- Storage/staging/candidate: view/projection-first; new store requires no-new proof, separate mission and Owner; Phase 3 NO-GO.
Không phát hiện hidden writable island. RR2-01 chỉ là prose inconsistency về KG target.
10. Điều 39 / KG Check
PASS WITH MINOR WORDING PATCH.
§8 phân loại đúng:
- Điều 39 v2.3 = ENACTED at law level.
- Runtime = NOT READY/EMPTY.
- Substrate = UNKNOWN.
- universal_edges không được chọn trước verification.
- IU-native edge + governed read-only projection vẫn được giữ.
- TBox human-approved; ABox AI-proposed.
- provenance/trust bắt buộc; no self-learning into canonical; no second KG SoT.
Cần sửa hai residual phrases ở §2.1/§4 để toàn tài liệu không còn shorthand preselection.
11. Phase 3 / Pilot Gate Check
PASS về authorization safety; MINOR PATCH REQUIRED về dependency consistency.
Rev2 ghi rõ:
- FUTURE HYPOTHESIS / NOT READY / NO-GO.
- F2 không cho chọn pilot trong gate hiện tại.
- Technical prerequisites và Owner authorization đều bắt buộc.
- Owner approval alone không thay technical prerequisites.
- Không macro nào ở §13 mở Phase 3.
- Default = HOLD.
Vì vậy Owner không thể hợp lệ mở Phase 3 chỉ bằng approval. Tuy nhiên cần sửa RR2-02 và RR2-03 để roadmap/final caveat không tự mâu thuẫn với gate này.
12. Final Recommendation
GPT/Owner có thể review rev2 ngay ở cấp định hướng. Các blocker cũ đã được đóng đủ để không còn nguy cơ approve nhầm DRAFT authority, reuse legacy AS_IS hoặc mở pilot trực tiếp.
Trước khi ghi nhận patch closure hoàn tất hoặc dùng target làm decision packet chính thức, áp dụng ba minor patches RR2-01…03:
- bỏ residual universal_edges preselection;
- phá dependency cycle Phase 3/4;
- tách governance decisions khỏi technical readiness blockers trong final caveat.
Không implementation, không live apply, không approval/enactment, không self-approval. Không sửa target trong mission này.
Evidence theo quy trình 0→6
- Bước 0: đã đọc skill 36 mục; OR v7.58; Hiến pháp v4.6.3; luật/ngữ cảnh cần thiết qua search_knowledge.
- Bước 1: mission duy nhất là rev2 patch-closure review; ba tài liệu bắt buộc đã đọc toàn văn.
- Bước 2: kiểm theo 8 issue cũ; chỉ mở một regression thực về phase dependency.
- Bước 3: N/A — không code/schema/runtime mutation.
- Bước 4: N/A — không PR/merge/deploy.
- Bước 5: N/A — không deploy, không production verification; current-pass live proof = none.
- Bước 6: chỉ tạo report này. OR/TD/handoff không cập nhật vì mission cho phép đúng một review report và không thay đổi runtime/operating state.