@sun-asterisk/sungen 3.0.1 → 3.1.1

package/dist/orchestrator/templates/ai-instructions/github-skill-sungen-tc-generation.md

@@ -54,6 +54,25 @@ user-invocable: false
   OR condition: generate 1 scenario per branch where that branch alone triggers the outcome.
   → Happy-path only = missing the most common multi-condition implementation bug.
 
+- **Many inputs, same steps → ONE data-driven scenario (`@cases`), not N copies:**
+  When a rule needs lots of inputs with the *same* step shape (email/format validation,
+  BVA boundary triples, EP classes, decision-table rows), tag one scenario `@cases:<dataset>`,
+  reference each row's columns as `{{col}}`, and put the rows as a LIST in test-data:
+  ```gherkin
+  @high @cases:email_validation
+  Scenario: VP-VAL-001 The email field rejects invalid formats
+    When User fill [Email] field with {{email}}
+    Then User see [Error] message with {{expected_error}}
+  ```
+  ```yaml
+  email_validation:
+    - { case: "no @",  email: "plainaddress", expected_error: "Invalid email" }
+    - { case: "valid", email: "ok@x.com",     expected_error: "" }
+  ```
+  → one `test()` per row, each labelled by `case`. Adding inputs = editing test-data (no recompile),
+  and env overlays apply. Prefer this over duplicating a scenario per value. (Gherkin
+  `Scenario Outline`/`Examples` is NOT supported — use `@cases`.)
+
 ---
 
 ## Tier System

package/dist/orchestrator/templates/ai-instructions/github-skill-sungen-tc-review.md

@@ -120,6 +120,7 @@ Build a mapping table: for each applicable group, does the feature have a matchi
 - **EP**: keep only **one representative** per invalid class; same-class duplicates → flag as redundant.
 - **BVA**: spec defines min/max → cover `min-1`, `min`, `max`, `max+1` (Maxlength, counts…).
 - Error messages must match the spec **word-for-word**, not generic.
+- **Data-driven (`@cases`)**: a `@cases:<dataset>` scenario legitimately covers many inputs in ONE scenario (one row per EP class / boundary / rule). Do **not** flag it as "too few negative cases" or as duplication — instead review the **dataset rows**: are all EP classes / boundary triples present, each labelled, expected values exact? N near-identical scenarios that differ only by input value → flag and recommend collapsing to `@cases`.
 
 ---
 

package/dist/orchestrator/templates/ai-instructions/github-skill-sungen-test-design-techniques.md

@@ -17,6 +17,12 @@ Apply selectively — not every screen needs all four techniques. Use the techni
 
 **Rule:** These techniques determine **how many** and **which** scenarios to generate. `sungen-viewpoint` determines **which viewpoints** to cover.
 
+**Implementing the data table → `@cases` (data-driven):** when EP classes / BVA boundary triples /
+decision-table rows share the *same step shape* and differ only by input/expected values, encode
+them as ONE `@cases:<dataset>` scenario (each class/boundary/rule = one row in the test-data list,
+labelled by a `case` column) instead of N near-duplicate scenarios. The technique still decides the
+rows; `@cases` is how you write them compactly. See `sungen-gherkin-syntax` → Data-driven.
+
 ---
 
 ## 1. Equivalence Partitioning (EP)

package/dist/orchestrator/templates/specs-db.d.ts

@@ -0,0 +1,26 @@
+declare class DataSource {
+    private configs;
+    private engines;
+    private cfg;
+    private engine;
+    private build;
+    /** A row matching `filter` must exist; if `expected` given, assert those columns on the first match. */
+    assertRow(table: string, filter: Record<string, any>, expected?: Record<string, any>, datasource?: string): Promise<void>;
+    /** No row matching `filter` may exist. */
+    assertNoRow(table: string, filter: Record<string, any>, datasource?: string): Promise<void>;
+    /** Exactly `count` rows must match `filter`. */
+    assertCount(table: string, filter: Record<string, any>, count: number, datasource?: string): Promise<void>;
+    /** Rewrites $1/$2 placeholders to `?` for SQLite. */
+    private sqlFor;
+    /** Read-only guard (second layer): a named query must be a single SELECT/WITH statement. */
+    private assertSelectOnly;
+    /**
+     * Run a catalog query (read-only) and return its rows. The result is bound to a `{{name}}`
+     * variable via `testData.bind(...)`, so the scenario asserts on it with `expect …` steps and
+     * path access (`{{name.count}}`, `{{name.first.col}}`, `{{name[2].col}}`).
+     */
+    fetchQuery(label: string, sql: string, params: any[], datasource?: string): Promise<any[]>;
+}
+export declare const db: DataSource;
+export {};
+//# sourceMappingURL=specs-db.d.ts.map

package/dist/orchestrator/templates/specs-db.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"specs-db.d.ts","sourceRoot":"","sources":["../../../src/orchestrator/templates/specs-db.ts"],"names":[],"mappings":"AA+DA,cAAM,UAAU;IACd,OAAO,CAAC,OAAO,CAAiD;IAChE,OAAO,CAAC,OAAO,CAA6B;IAE5C,OAAO,CAAC,GAAG;YAQG,MAAM;IAoBpB,OAAO,CAAC,KAAK;IAOb,wGAAwG;IAClG,SAAS,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,EAAE,QAAQ,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,EAAE,UAAU,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAc/H,0CAA0C;IACpC,WAAW,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,EAAE,UAAU,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAOjG,gDAAgD;IAC1C,WAAW,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,MAAM,EAAE,GAAG,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,UAAU,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,IAAI,CAAC;IAUhH,qDAAqD;IACrD,OAAO,CAAC,MAAM;IAKd,4FAA4F;IAC5F,OAAO,CAAC,gBAAgB;IASxB;;;;OAIG;IACG,UAAU,CAAC,KAAK,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,EAAE,EAAE,UAAU,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;CAKjG;AAMD,eAAO,MAAM,EAAE,YAAmB,CAAC"}

package/dist/orchestrator/templates/specs-db.js

@@ -0,0 +1,193 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.db = void 0;
+/* eslint-disable */
+/**
+ * Sungen Data Driver — runtime DB-verification helper (auto-generated into specs/db.ts).
+ *
+ * Read-only by construction: it only ever issues a single parameterized SELECT, and
+ * table/column identifiers are validated against a strict allowlist pattern (identifiers
+ * can't be bound as parameters). Secrets come from .env.qa / process.env, never inline.
+ *
+ * Engines: PostgreSQL (`pg`) and SQLite (`better-sqlite3`), lazy-loaded on first use.
+ * Config: a `datasources.yaml` at the project root (or qa/), with ${VAR} resolved from env.
+ *
+ * DO NOT EDIT — regenerated by `sungen generate`.
+ */
+const test_1 = require("@playwright/test");
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+const IDENT = /^[A-Za-z_][A-Za-z0-9_]*$/;
+const ident = (s) => {
+    if (!IDENT.test(s))
+        throw new Error(`Unsafe identifier: ${JSON.stringify(s)} (allowed: [A-Za-z_][A-Za-z0-9_]*)`);
+    return s;
+};
+function loadEnvQa() {
+    for (const name of ['.env.qa', `.env.qa.${process.env.SUNGEN_ENV || ''}`]) {
+        const p = path.join(process.cwd(), name);
+        if (!name.endsWith('.') && fs.existsSync(p)) {
+            for (const line of fs.readFileSync(p, 'utf8').split('\n')) {
+                const m = line.match(/^\s*([A-Za-z_][A-Za-z0-9_]*)\s*=\s*(.*?)\s*$/);
+                if (m && process.env[m[1]] === undefined) {
+                    process.env[m[1]] = m[2].replace(/^["']|["']$/g, '');
+                }
+            }
+        }
+    }
+}
+function loadConfig() {
+    loadEnvQa();
+    const file = [
+        path.join(process.cwd(), 'datasources.yaml'),
+        path.join(process.cwd(), 'qa', 'datasources.yaml'),
+    ].find((f) => fs.existsSync(f));
+    if (!file)
+        throw new Error('Data Driver: no datasources.yaml found (project root or qa/).');
+    let raw = fs.readFileSync(file, 'utf8').replace(/\$\{([A-Za-z_][A-Za-z0-9_]*)\}/g, (_, k) => process.env[k] ?? '');
+    const { parse } = require('yaml');
+    const doc = parse(raw) || {};
+    if (!doc.datasources || typeof doc.datasources !== 'object') {
+        throw new Error('Data Driver: datasources.yaml must define a top-level `datasources:` map.');
+    }
+    return doc.datasources;
+}
+class DataSource {
+    constructor() {
+        this.configs = null;
+        this.engines = new Map();
+    }
+    cfg(name) {
+        if (!this.configs)
+            this.configs = loadConfig();
+        const key = name || Object.keys(this.configs)[0];
+        const conf = this.configs[key];
+        if (!conf)
+            throw new Error(`Data Driver: datasource "${key}" not found in datasources.yaml`);
+        return { key, conf };
+    }
+    async engine(name) {
+        const { key, conf } = this.cfg(name);
+        if (this.engines.has(key))
+            return { engine: this.engines.get(key), conf };
+        if (!conf.url)
+            throw new Error(`Data Driver: datasource "${key}" has no url (set it in .env.qa).`);
+        let engine;
+        if (conf.engine === 'postgres') {
+            const { Pool } = require('pg');
+            const pool = new Pool({ connectionString: conf.url, max: 2, statement_timeout: conf.statement_timeout_ms ?? 4000 });
+            engine = { query: async (sql, params) => (await pool.query(sql, params)).rows };
+        }
+        else if (conf.engine === 'sqlite') {
+            const Database = require('better-sqlite3');
+            const db = new Database(conf.url.replace(/^sqlite:/, ''), { readonly: conf.readonly !== false });
+            engine = { query: async (sql, params) => db.prepare(sql).all(...params) };
+        }
+        else {
+            throw new Error(`Data Driver: engine "${conf.engine}" not supported yet (postgres | sqlite).`);
+        }
+        this.engines.set(key, engine);
+        return { engine, conf };
+    }
+    build(table, filter) {
+        const cols = Object.keys(filter);
+        const where = cols.map((c, i) => `${ident(c)} = $${i + 1}`).join(' AND ');
+        const sql = `SELECT * FROM ${ident(table)}${where ? ' WHERE ' + where : ''} LIMIT 50`;
+        return { sql, params: cols.map((c) => filter[c]) };
+    }
+    /** A row matching `filter` must exist; if `expected` given, assert those columns on the first match. */
+    async assertRow(table, filter, expected, datasource) {
+        const { engine, conf } = await this.engine(datasource);
+        const { sql, params } = this.build(table, filter);
+        const rows = await engine.query(this.sqlFor(conf, sql), params);
+        (0, test_1.expect)(rows.length, `Expected a row in "${table}" where ${desc(filter)} — found ${rows.length}`).toBeGreaterThanOrEqual(1);
+        if (expected) {
+            const row = rows[0];
+            for (const [col, val] of Object.entries(expected)) {
+                ident(col);
+                (0, test_1.expect)(String(row[col]), `Column "${col}" of "${table}" where ${desc(filter)}`).toBe(String(val));
+            }
+        }
+    }
+    /** No row matching `filter` may exist. */
+    async assertNoRow(table, filter, datasource) {
+        const { engine, conf } = await this.engine(datasource);
+        const { sql, params } = this.build(table, filter);
+        const rows = await engine.query(this.sqlFor(conf, sql), params);
+        (0, test_1.expect)(rows.length, `Expected NO row in "${table}" where ${desc(filter)} — found ${rows.length}`).toBe(0);
+    }
+    /** Exactly `count` rows must match `filter`. */
+    async assertCount(table, filter, count, datasource) {
+        const { engine, conf } = await this.engine(datasource);
+        const cols = Object.keys(filter);
+        const where = cols.map((c, i) => `${ident(c)} = $${i + 1}`).join(' AND ');
+        const sql = `SELECT count(*) AS n FROM ${ident(table)}${where ? ' WHERE ' + where : ''}`;
+        const rows = await engine.query(this.sqlFor(conf, sql), cols.map((c) => filter[c]));
+        const n = Number(rows[0]?.n ?? rows[0]?.['count(*)'] ?? 0);
+        (0, test_1.expect)(n, `Expected ${count} row(s) in "${table}"${cols.length ? ' where ' + desc(filter) : ''} — found ${n}`).toBe(Number(count));
+    }
+    /** Rewrites $1/$2 placeholders to `?` for SQLite. */
+    sqlFor(conf, sql) {
+        return conf.engine === 'sqlite' ? sql.replace(/\$\d+/g, '?') : sql;
+    }
+    // --- Named queries (catalog-backed; SQL is resolved + embedded at compile time) -----------
+    /** Read-only guard (second layer): a named query must be a single SELECT/WITH statement. */
+    assertSelectOnly(label, sql) {
+        const s = sql.trim().replace(/;\s*$/, '');
+        if (!/^(SELECT|WITH)\b/i.test(s))
+            throw new Error(`Data Driver: ${label} is not a read-only SELECT — refused.`);
+        if (s.includes(';'))
+            throw new Error(`Data Driver: ${label} contains multiple statements — refused.`);
+        if (/\b(INSERT|UPDATE|DELETE|DROP|ALTER|CREATE|TRUNCATE|GRANT|REVOKE|MERGE|REPLACE|CALL|EXEC|EXECUTE|ATTACH|PRAGMA|VACUUM)\b/i.test(s)) {
+            throw new Error(`Data Driver: ${label} contains a write/DDL keyword — refused.`);
+        }
+    }
+    /**
+     * Run a catalog query (read-only) and return its rows. The result is bound to a `{{name}}`
+     * variable via `testData.bind(...)`, so the scenario asserts on it with `expect …` steps and
+     * path access (`{{name.count}}`, `{{name.first.col}}`, `{{name[2].col}}`).
+     */
+    async fetchQuery(label, sql, params, datasource) {
+        this.assertSelectOnly(label, sql);
+        const { engine, conf } = await this.engine(datasource);
+        return engine.query(this.sqlFor(conf, sql), params);
+    }
+}
+function desc(filter) {
+    return Object.entries(filter).map(([k, v]) => `${k}=${JSON.stringify(v)}`).join(', ');
+}
+exports.db = new DataSource();
+//# sourceMappingURL=specs-db.js.map

package/dist/orchestrator/templates/specs-db.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"specs-db.js","sourceRoot":"","sources":["../../../src/orchestrator/templates/specs-db.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAAA,oBAAoB;AACpB;;;;;;;;;;;GAWG;AACH,2CAA0C;AAC1C,uCAAyB;AACzB,2CAA6B;AAE7B,MAAM,KAAK,GAAG,0BAA0B,CAAC;AACzC,MAAM,KAAK,GAAG,CAAC,CAAS,EAAU,EAAE;IAClC,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC;QAAE,MAAM,IAAI,KAAK,CAAC,sBAAsB,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,oCAAoC,CAAC,CAAC;IACjH,OAAO,CAAC,CAAC;AACX,CAAC,CAAC;AAUF,SAAS,SAAS;IAChB,KAAK,MAAM,IAAI,IAAI,CAAC,SAAS,EAAE,WAAW,OAAO,CAAC,GAAG,CAAC,UAAU,IAAI,EAAE,EAAE,CAAC,EAAE,CAAC;QAC1E,MAAM,CAAC,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,EAAE,IAAI,CAAC,CAAC;QACzC,IAAI,CAAC,IAAI,CAAC,QAAQ,CAAC,GAAG,CAAC,IAAI,EAAE,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,CAAC;YAC5C,KAAK,MAAM,IAAI,IAAI,EAAE,CAAC,YAAY,CAAC,CAAC,EAAE,MAAM,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;gBAC1D,MAAM,CAAC,GAAG,IAAI,CAAC,KAAK,CAAC,8CAA8C,CAAC,CAAC;gBACrE,IAAI,CAAC,IAAI,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,SAAS,EAAE,CAAC;oBACzC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,cAAc,EAAE,EAAE,CAAC,CAAC;gBACvD,CAAC;YACH,CAAC;QACH,CAAC;IACH,CAAC;AACH,CAAC;AAED,SAAS,UAAU;IACjB,SAAS,EAAE,CAAC;IACZ,MAAM,IAAI,GAAG;QACX,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,EAAE,kBAAkB,CAAC;QAC5C,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,EAAE,IAAI,EAAE,kBAAkB,CAAC;KACnD,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,CAAC;IAChC,IAAI,CAAC,IAAI;QAAE,MAAM,IAAI,KAAK,CAAC,+DAA+D,CAAC,CAAC;IAC5F,IAAI,GAAG,GAAG,EAAE,CAAC,YAAY,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC,OAAO,CAAC,iCAAiC,EAAE,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC;IACnH,MAAM,EAAE,KAAK,EAAE,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC;IAClC,MAAM,GAAG,GAAG,KAAK,CAAC,GAAG,CAAC,IAAI,EAAE,CAAC;IAC7B,IAAI,CAAC,GAAG,CAAC,WAAW,IAAI,OAAO,GAAG,CAAC,WAAW,KAAK,QAAQ,EAAE,CAAC;QAC5D,MAAM,IAAI,KAAK,CAAC,2EAA2E,CAAC,CAAC;IAC/F,CAAC;IACD,OAAO,GAAG,CAAC,WAAW,CAAC;AACzB,CAAC;AAID,MAAM,UAAU;IAAhB;QACU,YAAO,GAA4C,IAAI,CAAC;QACxD,YAAO,GAAG,IAAI,GAAG,EAAkB,CAAC;IAiG9C,CAAC;IA/FS,GAAG,CAAC,IAAa;QACvB,IAAI,CAAC,IAAI,CAAC,OAAO;YAAE,IAAI,CAAC,OAAO,GAAG,UAAU,EAAE,CAAC;QAC/C,MAAM,GAAG,GAAG,IAAI,IAAI,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC;QACjD,MAAM,IAAI,GAAG,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC;QAC/B,IAAI,CAAC,IAAI;YAAE,MAAM,IAAI,KAAK,CAAC,4BAA4B,GAAG,iCAAiC,CAAC,CAAC;QAC7F,OAAO,EAAE,GAAG,EAAE,IAAI,EAAE,CAAC;IACvB,CAAC;IAEO,KAAK,CAAC,MAAM,CAAC,IAAa;QAChC,MAAM,EAAE,GAAG,EAAE,IAAI,EAAE,GAAG,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;QACrC,IAAI,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,GAAG,CAAC;YAAE,OAAO,EAAE,MAAM,EAAE,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,GAAG,CAAE,EAAE,IAAI,EAAE,CAAC;QAC3E,IAAI,CAAC,IAAI,CAAC,GAAG;YAAE,MAAM,IAAI,KAAK,CAAC,4BAA4B,GAAG,mCAAmC,CAAC,CAAC;QACnG,IAAI,MAAc,CAAC;QACnB,IAAI,IAAI,CAAC,MAAM,KAAK,UAAU,EAAE,CAAC;YAC/B,MAAM,EAAE,IAAI,EAAE,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;YAC/B,MAAM,IAAI,GAAG,IAAI,IAAI,CAAC,EAAE,gBAAgB,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,EAAE,CAAC,EAAE,iBAAiB,EAAE,IAAI,CAAC,oBAAoB,IAAI,IAAI,EAAE,CAAC,CAAC;YACpH,MAAM,GAAG,EAAE,KAAK,EAAE,KAAK,EAAE,GAAG,EAAE,MAAM,EAAE,EAAE,CAAC,CAAC,MAAM,IAAI,CAAC,KAAK,CAAC,GAAG,EAAE,MAAM,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QAClF,CAAC;aAAM,IAAI,IAAI,CAAC,MAAM,KAAK,QAAQ,EAAE,CAAC;YACpC,MAAM,QAAQ,GAAG,OAAO,CAAC,gBAAgB,CAAC,CAAC;YAC3C,MAAM,EAAE,GAAG,IAAI,QAAQ,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,UAAU,EAAE,EAAE,CAAC,EAAE,EAAE,QAAQ,EAAE,IAAI,CAAC,QAAQ,KAAK,KAAK,EAAE,CAAC,CAAC;YACjG,MAAM,GAAG,EAAE,KAAK,EAAE,KAAK,EAAE,GAAG,EAAE,MAAM,EAAE,EAAE,CAAC,EAAE,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,GAAG,CAAC,GAAG,MAAM,CAAC,EAAE,CAAC;QAC5E,CAAC;aAAM,CAAC;YACN,MAAM,IAAI,KAAK,CAAC,wBAAwB,IAAI,CAAC,MAAM,0CAA0C,CAAC,CAAC;QACjG,CAAC;QACD,IAAI,CAAC,OAAO,CAAC,GAAG,CAAC,GAAG,EAAE,MAAM,CAAC,CAAC;QAC9B,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;IAC1B,CAAC;IAEO,KAAK,CAAC,KAAa,EAAE,MAA2B;QACtD,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QACjC,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,GAAG,KAAK,CAAC,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QAC1E,MAAM,GAAG,GAAG,iBAAiB,KAAK,CAAC,KAAK,CAAC,GAAG,KAAK,CAAC,CAAC,CAAC,SAAS,GAAG,KAAK,CAAC,CAAC,CAAC,EAAE,WAAW,CAAC;QACtF,OAAO,EAAE,GAAG,EAAE,MAAM,EAAE,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;IACrD,CAAC;IAED,wGAAwG;IACxG,KAAK,CAAC,SAAS,CAAC,KAAa,EAAE,MAA2B,EAAE,QAA8B,EAAE,UAAmB;QAC7G,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,MAAM,IAAI,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC;QACvD,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,EAAE,MAAM,CAAC,CAAC;QAClD,MAAM,IAAI,GAAG,MAAM,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,EAAE,GAAG,CAAC,EAAE,MAAM,CAAC,CAAC;QAChE,IAAA,aAAM,EAAC,IAAI,CAAC,MAAM,EAAE,sBAAsB,KAAK,WAAW,IAAI,CAAC,MAAM,CAAC,YAAY,IAAI,CAAC,MAAM,EAAE,CAAC,CAAC,sBAAsB,CAAC,CAAC,CAAC,CAAC;QAC3H,IAAI,QAAQ,EAAE,CAAC;YACb,MAAM,GAAG,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;YACpB,KAAK,MAAM,CAAC,GAAG,EAAE,GAAG,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAC;gBAClD,KAAK,CAAC,GAAG,CAAC,CAAC;gBACX,IAAA,aAAM,EAAC,MAAM,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC,EAAE,WAAW,GAAG,SAAS,KAAK,WAAW,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC;YACpG,CAAC;QACH,CAAC;IACH,CAAC;IAED,0CAA0C;IAC1C,KAAK,CAAC,WAAW,CAAC,KAAa,EAAE,MAA2B,EAAE,UAAmB;QAC/E,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,MAAM,IAAI,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC;QACvD,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,GAAG,IAAI,CAAC,KAAK,CAAC,KAAK,EAAE,MAAM,CAAC,CAAC;QAClD,MAAM,IAAI,GAAG,MAAM,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,EAAE,GAAG,CAAC,EAAE,MAAM,CAAC,CAAC;QAChE,IAAA,aAAM,EAAC,IAAI,CAAC,MAAM,EAAE,uBAAuB,KAAK,WAAW,IAAI,CAAC,MAAM,CAAC,YAAY,IAAI,CAAC,MAAM,EAAE,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAC5G,CAAC;IAED,gDAAgD;IAChD,KAAK,CAAC,WAAW,CAAC,KAAa,EAAE,MAA2B,EAAE,KAAa,EAAE,UAAmB;QAC9F,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,MAAM,IAAI,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC;QACvD,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;QACjC,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,GAAG,KAAK,CAAC,CAAC,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;QAC1E,MAAM,GAAG,GAAG,6BAA6B,KAAK,CAAC,KAAK,CAAC,GAAG,KAAK,CAAC,CAAC,CAAC,SAAS,GAAG,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC;QACzF,MAAM,IAAI,GAAG,MAAM,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,EAAE,GAAG,CAAC,EAAE,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QACpF,MAAM,CAAC,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,IAAI,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3D,IAAA,aAAM,EAAC,CAAC,EAAE,YAAY,KAAK,eAAe,KAAK,IAAI,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,SAAS,GAAG,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,EAAE,YAAY,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC;IACrI,CAAC;IAED,qDAAqD;IAC7C,MAAM,CAAC,IAAsB,EAAE,GAAW;QAChD,OAAO,IAAI,CAAC,MAAM,KAAK,QAAQ,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,QAAQ,EAAE,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC;IACrE,CAAC;IAED,6FAA6F;IAC7F,4FAA4F;IACpF,gBAAgB,CAAC,KAAa,EAAE,GAAW;QACjD,MAAM,CAAC,GAAG,GAAG,CAAC,IAAI,EAAE,CAAC,OAAO,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC;QAC1C,IAAI,CAAC,mBAAmB,CAAC,IAAI,CAAC,CAAC,CAAC;YAAE,MAAM,IAAI,KAAK,CAAC,gBAAgB,KAAK,uCAAuC,CAAC,CAAC;QAChH,IAAI,CAAC,CAAC,QAAQ,CAAC,GAAG,CAAC;YAAE,MAAM,IAAI,KAAK,CAAC,gBAAgB,KAAK,0CAA0C,CAAC,CAAC;QACtG,IAAI,0HAA0H,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC;YACvI,MAAM,IAAI,KAAK,CAAC,gBAAgB,KAAK,0CAA0C,CAAC,CAAC;QACnF,CAAC;IACH,CAAC;IAED;;;;OAIG;IACH,KAAK,CAAC,UAAU,CAAC,KAAa,EAAE,GAAW,EAAE,MAAa,EAAE,UAAmB;QAC7E,IAAI,CAAC,gBAAgB,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;QAClC,MAAM,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG,MAAM,IAAI,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC;QACvD,OAAO,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,IAAI,EAAE,GAAG,CAAC,EAAE,MAAM,CAAC,CAAC;IACtD,CAAC;CACF;AAED,SAAS,IAAI,CAAC,MAA2B;IACvC,OAAO,MAAM,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,CAAC,IAAI,IAAI,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AACxF,CAAC;AAEY,QAAA,EAAE,GAAG,IAAI,UAAU,EAAE,CAAC"}

package/dist/orchestrator/templates/specs-db.ts

@@ -0,0 +1,169 @@
+/* eslint-disable */
+/**
+ * Sungen Data Driver — runtime DB-verification helper (auto-generated into specs/db.ts).
+ *
+ * Read-only by construction: it only ever issues a single parameterized SELECT, and
+ * table/column identifiers are validated against a strict allowlist pattern (identifiers
+ * can't be bound as parameters). Secrets come from .env.qa / process.env, never inline.
+ *
+ * Engines: PostgreSQL (`pg`) and SQLite (`better-sqlite3`), lazy-loaded on first use.
+ * Config: a `datasources.yaml` at the project root (or qa/), with ${VAR} resolved from env.
+ *
+ * DO NOT EDIT — regenerated by `sungen generate`.
+ */
+import { expect } from '@playwright/test';
+import * as fs from 'fs';
+import * as path from 'path';
+
+const IDENT = /^[A-Za-z_][A-Za-z0-9_]*$/;
+const ident = (s: string): string => {
+  if (!IDENT.test(s)) throw new Error(`Unsafe identifier: ${JSON.stringify(s)} (allowed: [A-Za-z_][A-Za-z0-9_]*)`);
+  return s;
+};
+
+interface DataSourceConfig {
+  engine: 'postgres' | 'mysql' | 'sqlite';
+  url: string;
+  readonly?: boolean;
+  statement_timeout_ms?: number;
+  max_rows?: number;
+}
+
+function loadEnvQa(): void {
+  for (const name of ['.env.qa', `.env.qa.${process.env.SUNGEN_ENV || ''}`]) {
+    const p = path.join(process.cwd(), name);
+    if (!name.endsWith('.') && fs.existsSync(p)) {
+      for (const line of fs.readFileSync(p, 'utf8').split('\n')) {
+        const m = line.match(/^\s*([A-Za-z_][A-Za-z0-9_]*)\s*=\s*(.*?)\s*$/);
+        if (m && process.env[m[1]] === undefined) {
+          process.env[m[1]] = m[2].replace(/^["']|["']$/g, '');
+        }
+      }
+    }
+  }
+}
+
+function loadConfig(): Record<string, DataSourceConfig> {
+  loadEnvQa();
+  const file = [
+    path.join(process.cwd(), 'datasources.yaml'),
+    path.join(process.cwd(), 'qa', 'datasources.yaml'),
+  ].find((f) => fs.existsSync(f));
+  if (!file) throw new Error('Data Driver: no datasources.yaml found (project root or qa/).');
+  let raw = fs.readFileSync(file, 'utf8').replace(/\$\{([A-Za-z_][A-Za-z0-9_]*)\}/g, (_, k) => process.env[k] ?? '');
+  const { parse } = require('yaml');
+  const doc = parse(raw) || {};
+  if (!doc.datasources || typeof doc.datasources !== 'object') {
+    throw new Error('Data Driver: datasources.yaml must define a top-level `datasources:` map.');
+  }
+  return doc.datasources;
+}
+
+type Engine = { query(sql: string, params: any[]): Promise<any[]>; };
+
+class DataSource {
+  private configs: Record<string, DataSourceConfig> | null = null;
+  private engines = new Map<string, Engine>();
+
+  private cfg(name?: string): { key: string; conf: DataSourceConfig } {
+    if (!this.configs) this.configs = loadConfig();
+    const key = name || Object.keys(this.configs)[0];
+    const conf = this.configs[key];
+    if (!conf) throw new Error(`Data Driver: datasource "${key}" not found in datasources.yaml`);
+    return { key, conf };
+  }
+
+  private async engine(name?: string): Promise<{ engine: Engine; conf: DataSourceConfig }> {
+    const { key, conf } = this.cfg(name);
+    if (this.engines.has(key)) return { engine: this.engines.get(key)!, conf };
+    if (!conf.url) throw new Error(`Data Driver: datasource "${key}" has no url (set it in .env.qa).`);
+    let engine: Engine;
+    if (conf.engine === 'postgres') {
+      const { Pool } = require('pg');
+      const pool = new Pool({ connectionString: conf.url, max: 2, statement_timeout: conf.statement_timeout_ms ?? 4000 });
+      engine = { query: async (sql, params) => (await pool.query(sql, params)).rows };
+    } else if (conf.engine === 'sqlite') {
+      const Database = require('better-sqlite3');
+      const db = new Database(conf.url.replace(/^sqlite:/, ''), { readonly: conf.readonly !== false });
+      engine = { query: async (sql, params) => db.prepare(sql).all(...params) };
+    } else {
+      throw new Error(`Data Driver: engine "${conf.engine}" not supported yet (postgres | sqlite).`);
+    }
+    this.engines.set(key, engine);
+    return { engine, conf };
+  }
+
+  private build(table: string, filter: Record<string, any>): { sql: string; params: any[] } {
+    const cols = Object.keys(filter);
+    const where = cols.map((c, i) => `${ident(c)} = $${i + 1}`).join(' AND ');
+    const sql = `SELECT * FROM ${ident(table)}${where ? ' WHERE ' + where : ''} LIMIT 50`;
+    return { sql, params: cols.map((c) => filter[c]) };
+  }
+
+  /** A row matching `filter` must exist; if `expected` given, assert those columns on the first match. */
+  async assertRow(table: string, filter: Record<string, any>, expected?: Record<string, any>, datasource?: string): Promise<void> {
+    const { engine, conf } = await this.engine(datasource);
+    const { sql, params } = this.build(table, filter);
+    const rows = await engine.query(this.sqlFor(conf, sql), params);
+    expect(rows.length, `Expected a row in "${table}" where ${desc(filter)} — found ${rows.length}`).toBeGreaterThanOrEqual(1);
+    if (expected) {
+      const row = rows[0];
+      for (const [col, val] of Object.entries(expected)) {
+        ident(col);
+        expect(String(row[col]), `Column "${col}" of "${table}" where ${desc(filter)}`).toBe(String(val));
+      }
+    }
+  }
+
+  /** No row matching `filter` may exist. */
+  async assertNoRow(table: string, filter: Record<string, any>, datasource?: string): Promise<void> {
+    const { engine, conf } = await this.engine(datasource);
+    const { sql, params } = this.build(table, filter);
+    const rows = await engine.query(this.sqlFor(conf, sql), params);
+    expect(rows.length, `Expected NO row in "${table}" where ${desc(filter)} — found ${rows.length}`).toBe(0);
+  }
+
+  /** Exactly `count` rows must match `filter`. */
+  async assertCount(table: string, filter: Record<string, any>, count: number, datasource?: string): Promise<void> {
+    const { engine, conf } = await this.engine(datasource);
+    const cols = Object.keys(filter);
+    const where = cols.map((c, i) => `${ident(c)} = $${i + 1}`).join(' AND ');
+    const sql = `SELECT count(*) AS n FROM ${ident(table)}${where ? ' WHERE ' + where : ''}`;
+    const rows = await engine.query(this.sqlFor(conf, sql), cols.map((c) => filter[c]));
+    const n = Number(rows[0]?.n ?? rows[0]?.['count(*)'] ?? 0);
+    expect(n, `Expected ${count} row(s) in "${table}"${cols.length ? ' where ' + desc(filter) : ''} — found ${n}`).toBe(Number(count));
+  }
+
+  /** Rewrites $1/$2 placeholders to `?` for SQLite. */
+  private sqlFor(conf: DataSourceConfig, sql: string): string {
+    return conf.engine === 'sqlite' ? sql.replace(/\$\d+/g, '?') : sql;
+  }
+
+  // --- Named queries (catalog-backed; SQL is resolved + embedded at compile time) -----------
+  /** Read-only guard (second layer): a named query must be a single SELECT/WITH statement. */
+  private assertSelectOnly(label: string, sql: string): void {
+    const s = sql.trim().replace(/;\s*$/, '');
+    if (!/^(SELECT|WITH)\b/i.test(s)) throw new Error(`Data Driver: ${label} is not a read-only SELECT — refused.`);
+    if (s.includes(';')) throw new Error(`Data Driver: ${label} contains multiple statements — refused.`);
+    if (/\b(INSERT|UPDATE|DELETE|DROP|ALTER|CREATE|TRUNCATE|GRANT|REVOKE|MERGE|REPLACE|CALL|EXEC|EXECUTE|ATTACH|PRAGMA|VACUUM)\b/i.test(s)) {
+      throw new Error(`Data Driver: ${label} contains a write/DDL keyword — refused.`);
+    }
+  }
+
+  /**
+   * Run a catalog query (read-only) and return its rows. The result is bound to a `{{name}}`
+   * variable via `testData.bind(...)`, so the scenario asserts on it with `expect …` steps and
+   * path access (`{{name.count}}`, `{{name.first.col}}`, `{{name[2].col}}`).
+   */
+  async fetchQuery(label: string, sql: string, params: any[], datasource?: string): Promise<any[]> {
+    this.assertSelectOnly(label, sql);
+    const { engine, conf } = await this.engine(datasource);
+    return engine.query(this.sqlFor(conf, sql), params);
+  }
+}
+
+function desc(filter: Record<string, any>): string {
+  return Object.entries(filter).map(([k, v]) => `${k}=${JSON.stringify(v)}`).join(', ');
+}
+
+export const db = new DataSource();

package/dist/orchestrator/templates/specs-test-data.ts

@@ -5,6 +5,8 @@ import yaml from 'yaml';
 
 export class TestDataLoader {
   private data: Record<string, any>;
+  // Data-driven (@cases): when set (via withRow), get() prefers this row's columns.
+  private row?: Record<string, any>;
 
   private constructor(data: Record<string, any>) {
     this.data = data;
@@ -41,23 +43,56 @@ export class TestDataLoader {
   }
 
   get(key: string): string {
-    // Captured/runtime vars (set() below) are stored under their literal — possibly
-    // dotted — key (e.g. "cart.product_name"), so check the flat key first.
-    let current: any = this.data[key];
-    if (current === undefined || current === null) {
-      // Fall back to nested navigation for YAML-structured keys (e.g. "cart.qty_two").
-      current = this.data;
-      for (const part of key.split('.')) {
-        if (current == null || typeof current !== 'object') {
-          throw new Error(`Test data key not found: ${key} (failed at '${part}')`);
-        }
-        current = current[part];
-      }
-    }
-    if (current === undefined || current === null) {
+    const value = this.resolve(key);
+    if (value === undefined || value === null) {
       throw new Error(`Test data key not found: ${key}`);
     }
-    return String(current);
+    return String(value);
+  }
+
+  /**
+   * Resolve a `{{...}}` reference to its raw value. Supports:
+   *  - flat keys (incl. captured runtime vars stored under a literal dotted key);
+   *  - `@cases` row columns (the current row wins);
+   *  - structured paths over nested data AND `@query`-bound result arrays:
+   *      `q.count` / `q.length`  → number of rows
+   *      `q.first.col` / `q.last.col` / `q[2].col` → a specific row's column
+   *      `q.col`                → shorthand for the first row's column
+   */
+  private resolve(key: string): any {
+    // 1. Exact flat key — captured vars (set()) live under a literal, possibly dotted, key.
+    if (this.row && key in this.row && this.row[key] !== undefined && this.row[key] !== null) {
+      return this.row[key];
+    }
+    if (this.data[key] !== undefined && this.data[key] !== null) {
+      return this.data[key];
+    }
+    // 2. Structured path: head from the row (cases) or shared data, then walk segments.
+    const tokens = String(key).replace(/\[(\d+)\]/g, '.$1').split('.');
+    let cur: any = (this.row && tokens[0] in this.row) ? this.row[tokens[0]] : this.data[tokens[0]];
+    for (let i = 1; i < tokens.length && cur != null; i++) cur = TestDataLoader.step(cur, tokens[i]);
+    return cur;
+  }
+
+  /** One navigation step over an array (with count/first/last/index/field-shorthand) or object. */
+  private static step(cur: any, token: string): any {
+    if (Array.isArray(cur)) {
+      if (token === 'count' || token === 'length') return cur.length;
+      if (token === 'first') return cur[0];
+      if (token === 'last') return cur[cur.length - 1];
+      if (/^\d+$/.test(token)) return cur[Number(token)];
+      return cur[0] == null ? undefined : cur[0][token]; // shorthand: first row's field
+    }
+    if (cur && typeof cur === 'object') return cur[token];
+    return undefined;
+  }
+
+  /**
+   * Bind a raw value (e.g. an `@query` result array) under `key` so `{{key.…}}` paths resolve.
+   * Unlike set(), the value is stored as-is (array/object), not coerced to a string.
+   */
+  bind(key: string, value: any): void {
+    this.data[key] = value;
   }
 
   /**
@@ -68,6 +103,32 @@ export class TestDataLoader {
   set(key: string, value: string): void {
     this.data[key] = value;
   }
+
+  /**
+   * Data-driven (@cases): return the list of rows at `key` (after env-overlay merge),
+   * each stamped with `__label` for the test title / report. Throws if missing or not a list.
+   */
+  cases(key: string): Array<Record<string, any>> {
+    const list = this.data[key];
+    if (!Array.isArray(list)) {
+      throw new Error(`@cases dataset "${key}" not found or not a list in test-data (got ${typeof list}).`);
+    }
+    return list.map((row: any, i: number) => {
+      const r: Record<string, any> = (row && typeof row === 'object' && !Array.isArray(row)) ? { ...row } : { value: row };
+      r.__label = String(r.case ?? r.name ?? r.label ?? `row ${i + 1}`);
+      return r;
+    });
+  }
+
+  /**
+   * Data-driven (@cases): a view whose get() prefers the given row's columns and falls
+   * back to the shared data. Used inside the per-row test() loop.
+   */
+  withRow(row: Record<string, any>): TestDataLoader {
+    const view = new TestDataLoader({ ...this.data }); // clone → per-row set() stays isolated
+    view.row = row;
+    return view;
+  }
 }
 
 function loadYamlSync(filePath: string): Record<string, any> | null {

package/docs/orchestration-spec.md

@@ -2,7 +2,7 @@
 
 > Spec triển khai cho lần refactor lớn: chuyển từ **"Context Engine + Skill" (workflow tuyến tính)** sang **Orchestration + Harness Engine**.
 > Mục tiêu chính: **nâng chất lượng testcase** (đo & gate thay vì nhờ AI). Bất biến: `generate`, auto-fix selector, `delivery`, reports.
-> Tài liệu nền: `reports/sungen_direction_solution.md`, `reports/sungen_refactor_spec.md`, `reports/sungen_home_gherkin_viewpoint_coverage_review.md`.
+> Tài liệu nền: `docs/spec/sungen_direction_solution.md`, `docs/spec/sungen_refactor_spec.md`.
 
 ---
 
@@ -153,7 +153,7 @@ Hết ngân sách repair (N vòng) mà vẫn FAIL:
 └─► KHÔNG lặp vô hạn. Ghi gap vào audit-report + đưa QA checkpoint (human-in-the-loop).
 ```
 
-**Quy tắc "khám phá thêm khi nông":** nếu assertion-depth sensor báo một viewpoint critical chỉ assert "see page/section" (vụ home: Cart/Detail/Filter), Orchestrator phải nhận diện đây là **cross-screen** → khuyến nghị chuyển sang **flow** (`add-flow`) và/hoặc cần **năng lực DSL capture biến** (xem `reports/sungen_refactor_spec.md` §5.4) — thay vì giả vờ pass bằng assertion nông.
+**Quy tắc "khám phá thêm khi nông":** nếu assertion-depth sensor báo một viewpoint critical chỉ assert "see page/section" (vụ home: Cart/Detail/Filter), Orchestrator phải nhận diện đây là **cross-screen** → khuyến nghị chuyển sang **flow** (`add-flow`) và/hoặc cần **năng lực DSL capture biến** (xem `docs/spec/sungen_refactor_spec.md` §5.4) — thay vì giả vờ pass bằng assertion nông.
 
 ---
 
@@ -180,7 +180,7 @@ Bốn sensor + một gate. Input: `.feature` (qua GherkinParser) + `test-viewpoi
 - Orchestrator đưa phản hồi này lại bước sinh. **Ngân sách N vòng** (mặc định 2–3, như vòng auto-fix selector). Hết ngân sách → báo gap, không lặp vô hạn.
 
 ### 5.4 QA Checkpoint
-- QA accept/reject/edit/add viewpoint. Mọi quyết định ghi `feedback.jsonl` (xem `reports/sungen_refactor_spec.md` §8).
+- QA accept/reject/edit/add viewpoint. Mọi quyết định ghi `feedback.jsonl` (xem `docs/spec/sungen_refactor_spec.md` §8).
 
 ---
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sun-asterisk/sungen",
-  "version": "3.0.1",
+  "version": "3.1.1",
   "description": "Deterministic E2E Test Compiler - Gherkin + Selectors → Playwright tests",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -82,7 +82,7 @@
     "dist",
     "bin",
     "src",
-    "docs",
+    "docs/orchestration-spec.md",
     "README.md",
     "LICENSE"
   ]

package/src/cli/commands/challenge.ts

@@ -38,11 +38,15 @@ export function registerChallengeCommand(program: Command): void {
         if (report.collectionClaimSingular.length) {
           for (const f of report.collectionClaimSingular) L(`     ⚠ ${f.scenario}\n        → ${f.suggestion}`);
         } else L('     ✓ none');
-        L('  ② Coverage — over-covered / shallow');
+        L('  ② Data-driven — scenarios that should be one `@cases` (collapse data-variants / cover EP classes)');
+        if (report.dataDriven.length) {
+          for (const f of report.dataDriven) L(`     ⚠ ${f.scenario}\n        → ${f.suggestion}`);
+        } else L('     ✓ none');
+        L('  ③ Coverage — over-covered / shallow');
         if (report.overCovered.length) for (const o2 of report.overCovered) L(`     • ${o2.bucket}: ${o2.note}`);
         if (report.shallowThemes.length) L(`     • shallow themes: ${report.shallowThemes.join(', ')}`);
         if (!report.overCovered.length && !report.shallowThemes.length) L('     ✓ balanced');
-        L('  ③ Novelty — prompts for the `sungen-challenge` agent (≤20% of official, no auto-merge)');
+        L('  ④ Novelty — prompts for the `sungen-challenge` agent (≤20% of official, no auto-merge)');
         for (const p of report.noveltyPrompts) L(`     • ${p}`);
         L('  ── Exploration readiness ──');
         for (const e of report.explorationReadiness) L(`     • ${e}`);

package/src/cli/commands/delivery.ts

@@ -200,11 +200,12 @@ function discoverLocaleVariants(cwd: string, target: DeliveryTarget): LocaleVari
   const prefix = `${target.featureBaseName}-test-result`;
   const variants: LocaleVariant[] = [];
 
-  const basePath = path.join(genDir, `${prefix}.json`);
+  // Base variant: per-target result file if present, else fall back to the global
+  // test-results/results.json (what playwright.config writes by default) via resolveResultsPath.
   variants.push({
     locale: '',
     displayCode: DEFAULT_BASE_LOCALE.toUpperCase(),
-    resultsPath: fs.existsSync(basePath) ? basePath : null,
+    resultsPath: resolveResultsPath(cwd, target),
   });
 
   if (fs.existsSync(genDir)) {