roadmapsmith 0.9.22 → 0.9.24

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "roadmapsmith",
-  "version": "0.9.22",
+  "version": "0.9.24",
   "description": "One-command evidence-backed ROADMAP.md generator and sync tool for AI coding agents, with shared RoadmapSmith plugin skills for Codex and Claude.",
   "author": {
     "name": "PapiScholz"

package/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "roadmapsmith",
-  "version": "0.9.22",
+  "version": "0.9.24",
   "description": "One-command evidence-backed ROADMAP.md generator and sync tool for AI coding agents, with shared RoadmapSmith plugin skills for Codex and Claude.",
   "author": {
     "name": "PapiScholz"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "roadmapsmith",
-  "version": "0.9.22",
+  "version": "0.9.24",
   "description": "One-command evidence-backed ROADMAP.md generator and sync tool for AI coding agents, with shared RoadmapSmith plugin skills for Codex and Claude.",
   "main": "src/index.js",
   "bin": {

package/skills.json

@@ -29,67 +29,67 @@
       "name": "roadmap",
       "path": "skills/roadmap",
       "description": "Native slash palette for RoadmapSmith commands and recommended entrypoints across supported hosts.",
-      "version": "0.9.22"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-zero",
       "path": "skills/roadmap-zero",
       "description": "Native slash entrypoint for the one-command Zero Mode CLI workflow.",
-      "version": "0.9.22"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-maintain",
       "path": "skills/roadmap-maintain",
       "description": "Native slash entrypoint for the preserve-first generate + sync + audit flow.",
-      "version": "0.9.22"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-status",
       "path": "skills/roadmap-status",
       "description": "Native slash readiness check grounded in roadmapsmith doctor JSON.",
-      "version": "0.9.22"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-init",
       "path": "skills/roadmap-init",
       "description": "Native slash entrypoint for creating ROADMAP.md and AGENTS.md.",
-      "version": "0.9.22"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-generate",
       "path": "skills/roadmap-generate",
       "description": "Native slash entrypoint for managed roadmap updates that require --full-regen before destructive replacement.",
-      "version": "0.9.22"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-validate",
       "path": "skills/roadmap-validate",
       "description": "Native slash entrypoint for evidence-backed roadmap validation.",
-      "version": "0.9.22"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-update",
       "path": "skills/roadmap-update",
       "description": "Native slash entrypoint for applying evidence-backed checklist sync.",
-      "version": "0.9.22"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-sync",
       "path": "skills/roadmap-sync",
       "description": "Legacy namespaced root plus policy guidance for RoadmapSmith slash workflows.",
-      "version": "0.9.22"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-audit",
       "path": "skills/roadmap-audit",
       "description": "Native slash entrypoint for the current sync-plus-audit workflow.",
-      "version": "0.9.22"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-setup",
       "path": "skills/roadmap-setup",
       "description": "Native slash entrypoint for generating RoadmapSmith host integration files.",
-      "version": "0.9.22"
+      "version": "0.9.24"
     }
   ]
 }

package/src/classifier/index.js

@@ -21,6 +21,7 @@ const ELECTRON_CONFIGS = [
   'forge.config.ts'
 ];
 const LANDING_ROUTE_RE = /(?:^|\/)(?:contact|services|about|pricing|hero|cta|landing)(?:\/|\.)/i;
+const FIXTURE_PATH_RE = /(^|\/)(?:test|tests)\/fixtures\//i;
 
 function readPackageDeps(projectRoot) {
   if (!projectRoot) return [];
@@ -52,6 +53,9 @@ function hasWorkspaces(projectRoot) {
 }
 
 function classifyProject({ projectRoot, files }) {
+  const candidateFiles = Array.isArray(files)
+    ? files.filter((file) => !FIXTURE_PATH_RE.test(String(file || '')))
+    : [];
   const signals = [];
 
   if (hasWorkspaces(projectRoot)) {
@@ -59,8 +63,8 @@ function classifyProject({ projectRoot, files }) {
     return { type: 'monorepo', confidence: 'high', signals };
   }
 
-  const hasPy = hasFilename(files, 'pyproject.toml') || hasFilename(files, 'setup.py');
-  if (hasPy && !files.some((f) => /\.[jt]sx?$/.test(f))) {
+  const hasPy = hasFilename(candidateFiles, 'pyproject.toml') || hasFilename(candidateFiles, 'setup.py');
+  if (hasPy && !candidateFiles.some((f) => /\.[jt]sx?$/.test(f))) {
     signals.push('pyproject.toml / setup.py, no JS files');
     return { type: 'python-package', confidence: 'high', signals };
   }
@@ -82,72 +86,72 @@ function classifyProject({ projectRoot, files }) {
   }
 
   for (const dir of WEB_DIRS) {
-    if (hasDir(files, dir)) {
+    if (hasDir(candidateFiles, dir)) {
       webScore += 2;
       signals.push(`directory: ${dir.replace(/\/$/, '')}`);
     }
   }
 
   for (const dir of ASSET_DIRS) {
-    if (hasDir(files, dir)) {
+    if (hasDir(candidateFiles, dir)) {
       webScore += 1;
       signals.push(`directory: ${dir.replace(/\/$/, '')}`);
     }
   }
 
-  if (hasDir(files, 'electron/')) {
+  if (hasDir(candidateFiles, 'electron/')) {
     electronScore += 3;
     signals.push('directory: electron');
   }
 
-  if (files.some((file) => /^electron\/.+\.(js|ts|cjs|mjs)$/.test(file))) {
+  if (candidateFiles.some((file) => /^electron\/.+\.(js|ts|cjs|mjs)$/.test(file))) {
     electronScore += 2;
     signals.push('electron main/preload sources');
   }
 
   for (const cfg of ELECTRON_CONFIGS) {
-    if (hasFilename(files, cfg)) {
+    if (hasFilename(candidateFiles, cfg)) {
       electronScore += 2;
       signals.push(`config: ${cfg}`);
     }
   }
 
   for (const cfg of WEB_CONFIGS) {
-    if (hasFilename(files, cfg)) {
+    if (hasFilename(candidateFiles, cfg)) {
       webScore += 3;
       signals.push(`config: ${cfg}`);
     }
   }
 
   for (const cfg of STYLE_CONFIGS) {
-    if (hasFilename(files, cfg)) {
+    if (hasFilename(candidateFiles, cfg)) {
       webScore += 1;
       signals.push(`config: ${cfg}`);
     }
   }
 
-  if (files.some((f) => /\.css$/.test(f))) {
+  if (candidateFiles.some((f) => /\.css$/.test(f))) {
     webScore += 1;
     signals.push('CSS files present');
   }
 
-  const landingRoutes = files.filter((f) => LANDING_ROUTE_RE.test(f));
+  const landingRoutes = candidateFiles.filter((f) => LANDING_ROUTE_RE.test(f));
   if (landingRoutes.length > 0) {
     landingScore += landingRoutes.length * 2;
     signals.push(`landing/service routes: ${landingRoutes.length}`);
   }
 
-  if (hasFilename(files, 'favicon.ico') || hasFilename(files, 'logo.png') || hasFilename(files, 'logo.svg')) {
+  if (hasFilename(candidateFiles, 'favicon.ico') || hasFilename(candidateFiles, 'logo.png') || hasFilename(candidateFiles, 'logo.svg')) {
     landingScore += 1;
     signals.push('branding asset in public/');
   }
 
-  if (webScore === 0 && (files.some((f) => f.startsWith('bin/')) || hasFilename(files, 'cli.js'))) {
+  if (webScore === 0 && (candidateFiles.some((f) => f.startsWith('bin/')) || hasFilename(candidateFiles, 'cli.js'))) {
     signals.push('bin/ directory or cli.js');
     return { type: 'cli-tool', confidence: 'medium', signals };
   }
 
-  if (webScore === 0 && hasFilename(files, 'package.json')) {
+  if (webScore === 0 && hasFilename(candidateFiles, 'package.json')) {
     signals.push('package.json, no web signals');
     return { type: 'npm-package', confidence: 'low', signals };
   }

package/src/generator/index.js

@@ -207,6 +207,10 @@ function renderAdditionTask(task) {
   return `- [ ] ${task.text} <!-- rs:task=${task.id} -->`;
 }
 
+function isGenericPreserveModeCandidate(candidate) {
+  return candidate && ['default', 'classifier', 'todo-hint'].includes(candidate.source);
+}
+
 function buildManagedAdditionsLines(tasks, options = {}) {
   const groups = groupByPhase(tasks);
   const lines = [];
@@ -465,6 +469,10 @@ function insertPreserveModeTasks(existingContent, parsedRoadmap, tasks) {
   return nextLines.join('\n');
 }
 
+function filterPreserveModeCandidates(candidates) {
+  return candidates.filter((candidate) => !isGenericPreserveModeCandidate(candidate));
+}
+
 function mergeWithExisting(candidates, existingTasks, options = {}) {
   const matchedExistingIds = new Set();
   const merged = [];
@@ -810,14 +818,18 @@ function generateRoadmapDocument(options) {
 
   if (hasSubstantiveManagedBlock(existing) && preserveManagedBlock && !forceFullRegenerate) {
     const unmatchedCandidates = allCandidates.filter((candidate) => {
-      return !findBestTaskMatch(candidate, existingManagedTasks, { allowFuzzy: false });
+      return !findBestTaskMatch(candidate, existingManagedTasks, {
+        allowFuzzy: true,
+        minScore: 0.72
+      });
     });
+    const preserveModeCandidates = filterPreserveModeCandidates(unmatchedCandidates);
 
-    if (unmatchedCandidates.length === 0) {
+    if (preserveModeCandidates.length === 0) {
       return existingContent;
     }
 
-    return insertPreserveModeTasks(existingContent, existing, unmatchedCandidates);
+    return insertPreserveModeTasks(existingContent, existing, preserveModeCandidates);
   }
 
   if (hasSubstantiveManagedBlock(existing) && !forceFullRegenerate) {

package/src/sync/index.js

@@ -3,6 +3,8 @@
 const { parseRoadmap } = require('../parser');
 const { ensureTrailingNewline } = require('../utils');
 
+const WARNING_REASON_PREFIX = 'attempted but validation failed:';
+
 function setChecklistState(line, checked) {
   return line.replace(/- \[( |x|X)\]/, `- [${checked ? 'x' : ' '}]`);
 }
@@ -11,6 +13,95 @@ function formatWarning(indent, reason) {
   return `${indent}  - ⚠️ attempted but validation failed: ${reason}`;
 }
 
+function isWhitespaceCharacter(char) {
+  return char === ' ' || char === '\t' || char === '\n' || char === '\r' || char === '\f' || char === '\v';
+}
+
+function stripLeadingWarningMarker(value) {
+  let index = 0;
+  const source = String(value || '');
+  while (index < source.length && isWhitespaceCharacter(source[index])) {
+    index += 1;
+  }
+  if (source.slice(index, index + 2) === '⚠️') {
+    index += 2;
+  }
+  while (index < source.length && isWhitespaceCharacter(source[index])) {
+    index += 1;
+  }
+  return source.slice(index);
+}
+
+function splitWarningReasonSegments(value) {
+  const source = String(value || '');
+  const segments = [];
+  let current = '';
+  let index = 0;
+
+  while (index < source.length) {
+    const char = source[index];
+    if (char === ';') {
+      const trimmed = current.trim();
+      if (trimmed) {
+        segments.push(trimmed);
+      }
+      current = '';
+      index += 1;
+      while (index < source.length && isWhitespaceCharacter(source[index])) {
+        index += 1;
+      }
+      continue;
+    }
+
+    current += char;
+    index += 1;
+  }
+
+  const trimmed = current.trim();
+  if (trimmed) {
+    segments.push(trimmed);
+  }
+
+  return segments;
+}
+
+function normalizeWarningReason(reason) {
+  let normalized = String(reason || '').trim();
+  if (!normalized) {
+    return '';
+  }
+
+  normalized = stripLeadingWarningMarker(normalized).trim();
+  const prefixIndex = normalized.indexOf(WARNING_REASON_PREFIX);
+  if (prefixIndex >= 0) {
+    normalized = normalized.slice(prefixIndex + WARNING_REASON_PREFIX.length).trim();
+  }
+
+  return normalized;
+}
+
+function normalizeWarningReasons(reasons) {
+  const normalized = [];
+  const seen = new Set();
+  for (const reason of Array.isArray(reasons) ? reasons : [reasons]) {
+    for (const chunk of splitWarningReasonSegments(reason)) {
+      const clean = normalizeWarningReason(chunk);
+      if (!clean || seen.has(clean)) {
+        continue;
+      }
+      seen.add(clean);
+      normalized.push(clean);
+    }
+  }
+  return normalized;
+}
+
+function shouldPreserveExistingWarning(existingReason, newReason) {
+  const cleanExisting = normalizeWarningReason(existingReason);
+  const cleanNew = normalizeWarningReason(newReason) || 'validation failed';
+  return cleanNew === 'validation failed' && cleanExisting && cleanExisting !== cleanNew;
+}
+
 function applySync(content, parsedTasks, results) {
   const parsed = parseRoadmap(content);
   const lines = [...parsed.lines];
@@ -30,7 +121,7 @@ function applySync(content, parsedTasks, results) {
 
     lines[lineIndex] = setChecklistState(lines[lineIndex], result.passed);
 
-    const reason = result.reasons.join('; ');
+    const reason = normalizeWarningReasons(result.reasons).join('; ');
     const warningText = formatWarning(task.indent || '', reason || 'validation failed');
     const hasWarning = task.warningLineIndex != null;
     const warningIndex = hasWarning ? task.warningLineIndex + offset : null;
@@ -47,9 +138,7 @@ function applySync(content, parsedTasks, results) {
     if (warningIndex != null && warningIndex >= 0 && warningIndex < lines.length) {
       const existingReason = lines[warningIndex].split('validation failed:')[1];
       const newReason = reason || 'validation failed';
-      // Preserve existing warning when it's more descriptive than the new generic message.
-      const existingIsMoreSpecific = existingReason && existingReason.trim().length > newReason.length;
-      if (!existingIsMoreSpecific) {
+      if (!shouldPreserveExistingWarning(existingReason, newReason)) {
         lines[warningIndex] = warningText;
       }
     } else {

package/src/validator/index.js

@@ -373,6 +373,10 @@ function isDocTask(taskText) {
   return /\b(add|create|write|update|init|initialize|introduce|setup|document)\b/.test(normalized);
 }
 
+function isImplementationTask(taskText) {
+  return !isDocTask(taskText) && (isCodeTask(taskText) || taskDescribesChange(taskText));
+}
+
 function findFilesByPathHints(pathHints, fileIndex) {
   const matches = [];
   for (const hint of pathHints) {
@@ -1177,7 +1181,13 @@ function validateTask(task, context, config, plugins) {
     uniqueReasons = Array.isArray(overrideResult.reasons) ? Array.from(new Set(overrideResult.reasons)) : [];
   }
 
-  const attempted = hasStrongEvidence || hasWeakEvidence || pathHints.length > 0 || symbolHints.length > 0 || authoritativeEvidence.active;
+  const hasConcreteReferenceEvidence = filesFromPurePathHints.length > 0 || filesFromSymbols.length > 0;
+  const attempted = authoritativeEvidence.active
+    || hasRuleGrantedEvidence
+    || evidence.code
+    || evidence.test
+    || evidence.artifact
+    || hasConcreteReferenceEvidence;
   const { categories: strongEvidenceCategories } = countStrongEvidenceCategories(task.text, evidence);
   const strongEvidenceCount = strongEvidenceCategories.length;
   // Only pure path hints (not line-reference hints like file.ts:169) count as direct evidence.
@@ -1220,6 +1230,7 @@ function validateTask(task, context, config, plugins) {
   // WHERE to implement, not that implementation is done. Unchecked tasks need authoritative
   // evidence, artifact evidence, or strong code+test threshold to pass.
   // Already-checked tasks with found path hints are preserved via shouldPreserveCheckedTask.
+  const hasHighConfidenceImplementationEvidence = meetsStrongThreshold && evidence.code && evidence.test;
   let passed = authoritativeEvidence.passed || hasArtifactTaskPass || hasTrustedRuleEvidencePass || meetsStrongThreshold;
 
   if (!passed && !task.checked && hasDirectReferencePass) {
@@ -1234,30 +1245,29 @@ function validateTask(task, context, config, plugins) {
   // human/agent judgment that the feature is incomplete.
   if (task.warningText && !task.checked && passed && !authoritativeEvidence.passed) {
     passed = false;
-    uniqueReasons.push(task.warningText);
-    uniqueReasons = Array.from(new Set(uniqueReasons));
+    if (uniqueReasons.length === 0) {
+      uniqueReasons.push('validation failed');
+    }
   }
   if (negativeSignalMatches.length > 0) {
     passed = false;
   }
 
-  // Action-verb gate (Causa 3): unchecked tasks that describe a change to be made
-  // (Agregar, Configurar, Add, Fix, Manejo, Recovery path, …) cannot pass on code token overlap alone.
-  // Requires either: an Evidence line (authoritativeEvidence.passed), high-confidence evidence
-  // (code + test), grant-evidence from config (hasTrustedRuleEvidencePass), or canonical artifact
-  // evidence (hasArtifactTaskPass — e.g. "Add SECURITY.md").
+  // Unchecked implementation tasks need explicit evidence or high-confidence implementation
+  // evidence. Weak token overlap, direct file references, or code-only matches are not enough.
   if (
     !task.checked &&
     passed &&
-    taskDescribesChange(task.text) &&
+    isImplementationTask(task.text) &&
     !authoritativeEvidence.passed &&
     !hasTrustedRuleEvidencePass &&
-    !hasArtifactTaskPass
+    !hasArtifactTaskPass &&
+    !hasHighConfidenceImplementationEvidence
   ) {
     passed = false;
-    const actionVerbReason = 'action task requires Evidence line or high-confidence evidence (code + test) to be marked complete';
-    if (!uniqueReasons.includes(actionVerbReason)) {
-      uniqueReasons.push(actionVerbReason);
+    const implementationReason = 'implementation task requires Evidence line or high-confidence evidence (code + test) to be marked complete';
+    if (!uniqueReasons.includes(implementationReason)) {
+      uniqueReasons.push(implementationReason);
     }
   }