march-cli 0.1.24 → 0.1.26

package/src/debug/model-context-dumper.mjs

@@ -1,52 +1,52 @@
-import { mkdirSync, writeFileSync } from "node:fs";
-import { basename, extname, join } from "node:path";
-
-export function createModelContextDumper({ enabled = false, rootDir = "", now = () => new Date() } = {}) {
-  let sequence = 0;
-  return {
-    enabled: Boolean(enabled),
-    rootDir,
-    dump({ kind = "model", prompt = "", metadata = {} } = {}) {
-      if (!enabled) return null;
-      sequence += 1;
-      mkdirSync(rootDir, { recursive: true });
-      const timestamp = now().toISOString();
-      const filename = `${sanitizeTimestamp(timestamp)}-${String(sequence).padStart(4, "0")}-${sanitizeKind(kind)}.md`;
-      const path = join(rootDir, filename);
-      writeFileSync(path, `${formatHeader({ kind, timestamp, ...metadata })}\n\n${prompt}`, "utf8");
-      return path;
-    },
-    dumpSidecar({ sourcePath, suffix, value } = {}) {
-      if (!enabled || !sourcePath || !suffix) return null;
-      const ext = extname(sourcePath);
-      const stem = basename(sourcePath, ext);
-      const path = join(rootDir, `${stem}-${sanitizeKind(suffix)}.json`);
-      mkdirSync(rootDir, { recursive: true });
-      writeFileSync(path, `${JSON.stringify(value, null, 2)}\n`, "utf8");
-      return path;
-    },
-  };
-}
-
-function formatHeader(metadata) {
-  const lines = ["---"];
-  for (const [key, value] of Object.entries(metadata)) {
-    if (value == null || value === "") continue;
-    lines.push(`${key}: ${formatValue(value)}`);
-  }
-  lines.push("---");
-  return lines.join("\n");
-}
-
-function formatValue(value) {
-  if (typeof value === "number" || typeof value === "boolean") return String(value);
-  return JSON.stringify(String(value));
-}
-
-function sanitizeTimestamp(timestamp) {
-  return timestamp.replace(/[:.]/g, "-");
-}
-
-function sanitizeKind(kind) {
-  return String(kind).replace(/[^a-z0-9_-]+/gi, "-").replace(/^-+|-+$/g, "") || "model";
-}
+import { mkdirSync, writeFileSync } from "node:fs";
+import { basename, extname, join } from "node:path";
+
+export function createModelContextDumper({ enabled = false, rootDir = "", now = () => new Date() } = {}) {
+  let sequence = 0;
+  return {
+    enabled: Boolean(enabled),
+    rootDir,
+    dump({ kind = "model", prompt = "", metadata = {} } = {}) {
+      if (!enabled) return null;
+      sequence += 1;
+      mkdirSync(rootDir, { recursive: true });
+      const timestamp = now().toISOString();
+      const filename = `${sanitizeTimestamp(timestamp)}-${String(sequence).padStart(4, "0")}-${sanitizeKind(kind)}.md`;
+      const path = join(rootDir, filename);
+      writeFileSync(path, `${formatHeader({ kind, timestamp, ...metadata })}\n\n${prompt}`, "utf8");
+      return path;
+    },
+    dumpSidecar({ sourcePath, suffix, value } = {}) {
+      if (!enabled || !sourcePath || !suffix) return null;
+      const ext = extname(sourcePath);
+      const stem = basename(sourcePath, ext);
+      const path = join(rootDir, `${stem}-${sanitizeKind(suffix)}.json`);
+      mkdirSync(rootDir, { recursive: true });
+      writeFileSync(path, `${JSON.stringify(value, null, 2)}\n`, "utf8");
+      return path;
+    },
+  };
+}
+
+function formatHeader(metadata) {
+  const lines = ["---"];
+  for (const [key, value] of Object.entries(metadata)) {
+    if (value == null || value === "") continue;
+    lines.push(`${key}: ${formatValue(value)}`);
+  }
+  lines.push("---");
+  return lines.join("\n");
+}
+
+function formatValue(value) {
+  if (typeof value === "number" || typeof value === "boolean") return String(value);
+  return JSON.stringify(String(value));
+}
+
+function sanitizeTimestamp(timestamp) {
+  return timestamp.replace(/[:.]/g, "-");
+}
+
+function sanitizeKind(kind) {
+  return String(kind).replace(/[^a-z0-9_-]+/gi, "-").replace(/^-+|-+$/g, "") || "model";
+}

package/src/extensions/discovery.mjs

@@ -1,40 +1,40 @@
-import { existsSync, readdirSync } from "node:fs";
-import { join, resolve } from "node:path";
-
-const EXTENSION_FILE_NAMES = new Set(["index.js", "index.ts", "index.mjs", "index.cjs"]);
-const EXTENSION_FILE_EXTENSIONS = new Set([".js", ".ts", ".mjs", ".cjs"]);
-
-export function discoverProjectExtensionPaths(cwd) {
-  const extensionsDir = resolve(cwd, ".march", "extensions");
-  if (!existsSync(extensionsDir)) return [];
-
-  const entries = readdirSync(extensionsDir, { withFileTypes: true })
-    .sort((a, b) => a.name.localeCompare(b.name));
-  const paths = [];
-
-  for (const entry of entries) {
-    const entryPath = join(extensionsDir, entry.name);
-    if (entry.isFile() && hasExtensionFileExtension(entry.name)) {
-      paths.push(entryPath);
-      continue;
-    }
-    if (!entry.isDirectory()) continue;
-
-    const indexPath = findExtensionIndex(entryPath);
-    if (indexPath) paths.push(indexPath);
-  }
-
-  return paths;
-}
-
-function findExtensionIndex(dir) {
-  for (const fileName of EXTENSION_FILE_NAMES) {
-    const filePath = join(dir, fileName);
-    if (existsSync(filePath)) return filePath;
-  }
-  return null;
-}
-
-function hasExtensionFileExtension(fileName) {
-  return EXTENSION_FILE_EXTENSIONS.has(fileName.slice(fileName.lastIndexOf(".")));
-}
+import { existsSync, readdirSync } from "node:fs";
+import { join, resolve } from "node:path";
+
+const EXTENSION_FILE_NAMES = new Set(["index.js", "index.ts", "index.mjs", "index.cjs"]);
+const EXTENSION_FILE_EXTENSIONS = new Set([".js", ".ts", ".mjs", ".cjs"]);
+
+export function discoverProjectExtensionPaths(cwd) {
+  const extensionsDir = resolve(cwd, ".march", "extensions");
+  if (!existsSync(extensionsDir)) return [];
+
+  const entries = readdirSync(extensionsDir, { withFileTypes: true })
+    .sort((a, b) => a.name.localeCompare(b.name));
+  const paths = [];
+
+  for (const entry of entries) {
+    const entryPath = join(extensionsDir, entry.name);
+    if (entry.isFile() && hasExtensionFileExtension(entry.name)) {
+      paths.push(entryPath);
+      continue;
+    }
+    if (!entry.isDirectory()) continue;
+
+    const indexPath = findExtensionIndex(entryPath);
+    if (indexPath) paths.push(indexPath);
+  }
+
+  return paths;
+}
+
+function findExtensionIndex(dir) {
+  for (const fileName of EXTENSION_FILE_NAMES) {
+    const filePath = join(dir, fileName);
+    if (existsSync(filePath)) return filePath;
+  }
+  return null;
+}
+
+function hasExtensionFileExtension(fileName) {
+  return EXTENSION_FILE_EXTENSIONS.has(fileName.slice(fileName.lastIndexOf(".")));
+}

package/src/extensions/lifecycle-adapter.mjs

@@ -1,210 +1,210 @@
-export const MARCH_LIFECYCLE_LAYERS = Object.freeze([
-  {
-    name: "pi-agent-turn",
-    owner: "pi runtime host",
-    effects: Object.freeze(["read-runtime-diagnostics"]),
-  },
-  {
-    name: "march-agent-runtime",
-    owner: "March runner",
-    effects: Object.freeze(["read-session-ref", "read-sidecar-metadata", "read-runtime-state"]),
-  },
-  {
-    name: "march-collaboration",
-    owner: "March orchestrator",
-    effects: Object.freeze(["read-group-ref", "read-workspace-ref", "read-diff-metadata"]),
-  },
-]);
-
-export const DEFAULT_MARCH_HOOK_POLICY = Object.freeze({
-  mode: "read-only",
-  defaultBlocking: false,
-  allowedEffects: Object.freeze([
-    "read-group-ref",
-    "read-agent-ref",
-    "read-workspace-ref",
-    "read-session-ref",
-    "read-sidecar-metadata",
-    "read-diff-metadata",
-    "read-runtime-diagnostics",
-    "write-diagnostics",
-  ]),
-  deniedEffects: Object.freeze([
-    "write-files",
-    "run-shell",
-    "switch-session",
-    "commit-worktree",
-    "read-private-agent-context",
-  ]),
-});
-
-export function createMarchLifecycleAdapter({
-  cwd,
-  projectMarchDir,
-  extensionPaths = [],
-  sessionBinding,
-  engine,
-  getSessionStats,
-  getRuntimeDiagnostics = () => [],
-  manifestHooks = [],
-  manifestDiagnostics = [],
-}) {
-  const hooks = new Map();
-  const adapterDiagnostics = [...manifestDiagnostics];
-
-  for (const manifestHook of manifestHooks) {
-    try {
-      registerManifestHook(hooks, manifestHook);
-    } catch (err) {
-      adapterDiagnostics.push({
-        type: "warning",
-        message: `Failed to register March lifecycle hook ${manifestHook?.id ?? "(unknown)"}: ${err.message}`,
-        path: manifestHook?.sourcePath,
-      });
-    }
-  }
-
-  return {
-    registerHook(hook) {
-      const normalized = normalizeHook(hook);
-      const deniedEffect = normalized.effects
-        .map((effect) => evaluateMarchHookEffect(effect))
-        .find((result) => !result.allowed);
-      if (deniedEffect) {
-        throw new Error(deniedEffect.reason);
-      }
-      hooks.set(normalized.id, normalized);
-      return normalized;
-    },
-
-    unregisterHook(id) {
-      return hooks.delete(id);
-    },
-
-    async runHook(kind, payload = {}) {
-      const matchingHooks = [...hooks.values()].filter((hook) => hook.kind === kind);
-      const results = [];
-      for (const hook of matchingHooks) {
-        const result = await runRegisteredHook(hook, {
-          facts: buildFacts({ cwd, projectMarchDir, engine, getSessionStats }),
-          payload,
-          canExecute: evaluateMarchHookEffect,
-        }, adapterDiagnostics);
-        results.push(result);
-      }
-      return results;
-    },
-
-    getState() {
-      return {
-        status: "read-only",
-        registeredHookCount: hooks.size,
-        extensionPathCount: extensionPaths.length,
-        hookKinds: [...new Set([...hooks.values()].map((hook) => hook.kind))].sort(),
-        facts: buildFacts({ cwd, projectMarchDir, engine, getSessionStats }),
-        layers: MARCH_LIFECYCLE_LAYERS,
-        policy: DEFAULT_MARCH_HOOK_POLICY,
-        diagnostics: [
-          {
-            type: "info",
-            message: hooks.size === 0
-              ? "March lifecycle hook adapter is read-only; no March hooks are registered."
-              : "March lifecycle hook adapter is read-only; registered hooks are permission-gated.",
-          },
-          ...adapterDiagnostics,
-          ...getRuntimeDiagnostics(),
-        ],
-      };
-    },
-
-    canExecute(effect) {
-      return evaluateMarchHookEffect(effect);
-    },
-
-    getActiveSession() {
-      return sessionBinding?.get?.() ?? null;
-    },
-  };
-}
-
-function registerManifestHook(hooks, manifestHook) {
-  const normalized = normalizeHook({
-    ...manifestHook,
-    handler: () => ({
-      sourcePath: manifestHook.sourcePath,
-      description: manifestHook.description,
-    }),
-  });
-  const deniedEffect = normalized.effects
-    .map((effect) => evaluateMarchHookEffect(effect))
-    .find((result) => !result.allowed);
-  if (deniedEffect) throw new Error(deniedEffect.reason);
-  hooks.set(normalized.id, normalized);
-  return normalized;
-}
-
-export function evaluateMarchHookEffect(effect) {
-  if (DEFAULT_MARCH_HOOK_POLICY.allowedEffects.includes(effect)) {
-    return { allowed: true };
-  }
-  return {
-    allowed: false,
-    reason: DEFAULT_MARCH_HOOK_POLICY.deniedEffects.includes(effect)
-      ? `March lifecycle hooks cannot ${effect} in read-only mode`
-      : `Unknown March lifecycle hook effect: ${effect}`,
-  };
-}
-
-function buildFacts({ cwd, projectMarchDir, engine, getSessionStats }) {
-  const stats = getSessionStats?.();
-  return {
-    cwd,
-    projectMarchDir,
-    sessionId: stats?.sessionId ?? null,
-    sessionFile: stats?.sessionFile ?? null,
-    persisted: Boolean(stats?.persisted),
-    runtimeHost: Boolean(stats?.runtimeHost),
-    modelId: engine?.modelId ?? null,
-    provider: engine?.provider ?? null,
-    thinkingLevel: engine?.thinkingLevel ?? null,
-    namespace: engine?.namespace ?? null,
-    turnCount: engine?.turns?.length ?? 0,
-  };
-}
-
-function normalizeHook(hook) {
-  if (!hook || typeof hook !== "object") throw new Error("March lifecycle hook must be an object");
-  if (!hook.id || typeof hook.id !== "string") throw new Error("March lifecycle hook requires a string id");
-  if (!hook.kind || typeof hook.kind !== "string") throw new Error("March lifecycle hook requires a string kind");
-  if (typeof hook.handler !== "function") throw new Error("March lifecycle hook requires a handler function");
-  return {
-    id: hook.id,
-    kind: hook.kind,
-    effects: [...new Set(hook.effects ?? [])].sort(),
-    blocking: Boolean(hook.blocking ?? DEFAULT_MARCH_HOOK_POLICY.defaultBlocking),
-    handler: hook.handler,
-  };
-}
-
-async function runRegisteredHook(hook, context, diagnostics) {
-  try {
-    for (const effect of hook.effects) {
-      const gate = evaluateMarchHookEffect(effect);
-      if (!gate.allowed) throw new Error(gate.reason);
-    }
-    const value = await hook.handler(Object.freeze({
-      facts: Object.freeze({ ...context.facts }),
-      payload: Object.freeze({ ...context.payload }),
-      canExecute: context.canExecute,
-    }));
-    return { id: hook.id, kind: hook.kind, ok: true, value };
-  } catch (err) {
-    const diagnostic = {
-      type: hook.blocking ? "error" : "warning",
-      message: `March lifecycle hook ${hook.id} failed: ${err.message}`,
-    };
-    diagnostics.push(diagnostic);
-    if (hook.blocking) throw err;
-    return { id: hook.id, kind: hook.kind, ok: false, error: err.message };
-  }
-}
+export const MARCH_LIFECYCLE_LAYERS = Object.freeze([
+  {
+    name: "pi-agent-turn",
+    owner: "pi runtime host",
+    effects: Object.freeze(["read-runtime-diagnostics"]),
+  },
+  {
+    name: "march-agent-runtime",
+    owner: "March runner",
+    effects: Object.freeze(["read-session-ref", "read-sidecar-metadata", "read-runtime-state"]),
+  },
+  {
+    name: "march-collaboration",
+    owner: "March orchestrator",
+    effects: Object.freeze(["read-group-ref", "read-workspace-ref", "read-diff-metadata"]),
+  },
+]);
+
+export const DEFAULT_MARCH_HOOK_POLICY = Object.freeze({
+  mode: "read-only",
+  defaultBlocking: false,
+  allowedEffects: Object.freeze([
+    "read-group-ref",
+    "read-agent-ref",
+    "read-workspace-ref",
+    "read-session-ref",
+    "read-sidecar-metadata",
+    "read-diff-metadata",
+    "read-runtime-diagnostics",
+    "write-diagnostics",
+  ]),
+  deniedEffects: Object.freeze([
+    "write-files",
+    "run-shell",
+    "switch-session",
+    "commit-worktree",
+    "read-private-agent-context",
+  ]),
+});
+
+export function createMarchLifecycleAdapter({
+  cwd,
+  projectMarchDir,
+  extensionPaths = [],
+  sessionBinding,
+  engine,
+  getSessionStats,
+  getRuntimeDiagnostics = () => [],
+  manifestHooks = [],
+  manifestDiagnostics = [],
+}) {
+  const hooks = new Map();
+  const adapterDiagnostics = [...manifestDiagnostics];
+
+  for (const manifestHook of manifestHooks) {
+    try {
+      registerManifestHook(hooks, manifestHook);
+    } catch (err) {
+      adapterDiagnostics.push({
+        type: "warning",
+        message: `Failed to register March lifecycle hook ${manifestHook?.id ?? "(unknown)"}: ${err.message}`,
+        path: manifestHook?.sourcePath,
+      });
+    }
+  }
+
+  return {
+    registerHook(hook) {
+      const normalized = normalizeHook(hook);
+      const deniedEffect = normalized.effects
+        .map((effect) => evaluateMarchHookEffect(effect))
+        .find((result) => !result.allowed);
+      if (deniedEffect) {
+        throw new Error(deniedEffect.reason);
+      }
+      hooks.set(normalized.id, normalized);
+      return normalized;
+    },
+
+    unregisterHook(id) {
+      return hooks.delete(id);
+    },
+
+    async runHook(kind, payload = {}) {
+      const matchingHooks = [...hooks.values()].filter((hook) => hook.kind === kind);
+      const results = [];
+      for (const hook of matchingHooks) {
+        const result = await runRegisteredHook(hook, {
+          facts: buildFacts({ cwd, projectMarchDir, engine, getSessionStats }),
+          payload,
+          canExecute: evaluateMarchHookEffect,
+        }, adapterDiagnostics);
+        results.push(result);
+      }
+      return results;
+    },
+
+    getState() {
+      return {
+        status: "read-only",
+        registeredHookCount: hooks.size,
+        extensionPathCount: extensionPaths.length,
+        hookKinds: [...new Set([...hooks.values()].map((hook) => hook.kind))].sort(),
+        facts: buildFacts({ cwd, projectMarchDir, engine, getSessionStats }),
+        layers: MARCH_LIFECYCLE_LAYERS,
+        policy: DEFAULT_MARCH_HOOK_POLICY,
+        diagnostics: [
+          {
+            type: "info",
+            message: hooks.size === 0
+              ? "March lifecycle hook adapter is read-only; no March hooks are registered."
+              : "March lifecycle hook adapter is read-only; registered hooks are permission-gated.",
+          },
+          ...adapterDiagnostics,
+          ...getRuntimeDiagnostics(),
+        ],
+      };
+    },
+
+    canExecute(effect) {
+      return evaluateMarchHookEffect(effect);
+    },
+
+    getActiveSession() {
+      return sessionBinding?.get?.() ?? null;
+    },
+  };
+}
+
+function registerManifestHook(hooks, manifestHook) {
+  const normalized = normalizeHook({
+    ...manifestHook,
+    handler: () => ({
+      sourcePath: manifestHook.sourcePath,
+      description: manifestHook.description,
+    }),
+  });
+  const deniedEffect = normalized.effects
+    .map((effect) => evaluateMarchHookEffect(effect))
+    .find((result) => !result.allowed);
+  if (deniedEffect) throw new Error(deniedEffect.reason);
+  hooks.set(normalized.id, normalized);
+  return normalized;
+}
+
+export function evaluateMarchHookEffect(effect) {
+  if (DEFAULT_MARCH_HOOK_POLICY.allowedEffects.includes(effect)) {
+    return { allowed: true };
+  }
+  return {
+    allowed: false,
+    reason: DEFAULT_MARCH_HOOK_POLICY.deniedEffects.includes(effect)
+      ? `March lifecycle hooks cannot ${effect} in read-only mode`
+      : `Unknown March lifecycle hook effect: ${effect}`,
+  };
+}
+
+function buildFacts({ cwd, projectMarchDir, engine, getSessionStats }) {
+  const stats = getSessionStats?.();
+  return {
+    cwd,
+    projectMarchDir,
+    sessionId: stats?.sessionId ?? null,
+    sessionFile: stats?.sessionFile ?? null,
+    persisted: Boolean(stats?.persisted),
+    runtimeHost: Boolean(stats?.runtimeHost),
+    modelId: engine?.modelId ?? null,
+    provider: engine?.provider ?? null,
+    thinkingLevel: engine?.thinkingLevel ?? null,
+    namespace: engine?.namespace ?? null,
+    turnCount: engine?.turns?.length ?? 0,
+  };
+}
+
+function normalizeHook(hook) {
+  if (!hook || typeof hook !== "object") throw new Error("March lifecycle hook must be an object");
+  if (!hook.id || typeof hook.id !== "string") throw new Error("March lifecycle hook requires a string id");
+  if (!hook.kind || typeof hook.kind !== "string") throw new Error("March lifecycle hook requires a string kind");
+  if (typeof hook.handler !== "function") throw new Error("March lifecycle hook requires a handler function");
+  return {
+    id: hook.id,
+    kind: hook.kind,
+    effects: [...new Set(hook.effects ?? [])].sort(),
+    blocking: Boolean(hook.blocking ?? DEFAULT_MARCH_HOOK_POLICY.defaultBlocking),
+    handler: hook.handler,
+  };
+}
+
+async function runRegisteredHook(hook, context, diagnostics) {
+  try {
+    for (const effect of hook.effects) {
+      const gate = evaluateMarchHookEffect(effect);
+      if (!gate.allowed) throw new Error(gate.reason);
+    }
+    const value = await hook.handler(Object.freeze({
+      facts: Object.freeze({ ...context.facts }),
+      payload: Object.freeze({ ...context.payload }),
+      canExecute: context.canExecute,
+    }));
+    return { id: hook.id, kind: hook.kind, ok: true, value };
+  } catch (err) {
+    const diagnostic = {
+      type: hook.blocking ? "error" : "warning",
+      message: `March lifecycle hook ${hook.id} failed: ${err.message}`,
+    };
+    diagnostics.push(diagnostic);
+    if (hook.blocking) throw err;
+    return { id: hook.id, kind: hook.kind, ok: false, error: err.message };
+  }
+}