@sun-asterisk/sungen 2.6.10 → 2.6.11

package/src/exporters/xlsx-exporter.ts

@@ -12,6 +12,7 @@
 import * as fs from 'fs';
 import * as path from 'path';
 import ExcelJS from 'exceljs';
+import JSZip from 'jszip';
 import { ScreenSummary, TestCaseRow } from './types';
 import { getPackageVersion } from './package-info';
 import { SUN_LOGO_PNG_BASE64 } from './sun-logo';
@@ -54,7 +55,7 @@ export function renderXlsx(
 
   // -- Column widths matching template_report.xlsx (Sample sheet) --
   ws.columns = [
-    { width: 6.38 },  // A — TC ID
+    { width: 20 },    // A — TC ID (wider to fit long flow IDs like FLOW-KUDO-…)
     { width: 12.5 },  // B — Screen/Function
     { width: 15.38 }, // C — Big item
     { width: 16.25 }, // D — Medium item
@@ -92,28 +93,24 @@ export function renderXlsx(
   ws.mergeCells('A4:F4');
   ws.mergeCells('G4:H4');
 
-  // A1 (logo band, merged A1:C3) — give it bordering + embed the Sun* logo.
+  // A1 (logo band, merged A1:C3) — border + embedded Sun* logo (base64 inline).
   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).
+    // Centre a fixed-size 90×51 logo inside A1:C3 using absolute EMU offsets.
+    // Col widths (A=20, B=12.5, C=15.38) → pixels via Excel's `w*7+5` rule:
+    //   A=145, B=92.5, C=112.66  ⇒  A1:C3 ≈ 350 px wide.
+    // Left padding for a 90-px image = (350−90)/2 ≈ 130 px = 1,237,500 EMU.
+    // 3 default rows × 20 px = 60 px; top padding ≈ 4.5 px ≈ 42,862 EMU.
     ws.addImage(imageId, {
-      tl: { nativeCol: 0, nativeColOff: 714375, nativeRow: 0, nativeRowOff: 47625 } as unknown as ExcelJS.Anchor,
-      ext: { width: 90, height: 50 },
+      tl: { nativeCol: 0, nativeColOff: 1237500, nativeRow: 0, nativeRowOff: 42862 } as unknown as ExcelJS.Anchor,
+      ext: { width: 90, height: 51 },
       editAs: 'oneCell',
     } as unknown as ExcelJS.ImageRange);
   } catch { /* logo is decorative — never block export */ }
@@ -161,8 +158,7 @@ export function renderXlsx(
   g4.alignment = { horizontal: 'right', vertical: 'middle', wrapText: true };
   g4.border = allBordersBlack;
 
-  // -- Row 5: spacer (height 12) --
-  ws.getRow(5).height = 12;
+  // -- Row 5: spacer (height auto-fit). --
 
   // -- Row 6: Summary headers (cols C..H), lavender fill --
   const sumLabels: Record<string, string> = {
@@ -176,7 +172,7 @@ export function renderXlsx(
     c.alignment = { horizontal: 'center', vertical: 'top' };
     c.border = allBordersBlack;
   }
-  ws.getRow(6).height = 12;
+  // No explicit height → Excel auto-fits this row's content.
 
   // -- 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
@@ -200,7 +196,7 @@ export function renderXlsx(
     c.border = allBordersBlack;
     if (col === 'H') c.numFmt = '#,##0';
   }
-  ws.getRow(7).height = 12;
+  // No explicit height → Excel auto-fits.
 
   // C8 has no value in the template but still needs a border so the band
   // visually closes underneath "Total TCs".
@@ -227,7 +223,7 @@ export function renderXlsx(
     c.border = allBordersBlack;
     c.numFmt = '0%';
   }
-  ws.getRow(8).height = 12;
+  // No explicit height → Excel auto-fits.
 
   // After rows 1-8 are populated, append a blank row 9 spacer.
   while (ws.rowCount < 9) ws.addRow([]);
@@ -353,6 +349,17 @@ export function renderXlsx(
     to: { row: ws.rowCount, column: COL_COUNT },
   };
 
+  // Sheet protection lets the picLocks on the embedded logo take effect
+  // (Excel honours image locks only when the sheet is protected). Empty
+  // password — QA can run Review → Unprotect Sheet if they need to edit.
+  // `objects: false` here is ExcelJS's inverted semantic that writes
+  // `objects="1"` in XML (= objects ARE locked).
+  (ws as unknown as { sheetProtection: object }).sheetProtection = {
+    sheet: true,
+    objects: false,
+    scenarios: false,
+  };
+
   return wb;
 }
 
@@ -368,9 +375,41 @@ export async function writeXlsx(
   const outDir = path.join(cwd, 'qa', 'deliverables');
   if (!fs.existsSync(outDir)) fs.mkdirSync(outDir, { recursive: true });
   const outPath = path.join(outDir, `${screen}-testcases.xlsx`);
-  await wb.xlsx.writeFile(outPath);
+  const buffer = await wb.xlsx.writeBuffer();
+  const locked = await lockEmbeddedImages(Buffer.from(buffer as ArrayBuffer));
+  fs.writeFileSync(outPath, locked);
   return outPath;
 }
 
+/**
+ * Post-process the workbook buffer: extend `<a:picLocks>` on every drawing
+ * with `noMove`, `noResize`, `noSelect`. Combined with the sheet protection
+ * above, the embedded logo can't be dragged, resized, or selected via the UI.
+ */
+export async function lockEmbeddedImages(buffer: Buffer): Promise<Buffer> {
+  const zip = await JSZip.loadAsync(buffer);
+  const drawingFiles = Object.keys(zip.files).filter(
+    (name) => /^xl\/drawings\/drawing\d+\.xml$/.test(name)
+  );
+  for (const name of drawingFiles) {
+    const file = zip.file(name);
+    if (!file) continue;
+    const xml = await file.async('string');
+    const patched = xml.replace(
+      /<a:picLocks([^/]*)\/>/g,
+      (_match, attrs: string) => {
+        const has = (k: string) => new RegExp(`\\b${k}=`).test(attrs);
+        const additions = ['noMove', 'noResize', 'noSelect']
+          .filter((k) => !has(k))
+          .map((k) => ` ${k}="1"`)
+          .join('');
+        return `<a:picLocks${attrs}${additions}/>`;
+      }
+    );
+    zip.file(name, patched);
+  }
+  return zip.generateAsync({ type: 'nodebuffer', compression: 'DEFLATE' });
+}
+
 void applyBorder;
 void ({} as AnyRow);

package/src/orchestrator/project-initializer.ts

@@ -450,8 +450,9 @@ export class ProjectInitializer {
     console.log(`📦 Installing ${missingDeps.join(', ')}...\n`);
     execSync(`npm install -D ${missingDeps.join(' ')}`, execOpts);
 
-    console.log('\n🎭 Installing Playwright browsers...\n');
-    execSync('npx playwright install', execOpts);
+    console.log('\n🎭 Installing Chromium (default browser)...\n');
+    execSync('npx playwright install --with-deps chromium', execOpts);
+    console.log('\n💡 To install other browsers: npm run install:browsers -- firefox webkit\n');
   }
 
   /**
@@ -477,7 +478,7 @@ export class ProjectInitializer {
       'test:ui': 'playwright test specs/generated/ --ui',
       'report': 'playwright show-report',
       'generate': 'sungen generate --all',
-      'install:browsers': 'npx playwright install chromium',
+      'install:browsers': 'npx playwright install',
     };
 
     let added = 0;