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

package/dist/harness/viewpoint-scan.d.ts

@@ -1,41 +0,0 @@
-import { ItemSource } from './viewpoint-scan-refs';
-import { ViewpointMode } from './viewpoint-mode';
-export interface VpRow {
-    id: string;
-    line: number;
-    /** Item numbers the row cites (normalised: `009` → `9`), and spec-declared ids it cites (`FR-BC-03`). */
-    refs: string[];
-    /** `# MANUAL/ENV` or `@manual` on the row or an enclosing heading. */
-    manual: boolean;
-    /** The Kỳ vọng / Expected cell has text (best effort when the table has no such header). */
-    hasExpected: boolean;
-}
-export interface ViewpointScan {
-    /** The create-test mode this file calls for (viewpoint-mode.ts): 2 = AI drafts, 1 = review only. */
-    mode: ViewpointMode;
-    modeReason: string;
-    vpRows: VpRow[];
-    /** Distinct VP ids among `vpRows` — a reused id is several rows but one id. */
-    distinctIds: number;
-    rowsWithoutExpected: string[];
-    manualRows: string[];
-    itemSource: ItemSource;
-    /** Spec/inventory items no VP row cites — a hint for the review, not a verdict. */
-    uncitedItems: {
-        ref: string;
-        name: string;
-    }[];
-    /** VP ids no feature file mentions; null when the unit has no feature file yet. */
-    withoutScenario: string[] | null;
-    /** Why `vpRows` is empty, else null. */
-    note: string | null;
-}
-export declare const NO_VP_IDS_NOTE = "file has no VP ids; review falls back to AI-only";
-/**
- * VP rows of one viewpoint text (already comment-stripped, as `readViewpointText` returns it).
- * `specIds`: the requirement ids spec.md declares — only those count as spec citations.
- */
-export declare function scanViewpointText(text: string, specIds?: ReadonlySet<string>): VpRow[];
-/** Scan one unit directory (`qa/screens/<n>`, `qa/flows/<n>`, …); `unit` names its feature files. */
-export declare function scanViewpointUnit(unitDir: string, unit: string): ViewpointScan;
-//# sourceMappingURL=viewpoint-scan.d.ts.map

package/dist/harness/viewpoint-scan.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"viewpoint-scan.d.ts","sourceRoot":"","sources":["../../src/harness/viewpoint-scan.ts"],"names":[],"mappings":"AAeA,OAAO,EAAwD,UAAU,EAAE,MAAM,uBAAuB,CAAC;AACzG,OAAO,EAAiB,aAAa,EAAE,MAAM,kBAAkB,CAAC;AAEhE,MAAM,WAAW,KAAK;IACpB,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,yGAAyG;IACzG,IAAI,EAAE,MAAM,EAAE,CAAC;IACf,sEAAsE;IACtE,MAAM,EAAE,OAAO,CAAC;IAChB,4FAA4F;IAC5F,WAAW,EAAE,OAAO,CAAC;CACtB;AAED,MAAM,WAAW,aAAa;IAC5B,oGAAoG;IACpG,IAAI,EAAE,aAAa,CAAC;IACpB,UAAU,EAAE,MAAM,CAAC;IACnB,MAAM,EAAE,KAAK,EAAE,CAAC;IAChB,+EAA+E;IAC/E,WAAW,EAAE,MAAM,CAAC;IACpB,mBAAmB,EAAE,MAAM,EAAE,CAAC;IAC9B,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,UAAU,EAAE,UAAU,CAAC;IACvB,mFAAmF;IACnF,YAAY,EAAE;QAAE,GAAG,EAAE,MAAM,CAAC;QAAC,IAAI,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IAC9C,mFAAmF;IACnF,eAAe,EAAE,MAAM,EAAE,GAAG,IAAI,CAAC;IACjC,wCAAwC;IACxC,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;CACrB;AAUD,eAAO,MAAM,cAAc,qDAAqD,CAAC;AAEjF;;;GAGG;AACH,wBAAgB,iBAAiB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,GAAE,WAAW,CAAC,MAAM,CAAa,GAAG,KAAK,EAAE,CAoBjG;AAED,qGAAqG;AACrG,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,aAAa,CA0B9E"}

package/dist/harness/viewpoint-scan.js

@@ -1,118 +0,0 @@
-"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.NO_VP_IDS_NOTE = void 0;
-exports.scanViewpointText = scanViewpointText;
-exports.scanViewpointUnit = scanViewpointUnit;
-/**
- * `sungen viewpoint scan` — the code half of reviewing a QA-written test-viewpoint.md.
- *
- * It does set arithmetic on ids and nothing else: which VP rows exist, what each cites, which spec
- * items nobody cites, which ids no scenario carries. It never judges meaning, never edits the file,
- * and feeds no audit finding, score or baseline.
- *
- * A VP row is a markdown table row whose FIRST cell is a full id (`VP-UI-001`, `FL-LOGIN-3`).
- * Priority-table rows (`VP-LOGIN (Auth)`), bullets and appendix text are not rows. This is
- * deliberately NOT `parseViewpointItems` (viewpoint-ledger.ts), which also reads id-less rows.
- */
-const fs = __importStar(require("fs"));
-const path = __importStar(require("path"));
-const read_text_1 = require("./read-text");
-const unit_paths_1 = require("./unit-paths");
-const viewpoint_scan_refs_1 = require("./viewpoint-scan-refs");
-const viewpoint_mode_1 = require("./viewpoint-mode");
-const VP_ID = /^(VP|FL)-[A-Z][A-Z0-9]*(-[A-Z0-9]+)*-\d+$/;
-const MANUAL = /\bMANUAL\/ENV\b|@manual\b/i;
-const EXPECTED_COL = /kỳ vọng|kết quả mong đợi|expected|期待/i;
-/** Whole-word match (Unicode-aware: `\b` fails on `ồ`) so `Resource` / `Datasource` are not a source column. */
-const SOURCE_COL = /(?<![\p{L}\p{N}])(?:nguồn|source|tham chiếu|ref|出典|根拠)(?![\p{L}\p{N}])/iu;
-const PRIORITY = /^(critical|high|med(?:ium)?|low|cao|trung bình|thấp)$/i;
-const BLANK = /^[\s\-–—…]*$/;
-const SPEC_ID = /(?<![\w-])[A-Z][A-Z0-9]*(?:-[A-Z0-9]+)*-\d+[A-Z]?(?![\w-])/gi;
-exports.NO_VP_IDS_NOTE = 'file has no VP ids; review falls back to AI-only';
-/**
- * VP rows of one viewpoint text (already comment-stripped, as `readViewpointText` returns it).
- * `specIds`: the requirement ids spec.md declares — only those count as spec citations.
- */
-function scanViewpointText(text, specIds = new Set()) {
-    const rows = [];
-    for (const r of (0, viewpoint_scan_refs_1.tableRows)(text)) {
-        const id = r.cells[0];
-        if (!VP_ID.test(id))
-            continue;
-        const exp = r.header?.findIndex((c) => EXPECTED_COL.test(c)) ?? -1;
-        const src = r.header?.findIndex((c) => SOURCE_COL.test(c)) ?? -1;
-        const cites = src > 0 ? r.cells[src] ?? '' : r.cells.slice(1).join(' | ');
-        rows.push({
-            id,
-            line: r.line,
-            refs: [...(0, viewpoint_scan_refs_1.extractRefs)(cites), ...[...cites.matchAll(SPEC_ID)].map((m) => m[0].toUpperCase()).filter((id) => specIds.has(id))],
-            manual: MANUAL.test(r.raw) || MANUAL.test(r.section),
-            // Without a Kỳ vọng header: any later cell with text that is neither a priority word nor a ref.
-            hasExpected: exp > 0
-                ? !BLANK.test(r.cells[exp] ?? '')
-                : r.cells.slice(2).some((c) => !BLANK.test(c) && !PRIORITY.test(c) && (0, viewpoint_scan_refs_1.extractRefs)(c).length === 0),
-        });
-    }
-    return rows;
-}
-/** Scan one unit directory (`qa/screens/<n>`, `qa/flows/<n>`, …); `unit` names its feature files. */
-function scanViewpointUnit(unitDir, unit) {
-    const reqDir = path.join(unitDir, 'requirements');
-    const vpPath = path.join(reqDir, 'test-viewpoint.md');
-    const vpText = (0, read_text_1.readViewpointText)(vpPath);
-    const universes = (0, viewpoint_scan_refs_1.itemUniverses)(reqDir);
-    const vpRows = scanViewpointText(vpText, new Set(universes.specIds.map((i) => i.ref)));
-    const { mode, reason } = (0, viewpoint_mode_1.viewpointMode)(fs.existsSync(vpPath) ? (0, read_text_1.readTextFile)(vpPath) : null, vpRows);
-    const cited = new Set(vpRows.flatMap((r) => r.refs));
-    const { source, items } = (0, viewpoint_scan_refs_1.citedUniverse)(universes, cited, /(?<![\w-])UI-\d{1,3}\b/i.test(vpText));
-    const features = (0, unit_paths_1.featureFilesFor)(unitDir, unit).filter((f) => fs.existsSync(f));
-    // Same rule as the ledger (viewpoint-ledger.ts): an id is carried when the feature text contains it.
-    const featureText = features.map((f) => (0, read_text_1.readTextFile)(f).toLowerCase()).join('\n');
-    return {
-        mode,
-        modeReason: reason,
-        vpRows,
-        distinctIds: new Set(vpRows.map((r) => r.id)).size,
-        rowsWithoutExpected: vpRows.filter((r) => !r.hasExpected).map((r) => r.id),
-        manualRows: vpRows.filter((r) => r.manual).map((r) => r.id),
-        itemSource: source,
-        uncitedItems: items.filter((i) => !cited.has(i.ref)),
-        withoutScenario: features.length === 0 ? null : vpRows.filter((r) => !featureText.includes(r.id.toLowerCase())).map((r) => r.id),
-        note: vpRows.length > 0 ? null
-            : fs.existsSync(vpPath) ? exports.NO_VP_IDS_NOTE : 'requirements/test-viewpoint.md is missing',
-    };
-}
-//# sourceMappingURL=viewpoint-scan.js.map

package/dist/harness/viewpoint-scan.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"viewpoint-scan.js","sourceRoot":"","sources":["../../src/harness/viewpoint-scan.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;AA6DA,8CAoBC;AAGD,8CA0BC;AA9GD;;;;;;;;;;GAUG;AACH,uCAAyB;AACzB,2CAA6B;AAC7B,2CAA8D;AAC9D,6CAA+C;AAC/C,+DAAyG;AACzG,qDAAgE;AA+BhE,MAAM,KAAK,GAAG,2CAA2C,CAAC;AAC1D,MAAM,MAAM,GAAG,4BAA4B,CAAC;AAC5C,MAAM,YAAY,GAAG,uCAAuC,CAAC;AAC7D,gHAAgH;AAChH,MAAM,UAAU,GAAG,0EAA0E,CAAC;AAC9F,MAAM,QAAQ,GAAG,wDAAwD,CAAC;AAC1E,MAAM,KAAK,GAAG,cAAc,CAAC;AAC7B,MAAM,OAAO,GAAG,8DAA8D,CAAC;AAClE,QAAA,cAAc,GAAG,kDAAkD,CAAC;AAEjF;;;GAGG;AACH,SAAgB,iBAAiB,CAAC,IAAY,EAAE,UAA+B,IAAI,GAAG,EAAE;IACtF,MAAM,IAAI,GAAY,EAAE,CAAC;IACzB,KAAK,MAAM,CAAC,IAAI,IAAA,+BAAS,EAAC,IAAI,CAAC,EAAE,CAAC;QAChC,MAAM,EAAE,GAAG,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC;QACtB,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC;YAAE,SAAS;QAC9B,MAAM,GAAG,GAAG,CAAC,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC;QACnE,MAAM,GAAG,GAAG,CAAC,CAAC,MAAM,EAAE,SAAS,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC;QACjE,MAAM,KAAK,GAAG,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,GAAG,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QAC1E,IAAI,CAAC,IAAI,CAAC;YACR,EAAE;YACF,IAAI,EAAE,CAAC,CAAC,IAAI;YACZ,IAAI,EAAE,CAAC,GAAG,IAAA,iCAAW,EAAC,KAAK,CAAC,EAAE,GAAG,CAAC,GAAG,KAAK,CAAC,QAAQ,CAAC,OAAO,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC,CAAC;YAC7H,MAAM,EAAE,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,GAAG,CAAC,IAAI,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC;YACpD,gGAAgG;YAChG,WAAW,EAAE,GAAG,GAAG,CAAC;gBAClB,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,KAAK,CAAC,GAAG,CAAC,IAAI,EAAE,CAAC;gBACjC,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,IAAA,iCAAW,EAAC,CAAC,CAAC,CAAC,MAAM,KAAK,CAAC,CAAC;SACrG,CAAC,CAAC;IACL,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED,qGAAqG;AACrG,SAAgB,iBAAiB,CAAC,OAAe,EAAE,IAAY;IAC7D,MAAM,MAAM,GAAG,IAAI,CAAC,IAAI,CAAC,OAAO,EAAE,cAAc,CAAC,CAAC;IAClD,MAAM,MAAM,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,mBAAmB,CAAC,CAAC;IACtD,MAAM,MAAM,GAAG,IAAA,6BAAiB,EAAC,MAAM,CAAC,CAAC;IACzC,MAAM,SAAS,GAAG,IAAA,mCAAa,EAAC,MAAM,CAAC,CAAC;IACxC,MAAM,MAAM,GAAG,iBAAiB,CAAC,MAAM,EAAE,IAAI,GAAG,CAAC,SAAS,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;IACvF,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,GAAG,IAAA,8BAAa,EAAC,EAAE,CAAC,UAAU,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,IAAA,wBAAY,EAAC,MAAM,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;IACpG,MAAM,KAAK,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC;IACrD,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,GAAG,IAAA,mCAAa,EAAC,SAAS,EAAE,KAAK,EAAE,yBAAyB,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC;IAClG,MAAM,QAAQ,GAAG,IAAA,4BAAe,EAAC,OAAO,EAAE,IAAI,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,EAAE,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,CAAC;IAChF,qGAAqG;IACrG,MAAM,WAAW,GAAG,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,IAAA,wBAAY,EAAC,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAElF,OAAO;QACL,IAAI;QACJ,UAAU,EAAE,MAAM;QAClB,MAAM;QACN,WAAW,EAAE,IAAI,GAAG,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI;QAClD,mBAAmB,EAAE,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,WAAW,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC1E,UAAU,EAAE,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC3D,UAAU,EAAE,MAAM;QAClB,YAAY,EAAE,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC;QACpD,eAAe,EAAE,QAAQ,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,WAAW,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,CAAC,WAAW,EAAE,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAChI,IAAI,EAAE,MAAM,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,IAAI;YAC5B,CAAC,CAAC,EAAE,CAAC,UAAU,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,sBAAc,CAAC,CAAC,CAAC,2CAA2C;KACzF,CAAC;AACJ,CAAC"}

package/dist/orchestrator/templates/ai-src/skills/sungen-viewpoint/review-checklist.md

@@ -1,63 +0,0 @@
-# Viewpoint review checklist
-
-create-test runs this review of `requirements/test-viewpoint.md` inline, in both modes, and asks
-no question. Inputs: the viewpoint; the scan JSON (`sungen viewpoint scan --screen <name> --json`,
-a flow takes `--flow <name>`); `spec.md`, `ui-inventory.md` and the message table when present.
-Output: `requirements/viewpoint-review.md` (format below).
-
-## Rules
-- **Mode 1 — QA file (no `drafted-by: ai`): read only.** Never edit `test-viewpoint.md`. Every gap
-  goes into the review file; nothing is filled in. Look for what is **missing or contradicts the
-  spec**, not for how the QA writes: run only the checks the `Mode 1` column allows, as narrowed
-  there. A value the spec does not mention is not "invented". Wording the QA checks by eye ("Hiển thị đúng", "Khớp design"), their question style and
-  their environment marks are the QA's convention, never a finding.
-- **Mode 2 — `drafted-by: ai`: review your own draft.** Run the checks, fix the draft, run them
-  again — checks 4 and 9 field by field, before the review is written. The review file keeps only
-  what needs a QA decision: questions, inferred expectations, scope calls.
-- **Cite or drop.** Every finding names at least one VP id from the scan's `vpRows`, or an item ref
-  (`item N`, a spec id). A gap with no VP yet cites the item or spec line it comes from. A finding
-  you cannot anchor is not written.
-- **Distinct ids.** The scan lists a row per table occurrence: an id repeated in the questions table
-  (mục 5) is a reference, not a second VP. Count and cite each id once.
-- **No VP ids** (scan `note`: file has no VP ids): cite the file's own row labels (`DT-2 case 3`, a
-  field name) with the line number, plus item refs.
-- **Meaning, not format.** Headings, column order, wording style and language are never findings.
-- The scan's `withoutScenario` is generation work (a scenario per VP id), not a review finding.
-- **Audit findings** (create-test may route one here, check `audit`) enter only when they cite a VP id
-  from the scan's `vpRows` and the fix is an edit the QA can make in this file. A finding about
-  sungen itself — the page type, an id it read in a message code or a section-8 table — stays in the
-  hand-back, never in the review.
-
-## Checks
-One check, one fix hint. **Output order: 2, 9, 3, 6, 1, 4, 5, 7, 8** — highest impact first.
-
-| # | Check | Mode 1 | Read | Suggestion to write |
-|---|-------|--------|------|---------------------|
-| 1 | Each VP row says what to test and what to expect: `Kỳ vọng` is not empty and not vague ("OK", "đúng", "hiển thị đúng", "như spec"). | empty `Kỳ vọng` only | scan `rowsWithoutExpected`, the rows | Name the observable: message code, target screen, control state, value. |
-| 2 | Every expectation is traceable to the spec, the UI, the message table, a Design Decision (mục 6) or a Shared Check of this skill. One found in none of them is invented, unless mục 5 holds its question (`Nguồn: Suy luận → Q-0x`). | only an expectation that contradicts what the spec states; a value the spec does not mention (a URL, a screen id, a rule of thumb) is the QA's knowledge, not a finding | spec, UI, message table, mục 6 | Cite the source, or turn the expectation into a closed question in mục 5. |
-| 3 | A rule the spec applies to several fields names every field (`Ô` column, or one row per field). | ✓ | spec rules | Name the missing fields. |
-| 4 | Each accepted-input class in the spec (whitelist, value list, allowed characters, conversion such as trim, case, full→half width) has a positive VP — also a whitelist or list the spec names but does not reproduce. | ✓ | spec rules | Add an "accepted, no error" VP for the class, citing the list by name; not a question in mục 5. |
-| 5 | A VP that needs a real device, account, external service or elapsed time is marked `# MANUAL/ENV`. | — | scan `manualRows`, the rows | Mark the row or its group heading. |
-| 6 | Each entry of the scan's `uncitedItems`: deliberately out of scope (mục 6 or the scope note says so), or a gap? | ✓ | scan `uncitedItems`, mục 6, mục 7 | Add a VP citing `item N`, or a Design Decision saying why it is out. |
-| 7 | Questions are closed (yes/no, or pick one of the listed options) and carry `Major` / `Minor`. | — | mục 5 | Rewrite as a closed question; add the level. |
-| 8 | An edge case (mục 4) or decision (mục 6) states testable behaviour that no VP id carries — alone or together: an edge case split across several rows is carried. | ✓ | mục 4, mục 6 | Add a VP row. If a row already covers it, name that id in this finding — not in mục 4, where an id counts as a second declaration (`VIEWPOINT-ID-REUSED`). |
-| 9 | A row's `Kỳ vọng` is only a question pointer (`→ Q-0x`) although the spec states the expected result for that condition. | ✓ | spec, mục 5 | Write the spec's expectation in the row; keep the question for the part the spec leaves open. |
-
-## Output — `requirements/viewpoint-review.md`
-Overwrite it on every run, never append. Write Problem and Suggestion in the viewpoint's language;
-one row per finding, or one row listing every id that shares the same fix.
-
-```markdown
-# Viewpoint review — <unit>
-> Date: <YYYY-MM-DD> · test-viewpoint.md sha256: <hash> · Mode: <1 — QA file, review only | 2 — AI draft>
-> QA: tick `[x]` once handled, or delete the row. `sungen audit` reports the rows still open.
-
-| # | VP / item | Check | Problem | Suggestion | [ ] |
-|---|-----------|-------|---------|------------|-----|
-| 1 | VP-…-NNN · item N | 2 | <what is wrong, in one sentence> | <the fix> | [ ] |
-```
-
-- `sha256` of `test-viewpoint.md` as reviewed (`shasum -a 256` / `sha256sum`); mode 2: after your fixes.
-- At most 20 rows, in check order. Past 20, one line under the table: `+N more (checks …)`.
-- Nothing to report: keep the header, write `No open points.` instead of the table.
-- The hand-back counts the rows: `Viewpoint review: N điểm cần QA xem`.

package/dist/orchestrator/templates/ai-src/skills/sungen-viewpoint/viewpoint-template.md

@@ -1,158 +0,0 @@
-# Test Viewpoint — <MÃ MÀN> <Tên màn>
-<!--
-Template Test Viewpoint (skill sungen-viewpoint). Dòng mẫu và hướng dẫn nằm trong comment, nên khung chưa điền khai báo 0 viewpoint. Điền: chép dòng mẫu ra ngoài comment rồi sửa; nhóm không áp dụng thì xoá, ghi lý do ở bảng Coverage (mục 7).
-Nháp do AI soạn (unit chưa có viewpoint QA): dòng ĐẦU TIÊN của file là `drafted-by: ai`, đặt ngoài comment. Nháp không bao giờ được lấy làm chuẩn đo; QA duyệt xong thì accept (`sungen audit … --accept-viewpoint`), accept xoá dòng đó và từ đó file thuộc QA. `sungen add` / `sungen add-flow` không ghi dòng này.
--->
-> **Tài liệu đầu vào:** <đặc tả / design / bảng message của màn>
-> **Loại tài liệu:** bảng quan điểm kiểm thử — KHÔNG phải test case chi tiết. Mỗi quan điểm gắn mã `VP-…` để truy vết sang test case.
-> **Nguyên tắc:** mỗi dòng trả lời câu hỏi "test CÁI GÌ" và "kỳ vọng ra sao"; điểm spec chưa rõ thì ghi ở mục 5, **không tự bịa kỳ vọng** — spec đã nói thì ghi kỳ vọng theo spec, chỉ phần spec bỏ ngỏ mới thành câu hỏi. Quan điểm cần môi trường/thiết bị/thời gian thật: đánh dấu `# MANUAL/ENV`.
-
-## 0. Metadata
-| Trường | Giá trị |
-|--------|---------|
-| Màn hình | <mã + tên gốc / tên tiếng Việt, route> |
-| Vị trí trong flow | <vào từ đâu, đi tiếp tới đâu> |
-| Primary UI pattern | <form / list / detail / create / login / search…> |
-| Nền tảng | <trình duyệt / thiết bị / độ phân giải cần test> |
-| Đặc thù cần nhấn | <rủi ro hoặc đặc điểm quan trọng nhất của màn> |
-| Tiền điều kiện | <đăng nhập? role? dữ liệu cần chuẩn bị?> |
-| Ngày tạo viewpoint | <YYYY-MM-DD> |
-
-## 1. UI Patterns Identified
-*(Nhận diện màn dùng pattern nào → quyết định nhóm viewpoint nào cần áp dụng.)*
-
-| # | Pattern | Áp dụng? | Ghi chú |
-|---|---------|----------|---------|
-<!-- Áp dụng? = ✅ / ❌ (❌ thì ghi lý do). Một dòng cho mỗi pattern, số + tên theo catalog sungen-viewpoint (web 1–17, mobile thêm 18–32):
-| 1 | Form & Inputs | ✅ | <các ô nhập, validate lúc nào> |
-| 12 | Modal / Dialog | ❌ | <lý do không áp dụng> |
--->
-
-## 2. Priority Viewpoints
-### Viewpoint grouping — ưu tiên theo nhóm VP
-| Nhóm VP | Priority | Lý do |
-|---------|----------|-------|
-<!-- Priority = High / Med / Low; một dòng cho mỗi nhóm có ở mục 3: VP-VAL (Validation) · VP-LOGIC (Nghiệp vụ) · VP-UI-BTN (Nút & trạng thái) · VP-NAV (Điều hướng) · VP-AUTH (Quyền / tiền điều kiện) · VP-SEC (Bảo mật) · VP-UI (Hiển thị). Ví dụ:
-| VP-VAL (Validation) | High | <lý do> | -->
-
-## 3. Viewpoints chi tiết
-*(Pri = priority. Nguồn = số item trong spec/UI. Mỗi dòng = "test cái gì".)*
-<!--
-- Mọi bảng mục 3 dùng cùng cột. Mã VP-<NHÓM>[-<FIELD>]-NNN ở cột đầu, mỗi mã khai báo một lần. Flow: mã FL-…, thêm nhóm liên màn (dữ liệu / quyền giữ qua các màn).
-- Kỳ vọng = thứ nhìn thấy được: mã message, màn đích, trạng thái control. Kỳ vọng trống thì dòng KHÔNG tính là viewpoint.
-- Ô: để trống khi cả nhóm là một ô. Một luật áp cho nhiều ô: liệt kê đủ từng ô, không lấy 1 ô làm đại diện.
-- Pri = High / Med / Low. Nguồn = `item N` · `SPEC <id>` · `Suy luận → Q-0x` (suy luận bắt buộc có câu hỏi ở mục 5). Cần thiết bị / tài khoản / thời gian thật: `# MANUAL/ENV` trong ô Quan điểm hoặc tiêu đề nhóm.
--->
-
-### A. VP-UI — Hiển thị tĩnh & trạng thái ban đầu
-| VP | Quan điểm kiểm thử | Ô | Kỳ vọng | Pri | Nguồn |
-|----|--------------------|---|---------|-----|-------|
-<!-- | VP-UI-001 | Trạng thái lúc mở màn | | <nhãn, placeholder, giá trị mặc định, nút chính disabled / enabled> | | item N | -->
-
-### B. VP-VAL-<FIELD> — <Tên field> (item x)
-| VP | Quan điểm kiểm thử | Ô | Kỳ vọng | Pri | Nguồn |
-|----|--------------------|---|---------|-----|-------|
-<!--
-| VP-VAL-…-001 | Rỗng → blur / submit | | → <mã message> | High | |
-| VP-VAL-…-002 | Chỉ khoảng trắng / khoảng trắng đầu-cuối | | | | |
-| VP-VAL-…-003 | Biên độ dài: min−1 / min / max / max+1 | | | High | |
-| VP-VAL-…-004 | Sai định dạng / ký tự không cho phép | | | | |
-| VP-VAL-…-008 | Mọi ký tự / giá trị spec cho phép (whitelist, danh sách, chuyển đổi) được chấp nhận | | Không lỗi, giá trị giữ nguyên / đã chuyển đổi | High | |
-| VP-VAL-…-005 | Full-width / half-width | | | | |
-| VP-VAL-…-006 | Paste / autofill | | | | |
-| VP-VAL-…-007 | Đang lỗi → sửa đúng → message biến mất | | | | |
--->
-*(Lặp nhóm B cho từng field.)*
-
-### C. VP-LOGIC — Nghiệp vụ chính / submit
-| VP | Quan điểm kiểm thử | Ô | Kỳ vọng | Pri | Nguồn |
-|----|--------------------|---|---------|-----|-------|
-<!--
-| VP-LOGIC-001 | Happy path | | | High | |
-| VP-LOGIC-002 | Server trả lỗi nghiệp vụ | | | | |
-| VP-LOGIC-003 | Lỗi mạng / server `# MANUAL/ENV` | | | | |
--->
-
-### D. VP-UI-BTN — Nút & trạng thái
-| VP | Quan điểm kiểm thử | Ô | Kỳ vọng | Pri | Nguồn |
-|----|--------------------|---|---------|-----|-------|
-<!--
-| VP-UI-BTN-001 | Chưa đủ điều kiện → disabled | | | High | |
-| VP-UI-BTN-002 | Đủ điều kiện → enabled | | | High | |
-| VP-UI-BTN-003 | Loading sau khi bấm | | | | |
-| VP-UI-BTN-004 | Bấm kép → chỉ 1 request `# MANUAL/ENV` | | | High | |
--->
-
-### E. VP-NAV — Điều hướng
-| VP | Quan điểm kiểm thử | Ô | Kỳ vọng | Pri | Nguồn |
-|----|--------------------|---|---------|-----|-------|
-<!-- | VP-NAV-001 | | | → <màn đích>, cùng tab / tab mới | | | -->
-
-### F. VP-AUTH — Quyền / tiền điều kiện truy cập (nếu có)
-### G. VP-SEC — Bảo mật (nếu có)
-### H. VP-UI-PLTF — Nền tảng / thiết bị (nếu có)
-
-### DT-1 — Bảng quyết định: <luật phụ thuộc ≥ 2 điều kiện> (nếu có)
-<!-- Mỗi Case = 1 dòng có mã VP. Cột giữa là điều kiện (✓ / ✗ / giá trị), rồi Hành động và Kỳ vọng. Ví dụ:
-| VP | <Điều kiện 1> | <Điều kiện 2> | Hành động | Kỳ vọng | Pri | Nguồn |
-|----|---------------|---------------|-----------|---------|-----|-------|
-| VP-LOGIC-010 | ✓ | ✗ | Tick nốt ô con còn lại | "Chọn tất cả" tự → ✓ | High | item N |
--->
-
-## 4. Edge Cases
-*(Ca dễ bị sót khi viết test case.)*
-<!--
-- <biên nối tiếp: max rồi max+1>
-- <bấm nút đúng lúc field vừa thành không hợp lệ>
-- <mất mạng trước khi bấm vs trong lúc chờ phản hồi>
-- <phiên hết hạn giữa chừng>
--->
-
-## 5. Open questions / Điểm CHƯA CHỐT
-*(Spec chưa nêu → KHÔNG tự bịa kỳ vọng; ghi thành câu hỏi đóng để hỏi BA/dev.)*
-
-| Mã | Mức | VP liên quan | Câu hỏi đóng cần BA/dev xác nhận |
-|-----|-----|--------------|----------------------------------|
-<!-- Mức: Major = test case của VP liên quan chờ câu trả lời; Minor = không chặn. -->
-<!-- | Q-01 | Major / Minor | VP-… | <câu hỏi cụ thể, trả lời được bằng có/không hoặc chọn 1 phương án> | -->
-
-## Known issues — lỗi đã biết (KHÔNG phải câu hỏi)
-*(Lỗi đã xác nhận, còn tồn tại: scenario của VP đó gắn `@known-defect:DEF-NNN` (bản ghi ở `qa/defects/`), đỏ có chủ đích. Spec chưa rõ thì ghi ở mục 5.)*
-
-| KI | VP liên quan | Lỗi | Hiện tượng |
-|----|--------------|-----|------------|
-<!-- | KI-1 | VP-… | DEF-001 | <thấy gì, khác Kỳ vọng ở đâu> | -->
-
-## 6. Design Decisions
-*(Quyết định có chủ đích của spec — KHÔNG báo "thiếu" hoặc "bug" cho các mục này.)*
-<!--
-- D1: <vd: nút disabled tới khi đủ điều kiện> (SPEC <id>)
-- D2: <vd: lỗi đăng nhập không nói rõ sai email hay sai mật khẩu>
--->
-
-## 7. Traceability & tự nhận giới hạn
-- **Tổng: N viewpoint** — liệt kê số lượng theo từng nhóm. Mọi item trong spec có ít nhất 1 VP trỏ về.
-- ⚠ **Giới hạn:** bảng chứng minh "mọi VP đã liệt kê có kỳ vọng" — KHÔNG chứng minh "đã liệt kê ĐỦ VP". Đối chiếu lại với đặc tả gốc + bảng message trước khi viết test case.
-- **Scope note:** <VP nào cần môi trường/thiết bị thật, VP nào test ở tầng API/E2E>.
-- Còn **K điểm chưa chốt** (Q-…) — test case cho VP liên quan chờ câu trả lời.
-
-### Coverage — độ phủ tự kiểm
-<!-- Mỗi nhóm một dòng. Trạng thái: `có VP` (✓) · `để sau → <đích>` (# MANUAL/ENV, unit/flow khác, Q-0x) · `Không áp dụng` (➖) kèm lý do.
-Thêm dòng nền tảng theo catalog sungen-viewpoint khi màn có: web — XSS / SQL injection / URL (Shared Checks); mobile — Bàn phím ảo (30) · Vòng đời app & xoay màn (32) · Offline (27) · Quyền hệ thống (22) · Deep-link (26). -->
-| Nhóm kiểm tra | Trạng thái | VP / lý do |
-|---------------|------------|------------|
-| Happy path | | |
-| Negative | | |
-| Biên | | |
-| Charset | | |
-| Trạng thái nút | | |
-| Điều hướng | | |
-| Quyền | | |
-| Bảo mật | | |
-| Nền tảng | | |
-| Trạng thái tải / rỗng / lỗi | | |
-
-## 8. References · Phụ lục — Mã thông báo dùng trong tài liệu
-| Mã | Nghĩa | Kiểu hiển thị |
-|----|-------|---------------|
-<!-- | M01 | <nội dung message> | <dưới ô nhập / trên form / toast / mail> | -->

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

@@ -1,3 +0,0 @@
-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

@@ -1 +0,0 @@
-{"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

@@ -1,70 +0,0 @@
-"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

@@ -1 +0,0 @@
-{"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/src/cli/commands/viewpoint.ts

@@ -1,58 +0,0 @@
-import { Command } from 'commander';
-import { resolveUnit } from '../resolve-unit';
-import { scanViewpointUnit, ViewpointScan } from '../../harness/viewpoint-scan';
-
-const SHOWN = 10;
-
-/** `a, b, c … +N more` — a list long enough to scroll past is a count plus a sample. */
-const sample = (xs: string[]): string => (xs.length <= SHOWN ? xs.join(', ') : `${xs.slice(0, SHOWN).join(', ')} … +${xs.length - SHOWN} more`);
-
-function printScan(name: string, s: ViewpointScan): void {
-  const L = console.log;
-  L(`\n━━━ Viewpoint scan: ${name} ━━━`);
-  L(`  mode ${s.mode} (${s.modeReason}) — ${s.mode === 2 ? 'the AI drafts / revises its own draft' : 'review only, the file is never edited'}`);
-  L(`  ${s.vpRows.length} rows (${s.distinctIds} distinct ids)   without Kỳ vọng: ${s.rowsWithoutExpected.length}   manual: ${s.manualRows.length}`);
-  if (s.note) L(`  ⚠ ${s.note}`);
-  if (s.rowsWithoutExpected.length) L(`    no Kỳ vọng: ${sample(s.rowsWithoutExpected)}`);
-  if (s.manualRows.length) L(`    manual: ${sample(s.manualRows)}`);
-  if (s.itemSource === 'none') L('  item source: none (the viewpoint cites no spec id, spec item or UI-NNN)');
-  else L(`  item source: ${s.itemSource}   uncited items: ${s.uncitedItems.length}`);
-  if (s.uncitedItems.length) L(`    ${sample(s.uncitedItems.map((i) => (i.name ? `${i.ref} (${i.name})` : i.ref)))}`);
-  if (s.withoutScenario === null) L('  scenarios: no feature file yet');
-  else L(`  VP ids with no scenario: ${s.withoutScenario.length}${s.withoutScenario.length ? `\n    ${sample(s.withoutScenario)}` : ''}`);
-  L('');
-}
-
-/**
- * `sungen viewpoint scan` — ids and item refs only. Read-only: no audit finding, no score, no
- * baseline effect; the file is never edited. What the rows MEAN is the review skill's job.
- */
-export function registerViewpointCommand(program: Command): void {
-  const viewpoint = program
-    .command('viewpoint')
-    .description('Read a unit\'s requirements/test-viewpoint.md');
-
-  viewpoint
-    .command('scan')
-    .description('List VP rows (full-id first cell), the item numbers they cite, uncited spec items and VP ids no scenario carries — read-only')
-    .option('-s, --screen <name>', 'Screen (or flow) name')
-    .option('--flow <name>', 'Flow name (qa/flows/<name>)')
-    .option('--json', 'Output the scan as JSON')
-    .action((o: { screen?: string; flow?: string; json?: boolean }) => {
-      try {
-        if (o.screen && o.flow) throw new Error('Provide --screen <name> or --flow <name>, not both');
-        const name = o.screen || o.flow;
-        if (!name) throw new Error('Provide --screen <name> or --flow <name>');
-        const unit = resolveUnit(name);
-        if (!unit) throw new Error(`Not found: qa/screens/${name} or qa/flows/${name}`);
-        if (o.flow && unit.kind !== 'flow') throw new Error(`Not a flow: ${name} resolves to qa/${unit.unitId} (use --screen)`);
-
-        const scan = scanViewpointUnit(unit.dir, name);
-        if (o.json) console.log(JSON.stringify(scan, null, 2));
-        else printScan(name, scan);
-      } catch (e) {
-        console.error('Error:', e instanceof Error ? e.message : e);
-        process.exit(1);
-      }
-    });
-}

package/src/harness/viewpoint-draft-header.ts

@@ -1,55 +0,0 @@
-import * as fs from 'fs';
-import { stripHtmlComments } from './read-text';
-import { writeFileAtomic } from './write-atomic';
-
-/**
- * `drafted-by: ai` — provenance, not acceptance. The AI writes it as the first header line of a
- * viewpoint it drafted; while it is there the file is an unreviewed draft: the audit records no
- * baseline for it and holds the two axes measured against it. `--accept-viewpoint` removes it.
- *
- * Read by a line scan, not by the viewpoint parser: the key must be found in a file the parser
- * judges nothing about, and in a QA-written file it simply is not there.
- */
-const DRAFT_KEY = /^\s*drafted-by:\s*ai\b/i;
-/** The key is a header line: looking further down would read prose that merely mentions it. */
-const HEADER_LINES = 15;
-
-/** A line with its own terminator, so a removal keeps CRLF and a lone `\r` exactly as written. */
-const withTerminator = (raw: string): string[] => raw.match(/[^\r\n]*(?:\r\n|\r|\n)|[^\r\n]+$/g) ?? [];
-
-/** Index of the draft line in `raw`, or -1. Comments are blanked line-for-line, so indexes agree with the raw file. */
-function draftLineIndex(raw: string): number {
-  const visible = stripHtmlComments(raw.replace(/\r\n?/g, '\n')).split('\n');
-  let seen = 0;
-  for (let i = 0; i < visible.length && seen < HEADER_LINES; i++) {
-    if (visible[i].trim() === '') continue;
-    if (DRAFT_KEY.test(visible[i])) return i;
-    seen++;
-  }
-  return -1;
-}
-
-export function isAiDraft(text: string): boolean {
-  return draftLineIndex(text) >= 0;
-}
-
-/** `raw` without the one draft line; every other byte is untouched. A file with no draft line comes back as is. */
-export function removeDraftLine(raw: string): string {
-  const at = draftLineIndex(raw);
-  if (at < 0) return raw;
-  return withTerminator(raw).filter((_, i) => i !== at).join('');
-}
-
-export function viewpointFileIsAiDraft(viewpointPath: string): boolean {
-  try { return isAiDraft(fs.readFileSync(viewpointPath, 'utf-8')); } catch { return false; }
-}
-
-/** Remove the draft line from the file, atomically. True when a line was removed; throws a clear error when the write fails. */
-export function removeDraftLineFromFile(viewpointPath: string): boolean {
-  const raw = fs.readFileSync(viewpointPath, 'utf-8');
-  if (!isAiDraft(raw)) return false;
-  try { writeFileAtomic(viewpointPath, removeDraftLine(raw)); } catch (err) {
-    throw new Error(`could not remove the \`drafted-by: ai\` line from ${viewpointPath} (${err instanceof Error ? err.message : err}) — the file is unchanged; remove the line by hand`);
-  }
-  return true;
-}