simt-dev 0.7.0 → 0.8.0

package/src/commands/ignore.mjs

@@ -11,12 +11,13 @@ import { join } from 'node:path';
 
 import { EXIT, emit, emitJson, fail, isJson, note, writeMode } from '../output/index.mjs';
 import { CLI } from '../name.mjs';
-import { commitJournal, journaledWriteFile } from '../journal.mjs';
+import { applyPlan, change } from '../write/plan.mjs';
 import { diffField, emitDiffs, readBefore } from '../diff.mjs';
 import { CLAUDE_SETTINGS, IGNORE_FILE, parseIgnoreList, planIgnore } from '../ignore.mjs';
+import { commandRoot } from '../repo-root.mjs';
 
 export async function ignoreCommand(args, flags) {
-  const root = flags.root ?? process.cwd();
+  const root = commandRoot(flags);
 
   let source;
   try {
@@ -124,12 +125,16 @@ export async function ignoreCommand(args, flags) {
 
   // Journaled so `undo` restores this run (R49). Every note writes to stderr,
   // so the `--json` document remains the only bytes on stdout.
-  const journal = [];
-  for (const t of changed) {
-    await journaledWriteFile(root, journal, join(root, t.path), t.after);
-    note(`wrote    ${t.path} (${t.action})`);
-  }
-  await commitJournal(root, 'ignore', journal);
+  await applyPlan(
+    root,
+    await Promise.all(
+      changed.map(async (t) =>
+        change({ path: t.path, before: await readBefore(join(root, t.path)), after: t.after, owner: 'ignore', reason: t.action }),
+      ),
+    ),
+    { mode: { apply: true }, label: 'ignore' },
+  );
+  for (const t of changed) note(`wrote    ${t.path} (${t.action})`);
 
   if (isJson(flags)) emitJson(payload(true));
   // After the report: the report says what, the diff shows it (R50).

package/src/commands/impact.mjs

@@ -4,12 +4,10 @@
 // the file, the CODEOWNERS line, and — when a workspace is configured —
 // another repository that references a contract this change touches.
 
-import { execFile } from 'node:child_process';
 import { readFile } from 'node:fs/promises';
 import { basename, join, resolve } from 'node:path';
-import { promisify } from 'node:util';
 
-import { EXIT, emit, emitJson, fail, isJson } from '../output/index.mjs';
+import { EXIT, fail, present } from '../output/index.mjs';
 import { CLI } from '../name.mjs';
 import { discover } from '../compare/discover.mjs';
 import { extract } from '../index/extract.mjs';
@@ -22,11 +20,12 @@ import {
   ownersOf,
   relatedContracts,
 } from '../index/impact.mjs';
-import { decisionHits, scopeHits, sortHits, sourceRoots } from '../index/govern.mjs';
+import { governs, sourceRoots } from '../index/govern.mjs';
 import { decisionRecords, ruleScopes } from './why.mjs';
 import { resolveWorkspace } from '../workspace.mjs';
+import { commandRoot } from '../repo-root.mjs';
+import { gitPaths } from '../sys/git.mjs';
 
-const exec = promisify(execFile);
 const TEXT_EXT = new Set(['js', 'mjs', 'cjs', 'jsx', 'ts', 'tsx', 'json', 'yaml', 'yml', 'md', 'toml', 'xml', 'py', 'go', 'kt', 'java', 'rs']);
 const READ_CAP = 256 * 1024;
 const FILE_CAP = 2000;
@@ -35,7 +34,7 @@ export async function impactCommand(args, flags, commands = []) {
   if (args.length > 1) return fail(`unknown impact argument ${args[1]}`, EXIT.USAGE, `run: ${CLI} impact <file|symbol>`);
   const target = args[0];
   if (!target) return fail(`a file or a symbol is required`, EXIT.USAGE, `run: ${CLI} impact <file|symbol>`);
-  const root = flags.root ?? process.cwd();
+  const root = commandRoot(flags);
   const listed = await tracked(root);
   if (listed.error) return listed.error;
   const files = new Set(listed.files);
@@ -63,11 +62,7 @@ export async function impactCommand(args, flags, commands = []) {
     consumers: consumers.references,
     truncated,
   };
-  if (isJson(flags)) {
-    emitJson(payload, { pretty: flags.pretty === true, maxBytes: flags.maxBytes ?? 0 });
-    return EXIT.OK;
-  }
-  emit(render(payload), { maxBytes: flags.maxBytes ?? 0 });
+  present(payload, () => render(payload), flags);
   return EXIT.OK;
 }
 
@@ -91,8 +86,7 @@ async function governance(root, file, files, commands) {
   const records = await decisionRecords(root);
   const rules = await ruleScopes(root);
   const ctx = { files, commands: new Set(commands), roots: sourceRoots(files) };
-  const { hits } = decisionHits(records, file, ctx);
-  return sortHits([...hits, ...scopeHits(file, { rules })]);
+  return governs(file, { records, rules, ctx });
 }
 
 async function readOwners(root, file) {
@@ -193,8 +187,7 @@ async function tracked(root) {
   try {
     // `-z`: without it git C-quotes a non-ASCII path and the reader's real
     // filename never matches it (review 0003 Q39).
-    const { stdout } = await exec('git', ['ls-files', '-z'], { cwd: root, maxBuffer: 16 << 20 });
-    return { files: stdout.split('\0').filter(Boolean) };
+    return { files: await gitPaths(root, ['ls-files']) };
   } catch (error) {
     return {
       error: fail(