@sun-asterisk/sungen 2.6.8 → 2.6.11

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sun-asterisk/sungen",
-  "version": "2.6.8",
+  "version": "2.6.11",
   "description": "Deterministic E2E Test Compiler - Gherkin + Selectors → Playwright tests",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

package/src/cli/commands/dashboard.ts

@@ -3,7 +3,8 @@
  *
  * Builds qa/dashboard/index.html — a single-file, share-ready test report —
  * from existing Gherkin features, compiled .spec.ts files, and Playwright
- * results. Snapshots are persisted under qa/dashboard/history/ (max 20).
+ * results. Snapshots are persisted under qa/dashboard/history/ (never deleted;
+ * --max-history only controls how many are included in dashboard stats).
  */
 
 import { Command } from 'commander';
@@ -68,7 +69,7 @@ export function registerDashboardCommand(program: Command): void {
     .description('Build a single-file HTML dashboard summarising all test cases & results')
     .argument('[names...]', 'Specific screen or flow names. Omit to include all.')
     .option('--no-history', 'Do not persist this run under qa/dashboard/history/')
-    .option('--max-history <n>', `Cap retained history files (default: ${DEFAULT_MAX_HISTORY})`)
+    .option('--max-history <n>', `Limit snapshots included in dashboard stats (default: ${DEFAULT_MAX_HISTORY}). Older files are kept on disk.`)
     .option('--open', 'Open the rendered HTML in the default browser when done')
     .action(async (names: string[], options: DashboardOptions) => {
       try {
@@ -103,18 +104,17 @@ export function registerDashboardCommand(program: Command): void {
           `${COLOR.gray}(${snapshot.screens.length} screens, ${snapshot.summary.total} TCs)${COLOR.reset}`
         );
 
-        // 3. Persist history (unless disabled)
+        // 3. Persist history (unless disabled). Files are never deleted —
+        //    `max` only controls how many feed into the dashboard payload.
+        const max = options.maxHistory ? Math.max(1, parseInt(options.maxHistory, 10) || DEFAULT_MAX_HISTORY) : DEFAULT_MAX_HISTORY;
         if (!options.noHistory) {
-          const max = options.maxHistory ? Math.max(1, parseInt(options.maxHistory, 10) || DEFAULT_MAX_HISTORY) : DEFAULT_MAX_HISTORY;
           const result = writeSnapshotToHistory(cwd, snapshot, max);
           log(`  ${COLOR.green}✓${COLOR.reset} history  ${path.relative(cwd, result.written)}`);
-          if (result.pruned.length > 0) {
-            log(`  ${COLOR.gray}  pruned ${result.pruned.length} older snapshot(s)${COLOR.reset}`);
-          }
+          log(`  ${COLOR.gray}  ${result.total} on disk · ${result.included.length} within limit (${max})${result.archived.length > 0 ? ` · ${result.archived.length} archived (kept)` : ''}${COLOR.reset}`);
         }
 
-        // 4. Render HTML
-        const payload = buildPayload(cwd, snapshot);
+        // 4. Render HTML — limit how many history files feed the payload.
+        const payload = buildPayload(cwd, snapshot, max);
         const rendered = renderDashboardHtml(cwd, payload);
         log(`  ${COLOR.green}✓${COLOR.reset} html     ${path.relative(cwd, rendered.outputPath)}  ${COLOR.gray}(${(rendered.bytes / 1024 / 1024).toFixed(2)} MB)${COLOR.reset}`);
 

package/src/cli/commands/delivery.ts

@@ -7,6 +7,7 @@ import { Command } from 'commander';
 import * as path from 'path';
 import * as fs from 'fs';
 import { execSync } from 'child_process';
+import { parse as parseYaml } from 'yaml';
 import { parseFeatureMetadata } from '../../exporters/feature-parser';
 import { parseSpecFile } from '../../exporters/spec-parser';
 import { loadTestData } from '../../exporters/test-data-resolver';
@@ -102,6 +103,26 @@ function resolveTestDataPathForTarget(cwd: string, target: DeliveryTarget): stri
   return path.join(qaDir(cwd, target), 'test-data', `${target.name}.yaml`);
 }
 
+/**
+ * Flows record their entry-point URL inside selectors/<name>.yaml under the
+ * page-selector block (entry with `type: 'page'`). Return its `value`, or
+ * undefined when the file is missing or has no page selector.
+ */
+function readFlowPagePath(selectorsFile: string): string | undefined {
+  if (!fs.existsSync(selectorsFile)) return undefined;
+  try {
+    const parsed = parseYaml(fs.readFileSync(selectorsFile, 'utf-8'));
+    if (!parsed || typeof parsed !== 'object') return undefined;
+    for (const v of Object.values(parsed as Record<string, unknown>)) {
+      if (v && typeof v === 'object' && (v as { type?: string }).type === 'page') {
+        const value = (v as { value?: unknown }).value;
+        if (typeof value === 'string' && value.length > 0) return value;
+      }
+    }
+  } catch { /* malformed YAML — ignore */ }
+  return undefined;
+}
+
 function runPreflight(cwd: string, target: DeliveryTarget): PreflightCheck {
   const base = qaDir(cwd, target);
   const genBase = generatedDir(cwd, target);
@@ -240,9 +261,18 @@ async function exportTarget(
     const results = resultsFile ? loadPlaywrightReport(resultsFile) : null;
 
     const merged = mergeFeatureAndSpec(feature, spec);
+    // Screens get their URL path from the .feature file directly. Flows record
+    // it in selectors/<name>.yaml under the page-selector block — fall back to
+    // that so the Environment column shows a full URL for flows too.
+    let featurePath = feature.featurePath;
+    if (!featurePath && target.isFlow) {
+      const selectorsFile = path.join(base, 'selectors', `${target.name}.yaml`);
+      featurePath = readFlowPagePath(selectorsFile);
+    }
     const rows = buildTestCaseRows({
       screen: label,
       featureName: feature.featureName,
+      featurePath,
       merged,
       testData,
       results,

package/src/cli/index.ts

@@ -21,7 +21,7 @@ async function main() {
   program
     .name('sungen')
     .description('Deterministic E2E Test Compiler — Gherkin + Selectors → Playwright')
-    .version('2.6.8');
+    .version('2.6.11');
 
   // Global options
   program

package/src/dashboard/history-store.ts

@@ -1,7 +1,8 @@
 /**
- * Persist dashboard snapshots under qa/dashboard/history/ and prune to a fixed
- * max count (oldest by mtime are deleted). Used by `sungen dashboard` to
- * accumulate runs for Trends / Compare views.
+ * Persist dashboard snapshots under qa/dashboard/history/. Files are never
+ * deleted — `max` only controls how many are *included* in dashboard stats /
+ * payload. Older files beyond `max` are retained on disk as `archived` so
+ * trends / compare data is recoverable.
  */
 
 import * as fs from 'fs';
@@ -12,8 +13,9 @@ export const DEFAULT_MAX_HISTORY = 20;
 
 export interface HistoryWriteResult {
   written: string;                     // absolute path written
-  pruned: string[];                    // absolute paths removed
-  retained: string[];                  // absolute paths still on disk (newest → oldest)
+  total: number;                       // total files on disk (after write)
+  included: string[];                  // newest → oldest, capped at max
+  archived: string[];                  // files beyond max, still on disk
 }
 
 export function historyDir(cwd: string): string {
@@ -21,8 +23,8 @@ export function historyDir(cwd: string): string {
 }
 
 /**
- * Write `snapshot` as <runId>.json and prune older files until at most `max`
- * files remain. Files are sorted by mtime so manual edits (rare) still work.
+ * Write `snapshot` as <runId>.json. Never deletes existing files; `max` only
+ * partitions the on-disk list into `included` (newest N) vs `archived`.
  */
 export function writeSnapshotToHistory(
   cwd: string,
@@ -36,10 +38,10 @@ export function writeSnapshotToHistory(
   const target = path.join(dir, filename);
   fs.writeFileSync(target, JSON.stringify(snapshot, null, 2), 'utf-8');
 
-  const pruned = pruneHistory(dir, max);
-
-  const retained = listHistoryFiles(dir);
-  return { written: target, pruned, retained };
+  const all = listHistoryFiles(dir);          // newest first
+  const included = all.slice(0, max);
+  const archived = all.slice(max);
+  return { written: target, total: all.length, included, archived };
 }
 
 /**
@@ -54,11 +56,17 @@ export function listHistoryFiles(dir: string): string[] {
 }
 
 /**
- * Read every history JSON, oldest → newest. Skips files that fail to parse.
+ * Read history JSONs, oldest → newest. When `limit` is provided, only the
+ * newest `limit` files are loaded (older ones stay on disk but are skipped).
+ * Skips files that fail to parse.
  */
-export function readHistory(cwd: string): DashboardSnapshot[] {
+export function readHistory(cwd: string, limit?: number): DashboardSnapshot[] {
   const dir = historyDir(cwd);
-  const files = listHistoryFiles(dir).reverse(); // oldest → newest
+  let files = listHistoryFiles(dir);          // newest first
+  if (typeof limit === 'number' && limit >= 0) {
+    files = files.slice(0, limit);
+  }
+  files = files.reverse();                    // oldest → newest
   const out: DashboardSnapshot[] = [];
   for (const f of files) {
     try {
@@ -70,17 +78,3 @@ export function readHistory(cwd: string): DashboardSnapshot[] {
   }
   return out;
 }
-
-/**
- * Delete oldest files in `dir` until at most `max` remain.
- * Returns the absolute paths removed.
- */
-function pruneHistory(dir: string, max: number): string[] {
-  const files = listHistoryFiles(dir);              // newest first
-  if (files.length <= max) return [];
-  const toRemove = files.slice(max);
-  for (const f of toRemove) {
-    try { fs.unlinkSync(f); } catch { /* ignore */ }
-  }
-  return toRemove;
-}

package/src/dashboard/html-renderer.ts

@@ -42,8 +42,12 @@ export function resolveTemplatePath(): string {
   );
 }
 
-export function buildPayload(cwd: string, current: DashboardSnapshot): DashboardPayload {
-  const all = readHistory(cwd);
+export function buildPayload(
+  cwd: string,
+  current: DashboardSnapshot,
+  historyLimit?: number,
+): DashboardPayload {
+  const all = readHistory(cwd, historyLimit);
   // Exclude `current` from history if it's been written there already.
   const history = all.filter((s) => s.runId !== current.runId);
   return { current, history };

package/src/dashboard/snapshot-builder.ts

@@ -14,6 +14,7 @@
 import * as fs from 'fs';
 import * as path from 'path';
 import { execSync } from 'child_process';
+import { parse as parseYaml } from 'yaml';
 import { parseFeatureMetadata } from '../exporters/feature-parser';
 import { parseSpecFile } from '../exporters/spec-parser';
 import { loadTestData } from '../exporters/test-data-resolver';
@@ -99,11 +100,20 @@ function buildOneScreen(
   const label = target.isFlow ? `flow/${target.name}` : target.name;
   const specLink = fs.existsSync(specMdFile) ? path.relative(cwd, specMdFile) : undefined;
 
+  // Screens declare their URL inline in the .feature ("Path: /awards"). Flows
+  // don't have one there, so fall back to the first `type: 'page'` entry in
+  // the flow's selectors YAML — that's where the entry point is recorded.
+  let featurePath = feature.featurePath;
+  if (!featurePath && target.isFlow) {
+    const selectorsFile = path.join(base, 'selectors', `${target.name}.yaml`);
+    featurePath = readPagePathFromSelectors(selectorsFile);
+  }
+
   return buildScreenSnapshot({
     screen: label,
     isFlow: target.isFlow,
     featureName: feature.featureName,
-    featurePath: feature.featurePath,
+    featurePath,
     specLink,
     merged,
     testData,
@@ -125,7 +135,7 @@ function aggregateSummary(screens: ScreenSnapshot[]): AggregateSummary {
     na: 0,
     notCompiled: 0,
     passRate: 0,
-    byPriority: { Critical: 0, High: 0, Normal: 0, Low: 0 },
+    byPriority: { High: 0, Normal: 0, Low: 0 },
     byCategory: { Accessing: 0, GUI: 0, Function: 0 },
     byType: { Auto: 0, Manual: 0, 'Not compiled': 0 },
   };
@@ -269,5 +279,29 @@ function collectEnvironment(cwd: string, env: EnvironmentInfo) {
   };
 }
 
+/**
+ * Read the entry-point path for a flow from its selectors YAML. We pick the
+ * first selector whose `type` is `page` — that's the convention sungen uses
+ * for the page-selector block in flow YAML files. Returns undefined when the
+ * file is missing or has no page entry.
+ */
+function readPagePathFromSelectors(selectorsFile: string): string | undefined {
+  if (!fs.existsSync(selectorsFile)) return undefined;
+  try {
+    const raw = fs.readFileSync(selectorsFile, 'utf-8');
+    const parsed = parseYaml(raw);
+    if (!parsed || typeof parsed !== 'object') return undefined;
+    for (const value of Object.values(parsed as Record<string, unknown>)) {
+      if (value && typeof value === 'object' && (value as { type?: string }).type === 'page') {
+        const v = (value as { value?: unknown }).value;
+        if (typeof v === 'string' && v.length > 0) return v;
+      }
+    }
+  } catch {
+    // Malformed YAML — fall through to undefined.
+  }
+  return undefined;
+}
+
 // Helper for ScenarioSnapshot tree filters in UI (re-exported for callers)
 export type { ScenarioSnapshot };