dotmd-cli 0.31.2 → 0.31.3

package/bin/dotmd.mjs

@@ -690,10 +690,11 @@ async function main() {
 
   const config = await resolveConfig(process.cwd(), explicitConfig);
 
-  // Init — now has access to config for Claude command generation
+  // Init — runInit re-resolves the config from disk internally (after any
+  // starter-config write), so we don't need to pre-pass it.
   if (command === 'init') {
     const { runInit } = await import('../src/init.mjs');
-    runInit(process.cwd(), config.configFound ? config : null, { dryRun });
+    await runInit(process.cwd(), config, { dryRun });
     return;
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dotmd-cli",
-  "version": "0.31.2",
+  "version": "0.31.3",
   "description": "CLI for managing markdown documents with YAML frontmatter — index, query, validate, graph, export, Notion sync, AI summaries.",
   "type": "module",
   "license": "MIT",

package/src/doctor.mjs

@@ -67,27 +67,39 @@ export function runDoctor(argv, config, opts = {}) {
   process.stdout.write('\n' + bold('3. Syncing dates from git...') + '\n');
   runTouch(['--git'], config, { dryRun });
 
-  // Step 4: Regenerate index
-  if (config.indexPath) {
-    process.stdout.write('\n' + bold('4. Regenerating index...') + '\n');
-    if (!dryRun) {
-      const index = buildIndex(config);
-      writeIndex(renderIndexFile(index, config), config);
-      process.stdout.write('Index updated.\n');
-    } else {
-      process.stdout.write('[dry-run] Would regenerate index.\n');
-    }
+  // Step 4: Regenerate index. Heading always prints so the numbering stays
+  // `1,2,3,4,5,6` even when `index.path` isn't configured — pre-fix this was
+  // gated on `config.indexPath`, producing `1,2,3,5,6` on repos with no index.
+  process.stdout.write('\n' + bold('4. Regenerating index...') + '\n');
+  if (!config.indexPath) {
+    process.stdout.write('No index path configured (skip).\n');
+  } else if (dryRun) {
+    process.stdout.write('[dry-run] Would regenerate index.\n');
+  } else {
+    const index = buildIndex(config);
+    writeIndex(renderIndexFile(index, config), config);
+    process.stdout.write('Index updated.\n');
   }
 
-  // Step 5: Refresh Claude Code commands
-  const claudeResults = dryRun ? [] : scaffoldClaudeCommands(config.repoRoot, config);
-  if (claudeResults.some(r => r.action !== 'current' && r.action !== 'skipped')) {
-    process.stdout.write('\n' + bold('5. Claude Code commands:') + '\n');
-    for (const r of claudeResults) {
-      if (r.action === 'updated') {
-        process.stdout.write(`${green('Updated')} .claude/commands/${r.name} (v${r.from} → v${r.to})\n`);
-      } else if (r.action === 'created') {
-        process.stdout.write(`${green('Created')} .claude/commands/${r.name}\n`);
+  // Step 5: Refresh Claude Code commands. Always print the heading so the
+  // numbering stays `1,2,3,4,5,6` — pre-fix it was conditional, so a doctor
+  // run where everything was already current printed `1,2,3,4,6` with `5.`
+  // silently missing.
+  process.stdout.write('\n' + bold('5. Claude Code commands:') + '\n');
+  if (dryRun) {
+    process.stdout.write('[dry-run] Would refresh .claude/commands/ if outdated.\n');
+  } else {
+    const claudeResults = scaffoldClaudeCommands(config.repoRoot, config);
+    const changes = claudeResults.filter(r => r.action === 'updated' || r.action === 'created');
+    if (changes.length === 0) {
+      process.stdout.write('Nothing to refresh.\n');
+    } else {
+      for (const r of changes) {
+        if (r.action === 'updated') {
+          process.stdout.write(`${green('Updated')} .claude/commands/${r.name} (v${r.from} → v${r.to})\n`);
+        } else if (r.action === 'created') {
+          process.stdout.write(`${green('Created')} .claude/commands/${r.name}\n`);
+        }
       }
     }
   }

package/src/init.mjs

@@ -4,6 +4,7 @@ import { extractFrontmatter, parseSimpleFrontmatter } from './frontmatter.mjs';
 import { green, dim, yellow } from './color.mjs';
 import { warn } from './util.mjs';
 import { scaffoldClaudeCommands } from './claude-commands.mjs';
+import { resolveConfig } from './config.mjs';
 
 // Subdirectories scaffolded under docsRoot and tracked separately during scans.
 // Each maps to a builtin type (plan, prompt). New types added here should also
@@ -151,7 +152,7 @@ function generateDetectedConfig(scan, rootPath) {
   return lines.join('\n');
 }
 
-export function runInit(cwd, config, opts = {}) {
+export async function runInit(cwd, config, opts = {}) {
   const { dryRun = false } = opts;
   const configPath = path.join(cwd, 'dotmd.config.mjs');
   const docsDir = path.join(cwd, 'docs');
@@ -260,11 +261,20 @@ export function runInit(cwd, config, opts = {}) {
   }
 
   // Claude Code integration — auto-detect .claude/ directory.
+  // Re-resolve config so the scaffold sees whatever we (may have) just written.
+  // Pre-fix: the dispatcher passed `null` to runInit on a fresh repo because
+  // resolveConfig was called before init wrote the starter, so the `if (config)`
+  // gate below silently skipped slash-command scaffolding entirely on first init.
+  // Re-resolving picks up STARTER_CONFIG (or any pre-existing config) in the
+  // real-run path; in dry-run with no on-disk config, it returns the merged
+  // DEFAULTS, which is enough for the preview line (`would create…`).
+  //
   // Reports all four scaffold outcomes so the user can't be surprised by
   // either a silent regenerate (pre-fix: `updated` was unreported) or by
   // dotmd skipping a user-managed file (pre-fix: `skipped` was unreported).
-  if (config) {
-    const results = scaffoldClaudeCommands(cwd, config, { dryRun });
+  const scaffoldConfig = await resolveConfig(cwd);
+  if (scaffoldConfig) {
+    const results = scaffoldClaudeCommands(cwd, scaffoldConfig, { dryRun });
     for (const r of results) {
       const filename = `.claude/commands/${r.name}`;
       if (r.action === 'created') {

package/src/pickup-card.mjs

@@ -1,7 +1,7 @@
 import { readFileSync, existsSync } from 'node:fs';
 import path from 'node:path';
 import { extractFrontmatter, parseSimpleFrontmatter } from './frontmatter.mjs';
-import { asString, toRepoPath, resolveDocPath } from './util.mjs';
+import { asString, toRepoPath, resolveDocPath, resolveRefPath } from './util.mjs';
 import { walkSections, findSection, findActivePhase, summarizePhases, isPhaseHeading, detectMarker } from './section.mjs';
 import { dim, green } from './color.mjs';
 
@@ -29,7 +29,14 @@ function statusSummary(counts) {
   return order.filter(k => counts[k]).map(k => `${counts[k]}${icons[k]}`).join(' ');
 }
 
-function readRelatedSummary(rawList, config) {
+// `docDir` is the directory of the doc whose frontmatter we're reading.
+// Pre-fix: this used `resolveDocPath` which only tries repo-root and docsRoots-
+// relative, NOT doc-relative — so a bare basename like `sibling-plan.md` written
+// in `docs/plans/foo.md`'s `related_plans:` always showed `(missing)`, even
+// though graph/validate resolve the same ref fine via `resolveRefPath`. Now
+// matches graph's resolver semantics: doc-relative first, then repo-relative;
+// docsRoots-relative is kept as a final fallback for legacy refs.
+function readRelatedSummary(rawList, config, docDir) {
   const list = Array.isArray(rawList) ? rawList : (typeof rawList === 'string' && rawList.trim() ? [rawList] : []);
   const out = [];
   for (const ref of list) {
@@ -37,7 +44,10 @@ function readRelatedSummary(rawList, config) {
     const refStr = String(ref).trim();
     if (!refStr) continue;
     let abs = null;
-    try { abs = resolveDocPath(refStr, config); } catch { abs = null; }
+    try {
+      abs = resolveRefPath(refStr, docDir, config.repoRoot)
+        ?? resolveDocPath(refStr, config);
+    } catch { abs = null; }
     if (!abs || !existsSync(abs)) {
       out.push({ ref: refStr, status: null, missing: true });
       continue;
@@ -96,10 +106,13 @@ export function buildCard(filePath, raw, config) {
   const currentState = truncate(cleanInline(fm.current_state), CAPS.currentState);
   const nextStep = truncate(cleanInline(fm.next_step), CAPS.nextStep);
 
-  // Related plans (compressed: slug + status only — show all, don't cap count)
+  // Related plans (compressed: slug + status only — show all, don't cap count).
+  // docDir lets the resolver try same-dir basenames first — graph/validate do this
+  // already; pickup-card now matches.
+  const docDir = path.dirname(filePath);
   const related = [
-    ...readRelatedSummary(fm.parent_plan, config).map(r => ({ ...r, kind: 'parent' })),
-    ...readRelatedSummary(fm.related_plans, config).map(r => ({ ...r, kind: 'related' })),
+    ...readRelatedSummary(fm.parent_plan, config, docDir).map(r => ({ ...r, kind: 'parent' })),
+    ...readRelatedSummary(fm.related_plans, config, docDir).map(r => ({ ...r, kind: 'related' })),
   ];
 
   // Phases summary + active phase (pointer only, no body)

package/src/render.mjs

@@ -297,7 +297,13 @@ export function renderBriefing(index, config) {
   if (parts.length) lines.push(parts.join(' | '));
 
   const stale = index.docs.filter(d => d.isStale && !config.lifecycle.skipStaleFor.has(d.status)).length;
-  lines.push(`Stale: ${stale} | Errors: ${index.errors.length} | Warnings: ${index.warnings.length}`);
+  // Append a hint when errors are present — otherwise the user sees `Errors: 1`
+  // with no clue what or where. `dotmd check` is the canonical detail view.
+  const errorCount = index.errors.length;
+  const errorPart = errorCount > 0
+    ? `Errors: ${errorCount} ${dim('(run `dotmd check` to see)')}`
+    : `Errors: ${errorCount}`;
+  lines.push(`Stale: ${stale} | ${errorPart} | Warnings: ${index.warnings.length}`);
 
   try {
     const staleLeases = findStaleLeases(config);