roadmapsmith 0.4.0 → 0.5.1

package/src/io.js

@@ -1,228 +1,264 @@
-'use strict';
-
-const fs = require('fs');
-const path = require('path');
-const { ensureTrailingNewline, toPosix } = require('./utils');
-
-const DEFAULT_IGNORED_DIRS = new Set([
-  '.git',
-  '.idea',
-  '.vscode',
-  '.next',
-  '.nuxt',
-  '.turbo',
-  '.cache',
-  'dist',
-  'build',
-  'coverage',
-  'target',
-  'node_modules',
-  '.venv',
-  'venv',
-  '__pycache__',
-  '.pytest_cache',
-  '.mypy_cache',
-  '.ruff_cache'
-]);
-
-function readTextIfExists(filePath) {
-  try {
-    return fs.readFileSync(filePath, 'utf8');
-  } catch {
-    return null;
-  }
-}
-
-function ensureDirForFile(filePath) {
-  const dir = path.dirname(filePath);
-  fs.mkdirSync(dir, { recursive: true });
-}
-
-function writeText(filePath, content, options = {}) {
-  const next = ensureTrailingNewline(content);
-  const before = readTextIfExists(filePath);
-  const changed = before == null || before !== next;
-
-  if (!changed) {
-    return {
-      changed: false,
-      before,
-      after: next,
-      path: filePath
-    };
-  }
-
-  if (!options.dryRun) {
-    ensureDirForFile(filePath);
-    fs.writeFileSync(filePath, next, 'utf8');
-  }
-
-  return {
-    changed: true,
-    before,
-    after: next,
-    path: filePath
-  };
-}
-
-function walkFiles(rootPath, options = {}) {
-  const ignoredDirs = options.ignoredDirs || DEFAULT_IGNORED_DIRS;
-  const result = [];
-
-  function walk(current) {
-    const entries = fs.readdirSync(current, { withFileTypes: true });
-    entries.sort((left, right) => left.name.localeCompare(right.name));
-
-    for (const entry of entries) {
-      const absolutePath = path.join(current, entry.name);
-      const relativePath = toPosix(path.relative(rootPath, absolutePath));
-      if (entry.isDirectory()) {
-        if (ignoredDirs.has(entry.name)) {
-          continue;
-        }
-        walk(absolutePath);
-        continue;
-      }
-      result.push(relativePath);
-    }
-  }
-
-  walk(rootPath);
-  return result;
-}
-
-function parseJsonIfExists(filePath) {
-  const content = readTextIfExists(filePath);
-  if (!content) {
-    return null;
-  }
-  try {
-    return JSON.parse(content);
-  } catch {
-    return null;
-  }
-}
-
-function detectLanguages(files) {
-  const languageByExtension = {
-    '.js': 'JavaScript',
-    '.cjs': 'JavaScript',
-    '.mjs': 'JavaScript',
-    '.ts': 'TypeScript',
-    '.tsx': 'TypeScript',
-    '.jsx': 'JavaScript',
-    '.py': 'Python',
-    '.go': 'Go',
-    '.rs': 'Rust',
-    '.java': 'Java',
-    '.kt': 'Kotlin',
-    '.swift': 'Swift',
-    '.rb': 'Ruby',
-    '.php': 'PHP',
-    '.cs': 'C#',
-    '.cpp': 'C++',
-    '.c': 'C',
-    '.h': 'C',
-    '.sh': 'Shell'
-  };
-
-  const languages = new Set();
-  for (const file of files) {
-    const ext = path.extname(file).toLowerCase();
-    if (languageByExtension[ext]) {
-      languages.add(languageByExtension[ext]);
-    }
-  }
-  return Array.from(languages).sort((left, right) => left.localeCompare(right));
-}
-
-function detectTestFrameworks(projectRoot, files) {
-  const frameworks = new Set();
-
-  const packageJson = parseJsonIfExists(path.join(projectRoot, 'package.json'));
-  if (packageJson) {
-    const scripts = packageJson.scripts || {};
-    if (scripts.test) {
-      frameworks.add('node-test-script');
-    }
-    const deps = {
-      ...(packageJson.dependencies || {}),
-      ...(packageJson.devDependencies || {})
-    };
-    if (deps.jest) frameworks.add('jest');
-    if (deps.vitest) frameworks.add('vitest');
-    if (deps.mocha) frameworks.add('mocha');
-    if (deps.ava) frameworks.add('ava');
-    if (deps.tap) frameworks.add('tap');
-  }
-
-  if (files.some((file) => file.endsWith('pyproject.toml')) || files.some((file) => file.endsWith('pytest.ini'))) {
-    frameworks.add('pytest');
-  }
-  if (files.some((file) => /(^|\/)test_.*\.py$/.test(file)) || files.some((file) => /(^|\/)tests\//.test(file))) {
-    frameworks.add('python-tests');
-  }
-  if (files.some((file) => file.endsWith('go.mod'))) {
-    frameworks.add('go');
-  }
-  if (files.some((file) => file.endsWith('_test.go'))) {
-    frameworks.add('go-test');
-  }
-  if (files.some((file) => file.endsWith('Cargo.toml'))) {
-    frameworks.add('rust');
-  }
-  if (files.some((file) => /(^|\/)tests\//.test(file) && file.endsWith('.rs'))) {
-    frameworks.add('cargo-test');
-  }
-
-  if (files.some((file) => /(^|\/)(__tests__|tests)\//.test(file) || /\.test\.|\.spec\./.test(file))) {
-    frameworks.add('generic-tests');
-  }
-
-  return Array.from(frameworks).sort((left, right) => left.localeCompare(right));
-}
-
-function lineDiff(before, after) {
-  const left = (before || '').split(/\r?\n/);
-  const right = (after || '').split(/\r?\n/);
-  const max = Math.max(left.length, right.length);
-  const changes = [];
-
-  for (let i = 0; i < max; i += 1) {
-    const oldLine = left[i] == null ? '' : left[i];
-    const newLine = right[i] == null ? '' : right[i];
-    if (oldLine !== newLine) {
-      changes.push({ index: i + 1, oldLine, newLine });
-    }
-    if (changes.length >= 20) {
-      break;
-    }
-  }
-
-  return changes;
-}
-
-function printDryRunDiff(filePath, before, after) {
-  const changes = lineDiff(before, after);
-  console.log(`Dry run: ${filePath}`);
-  if (changes.length === 0) {
-    console.log('- no line changes');
-    return;
-  }
-
-  for (const change of changes) {
-    console.log(`L${change.index} - ${change.oldLine}`);
-    console.log(`L${change.index} + ${change.newLine}`);
-  }
-}
-
-module.exports = {
-  DEFAULT_IGNORED_DIRS,
-  detectLanguages,
-  detectTestFrameworks,
-  parseJsonIfExists,
-  printDryRunDiff,
-  readTextIfExists,
-  walkFiles,
-  writeText
+'use strict';
+
+const fs = require('fs');
+const path = require('path');
+const { ensureTrailingNewline, toPosix } = require('./utils');
+
+const DEFAULT_IGNORED_DIRS = new Set([
+  '.git',
+  '.idea',
+  '.vscode',
+  '.next',
+  '.nuxt',
+  '.turbo',
+  '.cache',
+  'dist',
+  'build',
+  'coverage',
+  'target',
+  'node_modules',
+  '.venv',
+  'venv',
+  '__pycache__',
+  '.pytest_cache',
+  '.mypy_cache',
+  '.ruff_cache'
+]);
+
+function readTextIfExists(filePath) {
+  try {
+    return fs.readFileSync(filePath, 'utf8');
+  } catch {
+    return null;
+  }
+}
+
+function ensureDirForFile(filePath) {
+  const dir = path.dirname(filePath);
+  fs.mkdirSync(dir, { recursive: true });
+}
+
+function writeText(filePath, content, options = {}) {
+  const next = ensureTrailingNewline(content);
+  const before = readTextIfExists(filePath);
+  const changed = before == null || before !== next;
+
+  if (!changed) {
+    return {
+      changed: false,
+      before,
+      after: next,
+      path: filePath
+    };
+  }
+
+  if (!options.dryRun) {
+    ensureDirForFile(filePath);
+    fs.writeFileSync(filePath, next, 'utf8');
+  }
+
+  return {
+    changed: true,
+    before,
+    after: next,
+    path: filePath
+  };
+}
+
+function walkFiles(rootPath, options = {}) {
+  const ignoredDirs = options.ignoredDirs || DEFAULT_IGNORED_DIRS;
+  const result = [];
+
+  function walk(current) {
+    const entries = fs.readdirSync(current, { withFileTypes: true });
+    entries.sort((left, right) => left.name.localeCompare(right.name));
+
+    for (const entry of entries) {
+      const absolutePath = path.join(current, entry.name);
+      const relativePath = toPosix(path.relative(rootPath, absolutePath));
+      if (entry.isDirectory()) {
+        if (ignoredDirs.has(entry.name)) {
+          continue;
+        }
+        walk(absolutePath);
+        continue;
+      }
+      result.push(relativePath);
+    }
+  }
+
+  walk(rootPath);
+  return result;
+}
+
+function parseJsonIfExists(filePath) {
+  const content = readTextIfExists(filePath);
+  if (!content) {
+    return null;
+  }
+  try {
+    return JSON.parse(content);
+  } catch {
+    return null;
+  }
+}
+
+function detectLanguages(files) {
+  const languageByExtension = {
+    '.js': 'JavaScript',
+    '.cjs': 'JavaScript',
+    '.mjs': 'JavaScript',
+    '.ts': 'TypeScript',
+    '.tsx': 'TypeScript',
+    '.jsx': 'JavaScript',
+    '.py': 'Python',
+    '.go': 'Go',
+    '.rs': 'Rust',
+    '.java': 'Java',
+    '.kt': 'Kotlin',
+    '.swift': 'Swift',
+    '.rb': 'Ruby',
+    '.php': 'PHP',
+    '.cs': 'C#',
+    '.cpp': 'C++',
+    '.c': 'C',
+    '.h': 'C',
+    '.sh': 'Shell'
+  };
+
+  const languages = new Set();
+  for (const file of files) {
+    const ext = path.extname(file).toLowerCase();
+    if (languageByExtension[ext]) {
+      languages.add(languageByExtension[ext]);
+    }
+  }
+  return Array.from(languages).sort((left, right) => left.localeCompare(right));
+}
+
+function detectTestFrameworks(projectRoot, files) {
+  const frameworks = new Set();
+
+  const packageJson = parseJsonIfExists(path.join(projectRoot, 'package.json'));
+  if (packageJson) {
+    const scripts = packageJson.scripts || {};
+    if (scripts.test) {
+      frameworks.add('node-test-script');
+    }
+    const deps = {
+      ...(packageJson.dependencies || {}),
+      ...(packageJson.devDependencies || {})
+    };
+    if (deps.jest) frameworks.add('jest');
+    if (deps.vitest) frameworks.add('vitest');
+    if (deps.mocha) frameworks.add('mocha');
+    if (deps.ava) frameworks.add('ava');
+    if (deps.tap) frameworks.add('tap');
+  }
+
+  if (files.some((file) => file.endsWith('pyproject.toml')) || files.some((file) => file.endsWith('pytest.ini'))) {
+    frameworks.add('pytest');
+  }
+  if (files.some((file) => /(^|\/)test_.*\.py$/.test(file)) || files.some((file) => /(^|\/)tests\//.test(file))) {
+    frameworks.add('python-tests');
+  }
+  if (files.some((file) => file.endsWith('go.mod'))) {
+    frameworks.add('go');
+  }
+  if (files.some((file) => file.endsWith('_test.go'))) {
+    frameworks.add('go-test');
+  }
+  if (files.some((file) => file.endsWith('Cargo.toml'))) {
+    frameworks.add('rust');
+  }
+  if (files.some((file) => /(^|\/)tests\//.test(file) && file.endsWith('.rs'))) {
+    frameworks.add('cargo-test');
+  }
+
+  if (files.some((file) => /(^|\/)(__tests__|tests)\//.test(file) || /\.test\.|\.spec\./.test(file))) {
+    frameworks.add('generic-tests');
+  }
+
+  return Array.from(frameworks).sort((left, right) => left.localeCompare(right));
+}
+
+// Detects workspace package roots (one level deep) from package.json workspaces field
+// and physical scan of packages/, apps/, tools/. Supports prefix/* and prefix/** globs.
+function detectWorkspaces(projectRoot, files) {
+  const packages = new Set();
+  const workspacePrefixes = ['packages/', 'apps/', 'tools/'];
+
+  for (const file of files) {
+    if (!file.endsWith('/package.json')) continue;
+    for (const prefix of workspacePrefixes) {
+      if (!file.startsWith(prefix)) continue;
+      const rest = file.slice(prefix.length);
+      const segments = rest.split('/');
+      if (segments.length === 2 && segments[1] === 'package.json') {
+        packages.add(prefix + segments[0]);
+      }
+    }
+  }
+
+  const pkg = parseJsonIfExists(path.join(projectRoot, 'package.json'));
+  const globs = Array.isArray(pkg && pkg.workspaces) ? pkg.workspaces
+    : (pkg && pkg.workspaces && Array.isArray(pkg.workspaces.packages)) ? pkg.workspaces.packages
+    : [];
+  for (const glob of globs) {
+    const match = glob.match(/^([A-Za-z0-9_.-]+)\/\*{1,2}$/);
+    if (!match) continue;
+    const prefix = match[1] + '/';
+    for (const file of files) {
+      if (!file.startsWith(prefix) || !file.endsWith('/package.json')) continue;
+      const rest = file.slice(prefix.length);
+      const segments = rest.split('/');
+      if (segments.length === 2) packages.add(prefix + segments[0]);
+    }
+  }
+
+  return Array.from(packages).sort((a, b) => a.localeCompare(b));
+}
+
+function lineDiff(before, after) {
+  const left = (before || '').split(/\r?\n/);
+  const right = (after || '').split(/\r?\n/);
+  const max = Math.max(left.length, right.length);
+  const changes = [];
+
+  for (let i = 0; i < max; i += 1) {
+    const oldLine = left[i] == null ? '' : left[i];
+    const newLine = right[i] == null ? '' : right[i];
+    if (oldLine !== newLine) {
+      changes.push({ index: i + 1, oldLine, newLine });
+    }
+    if (changes.length >= 20) {
+      break;
+    }
+  }
+
+  return changes;
+}
+
+function printDryRunDiff(filePath, before, after) {
+  const changes = lineDiff(before, after);
+  console.log(`Dry run: ${filePath}`);
+  if (changes.length === 0) {
+    console.log('- no line changes');
+    return;
+  }
+
+  for (const change of changes) {
+    console.log(`L${change.index} - ${change.oldLine}`);
+    console.log(`L${change.index} + ${change.newLine}`);
+  }
+}
+
+module.exports = {
+  detectLanguages,
+  detectTestFrameworks,
+  detectWorkspaces,
+  printDryRunDiff,
+  readTextIfExists,
+  walkFiles,
+  writeText
 };

package/src/match.js

@@ -1,86 +1,86 @@
-'use strict';
-
-const { similarityScore, slugify, tokenize, uniqueBy } = require('./utils');
-const { PHASE_ORDER } = require('./model');
-
-function canonicalSignature(text) {
-  const tokens = uniqueBy(tokenize(text), (token) => token).slice(0, 8);
-  return tokens.join('-') || slugify(text);
-}
-
-function inferPriorityWeight(priority) {
-  const normalized = String(priority || '').toUpperCase();
-  if (normalized === 'P0') return 0;
-  if (normalized === 'P1') return 1;
-  if (normalized === 'P2') return 2;
-  return 3;
-}
-
-function findBestTaskMatch(candidate, existingTasks, minScore = 0.55) {
-  const direct = existingTasks.find((task) => task.id === candidate.id);
-  if (direct) {
-    return { task: direct, score: 1 };
-  }
-
-  let best = null;
-  for (const task of existingTasks) {
-    const score = similarityScore(candidate.text, task.text);
-    if (score < minScore) {
-      continue;
-    }
-    if (!best || score > best.score) {
-      best = { task, score };
-    }
-  }
-
-  return best;
-}
-
-function dedupeTasks(tasks) {
-  const grouped = new Map();
-
-  for (const task of tasks) {
-    const signature = canonicalSignature(task.text);
-    if (!grouped.has(signature)) {
-      grouped.set(signature, task);
-      continue;
-    }
-
-    const current = grouped.get(signature);
-    const candidate = task;
-
-    if (current.checked !== candidate.checked) {
-      grouped.set(signature, candidate.checked ? candidate : current);
-      continue;
-    }
-
-    const currentWeight = inferPriorityWeight(current.priority);
-    const candidateWeight = inferPriorityWeight(candidate.priority);
-    if (candidateWeight < currentWeight) {
-      grouped.set(signature, candidate);
-      continue;
-    }
-
-    if (candidateWeight === currentWeight && candidate.text.length < current.text.length) {
-      grouped.set(signature, candidate);
-    }
-  }
-
-  return Array.from(grouped.values()).sort((left, right) => {
-    const leftPhase = PHASE_ORDER.indexOf(left.phase);
-    const rightPhase = PHASE_ORDER.indexOf(right.phase);
-    if (leftPhase !== rightPhase) {
-      return leftPhase - rightPhase;
-    }
-    if (left.priority !== right.priority) {
-      return inferPriorityWeight(left.priority) - inferPriorityWeight(right.priority);
-    }
-    return left.text.localeCompare(right.text);
-  });
-}
-
-module.exports = {
-  canonicalSignature,
-  dedupeTasks,
-  findBestTaskMatch
+'use strict';
+
+const { similarityScore, slugify, tokenize, uniqueBy } = require('./utils');
+const { PHASE_ORDER } = require('./model');
+
+function canonicalSignature(text) {
+  const tokens = uniqueBy(tokenize(text), (token) => token).slice(0, 8);
+  return tokens.join('-') || slugify(text);
+}
+
+function inferPriorityWeight(priority) {
+  const normalized = String(priority || '').toUpperCase();
+  if (normalized === 'P0') return 0;
+  if (normalized === 'P1') return 1;
+  if (normalized === 'P2') return 2;
+  if (normalized === 'P3') return 3;
+  return 4;
+}
+
+function findBestTaskMatch(candidate, existingTasks, minScore = 0.55) {
+  const direct = existingTasks.find((task) => task.id === candidate.id);
+  if (direct) {
+    return { task: direct, score: 1 };
+  }
+
+  let best = null;
+  for (const task of existingTasks) {
+    const score = similarityScore(candidate.text, task.text);
+    if (score < minScore) {
+      continue;
+    }
+    if (!best || score > best.score) {
+      best = { task, score };
+    }
+  }
+
+  return best;
+}
+
+function dedupeTasks(tasks) {
+  const grouped = new Map();
+
+  for (const task of tasks) {
+    const signature = canonicalSignature(task.text);
+    if (!grouped.has(signature)) {
+      grouped.set(signature, task);
+      continue;
+    }
+
+    const current = grouped.get(signature);
+    const candidate = task;
+
+    if (current.checked !== candidate.checked) {
+      grouped.set(signature, candidate.checked ? candidate : current);
+      continue;
+    }
+
+    const currentWeight = inferPriorityWeight(current.priority);
+    const candidateWeight = inferPriorityWeight(candidate.priority);
+    if (candidateWeight < currentWeight) {
+      grouped.set(signature, candidate);
+      continue;
+    }
+
+    if (candidateWeight === currentWeight && candidate.text.length < current.text.length) {
+      grouped.set(signature, candidate);
+    }
+  }
+
+  return Array.from(grouped.values()).sort((left, right) => {
+    const leftPhase = PHASE_ORDER.indexOf(left.phase);
+    const rightPhase = PHASE_ORDER.indexOf(right.phase);
+    if (leftPhase !== rightPhase) {
+      return leftPhase - rightPhase;
+    }
+    if (left.priority !== right.priority) {
+      return inferPriorityWeight(left.priority) - inferPriorityWeight(right.priority);
+    }
+    return left.text.localeCompare(right.text);
+  });
+}
+
+module.exports = {
+  dedupeTasks,
+  findBestTaskMatch
 };