fix: align display output with CLI UX guidelines spec

Bold section headers in session_start (Threads, Decisions), use
STATUS.pass constants instead of inline ANSI in session-close and
confirm-scars, fix recall verification header color from red to
yellow per spec.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: 2 people

src/tools/confirm-scars.ts

@@ -102,7 +102,7 @@ function formatResponse(
     lines.push(`${STATUS.ok} SCAR CONFIRMATIONS ACCEPTED`);
     lines.push("");
     for (const conf of confirmations) {
-      const indicator = conf.decision === "APPLYING" ? `${ANSI.green}+${ANSI.reset}` : conf.decision === "N_A" ? `${ANSI.dim}-${ANSI.reset}` : `${ANSI.yellow}!${ANSI.reset}`;
+      const indicator = conf.decision === "APPLYING" ? STATUS.pass : conf.decision === "N_A" ? `${ANSI.dim}-${ANSI.reset}` : `${ANSI.yellow}!${ANSI.reset}`;
       lines.push(`${indicator} **${conf.scar_title}** → ${conf.decision}`);
     }
     lines.push("");

src/tools/recall.ts

@@ -150,7 +150,7 @@ No past lessons match this plan closely enough. Scars accumulate as you work —
 
   // Display blocking verification requirements FIRST and prominently
   if (scarsWithVerification.length > 0) {
-    lines.push(`${ANSI.red}[!!] VERIFICATION REQUIRED BEFORE PROCEEDING${ANSI.reset}`);
+    lines.push(`${ANSI.yellow}VERIFICATION REQUIRED${ANSI.reset}`);
     lines.push("");
 
     for (const scar of scarsWithVerification) {
@@ -167,7 +167,7 @@ No past lessons match this plan closely enough. Scars accumulate as you work —
       lines.push("");
       lines.push(`**MUST SHOW:** ${rv.must_show}`);
       lines.push("");
-      lines.push(`${ANSI.red}[!!] DO NOT proceed until verification output is shown above.${ANSI.reset}`);
+      lines.push(`${ANSI.red}Do not proceed until verification output is shown.${ANSI.reset}`);
       lines.push("");
     }
 

src/tools/session-close.ts

@@ -339,7 +339,7 @@ function formatCloseDisplay(
     if (r.what_worked || r.do_differently) {
       lines.push("");
       if (r.what_worked) {
-        lines.push(`${ANSI.green}+${ANSI.reset} ${truncate(r.what_worked, 72)}`);
+        lines.push(`${STATUS.pass} ${truncate(r.what_worked, 72)}`);
       }
       if (r.do_differently) {
         lines.push(`${ANSI.yellow}>${ANSI.reset} ${truncate(r.do_differently, 72)}`);

src/tools/session-start.ts

@@ -38,7 +38,7 @@ import { setGitmemDir, getGitmemDir, getSessionPath, getConfigProject } from "..
 import { registerSession, findSessionByHostPid, pruneStale, migrateFromLegacy } from "../services/active-sessions.js";
 import * as os from "os";
 import { formatDate } from "../services/timezone.js";
-import { productLine, dimText } from "../services/display-protocol.js";
+import { productLine, dimText, boldText } from "../services/display-protocol.js";
 // Suggested threads removed from start display
 import type { PerformanceBreakdown, ComponentPerformance, SurfacedScar, Observation, SessionChild } from "../types/index.js";
 import type {
@@ -833,7 +833,7 @@ function formatStartDisplay(result: SessionStartResult, displayInfoMap?: Map<str
 
   if (hasThreads) {
     visual.push("");
-    visual.push(`Threads (${result.open_threads!.length})`);
+    visual.push(boldText(`Threads (${result.open_threads!.length})`));
 
     const enriched = result.open_threads!.map(t => ({
       thread: t,
@@ -856,7 +856,7 @@ function formatStartDisplay(result: SessionStartResult, displayInfoMap?: Map<str
   // Decisions section — top 3 with compact date
   if (hasDecisions) {
     visual.push("");
-    visual.push(`Decisions (${result.recent_decisions!.length})`);
+    visual.push(boldText(`Decisions (${result.recent_decisions!.length})`));
     for (const d of result.recent_decisions!.slice(0, 3)) {
       const title = d.title.length > 50 ? d.title.slice(0, 47) + "..." : d.title;
       visual.push(`  ${title} · ${d.date}`);