dw-kit 1.9.1 → 1.9.3

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/task-md-utils.mjs

@@ -0,0 +1,78 @@
+// task-md-utils.mjs — shared parsers for task.md v3 files.
+//
+// Used by board-data.mjs + goal-progress.mjs. Extracted into its own module
+// to break the cycle that would otherwise form between those two
+// (board-data attaches progress.computeGoalProgress, goal-progress reads the
+// same parseSubtasks board-data uses).
+//
+// Per ADR-0001: zero-dep beyond js-yaml.
+
+import { readFileSync, existsSync, readdirSync, statSync } from 'node:fs';
+import { join } from 'node:path';
+import yaml from 'js-yaml';
+
+const TASKS_DIR = '.dw/tasks';
+
+const STATUS_ICONS = {
+  '⬜': 'pending',
+  '🟡': 'in_progress',
+  '✅': 'done',
+  '🔴': 'blocked',
+  '⏸': 'paused',
+};
+
+/** Extract frontmatter (between --- markers) from a markdown file. */
+export function readFrontmatter(file) {
+  if (!existsSync(file)) return {};
+  let txt;
+  try { txt = readFileSync(file, 'utf8'); } catch { return {}; }
+  const m = txt.match(/^---\r?\n([\s\S]*?)\r?\n---/);
+  if (!m) return {};
+  try { return yaml.load(m[1]) || {}; } catch { return {}; }
+}
+
+/**
+ * Parse Section 3 Subtask Tracker rows from a task.md.
+ * Returns rows shaped { st_id, title, status_bucket, status_icon, status_label, date, notes }.
+ */
+export function parseSubtasks(taskPath) {
+  if (!existsSync(taskPath)) return [];
+  let txt;
+  try { txt = readFileSync(taskPath, 'utf8'); } catch { return []; }
+  const sec = txt.match(/^## 3\.[^\n]*\n([\s\S]*?)(?=^## 4\.|$(?![\s\S]))/m);
+  if (!sec) return [];
+  const out = [];
+  for (const line of sec[1].split('\n')) {
+    // `u` flag is essential — 🟡 (U+1F7E1) and 🔴 (U+1F534) are surrogate
+    // pairs; the character class would otherwise match the low surrogate
+    // instead of the emoji, silently dropping in_progress / blocked rows.
+    const m = line.match(/^\|\s*(ST-[\w.-]+)\s*\|\s*(.+?)\s*\|\s*([⬜🟡✅🔴⏸])\s*([A-Za-z ]+)?\s*\|\s*([^|]*)\|\s*([^|]*)\|/u);
+    if (!m) continue;
+    const icon = m[3];
+    const statusBucket = STATUS_ICONS[icon] || 'unknown';
+    const statusLabel = (m[4] || '').trim();
+    out.push({
+      st_id: m[1].trim(),
+      title: m[2].replace(/`/g, '').trim(),
+      status_bucket: statusBucket,
+      status_icon: icon,
+      status_label: statusLabel,
+      date: m[5].trim(),
+      notes: m[6].trim().slice(0, 160),
+    });
+  }
+  return out;
+}
+
+/** List task directories under `.dw/tasks/` (skipping archive + dotfiles). */
+export function listTaskDirs(rootDir) {
+  const dir = join(rootDir, TASKS_DIR);
+  if (!existsSync(dir)) return [];
+  return readdirSync(dir)
+    .filter((entry) => {
+      if (entry.startsWith('.') || entry === 'archive') return false;
+      try { return statSync(join(dir, entry)).isDirectory(); } catch { return false; }
+    });
+}
+
+export { TASKS_DIR, STATUS_ICONS };

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(() => {});
+}