dotmd-cli 0.68.0 → 0.70.0

package/src/rename.mjs

@@ -1,107 +1,99 @@
-import { existsSync, readFileSync, writeFileSync } from 'node:fs';
+import { existsSync, readFileSync } from 'node:fs';
 import path from 'node:path';
 import { toRepoPath, die, warn } from './util.mjs';
 import { collectDocFiles, resolveDocArg } from './index.mjs';
 import { regenIndex } from './lifecycle.mjs';
-import { gitMv } from './git.mjs';
+import { captureGitIndexGeneration, isTracked } from './git.mjs';
 import { green, dim } from './color.mjs';
 import { isInteractive, promptText } from './prompt.mjs';
+import { authorizeManagedDestination, authorizeManagedSource, authorizeManagedSweep } from './managed-path.mjs';
+import { availableSessionId, prepareOwnershipMigration } from './pickup.mjs';
+import { moveFileAtomic } from './atomic-mutation.mjs';
+import { configuredReferenceFields, createReferenceIdentitySet, planReferenceMove, rewriteDocumentReferences } from './reference-planner.mjs';
 
 export async function runRename(argv, config, opts = {}) {
   const { dryRun } = opts;
-
-  // Parse positional args (skip flags)
-  const positional = [];
-  for (let i = 0; i < argv.length; i++) {
-    if (argv[i].startsWith('-')) continue;
-    positional.push(argv[i]);
-  }
-
+  const positional = argv.filter(arg => !arg.startsWith('-'));
   const oldInput = positional[0];
   let newInput = positional[1];
-
-  if (!oldInput) { die('Usage: dotmd rename <old> <new>'); }
+  if (!oldInput) die('Usage: dotmd rename <old> <new>');
   if (!newInput) {
-    if (isInteractive()) {
-      newInput = await promptText('New name: ');
-      if (!newInput) die('No name provided.');
-    } else {
-      die('Usage: dotmd rename <old> <new>');
-    }
+    if (!isInteractive()) die('Usage: dotmd rename <old> <new>');
+    newInput = await promptText('New name: ');
+    if (!newInput) die('No name provided.');
   }
 
-  // Resolve old path
-  const oldPath = resolveDocArg(oldInput, config);
-
-  // Compute new path — cross-directory if input has slashes, same directory otherwise
+  let oldPath = resolveDocArg(oldInput, config);
+  const sourceAuthorization = authorizeManagedSource(oldPath, config, { kind: 'Rename source' });
+  oldPath = sourceAuthorization.path;
   let newPath;
   if (newInput.includes('/') || newInput.includes(path.sep)) {
-    let resolved = newInput;
-    if (!resolved.endsWith('.md')) resolved += '.md';
-    newPath = path.resolve(config.repoRoot, resolved);
+    newPath = path.resolve(config.repoRoot, newInput.endsWith('.md') ? newInput : `${newInput}.md`);
   } else {
-    let newBasename = newInput;
-    if (!newBasename.endsWith('.md')) newBasename += '.md';
-    newPath = path.join(path.dirname(oldPath), newBasename);
-  }
-
-  if (existsSync(newPath)) {
-    die(`Target already exists: ${toRepoPath(newPath, config.repoRoot)}`);
-    return;
+    newPath = path.join(path.dirname(oldPath), newInput.endsWith('.md') ? newInput : `${newInput}.md`);
   }
+  if (existsSync(newPath)) die(`Target already exists: ${toRepoPath(newPath, config.repoRoot)}`);
+  newPath = authorizeManagedDestination(newPath, config, { root: sourceAuthorization.root, kind: 'Rename destination' }).path;
 
   const oldRepoPath = toRepoPath(oldPath, config.repoRoot);
   const newRepoPath = toRepoPath(newPath, config.repoRoot);
-  const oldBasename = path.basename(oldPath);
-  const newBasename = path.basename(newPath);
-
-  // Scan for references in other docs
   const allFiles = collectDocFiles(config);
-  const filesToUpdate = [];
-
-  for (const filePath of allFiles) {
-    if (filePath === oldPath) continue;
-    const raw = readFileSync(filePath, 'utf8');
-    if (raw.includes(oldBasename)) {
-      filesToUpdate.push(filePath);
-    }
+  authorizeManagedSweep(allFiles, config, { kind: 'Rename reference rewrite source' });
+  const documents = allFiles.map(filePath => ({ path: filePath, content: readFileSync(filePath, 'utf8') }));
+  let sourceDocument = documents.find(document => document.path === oldPath);
+  if (!sourceDocument) {
+    sourceDocument = { path: oldPath, content: readFileSync(oldPath, 'utf8') };
+    documents.push(sourceDocument);
   }
+  const sourceContent = sourceDocument.content;
+  const referenceFields = configuredReferenceFields(config);
+  const referencePlan = planReferenceMove({ documents, oldPath, newPath, repoRoot: config.repoRoot, referenceFields });
+  const ownership = prepareOwnershipMigration(oldRepoPath, newPath, config, { sessionId: availableSessionId() });
 
   if (dryRun) {
     const prefix = dim('[dry-run]');
     process.stdout.write(`${prefix} Would rename: ${oldRepoPath} → ${newRepoPath}\n`);
-    if (filesToUpdate.length > 0) {
-      process.stdout.write(`${prefix} Would update references in ${filesToUpdate.length} file(s):\n`);
-      for (const f of filesToUpdate) {
-        process.stdout.write(`${prefix}   ${toRepoPath(f, config.repoRoot)}\n`);
-      }
+    if (referencePlan.updates.length) {
+      process.stdout.write(`${prefix} Would update references in ${referencePlan.updates.length} file(s):\n`);
+      for (const item of referencePlan.updates) process.stdout.write(`${prefix}   ${toRepoPath(item.path, config.repoRoot)}\n`);
     }
+    if (ownership) process.stdout.write(`${prefix} Would migrate this session's ownership record.\n`);
     return;
   }
 
-  // Perform git mv
-  const result = gitMv(oldPath, newPath, config.repoRoot);
-  if (result.status !== 0) {
-    die(result.stderr || 'git mv failed.');
-  }
-
-  // Update all references (frontmatter + body) in other docs
-  let updatedCount = 0;
-  for (const filePath of filesToUpdate) {
-    const raw = readFileSync(filePath, 'utf8');
-    const updated = raw.split(oldBasename).join(newBasename);
-    if (updated !== raw) {
-      writeFileSync(filePath, updated, 'utf8');
-      updatedCount++;
-    }
-  }
-
+  const tracked = isTracked(oldPath, config.repoRoot);
+  const gitIndex = tracked ? captureGitIndexGeneration(config.repoRoot) : null;
+  const identities = createReferenceIdentitySet(allFiles);
+  const result = moveFileAtomic(oldPath, newPath, current => rewriteDocumentReferences(current, {
+    sourcePath: oldPath,
+    outputPath: newPath,
+    repoRoot: config.repoRoot,
+    identities,
+    oldPath,
+    newPath,
+    referenceFields,
+    rebaseAll: true,
+  }), {
+    repoRoot: config.repoRoot,
+    config,
+    updates: allFiles.filter(filePath => filePath !== oldPath).map(filePath => ({
+      path: filePath,
+      render: current => rewriteDocumentReferences(current, {
+        sourcePath: filePath, repoRoot: config.repoRoot, identities, oldPath, newPath, referenceFields,
+      }),
+    })),
+    creations: ownership ? [{ path: ownership.newRecordPath, content: ownership.newContent, label: 'ownership' }] : [],
+    deletions: ownership ? [{ path: ownership.oldRecordPath, expectedContent: ownership.oldContent, label: 'ownership' }] : [],
+    gitMove: tracked,
+    gitIndex,
+    operation: 'rename',
+    sessionId: availableSessionId(),
+    testHooks: opts.testHooks,
+  });
   regenIndex(config);
-
   process.stdout.write(`${green('Renamed')}: ${oldRepoPath} → ${newRepoPath}\n`);
-  if (updatedCount > 0) {
-    process.stdout.write(`Updated references in ${updatedCount} file(s).\n`);
-  }
-
-  try { config.hooks.onRename?.({ oldPath: oldRepoPath, newPath: newRepoPath, referencesUpdated: updatedCount }); } catch (err) { warn(`Hook 'onRename' threw: ${err.message}`); }
+  if (result.updatedPaths.length) process.stdout.write(`Updated references in ${result.updatedPaths.length} file(s).\n`);
+  try { config.hooks.onRename?.({ oldPath: oldRepoPath, newPath: newRepoPath, referencesUpdated: result.updatedPaths.length }); }
+  catch (err) { warn(`Hook 'onRename' threw: ${err.message}`); }
+  return { oldRepoPath, newRepoPath, referencePaths: result.updatedPaths.map(item => toRepoPath(item, config.repoRoot)) };
 }

package/src/render.mjs

@@ -6,6 +6,7 @@ import { summarizeDocBody } from './ai.mjs';
 import { bold, red, yellow, green, dim } from './color.mjs';
 import { categorizeWarnings } from './check-collapse.mjs';
 import { buildCoordinationIndex, isRoadmapHub } from './runlist.mjs';
+import { resolveStatusMetadata, statusMetadataFor } from './status-metadata.mjs';
 
 // Render `currentState` with an `(auto)` prefix when the value was body-scraped
 // rather than read from frontmatter. Lets a reader see at a glance which docs
@@ -20,7 +21,7 @@ export function formatCurrentState(doc) {
 
 export function renderCompactList(index, config) {
   const defaultRenderer = (idx) => _renderCompactList(idx, config);
-  if (config.hooks.renderCompactList) {
+  if (!config._execution?.suppressSideEffects && config.hooks.renderCompactList) {
     try { return config.hooks.renderCompactList(index, defaultRenderer); }
     catch (err) { warn(`Hook 'renderCompactList' threw: ${err.message}`); }
   }
@@ -162,7 +163,7 @@ export function renderVerboseList(index, config) {
 
 export function renderContext(index, config, opts = {}) {
   const defaultRenderer = (idx) => _renderContext(idx, config, opts);
-  if (config.hooks.renderContext) {
+  if (!config._execution?.suppressSideEffects && config.hooks.renderContext) {
     try { return config.hooks.renderContext(index, defaultRenderer); }
     catch (err) { warn(`Hook 'renderContext' threw: ${err.message}`); }
   }
@@ -190,7 +191,7 @@ function _renderContextSection(docs, ctx, opts, config, lines) {
         ? truncate(doc.nextStep, ctx.truncateNextStep || 80)
         : '(no next step)';
       lines.push(`  ${slug}${ageTag}  next: ${next}`);
-      if (opts.summarize) {
+      if (opts.summarize && !config._execution?.suppressSideEffects) {
         try {
           const absPath = path.resolve(config.repoRoot, doc.path);
           const raw = readFileSync(absPath, 'utf8');
@@ -225,6 +226,9 @@ function _renderContextSection(docs, ctx, opts, config, lines) {
 function _renderContext(index, config, opts = {}) {
   const today = new Date().toISOString().slice(0, 10);
   const lines = [`BRIEFING (${today})`, ''];
+  if (opts.summarize && config._execution?.suppressSideEffects) {
+    lines.push(dim('[preview] AI summaries skipped; models and custom summarizeDoc hooks are not invoked.'), '');
+  }
 
   // Group docs by type
   const typeOrder = ['plan', 'doc', 'research'];
@@ -352,11 +356,16 @@ export function renderBriefing(index, config) {
     // headline plan count entirely (they get their own `N runlists · dotmd
     // runlists` pointer line below). The breakdown is leaf statuses only and sums
     // back to the leaf count, so "N live plans" means N things to actually work on.
-    const counts = Object.entries(bySt).map(([s, n]) => `${n} ${s}`).join(', ');
+    const planStatusOrder = (resolveStatusMetadata(config).byType.plan ?? []).map(item => item.name);
+    const orderedStatuses = [
+      ...planStatusOrder.filter(status => bySt[status]),
+      ...Object.keys(bySt).filter(status => !planStatusOrder.includes(status)).sort(),
+    ];
+    const counts = orderedStatuses.map(status => `${bySt[status]} ${status}`).join(', ');
     const closedCount = plans.length - live.length;
     const closedPart = closedCount ? ` (${closedCount} archived)` : '';
     lines.push(liveLeaves ? `${liveLeaves} live plans${closedPart}: ${counts}` : `0 live plans${closedPart}`);
-    const show = plans.filter(p => (p.status === 'in-session' || p.status === 'active') && !isHub(p));
+    const show = plans.filter(p => statusMetadataFor(config, 'plan', p.status)?.context === 'expanded' && !isHub(p));
     for (const p of show) {
       const next = p.nextStep ? `next: ${p.nextStep}` : '(no next step)';
       lines.push(`  > ${path.basename(p.path, '.md')} (${p.status}) ${next}`);
@@ -382,7 +391,7 @@ export function renderBriefing(index, config) {
   if (untyped.length) parts.push(`${untyped.length} untyped`);
   if (parts.length) lines.push(parts.join(' | '));
 
-  const stale = index.docs.filter(d => d.isStale && !config.lifecycle.skipStaleFor.has(d.status)).length;
+  const stale = index.docs.filter(d => d.isStale && !statusMetadataFor(config, d.type, d.status)?.skipStale).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;
@@ -396,7 +405,7 @@ export function renderBriefing(index, config) {
 
 export function renderCheck(index, config, opts = {}) {
   const defaultRenderer = (idx) => _renderCheck(idx, opts);
-  if (config.hooks.renderCheck) {
+  if (!config._execution?.suppressSideEffects && config.hooks.renderCheck) {
     try { return config.hooks.renderCheck(index, defaultRenderer); }
     catch (err) { warn(`Hook 'renderCheck' threw: ${err.message}`); }
   }
@@ -586,7 +595,7 @@ export function renderProgressBar(checklist) {
 
 export function formatSnapshot(doc, config) {
   const defaultFormatter = (d) => _formatSnapshot(d, config);
-  if (config.hooks.formatSnapshot) {
+  if (!config._execution?.suppressSideEffects && config.hooks.formatSnapshot) {
     try { return config.hooks.formatSnapshot(doc, defaultFormatter); }
     catch (err) { warn(`Hook 'formatSnapshot' threw: ${err.message}`); }
   }