roadmapsmith 0.9.28 → 0.9.30

package/src/slash.js

@@ -3,65 +3,78 @@
 const SLASH_ACTIONS = [
   {
     id: 'zero',
+    tier: 'canonical',
     description: 'Interview the developer in terminal and generate the first roadmap for an empty or low-context repo.',
     classicCliExample: 'roadmapsmith zero',
     taskLabel: 'RoadmapSmith: Zero Mode'
   },
   {
     id: 'maintain',
+    tier: 'canonical',
     description: 'Preserve-first existing-repo flow: update, sync, and audit the roadmap without rebuilding substantive domain content.',
     classicCliExample: 'roadmapsmith maintain',
     taskLabel: 'RoadmapSmith: Maintain'
   },
   {
     id: 'status',
+    tier: 'canonical',
     description: 'Inspect CLI, roadmap, VS Code task, Codex, and Claude readiness.',
     classicCliExample: 'roadmapsmith status --json',
     taskLabel: 'RoadmapSmith: Status'
   },
+  {
+    id: 'validate',
+    tier: 'canonical',
+    description: 'Inspect per-task evidence status as JSON.',
+    classicCliExample: 'roadmapsmith validate --json --project-root .',
+    taskLabel: 'RoadmapSmith: Validate'
+  },
+  {
+    id: 'update',
+    tier: 'canonical',
+    aliases: ['sync'],
+    description: 'Apply evidence-backed checklist refresh to ROADMAP.md or complete one task with verified evidence.',
+    classicCliExample: 'roadmapsmith update --project-root .',
+    taskLabel: 'RoadmapSmith: Update'
+  },
+  {
+    id: 'setup',
+    tier: 'canonical',
+    description: 'Generate visible VS Code tasks and optional Claude hook wiring.',
+    classicCliExample: 'roadmapsmith setup',
+    taskLabel: 'RoadmapSmith: Refresh Setup'
+  },
   {
     id: 'init',
+    tier: 'advanced',
     description: 'Create ROADMAP.md and AGENTS.md when they are missing.',
     classicCliExample: 'roadmapsmith init',
     taskLabel: 'RoadmapSmith: Init'
   },
   {
     id: 'generate',
+    tier: 'advanced',
     description: 'Generate or update ROADMAP.md, refusing destructive replacement unless rerun with --full-regen.',
     classicCliExample: 'roadmapsmith generate --project-root .',
     taskLabel: 'RoadmapSmith: Generate'
   },
-  {
-    id: 'validate',
-    description: 'Inspect per-task evidence status as JSON.',
-    classicCliExample: 'roadmapsmith validate --json --project-root .',
-    taskLabel: 'RoadmapSmith: Validate'
-  },
-  {
-    id: 'sync',
-    description: 'Apply evidence-backed checklist sync to ROADMAP.md.',
-    classicCliExample: 'roadmapsmith sync --project-root .',
-    taskLabel: 'RoadmapSmith: Sync'
-  },
   {
     id: 'audit',
+    tier: 'advanced',
     description: 'Run sync and print the post-sync mismatch summary.',
     classicCliExample: 'roadmapsmith sync --audit --project-root .',
     taskLabel: 'RoadmapSmith: Sync Audit'
-  },
-  {
-    id: 'setup',
-    description: 'Generate visible VS Code tasks and optional Claude hook wiring.',
-    classicCliExample: 'roadmapsmith setup',
-    taskLabel: 'RoadmapSmith: Refresh Setup'
   }
 ];
 
 const SLASH_ROOT_ALIASES = new Set(['/roadmap', '/road']);
 const LEGACY_ROUTER_ALIAS = '/roadmap-sync';
+const CANONICAL_ACTION_IDS = Object.freeze(SLASH_ACTIONS.filter((action) => action.tier === 'canonical').map((action) => action.id));
+const ADVANCED_ACTION_IDS = Object.freeze(SLASH_ACTIONS.filter((action) => action.tier === 'advanced').map((action) => action.id));
+const COMPATIBILITY_ACTION_IDS = Object.freeze([]);
 
 function getNamespacedDirectSlash(actionId) {
-  return actionId === 'sync' ? '/roadmap-update' : `/roadmap-${actionId}`;
+  return `/roadmap-${actionId}`;
 }
 
 const DIRECT_HOST_NATIVE_ALIAS_TO_ACTION = Object.freeze(
@@ -69,25 +82,64 @@ const DIRECT_HOST_NATIVE_ALIAS_TO_ACTION = Object.freeze(
 );
 
 const DIRECT_DEPRECATED_CLI_ALIAS_TO_ACTION = Object.freeze(
-  Object.fromEntries(SLASH_ACTIONS.map((action) => [`/${action.id}`, action.id]))
+  Object.fromEntries(
+    SLASH_ACTIONS.flatMap((action) => {
+      const aliases = Array.isArray(action.aliases) ? action.aliases : [];
+      return [action.id, ...aliases].map((alias) => [`/${alias}`, action.id]);
+    })
+  )
 );
 
-function getHostNativeSkillNames() {
+const ACTION_ALIAS_TO_ID = Object.freeze(
+  Object.fromEntries(
+    SLASH_ACTIONS.flatMap((action) => {
+      const aliases = Array.isArray(action.aliases) ? action.aliases : [];
+      return [action.id, ...aliases].map((alias) => [alias, action.id]);
+    })
+  )
+);
+
+function mapActionIdsToSkillNames(actionIds) {
   return [
     'roadmap',
-    'roadmap-zero',
-    'roadmap-maintain',
-    'roadmap-status',
-    'roadmap-init',
-    'roadmap-generate',
-    'roadmap-validate',
-    'roadmap-update',
-    'roadmap-sync',
-    'roadmap-audit',
-    'roadmap-setup'
+    ...actionIds.map((actionId) => getNamespacedDirectSlash(actionId).slice(1))
   ];
 }
 
+function getCanonicalHostNativeSkillNames() {
+  return mapActionIdsToSkillNames(CANONICAL_ACTION_IDS);
+}
+
+function getAdvancedHostNativeSkillNames() {
+  return ADVANCED_ACTION_IDS.map((actionId) => getNamespacedDirectSlash(actionId).slice(1));
+}
+
+function getCompatibilityHostNativeSkillNames() {
+  return [
+    'roadmap-sync'
+  ];
+}
+
+function getHostNativeSkillNames() {
+  return [
+    ...getCanonicalHostNativeSkillNames(),
+    ...getAdvancedHostNativeSkillNames(),
+    ...getCompatibilityHostNativeSkillNames()
+  ];
+}
+
+function getCanonicalHostNativeSlashCommands() {
+  return getCanonicalHostNativeSkillNames().map((name) => `/${name}`);
+}
+
+function getAdvancedHostNativeSlashCommands() {
+  return getAdvancedHostNativeSkillNames().map((name) => `/${name}`);
+}
+
+function getCompatibilityHostNativeSlashCommands() {
+  return getCompatibilityHostNativeSkillNames().map((name) => `/${name}`);
+}
+
 function getHostNativeSlashCommands() {
   return getHostNativeSkillNames().map((name) => `/${name}`);
 }
@@ -97,23 +149,29 @@ function normalizeActionId(value) {
   if (normalized.startsWith('roadmap-')) {
     normalized = normalized.slice('roadmap-'.length);
   }
-  if (normalized === 'update') {
-    normalized = 'sync';
-  }
   return normalized;
 }
 
+function canonicalizeActionId(value) {
+  const normalized = normalizeActionId(value);
+  return ACTION_ALIAS_TO_ID[normalized] || normalized;
+}
+
 function isSlashToken(value) {
   return typeof value === 'string' && value.trim().startsWith('/');
 }
 
 function getSlashAction(actionId) {
-  const normalized = normalizeActionId(actionId);
+  const normalized = canonicalizeActionId(actionId);
   return SLASH_ACTIONS.find((action) => action.id === normalized) || null;
 }
 
 function getLegacyRouterSlash(action) {
-  return `/roadmap-sync ${action.id === 'sync' ? 'update' : action.id}`;
+  return `/roadmap-sync ${action.id}`;
+}
+
+function actionSearchTerms(action) {
+  return [action.id, ...(Array.isArray(action.aliases) ? action.aliases : [])];
 }
 
 function getSlashActionSpecs() {
@@ -127,13 +185,22 @@ function getSlashActionSpecs() {
 
 function getSlashSuggestions(query) {
   const normalized = normalizeActionId(query);
+  const visibleActions = SLASH_ACTIONS.filter((action) => action.tier !== 'compatibility');
   if (!normalized) {
-    return getSlashActionSpecs();
+    return visibleActions.map((action) => ({
+      ...action,
+      directSlash: getNamespacedDirectSlash(action.id),
+      routerSlash: `/roadmap ${action.id}`,
+      legacyRouterSlash: getLegacyRouterSlash(action)
+    }));
   }
 
-  const startsWithMatches = SLASH_ACTIONS.filter((action) => action.id.startsWith(normalized));
-  const containsMatches = SLASH_ACTIONS.filter((action) => {
-    return !action.id.startsWith(normalized) && action.id.includes(normalized);
+  const startsWithMatches = visibleActions.filter((action) => {
+    return actionSearchTerms(action).some((term) => term.startsWith(normalized));
+  });
+  const containsMatches = visibleActions.filter((action) => {
+    return !actionSearchTerms(action).some((term) => term.startsWith(normalized))
+      && actionSearchTerms(action).some((term) => term.includes(normalized));
   });
 
   return [...startsWithMatches, ...containsMatches].map((action) => ({
@@ -175,12 +242,12 @@ function resolveSlashInvocation(command, args = []) {
   const normalizedCommand = String(command).trim().toLowerCase();
 
   if (normalizedCommand === LEGACY_ROUTER_ALIAS) {
+    const deprecationMessage = 'Legacy CLI compatibility root /roadmap-sync <action> is deprecated. Use /roadmap <action> or the direct /roadmap-* commands.';
     if (args.length === 0) {
-      return paletteResponse(normalizedCommand, '');
+      return paletteResponse(normalizedCommand, '', true, deprecationMessage);
     }
 
     const queryToken = normalizeActionId(args[0]);
-    const deprecationMessage = 'Legacy CLI compatibility root /roadmap-sync <action> is deprecated. Use /roadmap <action> or the direct /roadmap-* commands.';
     const exactAction = getSlashAction(queryToken);
     if (exactAction) {
       return executeResponse(normalizedCommand, exactAction.id, queryToken, true, deprecationMessage);
@@ -268,7 +335,9 @@ function renderSlashPalette(options = {}) {
     suggestions.forEach((action) => {
       lines.push(`- ${action.directSlash}: ${action.description}`);
       lines.push(`  Router form: ${action.routerSlash}`);
-      lines.push(`  Legacy router: ${action.legacyRouterSlash}`);
+      if (options.deprecated || source === LEGACY_ROUTER_ALIAS) {
+        lines.push(`  Legacy router: ${action.legacyRouterSlash}`);
+      }
       lines.push(`  Classic CLI: ${action.classicCliExample}`);
       lines.push(`  VS Code task: ${action.taskLabel}`);
     });
@@ -280,7 +349,9 @@ function renderSlashPalette(options = {}) {
   lines.push('- roadmapsmith /roadmap maintain');
   lines.push('- roadmapsmith /roadmap-maintain');
   lines.push('- roadmapsmith /roadmap-update');
-  lines.push('- roadmapsmith /roadmap-sync validate');
+  if (options.deprecated || source === LEGACY_ROUTER_ALIAS) {
+    lines.push('- roadmapsmith /roadmap-sync validate');
+  }
   lines.push('');
   lines.push('Installing the skill alone does not expose CLI behavior in VS Code. Use roadmapsmith setup for the visible task/launcher layer.');
 
@@ -288,10 +359,20 @@ function renderSlashPalette(options = {}) {
 }
 
 module.exports = {
+  ACTION_ALIAS_TO_ID,
   DIRECT_DEPRECATED_CLI_ALIAS_TO_ACTION,
   DIRECT_HOST_NATIVE_ALIAS_TO_ACTION,
+  ADVANCED_ACTION_IDS,
+  CANONICAL_ACTION_IDS,
+  COMPATIBILITY_ACTION_IDS,
   LEGACY_ROUTER_ALIAS,
   SLASH_ROOT_ALIASES,
+  getAdvancedHostNativeSkillNames,
+  getAdvancedHostNativeSlashCommands,
+  getCanonicalHostNativeSkillNames,
+  getCanonicalHostNativeSlashCommands,
+  getCompatibilityHostNativeSkillNames,
+  getCompatibilityHostNativeSlashCommands,
   getHostNativeSkillNames,
   getHostNativeSlashCommands,
   getNamespacedDirectSlash,
@@ -299,6 +380,7 @@ module.exports = {
   getSlashActionSpecs,
   getSlashSuggestions,
   isSlashToken,
+  canonicalizeActionId,
   normalizeActionId,
   renderSlashPalette,
   resolveSlashInvocation

package/src/sync/index.js

@@ -112,6 +112,23 @@ function shouldPreserveExistingWarning(existingReason, newReason) {
   return cleanNew === 'validation failed' && cleanExisting && cleanExisting !== cleanNew;
 }
 
+function formatVerificationRecipe(indent, recipe) {
+  return `${indent}  - Verification recipe: ${recipe}`;
+}
+
+function formatTestEvidence(indent, evidence) {
+  return `${indent}  - Test evidence: ${evidence}`;
+}
+
+function findVerificationRecipeIndex(lines, taskLineIndex) {
+  for (let index = taskLineIndex + 1; index < lines.length; index += 1) {
+    const line = lines[index];
+    if (!line.trim() || /^\s*#{2,}\s/.test(line) || /^\s*- \[[ xX]\]\s/.test(line)) break;
+    if (/^\s*- Verification recipe:/i.test(line)) return index;
+  }
+  return null;
+}
+
 function applySync(content, parsedTasks, results) {
   const parsed = parseRoadmap(content);
   const lines = [...parsed.lines];
@@ -142,6 +159,19 @@ function applySync(content, parsedTasks, results) {
         lines.splice(warningIndex, 1);
         offset -= 1;
       }
+      const recipeIndex = findVerificationRecipeIndex(lines, lineIndex);
+      if (recipeIndex != null && recipeIndex >= 0 && recipeIndex < lines.length) {
+        lines.splice(recipeIndex, 1);
+        offset -= 1;
+      }
+      if (
+        result.generatedTestEvidence &&
+        (!Array.isArray(task.testEvidenceLines) || task.testEvidenceLines.length === 0)
+      ) {
+        const insertionIndex = Math.max(lineIndex + 1, (task.lastChildLineIndex + offset) + 1);
+        lines.splice(insertionIndex, 0, formatTestEvidence(task.indent || '', result.generatedTestEvidence));
+        offset += 1;
+      }
       if (
         result.staleEvidenceResolved &&
         (!Array.isArray(task.evidenceLines) || task.evidenceLines.length === 0) &&
@@ -154,10 +184,7 @@ function applySync(content, parsedTasks, results) {
         lines.splice(insertionIndex, 0, `${task.indent || ''}  - Evidence: ${result.discoveredEvidence}`);
         offset += 1;
       }
-      continue;
-    }
-
-    if (warningIndex != null && warningIndex >= 0 && warningIndex < lines.length) {
+    } else if (warningIndex != null && warningIndex >= 0 && warningIndex < lines.length) {
       const existingReason = normalizeWarningReason(lines[warningIndex]);
       const newReason = reason || 'validation failed';
       if (!shouldPreserveExistingWarning(existingReason, newReason)) {
@@ -167,6 +194,23 @@ function applySync(content, parsedTasks, results) {
       lines.splice(lastChildLineIndex + 1, 0, warningText);
       offset += 1;
     }
+
+    const recipeIndex = findVerificationRecipeIndex(lines, lineIndex);
+    if (result.passed) {
+      continue;
+    }
+    if (result.verificationRecipe) {
+      const recipeLine = formatVerificationRecipe(task.indent || '', result.verificationRecipe);
+      if (recipeIndex != null && recipeIndex >= 0 && recipeIndex < lines.length) {
+        lines[recipeIndex] = recipeLine;
+      } else {
+        lines.splice(lastChildLineIndex + 1, 0, recipeLine);
+        offset += 1;
+      }
+    } else if (recipeIndex != null && recipeIndex >= 0 && recipeIndex < lines.length) {
+      lines.splice(recipeIndex, 1);
+      offset -= 1;
+    }
   }
 
   return ensureTrailingNewline(lines.join('\n'));