@trungtamcongnghedigital/vsaf 1.0.0

package/skills/NTAPP/pack.yaml

@@ -0,0 +1,177 @@
+name: NTAPP
+version: 1.0.0
+description: NTAPP delivery flow — triage, slice planning, slice loop, hotfix, QA pipeline
+author: "@trungtamcongnghedigital/vsaf"
+
+# Deployed with the pack prefix, so these keep the NTAPP-* names their workflows,
+# hooks and documents already use today. No renaming.
+skills:
+  # Discovery & design
+  - business-overview-confluence
+  - system-overview
+  - market-research
+  - product-research
+  - architecture-design
+  - technical-design
+  - feature-design
+  - dd-design
+  - integration-inspection
+  - keycloak-integrate
+
+  # Triage & planning
+  - feature-triage
+  - slice-planning
+  - jira-subtask-creator
+  - sprint-worklog-from-git
+
+  # Design system & UI
+  - ds-bootstrap
+  - ds-build-component
+  - figma-ingest
+  - ui-generation
+  - ux-design
+  - ux-sketch
+
+  # Bug & hotfix
+  - bug-fix-engine
+  - fix-bug
+  - log-bug
+
+  # QA pipeline
+  - qa-audit
+  - qa-gap-resolver
+  - qa-generate-testcase
+  - qa-robot
+  - qa-setup-data
+
+  # Knowledge bundle (OKF) + đồng bộ skill ra máy
+  - okf-refresh
+  - sync-skills-global
+
+  # Confluence publishing
+  - dd-to-confluence
+  - srs-to-confluence
+  - kbnt-to-confluence
+
+  # Bitbucket
+  - create-pull-request
+
+  # Workflow entry points
+  - vsaf-run
+  - vsaf-resume
+  - vsaf-ad-hoc-per-unit
+  - vsaf-express-hotfix
+  - vsaf-fix-bug
+  - vsaf-path-b-quick-dev
+  - vsaf-reproduce-architecture
+  - vsaf-slice-loop
+  - vsaf-staging-deploy
+
+# Deployed without a prefix — the 15 skills that NTAPP skills and workflows call by
+# name, plus what those skills call in turn. Installing NTAPP must not require
+# installing sdlc, so these travel with the pack.
+bundled:
+  - bmad-check-implementation-readiness
+  - bmad-code-review
+  - bmad-create-epics-and-stories
+  - bmad-create-prd
+  - bmad-create-story
+  - bmad-quick-dev
+  - bmad-testarch-atdd
+  - bmad-testarch-ci
+  - bmad-testarch-framework
+  - bmad-testarch-nfr
+  - bmad-testarch-trace
+  - c4-diagram-optimizer
+  - grill-with-docs
+  - lavish
+  # Called by a workflow node, not by any skill file
+  - superpowers-finishing-a-development-branch
+
+# Pack này mang theo script nối dây của chính nó, và nó là nguồn sự thật cho 17 bước
+# đó. Khai ở đây thì install bỏ qua khối nối dây nội bộ của vsaf và gọi script — nếu
+# cả hai cùng chạy thì hai bên ghi lên nhau. Pack không khai (sdlc) đi nhánh cũ,
+# không đổi gì.
+#
+# `env` là các biến bắt buộc phải đặt: script có hỏi tên project và hỏi xác nhận
+# phạm vi repo, gọi mà không có TTY thì nó đứng chờ mãi.
+wiring:
+  script: scripts/bootstrap.sh
+  env: [PROJECT_NAME, AUTO_CONFIRM]
+  passthrough: ["--refresh", "--upgrade", "--dry-run"]
+
+# Nguồn skill của pack này nằm ở bố cục của base template, không phẳng ở gốc pack.
+# Nhờ vậy script sync của NTAPP chạy trực tiếp trong thư mục pack mà không cần dựng
+# thư mục tạm nào.
+skillsRoot: .agent/skills
+skillsPrefixed: true
+
+# Skill đi theo project thay vì ra máy. Cả thư viện sống ở global, nên bản copy vào
+# project chỉ giữ đúng skill dùng để làm mới global về sau — không thì một máy mới,
+# global còn trống, sẽ không có đường nào tự đồng bộ mà không clone lại base template.
+skillsKeepInProject:
+  - NTAPP-sync-skills-global
+
+# Thứ chỉ vsaf đọc, không phải nội dung của project — loại khỏi bản copy giống
+# pack.yaml. Bốn file này là bản đối chiếu của vsaf cho mấy asset mà upstream đặt ở
+# scripts/templates/; script nối dây dùng bản của nó, nên copy vào project chỉ để lại
+# hai bản của cùng một thứ và không ai biết bản nào đang có hiệu lực.
+packOnly:
+  - config
+  - git-hooks
+  - hooks/settings.json
+  - rules/rules.json
+
+# Script đẩy skill + rule ra ngoài project. Cùng lý do như `wiring`: dùng bản của
+# NTAPP thay vì bản của vsaf, để hai bên không lệch nhau.
+globalSync:
+  script: scripts/sync-skills-to-user.sh
+  passthrough: ["--dry-run", "--skills-only", "--rules-only", "--clean"]
+
+# Where this pack's skills write their deliverables. `vsaf init` copies this into
+# the project's telemetry config, so the per-node output metrics count NTAPP's
+# documents as artifacts. Without it the engine default (`.vsaf/docs/`, the sdlc
+# pack's layout) applies and every NTAPP run reports artifacts_created: 0 while
+# still counting the same files under the generic file total.
+telemetry:
+  artifacts:
+    include:
+      - NTAPP-output/
+
+# The pack's own skills are reinstallable, so they live where git ignores them and
+# get mirrored to the other IDE directory.
+#
+# NOT `.agent/skills`: in a spec repo that path means something else — it is the
+# place for skills written for this one project, kept empty by default and meant to
+# be committed. Putting 55 reinstallable skills there would commit 9 MB of them.
+canonicalSkillsDir: .claude/skills
+
+# Paths this pack generates per machine, kept out of git. Two policies because a
+# spec repo keeps its canonical skills and its hand-edited config in version
+# control, while a code repo regenerates both on every machine.
+gitignore:
+  spec:
+    - .claude/skills/
+    - .cursor/skills/
+    - .claude/hooks
+    - "*.bak.*"
+  code:
+    - .agent/skills
+    - .claude/skills/
+    - .cursor/skills/
+    - .claude/hooks
+    - .claude/settings.json
+    - .bmm-config.yaml
+    # Regenerated per machine from the pack, like the config above. The spec repo
+    # commits its copy; a code repo receives one and has nothing to review in it.
+    - rules/
+    - "*.bak.*"
+
+# Writing outside the project needs the user to say yes, and everything written is
+# recorded in a manifest so it can be taken back without touching personal skills.
+global:
+  skillTargets:
+    - .claude/skills
+    - .gemini/config/skills
+    - .codeium/skills
+  ruleTarget: .claude/rules

package/skills/NTAPP/rules/be-finance-rules.md

@@ -0,0 +1,139 @@
+# Rule khi code Backend (hệ thống tài chính số)
+
+Các rule dưới đây áp dụng khi sửa/tạo file Backend (Java/Spring Boot, hoặc service code nói chung). Với file Frontend hoặc file không thuộc BE thì bỏ qua toàn bộ mục này. Cách áp dụng giống rule Sonar: review phần code BE đã đụng tới TRƯỚC khi báo hoàn thành, phát hiện vi phạm thì sửa luôn và nêu trong báo cáo cuối.
+
+# Kiểm tra Sonar sau khi sửa code
+
+Mỗi khi sửa code xong (bất kỳ task nào có chỉnh sửa file code), TRƯỚC khi báo hoàn thành phải tự review các file vừa sửa/tạo theo các rule Sonar dưới đây. Nếu phát hiện vi phạm → sửa luôn rồi mới kết thúc. Chỉ review phần code đã đụng tới trong task, không quét toàn repo.
+
+Báo cáo ngắn gọn ở cuối: liệt kê file đã review + rule nào vi phạm (nếu có) + đã sửa thế nào, hoặc "không vi phạm".
+
+## Các rule chính cần kiểm tra
+
+**`java:S3776` — Cognitive Complexity of methods should not be too high**
+- Ngưỡng mặc định: cognitive complexity của 1 method **> 15** là vi phạm.
+- Tăng điểm khi: mỗi cấu trúc rẽ nhánh/lặp (`if`, `else if`, `else`, `for`, `while`, `do`, `catch`, ternary `?:`, `switch`); mỗi mức lồng nhau (nesting) cộng thêm penalty; chuỗi toán tử logic hỗn hợp (`&&`, `||`) xen kẽ; `break`/`continue` có label; đệ quy.
+- Cách sửa: tách method lớn thành các method con (extract method), early-return / guard clause để giảm nesting, gộp điều kiện trùng, dùng polymorphism/map thay cho switch dài, tách vòng lặp lồng nhau ra hàm riêng.
+
+**`java:S1192` — String literals should not be duplicated**
+- Ngưỡng mặc định: cùng một string literal (≥ 5 ký tự) lặp lại **≥ 3 lần** trong cùng 1 file là vi phạm.
+- Bỏ qua các literal quá ngắn (< 5 ký tự) và annotation values.
+- Cách sửa: tách thành `private static final String` constant đặt tên có nghĩa, rồi thay thế tất cả chỗ dùng.
+
+**`java:S125` — Sections of code should not be commented out**
+- BẮT BUỘC kiểm tra mọi file vừa sửa/tạo. Mọi dòng/khối **code** bị comment-out (không phải comment giải thích) đều là vi phạm — kể cả khi chỉ có **1 dòng**, không có ngưỡng tối thiểu.
+- 🔴 **TIÊU CHÍ SẮC — chạy TRƯỚC khi báo hoàn thành, không có ngoại lệ.** Signature bắt được chắc chắn (đã kiểm chứng thực nghiệm, xem mục "Bằng chứng đo được" bên dưới) là: **line comment `//` KẾT THÚC bằng dấu `;`**. Quét cơ học bằng đúng 1 lệnh trên các file `.java` vừa sửa:
+  ```bash
+  grep -nE '^\s*//.*;\s*$' <file1.java> <file2.java> ...
+  ```
+  Có bất kỳ dòng nào trả về → **PHẢI sửa** (tách câu bằng dấu chấm) rồi mới báo hoàn thành. Không có dòng nào → qua cửa này.
+  ⚠️ Bước grep này là BẮT BUỘC, không được thay bằng "tôi đã đọc lại và thấy comment là văn xuôi nên an toàn" — chính suy nghĩ đó đã làm lọt 3 lần.
+- 📊 **Bằng chứng đo được (repo `merchant-composite-service`, quét toàn `src/`):**
+
+  | Dạng comment chứa `;` | Số dòng | Sonar flag? |
+  |---|---|---|
+  | Javadoc `/** ... ; */` | 26 | ❌ không |
+  | `//` có `;` **giữa dòng** (vd `at 422; HUB-04 reuses...`) | ~9 | ❌ không |
+  | `//` có `;` **cuối dòng** | 2 | ✅ **cả 2 đều bị flag** |
+
+  → Đừng phát biểu rule kiểu "`;` ở bất kỳ đâu là nguy hiểm": ~94% trường hợp không sao, nên rule rộng bị đọc như cảnh báo nền rồi bỏ qua. Rủi ro thật tập trung ở `//` kết thúc bằng `;`.
+- ⚠️ **`;` là dấu câu hợp lệ trong tiếng Việt — và vị trí nó rơi vào cuối dòng là NGẪU NHIÊN.** Cùng một câu, wrap lệch 3 chữ là thoát/dính. Vì vậy KHÔNG dùng `;` để ngăn hai vế câu trong comment — dùng dấu chấm hoặc dấu phẩy. Đây là nguồn tái phạm duy nhất qua cả 3 lần dính.
+- ✅ **Lối thoát hợp pháp:** ghi chú dài, buộc phải nhắc tên kỹ thuật/contract → đặt trong **Javadoc `/** ... */`** (S125 thực tế bỏ qua Javadoc — 26/26 dòng Javadoc có `;` trong repo đều sạch), KHÔNG viết thành khối `//` nhiều dòng.
+- Dấu hiệu nhận biết "code bị comment" (khác với comment giải thích bằng ngôn ngữ tự nhiên): bên trong comment có cú pháp code — dấu `;` **cuối dòng** (rủi ro cao nhất), `{` `}` `()`, từ khóa (`if`, `for`, `return`, `import`, khai báo biến/method), gọi hàm, annotation `@...`, identifier dạng `snake_case`/`camelCase` (tên cột/field/biến viết raw, vd `current_status`, `last_synced_at`), **từ ghép nối bằng gạch ngang** (vd `sub-link` — parser đọc ra phép trừ `sub - link`), toán tử (`+`, `=`, `->`), hoặc một statement/expression hoàn chỉnh.
+- ⚠️ **Cạm bẫy FALSE-SAFE — đừng dựa vào "đây là câu giải thích nên an toàn":** engine Sonar chấm S125 theo **cú pháp**, KHÔNG theo ý nghĩa. Một comment **đọc ra vẫn là giải thích** nhưng có chèn token code thì VẪN bị flag. Token nguy hiểm cần tránh trong comment:
+  - lời gọi method `tênHàm(...)`, dấu `{...}`/`(...)` bọc token code, path template kiểu `{id}` / `{reason}`, hoặc HTTP verb + path (`PUT /x/{id}`);
+  - **tên cột/field/biến viết raw dạng `snake_case`/`camelCase`** (vd `current_status`, `last_synced_at`, `updated_at`) — Sonar đọc ra là identifier;
+  - **dấu `;` ở CUỐI dòng comment** và **mệnh đề bọc trong `(...)`** chứa identifier/từ khóa (vd `(Schema has no updated_at column;`) — cặp `(...)` + `;` rất dễ bị chấm là statement.
+  - Ví dụ thực tế đã dính (3 lần): (1) comment mô tả contract API chứa `dto.getReason()` và `{reason}`; (2) comment mô tả luồng DB chứa `current_status + last_synced_at` và `(Schema has no updated_at column;`; (3) **hai comment văn xuôi tiếng Việt 100% không có token code nào**, chỉ vì dùng `;` ngăn câu và nó rơi đúng cuối dòng → vẫn bị S125 bắt. Cả ba đều là giải thích, không phải code chết.
+  - → Khi mô tả API/contract/luồng/DB trong comment, diễn đạt **bằng lời thuần** (vd "cập nhật trạng thái hiện tại và mốc thời gian đồng bộ"), KHÔNG chèn cú pháp code, KHÔNG nhắc tên cột/field/biến ở dạng raw token; nếu buộc phải nhắc tên kỹ thuật thì để ở Javadoc/tài liệu, không nhúng dạng code vào comment dòng.
+- ✅ **Quét thủ công bổ sung cho MỌI comment giải thích nhiều dòng** (chạy SAU lệnh grep ở trên, và quét cả comment có sẵn nằm trong vùng code đã đụng tới): đọc từng dòng comment và tự hỏi "dòng này có chứa `()`/`{}`, `=`/`+`/`->`, `@...`, identifier `snake_case`/`camelCase`, từ ghép gạch ngang, hay path `{...}` không?". Nếu CÓ ≥ 1 token → viết lại dòng đó bằng văn xuôi thuần trước khi báo hoàn thành. Quy tắc an toàn: khối `//` nhiều dòng = **văn xuôi thuần tiếng Việt/tiếng Anh, zero token code, không `;`**.
+- Áp dụng cho mọi kiểu comment: `//`, `/* ... */`, `/** ... */`, và comment nhiều dòng bao quanh block code. Lưu ý phân biệt: **code thật bị comment-out** thì kiểu comment nào cũng phải xóa; còn **văn xuôi giải thích bị chấm nhầm** thì rủi ro dồn vào khối `//` — nên đó là lý do đưa ghi chú dài sang Javadoc.
+- Cách sửa: **xóa hẳn** khối code đã comment (git/lịch sử đã lưu, không cần giữ). KHÔNG để lại "code chết". Nếu thật sự cần giữ ghi chú thì chuyển thành câu giải thích bằng tiếng Việt, không để nguyên dạng code.
+- Khi review cuối task, quét lại từng file đã đụng tới: nếu còn bất kỳ khối code comment-out nào → xóa rồi mới báo hoàn thành.
+
+**`java:S1128` — Unnecessary imports should be removed**
+- BẮT BUỘC kiểm tra mọi file `.java` vừa sửa/tạo. Mỗi `import` (kể cả `import static`) khai báo nhưng KHÔNG được dùng ở bất kỳ đâu trong file đều là vi phạm — không có ngưỡng tối thiểu, 1 import thừa cũng bị flag.
+- Nguồn phát sinh phổ biến: bỏ bớt code sau refactor nhưng quên xóa import; copy import từ file khác; import static của Mockito/JUnit/AssertJ (`org.mockito.Mockito.when`, `org.mockito.ArgumentMatchers.any`, `assertThrows`, ...) trong file test nhưng method tương ứng không còn được gọi.
+- Cũng tính vi phạm: import trùng lặp, import từ chính package hiện tại (redundant), và import `java.lang.*` (luôn thừa).
+- Cách kiểm: với mỗi dòng import, tìm định danh cuối (tên class / tên method static) trong phần thân file; nếu không xuất hiện lần nào (ngoài chính dòng import) → xóa dòng import đó. Ưu tiên dùng "Organize Imports" của IDE, nhưng vẫn phải rà lại import static vì IDE đôi khi giữ lại nhầm.
+- Cách sửa: **xóa hẳn** dòng import thừa. KHÔNG thay bằng import wildcard `*` để né lỗi.
+
+## Mở rộng Sonar
+
+Đây là các rule trọng tâm; nếu thấy các code smell Sonar phổ biến khác trong file vừa sửa (vd `java:S1186` empty method, `java:S106` dùng `System.out`, unused variable, `java:S2589` điều kiện luôn đúng/sai) thì cũng nên sửa kèm và nêu trong báo cáo. Khi không chắc một literal/method có vượt ngưỡng hay không, ưu tiên đếm cụ thể trước khi kết luận.
+
+# SENIOR BACKEND RULES — DIGITAL FINANCE SYSTEM (OPTIMIZED FOR LLM)
+
+> Stack: Java 17+ · Spring Boot 3.x · JPA/Hibernate · Keycloak · PostgreSQL/Oracle.
+> Nguyên tắc xuyên suốt: TIỀN KHÔNG ĐƯỢC SAI — đúng trước, nhanh sau.
+
+## 1. XỬ LÝ TIỀN TỆ & SỐ HỌC
+* **CẤM `double`/`float` cho tiền**: bắt buộc `BigDecimal` (Java) / `DECIMAL`,`NUMERIC` (DB). Áp cho cả tỷ giá, lãi suất, phí, phần trăm.
+* **So sánh BigDecimal**: dùng `compareTo()`, CẤM `equals()` (`0.10` vs `0.1` không equals nhưng compareTo = 0).
+* **Rounding tường minh**: mọi phép chia/nhân tỷ lệ phải khai báo `RoundingMode` + scale rõ ràng (thường `HALF_UP`/`HALF_EVEN` theo quy định nghiệp vụ). CẤM để default.
+* **Tiền luôn đi kèm currency**: model dạng `(amount, currencyCode)`; CẤM cộng trừ hai amount khác currency mà không qua conversion tường minh.
+* **CẤM tự tính lại số dư từ phía client gửi lên**: amount/balance chỉ tính từ dữ liệu server-side.
+
+## 2. IDEMPOTENCY & CHỐNG DOUBLE-PROCESSING
+* **Mọi API ghi tạo giao dịch tiền phải idempotent**: nhận `Idempotency-Key` (hoặc client request id) + unique constraint ở DB; request trùng key trả về kết quả lần đầu, KHÔNG xử lý lại.
+* **Chống double-submit ở tầng DB, không chỉ ở code**: unique constraint / insert-first pattern; CẤM chỉ dựa vào check-then-act trong code (race condition).
+* **Consumer message/event phải idempotent**: xử lý được message trùng (at-least-once delivery); dedupe bằng message id + bảng processed.
+* **Retry chỉ cho operation idempotent**: CẤM auto-retry cho call ghi không idempotent; timeout với đối tác thanh toán → trạng thái `PENDING/UNKNOWN` + job đối soát truy vấn lại, KHÔNG tự đoán thành công/thất bại.
+
+## 3. TRANSACTION & CONSISTENCY
+* **Ranh giới `@Transactional` gọn nhất có thể**: CẤM gọi API ngoài / gửi message / I/O chậm bên trong transaction DB.
+* **Hiểu proxy Spring**: `@Transactional` không có tác dụng khi self-invocation (gọi method cùng class); tách sang bean khác nếu cần.
+* **Update số dư phải chống race**: dùng optimistic lock (`@Version`) hoặc pessimistic lock (`SELECT ... FOR UPDATE`) / atomic update (`UPDATE ... SET balance = balance - :x WHERE balance >= :x`). CẤM read-modify-write không lock.
+* **Ghi sổ dạng ledger append-only**: giao dịch tiền ghi bút toán (entry) mới, CẤM update/delete bản ghi giao dịch đã ghi; sửa sai → bút toán đảo (reversal).
+* **Cross-service consistency**: dùng outbox pattern / saga cho luồng chạm nhiều service; CẤM two-phase tự chế bằng try-catch gọi API rollback tay.
+* **Trạng thái giao dịch là state machine tường minh**: khai báo transition hợp lệ (vd `INIT → PENDING → SUCCESS/FAILED → REVERSED`); CẤM set status tự do bằng setter.
+
+## 4. VALIDATION & INPUT
+* **Validate tại biên**: Bean Validation (`@Valid`, `@NotNull`, `@Positive`, `@Digits`) trên DTO request; business rule validate ở service.
+* **Amount phải check**: > 0, đúng scale theo currency, trong hạn mức (min/max, hạn mức ngày/user).
+* **CẤM bind entity JPA trực tiếp từ request** (mass assignment): luôn qua DTO + mapping tường minh.
+* **Whitelist, không blacklist**: enum/status/type từ client → parse về enum Java, CẤM nhận string tự do rồi so sánh chuỗi.
+
+## 5. BẢO MẬT & PHÂN QUYỀN
+* **AuthZ ở server cho từng resource**: check user có quyền trên CHÍNH resource đó (ownership check — chủ sở hữu account phải trùng principal đang đăng nhập), không chỉ check role. Chống IDOR: CẤM tin resource id client gửi lên.
+* **Method-level security**: `@PreAuthorize` cho operation nhạy cảm; mapping role Keycloak tập trung một chỗ, CẤM rải string role khắp code.
+* **CẤM secret trong code/config commit**: credential, API key qua env/vault. Log/response CẤM chứa secret.
+* **SQL Injection**: chỉ dùng parameterized query / JPA; CẤM nối chuỗi SQL từ input — kể cả trong native query, `ORDER BY` động phải whitelist.
+* **Dữ liệu nhạy cảm at-rest**: PAN/số thẻ/CCCD mã hóa hoặc tokenize theo chuẩn (PCI-DSS nếu chạm thẻ); CẤM lưu CVV dưới mọi hình thức.
+
+## 6. AUDIT & LOGGING
+* **Audit trail bất biến cho mọi thao tác tiền + thao tác admin**: ai (userId) — làm gì (action) — trên gì (resource) — khi nào (UTC) — từ đâu (IP/channel) — giá trị trước/sau. Ghi append-only.
+* **CẤM log PII/dữ liệu nhạy cảm dạng thô**: số thẻ, mật khẩu, token, OTP, số dư kèm định danh → mask (`****1234`) hoặc bỏ. Review mọi lệnh log có object dump.
+* **Correlation id xuyên suốt**: mọi request có trace/correlation id, propagate qua service call + message; log qua MDC.
+* **Log có cấu trúc**: key-value/JSON, đủ context để trace 1 giao dịch end-to-end bằng transaction id.
+* **CẤM `System.out.println`** (Sonar S106): dùng logger.
+
+## 7. THỜI GIAN & DỮ LIỆU
+* **Lưu UTC, hiển thị theo timezone user**: DB dùng `timestamptz`/UTC; code dùng `Instant`/`OffsetDateTime`; CẤM `java.util.Date`, `Calendar` cho code mới.
+* **Cut-off time nghiệp vụ khai báo tường minh** (ngày giao dịch khác ngày calendar): logic ngày làm việc/cut-off tách thành service riêng, có test.
+* **Soft-delete cho dữ liệu nghiệp vụ**: CẤM hard delete bản ghi tài chính; dùng flag + audit.
+* **DB migration qua Flyway/Liquibase**: CẤM sửa schema tay; migration script immutable sau khi merge — sửa bằng script mới.
+
+## 8. API DESIGN & ERROR HANDLING
+* **Error response chuẩn hóa 1 format**: error code nghiệp vụ (catalog tập trung) + message + traceId; CẤM trả stack trace / message exception thô ra client.
+* **HTTP status đúng nghĩa**: 400 validate, 401/403 auth, 404 not found, 409 conflict/duplicate, 422 business rule, 500 lỗi hệ thống. CẤM trả 200 kèm `success: false` cho lỗi.
+* **Exception handling tập trung**: `@RestControllerAdvice`; CẤM try-catch nuốt exception (catch rồi chỉ log, không rethrow / không chuyển trạng thái).
+* **Response tiền dạng string hoặc number đúng scale**: thống nhất toàn hệ thống, tránh JS làm tròn sai với số lớn vượt độ chính xác 53 bit.
+* **Versioning + backward compatible**: thêm field mới phải optional; breaking change → version mới, CẤM sửa contract đang chạy.
+
+## 9. TÍCH HỢP ĐỐI TÁC (PAYMENT GATEWAY, CORE BANKING)
+* **Timeout + circuit breaker bắt buộc** cho mọi outbound call (Resilience4j): CẤM call không timeout.
+* **Lưu raw request/response với đối tác** (đã mask dữ liệu nhạy cảm) phục vụ đối soát/tranh chấp.
+* **Verify chữ ký/checksum callback**: mọi webhook/callback từ đối tác phải verify signature + chống replay (timestamp + nonce); CẤM tin callback chỉ vì đúng format.
+* **Đối soát (reconciliation) là bắt buộc**: mọi luồng tiền qua đối tác phải có job đối soát định kỳ so khớp 2 bên; thiết kế luồng phải trả lời được câu "lệch thì phát hiện thế nào?".
+
+## 10. CONCURRENCY & JOBS
+* **Scheduled job chạy multi-instance phải có distributed lock** (ShedLock/DB lock): CẤM `@Scheduled` trần khi deploy nhiều node.
+* **Job xử lý batch phải resumable + idempotent**: chết giữa chừng chạy lại không double-process (checkpoint/status per record).
+* **CẤM shared mutable state trong bean singleton**: service Spring stateless; cần state → DB/cache.
+
+## 11. TESTING
+* **Bắt buộc test cho logic tiền**: tính phí, lãi, làm tròn, chia tách — test boundary (0, âm, max, scale lẻ, currency không decimal như VND/JPY).
+* **Test concurrency cho luồng số dư**: ít nhất 1 test chứng minh không âm quỹ / không double-spend khi 2 request song song.
+* **Test state machine**: mọi transition không hợp lệ phải bị chặn.
+* **Integration test dùng Testcontainers** (DB thật), không chỉ H2 — hành vi lock/constraint khác nhau giữa DB.

package/skills/NTAPP/rules/fe-angular-rules.md

@@ -0,0 +1,96 @@
+# Rule khi code Frontend (Angular)
+
+Các rule dưới đây CHỈ áp dụng khi sửa/tạo file Frontend Angular (`*.component.ts` và các `*.ts` thuộc FE, `*.html`, `*.scss`, `*.css`, `*.sass`, `*.less`). Với file Backend hoặc file không thuộc FE thì bỏ qua toàn bộ mục này. Cách áp dụng giống rule Sonar: review phần code FE đã đụng tới TRƯỚC khi báo hoàn thành, phát hiện vi phạm thì sửa luôn và nêu trong báo cáo cuối.
+
+# SENIOR ANGULAR & FRONTEND RULES (OPTIMIZED FOR LLM)
+
+> Stack: Angular 16+ · ngx-translate · Keycloak · SCSS.
+> Mỗi rule CẤM đều có mục **Ngoại lệ** — ngoài ngoại lệ đó ra, không được vi phạm.
+> [Legacy ≤15] = chỉ áp dụng khi project dùng Angular 15 trở xuống.
+
+## 1. RXJS & DECLARATIVE PROGRAMMING
+* **CẤM nested subscription**: dùng higher-order operator để flatten. Chọn đúng operator theo ngữ cảnh:
+  - `switchMap`: search/autocomplete/param đổi liên tục (hủy request cũ).
+  - `exhaustMap`: submit button, action ghi dữ liệu (chống double-click).
+  - `concatMap`: các thao tác ghi cần đúng thứ tự.
+  - `mergeMap`: chỉ khi cần chạy song song và thứ tự không quan trọng.
+* **Ưu tiên declarative**: expose stream qua `async` pipe hoặc `toSignal()`; hạn chế subscribe trong TS.
+  - **Ngoại lệ được subscribe trong TS**: side-effect thuần (navigation, toast, dialog, gọi API ghi) — khi đó BẮT BUỘC áp Rule 2.
+* **Cache API dùng chung**: `shareReplay({ bufferSize: 1, refCount: true })`.
+  - Master-data/config cần sống suốt session → cache trong service `providedIn: 'root'` (BehaviorSubject/signal), không dựa vào `refCount`.
+* **Error không được kill stream**: `catchError` đặt ở INNER observable (bên trong `switchMap`/...), không đặt ở outer pipe của stream long-lived (valueChanges, router events).
+
+## 2. QUẢN LÝ SUBSCRIPTION & MEMORY LEAK
+* **Mọi subscribe trong TS phải tự hủy**: dùng `takeUntilDestroyed(this.destroyRef)` — đặt CUỐI CÙNG trong `pipe()`.
+  - [Legacy ≤15] dùng `takeUntil(this.destroy$)` + `destroy$.next(); destroy$.complete();` trong `ngOnDestroy()`.
+  - **Ngoại lệ**: observable chắc chắn complete ngay (HttpClient one-shot trong service không giữ tham chiếu component) — vẫn khuyến khích áp dụng cho đồng nhất.
+* **Hủy tài nguyên ngoài trong `ngOnDestroy()`**: `setTimeout`/`setInterval`, `Renderer2.listen`, ResizeObserver/IntersectionObserver, instance thư viện ngoài (chart, map, editor...).
+
+## 3. HIỆU NĂNG & CHANGE DETECTION
+* **OnPush bắt buộc**: `changeDetection: ChangeDetectionStrategy.OnPush` cho mọi component. State thay đổi qua signal / async pipe / immutable `@Input`.
+  - `markForCheck()` là biện pháp CUỐI CÙNG — chỉ dùng cho callback từ thư viện ngoài chạy ngoài zone. KHÔNG rắc `markForCheck()` như thói quen.
+* **CẤM gọi function/getter trong template binding**: `{{ calc() }}`, `[disabled]="check()"` chạy lại mỗi CD cycle. Thay bằng: pure pipe, signal `computed()`, hoặc biến tính sẵn khi data đổi.
+  - **Ngoại lệ**: signal call (`count()`), hàm `track` của `@for`.
+* **Vòng lặp**: dùng `@for (item of items; track item.id)`. [Legacy ≤15] `*ngFor` + `trackBy` bắt buộc.
+* **Chạy ngoài zone**: event tần suất cao (scroll, mousemove, drag, polling) → `ngZone.runOutsideAngular()`; chỉ `ngZone.run()` khi cần cập nhật UI.
+* **Lazy loading bắt buộc**: route-level lazy load cho mọi feature (`loadChildren`/`loadComponent`). Block render nặng dưới màn hình đầu → `@defer`.
+* **List dài (> ~100 item)**: dùng `cdk-virtual-scroll-viewport`, không render thẳng.
+
+## 4. TYPESCRIPT STRICTNESS
+* **CẤM `any`**: dùng type tường minh hoặc `unknown` + type guard. Response API phải có interface/type riêng — CẤM dùng object trần.
+* **tsconfig**: bật `strict: true` + `strictTemplates: true`. Code mới không được sinh lỗi strict.
+* **CẤM non-null assertion `!` tràn lan**: chỉ dùng khi có bằng chứng rõ ràng (đã guard trước đó); ưu tiên optional chaining + default value.
+* **DI**: dùng `inject()` thay constructor injection cho code mới. [Legacy ≤15] constructor injection.
+
+## 5. FORMS & CUSTOM CONTROLS (CVA)
+* **Strict Typed Forms**: `FormGroup`/`FormControl` phải có type tường minh (`FormControl<string>`, `FormGroup<LoginForm>`). CẤM untyped form cho code mới.
+* **ControlValueAccessor**: mọi component dùng chung trong form phải implement đủ 4 method: `writeValue`, `registerOnChange`, `registerOnTouched`, `setDisabledState`.
+* **Normalize input trong `writeValue()`**: CVA không được giả định kiểu dữ liệu bên ngoài truyền vào — normalize về internal shape (vd nhận cả `['A','B']` lẫn `'A,B'` thì convert về một dạng duy nhất trước khi xử lý).
+* **Validator dùng chung**: tách thành function/class riêng tái sử dụng, không viết inline lặp lại.
+
+## 6. KIẾN TRÚC COMPONENT & STATE
+* **Smart / Presentation tách bạch**: Presentation (dumb) chỉ dùng `@Input`/`@Output` (hoặc `input()`/`output()`), không inject service nghiệp vụ.
+* **Facade Pattern**: logic xử lý, state, gọi API tách khỏi `.component.ts` sang facade service. Component < 400 dòng.
+* **Local state**: ưu tiên signal (`signal`, `computed`); RxJS cho async stream (API, events); `toSignal()`/`toObservable()` ở biên chuyển đổi.
+* **CẤM logic trong constructor**: constructor chỉ để DI; init logic đặt ở `ngOnInit` hoặc field initializer.
+
+## 7. BẢO MẬT (XSS + KEYCLOAK)
+* **CẤM `bypassSecurityTrust*`** khi chưa có review rõ ràng nguồn dữ liệu — mỗi chỗ dùng phải có comment giải thích vì sao an toàn.
+* **Hạn chế `[innerHTML]`**: chỉ cho nội dung đã qua sanitize của Angular; CẤM nối chuỗi HTML thủ công từ input người dùng.
+* **CẤM thao tác DOM trực tiếp**: không `document.*`, không đọc/ghi `ElementRef.nativeElement` tùy tiện — dùng `Renderer2` / binding.
+* **Guards & Resolvers**: kiểm tra quyền bằng functional guard (`CanActivateFn`); preload data màn hình bằng `ResolveFn` để tránh blank/flashing UI.
+* **HTTP Interceptor**: chèn Keycloak access token + xử lý token refresh tập trung tại interceptor. CẤM gắn token thủ công ở từng service; CẤM tự lưu token vào `localStorage`.
+* **Phân quyền UI**: ẩn/hiện element theo role qua directive/service dùng chung, không if-else role rải rác trong component.
+
+## 8. ERROR HANDLING
+* **Global**: có `ErrorHandler` custom + interceptor xử lý lỗi HTTP tập trung (401 → refresh/logout, 403 → thông báo quyền, 5xx → toast chung).
+* **CẤM nuốt lỗi im lặng**: `catchError` phải log hoặc chuyển trạng thái UI (error state), không `catchError(() => EMPTY)` trống.
+* **UI phải có 3 trạng thái**: loading / error / empty cho mọi màn hình gọi API — không để user nhìn màn hình trắng.
+
+## 9. ĐA NGÔN NGỮ (i18n)
+* **CẤM hardcode text**: 100% text hiển thị đi qua i18n.
+* **Key JSON**: `UPPERCASE_SNAKE_CASE`, phân cấp dot notation `MODULE.SCREEN.ELEMENT`.
+* **CẤM key động hoàn toàn** (`translate.instant(prefix + variable)`): làm mất khả năng trace key. Nếu cần map theo enum → khai báo map tường minh key literal.
+* **Binding**: pipe `translate` trong HTML; trong TS dùng `TranslateService` dạng stream (`get`/`stream`) — `instant` chỉ khi chắc chắn file dịch đã load.
+* **Interpolation**: dùng tham số hóa (`"KEY": "Hello {{name}}"`); CẤM cộng chuỗi thủ công trong TS.
+
+## 10. SCSS / CSS
+* **Style scoping**: style component đặt trong `:host`. `::ng-deep` (deprecated) chỉ để đè UI thư viện ngoài và PHẢI đi kèm `:host ::ng-deep`; ưu tiên CSS variables xuyên component nếu thư viện hỗ trợ.
+* **Nesting tối đa 3 cấp**.
+* **CẤM hardcode màu** (HEX/RGB/tên màu): dùng CSS variables của theme (Light/Dark).
+* **CẤM animation gây reflow** (`width`/`height`/`margin`/`top`/`left`): dùng `transform` (`translate3d`, `scale`) + `opacity`.
+* **CẤM inline style tĩnh** `style="..."` và tiêm HTML style qua TS (`innerHTML`).
+  - **Ngoại lệ**: binding giá trị runtime (`[style.width.%]="progress"`, `[style.--var]`) là hợp lệ.
+* **CẤM viết CSS trong metadata** `styles: [...]`: tách file `.scss` riêng.
+* **Layout**: Grid/Flexbox first; `clamp()`/`calc()` cho fluid layout; CSS variables cho theming.
+
+## 11. ACCESSIBILITY (A11Y)
+* **Semantic HTML**: element có hành vi click phải là `<button>`/`<a>`, CẤM `div (click)`.
+* **Ảnh/icon có nghĩa** phải có `alt`/`aria-label`; icon trang trí → `aria-hidden="true"`.
+* **Keyboard**: component tương tác custom (dropdown, modal, tab) phải điều khiển được bằng bàn phím + quản lý focus (trap focus trong modal, trả focus khi đóng).
+* **CẤM bỏ outline focus** mà không có focus style thay thế.
+
+## 12. NAMING & CẤU TRÚC
+* **File theo feature**: `feature/` chứa components / services / models của riêng feature; dùng chung → `shared/`; singleton toàn app → `core/`.
+* **Đặt tên**: component `PascalCase` + suffix theo loại (`UserListComponent`, `AuthFacade`, `DateFormatPipe`); observable suffix `$`; signal không suffix.
+* **Barrel/import**: CẤM feature import chéo feature khác trực tiếp — đi qua `shared`/`core` hoặc public API của feature.

package/skills/NTAPP/rules/rules.json

@@ -0,0 +1,4 @@
+{
+  "be": "be-finance-rules.md",
+  "fe": "fe-angular-rules.md"
+}

package/skills/NTAPP/scripts/antigravity/README.md

@@ -0,0 +1,90 @@
+# Antigravity host for VSAF — PoC driver (`vsaf-agy-driver.cjs`)
+
+Run NTAPP VSAF workflows through **Antigravity CLI (`agy -p`)** instead of Claude Code,
+**deterministically**. Additive & hybrid: this does NOT touch the Claude path
+(`/NTAPP-vsaf-run` MCP glue) — Claude stays primary, Antigravity is the fallback host.
+
+## Why this design
+
+VSAF's engine (`@vsaf/core`) is a **host-agnostic DAG orchestrator**. Today the step loop
+is driven by an LLM following `NTAPP-vsaf-run/workflow.md` over the `vsaf_start`/`vsaf_next`
+MCP tools — that LLM-driven loop is the consistency-weak link.
+
+This driver replaces the loop with **code**: it imports `@vsaf/core`, runs
+`DAGExecutor.step()` in a plain JS `while` loop, and executes each yielded AI node by
+shelling out to `agy -p`. Orchestration (deps, fanout, concurrency caps, retries,
+skip/idempotency) is deterministic engine code; only per-node *content* is model-driven.
+Each AI node = one fresh `agy -p` process = a literal fresh context — exactly what VSAF
+fanout sub-tasks want.
+
+## What the PoC proves (run on this repo, June 2026)
+
+- `ad-hoc-per-unit` (12 nodes): driver walked the full real DAG in dependency order to
+  `done`, exit 0. Orchestration is host-free.
+- `slice-loop` fanout: `dd-per-unit` (mc:5) yielded 4 parallel sub-nodes
+  (`~fanout~0..3`); `ds-build-per-fe-unit` (mc:1) yielded one at a time. **The engine's
+  `max_concurrency` is honored for free** — it only yields ≤ mc; the driver runs what it gets.
+
+LIVE proven on agy **1.0.14**: the echo smoke workflow ran end-to-end through real agy
+(Gemini), and `/NTAPP-agy-vsaf-slice-loop SLICE-01 --dry-run` invoked via `agy -p` drove the
+full slice-loop DAG to `FINISHED: done`.
+
+## Usage — 3 layers (simplest last)
+
+```bash
+# (a) raw driver
+node scripts/antigravity/vsaf-agy-driver.cjs NTAPP/ad-hoc-per-unit --args "<intent>" --yolo
+
+# (b) wrapper (auto-prefixes NTAPP/, default --yolo, --yolo off in dry-run)
+./scripts/antigravity/vsaf-agy ad-hoc-per-unit "<intent>"
+./scripts/antigravity/vsaf-agy slice-loop SLICE-01 --dry-run
+
+# (c) slash command inside agy (after install-agy-skills.sh) — the requested UX
+#     type in the agy TUI, or: agy -p "/NTAPP-agy-vsaf-slice-loop SLICE-01"
+/NTAPP-agy-vsaf-slice-loop SLICE-01
+/NTAPP-agy-vsaf-ad-hoc-per-unit "Thêm endpoint health-check"
+```
+
+Driver flags: `--args` `--project` `--dry-run` `--agy <bin>` `--model` `--print-timeout` `--yolo` `--max-parallel N` `--no-hooks`.
+Wrapper env: `VSAF_AGY_MODEL`, `VSAF_AGY_NO_YOLO=1`. Hooks ON by default — see `hooks/README.md`; `--dry-run` prints the hook plan per node.
+
+## Install (machine-local, idempotent)
+
+```bash
+bash scripts/antigravity/install-agy-skills.sh   # vsaf-agy -> ~/.local/bin ; glue skills -> ~/.gemini/config/skills
+```
+
+## Verified facts (agy 1.0.14) — flags in `buildAgyArgs()`
+
+- `-p`/`--print` non-interactive (accepts large prompts as argv); **no `--output-format`** — print emits plain text, so the driver never sets `structuredOutput` (neither target workflow uses `outputFormat`).
+- `--model "<display name>"` e.g. `"Gemini 3.1 Pro (High)"`, `"Gemini 3.5 Flash (Low)"` (from `agy models`).
+- `--print-timeout` (agy default 5m; driver default 15m for long dev/review nodes).
+- `--dangerously-skip-permissions` (YOLO) needed for headless; or add the repo to `trustedWorkspaces`.
+- Driver `spawn` sets `stdio[0]='ignore'` — a default stdin pipe makes agy print mode hang.
+- **Skill discovery**: agy 1.0.14 loads user skills only from global `~/.gemini/config/skills/`; in-repo `.agents/skills/` is NOT auto-discovered. Slash command = a skill activated by its `description`.
+- **Native hooks**: agy 1.0.14 HAS hooks — 5 events `PreToolUse / PostToolUse / PreInvocation / PostInvocation / Stop` (proto `exa.hooks_pb`; entry `{name,command,args,event,matcher,timeout_millis}`; can block with a reason). No `agy hooks` CLI subcommand and no hooks file until you create one — managed via the interactive `/hooks` form. We use agy-native only for Hook 9 (per-edit PostToolUse); the workflow gate hooks live at driver node boundaries instead (see `hooks/README.md` for why).
+
+## Remaining porting work (not in this PoC)
+
+1. **Leaf skills** → the workflow nodes call `/ntapp-technical-design`, `/NTAPP-feature-design`,
+   `/bmad-dev-story`, etc. (125 skills in `.agent/skills/`). For a non-dry-run run, expose these
+   to agy's global dir `~/.gemini/config/skills/` (symlink per skill), EXCLUDING the Claude
+   `NTAPP-vsaf-*` glue (agy uses the in-repo `agy-skills/` versions instead). Audit skills that
+   spawn parallel sub-agents (NTAPP-feature-design F0–F4, NTAPP-ux-design, qa pipelines) —
+   repoint their subagent step to Antigravity subagents. *(Glue slash commands: DONE — see
+   `agy-skills/` + `install-agy-skills.sh`.)*
+2. **Hooks** → **DONE for slice-loop + ad-hoc-per-unit** (see `hooks/README.md`). The 8 gate
+   hooks those workflows reference (3,4,5,5b,6,7,8,10) fire at **driver node boundaries**
+   (`hooks/run-hooks.cjs`, reusing `hooks/*.sh` verbatim) — a blocking gate halts the run.
+   Hook 9 (build_loop, per-edit) runs as a driver post-node sweep AND as an **agy-native
+   PostToolUse** hook (`hooks/agy-posttool-shim.cjs`, matcher `write_to_file|replace_file_content`).
+   git pre-commit (Rules 1–9) reused unchanged. Per-unit `HOOK_TARGET_ID` removes the parallel
+   fanout race (Claude path unchanged — `hooks/tests/run.sh` still 15/15).
+   *Remaining*: the other ~7 event hooks (1,2,12,13,14,15,16) for the batch-phase workflows
+   (create_prd_epics, reproduce_architecture, slice-planning, feature-triage) — same pattern.
+3. **MCP** → move gitnexus/figma/etc. into `~/.gemini/config/mcp_config.json`.
+4. **Engine** → optionally upstream this as a `provider: antigravity` patch under
+   `scripts/engine-patches/` so `vsaf run` handles AI nodes headlessly (today it rejects them).
+5. **Hardening** → convert `resolve-slice-units` / `resolve-slice-modules` (and ad-hoc's
+   `.current-unit-id` plumbing) from `prompt:` → `bash:` nodes so the engine runs them
+   directly (zero LLM) — removes fanout-prep flakiness regardless of host.

package/skills/NTAPP/scripts/antigravity/agy-skills/NTAPP-agy-vsaf-ad-hoc-per-unit/SKILL.md

@@ -0,0 +1,33 @@
+---
+name: NTAPP-agy-vsaf-ad-hoc-per-unit
+description: 'Antigravity host for the NTAPP ad-hoc-per-unit workflow (Lane 4). Activate when the user types "/NTAPP-agy-vsaf-ad-hoc-per-unit <intent | unit_id>" or asks to spec-build-demo one independent unit. Runs the deterministic VSAF→Antigravity driver (vsaf-agy). Do NOT interpret the workflow yourself.'
+---
+
+# NTAPP VSAF Ad-hoc-per-unit — Antigravity host
+
+Antigravity counterpart of the Claude `/NTAPP-vsaf-ad-hoc-per-unit` glue. Runs the
+deterministic `vsaf-agy` driver (it imports `@vsaf/core`, walks the DAG in code,
+executes each node via `agy -p`). Your only job is to launch it.
+
+## What to do when activated
+
+The user invokes: `/NTAPP-agy-vsaf-ad-hoc-per-unit <intent | file-path | unit_id> [extra flags]`.
+
+Do EXACTLY this, nothing else:
+
+1. Treat everything after the command name as ARGS (the intent/unit id plus any flags).
+   If the intent contains spaces, keep it as a single quoted string.
+2. From the current project root, run this terminal command and stream its full output:
+
+   ```
+   vsaf-agy ad-hoc-per-unit "<ARGS>"
+   ```
+
+3. Report the driver's final status (`FINISHED: done|failed`) and changed artifacts.
+   Do NOT re-run or re-interpret nodes yourself.
+
+## Notes
+
+- Precondition: WBS locked; unit resolves into the WBS (the driver's first node appends it if new).
+- Add `--dry-run` to preview without calling agy per node.
+- If `vsaf-agy: command not found`, run `bash scripts/antigravity/install-agy-skills.sh` first.

package/skills/NTAPP/scripts/antigravity/agy-skills/NTAPP-agy-vsaf-slice-loop/SKILL.md

@@ -0,0 +1,39 @@
+---
+name: NTAPP-agy-vsaf-slice-loop
+description: 'Antigravity host for the NTAPP slice-loop workflow. Activate when the user types "/NTAPP-agy-vsaf-slice-loop <slice_id>" or asks to run / build a slice end-to-end (e.g. SLICE-01). Runs the deterministic VSAF→Antigravity driver (vsaf-agy); it orchestrates Stage 3→4→5 for the slice. Do NOT interpret the workflow yourself.'
+---
+
+# NTAPP VSAF Slice Loop — Antigravity host
+
+This is the Antigravity counterpart of the Claude `/NTAPP-vsaf-slice-loop` glue.
+Difference: it does NOT drive the `vsaf_start`/`vsaf_next` MCP loop. It runs a
+**deterministic code driver** (`vsaf-agy`) that imports `@vsaf/core`, walks the DAG
+in code, and executes each node via `agy -p`. Your only job is to launch it.
+
+## What to do when activated
+
+The user invokes: `/NTAPP-agy-vsaf-slice-loop <slice_id> [extra flags]`
+(e.g. `/NTAPP-agy-vsaf-slice-loop SLICE-01`, or `... SLICE-01 --dry-run`).
+
+Do EXACTLY this, nothing else:
+
+1. Treat everything after the command name as ARGS (the slice id plus any flags).
+2. From the current project root, run this terminal command and stream its full output:
+
+   ```
+   vsaf-agy slice-loop <ARGS>
+   ```
+
+   `vsaf-agy` is on PATH (installed by `scripts/antigravity/install-agy-skills.sh`).
+   It runs `scripts/antigravity/vsaf-agy-driver.cjs` which owns all orchestration
+   (dependencies, fanout, max_concurrency, idempotent skip, resume).
+
+3. When it finishes, report the driver's final status line (`FINISHED: done|failed`)
+   and any changed artifacts. Do NOT re-run, re-order, or re-interpret nodes yourself —
+   the driver is the source of truth.
+
+## Notes
+
+- Preconditions (same as Claude lane): WBS locked + slice plan approved + `<slice_id>` exists.
+- Add `--dry-run` to preview the DAG without calling agy per node.
+- If `vsaf-agy: command not found`, run `bash scripts/antigravity/install-agy-skills.sh` first.