dw-kit 1.9.0 → 1.9.2

package/src/lib/prompt-suggest.mjs

@@ -1,84 +1,84 @@
-import { execSync } from 'node:child_process';
-
-const TEMPLATE_SUGGESTIONS = [
-  'fix: ',
-  'fix authentication redirect after login',
-  'fix null pointer / undefined error in ',
-  'fix performance issue in ',
-  'feat: add ',
-  'feat: implement ',
-  'feat: support ',
-  'refactor: simplify ',
-  'refactor: extract ',
-  'refactor: rename ',
-  'perf: optimize ',
-  'perf: reduce load time of ',
-  'chore: update dependencies',
-  'docs: update README',
-  'test: add tests for ',
-];
-
-// Common verbs to detect whether a description has intent
-const INTENT_VERBS = [
-  'fix', 'add', 'update', 'remove', 'delete', 'create', 'implement', 'refactor',
-  'rename', 'move', 'improve', 'optimize', 'support', 'extract', 'migrate',
-  'replace', 'upgrade', 'enable', 'disable', 'integrate', 'patch',
-];
-
-function getTemplateSuggestions() {
-  return [...TEMPLATE_SUGGESTIONS];
-}
-
-export function getGitSuggestions(cwd) {
-  try {
-    const out = execSync('git log --oneline -50 --no-merges', {
-      cwd,
-      encoding: 'utf-8',
-      timeout: 3000,
-      stdio: ['ignore', 'pipe', 'ignore'],
-    });
-    return out
-      .trim()
-      .split('\n')
-      .filter(Boolean)
-      .map((line) => line.replace(/^[a-f0-9]+ /, '').trim())  // strip hash
-      .filter((msg) => msg.length > 5)
-      .slice(0, 30);
-  } catch {
-    return [];
-  }
-}
-
-export function getSuggestions(cwd = process.cwd()) {
-  const git = getGitSuggestions(cwd);
-  const templates = getTemplateSuggestions();
-  // git log first (most relevant to this repo), then templates
-  const merged = [...git, ...templates];
-  // dedupe
-  return [...new Set(merged)].slice(0, 20);
-}
-
-/**
- * Returns true if the description is likely too vague to give Claude good context.
- */
-export function isVague(text) {
-  const trimmed = (text || '').trim();
-  if (trimmed.length < 50) return true;
-  const lower = trimmed.toLowerCase();
-  const hasVerb = INTENT_VERBS.some((v) => lower.startsWith(v) || lower.includes(` ${v} `));
-  return !hasVerb;
-}
-
-/**
- * Expand a short description into a structured prompt.
- * @param {string} text - core task description
- * @param {{ area?: string, outcome?: string }} extras
- * @returns {string}
- */
-export function expandTemplate(text, { area = '', outcome = '' } = {}) {
-  const base = text.trim();
-  const parts = [base];
-  if (area) parts.push(`Scope: ${area.trim()}.`);
-  if (outcome) parts.push(`Expected: ${outcome.trim()}.`);
-  return parts.join('\n');
-}
+import { execSync } from 'node:child_process';
+
+const TEMPLATE_SUGGESTIONS = [
+  'fix: ',
+  'fix authentication redirect after login',
+  'fix null pointer / undefined error in ',
+  'fix performance issue in ',
+  'feat: add ',
+  'feat: implement ',
+  'feat: support ',
+  'refactor: simplify ',
+  'refactor: extract ',
+  'refactor: rename ',
+  'perf: optimize ',
+  'perf: reduce load time of ',
+  'chore: update dependencies',
+  'docs: update README',
+  'test: add tests for ',
+];
+
+// Common verbs to detect whether a description has intent
+const INTENT_VERBS = [
+  'fix', 'add', 'update', 'remove', 'delete', 'create', 'implement', 'refactor',
+  'rename', 'move', 'improve', 'optimize', 'support', 'extract', 'migrate',
+  'replace', 'upgrade', 'enable', 'disable', 'integrate', 'patch',
+];
+
+function getTemplateSuggestions() {
+  return [...TEMPLATE_SUGGESTIONS];
+}
+
+export function getGitSuggestions(cwd) {
+  try {
+    const out = execSync('git log --oneline -50 --no-merges', {
+      cwd,
+      encoding: 'utf-8',
+      timeout: 3000,
+      stdio: ['ignore', 'pipe', 'ignore'],
+    });
+    return out
+      .trim()
+      .split('\n')
+      .filter(Boolean)
+      .map((line) => line.replace(/^[a-f0-9]+ /, '').trim())  // strip hash
+      .filter((msg) => msg.length > 5)
+      .slice(0, 30);
+  } catch {
+    return [];
+  }
+}
+
+export function getSuggestions(cwd = process.cwd()) {
+  const git = getGitSuggestions(cwd);
+  const templates = getTemplateSuggestions();
+  // git log first (most relevant to this repo), then templates
+  const merged = [...git, ...templates];
+  // dedupe
+  return [...new Set(merged)].slice(0, 20);
+}
+
+/**
+ * Returns true if the description is likely too vague to give Claude good context.
+ */
+export function isVague(text) {
+  const trimmed = (text || '').trim();
+  if (trimmed.length < 50) return true;
+  const lower = trimmed.toLowerCase();
+  const hasVerb = INTENT_VERBS.some((v) => lower.startsWith(v) || lower.includes(` ${v} `));
+  return !hasVerb;
+}
+
+/**
+ * Expand a short description into a structured prompt.
+ * @param {string} text - core task description
+ * @param {{ area?: string, outcome?: string }} extras
+ * @returns {string}
+ */
+export function expandTemplate(text, { area = '', outcome = '' } = {}) {
+  const base = text.trim();
+  const parts = [base];
+  if (area) parts.push(`Scope: ${area.trim()}.`);
+  if (outcome) parts.push(`Expected: ${outcome.trim()}.`);
+  return parts.join('\n');
+}

package/src/lib/timeline-parser.mjs

@@ -29,29 +29,68 @@ export function parseTimeline(content) {
   };
 }
 
+// Header-driven so the column ORDER is not load-bearing (#24): the §3 tracker
+// may be `# | Subtask | Status | Date | Notes` (legacy) or
+// `# | Subtask | Acceptance | Est | Status | Notes` (current) — we read columns
+// by name. Falls back to legacy positional mapping when no header row is found,
+// so older / hand-written tables keep parsing.
+const TRACKER_COLS = {
+  id: ['#', 'id'],
+  name: ['subtask', 'task', 'name'],
+  status: ['status'],
+  date: ['date'],
+  notes: ['notes'],
+  acceptance: ['acceptance'],
+  est: ['est', 'estimate'],
+};
+const LEGACY_POS = { id: 0, name: 1, status: 2, date: 3, notes: 4 };
+
+function splitRow(line) {
+  // Drop the empty cells produced by the leading/trailing pipes.
+  return line.split('|').map((c) => c.trim()).filter((_, i, arr) => i > 0 && i < arr.length - 1);
+}
+
 export function parseSubtaskTracker(sectionText) {
   if (!sectionText) return [];
   const lines = sectionText.split('\n').map((l) => l.trim()).filter(Boolean);
+  let colMap = null; // canonical-name -> column index
   const rows = [];
-  let inTable = false;
   for (const line of lines) {
-    if (line.startsWith('|---') || line.match(/^\|[\s|:-]+\|$/)) { inTable = true; continue; }
-    if (!line.startsWith('|')) { inTable = false; continue; }
-    const cells = line.split('|').map((c) => c.trim()).filter((_, i, arr) => i > 0 && i < arr.length - 1);
+    if (!line.startsWith('|')) continue;
+    if (line.match(/^\|[\s|:-]+\|$/)) continue; // separator row
+    const cells = splitRow(line);
     if (cells.length < 2) continue;
-    if (!inTable && (cells[0].toLowerCase().startsWith('#') || cells[0].toLowerCase() === 'ws' || cells[0].toLowerCase() === 'st')) {
-      // header row
+
+    const low = cells.map((c) => c.toLowerCase());
+    // Header row: first cell is '#'/'id', or it names known columns. Detect once.
+    const looksLikeHeader = low[0] === '#' || low[0] === 'id'
+      || low.includes('status') || low.includes('subtask');
+    if (!colMap && looksLikeHeader) {
+      colMap = {};
+      for (const [canon, aliases] of Object.entries(TRACKER_COLS)) {
+        const idx = low.findIndex((c) => aliases.includes(c));
+        if (idx >= 0) colMap[canon] = idx;
+      }
       continue;
     }
-    if (cells.length >= 3) {
-      rows.push({
-        id: cells[0],
-        name: cells[1],
-        status: cells[2],
-        date: cells[3] || '',
-        notes: cells[4] || '',
-      });
-    }
+
+    // Data row. Skip stray non-subtask rows when using positional fallback.
+    if (!colMap && !/^(ST|WS)-/i.test(cells[0]) && !/^\d/.test(cells[0])) continue;
+    if (cells.length < 3 && !colMap) continue;
+
+    const get = (canon) => {
+      const idx = colMap ? colMap[canon] : LEGACY_POS[canon];
+      return idx != null && idx >= 0 ? (cells[idx] ?? '') : '';
+    };
+    rows.push({
+      id: get('id'),
+      name: get('name'),
+      status: get('status'),
+      date: get('date'),
+      notes: get('notes'),
+      acceptance: get('acceptance'),
+      est: get('est'),
+    });
   }
   return rows;
 }

package/src/lib/ui.mjs

@@ -1,66 +1,66 @@
-import chalk from 'chalk';
-import { createInterface } from 'node:readline/promises';
-import { stdin, stdout } from 'node:process';
-
-export const BANNER = `
-  ██████╗ ██╗    ██╗   ██╗  ██╗██╗████████╗
-  ██╔══██╗██║    ██║   ██║ ██╔╝██║╚══██╔══╝
-  ██║  ██║██║ █╗ ██║   █████╔╝ ██║   ██║
-  ██║  ██║██║███╗██║   ██╔═██╗ ██║   ██║
-  ██████╔╝╚███╔███╔╝   ██║  ██╗██║   ██║
-  ╚═════╝  ╚══╝╚══╝    ╚═╝  ╚═╝╚═╝   ╚═╝`;
-
-export function banner(subtitle) {
-  console.log(chalk.cyan.bold(BANNER));
-  if (subtitle) console.log(chalk.cyan(`  ${subtitle}`));
-  console.log();
-}
-
-export function header(text) {
-  console.log();
-  console.log(chalk.bold(`══════════════════════════════════════════`));
-  console.log(chalk.bold(`  ${text}`));
-  console.log(chalk.bold(`══════════════════════════════════════════`));
-}
-
-export const log = (msg) => console.log(`  ${msg}`);
-export const ok = (msg) => console.log(chalk.green(`  ✓  ${msg}`));
-export const warn = (msg) => console.log(chalk.yellow(`  ⚠  ${msg}`));
-export const err = (msg) => console.log(chalk.red(`  ✗  ${msg}`));
-export const info = (msg) => { console.log(); console.log(chalk.cyan(`▶ ${msg}`)); };
-export const dry = (msg) => console.log(chalk.dim(`  [dry-run] ${msg}`));
-
-export async function ask(question, defaultValue) {
-  const rl = createInterface({ input: stdin, output: stdout });
-  const suffix = defaultValue ? ` [${defaultValue}]` : '';
-  try {
-    const answer = await rl.question(chalk.bold(`  ${question}${suffix}: `));
-    return answer.trim() || defaultValue || '';
-  } finally {
-    rl.close();
-  }
-}
-
-export async function choose(question, options, defaultValue) {
-  console.log(chalk.bold(`  ${question}`));
-  for (const opt of options) {
-    const marker = opt.value === defaultValue ? chalk.cyan(' [default]') : '';
-    console.log(`    ${opt.value} = ${opt.label}${marker}`);
-  }
-  const allowed = new Set(options.map((o) => o.value));
-  while (true) {
-    const answer = await ask('>', defaultValue);
-    if (allowed.has(answer)) return answer;
-    warn(`Invalid choice: "${answer}". Allowed: ${[...allowed].join(', ')}`);
-  }
-}
-
-export async function multiSelect(question, options, defaultValues) {
-  console.log(chalk.bold(`  ${question}`));
-  for (const opt of options) {
-    const marker = defaultValues?.includes(opt.value) ? chalk.dim(' (included)') : '';
-    console.log(`    ${opt.key} = ${opt.label}${marker}`);
-  }
-  const hint = defaultValues ? defaultValues.join(',') : '';
-  return ask('Enter numbers separated by comma', hint);
-}
+import chalk from 'chalk';
+import { createInterface } from 'node:readline/promises';
+import { stdin, stdout } from 'node:process';
+
+export const BANNER = `
+  ██████╗ ██╗    ██╗   ██╗  ██╗██╗████████╗
+  ██╔══██╗██║    ██║   ██║ ██╔╝██║╚══██╔══╝
+  ██║  ██║██║ █╗ ██║   █████╔╝ ██║   ██║
+  ██║  ██║██║███╗██║   ██╔═██╗ ██║   ██║
+  ██████╔╝╚███╔███╔╝   ██║  ██╗██║   ██║
+  ╚═════╝  ╚══╝╚══╝    ╚═╝  ╚═╝╚═╝   ╚═╝`;
+
+export function banner(subtitle) {
+  console.log(chalk.cyan.bold(BANNER));
+  if (subtitle) console.log(chalk.cyan(`  ${subtitle}`));
+  console.log();
+}
+
+export function header(text) {
+  console.log();
+  console.log(chalk.bold(`══════════════════════════════════════════`));
+  console.log(chalk.bold(`  ${text}`));
+  console.log(chalk.bold(`══════════════════════════════════════════`));
+}
+
+export const log = (msg) => console.log(`  ${msg}`);
+export const ok = (msg) => console.log(chalk.green(`  ✓  ${msg}`));
+export const warn = (msg) => console.log(chalk.yellow(`  ⚠  ${msg}`));
+export const err = (msg) => console.log(chalk.red(`  ✗  ${msg}`));
+export const info = (msg) => { console.log(); console.log(chalk.cyan(`▶ ${msg}`)); };
+export const dry = (msg) => console.log(chalk.dim(`  [dry-run] ${msg}`));
+
+export async function ask(question, defaultValue) {
+  const rl = createInterface({ input: stdin, output: stdout });
+  const suffix = defaultValue ? ` [${defaultValue}]` : '';
+  try {
+    const answer = await rl.question(chalk.bold(`  ${question}${suffix}: `));
+    return answer.trim() || defaultValue || '';
+  } finally {
+    rl.close();
+  }
+}
+
+export async function choose(question, options, defaultValue) {
+  console.log(chalk.bold(`  ${question}`));
+  for (const opt of options) {
+    const marker = opt.value === defaultValue ? chalk.cyan(' [default]') : '';
+    console.log(`    ${opt.value} = ${opt.label}${marker}`);
+  }
+  const allowed = new Set(options.map((o) => o.value));
+  while (true) {
+    const answer = await ask('>', defaultValue);
+    if (allowed.has(answer)) return answer;
+    warn(`Invalid choice: "${answer}". Allowed: ${[...allowed].join(', ')}`);
+  }
+}
+
+export async function multiSelect(question, options, defaultValues) {
+  console.log(chalk.bold(`  ${question}`));
+  for (const opt of options) {
+    const marker = defaultValues?.includes(opt.value) ? chalk.dim(' (included)') : '';
+    console.log(`    ${opt.key} = ${opt.label}${marker}`);
+  }
+  const hint = defaultValues ? defaultValues.join(',') : '';
+  return ask('Enter numbers separated by comma', hint);
+}

package/src/lib/update-checker.mjs

@@ -1,73 +1,73 @@
-import { readFileSync, writeFileSync, mkdirSync } from 'node:fs';
-import { join } from 'node:path';
-import { homedir } from 'node:os';
-
-const CACHE_TTL_MS = 24 * 60 * 60 * 1000; // 24 hours
-const REGISTRY_URL = 'https://registry.npmjs.org/dw-kit/latest';
-const CACHE_DIR = join(homedir(), '.dw-kit');
-const CACHE_FILE = join(CACHE_DIR, 'update-cache.json');
-
-function parseSemver(v) {
-  const parts = String(v).replace(/^v/, '').split('.').map(Number);
-  return { major: parts[0] || 0, minor: parts[1] || 0, patch: parts[2] || 0 };
-}
-
-function isNewer(latest, current) {
-  const l = parseSemver(latest);
-  const c = parseSemver(current);
-  if (l.major !== c.major) return l.major > c.major;
-  if (l.minor !== c.minor) return l.minor > c.minor;
-  return l.patch > c.patch;
-}
-
-function readCache() {
-  try {
-    return JSON.parse(readFileSync(CACHE_FILE, 'utf-8'));
-  } catch {
-    return null;
-  }
-}
-
-function writeCache(data) {
-  try {
-    mkdirSync(CACHE_DIR, { recursive: true });
-    writeFileSync(CACHE_FILE, JSON.stringify(data), 'utf-8');
-  } catch {
-    // ignore write errors (permission, disk full, etc.)
-  }
-}
-
-async function fetchLatestVersion() {
-  const res = await fetch(REGISTRY_URL, { signal: AbortSignal.timeout(3000) });
-  if (!res.ok) throw new Error(`HTTP ${res.status}`);
-  const data = await res.json();
-  return data.version;
-}
-
-/**
- * Returns latest version string if an update is available (from cache), null otherwise.
- * Never throws, never makes network calls.
- */
-export function getUpdateNotice(currentVersion) {
-  if (process.env.DW_NO_UPDATE_CHECK) return null;
-  const cache = readCache();
-  if (!cache?.latest) return null;
-  return isNewer(cache.latest, currentVersion) ? cache.latest : null;
-}
-
-/**
- * Fires off an async check against npm registry and updates the cache.
- * Non-blocking — caller should NOT await this.
- * Skips the check if cache is still fresh (< 24h).
- */
-export function scheduleUpdateCheck(currentVersion) {
-  if (process.env.DW_NO_UPDATE_CHECK) return;
-
-  const cache = readCache();
-  const now = Date.now();
-  if (cache?.checkedAt && now - cache.checkedAt < CACHE_TTL_MS) return;
-
-  fetchLatestVersion()
-    .then((latest) => writeCache({ latest, checkedAt: now, current: currentVersion }))
-    .catch(() => {});
-}
+import { readFileSync, writeFileSync, mkdirSync } from 'node:fs';
+import { join } from 'node:path';
+import { homedir } from 'node:os';
+
+const CACHE_TTL_MS = 24 * 60 * 60 * 1000; // 24 hours
+const REGISTRY_URL = 'https://registry.npmjs.org/dw-kit/latest';
+const CACHE_DIR = join(homedir(), '.dw-kit');
+const CACHE_FILE = join(CACHE_DIR, 'update-cache.json');
+
+function parseSemver(v) {
+  const parts = String(v).replace(/^v/, '').split('.').map(Number);
+  return { major: parts[0] || 0, minor: parts[1] || 0, patch: parts[2] || 0 };
+}
+
+function isNewer(latest, current) {
+  const l = parseSemver(latest);
+  const c = parseSemver(current);
+  if (l.major !== c.major) return l.major > c.major;
+  if (l.minor !== c.minor) return l.minor > c.minor;
+  return l.patch > c.patch;
+}
+
+function readCache() {
+  try {
+    return JSON.parse(readFileSync(CACHE_FILE, 'utf-8'));
+  } catch {
+    return null;
+  }
+}
+
+function writeCache(data) {
+  try {
+    mkdirSync(CACHE_DIR, { recursive: true });
+    writeFileSync(CACHE_FILE, JSON.stringify(data), 'utf-8');
+  } catch {
+    // ignore write errors (permission, disk full, etc.)
+  }
+}
+
+async function fetchLatestVersion() {
+  const res = await fetch(REGISTRY_URL, { signal: AbortSignal.timeout(3000) });
+  if (!res.ok) throw new Error(`HTTP ${res.status}`);
+  const data = await res.json();
+  return data.version;
+}
+
+/**
+ * Returns latest version string if an update is available (from cache), null otherwise.
+ * Never throws, never makes network calls.
+ */
+export function getUpdateNotice(currentVersion) {
+  if (process.env.DW_NO_UPDATE_CHECK) return null;
+  const cache = readCache();
+  if (!cache?.latest) return null;
+  return isNewer(cache.latest, currentVersion) ? cache.latest : null;
+}
+
+/**
+ * Fires off an async check against npm registry and updates the cache.
+ * Non-blocking — caller should NOT await this.
+ * Skips the check if cache is still fresh (< 24h).
+ */
+export function scheduleUpdateCheck(currentVersion) {
+  if (process.env.DW_NO_UPDATE_CHECK) return;
+
+  const cache = readCache();
+  const now = Date.now();
+  if (cache?.checkedAt && now - cache.checkedAt < CACHE_TTL_MS) return;
+
+  fetchLatestVersion()
+    .then((latest) => writeCache({ latest, checkedAt: now, current: currentVersion }))
+    .catch(() => {});
+}