@tekyzinc/gsd-t 4.0.22 → 4.0.24

package/CHANGELOG.md

@@ -2,6 +2,30 @@
 
 All notable changes to GSD-T are documented here. Updated with each release.
 
+## [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
+
+The scan register used emoji severity markers (🔴🟠🟡🟢) and em-dashes. In a non-UTF-8 terminal/pager these display as garbage boxes (`βPADCCH`/`πAPCCCH`). The file bytes were valid UTF-8 — purely a render problem — but the emoji added nothing and made the register unreadable in common viewers.
+
+- `templates/workflows/gsd-t-scan.workflow.js`: added `ascii()` sanitizer (strips emoji/symbols, normalizes em/en dashes → `-`, smart quotes → ASCII, ellipsis → `...`). `fmtChunks` now emits plain-ASCII headers/tables and sanitizes every user-supplied field (finder text can contain these too). Document-phase agents instructed "ASCII ONLY".
+- `test/m76-ascii-clean-register.test.js`: +5 tests (sanitizer behavior + a structural guard that fmtChunks' output literals contain no emoji/em-dash).
+- One-off: cleaned the existing HiloAviation scan docs (techdebt.md + plain-english + 5 dimension files) — 0 emoji / 0 em-dashes remaining.
+
+Suite: 1311 pass / 0 fail / 4 skip — zero regressions.
+
 ## [4.0.22] - 2026-06-02 (M75 Deterministic Chunked Register Write — patch)
 
 ### Fixed — synthesis no longer stalls writing a large register

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tekyzinc/gsd-t",
-  "version": "4.0.22",
+  "version": "4.0.24",
   "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,30 +494,44 @@ 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 (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(/[—–]/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 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"})`);
+  head.push(`# Tech Debt Register - ${projectDir.split("/").pop()}`, "");
+  if (scanNumber) head.push(`**Scan #${scanNumber}** - Deep codebase scan (runtime-native, ${coverageComplete ? "full coverage" : "PARTIAL coverage"})`);
   head.push(`**Date:** ${today}`);
-  head.push(`**Slices run:** ${slices.length} | **Coverage:** ${coverageComplete ? `FULL — all ${slices.length} slices succeeded` : `PARTIAL — ${succeededCount}/${slices.length} succeeded`}`);
+  head.push(`**Slices run:** ${slices.length} | **Coverage:** ${coverageComplete ? `FULL - all ${slices.length} slices succeeded` : `PARTIAL - ${succeededCount}/${slices.length} succeeded`}`);
   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): ${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} |`,
     `| **Total** | **${counts.total}** |`, "", "---", "");
 
   function itemMd(f, td) {
-    const L = [`### TD-${td} — ${f.title || "(untitled)"}`,
-      `- **Area:** ${f.area || "—"}`, `- **Severity:** ${f.severity}`, `- **Status:** OPEN`,
-      `- **Location:** ${(f.files && f.files.length) ? f.files.join(", ") : "—"}`];
-    if (f.detail) L.push(`- **Description:** ${f.detail}`);
-    if (f.impact) L.push(`- **Impact:** ${f.impact}`);
-    if (f.recommendation) L.push(`- **Remediation:** ${f.recommendation}`);
-    if (f.slice) L.push(`- **Found in slice:** ${f.slice}`);
+    const L = [`### TD-${td} - ${ascii(f.title) || "(untitled)"}`,
+      `- **Area:** ${ascii(f.area) || "(none)"}`, `- **Severity:** ${f.severity}`, `- **Status:** OPEN`,
+      `- **Location:** ${(f.files && f.files.length) ? ascii(f.files.join(", ")) : "(none)"}`];
+    if (f.detail) L.push(`- **Description:** ${ascii(f.detail)}`);
+    if (f.impact) L.push(`- **Impact:** ${ascii(f.impact)}`);
+    if (f.recommendation) L.push(`- **Remediation:** ${ascii(f.recommendation)}`);
+    if (f.slice) L.push(`- **Found in slice:** ${ascii(f.slice)}`);
     L.push("");
     return L.join("\n");
   }
@@ -646,7 +660,8 @@ 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).`,
-      `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.`,
+      `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 {
       return await agent(prompt, { label: d.label, phase: "Document", schema: DOC_RESULT_SCHEMA, model: "sonnet" });