dotmd-cli 0.48.1 → 0.48.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dotmd-cli",
-  "version": "0.48.1",
+  "version": "0.48.3",
   "description": "CLI for managing markdown documents with YAML frontmatter — index, query, validate, graph, export, Notion sync, AI summaries.",
   "type": "module",
   "license": "MIT",
@@ -40,6 +40,7 @@
   "scripts": {
     "test": "node --test test/*.test.mjs",
     "preversion": "npm test",
+    "version": "node bin/dotmd.mjs hud >/dev/null 2>&1; git add .claude/commands docs/docs.md 2>/dev/null; true",
     "postversion": "git push origin main --tags && gh release create v$npm_package_version --generate-notes --title v$npm_package_version && sleep 5 && gh run watch $(gh run list --workflow=publish.yml --limit 1 --json databaseId --jq '.[0].databaseId') --exit-status && sleep 10 && npm cache clean --force && npm install -g dotmd-cli@$npm_package_version"
   },
   "engines": {

package/src/claude-commands.mjs

@@ -229,22 +229,12 @@ export function refreshStaleSlashCommands(config) {
   return results.filter(r => r.action === 'updated');
 }
 
-export function checkClaudeCommands(cwd, opts = {}) {
-  const { version = pkg.version } = opts;
-  const commandsDir = path.join(cwd, '.claude', 'commands');
-  if (!existsSync(commandsDir)) return [];
-
-  const warnings = [];
-  for (const name of ['plans.md', 'docs.md', 'baton.md']) {
-    const filePath = path.join(commandsDir, name);
-    const installedVersion = getInstalledVersion(filePath);
-    if (installedVersion && installedVersion !== version) {
-      warnings.push({
-        path: `.claude/commands/${name}`,
-        level: 'warning',
-        message: `Claude command outdated (v${installedVersion} → v${version}). Run \`dotmd doctor\` to update.`,
-      });
-    }
-  }
-  return warnings;
+// Intentionally returns []. Slash-command stamp drift is auto-healed every
+// time `dotmd hud` runs (SessionStart hook), and `dotmd doctor` regens them
+// on demand. Surfacing a warning at `dotmd check` time was pure noise — it
+// fired on every release until the next session, despite the user having no
+// action to take (the heal is automatic). Kept the function for API stability
+// in case downstream callers import it.
+export function checkClaudeCommands(_cwd, _opts = {}) {
+  return [];
 }

package/src/new.mjs

@@ -22,6 +22,23 @@ function surfacesScaffold(ctx) {
   return 'surfaces:';
 }
 
+// Module-taxonomy parallel of surfacesScaffold. When `taxonomy.modules` is set,
+// scaffold lists valid values + a `- none` placeholder so the validator's
+// modules-required check passes by default and the author can swap to a real
+// module from the listed taxonomy. When unset, the project doesn't enumerate
+// modules — drop the placeholder entirely so new plans aren't sprinkled with a
+// meaningless sentinel.
+function modulesScaffold(ctx, kind /* 'doc' | 'plan' */) {
+  const valid = ctx?.validModules;
+  if (Array.isArray(valid) && valid.length > 0) {
+    const comment = kind === 'plan'
+      ? `# modules — valid: ${valid.join(', ')} (or \`none\` for tooling/infra plans)`
+      : `# modules — valid: ${valid.join(', ')} (or \`none\` for platform/infra docs)`;
+    return `${comment}\nmodules:\n  - none`;
+  }
+  return 'modules:';
+}
+
 const BUILTIN_TEMPLATES = {
   doc: {
     description: 'Reference doc, design note, module overview — build-up shape lite',
@@ -35,9 +52,7 @@ const BUILTIN_TEMPLATES = {
       `status: ${s}`,
       `created: ${d}`,
       `updated: ${d}`,
-      '# modules — real module name(s), or `none` for platform/infra docs',
-      'modules:',
-      '  - none',
+      modulesScaffold(ctx, 'doc'),
       surfacesScaffold(ctx),
       'domain:',
       'audience: internal',
@@ -75,9 +90,7 @@ ${ctx?.bodyInput?.trim() ?? ''}
       `created: ${d}`,
       `updated: ${d}`,
       surfacesScaffold(ctx),
-      '# modules — real module name(s), or `none` for tooling/infra plans',
-      'modules:',
-      '  - none',
+      modulesScaffold(ctx, 'plan'),
       'domain:',
       'audience: internal',
       'parent_plan:',
@@ -477,7 +490,8 @@ export async function runNew(argv, config, opts = {}) {
   // Generate content
   let content;
   const validSurfaces = config.raw?.taxonomy?.surfaces ?? (config.validSurfaces ? [...config.validSurfaces] : null);
-  const tmplCtx = { status, title: docTitle, today, bodyInput, validSurfaces };
+  const validModules = config.raw?.taxonomy?.modules ?? (config.validModules ? [...config.validModules] : null);
+  const tmplCtx = { status, title: docTitle, today, bodyInput, validSurfaces, validModules };
   if (typeof template === 'function') {
     content = template(name, tmplCtx);
   } else {

package/src/runlist.mjs

@@ -1,4 +1,4 @@
-import { readFileSync } from 'node:fs';
+import { existsSync, readFileSync } from 'node:fs';
 import path from 'node:path';
 import { extractFrontmatter, parseSimpleFrontmatter } from './frontmatter.mjs';
 import {
@@ -14,6 +14,34 @@ import { bold, cyan, dim, green, red, yellow } from './color.mjs';
 
 const PICKUPABLE_STATUSES = new Set(['active', 'planned', 'in-session']);
 
+// resolveDocPath only tries cwd / repo-root / docs-root joins, so bare plan
+// slugs like `clear-the-deck` don't resolve when plans live under
+// `<docsRoot>/plans/`. The other commands (`dotmd set <slug>`, `dotmd plans`)
+// take slugs — runlist should too. Try the direct resolve first, then
+// fall back to `<root>/plans/<slug>.md` under each configured doc root.
+function resolveHubInput(input, config) {
+  const direct = resolveDocPath(input, config);
+  if (direct) return direct;
+
+  if (!input.endsWith('.md')) {
+    const withExt = resolveDocPath(input + '.md', config);
+    if (withExt) return withExt;
+  }
+
+  const slugFile = input.endsWith('.md') ? input : `${input}.md`;
+  const roots = config.docsRoots || (config.docsRoot ? [config.docsRoot] : []);
+  for (const root of roots) {
+    const candidate = path.join(root, 'plans', slugFile);
+    if (existsSync(candidate)) return candidate;
+    // Multi-root layouts may already have a `plans` root, so also try the
+    // root itself.
+    const rootCandidate = path.join(root, slugFile);
+    if (existsSync(rootCandidate)) return rootCandidate;
+  }
+
+  return null;
+}
+
 // Read a hub plan's `runlist:` and resolve each entry to a repo-relative path
 // plus its current status. Missing files are reported with `missing: true`;
 // callers decide how to render them. Pure: no IO beyond file reads.
@@ -112,7 +140,7 @@ export async function runRunlist(argv, config, opts = {}) {
       : 'Usage: dotmd runlist <hub-plan>');
   }
 
-  const hubAbs = resolveDocPath(hubInput, config);
+  const hubAbs = resolveHubInput(hubInput, config);
   if (!hubAbs) die(`Hub plan not found: ${hubInput}`);
   const hubRepoPath = toRepoPath(hubAbs, config.repoRoot);