roadmapsmith 0.9.23 → 0.9.24

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "roadmapsmith",
-  "version": "0.9.23",
+  "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.23",
+  "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.23",
+  "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.23"
+      "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.23"
+      "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.23"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-status",
       "path": "skills/roadmap-status",
       "description": "Native slash readiness check grounded in roadmapsmith doctor JSON.",
-      "version": "0.9.23"
+      "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.23"
+      "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.23"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-validate",
       "path": "skills/roadmap-validate",
       "description": "Native slash entrypoint for evidence-backed roadmap validation.",
-      "version": "0.9.23"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-update",
       "path": "skills/roadmap-update",
       "description": "Native slash entrypoint for applying evidence-backed checklist sync.",
-      "version": "0.9.23"
+      "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.23"
+      "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.23"
+      "version": "0.9.24"
     },
     {
       "name": "roadmap-setup",
       "path": "skills/roadmap-setup",
       "description": "Native slash entrypoint for generating RoadmapSmith host integration files.",
-      "version": "0.9.23"
+      "version": "0.9.24"
     }
   ]
 }

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);
     }
   }