spec-lite 1.1.8 → 1.1.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spec-lite",
-  "version": "1.1.8",
+  "version": "1.1.10",
   "description": "Spec-driven development kit for Claude Code",
   "type": "module",
   "bin": {

package/skills/ado-create/SKILL.md

@@ -26,20 +26,24 @@ description: Tạo Feature và User Story tickets trên ADO từ spec.md của m
 Chưa có .claude/ado.yaml. Hãy chạy /ado-config trước.
 ```
 
-**Xác định current user:** Đọc `git config user.email`. Tìm email này trong toàn bộ `team.PE + team.SE + team.DE` của ado.yaml.
+**Xác định current user:** Gọi `mcp__azure-devops__core_get_identity_ids` với descriptor `me` hoặc `[me]` để lấy identity của user đang authenticated với ADO. Từ kết quả trả về, lấy `providerDisplayName` hoặc `subjectDescriptor` rồi tìm email tương ứng trong toàn bộ `team.PE + team.SE + team.DE` của ado.yaml.
 
-**Nếu tìm thấy** → dùng email đó làm assignee.
+**Nếu tìm thấy** → dùng email đó làm assignee, dùng identity ID từ kết quả MCP (không cần gọi lại).
 
-**Nếu không tìm thấy** → hỏi bằng `AskUserQuestion`:
+**Nếu không resolve được (tool lỗi hoặc không match được email)** → hỏi bằng `AskUserQuestion`:
 
 ```
 question: "Bạn là ai trong team? (dùng để assign ticket)"
 options: [mỗi email trong team.PE + team.SE + team.DE thành 1 option]
 ```
 
-Sau khi xác định email, resolve thành ADO identity ID bằng `mcp__azure-devops__core_get_identity_ids`.
+Sau khi user chọn email, resolve thành ADO identity ID bằng `mcp__azure-devops__core_get_identity_ids`.
 
-Nếu resolve thất bại → tiếp tục nhưng không set assignee; thông báo để user tự assign trên ADO sau.
+Nếu resolve thất bại → **dừng và báo lỗi**. Không tạo ticket nếu không xác định được assignee:
+
+```
+Không resolve được ADO identity ID cho <email>. Kiểm tra lại PAT token hoặc quyền truy cập ADO rồi thử lại.
+```
 
 ---
 
@@ -300,6 +304,7 @@ Tìm section `## ADO Tickets` trong file.
 
 Trước khi tạo ticket:
 - [ ] `.claude/ado.yaml` tồn tại và `epic.id` là số nguyên hợp lệ
+- [ ] Current ADO user đã được xác định và resolve thành identity ID — **nếu không xác định được thì dừng, không tạo ticket không có assignee**
 - [ ] Integration được chọn có `spec.md` chứa ít nhất 1 Feature
 - [ ] `frd.md` tìm được cho mỗi Feature (cảnh báo nếu thiếu, không dừng)
 

package/skills/build/SKILL.md

@@ -38,6 +38,38 @@ Chọn số integration:
   - Nếu chỉ có 1 integration có task pending → chọn luôn, không hỏi
   - Nếu tất cả đều done → thông báo: "Tất cả integration đã hoàn thành."
 
+### Bước 0a: Gate check — plan.md phải approved
+
+Đọc frontmatter của `specs/integrations/{slug}/plan.md`.
+
+Nếu `plan.md` chưa tồn tại:
+
+> ⛔ Chưa có `plan.md` cho integration **{slug}**.
+>
+> Để tiếp tục:
+> 1. Chạy `/plan` để tạo plan.md và todo.md
+> 2. Review và đổi `status: approved` trong plan.md
+>
+> Sau đó chạy lại `/build`.
+
+**Dừng hoàn toàn.**
+
+Nếu `plan.md` tồn tại nhưng `status` **không phải** `approved`:
+
+> ⛔ `plan.md` của integration **{slug}** chưa được approve (status hiện tại: `{status}`).
+>
+> Để tiếp tục:
+> 1. Review `specs/integrations/{slug}/plan.md`
+> 2. Đổi `status: draft` → `status: approved` trong frontmatter
+>
+> Sau khi approve xong, chạy lại `/build`.
+
+**Dừng hoàn toàn. Không tiếp tục.**
+
+Nếu `status: approved` → tiếp tục sang Bước 0b.
+
+---
+
 ### Bước 0b: Tạo branch và chọn chế độ worktree
 
 **Tạo branch (bắt buộc):**

package/skills/plan/SKILL.md

@@ -53,6 +53,32 @@ Nếu plan.md / todo.md đã tồn tại → hỏi trước khi ghi đè.
 
 ---
 
+### Bước 1b: Gate check — spec.md và tech.md phải approved
+
+Đọc frontmatter của `specs/integrations/{slug}/spec.md` và `specs/integrations/{slug}/tech.md`.
+
+Nếu `tech.md` chưa tồn tại → coi như `status: missing`.
+
+Kiểm tra từng file — nếu **bất kỳ file nào** chưa `approved`:
+
+> ⛔ Không thể tạo plan.md — các artifact sau chưa được approve:
+>
+> - `spec.md` — status: `{status}` ← chỉ liệt kê nếu chưa approved
+> - `tech.md` — status: `{status}` ← chỉ liệt kê nếu chưa approved hoặc chưa tồn tại
+>
+> Để tiếp tục:
+> 1. Review từng file chưa approved (chạy `/spec-tech` trước nếu `tech.md` chưa có)
+> 2. Apply Cascade Proposals (nếu có)
+> 3. Đổi `status` thành `approved` trong frontmatter
+>
+> Sau khi approve xong, chạy lại `/plan`.
+
+**Dừng hoàn toàn. Không tiếp tục.**
+
+Nếu cả hai đều `approved` → tiếp tục sang Bước 2.
+
+---
+
 ### Bước 2: Load và surface context
 
 Load:

package/skills/review/SKILL.md

@@ -31,5 +31,139 @@ git diff $(git merge-base HEAD $BASE)...HEAD
 
 ## Sau khi review
 
-- **Có Critical/Important:** Dừng, không tiếp tục `/build`. Sửa và re-review.
-- **Chỉ Suggestion hoặc không có finding:** Approve, tiếp tục `/build` task tiếp theo.
+### Chỉ Nit/Optional/FYI hoặc không có finding
+
+Approve và thông báo tiếp tục `/build` task tiếp theo.
+
+### Có Critical hoặc Important
+
+**Bước 1 — Tổng hợp findings thành tasks:**
+
+Nhóm các findings thành tasks cụ thể có thể implement được. Mỗi finding Critical/Important là ít nhất một task. Đặt tên task bắt đầu bằng động từ, mô tả rõ việc phải sửa.
+
+**Bước 2 — Đề xuất thêm phase mới vào plan.md và todo.md:**
+
+Hiển thị preview phase sẽ được thêm:
+
+```
+Phát hiện {N} Critical / {M} Important findings. Đề xuất thêm phase mới:
+
+Phase Review: Fix review findings
+
+  T0XX · Fix {mô tả finding 1}
+  T0XY · Fix {mô tả finding 2}
+  ...
+
+Thêm phase này vào plan.md và todo.md, rồi chạy /build để fix?
+```
+
+Hỏi user bằng `AskUserQuestion`:
+
+```
+question: "Thêm phase Review vào plan.md/todo.md và chạy /build ngay?"
+options:
+  - label: "Thêm phase và chạy /build"
+    description: "Thêm các tasks fix findings vào plan.md và todo.md, rồi gọi /build để thực thi"
+  - label: "Thêm phase nhưng không chạy /build"
+    description: "Chỉ cập nhật plan.md và todo.md, tự chạy /build sau"
+  - label: "Lưu vào techdebt.md"
+    description: "Ghi findings vào specs/integrations/{slug}/techdebt.md để theo dõi sau, không fix ngay"
+```
+
+**Bước 3 — Nếu user chọn "Thêm phase" (cả hai option đầu):**
+
+Xác định số phase tiếp theo từ plan.md (đếm số `### Phase` hiện có, cộng 1).
+
+Xác định task ID tiếp theo từ plan.md (lấy task ID lớn nhất hiện có, cộng 1).
+
+Append vào `plan.md` (trước section `## Definition of Done`):
+
+```markdown
+### Phase {N}: Review Findings
+
+#### T{XXX} · {task title — bắt đầu bằng động từ}
+
+**Mục tiêu:** {mô tả finding cần fix}
+
+**depends:** {task ID cuối cùng trước phase này}
+
+**Các bước:**
+1. {bước fix cụ thể từ finding}
+
+**Verification:**
+- [ ] {điều kiện verify finding đã được fix}
+
+---
+```
+
+Append vào `todo.md`:
+
+```markdown
+### Phase {N}: Review Findings
+- [ ] T{XXX} — {mô tả task fix}
+```
+
+**Bước 4 — Nếu user chọn "Thêm phase và chạy /build":**
+
+Sau khi cập nhật xong plan.md và todo.md, invoke ngay `/build` để thực thi phase Review vừa thêm.
+
+**Bước 4 — Nếu user chọn "Lưu vào techdebt.md":**
+
+Lấy short SHA hiện tại: `git rev-parse --short HEAD`.
+
+Ghi findings vào `specs/integrations/{slug}/techdebt.md`.
+
+Nếu file chưa tồn tại, tạo mới từ `.claude/templates/integrations/techdebt-template.md`, điền `slug`, `title`, `created`.
+
+Append section mới vào cuối file (không ghi đè section cũ):
+
+```markdown
+
+## {YYYY-MM-DD} — {short-sha}
+
+- [ ] **Critical** — {mô tả finding 1}
+- [ ] **Important** — {mô tả finding 2}
+```
+
+Sau khi ghi xong, thông báo:
+
+```
+Đã lưu {N} findings vào specs/integrations/{slug}/techdebt.md.
+Integration được đánh dấu có tech debt chưa xử lý.
+```
+
+---
+
+## Đánh dấu techdebt đã resolved
+
+Khi `/build` hoàn thành **thành công** một Phase Review, nó phải tìm và mark các techdebt findings tương ứng là resolved.
+
+**Cơ chế liên kết:** Khi append Phase Review vào `plan.md` (Bước 3 ở trên), thêm field `resolves-techdebt` vào task đầu tiên của phase, trỏ đến section date trong techdebt.md:
+
+```markdown
+#### T{XXX} · Fix {mô tả finding 1}
+
+**Mục tiêu:** {mô tả finding cần fix}
+
+**depends:** {task ID cuối cùng trước phase này}
+
+**resolves-techdebt:** {YYYY-MM-DD} — {short-sha}
+
+**Các bước:**
+...
+```
+
+**Khi `/build` hoàn thành Phase Review:**
+
+1. Đọc field `resolves-techdebt` từ task đầu tiên của Phase Review trong `plan.md`
+2. Tìm section tương ứng trong `techdebt.md` (match theo `## {YYYY-MM-DD} — {short-sha}`)
+3. Với mỗi dòng `- [ ]` trong section đó → đổi thành `- [x]` và append ` *(resolved by T{XXX})*`:
+
+```markdown
+## 2026-05-08 — abc1234
+
+- [x] **Critical** — {mô tả finding 1} *(resolved by T005)*
+- [x] **Important** — {mô tả finding 2} *(resolved by T005)*
+```
+
+4. Nếu không tìm được field `resolves-techdebt` hoặc không tìm được section trong techdebt.md → bỏ qua, không báo lỗi.

package/skills/spec-tech/SKILL.md

@@ -56,6 +56,27 @@ Nếu integration đã có `tech.md` → hỏi trước khi tiếp tục:
 
 ---
 
+### Bước 1b: Gate check — spec.md phải approved
+
+Đọc frontmatter của `specs/integrations/{slug}/spec.md`.
+
+Nếu `status` **không phải** `approved`:
+
+> ⛔ `spec.md` của integration **{slug}** chưa được approve (status hiện tại: `{status}`).
+>
+> Để tiếp tục:
+> 1. Review `specs/integrations/{slug}/spec.md`
+> 2. Apply Cascade Proposals vào main artifacts (nếu có)
+> 3. Đổi `status: draft` → `status: approved` trong frontmatter
+>
+> Sau khi approve xong, chạy lại `/spec-tech`.
+
+**Dừng hoàn toàn. Không tiếp tục.**
+
+Nếu `status: approved` → tiếp tục sang Bước 2.
+
+---
+
 ### Bước 2: Load context
 
 Load các file sau:

package/templates/integrations/techdebt-template.md

@@ -0,0 +1,13 @@
+---
+slug: "{slug}"
+created: {YYYY-MM-DD}
+referenced_by:
+  - skills/review/SKILL.md > Bước 4 — Lưu vào techdebt.md
+---
+
+# Tech Debt — {title}
+
+## {YYYY-MM-DD} — {short-sha}
+
+- [ ] **Critical** — {mô tả finding}
+- [ ] **Important** — {mô tả finding}