@smartmemory/compose 0.2.21-beta → 0.2.22-beta

package/lib/feature-validator.js

@@ -32,7 +32,7 @@ import fs from 'node:fs';
 import path from 'node:path';
 import { fileURLToPath } from 'node:url';
 import { FEATURE_CODE_RE_STRICT, validateCode } from './feature-code.js';
-import { parseRoadmap } from './roadmap-parser.js';
+import { parseRoadmap, splitRoadmapCells } from './roadmap-parser.js';
 import { listFeatures, readFeature } from './feature-json.js';
 import { loadExternalPrefixes } from './project-paths.js';
 import { checkRoundtrip, LOSSY_LABELS } from './roadmap-roundtrip.js';
@@ -161,7 +161,9 @@ export function loadValidationContext(cwd, options = {}) {
       const rowMatch = rawLine.match(/^\|(.+)\|\s*$/);
       if (!rowMatch) { inTable = false; sawSeparator = false; continue; }
 
-      const cols = rowMatch[1].split('|').map((c) => c.trim());
+      // Escaped-pipe-aware split (COMP-MCP-VALIDATE-4): a `\|` in a description
+      // cell must not shift status-column detection and read prose as the status.
+      const cols = splitRoadmapCells(rawLine);
 
       // Detect header row by column names. Recognize common column-name variants
       // (feature/code/item/name) and (status/state) so non-canonical tables that

package/lib/feature-write-guard.js

@@ -19,6 +19,7 @@ import { join, resolve, dirname } from 'path';
 import { fileURLToPath } from 'url';
 import { SchemaValidator } from '../server/schema-validator.js';
 import { FEATURE_CODE_RE_STRICT } from './feature-code.js';
+import { splitRoadmapCells } from './roadmap-parser.js';
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const SCHEMA_PATH = resolve(__dirname, '../contracts/feature-json.schema.json');
@@ -108,7 +109,9 @@ export function scanRoadmapRows(roadmapPath) {
     if (/^##\s+/.test(rawLine)) { inTable = false; sawSeparator = false; codeIdx = statusIdx = -1; continue; }
     const rowMatch = rawLine.match(/^\|(.+)\|\s*$/);
     if (!rowMatch) { inTable = false; sawSeparator = false; continue; }
-    const cols = rowMatch[1].split('|').map((c) => c.trim());
+    // Escaped-pipe-aware split (COMP-MCP-VALIDATE-4): keep column detection stable
+    // when a description cell contains `\|`.
+    const cols = splitRoadmapCells(rawLine);
     const lower = cols.map((c) => c.toLowerCase());
     const featureColIdx = lower.findIndex((c) => ['feature', 'code', 'item', 'name'].includes(c));
     const statusColIdx = lower.findIndex((c) => ['status', 'state'].includes(c));

package/lib/roadmap-parser.js

@@ -21,6 +21,24 @@ const MILESTONE_HEADING_RE = /^###\s+(.+?)(?:\s*:\s*(.+))?$/;
 const TABLE_ROW_RE = /^\|(.+)\|$/;
 const FENCE_RE = /^```/;
 
+/**
+ * Split a markdown table row into trimmed cells, honoring escaped pipes.
+ * Splits on UNESCAPED `|` only (`/(?<!\\)\|/`), drops the leading/trailing empty
+ * cells from the outer pipes, and unescapes `\|` → `|` in each cell. Symmetric
+ * with `escCell()` in roadmap-gen.js. For pipe-free rows this is identical to a
+ * naive `split('|')`.
+ *
+ * The canonical row splitter — every ROADMAP-row parse site (validator,
+ * write-guard, this parser) must use it so a `\|` in a description cell can never
+ * shift status-column detection (COMP-MCP-VALIDATE-4).
+ *
+ * @param {string} rawLine - A full table row line, e.g. "| a | b \\| c | PLANNED |"
+ * @returns {string[]} trimmed, unescaped cells
+ */
+export function splitRoadmapCells(rawLine) {
+  return rawLine.trim().split(/(?<!\\)\|/).slice(1, -1).map((c) => c.trim().replace(/\\\|/g, '|'));
+}
+
 /**
  * @typedef {{ code: string, description: string, status: string, phaseId: string, position: number }} FeatureEntry
  */
@@ -112,10 +130,8 @@ export function parseRoadmap(text) {
       continue;
     }
 
-    // Split on UNESCAPED pipes only, then unescape `\|` → `|` in each cell.
-    // Symmetric with escCell() in roadmap-gen.js. For pipe-free rows this is
-    // identical to split('|') (no backslashes, no lookbehind matches).
-    const cells = trimmed.split(/(?<!\\)\|/).slice(1, -1).map(c => c.trim().replace(/\\\|/g, '|'));
+    // Escaped-pipe-aware cell split (the canonical splitter — see splitRoadmapCells).
+    const cells = splitRoadmapCells(trimmed);
 
     // Skip separator rows (|---|---|---|)
     if (cells.every(c => /^[-:]+$/.test(c))) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@smartmemory/compose",
-  "version": "0.2.21-beta",
+  "version": "0.2.22-beta",
   "description": "Structured AI dev pipeline — goal-to-product orchestration with gates, iteration loops, and feature lifecycle management.",
   "author": "SmartMemory",
   "license": "MIT",