Council Review Điều 43 v1.2 DRAFT — Round 1 — Gemini

Date: 2026-04-17 Reviewer: Gemini CLI Score: 9.5/10 Recommendation: APPROVE WITH MINOR

Q1 — Xung đột luật

Dựa trên rà soát 5 khoản amend, không phát hiện xung đột nghiêm trọng với các luật hiện hành.

• HP v4.6.1: Hoàn toàn tương thích, thậm chí là bước tiến lớn để hiện thực hóa NT2 (Tự động 100%) và NT4 (Thay đổi = config) mà bản v1.1 còn nợ.
• Điều 33 v2.0 (PostgreSQL): Tuân thủ tốt việc sử dụng PG Native (NT13) qua bảng reference và catalog.
• Điều 35 v5.1 (DOT): Cấu trúc 2 DOT cặp và dual-trigger giữ nguyên, đảm bảo tính nhất quán.
• Điều 22 (Self-Healing): Retry policy mới trong context_pack_requests bổ trợ cho cơ chế xử lý issue của Điều 22, không chồng chéo.
• Đánh giá: KHÔNG XUNG ĐỘT.

Q2 — Đủ NT2+NT4+NT11?

• Test 1 (Thêm DB thứ 4): PASS. Chỉ cần cập nhật JSON array trong dot_config('context_pack_scan_db_whitelist') hoặc để trống để auto-detect qua pg_database (NT11).
• Test 2 (Thêm folder code thứ 5): PASS. Cập nhật dot_config('context_pack_scan_paths'), script loop qua JSON array nên không cần sửa code.
• Test 3 (Thêm law pattern mới): PASS. Cập nhật dot_config('context_pack_watched_key_patterns').
• Test 4 (Thêm section thứ 9): PASS. INSERT 1 row vào context_pack_section_definitions. Script build loop qua table này nên sẽ tự động sinh thêm file.
• Test 5 (Thêm health check H10): PARTIAL. Nếu check_type mới đã có handler trong code (ví dụ dùng lại handler section_size) thì PASS. Nếu là logic kiểm tra hoàn toàn mới, cần amend code script để thêm handler (đã được dự liệu trong §9 là ngoại lệ có lý do). Tuy nhiên, không cần amend luật Điều 43.

Q3 — Điểm hardcode sót

Rà soát kỹ wording v1.2, phát hiện một vài điểm nhỏ cần lưu ý:

1. Advisory lock namespace (43, 1): Chấp nhận được (Protocol ID).
2. Path folder mặc định (§11 Bước 7): /opt/incomex/context-pack/. Mặc dù đã có config scan_paths, nhưng folder output vẫn đang fix cứng trong luật và lệnh mv. Đề xuất: Thêm key context_pack_output_root vào dot_config để triệt để NT4.
3. Regex strip VOLATILE HEADER (§7.1): Marker <!-- VOLATILE HEADER --> đang là chuỗi cứng. Nếu sau này template đổi format (không dùng MD), regex này có thể fail. Tuy nhiên ở mức v1.2, việc quy định marker này trong luật để các renderer tuân theo là chấp nhận được (Standardization).
4. Exit codes 0/1/2/3: Tuân thủ chuẩn POSIX/DOT, không vi phạm.
5. SHA256: Là tiêu chuẩn hệ thống hiện tại, chưa cần cấu hình hóa thuật toán.

Khuyến nghị trước ban hành (nếu APPROVE WITH CHANGES)

• Minor Patch 1 (NT4 triệt để): Chuyển folder output /opt/incomex/context-pack/ thành cấu hình context_pack_output_path trong dot_config (mặc định vẫn là path cũ).
• Minor Patch 2 (NT11): Trong context_pack_health_checks, với các check cần threshold (như H1, H4, H5), đảm bảo script verify đọc chính xác từ threshold_config JSONB thay vì dùng case statement hardcode giá trị fallback.

Kết luận

Bản v1.2 DRAFT là một bản nâng cấp "sạch" và cực kỳ chuyên nghiệp. Nó không chỉ sửa lỗi vi phạm Hiến pháp của v1.1 mà còn biến Law 43 thành một framework "meta-data driven" thực thụ. Việc chuyển dịch từ hardcode sang reference tables giúp hệ thống tự thích nghi (NT2) và mở rộng linh hoạt mà không cần can thiệp vào mã nguồn script build/verify. Khuyến nghị BAN HÀNH ngay sau khi apply 2 minor patches trên.