vibe-splain 2.0.2 → 2.0.3

package/dist/index.js

@@ -1508,6 +1508,13 @@ import { createHash as createHash2 } from "crypto";
 import { readFile as readFile8 } from "fs/promises";
 import { join as join7 } from "path";
 var CATEGORIES = ["Bottleneck", "Hack", "Smart-Move", "Risk", "Convention", "Dead-Weight"];
+function normalizeSnippet(s) {
+  let out = (s ?? "").replace(/\r\n/g, "\n");
+  if (/\\[nt]/.test(out)) {
+    out = out.replace(/\\r\\n/g, "\n").replace(/\\n/g, "\n").replace(/\\t/g, "  ");
+  }
+  return out.split("\n").map((l) => l.replace(/\s+$/, "")).join("\n").replace(/^\n+|\n+$/g, "");
+}
 var writeDecisionCardTool = {
   name: "write_decision_card",
   description: "Persists ONE Decision Card about ONE file. This is a hostile architecture review, not documentation. The thesis must be a VERDICT, not a description. The pillar MUST be one of the names from get_project_map (free-form is rejected). One card per file (duplicates rejected). Evidence must come from get_file_context hotSpans/smellSpans \u2014 never the header comment, never the whole file.",
@@ -1556,9 +1563,10 @@ async function handleWriteDecisionCard(args) {
   const tradeoff = args.tradeoff || null;
   const blastRadius = args.blastRadius || null;
   const confidence = args.confidence || "medium";
-  const evidence = args.evidence;
+  const rawEvidence = args.evidence;
+  const evidence = (rawEvidence || []).map((e) => ({ ...e, snippet: normalizeSnippet(e.snippet) }));
   const diagram = args.diagram || null;
-  if (!projectRoot || !pillar || !primaryFile || !title || !thesis || !category || !narrative || !evidence) {
+  if (!projectRoot || !pillar || !primaryFile || !title || !thesis || !category || !narrative || !rawEvidence || rawEvidence.length === 0) {
     throw new Error("projectRoot, pillar, primaryFile, title, thesis, category, narrative, and evidence are required");
   }
   if (!CATEGORIES.includes(category)) {

package/dist/mcp/tools/write_decision_card.js

@@ -4,6 +4,15 @@ import { createHash } from 'crypto';
 import { readFile } from 'fs/promises';
 import { join } from 'path';
 const CATEGORIES = ['Bottleneck', 'Hack', 'Smart-Move', 'Risk', 'Convention', 'Dead-Weight'];
+// Agents frequently emit snippets with literal "\n"/"\t" instead of real
+// newlines, which collapses the code into one line in the UI. Restore them.
+function normalizeSnippet(s) {
+    let out = (s ?? '').replace(/\r\n/g, '\n');
+    if (/\\[nt]/.test(out)) {
+        out = out.replace(/\\r\\n/g, '\n').replace(/\\n/g, '\n').replace(/\\t/g, '  ');
+    }
+    return out.split('\n').map(l => l.replace(/\s+$/, '')).join('\n').replace(/^\n+|\n+$/g, '');
+}
 export const writeDecisionCardTool = {
     name: 'write_decision_card',
     description: 'Persists ONE Decision Card about ONE file. This is a hostile architecture review, not documentation. The thesis must be a VERDICT, not a description. The pillar MUST be one of the names from get_project_map (free-form is rejected). One card per file (duplicates rejected). Evidence must come from get_file_context hotSpans/smellSpans — never the header comment, never the whole file.',
@@ -52,9 +61,10 @@ export async function handleWriteDecisionCard(args) {
     const tradeoff = args.tradeoff || null;
     const blastRadius = args.blastRadius || null;
     const confidence = args.confidence || 'medium';
-    const evidence = args.evidence;
+    const rawEvidence = args.evidence;
+    const evidence = (rawEvidence || []).map(e => ({ ...e, snippet: normalizeSnippet(e.snippet) }));
     const diagram = args.diagram || null;
-    if (!projectRoot || !pillar || !primaryFile || !title || !thesis || !category || !narrative || !evidence) {
+    if (!projectRoot || !pillar || !primaryFile || !title || !thesis || !category || !narrative || !rawEvidence || rawEvidence.length === 0) {
         throw new Error('projectRoot, pillar, primaryFile, title, thesis, category, narrative, and evidence are required');
     }
     if (!CATEGORIES.includes(category)) {