roadmapsmith 0.4.0 → 0.5.0

package/src/generator/index.js

@@ -2,7 +2,7 @@
 
 const fs = require('fs');
 const path = require('path');
-const { walkFiles, detectLanguages, detectTestFrameworks } = require('../io');
+const { walkFiles, detectLanguages, detectTestFrameworks, detectWorkspaces } = require('../io');
 const { createRoadmapModel, PHASE_ORDER } = require('../model');
 const { slugify, ensureTrailingNewline } = require('../utils');
 const { parseRoadmap, upsertManagedBlock } = require('../parser');
@@ -10,21 +10,52 @@ const { findBestTaskMatch, dedupeTasks } = require('../match');
 const { collectPluginContributions } = require('../config');
 const { renderBody } = require('../renderer');
 
+const IMPL_PATTERN_RE = /[/|]TODO|TODO[|/]|[/|]FIXME|FIXME[|/]/;
+const COMMENT_TODO_RE = /(?:\/\/|#|\*\s*).*\b(?:TODO|FIXME)\b/;
+
+function isTodoMarker(line) {
+  return COMMENT_TODO_RE.test(line) && !IMPL_PATTERN_RE.test(line);
+}
+
+const GENERIC_MODULE_NAMES = new Set(['index', 'main', 'utils', 'common', 'helpers', 'types', 'constants', 'model']);
+
 function detectModules(files) {
   const modules = new Set();
-  const roots = ['src/', 'apps/', 'packages/', 'lib/', 'cmd/', 'internal/'];
+  const rootPrefixes = ['src/', 'apps/', 'packages/', 'lib/', 'cmd/', 'internal/'];
 
   for (const file of files) {
-    const root = roots.find((candidate) => file.startsWith(candidate));
-    if (!root) {
-      continue;
+    let relative;
+
+    const directRoot = rootPrefixes.find((r) => file.startsWith(r));
+    if (directRoot) {
+      relative = file.slice(directRoot.length);
+    } else {
+      let found = false;
+      for (const r of rootPrefixes) {
+        const idx = file.indexOf('/' + r);
+        if (idx !== -1) {
+          // Only accept nested prefix when it appears within the first two path segments
+          // (e.g. "wrapper/src/..." is fine; "a/b/c/src/..." is too deep and likely a fixture or dependency)
+          if (file.slice(0, idx).split('/').length > 2) continue;
+          relative = file.slice(idx + 1 + r.length);
+          found = true;
+          break;
+        }
+      }
+      if (!found) continue;
     }
-    const relative = file.slice(root.length);
+
     const first = relative.split('/')[0];
-    if (!first || first.includes('.')) {
-      continue;
+    if (!first) continue;
+
+    if (first.includes('.')) {
+      const name = first.slice(0, first.lastIndexOf('.'));
+      if (name && !GENERIC_MODULE_NAMES.has(name)) {
+        modules.add(name);
+      }
+    } else {
+      modules.add(first);
     }
-    modules.add(first);
   }
 
   return Array.from(modules).sort((left, right) => left.localeCompare(right));
@@ -58,7 +89,7 @@ function collectTodoHints(projectRoot, files) {
 
     const lines = content.split(/\r?\n/);
     for (let i = 0; i < lines.length; i += 1) {
-      if (/TODO|FIXME/i.test(lines[i])) {
+      if (isTodoMarker(lines[i])) {
         hints.push({
           file,
           line: i + 1,
@@ -89,7 +120,7 @@ function collectCodeTodoHints(projectRoot, files) {
 
     const lines = content.split(/\r?\n/);
     for (let i = 0; i < lines.length; i += 1) {
-      if (/TODO|FIXME/i.test(lines[i])) {
+      if (isTodoMarker(lines[i])) {
         hints.push({
           file,
           line: i + 1,
@@ -113,6 +144,7 @@ function scanProject(projectRoot) {
   const commands = detectCommands(files);
   const todos = collectTodoHints(projectRoot, files);
   const codeTodos = collectCodeTodoHints(projectRoot, files);
+  const workspaces = detectWorkspaces(projectRoot, files);
 
   const implementedFiles = files.filter((file) => /\.(js|ts|tsx|py|go|rs|java|kt|cs)$/.test(file));
   const testFiles = files.filter((file) => /(^|\/)(__tests__|tests)\//.test(file) || /\.test\.|\.spec\.|_test\.go$/.test(file));
@@ -126,6 +158,7 @@ function scanProject(projectRoot) {
     commands,
     todos,
     codeTodos,
+    workspaces,
     implementedCount: implementedFiles.length,
     testCount: testFiles.length
   };
@@ -272,97 +305,6 @@ function groupByPhase(tasks) {
   return groups;
 }
 
-function taskLine(task) {
-  return `- [${task.checked ? 'x' : ' '}] ${task.text} <!-- rs:task=${task.id} -->`;
-}
-
-function checkedState(model, id) {
-  return Boolean(model.checkedById && model.checkedById[id]);
-}
-
-function renderManagedBody(model) {
-  const lines = [];
-
-  lines.push('# Project Roadmap');
-  lines.push('');
-  lines.push('## Product North Star');
-  lines.push(model.northStar);
-  lines.push('');
-
-  lines.push('## Current State');
-  lines.push(`- Implemented surface: ${model.currentState.implementedSummary}`);
-  lines.push(`- TODO surface: ${model.currentState.todoSummary}`);
-  lines.push(`- Detected stacks: ${model.currentState.stackSummary}`);
-  lines.push('');
-
-  lines.push('## Phased Roadmap');
-  lines.push('');
-  lines.push('### Phase P0 (Critical)');
-  for (const task of model.phases.P0) {
-    lines.push(taskLine(task));
-  }
-  lines.push('');
-  lines.push('### Phase P1 (Important)');
-  for (const task of model.phases.P1) {
-    lines.push(taskLine(task));
-  }
-  lines.push('');
-  lines.push('### Phase P2 (Optimization)');
-  for (const task of model.phases.P2) {
-    lines.push(taskLine(task));
-  }
-  lines.push('');
-
-  lines.push('## Release Milestones');
-  for (const milestone of model.milestones) {
-    const id = `milestone-${slugify(milestone.version)}`;
-    lines.push(`- [${checkedState(model, id) ? 'x' : ' '}] ${milestone.version}: ${milestone.goal} <!-- rs:task=${id} -->`);
-  }
-  lines.push('');
-
-  lines.push('## Command/Module Breakdown');
-  if (model.commandBreakdown.length === 0) {
-    const id = 'identify-command-module-boundaries';
-    lines.push(`- [${checkedState(model, id) ? 'x' : ' '}] Identify command/module boundaries for the next increment <!-- rs:task=${id} -->`);
-  } else {
-    for (const item of model.commandBreakdown) {
-      const id = `module-${slugify(item)}`;
-      lines.push(`- [${checkedState(model, id) ? 'x' : ' '}] ${item} <!-- rs:task=${id} -->`);
-    }
-  }
-  lines.push('');
-
-  lines.push('## Exit Criteria Per Phase');
-  for (const item of model.exitCriteria) {
-    const id = `exit-${slugify(item)}`;
-    lines.push(`- [${checkedState(model, id) ? 'x' : ' '}] ${item} <!-- rs:task=${id} -->`);
-  }
-  lines.push('');
-
-  for (const section of model.customSections) {
-    lines.push(`## ${section.title}`);
-    for (const line of section.items) {
-      lines.push(line);
-    }
-    lines.push('');
-  }
-
-  lines.push('## Risks and Anti-goals');
-  lines.push('### Risks');
-  for (const risk of model.risks) {
-    const id = `risk-${slugify(risk)}`;
-    lines.push(`- [${checkedState(model, id) ? 'x' : ' '}] ${risk} <!-- rs:task=${id} -->`);
-  }
-  lines.push('');
-  lines.push('### Anti-goals');
-  for (const antiGoal of model.antiGoals) {
-    const id = `anti-goal-${slugify(antiGoal)}`;
-    lines.push(`- [${checkedState(model, id) ? 'x' : ' '}] ${antiGoal} <!-- rs:task=${id} -->`);
-  }
-
-  return ensureTrailingNewline(lines.join('\n')).trimEnd();
-}
-
 function inferProjectName(projectRoot) {
   const pkgPath = path.join(projectRoot, 'package.json');
   try {
@@ -488,6 +430,7 @@ function createModel(scan, tasks, config, customSections, checkedById) {
     implemented,
     scaffold,
     knownLimitations,
+    workspaces: scan.workspaces || [],
     implementedSummary: `${scan.implementedCount} implementation files detected`,
     todoSummary: `${scan.todos.length} TODO/FIXME markers detected`,
     stackSummary: scan.languages.length > 0 ? scan.languages.join(', ') : 'No language-specific stack detected'
@@ -615,6 +558,5 @@ function generateRoadmapDocument(options) {
 
 module.exports = {
   generateRoadmapDocument,
-  renderManagedBody,
   scanProject
 };

package/src/index.js

@@ -1,8 +1,11 @@
-'use strict';
-
-module.exports = {
-  generator: require('./generator'),
-  parser: require('./parser'),
-  sync: require('./sync'),
-  validator: require('./validator')
+'use strict';
+
+module.exports = {
+  generator: require('./generator'),
+  parser: require('./parser'),
+  sync: require('./sync'),
+  validator: require('./validator'),
+  config: require('./config'),
+  model: require('./model'),
+  renderer: require('./renderer')
 };