Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Register
  • Sign in
  • A AI-first Engineering Playbook
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 2
    • Merge requests 2
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Terraform modules
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • holatech
  • AI-first Engineering Playbook
  • Merge requests
  • !3

feat: phase 1 s4 spec and s5 change requests

  • Review changes

  • Download
  • Patches
  • Plain diff
Open Khanh Pham requested to merge feat/phase1-s5 into main Nov 02, 2026
  • Overview 0
  • Commits 107
  • Pipelines 0
  • Changes 595

Tóm tắt

Phase 1 S4 (spec) và S5 (change request). Nhánh này cắt từ feat/phase1-s4 nên chứa cả hai. Nó còn mang theo 65 commit có trên main local nhưng chưa có trên origin/main (S2/S3 đã merge local và đổi model của agent intent sang Haiku 5.5).

S4: spec

  • Confirm brief → agent spec (claude-opus-5-5) sinh REQ/AMB/AC. Có version, sửa tay, diff, archive.
  • Gate review và trả lời AMB. Vòng revision dừng khi không còn tiến triển.
  • J3: Jev gắn cờ AC khó test (chỉ là gợi ý).
  • Link review cho guest (/r/:token): nginx headers, ẩn URL trong log, audit IP/UA.
  • FE: trang spec, gate và trang cho guest.

S5: change request

  • Migration change_requests. CrDelta, applyCrDelta và dryRunCrDelta nằm trong @cp/shared.
  • API CR: tạo kèm phân tích, sửa, phân tích lại, rút, quyết định. Khi duyệt sẽ áp delta thành version mới.
  • Agent cr-impact. J4 (cr.scope) và J5 (cr.ac_relevance, lọc AC khi có trên 15).
  • Archive spec sẽ rút CR đang mở. Thứ tự khóa: spec → run → CR → link.
  • Guest duyệt CR qua link review.
  • FE: tab CR, editor delta, panel quyết định, lịch sử "vN CR-n".

Migration

  • 1793491200000-specs.ts, 1793577600000-change-requests.ts.

Kiểm tra

  • bash scripts/verify.sh qua: lint, typecheck, unit, e2e. be e2e 309 test.
  • UI e2e (Playwright) 21/21, chạy với fake runtime.
  • Mỗi task đều qua review riêng. Final review cho S4 và S5 bằng convention-reviewer và spec-reviewer trên Opus: PASS.

Còn để lại (không chặn)

  • analyze không khóa spec: nếu archive xảy ra cùng lúc, một run vẫn gọi model rồi bỏ kết quả.
  • Gọi retry trên run cr-impact trả về 409; người dùng bấm "Phân tích lại".
  • TRUST_PROXY_HOPS chưa đặt trong cấu hình deploy nào, nên giới hạn theo IP của link review dùng req.ip.
  • Câu hỏi mở về SPEC: quyền đọc CR (15.7 ghi chỉ editor); impact_md nhắc tới AMB mới trong khi delta không thêm được AMB.

🤖 Generated with Claude Code

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: feat/phase1-s5