@sun-asterisk/sungen 2.6.7 → 2.6.10

package/src/exporters/test-data-resolver.ts

@@ -8,7 +8,11 @@ import { parse as parseYaml } from 'yaml';
 
 /**
  * Load test-data.yaml into a flat key→value map.
- * Returns empty map if file missing.
+ * Nested objects are flattened with dot notation so flow files like
+ *   kudos:
+ *     recipient: "An"
+ * become { "kudos.recipient": "An" }, matching {{kudos.recipient}} refs.
+ * Arrays are JSON-stringified. Returns empty map if file missing.
  */
 export function loadTestData(testDataFilePath: string): Record<string, string> {
   if (!fs.existsSync(testDataFilePath)) return {};
@@ -16,38 +20,58 @@ export function loadTestData(testDataFilePath: string): Record<string, string> {
   const parsed = parseYaml(content);
   if (!parsed || typeof parsed !== 'object') return {};
 
-  // Flatten shallow object to string values
   const result: Record<string, string> = {};
-  for (const [key, value] of Object.entries(parsed as Record<string, unknown>)) {
+  flatten(parsed as Record<string, unknown>, '', result);
+  return result;
+}
+
+function flatten(obj: Record<string, unknown>, prefix: string, out: Record<string, string>): void {
+  for (const [key, value] of Object.entries(obj)) {
     if (value === null || value === undefined) continue;
-    if (typeof value === 'object') continue; // skip nested structures
-    result[key] = String(value);
+    const path = prefix ? `${prefix}.${key}` : key;
+    if (Array.isArray(value)) {
+      out[path] = JSON.stringify(value);
+    } else if (typeof value === 'object') {
+      flatten(value as Record<string, unknown>, path, out);
+    } else {
+      out[path] = String(value);
+    }
   }
-  return result;
 }
 
 /**
- * Format a list of variable references into a "key: value; key2: value2" string
- * suitable for the CSV Test Data column.
+ * Format a list of variable references into a "key: value" string.
  *
- * Long values (>80 chars) are truncated with "…".
+ * - `separator` controls how pairs are joined. Default `'; '` keeps CSV/XLSX
+ *   cells on one line. Pass `'\n'` for surfaces that render multi-line text
+ *   (e.g. the dashboard modal uses whitespace-pre-wrap).
+ * - Test data is never truncated by default — values represent real test
+ *   inputs and clients need to see them in full. Pass a finite `maxLen` only
+ *   if the downstream surface requires a cap (e.g. terminal preview).
  */
 export function formatTestData(
   referencedVars: string[],
-  testData: Record<string, string>
+  testData: Record<string, string>,
+  maxLen: number = Infinity,
+  separator: string = '; ',
 ): string {
   if (referencedVars.length === 0) return '';
+  // When joining with newlines (dashboard multi-line view), prefix each line
+  // with a bullet point so the list scans easily. CSV-style "; " join skips
+  // the bullet — single-cell text shouldn't have list markers.
+  const useBullet = separator.includes('\n');
   const pairs: string[] = [];
   for (const key of referencedVars) {
     const value = testData[key];
     if (value === undefined) continue;
-    pairs.push(`${key}: ${truncate(value, 80)}`);
+    const line = `${key}: ${truncate(value, maxLen)}`;
+    pairs.push(useBullet ? `• ${line}` : line);
   }
-  return pairs.join('; ');
+  return pairs.join(separator);
 }
 
 function truncate(s: string, max: number): string {
-  if (s.length <= max) return s;
+  if (!isFinite(max) || s.length <= max) return s;
   return s.substring(0, max - 1) + '…';
 }
 

package/src/exporters/types.ts

@@ -15,7 +15,7 @@ export interface TestCaseRow {
   testData: string;          // semicolon-separated key: value
   steps: string;             // numbered steps
   expectedResults: string;   // numbered expected results
-  priority: string;          // Critical | High | Normal | Low
+  priority: string;          // High | Normal | Low
   testcaseType: string;      // Auto | Manual | Not compiled
   testResult: string;        // Passed | Failed | Pending | N/A
   executedDate: string;      // dd/mm/yyyy
@@ -59,9 +59,20 @@ export interface FeatureMetadata {
   featureName: string;                    // e.g., "Create Kudo Modal"
   featurePath?: string;                   // e.g., "/kudos"
   featureTags: string[];                  // Feature-level tags
+  /** Background block classified by bucket. Shared across all runnable
+   *  scenarios — the merger is responsible for prepending these.  */
+  backgroundGivenSteps: string[];
+  backgroundWhenSteps:  string[];
+  backgroundThenSteps:  string[];
+  backgroundOrderedSteps: OrderedStep[];
   scenarios: ScenarioMetadata[];
 }
 
+export interface OrderedStep {
+  text: string;
+  bucket: 'given' | 'when' | 'then';
+}
+
 export interface ScenarioMetadata {
   name: string;                           // Full scenario name (with VP prefix)
   tags: string[];                         // Scenario-level tags (merged with feature tags)
@@ -71,6 +82,12 @@ export interface ScenarioMetadata {
   rawGivenSteps: string[];                // Raw Given/And-after-Given step text
   rawWhenSteps: string[];                 // Raw When/And-after-When step text
   rawThenSteps: string[];                 // Raw Then/And-after-Then step text
+  /**
+   * Steps in original chronological order with each step labelled by its
+   * effective bucket (And/But inherits from the prior explicit keyword).
+   * Use this when ordering matters — e.g. mid-scenario `Given` after `When`.
+   */
+  orderedSteps: OrderedStep[];
 }
 
 /**

package/src/exporters/xlsx-exporter.ts

@@ -14,12 +14,13 @@ import * as path from 'path';
 import ExcelJS from 'exceljs';
 import { ScreenSummary, TestCaseRow } from './types';
 import { getPackageVersion } from './package-info';
+import { SUN_LOGO_PNG_BASE64 } from './sun-logo';
 
 const COL_COUNT = 16;
-const HEADER_FILL = 'FF1F4E78'; // deep blue band
-const HEADER_FONT = 'FFFFFFFF';
+const HEADER_FILL = 'FFD9D2E9'; // lavender — matches the summary-header band on row 6
+const HEADER_FONT = 'FF000000'; // black text reads better on the light lavender
 const META_FILL = 'FFDCE6F1'; // soft blue for meta rows
-const CATEGORY_FILL = 'FFFFF2CC'; // soft yellow for group dividers
+const CATEGORY_FILL = 'FFD9D2E9'; // same lavender for group dividers (Accessing / GUI / Function)
 const BORDER_COLOR = 'FFBFBFBF';
 
 type AnyCell = ExcelJS.Cell;
@@ -51,111 +52,185 @@ export function renderXlsx(
     return `${String(d.getDate()).padStart(2, '0')}/${String(d.getMonth() + 1).padStart(2, '0')}/${d.getFullYear()}`;
   })();
 
-  const total = summary.total || 1;
-  const pct = (n: number) => `${Math.round((n / total) * 100)}%`;
-
-  // -- Column widths (matches CSV column order) --
+  // -- Column widths matching template_report.xlsx (Sample sheet) --
   ws.columns = [
-    { width: 18 }, // TC ID
-    { width: 42 }, // Category 1
-    { width: 14 }, // Category 2
-    { width: 26 }, // Category 3
-    { width: 14 }, // Category 4
-    { width: 36 }, // Pre-condition
-    { width: 30 }, // Test Data
-    { width: 46 }, // Steps
-    { width: 52 }, // Expected results
-    { width: 10 }, // Priority
-    { width: 14 }, // Testcase type
-    { width: 14 }, // Test Result
-    { width: 14 }, // Executed Date
-    { width: 16 }, // Test Executor
-    { width: 38 }, // Test Environment
-    { width: 40 }, // Note
+    { width: 6.38 },  // A — TC ID
+    { width: 12.5 },  // B — Screen/Function
+    { width: 15.38 }, // C — Big item
+    { width: 16.25 }, // D — Medium item
+    { width: 17 },    // E — Test Object
+    { width: 25.5 },  // F — Pre-condition
+    { width: 24.88 }, // G — Test Data
+    { width: 24.88 }, // H — Steps
+    { width: 41 },    // I — Expected results
+    { width: 12 },    // J — Priority
+    { width: 13.5 },  // K — Testcase type
+    { width: 15.88 }, // L — Test Result
+    { width: 13.5 },  // M — Executed Date
+    { width: 16.5 },  // N — Test Executor
+    { width: 23 },    // O — Test Environment
+    { width: 29 },    // P — Note
   ];
 
-  // -- Top metadata band (rows 1-4) --
+  // -- Top metadata band (rows 1-4) — 1:1 with template_report.xlsx --
+  const TIMES = 'Times New Roman';
+  const ARIAL_FONT = 'Arial';
+  const BLACK = { argb: 'FF000000' };
+  const WHITE = { argb: 'FFFFFFFF' };
+  const LAVENDER = { argb: 'FFD9D2E9' };
+  const thinBlack = { style: 'thin' as const, color: BLACK };
+  const allBordersBlack = { top: thinBlack, left: thinBlack, bottom: thinBlack, right: thinBlack };
+
+  // Ensure rows 1..8 exist before merging.
+  for (let r = 1; r <= 8; r++) ws.getRow(r);
+
+  ws.mergeCells('A1:C3');
+  ws.mergeCells('D1:F3');
+  ws.mergeCells('G1:H1');
+  ws.mergeCells('G2:H2');
+  ws.mergeCells('G3:H3');
+  ws.mergeCells('A4:F4');
+  ws.mergeCells('G4:H4');
+
+  // A1 (logo band, merged A1:C3) — give it bordering + embed the Sun* logo.
+  const a1 = ws.getCell('A1');
+  a1.alignment = { horizontal: 'center', vertical: 'middle' };
+  a1.border = allBordersBlack;
+
+  // Embed Sun* logo using a oneCellAnchor + explicit pixel `ext`, exactly
+  // like the template. tl is offset into the merged band so the logo lands
+  // centred in A1:C3 instead of pinned to the top-left corner.
+  try {
+    const imageId = wb.addImage({
+      buffer: Buffer.from(SUN_LOGO_PNG_BASE64, 'base64') as unknown as ExcelJS.Buffer,
+      extension: 'png',
+    });
+    // A1:C3 ≈ 240 × 60 px. For a 90×50 logo centred horizontally / vertically:
+    //   left margin = (240 − 90)/2 ≈ 75 px → 9525 EMU/px → 714375 EMU
+    //   top margin  = (60 − 50)/2 ≈ 5 px  → 47625 EMU
+    // ExcelJS's fractional `col`/`row` clamps to a small EMU range, so we
+    // bypass it and write `native*` fields directly (the underlying XML uses
+    // the same field names).
+    ws.addImage(imageId, {
+      tl: { nativeCol: 0, nativeColOff: 714375, nativeRow: 0, nativeRowOff: 47625 } as unknown as ExcelJS.Anchor,
+      ext: { width: 90, height: 50 },
+      editAs: 'oneCell',
+    } as unknown as ExcelJS.ImageRange);
+  } catch { /* logo is decorative — never block export */ }
+
+  // D1 (TESTCASE title)
   const titleLabel = `${summary.screen.toUpperCase()} TESTCASE`;
-  const titleRow = ws.addRow(['', '', '', titleLabel, '', '', 'No: BM-2-901-13']);
-  titleRow.getCell(4).font = { bold: true, size: 16 };
-  titleRow.getCell(4).alignment = { horizontal: 'center', vertical: 'middle' };
-  ws.mergeCells(titleRow.number, 4, titleRow.number, 6);
-  titleRow.getCell(7).font = { bold: true };
-  titleRow.height = 22;
-
-  const versionRow = ws.addRow(['', '', '', '', '', '', `Version: ${getPackageVersion()}`]);
-  versionRow.getCell(7).font = { bold: true };
-  const dateRow = ws.addRow(['', '', '', '', '', '', `Issue Date: ${issueDate}`]);
-  dateRow.getCell(7).font = { bold: true };
-  const companyRow = ws.addRow([
-    'SUN ASTERISK VIETNAM CO., LTD',
-    '',
-    '',
-    '',
-    '',
-    '',
-    'ISO/IEC 27001:2022 & ISO 9001:2015',
-  ]);
-  companyRow.getCell(1).font = { bold: true };
-  companyRow.getCell(7).font = { bold: true };
+  const d1 = ws.getCell('D1');
+  d1.value = titleLabel;
+  d1.font = { bold: true, size: 18, name: TIMES };
+  d1.alignment = { horizontal: 'center', vertical: 'middle', wrapText: true };
+  d1.border = allBordersBlack;
 
-  ws.addRow([]); // spacer
+  // G1 — No: BM-2-901-13
+  const g1 = ws.getCell('G1');
+  g1.value = 'No: BM-2-901-13';
+  g1.font = { size: 12, name: TIMES };
+  g1.alignment = { vertical: 'middle' };
+  g1.border = { top: thinBlack, left: thinBlack, right: thinBlack };
 
-  // -- Summary band --
-  const summaryHeader = ws.addRow([
-    '',
-    '',
-    'Total TCs',
-    'Passed',
-    'Failed',
-    'Pending',
-    'N/A',
-    'Remaining',
-  ]);
-  for (let i = 3; i <= 8; i++) {
-    const c = summaryHeader.getCell(i);
-    c.font = { bold: true, color: { argb: HEADER_FONT } };
-    c.fill = { type: 'pattern', pattern: 'solid', fgColor: { argb: HEADER_FILL } };
-    c.alignment = { horizontal: 'center', vertical: 'middle' };
-    applyBorder(c);
+  // G2 — Version
+  const g2 = ws.getCell('G2');
+  g2.value = `Version: ${getPackageVersion()}`;
+  g2.font = { size: 12, name: TIMES };
+  g2.alignment = { vertical: 'middle' };
+  g2.border = { left: thinBlack, right: thinBlack };
+
+  // G3 — Issue Date
+  const g3 = ws.getCell('G3');
+  g3.value = `Issue Date: ${issueDate}`;
+  g3.font = { size: 12, name: TIMES };
+  g3.alignment = { vertical: 'middle' };
+  g3.border = { left: thinBlack, right: thinBlack, bottom: thinBlack };
+
+  // A4 — SUN ASTERISK VIETNAM CO., LTD
+  const a4 = ws.getCell('A4');
+  a4.value = 'SUN ASTERISK VIETNAM CO., LTD';
+  a4.font = { bold: true, size: 10, name: TIMES };
+  a4.alignment = { vertical: 'middle' };
+  a4.border = allBordersBlack;
+
+  // G4 — ISO/IEC ...
+  const g4 = ws.getCell('G4');
+  g4.value = 'ISO/IEC 27001:2022 & ISO 9001:2015';
+  g4.font = { bold: true, size: 10, name: TIMES };
+  g4.alignment = { horizontal: 'right', vertical: 'middle', wrapText: true };
+  g4.border = allBordersBlack;
+
+  // -- Row 5: spacer (height 12) --
+  ws.getRow(5).height = 12;
+
+  // -- Row 6: Summary headers (cols C..H), lavender fill --
+  const sumLabels: Record<string, string> = {
+    C: 'Total TCs', D: 'Passed', E: 'Failed', F: 'Pending', G: 'N/A', H: 'Remaining',
+  };
+  for (const [col, label] of Object.entries(sumLabels)) {
+    const c = ws.getCell(`${col}6`);
+    c.value = label;
+    c.font = { bold: true, size: 10, color: BLACK, name: ARIAL_FONT };
+    c.fill = { type: 'pattern', pattern: 'solid', fgColor: LAVENDER };
+    c.alignment = { horizontal: 'center', vertical: 'top' };
+    c.border = allBordersBlack;
   }
+  ws.getRow(6).height = 12;
 
-  const summaryCounts = ws.addRow([
-    '',
-    '',
-    summary.total,
-    summary.passed,
-    summary.failed,
-    summary.pending,
-    summary.na,
-    summary.pending + summary.na,
-  ]);
-  for (let i = 3; i <= 8; i++) {
-    const c = summaryCounts.getCell(i);
-    c.font = { bold: true };
-    c.alignment = { horizontal: 'center' };
-    c.fill = { type: 'pattern', pattern: 'solid', fgColor: { argb: META_FILL } };
-    applyBorder(c);
+  // -- Row 7: Counts (cols C..H), live formulas referencing the data area --
+  // Data table starts at row 15 (col header at row 14, divider at row 14? actually
+  // header at row 13 + divider at 14 + first data at 15). The formulas read col L
+  // (Test Result) for status counts, col A (TC ID) for total via COUNTA.
+  const COUNT_RANGE_END = 10000;
+  const r7Values: Record<string, ExcelJS.CellValue> = {
+    C: { formula: `COUNTA(A15:A${COUNT_RANGE_END})`, result: summary.total } as ExcelJS.CellFormulaValue,
+    D: { formula: `COUNTIF(L15:L${COUNT_RANGE_END},"Passed")`, result: summary.passed } as ExcelJS.CellFormulaValue,
+    E: { formula: `COUNTIF(L15:L${COUNT_RANGE_END},"Failed")`, result: summary.failed } as ExcelJS.CellFormulaValue,
+    F: { formula: `COUNTIF(L15:L${COUNT_RANGE_END},"Pending")`, result: summary.pending } as ExcelJS.CellFormulaValue,
+    G: { formula: `COUNTIF(L15:L${COUNT_RANGE_END},"N/A")`, result: summary.na } as ExcelJS.CellFormulaValue,
+    H: { formula: 'C7-D7-G7', result: summary.failed + summary.pending } as ExcelJS.CellFormulaValue,
+  };
+  for (const [col, val] of Object.entries(r7Values)) {
+    const c = ws.getCell(`${col}7`);
+    c.value = val;
+    c.font = { size: 10, color: BLACK, name: ARIAL_FONT };
+    c.fill = { type: 'pattern', pattern: 'solid', fgColor: WHITE };
+    c.alignment = { horizontal: 'center', vertical: 'top' };
+    c.border = allBordersBlack;
+    if (col === 'H') c.numFmt = '#,##0';
   }
+  ws.getRow(7).height = 12;
 
-  const summaryPct = ws.addRow([
-    '',
-    '',
-    '',
-    pct(summary.passed),
-    pct(summary.failed),
-    pct(summary.pending),
-    pct(summary.na),
-    pct(summary.pending + summary.na),
-  ]);
-  for (let i = 4; i <= 8; i++) {
-    const c = summaryPct.getCell(i);
-    c.alignment = { horizontal: 'center' };
-    c.fill = { type: 'pattern', pattern: 'solid', fgColor: { argb: META_FILL } };
-    applyBorder(c);
+  // C8 has no value in the template but still needs a border so the band
+  // visually closes underneath "Total TCs".
+  const c8 = ws.getCell('C8');
+  c8.font = { size: 10, color: BLACK, name: ARIAL_FONT };
+  c8.fill = { type: 'pattern', pattern: 'solid', fgColor: WHITE };
+  c8.alignment = { horizontal: 'center', vertical: 'top' };
+  c8.border = allBordersBlack;
+
+  // -- Row 8: Percentages (cols D..H), formula = count / (Total − N/A) --
+  const r8Values: Record<string, ExcelJS.CellValue> = {
+    D: { formula: 'IFERROR(D7/(C7-G7),0)', result: summary.passed / Math.max(1, summary.total - summary.na) } as ExcelJS.CellFormulaValue,
+    E: { formula: 'IFERROR(E7/(C7-G7),0)', result: summary.failed / Math.max(1, summary.total - summary.na) } as ExcelJS.CellFormulaValue,
+    F: { formula: 'IFERROR(F7/(C7-G7),0)', result: summary.pending / Math.max(1, summary.total - summary.na) } as ExcelJS.CellFormulaValue,
+    G: { formula: 'IFERROR(G7/(C7-G7),0)', result: summary.na / Math.max(1, summary.total - summary.na) } as ExcelJS.CellFormulaValue,
+    H: { formula: 'IFERROR(H7/(C7-G7),0)', result: (summary.failed + summary.pending) / Math.max(1, summary.total - summary.na) } as ExcelJS.CellFormulaValue,
+  };
+  for (const [col, val] of Object.entries(r8Values)) {
+    const c = ws.getCell(`${col}8`);
+    c.value = val;
+    c.font = { size: 10, color: BLACK, name: ARIAL_FONT };
+    c.fill = { type: 'pattern', pattern: 'solid', fgColor: WHITE };
+    c.alignment = { horizontal: 'center', vertical: 'top' };
+    c.border = allBordersBlack;
+    c.numFmt = '0%';
   }
+  ws.getRow(8).height = 12;
 
-  ws.addRow([]); // spacer
+  // After rows 1-8 are populated, append a blank row 9 spacer.
+  while (ws.rowCount < 9) ws.addRow([]);
 
   const specRow = ws.addRow(['Spec/Design link:', specLink]);
   specRow.getCell(1).font = { bold: true };
@@ -169,10 +244,10 @@ export function renderXlsx(
   // -- Column header row (bold, filled, wrapped) --
   const headerRow = ws.addRow([
     'TC ID*',
-    '{Category 1}',
-    '{Category 2}',
-    '{Category 3}',
-    '{Category 4}',
+    'Screen/Function',
+    'Big item',
+    'Medium item',
+    'Test Object',
     'Pre-condition',
     'Test Data',
     'Steps*',
@@ -222,10 +297,10 @@ export function renderXlsx(
     for (const r of groupRows) {
       const row = ws.addRow([
         r.tcId,
-        r.category1,
         '',
         '',
         '',
+        r.category1,
         r.precondition,
         r.testData,
         r.steps,

package/src/orchestrator/templates/ai-instructions/claude-cmd-create-test.md

@@ -51,7 +51,7 @@ Parse **name** from `$ARGUMENTS`. If missing, ask the user.
 
    Summarize what you found in requirements and present to the user.
 
-4. Follow the `sungen-tc-generation` skill for section identification, viewpoint generation, and output format. **For flows**, use the "Flow Test Generation" section in the skill. When requirements exist, use the "Requirements-Driven Generation" strategy.
+4. Follow the `sungen-tc-generation` skill for section identification, viewpoint generation, and output format. **For flows**, use the "Flow Test Generation" section in the skill. When requirements exist, use the "Requirements-Driven Generation" strategy. **For Tier 1**, apply the **Lightweight Guard** — verify required fields, validation rules, business rules, security checks, and key state transitions all have TCs after generation. **For Tier 2+**, **MUST** apply the full **Mapping Contract** — walk every `spec.md` section top-to-bottom and produce the indicated TCs per Table 1; handle `test-viewpoint.md` per Table 2. Do not silently skip sections.
 5. Generate or update `.feature` + `test-data.yaml` following `sungen-gherkin-syntax` and `sungen-tc-generation` skills. **For flows**: use `[Screen:Element]` namespace format, namespace test-data by phase, add `@flow` tag.
 6. Show summary, then use `AskUserQuestion` to offer next steps based on which tier was just generated: