march-cli 0.1.24 → 0.1.25

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 };
+  }
+}

package/src/extensions/lifecycle-manifest.mjs

@@ -1,69 +1,69 @@
-import { existsSync, readdirSync, readFileSync } from "node:fs";
-import { join, resolve } from "node:path";
-
-export function discoverProjectLifecycleHookManifestPaths(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() && entry.name.endsWith(".march-hooks.json")) {
-      paths.push(entryPath);
-      continue;
-    }
-    if (entry.isDirectory()) {
-      const manifestPath = join(entryPath, "march-hooks.json");
-      if (existsSync(manifestPath)) paths.push(manifestPath);
-    }
-  }
-  return paths;
-}
-
-export function loadProjectLifecycleHookManifests(cwd) {
-  return loadLifecycleHookManifests(discoverProjectLifecycleHookManifestPaths(cwd));
-}
-
-export function loadLifecycleHookManifests(paths) {
-  const hooks = [];
-  const diagnostics = [];
-  for (const path of paths) {
-    try {
-      const parsed = JSON.parse(readFileSync(path, "utf8"));
-      for (const hook of normalizeManifestHooks(parsed, path)) hooks.push(hook);
-    } catch (err) {
-      diagnostics.push({
-        type: "warning",
-        message: `Failed to load March lifecycle hook manifest: ${err.message}`,
-        path,
-      });
-    }
-  }
-  return { hooks, diagnostics };
-}
-
-function normalizeManifestHooks(parsed, path) {
-  const rawHooks = Array.isArray(parsed) ? parsed : parsed?.hooks;
-  if (!Array.isArray(rawHooks)) throw new Error("manifest must be an array or an object with hooks[]");
-  return rawHooks.map((hook, index) => normalizeManifestHook(hook, path, index));
-}
-
-function normalizeManifestHook(hook, path, index) {
-  if (!hook || typeof hook !== "object") throw new Error(`hook ${index + 1} must be an object`);
-  if (!hook.id || typeof hook.id !== "string") throw new Error(`hook ${index + 1} requires string id`);
-  if (!hook.kind || typeof hook.kind !== "string") throw new Error(`hook ${index + 1} requires string kind`);
-  const effects = Array.isArray(hook.effects) ? hook.effects : [];
-  if (!effects.every((effect) => typeof effect === "string")) {
-    throw new Error(`hook ${hook.id} effects must be strings`);
-  }
-  return {
-    id: hook.id,
-    kind: hook.kind,
-    effects,
-    blocking: Boolean(hook.blocking),
-    description: typeof hook.description === "string" ? hook.description : "",
-    sourcePath: path,
-  };
-}
+import { existsSync, readdirSync, readFileSync } from "node:fs";
+import { join, resolve } from "node:path";
+
+export function discoverProjectLifecycleHookManifestPaths(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() && entry.name.endsWith(".march-hooks.json")) {
+      paths.push(entryPath);
+      continue;
+    }
+    if (entry.isDirectory()) {
+      const manifestPath = join(entryPath, "march-hooks.json");
+      if (existsSync(manifestPath)) paths.push(manifestPath);
+    }
+  }
+  return paths;
+}
+
+export function loadProjectLifecycleHookManifests(cwd) {
+  return loadLifecycleHookManifests(discoverProjectLifecycleHookManifestPaths(cwd));
+}
+
+export function loadLifecycleHookManifests(paths) {
+  const hooks = [];
+  const diagnostics = [];
+  for (const path of paths) {
+    try {
+      const parsed = JSON.parse(readFileSync(path, "utf8"));
+      for (const hook of normalizeManifestHooks(parsed, path)) hooks.push(hook);
+    } catch (err) {
+      diagnostics.push({
+        type: "warning",
+        message: `Failed to load March lifecycle hook manifest: ${err.message}`,
+        path,
+      });
+    }
+  }
+  return { hooks, diagnostics };
+}
+
+function normalizeManifestHooks(parsed, path) {
+  const rawHooks = Array.isArray(parsed) ? parsed : parsed?.hooks;
+  if (!Array.isArray(rawHooks)) throw new Error("manifest must be an array or an object with hooks[]");
+  return rawHooks.map((hook, index) => normalizeManifestHook(hook, path, index));
+}
+
+function normalizeManifestHook(hook, path, index) {
+  if (!hook || typeof hook !== "object") throw new Error(`hook ${index + 1} must be an object`);
+  if (!hook.id || typeof hook.id !== "string") throw new Error(`hook ${index + 1} requires string id`);
+  if (!hook.kind || typeof hook.kind !== "string") throw new Error(`hook ${index + 1} requires string kind`);
+  const effects = Array.isArray(hook.effects) ? hook.effects : [];
+  if (!effects.every((effect) => typeof effect === "string")) {
+    throw new Error(`hook ${hook.id} effects must be strings`);
+  }
+  return {
+    id: hook.id,
+    kind: hook.kind,
+    effects,
+    blocking: Boolean(hook.blocking),
+    description: typeof hook.description === "string" ? hook.description : "",
+    sourcePath: path,
+  };
+}

package/src/image-gen/index.mjs

@@ -1,7 +1,7 @@
-import { createImageGenTool } from "./tool.mjs";
-
-export function initImageGen({ authStorage, projectMarchDir }) {
-  const credentials = authStorage?.get?.("openai-codex");
-  if (!credentials) return [];
-  return [createImageGenTool({ authStorage, projectMarchDir })];
-}
+import { createImageGenTool } from "./tool.mjs";
+
+export function initImageGen({ authStorage, projectMarchDir }) {
+  const credentials = authStorage?.get?.("openai-codex");
+  if (!credentials) return [];
+  return [createImageGenTool({ authStorage, projectMarchDir })];
+}