roadmapsmith 0.4.0 → 0.5.1

package/src/model.js

@@ -1,33 +1,32 @@
-'use strict';
-
-const PHASE_ORDER = ['P0', 'P1', 'P2'];
-
-function phaseWeight(phase) {
-  const idx = PHASE_ORDER.indexOf(phase);
-  return idx >= 0 ? idx : PHASE_ORDER.length;
-}
-
-function createRoadmapModel(input) {
-  return {
-    northStar: input.northStar,
-    product: input.product || {},
-    currentState: input.currentState,
-    phases: input.phases,
-    steps: input.steps || [],
-    phasesDetailed: input.phasesDetailed || [],
-    milestones: input.milestones,
-    commandBreakdown: input.commandBreakdown,
-    exitCriteria: input.exitCriteria,
-    risks: input.risks,
-    antiGoals: input.antiGoals,
-    successCriteria: input.successCriteria || [],
-    customSections: input.customSections || [],
-    checkedById: input.checkedById || {}
-  };
-}
-
-module.exports = {
-  PHASE_ORDER,
-  createRoadmapModel,
-  phaseWeight
-};
+'use strict';
+
+const PHASE_ORDER = ['P0', 'P1', 'P2'];
+
+function phaseWeight(phase) {
+  const idx = PHASE_ORDER.indexOf(phase);
+  return idx >= 0 ? idx : PHASE_ORDER.length;
+}
+
+function createRoadmapModel(input) {
+  return {
+    northStar: input.northStar,
+    product: input.product || {},
+    currentState: input.currentState,
+    phases: input.phases,
+    steps: input.steps || [],
+    phasesDetailed: input.phasesDetailed || [],
+    milestones: input.milestones,
+    commandBreakdown: input.commandBreakdown,
+    exitCriteria: input.exitCriteria,
+    risks: input.risks,
+    antiGoals: input.antiGoals,
+    successCriteria: input.successCriteria || [],
+    customSections: input.customSections || [],
+    checkedById: input.checkedById || {}
+  };
+}
+
+module.exports = {
+  PHASE_ORDER,
+  createRoadmapModel
+};

package/src/parser/index.js

@@ -1,109 +1,107 @@
-'use strict';
-
-const { slugify } = require('../utils');
-
-const TASK_LINE_RE = /^(\s*)- \[( |x|X)\] (.*?)(?:\s*<!--\s*rs:task=([a-z0-9-]+)\s*-->)?\s*$/;
-const WARNING_RE = /^\s*-\s+⚠️ attempted but validation failed:\s*(.+?)\s*$/;
-const HEADING_RE = /^#{2,3}\s+(.*)$/;
-
-const MANAGED_START = '<!-- rs:managed:start -->';
-const MANAGED_END = '<!-- rs:managed:end -->';
-
-function parseRoadmap(content) {
-  const lines = String(content || '').split(/\r?\n/);
-  const tasks = [];
-  let section = '';
-
-  for (let index = 0; index < lines.length; index += 1) {
-    const line = lines[index];
-    const headingMatch = line.match(HEADING_RE);
-    if (headingMatch) {
-      section = headingMatch[1].trim();
-    }
-
-    const taskMatch = line.match(TASK_LINE_RE);
-    if (!taskMatch) {
-      continue;
-    }
-
-    const indent = taskMatch[1] || '';
-    const checked = taskMatch[2].toLowerCase() === 'x';
-    const text = taskMatch[3].trim();
-    const markerId = taskMatch[4] || null;
-
-    let warningLineIndex = null;
-    let warningText = null;
-    if (index + 1 < lines.length) {
-      const nextLine = lines[index + 1];
-      const warningMatch = nextLine.match(WARNING_RE);
-      if (warningMatch) {
-        warningLineIndex = index + 1;
-        warningText = warningMatch[1].trim();
-      }
-    }
-
-    const id = markerId || slugify(text);
-    tasks.push({
-      id,
-      text,
-      checked,
-      lineIndex: index,
-      warningLineIndex,
-      warningText,
-      markerId,
-      indent,
-      section
-    });
-  }
-
-  return {
-    lines,
-    tasks
-  };
-}
-
-function findManagedRange(lines) {
-  let start = -1;
-  let end = -1;
-
-  for (let i = 0; i < lines.length; i += 1) {
-    if (lines[i].trim() === MANAGED_START) {
-      start = i;
-      continue;
-    }
-    if (lines[i].trim() === MANAGED_END) {
-      end = i;
-      break;
-    }
-  }
-
-  if (start >= 0 && end >= 0 && start < end) {
-    return { start, end };
-  }
-  return null;
-}
-
-function upsertManagedBlock(existingContent, managedBody) {
-  const existing = String(existingContent || '');
-  const lines = existing.split(/\r?\n/);
-  const range = findManagedRange(lines);
-  const bodyLines = managedBody.split(/\r?\n/);
-
-  if (!range) {
-    if (existing.trim().length === 0) {
-      return [MANAGED_START, ...bodyLines, MANAGED_END].join('\n');
-    }
-    return `${existing.replace(/\s+$/, '')}\n\n${MANAGED_START}\n${managedBody}\n${MANAGED_END}`;
-  }
-
-  const prefix = lines.slice(0, range.start + 1);
-  const suffix = lines.slice(range.end);
-  return [...prefix, ...bodyLines, ...suffix].join('\n');
-}
-
-module.exports = {
-  MANAGED_END,
-  MANAGED_START,
-  parseRoadmap,
-  upsertManagedBlock
+'use strict';
+
+const { slugify } = require('../utils');
+
+const TASK_LINE_RE = /^(\s*)- \[( |x|X)\] (.*?)(?:\s*<!--\s*rs:task=([a-z0-9-]+)\s*-->)?\s*$/;
+const WARNING_RE = /^\s*-\s+⚠️ attempted but validation failed:\s*(.+?)\s*$/;
+const HEADING_RE = /^#{2,3}\s+(.*)$/;
+
+const MANAGED_START = '<!-- rs:managed:start -->';
+const MANAGED_END = '<!-- rs:managed:end -->';
+
+function parseRoadmap(content) {
+  const lines = String(content || '').split(/\r?\n/);
+  const tasks = [];
+  let section = '';
+
+  for (let index = 0; index < lines.length; index += 1) {
+    const line = lines[index];
+    const headingMatch = line.match(HEADING_RE);
+    if (headingMatch) {
+      section = headingMatch[1].trim();
+    }
+
+    const taskMatch = line.match(TASK_LINE_RE);
+    if (!taskMatch) {
+      continue;
+    }
+
+    const indent = taskMatch[1] || '';
+    const checked = taskMatch[2].toLowerCase() === 'x';
+    const text = taskMatch[3].trim();
+    const markerId = taskMatch[4] || null;
+
+    let warningLineIndex = null;
+    let warningText = null;
+    if (index + 1 < lines.length) {
+      const nextLine = lines[index + 1];
+      const warningMatch = nextLine.match(WARNING_RE);
+      if (warningMatch) {
+        warningLineIndex = index + 1;
+        warningText = warningMatch[1].trim();
+      }
+    }
+
+    const id = markerId || slugify(text);
+    tasks.push({
+      id,
+      text,
+      checked,
+      lineIndex: index,
+      warningLineIndex,
+      warningText,
+      markerId,
+      indent,
+      section
+    });
+  }
+
+  return {
+    lines,
+    tasks
+  };
+}
+
+function findManagedRange(lines) {
+  let start = -1;
+  let end = -1;
+
+  for (let i = 0; i < lines.length; i += 1) {
+    if (lines[i].trim() === MANAGED_START) {
+      start = i;
+      continue;
+    }
+    if (lines[i].trim() === MANAGED_END) {
+      end = i;
+      break;
+    }
+  }
+
+  if (start >= 0 && end >= 0 && start < end) {
+    return { start, end };
+  }
+  return null;
+}
+
+function upsertManagedBlock(existingContent, managedBody) {
+  const existing = String(existingContent || '');
+  const lines = existing.split(/\r?\n/);
+  const range = findManagedRange(lines);
+  const bodyLines = managedBody.split(/\r?\n/);
+
+  if (!range) {
+    if (existing.trim().length === 0) {
+      return [MANAGED_START, ...bodyLines, MANAGED_END].join('\n');
+    }
+    return `${existing.replace(/\s+$/, '')}\n\n${MANAGED_START}\n${managedBody}\n${MANAGED_END}`;
+  }
+
+  const prefix = lines.slice(0, range.start + 1);
+  const suffix = lines.slice(range.end);
+  return [...prefix, ...bodyLines, ...suffix].join('\n');
+}
+
+module.exports = {
+  parseRoadmap,
+  upsertManagedBlock
 };

package/src/renderer/professional.js

@@ -73,6 +73,13 @@ function renderSection3CurrentState(model, lines) {
   }
   lines.push('');
 
+  if (model.currentState.workspaces && model.currentState.workspaces.length > 0) {
+    lines.push('### Workspace Packages');
+    lines.push('');
+    lines.push(`- Workspace packages detected: ${model.currentState.workspaces.join(', ')}`);
+    lines.push('');
+  }
+
   lines.push('### Known Limitations');
   lines.push('');
   if (model.currentState.knownLimitations && model.currentState.knownLimitations.length > 0) {
@@ -169,8 +176,10 @@ function renderSection5Milestones(model, lines) {
       lines.push('**What Must Be Stable:**');
       lines.push('');
       for (const item of milestone.mustBeStable) {
-        const id = `prof-ms-${msSlug}-stable-${slugify(item)}`;
-        lines.push(`- [${checkedState(model, id) ? 'x' : ' '}] \`[P1]\` ${item} <!-- rs:task=${id} -->`);
+        const text = typeof item === 'string' ? item : item.text;
+        const note = typeof item === 'object' && item.note ? ` — _${item.note}_` : '';
+        const id = `prof-ms-${msSlug}-stable-${slugify(text)}`;
+        lines.push(`- [${checkedState(model, id) ? 'x' : ' '}] \`[P1]\` ${text}${note} <!-- rs:task=${id} -->`);
       }
       lines.push('');
     }

package/src/utils.js

@@ -1,143 +1,142 @@
-'use strict';
-
-const path = require('path');
-
-const STOP_WORDS = new Set([
-  'a', 'an', 'and', 'are', 'as', 'at', 'be', 'by', 'for', 'from', 'in', 'into', 'is', 'it', 'of', 'on', 'or', 'that',
-  'the', 'to', 'with', 'this', 'these', 'those', 'via', 'per', 'task', 'tasks', 'phase', 'priority'
-]);
-
-function toPosix(input) {
-  return input.split(path.sep).join('/');
-}
-
-function slugify(text) {
-  return String(text || '')
-    .toLowerCase()
-    .replace(/[^a-z0-9]+/g, '-')
-    .replace(/^-+|-+$/g, '')
-    .replace(/-{2,}/g, '-') || 'task';
-}
-
-function normalizeText(text) {
-  return String(text || '')
-    .toLowerCase()
-    .replace(/[`*_~#>\[\](){}.!?,:;"']/g, ' ')
-    .replace(/\s+/g, ' ')
-    .trim();
-}
-
-function tokenize(text) {
-  return normalizeText(text)
-    .split(' ')
-    .filter(Boolean)
-    .filter((token) => !STOP_WORDS.has(token));
-}
-
-function uniqueBy(items, keyFn) {
-  const seen = new Set();
-  const result = [];
-  for (const item of items) {
-    const key = keyFn(item);
-    if (seen.has(key)) {
-      continue;
-    }
-    seen.add(key);
-    result.push(item);
-  }
-  return result;
-}
-
-function similarityScore(left, right) {
-  const leftTokens = new Set(tokenize(left));
-  const rightTokens = new Set(tokenize(right));
-  if (leftTokens.size === 0 || rightTokens.size === 0) {
-    return 0;
-  }
-
-  let shared = 0;
-  for (const token of leftTokens) {
-    if (rightTokens.has(token)) {
-      shared += 1;
-    }
-  }
-
-  const union = new Set([...leftTokens, ...rightTokens]);
-  return shared / union.size;
-}
-
-function ensureTrailingNewline(text) {
-  return text.endsWith('\n') ? text : `${text}\n`;
-}
-
-function escapeRegExp(value) {
-  return String(value).replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-}
-
-function parseArgv(argv) {
-  const flags = {};
-  const positionals = [];
-
-  for (let i = 0; i < argv.length; i += 1) {
-    const current = argv[i];
-    if (!current.startsWith('-')) {
-      positionals.push(current);
-      continue;
-    }
-
-    if (current.startsWith('--')) {
-      const withoutPrefix = current.slice(2);
-      const eqIndex = withoutPrefix.indexOf('=');
-      let key;
-      let value;
-
-      if (eqIndex >= 0) {
-        key = withoutPrefix.slice(0, eqIndex);
-        value = withoutPrefix.slice(eqIndex + 1);
-      } else {
-        key = withoutPrefix;
-        const next = argv[i + 1];
-        if (next && !next.startsWith('-')) {
-          value = next;
-          i += 1;
-        } else {
-          value = true;
-        }
-      }
-
-      if (Object.prototype.hasOwnProperty.call(flags, key)) {
-        if (Array.isArray(flags[key])) {
-          flags[key].push(value);
-        } else {
-          flags[key] = [flags[key], value];
-        }
-      } else {
-        flags[key] = value;
-      }
-      continue;
-    }
-
-    const short = current.slice(1);
-    flags[short] = true;
-  }
-
-  const command = positionals.length > 0 ? positionals[0] : null;
-  return {
-    command,
-    args: positionals.slice(1),
-    flags,
-    positionals
-  };
-}
-
-module.exports = {
-  escapeRegExp,
-  ensureTrailingNewline,
-  normalizeText,
-  parseArgv,
-  similarityScore,
-  slugify,
-  toPosix,
-  tokenize,
-  uniqueBy
+'use strict';
+
+const path = require('path');
+
+const STOP_WORDS = new Set([
+  'a', 'an', 'and', 'are', 'as', 'at', 'be', 'by', 'for', 'from', 'in', 'into', 'is', 'it', 'of', 'on', 'or', 'that',
+  'the', 'to', 'with', 'this', 'these', 'those', 'via', 'per', 'task', 'tasks', 'phase', 'priority'
+]);
+
+function toPosix(input) {
+  return input.split(path.sep).join('/');
+}
+
+function slugify(text) {
+  return String(text || '')
+    .toLowerCase()
+    .replace(/[^a-z0-9]+/g, '-')
+    .replace(/^-+|-+$/g, '')
+    .replace(/-{2,}/g, '-') || 'task';
+}
+
+function normalizeText(text) {
+  return String(text || '')
+    .toLowerCase()
+    .replace(/[`*_~#>\[\](){}.!?,:;"']/g, ' ')
+    .replace(/\s+/g, ' ')
+    .trim();
+}
+
+function tokenize(text) {
+  return normalizeText(text)
+    .split(' ')
+    .filter(Boolean)
+    .filter((token) => !STOP_WORDS.has(token));
+}
+
+function uniqueBy(items, keyFn) {
+  const seen = new Set();
+  const result = [];
+  for (const item of items) {
+    const key = keyFn(item);
+    if (seen.has(key)) {
+      continue;
+    }
+    seen.add(key);
+    result.push(item);
+  }
+  return result;
+}
+
+function similarityScore(left, right) {
+  const leftTokens = new Set(tokenize(left));
+  const rightTokens = new Set(tokenize(right));
+  if (leftTokens.size === 0 || rightTokens.size === 0) {
+    return 0;
+  }
+
+  let shared = 0;
+  for (const token of leftTokens) {
+    if (rightTokens.has(token)) {
+      shared += 1;
+    }
+  }
+
+  const union = new Set([...leftTokens, ...rightTokens]);
+  return shared / union.size;
+}
+
+function ensureTrailingNewline(text) {
+  return text.endsWith('\n') ? text : `${text}\n`;
+}
+
+function escapeRegExp(value) {
+  return String(value).replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+}
+
+function parseArgv(argv) {
+  const flags = {};
+  const positionals = [];
+
+  for (let i = 0; i < argv.length; i += 1) {
+    const current = argv[i];
+    if (!current.startsWith('-')) {
+      positionals.push(current);
+      continue;
+    }
+
+    if (current.startsWith('--')) {
+      const withoutPrefix = current.slice(2);
+      const eqIndex = withoutPrefix.indexOf('=');
+      let key;
+      let value;
+
+      if (eqIndex >= 0) {
+        key = withoutPrefix.slice(0, eqIndex);
+        value = withoutPrefix.slice(eqIndex + 1);
+      } else {
+        key = withoutPrefix;
+        const next = argv[i + 1];
+        if (next && !next.startsWith('-')) {
+          value = next;
+          i += 1;
+        } else {
+          value = true;
+        }
+      }
+
+      if (Object.prototype.hasOwnProperty.call(flags, key)) {
+        if (Array.isArray(flags[key])) {
+          flags[key].push(value);
+        } else {
+          flags[key] = [flags[key], value];
+        }
+      } else {
+        flags[key] = value;
+      }
+      continue;
+    }
+
+    const short = current.slice(1);
+    flags[short] = true;
+  }
+
+  const command = positionals.length > 0 ? positionals[0] : null;
+  return {
+    command,
+    args: positionals.slice(1),
+    flags,
+    positionals
+  };
+}
+
+module.exports = {
+  escapeRegExp,
+  ensureTrailingNewline,
+  parseArgv,
+  similarityScore,
+  slugify,
+  toPosix,
+  tokenize,
+  uniqueBy
 };

package/src/validator/index.js

@@ -56,6 +56,24 @@ function readFileIndex(projectRoot, files) {
   return index;
 }
 
+const KNOWN_PATH_ROOTS = [
+  'src/', 'lib/', 'bin/', 'test/', 'tests/', 'docs/', 'scripts/',
+  'packages/', 'apps/', 'tools/', '.github/', 'roadmap-skill/'
+];
+
+function hasFileExtension(token) {
+  const lastSegment = token.replace(/\\/g, '/').split('/').pop() || '';
+  return /\.[A-Za-z0-9]{1,10}$/.test(lastSegment);
+}
+
+function isLikelyPath(token) {
+  if (/^\.{1,2}\/|^\//.test(token)) return true;
+  if (hasFileExtension(token)) return true;
+  if (KNOWN_PATH_ROOTS.some((root) => token.startsWith(root))) return true;
+  if ((token.match(/\//g) || []).length >= 2) return true;
+  return false;
+}
+
 function extractExplicitPaths(text) {
   const results = new Set();
   const quoted = String(text).match(/`([^`]+)`/g) || [];
@@ -67,8 +85,9 @@ function extractExplicitPaths(text) {
   }
 
   const pathTokens = String(text).match(/([A-Za-z0-9_.-]+\/[A-Za-z0-9_./-]+)/g) || [];
-  for (const token of pathTokens) {
-    results.add(token);
+  for (const raw of pathTokens) {
+    const token = raw.replace(/[.,;:!?)]+$/, '');
+    if (isLikelyPath(token)) results.add(token);
   }
 
   return Array.from(results).sort((left, right) => left.localeCompare(right));
@@ -398,4 +417,4 @@ module.exports = {
   buildValidationContext,
   validateTask,
   validateTasks
-};
+};