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

package/src/harness/viewpoint-mode.ts

@@ -1,49 +0,0 @@
-/**
- * Which create-test mode a unit's test-viewpoint.md calls for — the table in create-test.md,
- * computed so the agent reads it from `sungen viewpoint scan` instead of inferring it.
- *
- *   mode 2 (AI drafts / re-reviews its draft): file missing · untouched scaffold · first line
- *          `drafted-by: ai` · VP rows exist but none has a Kỳ vọng
- *   mode 1 (review only, never edit): anything else — a QA-written file, with or without VP ids
- *
- * "Untouched scaffold" is checked on RAW lines against the template the scaffold is written from:
- * apart from the first `# ` line (the title), every non-blank line must be a template line —
- * comments included, so a QA note inside `<!-- -->` or under its own `# ` heading makes the file
- * the QA's (owner decision 2026-10-14: a QA file is never overwritten). Every doubt falls to
- * mode 1: a scaffold from an older sungen (other template lines) or an unreadable template.
- */
-import { isAiDraft } from './viewpoint-draft-header';
-import { viewpointScaffold } from '../orchestrator/viewpoint-scaffold';
-
-export type ViewpointMode = 1 | 2;
-
-/** Trimmed non-blank lines without the title (the first `# ` line), BOM and CRLF ignored. */
-function bodyLines(text: string): string[] {
-  const lines = text.replace(/^/, '').replace(/\r\n?/g, '\n').split('\n').map((l) => l.trim()).filter(Boolean);
-  const title = lines.findIndex((l) => l.startsWith('# '));
-  if (title >= 0) lines.splice(title, 1);
-  return lines;
-}
-
-let templateLines: Set<string> | null = null;
-function isUntouchedScaffold(raw: string): boolean {
-  if (!templateLines) {
-    try {
-      templateLines = new Set([false, true].flatMap((mobile) => bodyLines(viewpointScaffold('', mobile))));
-    } catch {
-      return false;   // no template to compare with → treat the file as the QA's
-    }
-  }
-  const known = templateLines;
-  return bodyLines(raw).every((l) => known.has(l));
-}
-
-/** `raw` is the file as written (null when missing); `rows` are its VP rows from the scan. */
-export function viewpointMode(raw: string | null, rows: { hasExpected: boolean }[]): { mode: ViewpointMode; reason: string } {
-  if (raw === null) return { mode: 2, reason: 'file missing' };
-  if (isAiDraft(raw)) return { mode: 2, reason: 'drafted-by: ai (re-review)' };
-  if (rows.length > 0) {
-    return rows.some((r) => r.hasExpected) ? { mode: 1, reason: 'QA-written' } : { mode: 2, reason: 'VP rows without Kỳ vọng' };
-  }
-  return isUntouchedScaffold(raw) ? { mode: 2, reason: 'untouched scaffold' } : { mode: 1, reason: 'QA-written, no VP ids' };
-}

package/src/harness/viewpoint-priority.ts

@@ -1,38 +0,0 @@
-/**
- * One reading of a priority — a viewpoint row's cell or a scenario's tag.
- *
- * The overview parser, the ledger's uniform-priority check and PRIORITY-CONTRADICTED each read a
- * priority their own way (`/high/i` substrings, raw words, a one-off medium→normal), so the same
- * row could be `Medium` to one and `Unknown` to another — `Med`, which QA writes, was `Unknown`.
- * The vocabulary is `high | normal | low`; `critical`, `medium` and `med` are read as aliases.
- */
-export type PriorityLevel = 'high' | 'normal' | 'low' | 'deferred' | 'skip' | 'unknown';
-
-const WORD: Record<string, PriorityLevel> = {
-  critical: 'high', high: 'high',
-  medium: 'normal', med: 'normal', normal: 'normal',
-  low: 'low', deferred: 'deferred',
-};
-/** "Declared not applicable" — one level, so every consumer checks one word. */
-const NOT_APPLICABLE = /^(?:skip(?:ped)?|n\/a|none|not applicable)$/;
-
-/**
- * A priority cell or tag → its level. The whole cell being a word is the clean case; otherwise the
- * overview has always read prose by SUBSTRING in precedence order (`High — complex form`,
- * `Medium-High`), and keeps doing so. The substring words stay unbounded for `high|medium|low`
- * (existing behaviour); the newer `critical|normal|med` need word boundaries (`Medical` is not `Med`).
- */
-export function parsePriority(cell: string): PriorityLevel {
-  const w = cell.replace(/[*_`]/g, '').replace(/\.+$/, '').trim().toLowerCase();
-  if (WORD[w]) return WORD[w];
-  if (NOT_APPLICABLE.test(w)) return 'skip';
-  if (/high|\bcritical\b/.test(w)) return 'high';
-  if (/medium|\bnormal\b|\bmed\b/.test(w)) return 'normal';
-  if (/low/.test(w)) return 'low';
-  return 'unknown';
-}
-
-/** The public `ViewpointEntry.priority` spelling of a level — the exported type is unchanged. */
-export function toEntryPriority(level: PriorityLevel): 'High' | 'Medium' | 'Low' | 'Unknown' {
-  return level === 'high' ? 'High' : level === 'normal' ? 'Medium' : level === 'low' ? 'Low' : 'Unknown';
-}

package/src/harness/viewpoint-review-open.ts

@@ -1,51 +0,0 @@
-/**
- * VIEWPOINT-REVIEW-OPEN (info) — the audit's one line about `requirements/viewpoint-review.md`.
- *
- * create-test reviews test-viewpoint.md with the sungen-viewpoint checklist and writes each point a
- * QA should look at as a table row whose last cell is `[ ]`; the QA ticks it `[x]` or deletes the
- * row. While rows stay unticked the audit says how many, once. Advisory only: no score, no gate, no
- * baseline effect. A missing or unreadable file says nothing — the review is optional input.
- */
-import * as fs from 'fs';
-import * as path from 'path';
-import { stripHtmlComments } from './read-text';
-
-export const VIEWPOINT_REVIEW_FILE = 'viewpoint-review.md';
-
-const SEPARATOR = /^\s*\|?\s*:?-{2,}:?\s*(\|\s*:?-{2,}:?\s*)*\|?\s*$/;
-/** A table separator, not a `---` rule under a row. */
-const isSeparator = (line: string): boolean => SEPARATOR.test(line) && line.includes('|');
-const UNTICKED = /^\[\s?\]$/;
-
-/** The last cell of a markdown table row, or null for a line that is not one. */
-function lastCell(line: string): string | null {
-  const t = line.trim();
-  if (!t.startsWith('|') || !t.endsWith('|') || t.length < 2) return null;
-  const cells = t.slice(1, -1).split(/(?<!\\)\|/);
-  return cells[cells.length - 1].trim();
-}
-
-/** Body rows whose last cell is `[ ]`. The header row (`… | [ ] |` above `|---|`) is not a point. */
-export function openReviewPoints(text: string): number {
-  const lines = stripHtmlComments(text.replace(/\r\n?/g, '\n')).split('\n');
-  let open = 0;
-  for (let i = 0; i < lines.length; i++) {
-    if (isSeparator(lines[i]) || isSeparator(lines[i + 1] ?? '')) continue;
-    const last = lastCell(lines[i]);
-    if (last !== null && UNTICKED.test(last)) open++;
-  }
-  return open;
-}
-
-/** The info finding for one unit's `requirements/` directory; empty when nothing is open. */
-export function viewpointReviewOpenFindings(requirementsDir: string): string[] {
-  let text: string;
-  try {
-    text = fs.readFileSync(path.join(requirementsDir, VIEWPOINT_REVIEW_FILE), 'utf-8');
-  } catch {
-    return [];
-  }
-  const open = openReviewPoints(text);
-  if (open === 0) return [];
-  return [`VIEWPOINT-REVIEW-OPEN (info): ${open} open point(s) in requirements/${VIEWPOINT_REVIEW_FILE} — the checklist review of test-viewpoint.md is waiting for QA. QA: fix the viewpoint where you agree, then tick the row \`[x]\` or delete it. Not scored; an agent does not tick rows for the QA.`];
-}

package/src/harness/viewpoint-scan-refs.ts

@@ -1,154 +0,0 @@
-/**
- * Building blocks of `sungen viewpoint scan` (viewpoint-scan.ts): the markdown table walker, the
- * item-reference reader and the item universe. Pure, read-only, format-tolerant by design — a
- * QA-written file is never judged here, only its ids and item numbers are read.
- */
-import * as fs from 'fs';
-import * as path from 'path';
-import { readTextFile, stripHtmlComments } from './read-text';
-import { parseUiInventory } from './ui-inventory';
-import { parseSpecClauses } from './spec-coverage';
-
-export interface TableRow {
-  /** 1-based line in the (comment-stripped, newline-preserving) text. */
-  line: number;
-  raw: string;
-  /** Cells with `**`, backticks and edge spaces removed. */
-  cells: string[];
-  /** The table's header cells; null for a table with no separator line. */
-  header: string[] | null;
-  /** Text of every heading enclosing the row (outermost first) — where a `# MANUAL/ENV` mark may sit. */
-  section: string;
-}
-
-const SEPARATOR = /^\s*\|?\s*:?-{2,}:?\s*(\|\s*:?-{2,}:?\s*)*\|?\s*$/;
-const splitCells = (line: string): string[] => line.trim().replace(/^\|/, '').replace(/\|$/, '')
-  .split(/(?<!\\)\|/).map((c) => c.replace(/\*\*|`/g, '').trim());
-
-/** Every body row of every markdown table, with its header and enclosing headings. */
-export function tableRows(text: string): TableRow[] {
-  const lines = text.split('\n');
-  const out: TableRow[] = [];
-  const headings: string[] = [];   // index = level − 1
-  let header: string[] | null = null;
-  for (let i = 0; i < lines.length; i++) {
-    const raw = lines[i];
-    const h = raw.match(/^(#{1,6})\s+(.*)$/);
-    if (h) { headings.length = h[1].length - 1; headings[h[1].length - 1] = h[2]; header = null; continue; }
-    if (!raw.includes('|')) { header = null; continue; }
-    if (SEPARATOR.test(raw)) continue;
-    if (i + 1 < lines.length && SEPARATOR.test(lines[i + 1])) { header = splitCells(raw); continue; }
-    out.push({ line: i + 1, raw, cells: splitCells(raw), header, section: headings.join(' ') });
-  }
-  return out;
-}
-
-const NUM = String.raw`(?<!\d)\d{1,3}(?:\.\d{1,3})*(?![\d])`;
-const LIST = new RegExp(`(?:\\bitems?|項目|(?<![\\w-])UI[\\s-]*#?|#)\\s*(${NUM}(?:\\s*(?:[-–—~〜～,;&/]|(?<![\\p{L}\\p{N}])(?:và|and)(?![\\p{L}\\p{N}]))\\s*${NUM})*)`, 'giu');
-
-/** `009` → `9`, `4.03` → `4.3`; null when the text is not a plain (dotted) number. */
-export function normRef(s: string): string | null {
-  const t = s.replace(/[[\]§#]/g, '').trim();
-  return /^\d+(\.\d+)*$/.test(t) ? t.split('.').map((p) => String(Number(p))).join('.') : null;
-}
-
-/**
- * The refs strictly between `a` and `b` (normalised): `4`–`6` → `5`, `5.4`–`5.8` → `5.5 … 5.7`.
- * Only the last level steps, so both ends need the same prefix; `5.4–6.2` or `3–5.2` cannot be
- * enumerated without the item list and give nothing.
- */
-function between(a: string, b: string): string[] {
-  const pa = a.split('.');
-  const pb = b.split('.');
-  if (pa.length !== pb.length || pa.slice(0, -1).join('.') !== pb.slice(0, -1).join('.')) return [];
-  const lo = Number(pa[pa.length - 1]);
-  const hi = Number(pb[pb.length - 1]);
-  if (!(hi > lo && hi - lo < 200)) return [];
-  const prefix = pa.slice(0, -1).map((p) => `${p}.`).join('');
-  return Array.from({ length: hi - lo - 1 }, (_, k) => `${prefix}${lo + 1 + k}`);
-}
-
-/**
- * Item numbers a cell cites: `item 9`, `item 2, 3, 7`, `item 4–6`, `item 5.4–5.8`, `項目 4.3〜4.6`, `UI #9`,
- * `UI-009`, `[9]`, `§4.3.2`. Prose (`Auth guard`), message codes (`M02`) and question ids
- * (`Q-01`) are not refs.
- */
-export function extractRefs(text: string): string[] {
-  const refs = new Set<string>();
-  const add = (s: string): void => { const r = normRef(s); if (r) refs.add(r); };
-  for (const m of text.matchAll(LIST)) {
-    let prev: string | null = null;
-    let range = false;
-    for (const tok of m[1].matchAll(new RegExp(`(${NUM})|([-–—~〜～])`, 'g'))) {
-      if (tok[2]) { range = prev !== null; continue; }
-      const n = normRef(tok[1]);
-      if (!n) continue;
-      if (range && prev) for (const r of between(prev, n)) refs.add(r);
-      refs.add(n); prev = n; range = false;
-    }
-  }
-  for (const m of text.matchAll(/\[(\d{1,3})\]/g)) add(m[1]);
-  for (const m of text.matchAll(/§\s*(\d{1,3}(?:\.\d{1,3})*)/g)) add(m[1]);
-  return [...refs];
-}
-
-export type ItemSource = 'spec-ids' | 'spec-table' | 'spec-ids+spec-table' | 'ui-inventory' | 'none';
-export interface UniverseItem { ref: string; name: string }
-/** Every list a citation could point at, each in its own notation. */
-export interface ItemUniverses { specIds: UniverseItem[]; specTable: UniverseItem[]; inventory: UniverseItem[] }
-
-const NO_COL = /^(no\.?|#|stt|№|番号)$/i;
-const TYPE_COL = /種別|\b(type|kind)\b|loại/i;
-const NAME_COL = /item|name|tên|項目|名|field|label|trường|thành phần|component/i;
-
-/**
- * The unit's lists, by where each id is DECLARED: requirement ids spec.md declares (`**FR-BC-03**:`),
- * the spec's item table (`No` + name + type columns), and ui-inventory ids (`UI-NNN`). An id's shape
- * says nothing about which list it belongs to — `item 9` on a mockup and `UI-009` in the inventory
- * number different things.
- */
-export function itemUniverses(reqDir: string): ItemUniverses {
-  const read = (f: string): string | null => {
-    const p = path.join(reqDir, f);
-    return fs.existsSync(p) ? stripHtmlComments(readTextFile(p)) : null;
-  };
-  const out: ItemUniverses = { specIds: [], specTable: [], inventory: [] };
-  const spec = read('spec.md');
-  if (spec) {
-    out.specIds = parseSpecClauses(path.join(reqDir, 'spec.md')).frs.map((f) => ({ ref: f.id, name: f.text.slice(0, 80) }));
-    for (const row of tableRows(spec)) {
-      const h = row.header;
-      const no = h?.findIndex((c) => NO_COL.test(c)) ?? -1;
-      const type = h?.findIndex((c) => TYPE_COL.test(c)) ?? -1;
-      const name = h?.findIndex((c, i) => i !== no && i !== type && NAME_COL.test(c)) ?? -1;
-      if (no < 0 || type < 0 || name < 0) continue;
-      const ref = normRef(row.cells[no] ?? '');
-      if (ref) out.specTable.push({ ref, name: row.cells[name] ?? '' });
-    }
-  }
-  const inv = read('ui-inventory.md');
-  if (inv) {
-    out.inventory = parseUiInventory(inv).items.flatMap((i) => {
-      const ref = normRef(i.id.replace(/^UI-/i, ''));
-      return ref ? [{ ref, name: i.item }] : [];
-    });
-  }
-  return out;
-}
-
-/**
- * The list(s) the viewpoint actually cites — a list nobody cites says nothing about gaps. The spec's
- * ids and its item table, each when cited (both when both are: the notations cannot collide, and
- * dropping one would under-report); the table also when nothing is cited at all (then every item is
- * uncited, which is true); the inventory only when the viewpoint writes `UI-NNN`; else none.
- */
-export function citedUniverse(u: ItemUniverses, cited: ReadonlySet<string>, writesInventoryIds: boolean): { source: ItemSource; items: UniverseItem[] } {
-  const hits = (items: UniverseItem[]): boolean => items.some((i) => cited.has(i.ref));
-  const ids = hits(u.specIds);
-  const table = u.specTable.length > 0 && (cited.size === 0 || hits(u.specTable));
-  if (ids && table) return { source: 'spec-ids+spec-table', items: [...u.specIds, ...u.specTable] };
-  if (ids) return { source: 'spec-ids', items: u.specIds };
-  if (table) return { source: 'spec-table', items: u.specTable };
-  if (writesInventoryIds && hits(u.inventory)) return { source: 'ui-inventory', items: u.inventory };
-  return { source: 'none', items: [] };
-}

package/src/harness/viewpoint-scan.ts

@@ -1,111 +0,0 @@
-/**
- * `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.
- */
-import * as fs from 'fs';
-import * as path from 'path';
-import { readViewpointText, readTextFile } from './read-text';
-import { featureFilesFor } from './unit-paths';
-import { tableRows, extractRefs, itemUniverses, citedUniverse, ItemSource } from './viewpoint-scan-refs';
-import { viewpointMode, 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;
-}
-
-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;
-export 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 function scanViewpointText(text: string, specIds: ReadonlySet<string> = new Set()): VpRow[] {
-  const rows: VpRow[] = [];
-  for (const r of 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: [...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) && extractRefs(c).length === 0),
-    });
-  }
-  return rows;
-}
-
-/** Scan one unit directory (`qa/screens/<n>`, `qa/flows/<n>`, …); `unit` names its feature files. */
-export function scanViewpointUnit(unitDir: string, unit: string): ViewpointScan {
-  const reqDir = path.join(unitDir, 'requirements');
-  const vpPath = path.join(reqDir, 'test-viewpoint.md');
-  const vpText = readViewpointText(vpPath);
-  const universes = itemUniverses(reqDir);
-  const vpRows = scanViewpointText(vpText, new Set(universes.specIds.map((i) => i.ref)));
-  const { mode, reason } = viewpointMode(fs.existsSync(vpPath) ? readTextFile(vpPath) : null, vpRows);
-  const cited = new Set(vpRows.flatMap((r) => r.refs));
-  const { source, items } = citedUniverse(universes, cited, /(?<![\w-])UI-\d{1,3}\b/i.test(vpText));
-  const features = 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) => 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) ? NO_VP_IDS_NOTE : 'requirements/test-viewpoint.md is missing',
-  };
-}

package/src/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/src/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> | -->