@tekyzinc/gsd-t 4.0.23 → 4.0.25

package/CHANGELOG.md

@@ -2,6 +2,26 @@
 
 All notable changes to GSD-T are documented here. Updated with each release.
 
+## [4.0.25] - 2026-06-03 (M77 HTML Report Reads Deep-Scan Table Format - patch)
+
+### Fixed - scan report showed 0 critical/0 high on a 322-finding register
+
+The HTML report renderer (`bin/scan-data-collector.js::parseDebtSummary`) parsed only the LEGACY breadth-scan prose format ("Critical items: N"). The deep-scan register uses a markdown severity TABLE (`| CRITICAL | 9 |`, emoji optional), so the report displayed 0 critical / 0 high. parseDebtSummary now reads both formats (prose first, then the table). +test/m77-renderer-table-summary.test.js (4 tests). Note: the per-item techDebt[] list section + the 10-finding sample still use legacy formats (secondary; headline cards are the fix).
+
+Suite: 1315 pass / 0 fail / 4 skip.
+
+## [4.0.24] - 2026-06-03 (M76 revision — keep severity color bullets)
+
+### Fixed — over-correction: v4.0.23 stripped the severity color bullets too
+
+v4.0.23 removed the severity emoji (🔴🟠🟡🟢) along with the em-dashes. But the emoji render fine and are wanted; the actual mojibake cause was only the em/en-dashes and smart quotes. Reverted the emoji strip:
+
+- `templates/workflows/gsd-t-scan.workflow.js`: `ascii()` now normalizes em/en-dashes, smart quotes, and ellipsis ONLY — it no longer strips emoji. The severity color bullets are kept in the summary table + section headers. Doc-phase punctuation instruction updated to explicitly allow the severity bullets.
+- `test/m76-ascii-clean-register.test.js`: updated to assert bullets are KEPT and dashes/quotes/ellipsis are normalized.
+- One-off: restored the color bullets in the HiloAviation register (summary table + 4 severity section headers) while keeping the em-dash fix.
+
+Suite: 1311 pass / 0 fail / 4 skip — zero regressions.
+
 ## [4.0.23] - 2026-06-03 (M76 ASCII-Clean Register Output — patch)
 
 ### Fixed — register/docs rendered as mojibake in non-UTF-8 terminals

package/bin/scan-data-collector.js

@@ -8,11 +8,25 @@ function read(filePath) {
 
 function parseDebtSummary(text) {
   const n = (pattern) => { const m = text.match(pattern); return m ? parseInt(m[1], 10) : 0; };
-  return {
+  // Legacy prose format ("Critical items: N").
+  let out = {
     debtCritical: n(/Critical items?:\s*(\d+)/i),
     debtHigh:     n(/High priority:\s*(\d+)/i),
     debtMedium:   n(/Medium priority:\s*(\d+)/i)
   };
+  // M77: deep-scan register uses a markdown SEVERITY TABLE instead, e.g.
+  //   | 🔴 CRITICAL | 9 |   /   | HIGH | 90 |   (emoji optional, case-insensitive).
+  // If the prose format yielded nothing, parse the table rows.
+  if (!out.debtCritical && !out.debtHigh && !out.debtMedium) {
+    const row = (label) => {
+      // a table row whose first cell contains the label and whose next cell is a number
+      const re = new RegExp("\\|[^|\\n]*\\b" + label + "\\b[^|\\n]*\\|\\s*\\**\\s*(\\d+)\\s*\\**\\s*\\|", "i");
+      const m = text.match(re);
+      return m ? parseInt(m[1], 10) : 0;
+    };
+    out = { debtCritical: row("CRITICAL"), debtHigh: row("HIGH"), debtMedium: row("MEDIUM") };
+  }
+  return out;
 }
 
 function parseTestCoverage(text) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tekyzinc/gsd-t",
-  "version": "4.0.23",
+  "version": "4.0.25",
   "description": "GSD-T: Contract-Driven Development for Claude Code — 54 slash commands with headless-by-default workflow spawning, unattended supervisor relay with event stream, graph-powered code analysis, real-time agent dashboard, task telemetry, doc-ripple enforcement, backlog management, impact analysis, test sync, milestone archival, and PRD generation",
   "author": "Tekyz, Inc.",
   "license": "MIT",

package/templates/workflows/gsd-t-scan.workflow.js

@@ -494,21 +494,22 @@ counts.total = finalFindings.length;
 // of a 466KB register truncates at ~165KB — verified). Chunk 0 is the header+summary
 // (Write/create); the rest are appends. Items are grouped so a chunk never splits an
 // item, and the severity-section heading rides with its first item's chunk.
-// M76: register output must be ASCII-clean — no emoji, no em/en-dashes, no smart
-// quotes. They render as mojibake boxes in non-UTF-8 terminals/pagers (the
-// βPADCCH/πAPCCCH garbage a user saw). Finder text can also contain these, so sanitize
-// every user-supplied field, not just our own template strings.
+// M76 (revised): the mojibake culprit was EM-DASHES (—) and smart quotes, NOT the
+// severity color bullets (🔴🟠🟡🟢) — those render fine and are intentional/wanted.
+// So ascii() normalizes em/en-dashes, smart quotes, and ellipsis (and tidies trailing
+// whitespace), but KEEPS emoji. It's applied to finder-supplied free-text fields so a
+// stray dash in a description doesn't mojibake; the severity bullets live in the
+// template below and are preserved.
 function ascii(s) {
   return String(s == null ? "" : s)
-    .replace(/[\u{1F300}-\u{1FAFF}\u{2600}-\u{27BF}\u{2190}-\u{21FF}\u{2B00}-\u{2BFF}️]/gu, "") // emoji/symbols
-    .replace(/[—–]/g, "-")        // em/en dash -> hyphen
+    .replace(/[—–]/g, "-")        // em/en dash -> hyphen (the actual mojibake cause)
     .replace(/[‘’]/g, "'")        // smart single quotes
     .replace(/[“”]/g, '"')        // smart double quotes
     .replace(/…/g, "...")              // ellipsis
     .replace(/[ \t]+\n/g, "\n");            // tidy trailing whitespace
 }
 function fmtChunks(today) {
-  const sevHead = { CRITICAL: "Critical", HIGH: "High", MEDIUM: "Medium", LOW: "Low" };
+  const sevHead = { CRITICAL: "🔴 Critical", HIGH: "🟠 High", MEDIUM: "🟡 Medium", LOW: "🟢 Low" };
   const head = [];
   head.push(`# Tech Debt Register - ${projectDir.split("/").pop()}`, "");
   if (scanNumber) head.push(`**Scan #${scanNumber}** - Deep codebase scan (runtime-native, ${coverageComplete ? "full coverage" : "PARTIAL coverage"})`);
@@ -517,10 +518,10 @@ function fmtChunks(today) {
   head.push(`**Verified findings:** ${counts.total}`, "");
   head.push(`> Effort estimates use GSD-T-native units (domain / wave / spawn / token-spend). Never human-hours.`);
   head.push(`> TD numbering continues from the prior register (if any, archived). This scan begins at **TD-${tdStart}**.`, "");
-  if (!coverageComplete) head.push(`> [!] **PARTIAL COVERAGE - ${failedSlices.length} of ${slices.length} codebase areas were NOT scanned this pass** (failed to return findings): ${ascii(failedSlices.join(", "))}. Findings UNDER-COUNT the real debt. Re-run (resume) for full coverage.`, "");
+  if (!coverageComplete) head.push(`> ⚠️ **PARTIAL COVERAGE - ${failedSlices.length} of ${slices.length} codebase areas were NOT scanned this pass** (failed to return findings): ${ascii(failedSlices.join(", "))}. Findings UNDER-COUNT the real debt. Re-run (resume) for full coverage.`, "");
   head.push(`## Summary`, "", `| Severity | Count |`, `|----------|-------|`,
-    `| CRITICAL | ${counts.critical} |`, `| HIGH | ${counts.high} |`,
-    `| MEDIUM | ${counts.medium} |`, `| LOW | ${counts.low} |`,
+    `| 🔴 CRITICAL | ${counts.critical} |`, `| 🟠 HIGH | ${counts.high} |`,
+    `| 🟡 MEDIUM | ${counts.medium} |`, `| 🟢 LOW | ${counts.low} |`,
     `| **Total** | **${counts.total}** |`, "", "---", "");
 
   function itemMd(f, td) {
@@ -659,7 +660,7 @@ const docResults = await parallel(
       d.prompt,
       ``,
       isLiving ? mergeNote : `Write the file fresh in the format described (use Bash \`mkdir -p\` for parent dirs if needed).`,
-      `ASCII ONLY: do NOT use emoji, em-dashes (use " - "), en-dashes, smart quotes, or other non-ASCII punctuation — they render as garbage in non-UTF-8 terminals. Plain ASCII markdown only.`,
+      `PUNCTUATION: do NOT use em-dashes (use " - "), en-dashes, smart quotes, or ellipsis characters — those render as garbage in non-UTF-8 terminals. Use plain ASCII hyphens and straight quotes. (Severity color bullets 🔴🟠🟡🟢 are fine to keep where used for severity.)`,
       `Read the actual code under the relevant slice paths for specifics - don't summarize only from findings. Use Write/Edit to write the file, then return JSON per the schema (status "written"/"merged"/"skipped"/"failed"). Do NOT commit - the workflow handles git at the end.`,
     ].filter(Boolean).join("\n");
     try {