dotmd-cli 0.4.0 → 0.5.0

package/bin/dotmd.mjs

@@ -282,7 +282,6 @@ async function main() {
   if (command === 'index') {
     if (!config.indexPath) {
       die('Index generation is not configured. Add an `index` section to your dotmd.config.mjs.');
-      return;
     }
     const write = args.includes('--write');
     const rendered = renderIndexFile(index, config);
@@ -306,5 +305,6 @@ async function main() {
 }
 
 main().catch(err => {
-  die(err.message);
+  process.stderr.write(`${err.message}\n`);
+  process.exitCode = 1;
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dotmd-cli",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "description": "Zero-dependency CLI for managing markdown documents with YAML frontmatter — index, query, validate, lifecycle.",
   "type": "module",
   "license": "MIT",

package/src/completions.mjs

@@ -90,7 +90,6 @@ export function runCompletions(argv) {
   const shell = argv[0];
   if (!shell) {
     die('Usage: dotmd completions <bash|zsh>');
-    return;
   }
   if (shell === 'bash') {
     process.stdout.write(bashCompletion() + '\n');

package/src/config.mjs

@@ -100,6 +100,15 @@ function validateConfig(userConfig, config, validStatuses, indexPath) {
     }
   }
 
+  // staleDays keys must exist in validStatuses
+  if (config.statuses?.staleDays) {
+    for (const key of Object.keys(config.statuses.staleDays)) {
+      if (!validStatuses.has(key)) {
+        warnings.push(`Config: statuses.staleDays contains unknown status '${key}'.`);
+      }
+    }
+  }
+
   // taxonomy.surfaces must be null or array
   if (config.taxonomy?.surfaces !== undefined && config.taxonomy.surfaces !== null && !Array.isArray(config.taxonomy.surfaces)) {
     warnings.push('Config: taxonomy.surfaces must be null or an array.');
@@ -149,22 +158,6 @@ export async function resolveConfig(cwd, explicitConfigPath) {
       mod = await import(configUrl);
     } catch (err) {
       die('Failed to load config: ' + configPath + '\n' + err.message + '\nRun `dotmd init` to create a starter config.');
-      // Return defaults so caller can still function
-      const defaults = deepMerge(DEFAULTS, {});
-      const defaultStatusOrder = defaults.statuses.order;
-      return {
-        raw: defaults, docsRoot: cwd, repoRoot: cwd, configDir: cwd,
-        configPath: configPath ?? null, configFound: Boolean(configPath),
-        archiveDir: defaults.archiveDir, excludeDirs: new Set(defaults.excludeDirs),
-        docsRootPrefix: '', statusOrder: defaultStatusOrder,
-        validStatuses: new Set(defaultStatusOrder), staleDaysByStatus: {},
-        lifecycle: { archiveStatuses: new Set(defaults.lifecycle.archiveStatuses), skipStaleFor: new Set(defaults.lifecycle.skipStaleFor), skipWarningsFor: new Set(defaults.lifecycle.skipWarningsFor) },
-        validSurfaces: null, moduleRequiredStatuses: new Set(),
-        indexPath: null, indexStartMarker: '<!-- GENERATED:dotmd:start -->', indexEndMarker: '<!-- GENERATED:dotmd:end -->', archivedHighlightLimit: 8,
-        context: defaults.context, display: defaults.display,
-        referenceFields: defaults.referenceFields, presets: defaults.presets, hooks: {},
-        configWarnings: [],
-      };
     }
 
     configDir = path.dirname(configPath);
@@ -189,8 +182,9 @@ export async function resolveConfig(cwd, explicitConfigPath) {
 
   const docsRoot = path.resolve(configDir, config.root);
 
+  const earlyWarnings = [];
   if (!existsSync(docsRoot)) {
-    warn('Docs root does not exist: ' + docsRoot);
+    earlyWarnings.push('Config: docs root does not exist: ' + docsRoot);
   }
 
   // Find repo root by walking up looking for .git
@@ -233,7 +227,7 @@ export async function resolveConfig(cwd, explicitConfigPath) {
   const skipStaleFor = new Set(lifecycle.skipStaleFor);
   const skipWarningsFor = new Set(lifecycle.skipWarningsFor);
 
-  const configWarnings = validateConfig(userConfig, config, validStatuses, indexPath);
+  const configWarnings = [...earlyWarnings, ...validateConfig(userConfig, config, validStatuses, indexPath)];
 
   return {
     raw: config,

package/src/diff.mjs

@@ -27,7 +27,6 @@ export function runDiff(argv, config) {
     const filePath = resolveDocPath(file, config);
     if (!filePath) {
       die(`File not found: ${file}\nSearched: ${toRepoPath(config.repoRoot, config.repoRoot) || '.'}, ${toRepoPath(config.docsRoot, config.repoRoot)}`);
-      return;
     }
 
     const raw = readFileSync(filePath, 'utf8');
@@ -37,7 +36,6 @@ export function runDiff(argv, config) {
 
     if (!since) {
       die(`No updated date found in ${file} and no --since provided.`);
-      return;
     }
 
     const relPath = toRepoPath(filePath, config.repoRoot);
@@ -80,9 +78,15 @@ function printFileDiff(relPath, since, diffOutput, opts) {
   process.stdout.write(bold(relPath) + dim(` (updated: ${since})`) + '\n');
 
   if (opts.summarize) {
-    const summary = opts.config?.hooks?.summarizeDiff
-      ? opts.config.hooks.summarizeDiff(diffOutput, relPath)
-      : summarizeWithMLX(diffOutput, relPath, opts.model);
+    let summary;
+    try {
+      summary = opts.config?.hooks?.summarizeDiff
+        ? opts.config.hooks.summarizeDiff(diffOutput, relPath)
+        : summarizeWithMLX(diffOutput, relPath, opts.model);
+    } catch (err) {
+      warn(`Hook 'summarizeDiff' threw: ${err.message}`);
+      summary = null;
+    }
     if (summary) {
       process.stdout.write(dim(`  Summary: ${summary}`) + '\n');
     } else {
@@ -95,6 +99,12 @@ function printFileDiff(relPath, since, diffOutput, opts) {
 }
 
 function summarizeWithMLX(diffText, filePath, model) {
+  const uvCheck = spawnSync('uv', ['--version'], { encoding: 'utf8' });
+  if (uvCheck.error) {
+    warn('uv is not installed. Install it to enable --summarize: https://docs.astral.sh/uv/');
+    return null;
+  }
+
   const prompt = `Summarize this git diff in 1-2 sentences. Focus on what changed semantically, not line counts.\n\nFile: ${filePath}\n\n${diffText.slice(0, 4000)}`;
 
   const result = spawnSync('uv', [

package/src/frontmatter.mjs

@@ -14,6 +14,14 @@ export function extractFrontmatter(raw) {
   };
 }
 
+export function replaceFrontmatter(raw, newFrontmatter) {
+  if (!raw.startsWith('---\n')) return raw;
+  const endMarker = raw.indexOf('\n---\n', 4);
+  if (endMarker === -1) return raw;
+  const body = raw.slice(endMarker + 5);
+  return `---\n${newFrontmatter}\n---\n${body}`;
+}
+
 export function parseSimpleFrontmatter(text) {
   const data = {};
   let currentArrayKey = null;
@@ -25,6 +33,10 @@ export function parseSimpleFrontmatter(text) {
     const keyMatch = line.match(/^([A-Za-z0-9_-]+):\s*(.*)$/);
     if (keyMatch) {
       const [, key, rawValue] = keyMatch;
+      if (Object.prototype.hasOwnProperty.call(data, key)) {
+        currentArrayKey = null;
+        continue;
+      }
       if (!rawValue.trim()) {
         data[key] = [];
         currentArrayKey = key;
@@ -48,7 +60,12 @@ export function parseSimpleFrontmatter(text) {
 }
 
 function parseScalar(value) {
-  const unquoted = value.replace(/^['"]|['"]$/g, '');
+  let unquoted = value;
+  if (value.length > 1 &&
+      ((value.startsWith("'") && value.endsWith("'")) ||
+       (value.startsWith('"') && value.endsWith('"')))) {
+    unquoted = value.slice(1, -1);
+  }
   if (unquoted === 'true') return true;
   if (unquoted === 'false') return false;
   return unquoted;

package/src/git.mjs

@@ -1,15 +1,26 @@
 import { spawnSync } from 'node:child_process';
 
+let gitChecked = false;
+function ensureGit() {
+  if (gitChecked) return;
+  const result = spawnSync('git', ['--version'], { encoding: 'utf8' });
+  if (result.error) {
+    throw new Error('git is not installed or not found in PATH. dotmd requires git for this operation.');
+  }
+  gitChecked = true;
+}
+
 export function getGitLastModified(relPath, repoRoot) {
   const result = spawnSync('git', ['log', '-1', '--format=%aI', '--', relPath], {
     cwd: repoRoot,
     encoding: 'utf8',
   });
-  if (result.status !== 0 || !result.stdout.trim()) return null;
+  if (result.error || result.status !== 0 || !result.stdout.trim()) return null;
   return result.stdout.trim();
 }
 
 export function gitMv(source, target, repoRoot) {
+  ensureGit();
   const result = spawnSync('git', ['mv', source, target], {
     cwd: repoRoot,
     encoding: 'utf8',
@@ -18,6 +29,7 @@ export function gitMv(source, target, repoRoot) {
 }
 
 export function gitDiffSince(relPath, sinceDate, repoRoot, opts = {}) {
+  ensureGit();
   // Find the last commit at or before sinceDate
   const baseline = spawnSync('git', [
     'log', '-1', '--before=' + sinceDate + 'T23:59:59', '--format=%H', '--', relPath

package/src/index.mjs

@@ -2,7 +2,7 @@ import { readdirSync, readFileSync } from 'node:fs';
 import path from 'node:path';
 import { extractFrontmatter, parseSimpleFrontmatter } from './frontmatter.mjs';
 import { extractFirstHeading, extractSummary, extractStatusSnapshot, extractNextStep, extractChecklistCounts } from './extractors.mjs';
-import { asString, normalizeStringList, normalizeBlockers, mergeUniqueStrings, toRepoPath } from './util.mjs';
+import { asString, normalizeStringList, normalizeBlockers, mergeUniqueStrings, toRepoPath, warn } from './util.mjs';
 import { validateDoc, checkBidirectionalReferences, checkGitStaleness, computeDaysSinceUpdate, computeIsStale, computeChecklistCompletionRate } from './validate.mjs';
 import { checkIndex } from './index-file.mjs';
 
@@ -19,20 +19,32 @@ export function buildIndex(config) {
   if (config.hooks.validate) {
     const ctx = { config, allDocs: docs, repoRoot: config.repoRoot };
     for (const doc of docs) {
-      const result = config.hooks.validate(doc, ctx);
-      if (result?.errors) {
-        doc.errors.push(...result.errors);
-        errors.push(...result.errors);
-      }
-      if (result?.warnings) {
-        doc.warnings.push(...result.warnings);
-        warnings.push(...result.warnings);
+      try {
+        const result = config.hooks.validate(doc, ctx);
+        if (result?.errors) {
+          doc.errors.push(...result.errors);
+          errors.push(...result.errors);
+        }
+        if (result?.warnings) {
+          doc.warnings.push(...result.warnings);
+          warnings.push(...result.warnings);
+        }
+      } catch (err) {
+        const hookError = { path: doc.path, level: 'error', message: `Hook 'validate' threw: ${err.message}` };
+        doc.errors.push(hookError);
+        errors.push(hookError);
       }
     }
   }
 
   const transformedDocs = config.hooks.transformDoc
-    ? docs.map(d => config.hooks.transformDoc(d) ?? d)
+    ? docs.map(d => {
+        try { return config.hooks.transformDoc(d) ?? d; }
+        catch (err) {
+          warnings.push({ path: d.path, level: 'warning', message: `Hook 'transformDoc' threw: ${err.message}` });
+          return d;
+        }
+      })
     : docs;
 
   const countsByStatus = Object.fromEntries(config.statusOrder.map(status => [

package/src/lifecycle.mjs

@@ -1,7 +1,7 @@
-import { existsSync, readFileSync, writeFileSync } from 'node:fs';
+import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
 import path from 'node:path';
 import { extractFrontmatter, parseSimpleFrontmatter } from './frontmatter.mjs';
-import { asString, toRepoPath, die, resolveDocPath } from './util.mjs';
+import { asString, toRepoPath, die, warn, resolveDocPath, escapeRegex } from './util.mjs';
 import { gitMv } from './git.mjs';
 import { buildIndex, collectDocFiles } from './index.mjs';
 import { renderIndexFile, writeIndex } from './index-file.mjs';
@@ -12,11 +12,11 @@ export function runStatus(argv, config, opts = {}) {
   const input = argv[0];
   const newStatus = argv[1];
 
-  if (!input || !newStatus) { die('Usage: dotmd status <file> <new-status>'); return; }
-  if (!config.validStatuses.has(newStatus)) { die(`Invalid status: ${newStatus}\nValid: ${[...config.validStatuses].join(', ')}`); return; }
+  if (!input || !newStatus) { die('Usage: dotmd status <file> <new-status>'); }
+  if (!config.validStatuses.has(newStatus)) { die(`Invalid status: ${newStatus}\nValid: ${[...config.validStatuses].join(', ')}`); }
 
   const filePath = resolveDocPath(input, config);
-  if (!filePath) { die(`File not found: ${input}\nSearched: ${toRepoPath(config.repoRoot, config.repoRoot) || '.'}, ${toRepoPath(config.docsRoot, config.repoRoot)}`); return; }
+  if (!filePath) { die(`File not found: ${input}\nSearched: ${toRepoPath(config.repoRoot, config.repoRoot) || '.'}, ${toRepoPath(config.docsRoot, config.repoRoot)}`); }
 
   const raw = readFileSync(filePath, 'utf8');
   const { frontmatter } = extractFrontmatter(raw);
@@ -57,18 +57,19 @@ export function runStatus(argv, config, opts = {}) {
   updateFrontmatter(filePath, { status: newStatus, updated: today });
 
   if (isArchiving) {
+    mkdirSync(archiveDir, { recursive: true });
     const targetPath = path.join(archiveDir, path.basename(filePath));
-    if (existsSync(targetPath)) { die(`Target already exists: ${toRepoPath(targetPath, config.repoRoot)}`); return; }
+    if (existsSync(targetPath)) { die(`Target already exists: ${toRepoPath(targetPath, config.repoRoot)}`); }
     const result = gitMv(filePath, targetPath, config.repoRoot);
-    if (result.status !== 0) { die(result.stderr || 'git mv failed.'); return; }
+    if (result.status !== 0) { die(result.stderr || 'git mv failed.'); }
     finalPath = targetPath;
   }
 
   if (isUnarchiving) {
     const targetPath = path.join(config.docsRoot, path.basename(filePath));
-    if (existsSync(targetPath)) { die(`Target already exists: ${toRepoPath(targetPath, config.repoRoot)}`); return; }
+    if (existsSync(targetPath)) { die(`Target already exists: ${toRepoPath(targetPath, config.repoRoot)}`); }
     const result = gitMv(filePath, targetPath, config.repoRoot);
-    if (result.status !== 0) { die(result.stderr || 'git mv failed.'); return; }
+    if (result.status !== 0) { die(result.stderr || 'git mv failed.'); }
     finalPath = targetPath;
   }
 
@@ -79,21 +80,21 @@ export function runStatus(argv, config, opts = {}) {
 
   process.stdout.write(`${green(toRepoPath(finalPath, config.repoRoot))}: ${oldStatus ?? 'unknown'} → ${newStatus}\n`);
 
-  config.hooks.onStatusChange?.({ path: toRepoPath(finalPath, config.repoRoot), oldStatus, newStatus }, {
+  try { config.hooks.onStatusChange?.({ path: toRepoPath(finalPath, config.repoRoot), oldStatus, newStatus }, {
     oldPath: toRepoPath(filePath, config.repoRoot),
     newPath: toRepoPath(finalPath, config.repoRoot),
-  });
+  }); } catch (err) { warn(`Hook 'onStatusChange' threw: ${err.message}`); }
 }
 
 export function runArchive(argv, config, opts = {}) {
   const { dryRun } = opts;
   const input = argv[0];
 
-  if (!input) { die('Usage: dotmd archive <file>'); return; }
+  if (!input) { die('Usage: dotmd archive <file>'); }
 
   const filePath = resolveDocPath(input, config);
-  if (!filePath) { die(`File not found: ${input}\nSearched: ${toRepoPath(config.repoRoot, config.repoRoot) || '.'}, ${toRepoPath(config.docsRoot, config.repoRoot)}`); return; }
-  if (filePath.includes(`/${config.archiveDir}/`)) { die(`Already archived: ${toRepoPath(filePath, config.repoRoot)}`); return; }
+  if (!filePath) { die(`File not found: ${input}\nSearched: ${toRepoPath(config.repoRoot, config.repoRoot) || '.'}, ${toRepoPath(config.docsRoot, config.repoRoot)}`); }
+  if (filePath.includes(`/${config.archiveDir}/`)) { die(`Already archived: ${toRepoPath(filePath, config.repoRoot)}`); }
 
   const raw = readFileSync(filePath, 'utf8');
   const { frontmatter } = extractFrontmatter(raw);
@@ -109,7 +110,7 @@ export function runArchive(argv, config, opts = {}) {
   if (dryRun) {
     const prefix = dim('[dry-run]');
     process.stdout.write(`${prefix} Would update frontmatter: status: ${oldStatus} → archived, updated: ${today}\n`);
-    if (existsSync(targetPath)) { die(`Target already exists: ${toRepoPath(targetPath, config.repoRoot)}`); return; }
+    if (existsSync(targetPath)) { die(`Target already exists: ${toRepoPath(targetPath, config.repoRoot)}`); }
     process.stdout.write(`${prefix} Would move: ${oldRepoPath} → ${newRepoPath}\n`);
     if (config.indexPath) process.stdout.write(`${prefix} Would regenerate index\n`);
 
@@ -133,10 +134,11 @@ export function runArchive(argv, config, opts = {}) {
 
   updateFrontmatter(filePath, { status: 'archived', updated: today });
 
-  if (existsSync(targetPath)) { die(`Target already exists: ${toRepoPath(targetPath, config.repoRoot)}`); return; }
+  mkdirSync(targetDir, { recursive: true });
+  if (existsSync(targetPath)) { die(`Target already exists: ${toRepoPath(targetPath, config.repoRoot)}`); }
 
   const result = gitMv(filePath, targetPath, config.repoRoot);
-  if (result.status !== 0) { die(result.stderr || 'git mv failed.'); return; }
+  if (result.status !== 0) { die(result.stderr || 'git mv failed.'); }
 
   if (config.indexPath) {
     const index = buildIndex(config);
@@ -163,17 +165,17 @@ export function runArchive(argv, config, opts = {}) {
   }
 
   process.stdout.write('\nNext: commit, then update references if needed.\n');
-  config.hooks.onArchive?.({ path: newRepoPath, oldStatus }, { oldPath: oldRepoPath, newPath: newRepoPath });
+  try { config.hooks.onArchive?.({ path: newRepoPath, oldStatus }, { oldPath: oldRepoPath, newPath: newRepoPath }); } catch (err) { warn(`Hook 'onArchive' threw: ${err.message}`); }
 }
 
 export function runTouch(argv, config, opts = {}) {
   const { dryRun } = opts;
   const input = argv[0];
 
-  if (!input) { die('Usage: dotmd touch <file>'); return; }
+  if (!input) { die('Usage: dotmd touch <file>'); }
 
   const filePath = resolveDocPath(input, config);
-  if (!filePath) { die(`File not found: ${input}\nSearched: ${toRepoPath(config.repoRoot, config.repoRoot) || '.'}, ${toRepoPath(config.docsRoot, config.repoRoot)}`); return; }
+  if (!filePath) { die(`File not found: ${input}\nSearched: ${toRepoPath(config.repoRoot, config.repoRoot) || '.'}, ${toRepoPath(config.docsRoot, config.repoRoot)}`); }
 
   const today = new Date().toISOString().slice(0, 10);
 
@@ -185,7 +187,7 @@ export function runTouch(argv, config, opts = {}) {
   updateFrontmatter(filePath, { updated: today });
   process.stdout.write(`${green('Touched')}: ${toRepoPath(filePath, config.repoRoot)} (updated → ${today})\n`);
 
-  config.hooks.onTouch?.({ path: toRepoPath(filePath, config.repoRoot) }, { path: toRepoPath(filePath, config.repoRoot), date: today });
+  try { config.hooks.onTouch?.({ path: toRepoPath(filePath, config.repoRoot) }, { path: toRepoPath(filePath, config.repoRoot), date: today }); } catch (err) { warn(`Hook 'onTouch' threw: ${err.message}`); }
 }
 
 export function updateFrontmatter(filePath, updates) {
@@ -199,7 +201,7 @@ export function updateFrontmatter(filePath, updates) {
   const body = raw.slice(endMarker + 5);
 
   for (const [key, value] of Object.entries(updates)) {
-    const regex = new RegExp(`^${key}:.*$`, 'm');
+    const regex = new RegExp(`^${escapeRegex(key)}:.*$`, 'm');
     if (regex.test(frontmatter)) {
       frontmatter = frontmatter.replace(regex, `${key}: ${value}`);
     } else {

package/src/lint.mjs

@@ -1,6 +1,6 @@
 import { readFileSync, writeFileSync } from 'node:fs';
-import { extractFrontmatter, parseSimpleFrontmatter } from './frontmatter.mjs';
-import { asString, toRepoPath } from './util.mjs';
+import { extractFrontmatter, parseSimpleFrontmatter, replaceFrontmatter } from './frontmatter.mjs';
+import { asString, toRepoPath, escapeRegex, warn } from './util.mjs';
 import { buildIndex, collectDocFiles } from './index.mjs';
 import { updateFrontmatter } from './lifecycle.mjs';
 import { bold, green, yellow, dim } from './color.mjs';
@@ -132,7 +132,7 @@ export function runLint(argv, config, opts = {}) {
         const { frontmatter: fm } = extractFrontmatter(raw);
         let newFm = fm;
         for (const kr of keyRenames) {
-          const regex = new RegExp(`^${kr.oldValue}:`, 'm');
+          const regex = new RegExp(`^${escapeRegex(kr.oldValue)}:`, 'm');
           newFm = newFm.replace(regex, `${kr.newValue}:`);
         }
         for (const tf of trimFixes) {
@@ -140,7 +140,7 @@ export function runLint(argv, config, opts = {}) {
           newFm = newFm.replace(regex, `$1${tf.newValue}`);
         }
         if (newFm !== fm) {
-          raw = raw.replace(`---\n${fm}\n---`, `---\n${newFm}\n---`);
+          raw = replaceFrontmatter(raw, newFm);
           writeFileSync(filePath, raw, 'utf8');
         }
       }
@@ -174,13 +174,10 @@ export function runLint(argv, config, opts = {}) {
     totalFixes += fixes.length;
 
     if (!dryRun) {
-      config.hooks.onLint?.({ path: repoPath, fixes });
+      try { config.hooks.onLint?.({ path: repoPath, fixes }); } catch (err) { warn(`Hook 'onLint' threw: ${err.message}`); }
     }
   }
 
   process.stdout.write(`\n${prefix}${totalFixes} fix${totalFixes !== 1 ? 'es' : ''} applied across ${fixable.length} file(s).\n`);
 }
 
-function escapeRegex(str) {
-  return str.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
-}

package/src/migrate.mjs

@@ -21,7 +21,6 @@ export function runMigrate(argv, config, opts = {}) {
 
   if (!field || !oldValue || !newValue) {
     die('Usage: dotmd migrate <field> <old-value> <new-value>');
-    return;
   }
 
   const allFiles = collectDocFiles(config);

package/src/new.mjs

@@ -1,6 +1,6 @@
 import { existsSync, writeFileSync } from 'node:fs';
 import path from 'node:path';
-import { toRepoPath, die } from './util.mjs';
+import { toRepoPath, die, warn } from './util.mjs';
 import { green, dim } from './color.mjs';
 
 export function runNew(argv, config, opts = {}) {
@@ -17,17 +17,16 @@ export function runNew(argv, config, opts = {}) {
   }
 
   const name = positional[0];
-  if (!name) { die('Usage: dotmd new <name> [--status <s>] [--title <t>]'); return; }
+  if (!name) { die('Usage: dotmd new <name> [--status <s>] [--title <t>]'); }
 
   // Validate status
   if (!config.validStatuses.has(status)) {
     die(`Invalid status: ${status}\nValid: ${[...config.validStatuses].join(', ')}`);
-    return;
   }
 
   // Slugify
   const slug = name.toLowerCase().replace(/[\s_]+/g, '-').replace(/[^a-z0-9-]/g, '').replace(/-+/g, '-').replace(/^-|-$/g, '');
-  if (!slug) { die('Name resolves to empty slug: ' + name); return; }
+  if (!slug) { die('Name resolves to empty slug: ' + name); }
 
   // Title
   const docTitle = title ?? name.replace(/[-_]/g, ' ').replace(/\b\w/g, c => c.toUpperCase());
@@ -38,7 +37,6 @@ export function runNew(argv, config, opts = {}) {
 
   if (existsSync(filePath)) {
     die(`File already exists: ${repoPath}`);
-    return;
   }
 
   if (dryRun) {
@@ -52,5 +50,5 @@ export function runNew(argv, config, opts = {}) {
   writeFileSync(filePath, content, 'utf8');
   process.stdout.write(`${green('Created')}: ${repoPath}\n`);
 
-  config.hooks.onNew?.({ path: repoPath, status, title: docTitle });
+  try { config.hooks.onNew?.({ path: repoPath, status, title: docTitle }); } catch (err) { warn(`Hook 'onNew' threw: ${err.message}`); }
 }

package/src/rename.mjs

@@ -1,6 +1,6 @@
 import { existsSync, readFileSync, writeFileSync } from 'node:fs';
 import path from 'node:path';
-import { extractFrontmatter, parseSimpleFrontmatter } from './frontmatter.mjs';
+import { extractFrontmatter, parseSimpleFrontmatter, replaceFrontmatter } from './frontmatter.mjs';
 import { toRepoPath, resolveDocPath, die, warn } from './util.mjs';
 import { collectDocFiles } from './index.mjs';
 import { gitMv } from './git.mjs';
@@ -21,14 +21,12 @@ export function runRename(argv, config, opts = {}) {
 
   if (!oldInput || !newInput) {
     die('Usage: dotmd rename <old> <new>');
-    return;
   }
 
   // Resolve old path
   const oldPath = resolveDocPath(oldInput, config);
   if (!oldPath) {
     die(`File not found: ${oldInput}\nSearched: ${toRepoPath(config.repoRoot, config.repoRoot) || '.'}, ${toRepoPath(config.docsRoot, config.repoRoot)}`);
-    return;
   }
 
   // Compute new path in same directory as old
@@ -109,7 +107,6 @@ export function runRename(argv, config, opts = {}) {
   const result = gitMv(oldPath, newPath, config.repoRoot);
   if (result.status !== 0) {
     die(result.stderr || 'git mv failed.');
-    return;
   }
 
   // Update references in frontmatter of other docs
@@ -127,7 +124,7 @@ export function runRename(argv, config, opts = {}) {
     }).join('\n');
 
     if (newFm !== fm) {
-      raw = raw.replace(`---\n${fm}\n---`, `---\n${newFm}\n---`);
+      raw = replaceFrontmatter(raw, newFm);
       writeFileSync(filePath, raw, 'utf8');
       updatedCount++;
     }
@@ -145,5 +142,5 @@ export function runRename(argv, config, opts = {}) {
     }
   }
 
-  config.hooks.onRename?.({ oldPath: oldRepoPath, newPath: newRepoPath, referencesUpdated: updatedCount });
+  try { config.hooks.onRename?.({ oldPath: oldRepoPath, newPath: newRepoPath, referencesUpdated: updatedCount }); } catch (err) { warn(`Hook 'onRename' threw: ${err.message}`); }
 }

package/src/render.mjs

@@ -1,10 +1,11 @@
-import { capitalize, toSlug, truncate } from './util.mjs';
+import { capitalize, toSlug, truncate, warn } from './util.mjs';
 import { bold, red, yellow, green } from './color.mjs';
 
 export function renderCompactList(index, config) {
   const defaultRenderer = (idx) => _renderCompactList(idx, config);
   if (config.hooks.renderCompactList) {
-    return config.hooks.renderCompactList(index, defaultRenderer);
+    try { return config.hooks.renderCompactList(index, defaultRenderer); }
+    catch (err) { warn(`Hook 'renderCompactList' threw: ${err.message}`); }
   }
   return defaultRenderer(index);
 }
@@ -63,7 +64,8 @@ export function renderVerboseList(index, config) {
 export function renderContext(index, config) {
   const defaultRenderer = (idx) => _renderContext(idx, config);
   if (config.hooks.renderContext) {
-    return config.hooks.renderContext(index, defaultRenderer);
+    try { return config.hooks.renderContext(index, defaultRenderer); }
+    catch (err) { warn(`Hook 'renderContext' threw: ${err.message}`); }
   }
   return defaultRenderer(index);
 }
@@ -144,7 +146,8 @@ function _renderContext(index, config) {
 export function renderCheck(index, config) {
   const defaultRenderer = (idx) => _renderCheck(idx);
   if (config.hooks.renderCheck) {
-    return config.hooks.renderCheck(index, defaultRenderer);
+    try { return config.hooks.renderCheck(index, defaultRenderer); }
+    catch (err) { warn(`Hook 'renderCheck' threw: ${err.message}`); }
   }
   return defaultRenderer(index);
 }
@@ -234,7 +237,8 @@ export function renderProgressBar(checklist) {
 export function formatSnapshot(doc, config) {
   const defaultFormatter = (d) => _formatSnapshot(d);
   if (config.hooks.formatSnapshot) {
-    return config.hooks.formatSnapshot(doc, defaultFormatter);
+    try { return config.hooks.formatSnapshot(doc, defaultFormatter); }
+    catch (err) { warn(`Hook 'formatSnapshot' threw: ${err.message}`); }
   }
   return defaultFormatter(doc);
 }

package/src/util.mjs

@@ -47,6 +47,10 @@ export function mergeUniqueStrings(...lists) {
   return [...new Set(lists.flat().filter(Boolean))];
 }
 
+export function escapeRegex(str) {
+  return str.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+}
+
 export function toRepoPath(absolutePath, repoRoot) {
   return path.relative(repoRoot, absolutePath).split(path.sep).join('/');
 }
@@ -55,9 +59,15 @@ export function warn(message) {
   process.stderr.write(`${dim(message)}\n`);
 }
 
+export class DotmdError extends Error {
+  constructor(message) {
+    super(message);
+    this.name = 'DotmdError';
+  }
+}
+
 export function die(message) {
-  process.stderr.write(`${message}\n`);
-  process.exitCode = 1;
+  throw new DotmdError(message);
 }
 
 export function resolveDocPath(input, config) {