dw-kit 1.9.0 → 1.9.1

package/.claude/skills/dw-kit-report/SKILL.md

@@ -17,6 +17,18 @@ $ARGUMENTS
 
 ---
 
+## Nguyên tắc riêng tư (ĐỌC TRƯỚC — bắt buộc)
+
+Skill này tạo issue trên **repo bên thứ ba `dv-workflow/dv-workflow`** (ngoài project của bạn). Vì vậy **KHÔNG bao giờ tự động đưa thông tin nhận dạng của project người report vào issue**:
+
+- ❌ KHÔNG kèm `project.name` từ config (đây là tên codebase/đơn vị của người dùng).
+- ❌ KHÔNG kèm tên task, mã ticket nội bộ (vd `PROJ-123`), đường dẫn file độc quyền, hay code snippet.
+- ✅ CHỈ gửi: loại feedback, component của **dw-kit**, mô tả vấn đề (từ `$ARGUMENTS`), và môi trường tối thiểu (OS, shell, dw version).
+
+Nếu chính `$ARGUMENTS` chứa thông tin nhận dạng nội bộ → cảnh báo người dùng và đề nghị lược bỏ. **Luôn preview + xin xác nhận trước khi gửi (Bước 4).**
+
+---
+
 ## Bước 1: Thu Thập Context
 
 **OS detection:**
@@ -24,9 +36,9 @@ $ARGUMENTS
 uname -s 2>/dev/null || echo "Windows"
 ```
 
-**dw version:** Đọc `_toolkit.core_version` từ `.dw/config/dw.config.yml`
+**dw version:** Đọc `_toolkit.core_version` từ `.dw/config/dw.config.yml` (KHÔNG đọc `project.name`).
 
-**Task context:** Kiểm tra `.dw/tasks/` — task nào đang In Progress?
+**Workflow context:** Có task In Progress không, và depth của nó (quick/standard/thorough) — chỉ để giúp reproduce. Ghi nhận CÓ/KHÔNG + depth, **KHÔNG lấy tên task**.
 
 ---
 
@@ -83,8 +95,8 @@ Ví dụ:
 [Nội dung từ $ARGUMENTS — đầy đủ, rõ ràng]
 
 ## Context
-- Task khi gặp vấn đề: [task name hoặc "general usage"]
-- Command/skill liên quan: [nếu biết]
+- Hoàn cảnh: ["general usage" hoặc "trong một task depth=X" — KHÔNG ghi tên task/ticket]
+- Command/skill dw-kit liên quan: [nếu biết]
 - Bước reproduce (nếu là bug):
   1. ...
   2. ...
@@ -95,12 +107,31 @@ Ví dụ:
 - [ ] Minor — annoying, có workaround dễ
 
 ---
-*Reported via `/dw:kit-report` | Project: [project.name từ config]*
+*Reported via `/dw:kit-report`*
 ```
 
+> Không thêm dòng `Project:` hay bất kỳ tên project/ticket/path nội bộ nào (xem Nguyên tắc riêng tư).
+
+---
+
+## Bước 4: Preview + Xác Nhận (BẮT BUỘC trước khi gửi)
+
+Issue sẽ được tạo trên repo bên thứ ba **công khai với dw-kit team**. Trước khi gửi, hiện đầy đủ title + body sẽ post và CHỜ người dùng đồng ý:
+
+```
+┌─ Sẽ gửi lên github.com/dv-workflow/dv-workflow ─────────────┐
+ TITLE: [title]
+
+ BODY:
+ [body đầy đủ]
+└─ Gõ "ok" để gửi · hoặc sửa nội dung trước ──────────────────┘
+```
+
+Trước khi hiện preview, **rà soát lần cuối**: nếu title/body (kể cả phần người dùng tự nhập trong `$ARGUMENTS`) còn chứa tên project, mã ticket nội bộ, đường dẫn file độc quyền, hay code snippet → cảnh báo rõ và đề nghị lược bỏ. **Chỉ sang Bước 5 sau khi người dùng xác nhận.**
+
 ---
 
-## Bước 4: Gửi Lên GitHub
+## Bước 5: Gửi Lên GitHub
 
 **Kiểm tra `gh` CLI:**
 ```bash
@@ -141,7 +172,7 @@ In ra:
 
 ---
 
-## Bước 5: Xác Nhận
+## Bước 6: Xác Nhận
 
 ```
 ✓ Issue đã được gửi: https://github.com/dv-workflow/dv-workflow/issues/[N]

package/.dw/core/templates/v3/task.md

@@ -66,14 +66,11 @@ genuinely changes.
 
 {Forcing function: deadline, incident, dependency, opportunity.}
 
-### Subtasks (in scope)
-
-**ST-1: {Subtask name}**
-- {Concrete action}
-- Acceptance: {verifiable criterion}
-
-**ST-2: {Subtask name}**
-- ...
+<!--
+Subtasks live in the Section 3 tracker (one list, with per-subtask Acceptance +
+Est columns) — NOT duplicated here. Section 2 stays the stable intent contract;
+Section 3 owns the subtask breakdown + status.
+-->
 
 ### Out of Scope
 
@@ -120,23 +117,19 @@ markers — those live only in Section 3).
 ## 3. Subtask Tracker
 
 <!--
-SINGLE SOURCE OF TRUTH for subtask status. This is the only section where
-status markers (⬜🟡✅🔴⏸) are allowed.
-
-Section 2 ("Subtasks (in scope)") and this tracker are deliberately separate:
-§2 owns the stable INTENT (name + actions + acceptance), this table owns the
-churning STATUS. They are NOT merged — that separation is what keeps status in
-exactly one place (drift-prevention). The Subtask name is restated here as a
-short label only.
+SINGLE SOURCE OF TRUTH for the subtask breakdown AND status. The subtask list is
+NOT duplicated in Section 2 (#24). This is the only section where status markers
+(⬜🟡✅🔴⏸) are allowed — Section 2 stays status-free (drift-prevention).
 
-`Est` is an optional effort estimate (hours / story-points / t-shirt per
-`estimation_unit`); leave `—` if unused. See `/dw:estimate`.
+Columns (read by name, order-tolerant): Acceptance = per-subtask verifiable
+criterion; Est = optional effort estimate (hours / story-points / t-shirt per
+`estimation_unit`, `—` if unused, see `/dw:estimate`).
 -->
 
-| # | Subtask | Status | Date | Notes | Est |
-|---|---------|--------|------|-------|-----|
-| ST-1 | ... | ⬜ Pending | — | | — |
-| ST-2 | ... | ⬜ Pending | — | | — |
+| # | Subtask | Acceptance | Est | Status | Notes |
+|---|---------|-----------|-----|--------|-------|
+| ST-1 | {name} | {verifiable criterion} | — | ⬜ Pending | |
+| ST-2 | {name} | ... | — | ⬜ Pending | |
 
 Status legend: ⬜ Pending · 🟡 In Progress · ✅ Done · 🔴 Blocked · ⏸ Paused
 

package/README.md

@@ -2,7 +2,7 @@
 
 > An AI development workflow toolkit for teams using agentic IDEs (Claude Code, Cursor) — from idea to review-ready commits.
 
-**v1.9.0** · `npm install -g dw-kit` · [Docs](docs/README.md) · [Get started](docs/get-started.md) · [Cheatsheet](docs/cheatsheet.md) · [Migration v1.3](MIGRATION-v1.3.md) · [Changelog](CHANGELOG.md)
+**v1.9.1** · `npm install -g dw-kit` · [Docs](docs/README.md) · [Get started](docs/get-started.md) · [Cheatsheet](docs/cheatsheet.md) · [Migration v1.3](MIGRATION-v1.3.md) · [Changelog](CHANGELOG.md)
 
 ---
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dw-kit",
-  "version": "1.9.0",
+  "version": "1.9.1",
   "description": "AI development workflow toolkit — structured, quality-assured, team-ready. From requirements to dashboard.",
   "type": "module",
   "bin": {

package/src/lib/goal-store.mjs

@@ -1,6 +1,7 @@
 import { readFileSync, writeFileSync, existsSync, mkdirSync, readdirSync, statSync } from 'node:fs';
 import { join, dirname } from 'node:path';
 import { parseFrontmatter, stringifyFrontmatter } from './frontmatter.mjs';
+import { parseSubtaskTracker } from './timeline-parser.mjs';
 
 const GOALS_DIR = '.dw/goals';
 const INDEX_FILE = '.dw/goals/goals-index.json';
@@ -145,7 +146,7 @@ export function computeGoalProgress(goalId, rootDir = process.cwd(), linkedTasks
     if (!existsSync(taskFile)) continue;
     const content = readFileSync(taskFile, 'utf8');
     const tracker = extractTrackerSection(content);
-    for (const row of parseTrackerRows(tracker)) {
+    for (const row of parseSubtaskTracker(tracker)) {
       total++;
       if (row.status.includes('✅') || /Done/i.test(row.status)) done++;
       else if (row.status.includes('🟡') || /In Progress/i.test(row.status)) inProgress++;
@@ -163,19 +164,6 @@ function extractTrackerSection(content) {
   return m ? m[0] : '';
 }
 
-function parseTrackerRows(section) {
-  const rows = [];
-  const lines = section.split('\n');
-  for (const line of lines) {
-    // Match table rows like: | ST-N | Description | Status | Date | Notes |
-    if (!/^\|\s*(ST|WS)-/.test(line)) continue;
-    const cells = line.split('|').map((c) => c.trim());
-    if (cells.length < 4) continue;
-    // cells[0] = "" (before first |), [1] = ID, [2] = subtask, [3] = status
-    rows.push({ id: cells[1], subtask: cells[2], status: cells[3] || '', date: cells[4] || '', notes: cells[5] || '' });
-  }
-  return rows;
-}
 
 export function removeIndexEntry(goalId, rootDir = process.cwd()) {
   const index = readGoalIndex(rootDir);

package/src/lib/timeline-parser.mjs

@@ -29,29 +29,68 @@ export function parseTimeline(content) {
   };
 }
 
+// Header-driven so the column ORDER is not load-bearing (#24): the §3 tracker
+// may be `# | Subtask | Status | Date | Notes` (legacy) or
+// `# | Subtask | Acceptance | Est | Status | Notes` (current) — we read columns
+// by name. Falls back to legacy positional mapping when no header row is found,
+// so older / hand-written tables keep parsing.
+const TRACKER_COLS = {
+  id: ['#', 'id'],
+  name: ['subtask', 'task', 'name'],
+  status: ['status'],
+  date: ['date'],
+  notes: ['notes'],
+  acceptance: ['acceptance'],
+  est: ['est', 'estimate'],
+};
+const LEGACY_POS = { id: 0, name: 1, status: 2, date: 3, notes: 4 };
+
+function splitRow(line) {
+  // Drop the empty cells produced by the leading/trailing pipes.
+  return line.split('|').map((c) => c.trim()).filter((_, i, arr) => i > 0 && i < arr.length - 1);
+}
+
 export function parseSubtaskTracker(sectionText) {
   if (!sectionText) return [];
   const lines = sectionText.split('\n').map((l) => l.trim()).filter(Boolean);
+  let colMap = null; // canonical-name -> column index
   const rows = [];
-  let inTable = false;
   for (const line of lines) {
-    if (line.startsWith('|---') || line.match(/^\|[\s|:-]+\|$/)) { inTable = true; continue; }
-    if (!line.startsWith('|')) { inTable = false; continue; }
-    const cells = line.split('|').map((c) => c.trim()).filter((_, i, arr) => i > 0 && i < arr.length - 1);
+    if (!line.startsWith('|')) continue;
+    if (line.match(/^\|[\s|:-]+\|$/)) continue; // separator row
+    const cells = splitRow(line);
     if (cells.length < 2) continue;
-    if (!inTable && (cells[0].toLowerCase().startsWith('#') || cells[0].toLowerCase() === 'ws' || cells[0].toLowerCase() === 'st')) {
-      // header row
+
+    const low = cells.map((c) => c.toLowerCase());
+    // Header row: first cell is '#'/'id', or it names known columns. Detect once.
+    const looksLikeHeader = low[0] === '#' || low[0] === 'id'
+      || low.includes('status') || low.includes('subtask');
+    if (!colMap && looksLikeHeader) {
+      colMap = {};
+      for (const [canon, aliases] of Object.entries(TRACKER_COLS)) {
+        const idx = low.findIndex((c) => aliases.includes(c));
+        if (idx >= 0) colMap[canon] = idx;
+      }
       continue;
     }
-    if (cells.length >= 3) {
-      rows.push({
-        id: cells[0],
-        name: cells[1],
-        status: cells[2],
-        date: cells[3] || '',
-        notes: cells[4] || '',
-      });
-    }
+
+    // Data row. Skip stray non-subtask rows when using positional fallback.
+    if (!colMap && !/^(ST|WS)-/i.test(cells[0]) && !/^\d/.test(cells[0])) continue;
+    if (cells.length < 3 && !colMap) continue;
+
+    const get = (canon) => {
+      const idx = colMap ? colMap[canon] : LEGACY_POS[canon];
+      return idx != null && idx >= 0 ? (cells[idx] ?? '') : '';
+    };
+    rows.push({
+      id: get('id'),
+      name: get('name'),
+      status: get('status'),
+      date: get('date'),
+      notes: get('notes'),
+      acceptance: get('acceptance'),
+      est: get('est'),
+    });
   }
   return rows;
 }