march-cli 0.1.21 → 0.1.23

package/src/memory/markdown-tools.mjs

@@ -1,103 +1,103 @@
-import { defineTool } from "@earendil-works/pi-coding-agent";
-import { Type } from "typebox";
-import { toolText } from "../agent/tool-result.mjs";
-
-export function createMarkdownMemoryTools(store) {
-  if (!store) return [];
-  return [
-    defineTool({
-      name: "memory_search",
-      label: "Memory Search",
-      description:
-        "Search the Markdown memory vault using ripgrep. This is literal text search over memory Markdown files, not semantic search and not March's internal tags-only memory hint recall. Use it when memory hints are insufficient or you need to find a memory by phrase, term, or detail.",
-      parameters: Type.Object({
-        query: Type.String({ description: "Ripgrep query to search in Markdown memory files" }),
-        limit: Type.Optional(Type.Number({ description: "Maximum matching lines to return. Default: 20" })),
-      }),
-      execute: async (_toolCallId, params) => {
-        const results = store.searchRipgrep(params.query, { limit: params.limit ?? 20 });
-        if (results.length === 0) return toolText(formatMemorySearchMiss(params.query));
-        return toolText(results.map((result) => result.line).join("\n"), { results });
-      },
-    }),
-
-    defineTool({
-      name: "memory_open",
-      label: "Memory Open",
-      description: "Open a Markdown memory by id or by path. Use this after memory hint or memory_search when you need the full memory body. To edit an existing memory, use the returned path with edit_file for targeted edits.",
-      parameters: Type.Object({
-        id: Type.Optional(Type.String({ description: "Memory id from memory hint, e.g. mem_..." })),
-        path: Type.Optional(Type.String({ description: "Path returned by memory_search" })),
-      }),
-      execute: async (_toolCallId, params) => {
-        try {
-          const identifier = params.id || params.path;
-          const opened = store.open(identifier);
-          return toolText(
-            `path: ${opened.path}\nUse edit_file with this path for targeted edits.\n\n---\n${opened.content}`,
-            opened,
-          );
-        } catch (err) {
-          return toolText(`Error: ${err.message}`, { error: true });
-        }
-      },
-    }),
-
-    defineTool({
-      name: "memory_save",
-      label: "Memory Save",
-      description:
-        "Create a Markdown memory or update whole fields on an existing memory. For targeted edits to an existing memory body or frontmatter, use memory_open to get the path, then edit_file. Before creating a new memory, merge related updates into an existing memory when they share the same topic or decision thread. New memories require name, description, body, and at least one tag because memory hints only use tags. When updating by id, omitted fields keep their existing values; passing tags replaces the full tag list.",
-      parameters: Type.Object({
-        id: Type.Optional(Type.String({ description: "Existing memory id to update. Omit to create a new memory." })),
-        name: Type.Optional(Type.String({ description: "Memory name. Required when creating." })),
-        description: Type.Optional(Type.String({ description: "Short natural-language summary shown in memory hint. Required when creating." })),
-        body: Type.Optional(Type.String({ description: "Markdown memory body. Required when creating." })),
-        tags: Type.Optional(Type.Array(Type.String(), { description: "Tags used for memory hint. Required and non-empty when creating; replaces tags when updating. Prefer stable retrieval keys: project name, technology, feature/domain, user/person, and decision topic. Use lowercase kebab-case when possible. Use shared tags to keep related facts discoverable and merge them into one memory when they belong to the same topic. Examples: ['march-cli', 'tooling', 'permissions'], ['memory', 'sqlite-index']." })),
-      }),
-      execute: async (_toolCallId, params) => {
-        try {
-          const entry = store.save(params);
-          return toolText(
-            `Saved ${entry.id}\npath: ${entry.path}\ntags: ${entry.tags.join(", ")}`,
-            { memory: entry },
-          );
-        } catch (err) {
-          return toolText(`Error: ${err.message}`, { error: true });
-        }
-      },
-    }),
-
-    defineTool({
-      name: "memory_delete",
-      label: "Memory Delete",
-      description: "Soft-delete a Markdown memory by id or path. This marks status as deleted so memory hint and memory_search exclude it; the file remains on disk for audit/recovery.",
-      parameters: Type.Object({
-        id: Type.Optional(Type.String({ description: "Memory id to delete, e.g. mem_..." })),
-        path: Type.Optional(Type.String({ description: "Memory file path to delete" })),
-      }),
-      execute: async (_toolCallId, params) => {
-        try {
-          const identifier = params.id || params.path;
-          const result = store.delete(identifier);
-          if (result.alreadyDeleted) return toolText(`Memory ${result.id} is already deleted.\npath: ${result.path}`, { memory: result });
-          return toolText(`Deleted ${result.id}\npath: ${result.path}`, { memory: result });
-        } catch (err) {
-          return toolText(`Error: ${err.message}`, { error: true });
-        }
-      },
-    }),
-  ];
-}
-
-function formatMemorySearchMiss(query) {
-  const text = String(query ?? "");
-  const lines = [
-    `No memory files matched "${text}".`,
-    "memory_search is literal ripgrep over Markdown files; it is not semantic search and does not use memory-hint hints.",
-  ];
-  if (/^mem_[a-z0-9_\-]+$/i.test(text.trim())) {
-    lines.push("This looks like a memory id. Use memory_open({ id }) to open it directly.");
-  }
-  return lines.join("\n");
-}
+import { defineTool } from "@earendil-works/pi-coding-agent";
+import { Type } from "typebox";
+import { toolText } from "../agent/tool-result.mjs";
+
+export function createMarkdownMemoryTools(store) {
+  if (!store) return [];
+  return [
+    defineTool({
+      name: "memory_search",
+      label: "Memory Search",
+      description:
+        "Search the Markdown memory vault using ripgrep. This is literal text search over memory Markdown files, not semantic search and not March's internal tags-only memory hint recall. Use it when memory hints are insufficient or you need to find a memory by phrase, term, or detail.",
+      parameters: Type.Object({
+        query: Type.String({ description: "Ripgrep query to search in Markdown memory files" }),
+        limit: Type.Optional(Type.Number({ description: "Maximum matching lines to return. Default: 20" })),
+      }),
+      execute: async (_toolCallId, params) => {
+        const results = store.searchRipgrep(params.query, { limit: params.limit ?? 20 });
+        if (results.length === 0) return toolText(formatMemorySearchMiss(params.query));
+        return toolText(results.map((result) => result.line).join("\n"), { results });
+      },
+    }),
+
+    defineTool({
+      name: "memory_open",
+      label: "Memory Open",
+      description: "Open a Markdown memory by id or by path. Use this after memory hint or memory_search when you need the full memory body. To edit an existing memory, use the returned path with edit_file for targeted edits.",
+      parameters: Type.Object({
+        id: Type.Optional(Type.String({ description: "Memory id from memory hint, e.g. mem_..." })),
+        path: Type.Optional(Type.String({ description: "Path returned by memory_search" })),
+      }),
+      execute: async (_toolCallId, params) => {
+        try {
+          const identifier = params.id || params.path;
+          const opened = store.open(identifier);
+          return toolText(
+            `path: ${opened.path}\nUse edit_file with this path for targeted edits.\n\n---\n${opened.content}`,
+            opened,
+          );
+        } catch (err) {
+          return toolText(`Error: ${err.message}`, { error: true });
+        }
+      },
+    }),
+
+    defineTool({
+      name: "memory_save",
+      label: "Memory Save",
+      description:
+        "Create a Markdown memory or update whole fields on an existing memory. For targeted edits to an existing memory body or frontmatter, use memory_open to get the path, then edit_file. Before creating a new memory, merge related updates into an existing memory when they share the same topic or decision thread. New memories require name, description, body, and at least one tag because memory hints only use tags. When updating by id, omitted fields keep their existing values; passing tags replaces the full tag list.",
+      parameters: Type.Object({
+        id: Type.Optional(Type.String({ description: "Existing memory id to update. Omit to create a new memory." })),
+        name: Type.Optional(Type.String({ description: "Memory name. Required when creating." })),
+        description: Type.Optional(Type.String({ description: "Short natural-language summary shown in memory hint. Required when creating." })),
+        body: Type.Optional(Type.String({ description: "Markdown memory body. Required when creating." })),
+        tags: Type.Optional(Type.Array(Type.String(), { description: "Tags used for memory hint. Required and non-empty when creating; replaces tags when updating. Prefer stable retrieval keys: project name, technology, feature/domain, user/person, and decision topic. Use lowercase kebab-case when possible. Use shared tags to keep related facts discoverable and merge them into one memory when they belong to the same topic. Examples: ['march-cli', 'tooling', 'permissions'], ['memory', 'sqlite-index']." })),
+      }),
+      execute: async (_toolCallId, params) => {
+        try {
+          const entry = store.save(params);
+          return toolText(
+            `Saved ${entry.id}\npath: ${entry.path}\ntags: ${entry.tags.join(", ")}`,
+            { memory: entry },
+          );
+        } catch (err) {
+          return toolText(`Error: ${err.message}`, { error: true });
+        }
+      },
+    }),
+
+    defineTool({
+      name: "memory_delete",
+      label: "Memory Delete",
+      description: "Soft-delete a Markdown memory by id or path. This marks status as deleted so memory hint and memory_search exclude it; the file remains on disk for audit/recovery.",
+      parameters: Type.Object({
+        id: Type.Optional(Type.String({ description: "Memory id to delete, e.g. mem_..." })),
+        path: Type.Optional(Type.String({ description: "Memory file path to delete" })),
+      }),
+      execute: async (_toolCallId, params) => {
+        try {
+          const identifier = params.id || params.path;
+          const result = store.delete(identifier);
+          if (result.alreadyDeleted) return toolText(`Memory ${result.id} is already deleted.\npath: ${result.path}`, { memory: result });
+          return toolText(`Deleted ${result.id}\npath: ${result.path}`, { memory: result });
+        } catch (err) {
+          return toolText(`Error: ${err.message}`, { error: true });
+        }
+      },
+    }),
+  ];
+}
+
+function formatMemorySearchMiss(query) {
+  const text = String(query ?? "");
+  const lines = [
+    `No memory files matched "${text}".`,
+    "memory_search is literal ripgrep over Markdown files; it is not semantic search and does not use memory-hint hints.",
+  ];
+  if (/^mem_[a-z0-9_\-]+$/i.test(text.trim())) {
+    lines.push("This looks like a memory id. Use memory_open({ id }) to open it directly.");
+  }
+  return lines.join("\n");
+}

package/src/network/environment.mjs

@@ -1,131 +1,131 @@
-import { readFileSync } from "node:fs";
-import { execFileSync } from "node:child_process";
-import tls from "node:tls";
-import { EnvHttpProxyAgent, setGlobalDispatcher } from "undici";
-
-export function installNetworkEnvironment(network = {}) {
-  const proxy = resolveProxySettings(network);
-  installProxyDispatcher(proxy);
-  const ca = installDefaultCertificates(network.ca ?? "system");
-  return { proxy, ca };
-}
-
-export function resolveProxySettings(network = {}, { env = process.env, platform = process.platform } = {}) {
-  const proxyMode = network.proxy ?? "system";
-  const explicitNoProxy = formatNoProxy(network.noProxy);
-
-  if (proxyMode === false || proxyMode === "none" || proxyMode === "direct") {
-    return { mode: "direct", httpProxy: null, httpsProxy: null, noProxy: explicitNoProxy ?? env.NO_PROXY ?? env.no_proxy ?? null };
-  }
-
-  if (typeof proxyMode === "string" && proxyMode.trim() && proxyMode !== "system") {
-    const proxy = normalizeProxyUrl(proxyMode.trim());
-    return { mode: "config", httpProxy: proxy, httpsProxy: proxy, noProxy: explicitNoProxy ?? env.NO_PROXY ?? env.no_proxy ?? null };
-  }
-
-  const envProxy = proxyFromEnv(env, explicitNoProxy);
-  if (envProxy.httpProxy || envProxy.httpsProxy) return { mode: "env", ...envProxy };
-
-  const systemProxy = platform === "win32" ? detectWindowsProxy() : null;
-  if (systemProxy?.httpProxy || systemProxy?.httpsProxy) {
-    return {
-      mode: "system",
-      httpProxy: systemProxy.httpProxy,
-      httpsProxy: systemProxy.httpsProxy,
-      noProxy: explicitNoProxy ?? systemProxy.noProxy ?? null,
-    };
-  }
-
-  return { mode: "direct", httpProxy: null, httpsProxy: null, noProxy: explicitNoProxy ?? null };
-}
-
-function installProxyDispatcher(proxy) {
-  setGlobalDispatcher(new EnvHttpProxyAgent({
-    httpProxy: proxy.httpProxy ?? "",
-    httpsProxy: proxy.httpsProxy ?? "",
-    noProxy: proxy.noProxy ?? "",
-    bodyTimeout: 0,
-    headersTimeout: 0,
-  }));
-}
-
-export function installDefaultCertificates(caConfig = "system") {
-  const entries = Array.isArray(caConfig) ? caConfig : [caConfig];
-  const wantsSystem = entries.includes("system");
-  const pemPaths = entries.filter((entry) => typeof entry === "string" && entry && entry !== "system");
-
-  if (!wantsSystem && pemPaths.length === 0) return { mode: "default", system: false, extraFiles: [] };
-  if (typeof tls.setDefaultCACertificates !== "function" || typeof tls.getCACertificates !== "function") {
-    return { mode: "unsupported", system: false, extraFiles: [] };
-  }
-
-  const certificates = [
-    ...tls.getCACertificates("default"),
-    ...(wantsSystem ? tls.getCACertificates("system") : []),
-    ...pemPaths.map((path) => readFileSync(path, "utf8")),
-  ];
-  tls.setDefaultCACertificates([...new Set(certificates)]);
-  return { mode: "installed", system: wantsSystem, extraFiles: pemPaths };
-}
-
-function proxyFromEnv(env, explicitNoProxy) {
-  const httpsProxy = env.HTTPS_PROXY ?? env.https_proxy ?? env.ALL_PROXY ?? env.all_proxy ?? null;
-  const httpProxy = env.HTTP_PROXY ?? env.http_proxy ?? env.ALL_PROXY ?? env.all_proxy ?? null;
-  const normalizedHttp = httpProxy ? normalizeProxyUrl(httpProxy) : null;
-  return {
-    httpProxy: normalizedHttp,
-    httpsProxy: httpsProxy ? normalizeProxyUrl(httpsProxy) : normalizedHttp,
-    noProxy: explicitNoProxy ?? env.NO_PROXY ?? env.no_proxy ?? null,
-  };
-}
-
-function detectWindowsProxy() {
-  try {
-    const output = execFileSync("reg", ["query", "HKCU\\Software\\Microsoft\\Windows\\CurrentVersion\\Internet Settings"], { encoding: "utf8", windowsHide: true });
-    const values = parseRegQuery(output);
-    if (values.ProxyEnable !== "0x1") return null;
-    return parseWindowsProxyServer(values.ProxyServer, values.ProxyOverride);
-  } catch {
-    return null;
-  }
-}
-
-function parseRegQuery(output) {
-  const values = {};
-  for (const line of output.split(/\r?\n/)) {
-    const match = line.trim().match(/^(\S+)\s+REG_\S+\s+(.+)$/);
-    if (match) values[match[1]] = match[2].trim();
-  }
-  return values;
-}
-
-function parseWindowsProxyServer(proxyServer, proxyOverride) {
-  if (!proxyServer) return null;
-  const entries = Object.fromEntries(proxyServer.split(";").map((part) => part.split("=")).filter((part) => part.length === 2));
-  const fallback = proxyServer.includes("=") ? null : proxyServer;
-  return {
-    httpProxy: normalizeProxyUrl(entries.http ?? fallback),
-    httpsProxy: normalizeProxyUrl(entries.https ?? entries.http ?? fallback),
-    noProxy: formatWindowsProxyOverride(proxyOverride),
-  };
-}
-
-function normalizeProxyUrl(value) {
-  if (!value) return null;
-  return /^[a-z][a-z0-9+.-]*:\/\//i.test(value) ? value : `http://${value}`;
-}
-
-function formatNoProxy(value) {
-  if (Array.isArray(value)) return value.join(",");
-  if (typeof value === "string") return value;
-  return null;
-}
-
-function formatWindowsProxyOverride(value) {
-  if (!value) return null;
-  return value
-    .split(";")
-    .map((entry) => entry.trim())
-    .filter((entry) => entry && entry !== "<local>")
-    .join(",") || null;
-}
+import { readFileSync } from "node:fs";
+import { execFileSync } from "node:child_process";
+import tls from "node:tls";
+import { EnvHttpProxyAgent, setGlobalDispatcher } from "undici";
+
+export function installNetworkEnvironment(network = {}) {
+  const proxy = resolveProxySettings(network);
+  installProxyDispatcher(proxy);
+  const ca = installDefaultCertificates(network.ca ?? "system");
+  return { proxy, ca };
+}
+
+export function resolveProxySettings(network = {}, { env = process.env, platform = process.platform } = {}) {
+  const proxyMode = network.proxy ?? "system";
+  const explicitNoProxy = formatNoProxy(network.noProxy);
+
+  if (proxyMode === false || proxyMode === "none" || proxyMode === "direct") {
+    return { mode: "direct", httpProxy: null, httpsProxy: null, noProxy: explicitNoProxy ?? env.NO_PROXY ?? env.no_proxy ?? null };
+  }
+
+  if (typeof proxyMode === "string" && proxyMode.trim() && proxyMode !== "system") {
+    const proxy = normalizeProxyUrl(proxyMode.trim());
+    return { mode: "config", httpProxy: proxy, httpsProxy: proxy, noProxy: explicitNoProxy ?? env.NO_PROXY ?? env.no_proxy ?? null };
+  }
+
+  const envProxy = proxyFromEnv(env, explicitNoProxy);
+  if (envProxy.httpProxy || envProxy.httpsProxy) return { mode: "env", ...envProxy };
+
+  const systemProxy = platform === "win32" ? detectWindowsProxy() : null;
+  if (systemProxy?.httpProxy || systemProxy?.httpsProxy) {
+    return {
+      mode: "system",
+      httpProxy: systemProxy.httpProxy,
+      httpsProxy: systemProxy.httpsProxy,
+      noProxy: explicitNoProxy ?? systemProxy.noProxy ?? null,
+    };
+  }
+
+  return { mode: "direct", httpProxy: null, httpsProxy: null, noProxy: explicitNoProxy ?? null };
+}
+
+function installProxyDispatcher(proxy) {
+  setGlobalDispatcher(new EnvHttpProxyAgent({
+    httpProxy: proxy.httpProxy ?? "",
+    httpsProxy: proxy.httpsProxy ?? "",
+    noProxy: proxy.noProxy ?? "",
+    bodyTimeout: 0,
+    headersTimeout: 0,
+  }));
+}
+
+export function installDefaultCertificates(caConfig = "system") {
+  const entries = Array.isArray(caConfig) ? caConfig : [caConfig];
+  const wantsSystem = entries.includes("system");
+  const pemPaths = entries.filter((entry) => typeof entry === "string" && entry && entry !== "system");
+
+  if (!wantsSystem && pemPaths.length === 0) return { mode: "default", system: false, extraFiles: [] };
+  if (typeof tls.setDefaultCACertificates !== "function" || typeof tls.getCACertificates !== "function") {
+    return { mode: "unsupported", system: false, extraFiles: [] };
+  }
+
+  const certificates = [
+    ...tls.getCACertificates("default"),
+    ...(wantsSystem ? tls.getCACertificates("system") : []),
+    ...pemPaths.map((path) => readFileSync(path, "utf8")),
+  ];
+  tls.setDefaultCACertificates([...new Set(certificates)]);
+  return { mode: "installed", system: wantsSystem, extraFiles: pemPaths };
+}
+
+function proxyFromEnv(env, explicitNoProxy) {
+  const httpsProxy = env.HTTPS_PROXY ?? env.https_proxy ?? env.ALL_PROXY ?? env.all_proxy ?? null;
+  const httpProxy = env.HTTP_PROXY ?? env.http_proxy ?? env.ALL_PROXY ?? env.all_proxy ?? null;
+  const normalizedHttp = httpProxy ? normalizeProxyUrl(httpProxy) : null;
+  return {
+    httpProxy: normalizedHttp,
+    httpsProxy: httpsProxy ? normalizeProxyUrl(httpsProxy) : normalizedHttp,
+    noProxy: explicitNoProxy ?? env.NO_PROXY ?? env.no_proxy ?? null,
+  };
+}
+
+function detectWindowsProxy() {
+  try {
+    const output = execFileSync("reg", ["query", "HKCU\\Software\\Microsoft\\Windows\\CurrentVersion\\Internet Settings"], { encoding: "utf8", windowsHide: true });
+    const values = parseRegQuery(output);
+    if (values.ProxyEnable !== "0x1") return null;
+    return parseWindowsProxyServer(values.ProxyServer, values.ProxyOverride);
+  } catch {
+    return null;
+  }
+}
+
+function parseRegQuery(output) {
+  const values = {};
+  for (const line of output.split(/\r?\n/)) {
+    const match = line.trim().match(/^(\S+)\s+REG_\S+\s+(.+)$/);
+    if (match) values[match[1]] = match[2].trim();
+  }
+  return values;
+}
+
+function parseWindowsProxyServer(proxyServer, proxyOverride) {
+  if (!proxyServer) return null;
+  const entries = Object.fromEntries(proxyServer.split(";").map((part) => part.split("=")).filter((part) => part.length === 2));
+  const fallback = proxyServer.includes("=") ? null : proxyServer;
+  return {
+    httpProxy: normalizeProxyUrl(entries.http ?? fallback),
+    httpsProxy: normalizeProxyUrl(entries.https ?? entries.http ?? fallback),
+    noProxy: formatWindowsProxyOverride(proxyOverride),
+  };
+}
+
+function normalizeProxyUrl(value) {
+  if (!value) return null;
+  return /^[a-z][a-z0-9+.-]*:\/\//i.test(value) ? value : `http://${value}`;
+}
+
+function formatNoProxy(value) {
+  if (Array.isArray(value)) return value.join(",");
+  if (typeof value === "string") return value;
+  return null;
+}
+
+function formatWindowsProxyOverride(value) {
+  if (!value) return null;
+  return value
+    .split(";")
+    .map((entry) => entry.trim())
+    .filter((entry) => entry && entry !== "<local>")
+    .join(",") || null;
+}