GPT Review — P7 Pilot Migration Plan v0 Round 1
GPT Review — P7 Pilot Migration Plan v0 Round 1
Kết luận
P7 v0 đúng hướng và đủ khung để làm pilot plan, nhưng chưa nên upload KB nguyên trạng. Trạng thái: PASS có điều kiện — cần patch trước khi upload.
Điểm đạt
- Giữ đúng phạm vi: P7 là pilot/migration plan, không migration thật, không SQL, không production write.
- Chọn subset 3 tài liệu có gradient hợp lý: HOW-TO-READ → C1A → P5.
- Có staging JSON, dry-run workflow, verification, rollback/export, PASS/FAIL và go/no-go.
- Component/BOM round 1 chọn Option 1 defer là hợp lý, giảm rủi ro.
- Có constitutional/cross-law check và không yêu cầu sửa P5/P5b/P6.
Patches bắt buộc
1. Khóa lại loại tài liệu: P7 là plan, không phải execution
Hiện một số câu như “chạy simulated checker”, “Agent parse 3 files” có thể bị đọc như yêu cầu thực hiện ngay. Cần thêm câu ở §1 hoặc §10:
P7 chỉ thiết kế dry-run workflow. Việc chạy dry-run thật sẽ là bước P7-run hoặc pilot execution riêng sau khi P7 plan PASS.
2. Kiểm tra path P5 thực tế
P7 đang dùng P5-schema-draft-v0-2.md. Theo KB hiện có path chính có thể là knowledge/dev/laws/dieu38-trien-khai/P5-schema-draft-v0-2.md hoặc title P5 — Schema Draft v0.2. Opus phải verify exact path trước khi final. Nếu chưa chắc, ghi: “exact path to be verified before dry-run”.
3. Đừng chốt quá sớm unit count estimate cho P5
P5 dài và nhiều pseudo-DDL; estimate 30–45 có thể thấp. Nên đổi thành range rộng hơn hoặc ghi là “initial expectation, not PASS threshold”. PASS threshold đang 20–60 hợp lý hơn. Patch source inventory thành: “initial estimate ~30–60, calibrated by dry-run”.
4. publication.version = file revision chưa ổn
File revision/KB revision không nên mặc định thành publication.version vì publication version là governance version. Patch:
publication.version = pilot version label, ví dụpilot-v0hoặcsource-rev-N/pilot-v0trong publication_profile.- KB revision/source hash nằm trong
publication_profile, không là governance version chính.
5. provenance = PROV-AI cần map Đ24 thực tế/deferred
P7 ghi provenance PROV-AI. Đúng theo P5 proposal nhưng phải thêm guard:
provenance value must be verified against Đ24 FAC-PROV registry before execution; if not registered, use approved mapping or leave as staging placeholder.
6. Đ24 labels: entity_code mapping deferred
P7 nói entity_labels.entity_code = canonical_address. P5/P5b đã để deferred nếu Đ24 format khác. Cần chỉnh thành:
staging label mapping uses canonical_address as proposed entity key, final entity_code mapping deferred to Đ24 verification.
7. Section_type seed không được tạo registry mới
P7 nói seed all 17 candidates. Cần làm rõ đây là staging controlled vocabulary proposal from C1A, không tạo registry song song Đ24 và không production seed.
8. Round-trip semantic match >90% cần định nghĩa
“>90% semantic match” dễ mơ hồ. Patch thành qualitative + measurable proxies:
- no lost headings
- no lost body blocks
- code blocks byte-preserved
- tables row-count preserved
- source/export block count delta within tolerance Semantic match score chỉ là advisory, không sole PASS.
9. Checker readiness cần bám P6 wording
P6 final chốt birth + pre-enact bắt buộc trước P7 pilot; daily ERROR smoke trước pilot start. Vì P7 v0 là plan, cần ghi rõ:
- P7 plan chỉ liệt kê readiness checklist.
- P7 execution không được bắt đầu nếu birth/pre-enact design-to-implementation status chưa được User duyệt hoặc explicitly waived for staging-only simulation.
10. Component/BOM defer nhưng vẫn cần “future component candidate log” format
Option 1 hợp lý, nhưng nên thêm mini schema cho candidate log:
- source_unit_address
- phrase/reference
- candidate_component_type
- reason
- decision = defer_to_round2 Không tạo component rows.
11. “No production data changed” phải có verification mechanism
Vì pilot không write production, cơ chế verify nên là:
- dry-run artifacts only in sandbox/staging path
- no Directus/PG mutation calls executed
- source hashes unchanged
- Agent action log attached
12. Cần explicit artifact outputs
Thêm section “Pilot artifacts to produce”:
- source snapshot manifest
- parsed tree JSON
- segmentation proposal JSON
- staging rows JSON
- label mapping proposal JSON
- simulated checker report
- round-trip export markdown
- pilot review report
13. P7 có thể phát hiện cần sửa P5/P5b/P6 — nhưng không sửa trong pilot
Đã nói đúng, nhưng nên thêm routing:
- model gap →
P7 findings report - severity high → stop pilot and open P5/P5b/P6 amendment task
- severity low → continue with note
14. Đ32/APR: pilot proposed publication không enact, nhưng change-set hook vẫn cần concept
Thêm: staging publication/change_set should be marked draft/proposed, apr_ref = null, and any enactment simulation must be clearly simulated-only.
15. Cần phân biệt “source file OFFICIAL” và “pilot source snapshot”
P7 chọn tài liệu OFFICIAL, nhưng snapshot là bản copy frozen. Không được mutate source. Add:
Source snapshot is read-only copy; source documents remain authoritative until real migration approval.
Khuyến nghị nhẹ
- Đổi
D38-HOWTO,D38-C1A,D38-P5thành “pilot doc_code proposal”, không final doc_code. - Thêm một bảng “Pilot round 1 excludes” gồm LSL-01/L1-L5/C3/P5b/P6 và lý do.
- Thêm “manual review checkpoints” sau segmentation proposal và sau round-trip export.
Chỉ đạo Opus
Patch P7 v0 thành v0.1 theo 15 điểm bắt buộc + 3 khuyến nghị. Không upload KB trước khi GPT review Round 2. Không chạy pilot, không tạo artifact dry-run thật, không sửa tài liệu nguồn, không gọi SQL/migration/DOT.