@educa-corp/sdd-framework 0.9.7 → 0.9.8

package/bin/qc-base-map.json

@@ -58,9 +58,9 @@
     },
     {
       "upstream": "agents/6-qa-runner.md",
-      "targets": [],
-      "state": "undecided",
-      "why": "dinh nghia agent cua upstream; framework dung command + gate _agent_mode thay cho agent file"
+      "targets": ["commands/qc-design-script.tmpl", "commands/qc-run-script.tmpl"],
+      "state": "replaced",
+      "why": "Dinh nghia agent cua upstream; framework dung LENH thay cho agent. Vai nay nay tach lam hai lenh: /qc-design-script (sinh) va /qc-run-script (chay + cham) — Dot 2 · b2. Chot o Buoc S · S2."
     },
     {
       "upstream": "agents/7-qa-gate.md",
@@ -126,14 +126,14 @@
     {
       "upstream": "skills/qa-gate/styled-report-feat011.py",
       "targets": [],
-      "state": "undecided",
-      "why": "script chay duoc, khong phai prompt — framework QC hien 100% markdown-first"
+      "state": "skipped",
+      "why": "Bao cao do CONG CU sinh, khong do script tu viet: Automation-Standards §1 + OQ-01 (dong 2026-06-02) chot Playwright HTML Report cho web/api; Mobile-Automation-Standards §1 chot Allure Report v2.x cho app. Hai file .py nay khong con dich de port — bo o Buoc S · S5 (2026-09-17). Lane QC van 100% markdown-first: do duoc 0 file chay duoc trong skills/qc."
     },
     {
       "upstream": "skills/qa-gate/styled-report-template.py",
       "targets": [],
-      "state": "undecided",
-      "why": "script chay duoc, khong phai prompt — framework QC hien 100% markdown-first"
+      "state": "skipped",
+      "why": "Bao cao do CONG CU sinh, khong do script tu viet: Automation-Standards §1 + OQ-01 (dong 2026-06-02) chot Playwright HTML Report cho web/api; Mobile-Automation-Standards §1 chot Allure Report v2.x cho app. Hai file .py nay khong con dich de port — bo o Buoc S · S5 (2026-09-17). Lane QC van 100% markdown-first: do duoc 0 file chay duoc trong skills/qc."
     },
     {
       "upstream": "skills/qa-io-contract/injection-scanner.md",
@@ -157,13 +157,15 @@
     },
     {
       "upstream": "skills/qa-runner/script-writer.md",
-      "targets": [],
-      "state": "undecided"
+      "targets": ["skills/qc/qa-script-designer/web/", "skills/qc/qa-script-designer/mobile/", "skills/qc/qa-script-designer/api/", "skills/qc/qa-script-designer/_shared/"],
+      "state": "replaced",
+      "why": "Tach lam hai: phan SINH script -> qa-script-designer (Buoc S · S2, 2026-09-17), ba lane web/mobile/api theo active_platform. Stack doi tu Python sang TypeScript + Playwright/WDIO-Appium theo qc-base-new (Approved)."
     },
     {
       "upstream": "skills/qa-runner/test-executor.md",
-      "targets": [],
-      "state": "undecided"
+      "targets": ["skills/qc/qa-script-runner/web/run.md", "skills/qc/qa-script-runner/mobile/run.md", "skills/qc/qa-script-runner/report.md"],
+      "state": "replaced",
+      "why": "Tach lam hai: phan CHAY + cham -> qa-script-runner (Buoc S · S2). Truoc day gop chung voi phan sinh script trong qa-runner, nen sua phan cham phai mo trang day cach viet."
     },
     {
       "upstream": "skills/qa-tc-analyst/acceptance-criteria.md",

package/bin/self-check.js

@@ -103,8 +103,26 @@ function allSourceFiles() {
   push(path.join(ROOT, 'templates'), () => true);
   push(path.join(ROOT, 'rules'),    f => f.endsWith('.md'));
   push(path.join(ROOT, 'modules'),  f => f.endsWith('.yaml') || f.endsWith('.md'));
+  // skills/ vào scan set từ Bước S · S0 (2026-09-17). Trước đó R4/R5/R6 mù với toàn bộ
+  // skills/ — và 21 trong 27 file mang stack QC cũ nằm đúng ở đó, nên R5 không canh được
+  // việc một stack đã bỏ quay trở lại. Đo trước khi mở: 11 pattern đang có × 64 file
+  // skills/**/*.md = 0 hit, tức mở scan set không làm đỏ thứ gì đang xanh.
+  push(path.join(ROOT, 'skills'),   f => f.endsWith('.md'));
   return out;
 }
+
+/**
+ * Glob tối giản cho `forbidden_patterns[].scope`. Hỗ trợ `*` (không qua `/`) và `**`
+ * (qua mọi cấp) — đủ cho ba dạng đang dùng và cố ý KHÔNG hơn: một glob engine đầy đủ ở
+ * đây là thứ không ai đọc lại, và scope sai thì rule im lặng chứ không kêu lên.
+ */
+function globMatch(glob, rel) {
+  const re = glob.split('/').map(seg =>
+    seg === '**' ? '.*'
+                 : seg.replace(/[.+^${}()|[\]\\]/g, '\\$&').replace(/\*/g, '[^/]*')
+  ).join('/').replace(/\/\.\*$/, '(?:/.*)?');
+  return new RegExp(`^${re}$`).test(rel);
+}
 const sources = allSourceFiles().map(p => ({
   rel: path.relative(ROOT, p).replace(/\\/g, '/'),
   text: fs.readFileSync(p, 'utf8'),
@@ -281,16 +299,43 @@ for (const fp of schema.forbidden_patterns) {
   // `except` = file được phép chứa pattern vì nó đang TRÍCH DẪN pattern trong một cảnh
   // báo ("đừng glob X"). Mỗi entry phải kèm lý do trong schema — không phải ignore mù.
   const except = new Set(Object.keys(fp.except || {}));
-  const hits = sources.filter(s => s.text.includes(fp.pattern) && !except.has(s.rel));
+  // `scope` (thêm ở Bước S · S0) = mảng glob giới hạn vùng file rule soi tới. KHÔNG khai
+  // scope = toàn scan set, nên mọi entry có từ trước giữ nguyên hành vi. Vì sao cần: `pytest`
+  // bị cấm trong phạm vi QC nhưng HỢP LỆ ở lane DEV — modules/context-engineering là module
+  // Python thật, dev-gen-test mặc định sinh test Python. Cấm toàn cục là 4 dương tính giả,
+  // và dương tính giả đẻ ra except bừa cho tới khi R5 rỗng ruột mà vẫn xanh.
+  const inScope = s => !fp.scope || fp.scope.some(g => globMatch(g, s.rel));
+  const hits = sources.filter(s => inScope(s) && s.text.includes(fp.pattern) && !except.has(s.rel));
   if (hits.length) {
     err('R5', `Pattern bị cấm quay lại: \`${fp.pattern}\`  (${fp.gap})`,
         `${fp.reason}\n        Thấy ở: ${hits.map(h => h.rel).join(', ')}`);
   }
-  // Cảnh báo nếu một except trở nên vô nghĩa — cảnh báo đã bị xoá thì except cũng nên bỏ
+  // Cảnh báo nếu một except trở nên vô nghĩa — cảnh báo đã bị xoá thì except cũng nên bỏ.
+  // Nằm ngoài `scope` cũng là vô nghĩa: file đó không bao giờ bị soi, giữ except chỉ làm
+  // người đọc tưởng nó đang che một thứ có thật.
   for (const f of except) {
     const src = sources.find(s => s.rel === f);
-    if (!src || !src.text.includes(fp.pattern)) {
-      warn('R5', `\`${fp.pattern}\`: except \`${f}\` không còn chứa pattern — bỏ except này khỏi schema`);
+    if (!src || !src.text.includes(fp.pattern) || !inScope(src)) {
+      warn('R5', `\`${fp.pattern}\`: except \`${f}\` không còn chứa pattern (hoặc ngoài scope) — bỏ except này khỏi schema`);
+    }
+  }
+}
+
+// Một `scope` glob khớp 0 file thì rule **trông như** đang canh một vùng, thực ra canh không
+// khí — đúng kiểu rỗng ruột mà cảnh báo `except` chết ở trên sinh ra để chống, chỉ ở đầu kia.
+// Đã suýt xảy ra thật ở S0: scope trỏ `qa-runner/**` (thư mục S2 sắp XOÁ) và không trỏ
+// `qa-script-designer/**` (thư mục S2 sắp TẠO) → sau S2 đèn xanh vì RỖNG, không vì SẠCH.
+// Chỉ WARN, không ERROR: trỏ vào thư mục chưa tạo là hợp lệ trong lúc đang chuyển stack.
+// Gộp theo glob, không theo entry — 6 entry dùng chung một scope thì cảnh báo một lần.
+{
+  const seen = new Set();
+  for (const fp of schema.forbidden_patterns) {
+    for (const g of (fp.scope || [])) {
+      if (seen.has(g)) continue;
+      seen.add(g);
+      if (!sources.some(s => globMatch(g, s.rel))) {
+        warn('R5', `scope \`${g}\` khớp 0 file — thư mục chưa tồn tại, hoặc đã bị xoá mà glob còn ở lại`);
+      }
     }
   }
 }