@sun-asterisk/sungen 3.2.33 → 3.2.34-beta.1

package/dist/orchestrator/trace-mode-install.d.ts

@@ -0,0 +1,14 @@
+/** The template's trace value: cheap on CI (retries exist there), a trace for every local failure. */
+export declare const TRACE_MODE_EXPRESSION = "process.env.CI ? 'on-first-retry' : 'retain-on-failure'";
+/**
+ *   'patched' — the old default was rewritten now
+ *   'present' — already the template expression (idempotent)
+ *   'custom'  — another value, no trace line, more than one old-default line, or one inside
+ *               `projects: [...]`; nothing written
+ *   'absent'  — no playwright.config.ts (a mobile-only project, or not a sungen project)
+ */
+export type TraceModeOutcome = 'patched' | 'present' | 'custom' | 'absent';
+export declare function patchTraceMode(cwd: string, dryRun?: boolean): TraceModeOutcome;
+/** `sungen update` step: patch the trace mode and say so when it changed. */
+export declare function installTraceMode(cwd: string, dryRun?: boolean): void;
+//# sourceMappingURL=trace-mode-install.d.ts.map

package/dist/orchestrator/trace-mode-install.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"trace-mode-install.d.ts","sourceRoot":"","sources":["../../src/orchestrator/trace-mode-install.ts"],"names":[],"mappings":"AAgBA,sGAAsG;AACtG,eAAO,MAAM,qBAAqB,4DAA4D,CAAC;AAE/F;;;;;;GAMG;AACH,MAAM,MAAM,gBAAgB,GAAG,SAAS,GAAG,SAAS,GAAG,QAAQ,GAAG,QAAQ,CAAC;AA6C3E,wBAAgB,cAAc,CAAC,GAAG,EAAE,MAAM,EAAE,MAAM,GAAE,OAAe,GAAG,gBAAgB,CAgBrF;AAED,6EAA6E;AAC7E,wBAAgB,gBAAgB,CAAC,GAAG,EAAE,MAAM,EAAE,MAAM,GAAE,OAAe,GAAG,IAAI,CAG3E"}

package/dist/orchestrator/trace-mode-install.js

@@ -0,0 +1,132 @@
+"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.TRACE_MODE_EXPRESSION = void 0;
+exports.patchTraceMode = patchTraceMode;
+exports.installTraceMode = installTraceMode;
+/**
+ * Move an existing project's `use.trace` off the old default so local runs keep a trace.
+ *
+ * The old template paired `trace: 'on-first-retry'` with `retries: process.env.CI ? 2 : 0`: off CI
+ * there is no retry, so there is never a "first retry" and a QA fixing a test on their machine never
+ * got a trace. The template now records `on-first-retry` on CI and `retain-on-failure` locally.
+ *
+ * `playwright.config.ts` is written once at `sungen init` and never re-synced (see
+ * report-sanitize-reporter-install.ts, the same problem for the sanitize reporter), so `sungen update`
+ * calls this. It rewrites the trace line ONLY while it is still the old default — any other value is
+ * a choice the user made, and a wrong edit to a user's config is worse than a missing trace. The same
+ * goes for a `trace` inside `projects: [...]`: that is a per-project override, not the template default.
+ */
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+/** The template's trace value: cheap on CI (retries exist there), a trace for every local failure. */
+exports.TRACE_MODE_EXPRESSION = "process.env.CI ? 'on-first-retry' : 'retain-on-failure'";
+const escapeRegExp = (s) => s.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+// Both are anchored to the start of a line, so a commented-out copy (`// trace: …`) never counts.
+// OLD_DEFAULT keeps the indentation in $1 and leaves the trailing comma, comment and EOL alone.
+const OLD_DEFAULT = /^([ \t]*trace:[ \t]*)(['"])on-first-retry\2/gm;
+const PRESENT = new RegExp(`^[ \\t]*trace:[ \\t]*${escapeRegExp(exports.TRACE_MODE_EXPRESSION)}`, 'm');
+// The old template's one-line comment, only when it sits directly above the (patched) trace line.
+// A comment the user wrote is theirs and stays.
+const OLD_COMMENT = new RegExp(`^([ \\t]*)/\\* Collect trace when retrying the failed test\\. See https://playwright\\.dev/docs/trace-viewer \\*/(\\r?\\n)(?=[ \\t]*trace:[ \\t]*${escapeRegExp(exports.TRACE_MODE_EXPRESSION)})`, 'm');
+const newComment = (indent, eol) => `${indent}/* Trace for a failed test. CI: recorded on the first retry (CI retries). Local: no retries, so${eol}` +
+    `${indent}   'on-first-retry' would never record one — keep the trace of every failed test instead.${eol}` +
+    `${indent}   See https://playwright.dev/docs/trace-viewer */${eol}`;
+/**
+ * [start, end] offsets of each live `projects: [...]` array, matched by bracket depth. Brackets inside
+ * a `//` or `/* *\/` comment or a quoted string are skipped: one stray `]` in a note would otherwise
+ * close the span early and expose a per-project override to the patch.
+ */
+function projectsSpans(src) {
+    const spans = [];
+    for (const m of src.matchAll(/^[ \t]*projects\s*:\s*\[/gm)) {
+        const open = m.index + m[0].length - 1;
+        let depth = 0;
+        for (let i = open; i < src.length; i++) {
+            const ch = src[i];
+            if (ch === '/' && src[i + 1] === '/') {
+                const nl = src.indexOf('\n', i);
+                i = nl < 0 ? src.length : nl;
+                continue;
+            }
+            if (ch === '/' && src[i + 1] === '*') {
+                const end = src.indexOf('*/', i + 2);
+                i = end < 0 ? src.length : end + 1;
+                continue;
+            }
+            if (ch === "'" || ch === '"' || ch === '`') {
+                for (i++; i < src.length && src[i] !== ch; i++)
+                    if (src[i] === '\\')
+                        i++;
+                continue;
+            }
+            if (ch === '[')
+                depth++;
+            else if (ch === ']' && --depth === 0) {
+                spans.push([open, i]);
+                break;
+            }
+        }
+    }
+    return spans;
+}
+function patchTraceMode(cwd, dryRun = false) {
+    const configPath = path.join(cwd, 'playwright.config.ts');
+    if (!fs.existsSync(configPath))
+        return 'absent';
+    const src = fs.readFileSync(configPath, 'utf8');
+    if (PRESENT.test(src))
+        return 'present';
+    const hits = [...src.matchAll(OLD_DEFAULT)];
+    if (hits.length !== 1)
+        return 'custom'; // none: the user's own value · several: ambiguous, don't guess
+    const at = hits[0].index;
+    if (projectsSpans(src).some(([s, e]) => at > s && at < e))
+        return 'custom'; // a per-project override
+    if (!dryRun) {
+        const patched = src
+            .replace(OLD_DEFAULT, `$1${exports.TRACE_MODE_EXPRESSION}`)
+            .replace(OLD_COMMENT, (_m, indent, eol) => newComment(indent, eol));
+        fs.writeFileSync(configPath, patched);
+    }
+    return 'patched';
+}
+/** `sungen update` step: patch the trace mode and say so when it changed. */
+function installTraceMode(cwd, dryRun = false) {
+    if (patchTraceMode(cwd, dryRun) !== 'patched')
+        return;
+    console.log(`${dryRun ? '📋 would update' : '✓ updated'} playwright.config.ts: local runs now keep a trace for every failed test (trace: ${exports.TRACE_MODE_EXPRESSION})`);
+}
+//# sourceMappingURL=trace-mode-install.js.map

package/dist/orchestrator/trace-mode-install.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"trace-mode-install.js","sourceRoot":"","sources":["../../src/orchestrator/trace-mode-install.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAuEA,wCAgBC;AAGD,4CAGC;AA7FD;;;;;;;;;;;;GAYG;AACH,uCAAyB;AACzB,2CAA6B;AAE7B,sGAAsG;AACzF,QAAA,qBAAqB,GAAG,yDAAyD,CAAC;AAW/F,MAAM,YAAY,GAAG,CAAC,CAAS,EAAU,EAAE,CAAC,CAAC,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,CAAC;AAErF,kGAAkG;AAClG,gGAAgG;AAChG,MAAM,WAAW,GAAG,+CAA+C,CAAC;AACpE,MAAM,OAAO,GAAG,IAAI,MAAM,CAAC,wBAAwB,YAAY,CAAC,6BAAqB,CAAC,EAAE,EAAE,GAAG,CAAC,CAAC;AAE/F,kGAAkG;AAClG,gDAAgD;AAChD,MAAM,WAAW,GAAG,IAAI,MAAM,CAC5B,oJAAoJ,YAAY,CAAC,6BAAqB,CAAC,GAAG,EAC1L,GAAG,CACJ,CAAC;AACF,MAAM,UAAU,GAAG,CAAC,MAAc,EAAE,GAAW,EAAU,EAAE,CACzD,GAAG,MAAM,kGAAkG,GAAG,EAAE;IAChH,GAAG,MAAM,4FAA4F,GAAG,EAAE;IAC1G,GAAG,MAAM,qDAAqD,GAAG,EAAE,CAAC;AAEtE;;;;GAIG;AACH,SAAS,aAAa,CAAC,GAAW;IAChC,MAAM,KAAK,GAA4B,EAAE,CAAC;IAC1C,KAAK,MAAM,CAAC,IAAI,GAAG,CAAC,QAAQ,CAAC,4BAA4B,CAAC,EAAE,CAAC;QAC3D,MAAM,IAAI,GAAG,CAAC,CAAC,KAAM,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC;QACxC,IAAI,KAAK,GAAG,CAAC,CAAC;QACd,KAAK,IAAI,CAAC,GAAG,IAAI,EAAE,CAAC,GAAG,GAAG,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YACvC,MAAM,EAAE,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC;YAClB,IAAI,EAAE,KAAK,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC,KAAK,GAAG,EAAE,CAAC;gBAAC,MAAM,EAAE,GAAG,GAAG,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC;gBAAC,CAAC,GAAG,EAAE,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC;gBAAC,SAAS;YAAC,CAAC;YAClH,IAAI,EAAE,KAAK,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC,KAAK,GAAG,EAAE,CAAC;gBAAC,MAAM,GAAG,GAAG,GAAG,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,GAAG,CAAC,CAAC,CAAC;gBAAC,CAAC,GAAG,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC,GAAG,GAAG,CAAC,CAAC;gBAAC,SAAS;YAAC,CAAC;YAC7H,IAAI,EAAE,KAAK,GAAG,IAAI,EAAE,KAAK,GAAG,IAAI,EAAE,KAAK,GAAG,EAAE,CAAC;gBAC3C,KAAK,CAAC,EAAE,EAAE,CAAC,GAAG,GAAG,CAAC,MAAM,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,EAAE;oBAAE,IAAI,GAAG,CAAC,CAAC,CAAC,KAAK,IAAI;wBAAE,CAAC,EAAE,CAAC;gBACzE,SAAS;YACX,CAAC;YACD,IAAI,EAAE,KAAK,GAAG;gBAAE,KAAK,EAAE,CAAC;iBACnB,IAAI,EAAE,KAAK,GAAG,IAAI,EAAE,KAAK,KAAK,CAAC,EAAE,CAAC;gBAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;gBAAC,MAAM;YAAC,CAAC;QACzE,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,SAAgB,cAAc,CAAC,GAAW,EAAE,SAAkB,KAAK;IACjE,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,sBAAsB,CAAC,CAAC;IAC1D,IAAI,CAAC,EAAE,CAAC,UAAU,CAAC,UAAU,CAAC;QAAE,OAAO,QAAQ,CAAC;IAChD,MAAM,GAAG,GAAG,EAAE,CAAC,YAAY,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC;IAChD,IAAI,OAAO,CAAC,IAAI,CAAC,GAAG,CAAC;QAAE,OAAO,SAAS,CAAC;IACxC,MAAM,IAAI,GAAG,CAAC,GAAG,GAAG,CAAC,QAAQ,CAAC,WAAW,CAAC,CAAC,CAAC;IAC5C,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,QAAQ,CAAC,CAAC,+DAA+D;IACvG,MAAM,EAAE,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC,KAAM,CAAC;IAC1B,IAAI,aAAa,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,GAAG,CAAC,IAAI,EAAE,GAAG,CAAC,CAAC;QAAE,OAAO,QAAQ,CAAC,CAAC,yBAAyB;IACrG,IAAI,CAAC,MAAM,EAAE,CAAC;QACZ,MAAM,OAAO,GAAG,GAAG;aAChB,OAAO,CAAC,WAAW,EAAE,KAAK,6BAAqB,EAAE,CAAC;aAClD,OAAO,CAAC,WAAW,EAAE,CAAC,EAAE,EAAE,MAAc,EAAE,GAAW,EAAE,EAAE,CAAC,UAAU,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC,CAAC;QACtF,EAAE,CAAC,aAAa,CAAC,UAAU,EAAE,OAAO,CAAC,CAAC;IACxC,CAAC;IACD,OAAO,SAAS,CAAC;AACnB,CAAC;AAED,6EAA6E;AAC7E,SAAgB,gBAAgB,CAAC,GAAW,EAAE,SAAkB,KAAK;IACnE,IAAI,cAAc,CAAC,GAAG,EAAE,MAAM,CAAC,KAAK,SAAS;QAAE,OAAO;IACtD,OAAO,CAAC,GAAG,CAAC,GAAG,MAAM,CAAC,CAAC,CAAC,iBAAiB,CAAC,CAAC,CAAC,WAAW,oFAAoF,6BAAqB,GAAG,CAAC,CAAC;AACvK,CAAC"}

package/dist/orchestrator/viewpoint-pattern-catalog.d.ts

@@ -0,0 +1,23 @@
+/**
+ * The sungen-viewpoint pattern catalog by number — the `## N. <Name>` headings of
+ * `templates/ai-src/skills/sungen-viewpoint/group-*.md` — for the "UI Patterns Identified" table
+ * `sungen add` scaffolds into `test-viewpoint.md` (#842).
+ *
+ * The scaffold used to hard-code its own ten rows: eight of them numbered differently from the
+ * catalog (Data Table 2 vs 7, Modal 9 vs 12…), plus a "Pagination" pattern the catalog does not
+ * have, so authors copied the wrong numbers next to a skill that cites the right ones.
+ * `tests/harness/viewpoint-comments-and-patterns.run.ts` pins this list to the catalog headings.
+ */
+export interface ViewpointPattern {
+    n: number;
+    name: string;
+    /** Group F — native-app patterns, read IN ADDITION to 1–17 for a mobile screen. */
+    mobile?: boolean;
+}
+export declare const VIEWPOINT_PATTERNS: readonly ViewpointPattern[];
+/**
+ * The scaffold's pattern-table rows: 1–17 for every screen, plus Group F on a mobile project.
+ * `Áp dụng?` is left empty for the QA to mark ✅ / ❌, as the QA template does.
+ */
+export declare function uiPatternRows(isMobile: boolean): string[];
+//# sourceMappingURL=viewpoint-pattern-catalog.d.ts.map

package/dist/orchestrator/viewpoint-pattern-catalog.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"viewpoint-pattern-catalog.d.ts","sourceRoot":"","sources":["../../src/orchestrator/viewpoint-pattern-catalog.ts"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AACH,MAAM,WAAW,gBAAgB;IAC/B,CAAC,EAAE,MAAM,CAAC;IACV,IAAI,EAAE,MAAM,CAAC;IACb,mFAAmF;IACnF,MAAM,CAAC,EAAE,OAAO,CAAC;CAClB;AAED,eAAO,MAAM,kBAAkB,EAAE,SAAS,gBAAgB,EAiCzD,CAAC;AAEF;;;GAGG;AACH,wBAAgB,aAAa,CAAC,QAAQ,EAAE,OAAO,GAAG,MAAM,EAAE,CAIzD"}

package/dist/orchestrator/viewpoint-pattern-catalog.js

@@ -0,0 +1,48 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.VIEWPOINT_PATTERNS = void 0;
+exports.uiPatternRows = uiPatternRows;
+exports.VIEWPOINT_PATTERNS = [
+    { n: 1, name: 'Form & Inputs' },
+    { n: 2, name: 'File Upload' },
+    { n: 3, name: 'Import / Export' },
+    { n: 4, name: 'Create / Add' },
+    { n: 5, name: 'Update / Edit' },
+    { n: 6, name: 'Delete' },
+    { n: 7, name: 'Data Table' },
+    { n: 8, name: 'Search' },
+    { n: 9, name: 'Filter' },
+    { n: 10, name: 'List / Card View' },
+    { n: 11, name: 'Chart / Analytics' },
+    { n: 12, name: 'Modal / Dialog' },
+    { n: 13, name: 'Navigation' },
+    { n: 14, name: 'Notification / Toast / Alert' },
+    { n: 15, name: 'Login / Logout' },
+    { n: 16, name: 'Register' },
+    { n: 17, name: 'Password Management' },
+    { n: 18, name: 'Bottom Sheet', mobile: true },
+    { n: 19, name: 'Tab Bar / Bottom Navigation', mobile: true },
+    { n: 20, name: 'Swipe-to-Delete Row', mobile: true },
+    { n: 21, name: 'Native Picker', mobile: true },
+    { n: 22, name: 'Permission Dialog', mobile: true },
+    { n: 23, name: 'Biometric Prompt', mobile: true },
+    { n: 24, name: 'Camera / Gallery Picker', mobile: true },
+    { n: 25, name: 'Push Notification', mobile: true },
+    { n: 26, name: 'Deep-link Entry', mobile: true },
+    { n: 27, name: 'Offline Banner', mobile: true },
+    { n: 28, name: 'Onboarding Carousel', mobile: true },
+    { n: 29, name: 'Pull-to-Refresh List', mobile: true },
+    { n: 30, name: 'Soft-Keyboard Overlap', mobile: true },
+    { n: 31, name: 'Settings Toggle List', mobile: true },
+    { n: 32, name: 'App Lifecycle & Orientation', mobile: true },
+];
+/**
+ * The scaffold's pattern-table rows: 1–17 for every screen, plus Group F on a mobile project.
+ * `Áp dụng?` is left empty for the QA to mark ✅ / ❌, as the QA template does.
+ */
+function uiPatternRows(isMobile) {
+    return exports.VIEWPOINT_PATTERNS
+        .filter((p) => isMobile || !p.mobile)
+        .map((p) => `| ${p.n} | ${p.name} | | |`);
+}
+//# sourceMappingURL=viewpoint-pattern-catalog.js.map

package/dist/orchestrator/viewpoint-pattern-catalog.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"viewpoint-pattern-catalog.js","sourceRoot":"","sources":["../../src/orchestrator/viewpoint-pattern-catalog.ts"],"names":[],"mappings":";;;AAwDA,sCAIC;AA3CY,QAAA,kBAAkB,GAAgC;IAC7D,EAAE,CAAC,EAAE,CAAC,EAAE,IAAI,EAAE,eAAe,EAAE;IAC/B,EAAE,CAAC,EAAE,CAAC,EAAE,IAAI,EAAE,aAAa,EAAE;IAC7B,EAAE,CAAC,EAAE,CAAC,EAAE,IAAI,EAAE,iBAAiB,EAAE;IACjC,EAAE,CAAC,EAAE,CAAC,EAAE,IAAI,EAAE,cAAc,EAAE;IAC9B,EAAE,CAAC,EAAE,CAAC,EAAE,IAAI,EAAE,eAAe,EAAE;IAC/B,EAAE,CAAC,EAAE,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE;IACxB,EAAE,CAAC,EAAE,CAAC,EAAE,IAAI,EAAE,YAAY,EAAE;IAC5B,EAAE,CAAC,EAAE,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE;IACxB,EAAE,CAAC,EAAE,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE;IACxB,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,kBAAkB,EAAE;IACnC,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,mBAAmB,EAAE;IACpC,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,gBAAgB,EAAE;IACjC,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,YAAY,EAAE;IAC7B,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,8BAA8B,EAAE;IAC/C,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,gBAAgB,EAAE;IACjC,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,UAAU,EAAE;IAC3B,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,qBAAqB,EAAE;IACtC,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,cAAc,EAAE,MAAM,EAAE,IAAI,EAAE;IAC7C,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,6BAA6B,EAAE,MAAM,EAAE,IAAI,EAAE;IAC5D,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,qBAAqB,EAAE,MAAM,EAAE,IAAI,EAAE;IACpD,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,eAAe,EAAE,MAAM,EAAE,IAAI,EAAE;IAC9C,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,mBAAmB,EAAE,MAAM,EAAE,IAAI,EAAE;IAClD,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,kBAAkB,EAAE,MAAM,EAAE,IAAI,EAAE;IACjD,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,yBAAyB,EAAE,MAAM,EAAE,IAAI,EAAE;IACxD,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,mBAAmB,EAAE,MAAM,EAAE,IAAI,EAAE;IAClD,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,iBAAiB,EAAE,MAAM,EAAE,IAAI,EAAE;IAChD,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,gBAAgB,EAAE,MAAM,EAAE,IAAI,EAAE;IAC/C,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,qBAAqB,EAAE,MAAM,EAAE,IAAI,EAAE;IACpD,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,sBAAsB,EAAE,MAAM,EAAE,IAAI,EAAE;IACrD,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,uBAAuB,EAAE,MAAM,EAAE,IAAI,EAAE;IACtD,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,sBAAsB,EAAE,MAAM,EAAE,IAAI,EAAE;IACrD,EAAE,CAAC,EAAE,EAAE,EAAE,IAAI,EAAE,6BAA6B,EAAE,MAAM,EAAE,IAAI,EAAE;CAC7D,CAAC;AAEF;;;GAGG;AACH,SAAgB,aAAa,CAAC,QAAiB;IAC7C,OAAO,0BAAkB;SACtB,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,QAAQ,IAAI,CAAC,CAAC,CAAC,MAAM,CAAC;SACpC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,IAAI,QAAQ,CAAC,CAAC;AAC9C,CAAC"}

package/dist/orchestrator/viewpoint-scaffold.d.ts

@@ -0,0 +1,3 @@
+export declare const VIEWPOINT_TEMPLATE_PATH: string;
+export declare function viewpointScaffold(title: string, isMobile: boolean): string;
+//# sourceMappingURL=viewpoint-scaffold.d.ts.map

package/dist/orchestrator/viewpoint-scaffold.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"viewpoint-scaffold.d.ts","sourceRoot":"","sources":["../../src/orchestrator/viewpoint-scaffold.ts"],"names":[],"mappings":"AAeA,eAAO,MAAM,uBAAuB,QAEnC,CAAC;AAKF,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,MAAM,EAAE,QAAQ,EAAE,OAAO,GAAG,MAAM,CAa1E"}

package/dist/orchestrator/viewpoint-scaffold.js

@@ -0,0 +1,70 @@
+"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.VIEWPOINT_TEMPLATE_PATH = void 0;
+exports.viewpointScaffold = viewpointScaffold;
+/**
+ * The `requirements/test-viewpoint.md` that `sungen add` and `sungen add-flow` write.
+ *
+ * It IS the QA template the AI drafts from (`skills/sungen-viewpoint/viewpoint-template.md`), read
+ * at run time so QA and AI always start from the same file. Two substitutions only: the title, and
+ * the template's sample UI-pattern rows → the full catalog for the project's platform (#842).
+ *
+ * Every sample row sits inside `<!-- -->`, so an untouched scaffold declares no viewpoint and the
+ * audit reports VIEWPOINT-MISSING instead of measuring the suite against samples (#841). The
+ * scaffold never carries an active `drafted-by: ai` line: that marks an AI draft, and this is not one.
+ */
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+const viewpoint_pattern_catalog_1 = require("./viewpoint-pattern-catalog");
+exports.VIEWPOINT_TEMPLATE_PATH = path.join(__dirname, 'templates', 'ai-src', 'skills', 'sungen-viewpoint', 'viewpoint-template.md');
+/** A UI-pattern row: `| 12 | Modal / Dialog | … |`. Only section 1 of the template has them. */
+const PATTERN_ROW = /^\|\s*\d+\s*\|/;
+function viewpointScaffold(title, isMobile) {
+    const lines = fs.readFileSync(exports.VIEWPOINT_TEMPLATE_PATH, 'utf-8').replace(/\r\n?/g, '\n').split('\n');
+    const titleAt = lines.findIndex((l) => l.startsWith('# '));
+    const rowsAt = lines.findIndex((l) => PATTERN_ROW.test(l));
+    // A template edit that drops either anchor must fail here, not ship a scaffold without them.
+    if (titleAt < 0 || rowsAt < 0) {
+        throw new Error(`viewpoint-template.md has no title or no UI-pattern sample rows: ${exports.VIEWPOINT_TEMPLATE_PATH}`);
+    }
+    let rowsEnd = rowsAt;
+    while (rowsEnd < lines.length && PATTERN_ROW.test(lines[rowsEnd]))
+        rowsEnd++;
+    lines[titleAt] = `# Test Viewpoint — ${title}`;
+    lines.splice(rowsAt, rowsEnd - rowsAt, ...(0, viewpoint_pattern_catalog_1.uiPatternRows)(isMobile));
+    return lines.join('\n');
+}
+//# sourceMappingURL=viewpoint-scaffold.js.map

package/dist/orchestrator/viewpoint-scaffold.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"viewpoint-scaffold.js","sourceRoot":"","sources":["../../src/orchestrator/viewpoint-scaffold.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AAsBA,8CAaC;AAnCD;;;;;;;;;;GAUG;AACH,uCAAyB;AACzB,2CAA6B;AAC7B,2EAA4D;AAE/C,QAAA,uBAAuB,GAAG,IAAI,CAAC,IAAI,CAC9C,SAAS,EAAE,WAAW,EAAE,QAAQ,EAAE,QAAQ,EAAE,kBAAkB,EAAE,uBAAuB,CACxF,CAAC;AAEF,gGAAgG;AAChG,MAAM,WAAW,GAAG,gBAAgB,CAAC;AAErC,SAAgB,iBAAiB,CAAC,KAAa,EAAE,QAAiB;IAChE,MAAM,KAAK,GAAG,EAAE,CAAC,YAAY,CAAC,+BAAuB,EAAE,OAAO,CAAC,CAAC,OAAO,CAAC,QAAQ,EAAE,IAAI,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IACpG,MAAM,OAAO,GAAG,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC;IAC3D,MAAM,MAAM,GAAG,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC;IAC3D,6FAA6F;IAC7F,IAAI,OAAO,GAAG,CAAC,IAAI,MAAM,GAAG,CAAC,EAAE,CAAC;QAC9B,MAAM,IAAI,KAAK,CAAC,oEAAoE,+BAAuB,EAAE,CAAC,CAAC;IACjH,CAAC;IACD,IAAI,OAAO,GAAG,MAAM,CAAC;IACrB,OAAO,OAAO,GAAG,KAAK,CAAC,MAAM,IAAI,WAAW,CAAC,IAAI,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;QAAE,OAAO,EAAE,CAAC;IAC7E,KAAK,CAAC,OAAO,CAAC,GAAG,sBAAsB,KAAK,EAAE,CAAC;IAC/C,KAAK,CAAC,MAAM,CAAC,MAAM,EAAE,OAAO,GAAG,MAAM,EAAE,GAAG,IAAA,yCAAa,EAAC,QAAQ,CAAC,CAAC,CAAC;IACnE,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sun-asterisk/sungen",
-  "version": "3.2.33",
+  "version": "3.2.34-beta.1",
   "description": "Deterministic E2E Test Compiler - Gherkin + Selectors → Playwright tests",
   "main": "src/index.ts",
   "types": "src/index.ts",
@@ -40,8 +40,8 @@
     "@babel/types": "^7.28.5",
     "@cucumber/gherkin": "^37.0.0",
     "@cucumber/messages": "^31.0.0",
-    "@sungen/driver-data-factory": "3.2.33",
-    "@sungen/driver-ui": "3.2.33",
+    "@sungen/driver-data-factory": "3.2.34-beta.1",
+    "@sungen/driver-ui": "3.2.34-beta.1",
     "chalk": "^5.6.2",
     "commander": "^14.0.2",
     "dotenv": "^17.2.3",

package/src/cli/commands/audit.ts

@@ -4,6 +4,7 @@ import * as fs from 'fs';
 import { runAudit, AuditReport, catalogIdFromScreenDir } from '../../harness/audit';
 import { reportSlug } from '../../harness/unit-paths';
 import { acceptViewpointBaseline } from '../../harness/viewpoint-baseline';
+import { removeDraftLineFromFile, viewpointFileIsAiDraft } from '../../harness/viewpoint-draft-header';
 import { findUnitDir } from '../resolve-unit';
 
 
@@ -15,6 +16,14 @@ export function bar(ratio: number, width = 20): string {
   return '█'.repeat(n) + '░'.repeat(width - n);
 }
 
+/** The ⑥ headline. No viewpoint ID read → nothing was matched, so `0%` would read as a measured
+ *  failure; say `n/a` and why instead (#832). Same with no scenario yet (#852). */
+export function traceabilityHeadline(trace: AuditReport['trace']): string {
+  if (trace.overviewIds === 0) return '  ⑥ Traceability — n/a (no viewpoint ID found in test-viewpoint.md)';
+  if (trace.total === 0) return '  ⑥ Traceability — n/a (no scenarios yet)';
+  return `  ⑥ Traceability — ${(trace.mappedRatio * 100).toFixed(0)}% scenarios linked to viewpoint-overview`;
+}
+
 function render(r: AuditReport): void {
   const L = console.log;
   L('');
@@ -82,7 +91,7 @@ function render(r: AuditReport): void {
   if (r.depth.shallowBusinessCritical.length > 8) L(`      … +${r.depth.shallowBusinessCritical.length - 8} more`);
   L('');
   L(`  ③ Claim-Proof — ${r.claim.proven}/${r.claim.withClaims} titled claims proven by steps  [${r.claim.verdict.toUpperCase()}]`);
-  if (r.claim.unproven.length === 0) L('      ✓ every claim (all/only/single/correct/changes/hidden) is backed by an assertion');
+  if (r.claim.unproven.length === 0) L('      ✓ every claim (all/only/single/correct/changes/hidden/order/limit) is backed by an assertion');
   for (const u of r.claim.unproven.slice(0, 8)) L(`      ${u.severity === 'fail' ? '✗' : '⚠'} [${u.claim}] ${u.name} → needs ${u.need}`);
   if (r.claim.unproven.length > 8) L(`      … +${r.claim.unproven.length - 8} more`);
   L('');
@@ -110,7 +119,7 @@ function render(r: AuditReport): void {
     }
   }
   L('');
-  L(`  ⑥ Traceability — ${(r.trace.mappedRatio * 100).toFixed(0)}% scenarios linked to viewpoint-overview`);
+  L(traceabilityHeadline(r.trace));
   L(`      ${r.trace.note}`);
   L('');
   if (r.spec.hasSpec && (r.spec.frTotal > 0 || r.spec.triggerGaps.length > 0 || r.spec.verdict !== 'pass')) {
@@ -135,7 +144,7 @@ function render(r: AuditReport): void {
       const held = new Set(r.score.axes.filter((x) => x.withheld).map((x) => x.key));
       const absent = na.filter((k) => !held.has(k));
       if (absent.length) L(`      not scored (no evidence in this unit): ${absent.join(', ')}`);
-      if (held.size) L(`      measured but held (viewpoint baseline changed — review, then --accept-viewpoint): ${[...held].join(', ')}`);
+      if (held.size) L(`      measured but held (viewpoint not accepted yet — changed, adopted after the suite, or an AI draft; review, then --accept-viewpoint): ${[...held].join(', ')}`);
     }
     L(`      weakest: ${r.calibration.weakest.axis} ${(r.calibration.weakest.value * 100).toFixed(0)}%${r.calibration.inflated ? '  ⚠ SCORE-INFLATED-BY-BREADTH' : ''}`);
     if (r.calibration.cappedAt !== undefined) {
@@ -165,14 +174,18 @@ export function registerAuditCommand(program: Command): void {
     .option('--area <name>', 'Alias of --api — an API-first area (qa/api/<name>)')
     .option('--json', 'Output the raw JSON report')
     .option('--accept-viewpoint', 'Accept the current test-viewpoint.md as the baseline the suite is measured against')
+    .option('--no-baseline', 'Measure without writing .sungen/viewpoint-baseline.json (audit a drafted viewpoint before generating)')
     .action((options) => {
       try {
         const name = options.screen || options.api || options.area;
         if (!name) throw new Error('Provide --screen <name> (or --api <area>)');
+        // Commander's `--no-*`: `baseline` is true unless the flag is passed.
+        const recordBaseline = options.baseline !== false;
+        if (!recordBaseline && options.acceptViewpoint) throw new Error('--no-baseline and --accept-viewpoint cannot be combined');
         const dir = findUnitDir(name);
         if (!dir) throw new Error(`Not found: qa/screens/${name}, qa/flows/${name}, or qa/api/${name}`);
 
-        const report = runAudit(dir, name);
+        let report = runAudit(dir, name);
 
         // The baseline is only ever written HERE, never by the measurement itself:
         // a first sighting has nothing to compare against, and any later change is
@@ -187,18 +200,43 @@ export function registerAuditCommand(program: Command): void {
         const unitId = catalogIdFromScreenDir(dir);
         const vb = report.viewpointBaseline;
         const postHoc = vb.status === 'new' && report.scenarioCount > 0;
+        // `drafted-by: ai` — a viewpoint the AI wrote is never accepted by the audit that measures it,
+        // with scenarios or without: only `--accept-viewpoint` does, and that removes the line.
+        const vpPath = path.join(dir, 'requirements', 'test-viewpoint.md');
+        const hasDraftMarker = viewpointFileIsAiDraft(vpPath);
+        // A draft with no VP id has nothing to accept — only VIEWPOINT-MISSING speaks for it. Fail
+        // here, before any write, so the file stays as it is and the QA is told why.
+        if (options.acceptViewpoint && vb.status === 'absent' && hasDraftMarker) {
+          throw new Error('the draft declares no VP ids — nothing to accept. Add the viewpoint ids (VP-… in the Priority Viewpoints table or checks), then run --accept-viewpoint again.');
+        }
+        const aiDraft = vb.status !== 'absent' && hasDraftMarker;
         // A re-parsed baseline is a CACHE migration, not an acceptance: the comparison already
         // resolved to `unchanged`, and rewriting the record only lets the NEXT real edit be
         // detected. Left unwritten it would re-migrate — and re-announce — on every run.
-        if (vb.reparsed) acceptViewpointBaseline(process.cwd(), unitId, { hash: vb.hash, ids: vb.ids, fileHash: vb.fileHash });
-        if ((vb.status === 'new' && !postHoc) || (options.acceptViewpoint && (vb.status === 'changed' || postHoc))) {
+        //
+        // `--no-baseline` skips every write here, the migration included: a draft audited before its
+        // scenarios exist is `new` and not post-hoc, and a plain audit would record it as the yardstick.
+        if (recordBaseline && !aiDraft && vb.reparsed) acceptViewpointBaseline(process.cwd(), unitId, { hash: vb.hash, ids: vb.ids, fileHash: vb.fileHash });
+        if (recordBaseline && ((vb.status === 'new' && !postHoc && !aiDraft) || (options.acceptViewpoint && (vb.status === 'changed' || postHoc || aiDraft)))) {
           acceptViewpointBaseline(process.cwd(), unitId, { hash: vb.hash, ids: vb.ids, fileHash: vb.fileHash });
-          if (options.acceptViewpoint && !options.json) {
-            console.log(`\n  ✓ test-viewpoint.md accepted as the baseline for ${name} (${vb.ids.length} viewpoint id(s)).`);
-            console.log('    Re-run the audit to score atomicLedger + traceability against it.\n');
+          if (options.acceptViewpoint) {
+            // The baseline is recorded first: if the line cannot be removed the draft stays flagged
+            // (held, finding, decide step) and the accept can simply be run again.
+            const lineRemoved = aiDraft && removeDraftLineFromFile(vpPath);
+            // Measure again against what was just accepted: the report printed and saved below (the
+            // one `sungen next` reads) must not be the pre-accept one, still holding the axes (#867).
+            report = runAudit(dir, name);
+            if (!options.json) {
+              console.log(`\n  ✓ test-viewpoint.md accepted as the baseline for ${name} (${vb.ids.length} viewpoint id(s)).`);
+              if (lineRemoved) console.log('    Removed the `drafted-by: ai` line from test-viewpoint.md — it is QA-reviewed from here on.');
+              console.log('    The report below is measured against it.\n');
+            }
           }
         } else if (options.acceptViewpoint && !options.json) {
-          console.log(`\n  ○ nothing to accept for ${name} — the viewpoint baseline is ${vb.status}.\n`);
+          // `absent` covers a file with no readable viewpoint id too — say which (#832).
+          console.log(vb.status === 'absent'
+            ? `\n  ○ nothing to accept for ${name} — no viewpoint ID found in requirements/test-viewpoint.md (missing file, or no section declares one).\n`
+            : `\n  ○ nothing to accept for ${name} — the viewpoint baseline is ${vb.status}.\n`);
         }
 
         // Persist report under .sungen/reports/ (flat slug: api flow `flows/x` → `flows-x-audit.json`).
@@ -212,6 +250,7 @@ export function registerAuditCommand(program: Command): void {
         } else {
           render(report);
           console.log(`  Report: ${path.relative(process.cwd(), outPath)}`);
+          if (!recordBaseline) console.log('  ○ baseline not recorded (--no-baseline)');
           console.log('');
         }
         // Non-zero exit when the gate fails — usable in CI / repair loop. Set exitCode (not

package/src/cli/commands/capability.ts

@@ -265,7 +265,9 @@ export function registerCapabilityCommand(program: Command): void {
           else console.log('  – skipped  appium-environment/ (already exists)');
 
           // Best-effort driver install — non-fatal so capability add still succeeds offline.
-          installMobileDriver(cwd);
+          // --skip-install promises no npm install; the driver + wdio/appium runtime is ~290 MB.
+          if (o.skipInstall) console.log('  – skipped  mobile driver install (--skip-install) — later: sungen capability add mobile');
+          else installMobileDriver(cwd);
 
           // --skip-install means "record only" — then provisioning the machine is not wanted either.
           runAppiumEnvironmentSetup(cwd, { skip: !!(o.skipEnvSetup || o.skipInstall) });

package/src/cli/commands/changed.ts

@@ -12,6 +12,7 @@
 import { Command } from 'commander';
 import { buildRunManifest, readRunManifest, writeRunManifest, selectChanged } from '../../harness/run-manifest';
 import { findUnitDir } from '../resolve-unit';
+import { titlesToGrep } from '../../harness/select-by-ids';
 import { execFileSync } from 'child_process';
 import * as path from 'path';
 
@@ -40,9 +41,6 @@ function gitChangedFiles(unitDir: string): string[] {
   }
 }
 
-/** Escape a scenario title for a `-g` / `--mochaOpts.grep` regex. */
-const toGrep = (titles: string[]): string =>
-  titles.map((t) => t.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')).join('|');
 
 export function registerChangedCommand(program: Command): void {
   program
@@ -75,7 +73,7 @@ export function registerChangedCommand(program: Command): void {
         if (options.grep) {
           // Empty output is meaningful: nothing to run. A caller must treat "" as "skip the run",
           // never as "match everything" — an empty grep would run the whole suite.
-          console.log(toGrep(selection.scenarios));
+          console.log(titlesToGrep(selection.scenarios));
           return;
         }
         if (options.json) { console.log(JSON.stringify(selection, null, 2)); return; }

package/src/cli/commands/dashboard.ts

@@ -11,8 +11,7 @@ import { Command } from 'commander';
 import * as path from 'path';
 import * as fs from 'fs';
 import { execSync } from 'child_process';
-import { EnvironmentInfo } from '../../exporters/types';
-import { resolveWebBaseUrl } from '../../exporters/env-qa';
+import { getEnvironment } from '../../exporters/run-environment';
 import {
   buildDashboardSnapshot,
   DashboardTarget,
@@ -37,30 +36,6 @@ const COLOR = {
 
 function log(msg: string): void { console.log(msg); }
 
-function getEnvironment(cwd: string): EnvironmentInfo {
-  let baseURL = '';
-  let projectName = 'chromium';
-  const configPath = path.join(cwd, 'playwright.config.ts');
-  if (fs.existsSync(configPath)) {
-    const c = fs.readFileSync(configPath, 'utf-8');
-    // Both config shapes: legacy literal and env-aware resolveBaseUrl('x') wrapper.
-    const baseMatch = c.match(/baseURL:\s*(?:resolveBaseUrl\(\s*)?['"]([^'"]+)['"]/);
-    if (baseMatch) baseURL = baseMatch[1];
-    const projMatch = c.match(/name:\s*['"]([^'"]+)['"]/);
-    if (projMatch) projectName = projMatch[1];
-  }
-  // Same SUNGEN_BASE_URL override as delivery — the two surfaces must agree.
-  baseURL = resolveWebBaseUrl(cwd, baseURL);
-
-  let executor = process.env.CI_USER || process.env.USER || '';
-  try {
-    const gitUser = execSync('git config user.name', { cwd, encoding: 'utf-8' }).trim();
-    if (gitUser) executor = gitUser;
-  } catch { /* ignore */ }
-
-  return { baseURL, projectName, executor };
-}
-
 interface DashboardOptions {
   noHistory?: boolean;
   open?: boolean;

package/src/cli/commands/defect.ts

@@ -0,0 +1,49 @@
+import { Command } from 'commander';
+import { resolveUnit } from '../resolve-unit';
+import { draftDefects } from '../../harness/defect-draft';
+import { readDefects } from '../../harness/defects';
+
+/**
+ * `sungen defect draft|list` — defect records under qa/defects/ (see harness/defects.ts).
+ * `draft` writes one record per product-candidate row of the unit's last `sungen triage`.
+ */
+export function registerDefectCommand(program: Command): void {
+  const defect = program.command('defect').description('Draft and list defect records (qa/defects/DEF-NNN-*.md) from red scenarios');
+
+  defect
+    .command('draft')
+    .description('Draft a defect record for each product-candidate row of the last `sungen triage`')
+    .option('-s, --screen <name>', 'Screen or flow name')
+    .option('--api <name>', 'API-first area or api flow (qa/api/<name>)')
+    .option('--scenario <title|id>', 'Only the scenario whose title contains this text, or that carries this VP id')
+    .option('--json', 'Output raw JSON')
+    .action((o: { screen?: string; api?: string; scenario?: string; json?: boolean }) => {
+      try {
+        const name = o.screen || o.api;
+        if (!name) throw new Error('Provide --screen <name> (or --api <area>)');
+        const unit = resolveUnit(name, process.cwd());
+        if (!unit) throw new Error(`Not found: qa/screens/${name}, qa/flows/${name}, or qa/api/${name}`);
+        const r = draftDefects(process.cwd(), unit, { scenario: o.scenario });
+        if (o.json) { console.log(JSON.stringify(r, null, 2)); return; }
+        if (r.missingTriage) { console.log(`\n  ${name}: no triage yet — run \`sungen triage --screen ${name}\` after the tests.\n`); return; }
+        for (const f of r.written) console.log(`  ✎ ${f}`);
+        for (const s of r.skipped) console.log(`  ○ already recorded as ${s.id}: ${s.scenario}`);
+        if (!r.written.length && !r.skipped.length) console.log(`  ✓ ${name}: no product-candidate rows to draft.`);
+      } catch (error) {
+        console.error('Error:', error instanceof Error ? error.message : error);
+        process.exitCode = 1;
+      }
+    });
+
+  defect
+    .command('list')
+    .description('List defect records (id, severity, status, units, scenario count)')
+    .option('--status <status>', 'Only records with this status (open, fixed, verified, closed, wont-fix)')
+    .option('--json', 'Output raw JSON')
+    .action((o: { status?: string; json?: boolean }) => {
+      const rows = readDefects(process.cwd()).filter((r) => !o.status || r.status === o.status);
+      if (o.json) { console.log(JSON.stringify(rows, null, 2)); return; }
+      if (!rows.length) { console.log('  (no defect records)'); return; }
+      for (const r of rows) console.log(`  ${r.id}  ${r.severity.padEnd(8)} ${r.status.padEnd(9)} ${r.units.join(',')}  ${r.scenarios.length} scenario(s)  ${r.title}`);
+    });
+}