arcane-cli 1.8.1 → 1.9.1

package/dist/assets/.mcp.json

@@ -1,9 +1,9 @@
 {
-  "$comment": "Example MCP server config. Replace or remove the example server below with your own. Set a per-server \"timeout\" (milliseconds) so a hung server aborts on a predictable schedule instead of running to the client's own default idle limit (as long as 30 minutes) -- see .arcane/governance/git-conventions.md's MCP fail-fast / fallback rule.",
+  "$comment": "Example MCP server config. Replace or remove the example server below with your own: the package argument is a placeholder that npm rejects, so the example does not run until you name a real server package. Set a per-server \"timeout\" (milliseconds) so a hung server aborts on a predictable schedule instead of running to the client's own default idle limit (as long as 30 minutes) -- see .arcane/governance/git-conventions.md's MCP fail-fast / fallback rule.",
   "mcpServers": {
     "example-server": {
       "command": "npx",
-      "args": ["-y", "@example/mcp-server"],
+      "args": ["-y", "<your-mcp-server-package>"],
       "timeout": 30000
     }
   }

package/dist/index.js

@@ -5896,8 +5896,10 @@ async function scanRepository(repoRoot, rules, extensions, excludePrefixes = [])
 
 // src/modules/exec.ts
 import { execFile as execFile2 } from "child_process";
+import { existsSync, readFileSync } from "fs";
+import { win32 } from "path";
 var EXTERNAL_CLI_TIMEOUT_MS = 1e4;
-function execFileWithTimeout(file, args, timeoutMs) {
+function execFileWithTimeout(file, args, timeoutMs, options = {}) {
   return new Promise((resolve5, reject) => {
     const child = execFile2(
       file,
@@ -5906,7 +5908,8 @@ function execFileWithTimeout(file, args, timeoutMs) {
         timeout: timeoutMs,
         killSignal: "SIGTERM",
         windowsHide: true,
-        encoding: "utf8"
+        encoding: "utf8",
+        ...options.env ? { env: { ...process.env, ...options.env } } : {}
       },
       (error, stdout, stderr) => {
         if (error) {
@@ -5919,6 +5922,36 @@ function execFileWithTimeout(file, args, timeoutMs) {
     child.stdin?.end();
   });
 }
+function defaultProbes() {
+  return {
+    platform: process.platform,
+    pathEnv: process.env["PATH"] ?? process.env["Path"],
+    readText: (path4) => {
+      try {
+        return readFileSync(path4, "utf8");
+      } catch {
+        return void 0;
+      }
+    },
+    exists: (path4) => existsSync(path4)
+  };
+}
+var MSI_LAUNCHER_LINE = /^\s*"%~dp0\\\.\.\\python\.exe"\s+-IBm\s+azure\.cli\s+%\*\s*$/im;
+function resolveAzureCli(probes = defaultProbes()) {
+  const plain = { file: "az", prefixArgs: [] };
+  if (probes.platform !== "win32") return plain;
+  for (const raw of (probes.pathEnv ?? "").split(";")) {
+    const dir = raw.trim().replace(/^"(.*)"$/, "$1");
+    if (!dir || !win32.isAbsolute(dir)) continue;
+    const launcher = probes.readText(win32.join(dir, "az.cmd"));
+    if (launcher === void 0) continue;
+    if (!MSI_LAUNCHER_LINE.test(launcher)) return plain;
+    const interpreter = win32.resolve(dir, "..", "python.exe");
+    if (!probes.exists(interpreter)) return plain;
+    return { file: interpreter, prefixArgs: ["-IBm", "azure.cli"], env: { AZ_INSTALLER: "MSI" } };
+  }
+  return plain;
+}
 
 // src/modules/platform-policy.ts
 var SANCTIONED_MERGE_METHODS = ["merge", "rebase"];
@@ -6028,10 +6061,13 @@ function evaluateAdoMergePolicy(policies) {
   }
   return compareToLadder([...effective], []);
 }
-async function fetchAdoMergeTypePolicies(org, project, repoName, branch) {
+async function fetchAdoMergeTypePolicies(org, project, repoName, branch, launch = resolveAzureCli()) {
+  const az = (args) => execFileWithTimeout(launch.file, [...launch.prefixArgs, ...args], EXTERNAL_CLI_TIMEOUT_MS, {
+    ...launch.env ? { env: launch.env } : {}
+  });
   try {
     const orgUrl = `https://dev.azure.com/${org}`;
-    const { stdout: repoOut } = await execFileWithTimeout("az", [
+    const { stdout: repoOut } = await az([
       "repos",
       "show",
       "--repository",
@@ -6044,10 +6080,10 @@ async function fetchAdoMergeTypePolicies(org, project, repoName, branch) {
       "id",
       "--output",
       "tsv"
-    ], EXTERNAL_CLI_TIMEOUT_MS);
+    ]);
     const repositoryId = repoOut.trim();
     if (!repositoryId) return null;
-    const { stdout } = await execFileWithTimeout("az", [
+    const { stdout } = await az([
       "repos",
       "policy",
       "list",
@@ -6061,7 +6097,7 @@ async function fetchAdoMergeTypePolicies(org, project, repoName, branch) {
       project,
       "--output",
       "json"
-    ], EXTERNAL_CLI_TIMEOUT_MS);
+    ]);
     return JSON.parse(stdout);
   } catch {
     return null;
@@ -6552,6 +6588,7 @@ async function checkGovernancePlaceholders(targetDir) {
     message: `unknown placeholder(s) in status: active doc(s) \u2014 ${listed.join("; ")}. Fill each one in; or, if an agent resolves it at use time, add it to the runtime-placeholders list in .arcane/governance/${PLACEHOLDER_STANDARD_FILE}; or, if the document is meant to be filled in by hand, mark it \`status: template\`.`
   };
 }
+var UNOWNED_EXAMPLE_SCOPE = "@example/";
 async function checkComponentRequires(targetDir) {
   const name = "Component prerequisites";
   let manifest;
@@ -6611,6 +6648,22 @@ async function checkMcpConfig(targetDir) {
   if (serverNames.length === 0) {
     return { name, passed: true, blocking: false, message: "no MCP servers configured" };
   }
+  const exampleHits = [];
+  for (const n of serverNames) {
+    const args = servers[n]?.args;
+    if (!Array.isArray(args)) continue;
+    for (const a of args) {
+      if (typeof a === "string" && a.startsWith(UNOWNED_EXAMPLE_SCOPE)) exampleHits.push(`${n} (${a})`);
+    }
+  }
+  if (exampleHits.length > 0) {
+    return {
+      name,
+      passed: false,
+      blocking: false,
+      message: `.mcp.json runs a package under the ${UNOWNED_EXAMPLE_SCOPE} scope nobody here owns: ${exampleHits.join(", ")}. The old \`spell init\` scaffold shipped it as a placeholder, and \`npx -y\` would run whoever publishes it. Replace the entry with your own MCP server package, or remove it; the file is yours, so doctor does not edit it`
+    };
+  }
   const missingTimeout = serverNames.filter((n) => typeof servers[n]?.timeout !== "number");
   if (missingTimeout.length > 0) {
     return {
@@ -7441,13 +7494,16 @@ async function getVersionAtRef(cwd, ref) {
     return null;
   }
 }
+function excludeOutputs(generatedOutputs) {
+  return generatedOutputs.length === 0 ? [] : ["--", ".", ...generatedOutputs.map((p) => `:(exclude)${p}`)];
+}
 async function getCloseCommit(cwd, completedDate, generatedOutputs = []) {
   if (!completedDate) {
-    const args = generatedOutputs.length === 0 ? ["rev-parse", "HEAD"] : ["log", "-1", "--format=%H", "HEAD", "--", ".", ...generatedOutputs.map((p) => `:(exclude)${p}`)];
+    const args = generatedOutputs.length === 0 ? ["rev-parse", "HEAD"] : ["log", "-1", "--format=%H", "HEAD", ...excludeOutputs(generatedOutputs)];
     const head = await runGitTextOrNull(cwd, args);
     return head === null || head === "" ? null : head;
   }
-  const log = await runGitTextOrNull(cwd, ["log", "--format=%H%x09%cs", "HEAD"]);
+  const log = await runGitTextOrNull(cwd, ["log", "--format=%H%x09%cs", "HEAD", ...excludeOutputs(generatedOutputs)]);
   if (log === null || log === "") return null;
   for (const line of log.split("\n")) {
     const [sha, landed] = line.split("	");
@@ -7735,6 +7791,33 @@ function renderShowReport(model, templateHtml) {
   });
 }
 
+// src/modules/show-report/theme.ts
+var REPORT_THEMES = ["auto", "light", "dark"];
+function radioTagPattern(id) {
+  return new RegExp(`<input\\b[^>]*\\bid="arcane-report-theme-${id}"[^>]*?/?>`);
+}
+function withoutChecked(tag) {
+  return tag.replace(/\s+checked(?:="[^"]*")?/, "");
+}
+function withChecked(tag) {
+  return /\bchecked\b/.test(tag) ? tag : tag.replace(/(\/?>)$/, " checked$1");
+}
+function applyReportTheme(html, theme) {
+  if (theme === "auto") return { html, applied: true };
+  if (!radioTagPattern(theme).test(html)) return { html, applied: false };
+  let result = html;
+  for (const id of REPORT_THEMES) {
+    const match = radioTagPattern(id).exec(result);
+    if (!match) continue;
+    const original = match[0];
+    const replaced = id === theme ? withChecked(original) : withoutChecked(original);
+    if (replaced !== original) {
+      result = result.slice(0, match.index) + replaced + result.slice(match.index + original.length);
+    }
+  }
+  return { html: result, applied: true };
+}
+
 // src/modules/show-report/generate.ts
 var TEMPLATE_VERSION = "v0-interim";
 var NO_EPICS_REASON = "no parseable `- [x] **ID \u2014 Title.**` epic bullets";
@@ -7801,7 +7884,7 @@ async function readOptionalFile(path4) {
     throw error;
   }
 }
-async function generateReport(rootDir, program2, templateHtml) {
+async function generateReport(rootDir, program2, templateHtml, theme = "auto") {
   const planContent = await readFile15(join18(rootDir, program2.planRelPath), "utf8");
   const frontmatter = parseFrontmatter(planContent);
   const model = await buildShowReportModel({
@@ -7813,19 +7896,24 @@ async function generateReport(rootDir, program2, templateHtml) {
     compiledAt: deterministicCompiledAt(frontmatter),
     templateVersion: TEMPLATE_VERSION
   });
+  const themed = applyReportTheme(renderShowReport(model, templateHtml), theme);
   return {
     json: `${JSON.stringify(model, null, 2)}
 `,
-    html: renderShowReport(model, templateHtml)
+    html: themed.html,
+    themeApplied: themed.applied
   };
 }
 async function runReportGeneration(options) {
   const templateHtml = await readFile15(options.templatePath, "utf8");
+  const theme = options.theme ?? "auto";
   const drifted = [];
   const repaired = [];
   const unwritten = [];
+  const themeSkipped = [];
   for (const program2 of options.programs) {
-    const generated = await generateReport(options.rootDir, program2, templateHtml);
+    const generated = await generateReport(options.rootDir, program2, templateHtml, theme);
+    if (!generated.themeApplied) themeSkipped.push(program2.slug);
     const missing = JSON.parse(generated.json).sections.flatMap((section) => section.rows).filter((row) => row.descriptionState === "unwritten").map((row) => row.id);
     if (missing.length > 0) unwritten.push({ slug: program2.slug, ids: missing });
     const outAbs = options.outDir ? isAbsolute(options.outDir) ? options.outDir : resolve4(options.rootDir, options.outDir) : resolve4(options.rootDir, dirname10(program2.planRelPath));
@@ -7846,7 +7934,7 @@ async function runReportGeneration(options) {
       }
     }
   }
-  return { drifted, repaired, unwritten };
+  return { drifted, repaired, unwritten, themeSkipped };
 }
 
 // src/commands/report.ts
@@ -7866,6 +7954,14 @@ async function runReport(targetDir, options = {}, assetsDir) {
       "--refresh: external-data snapshotting (gh PR titles/merge dates) is not implemented yet; regenerating from local sources."
     );
   }
+  const theme = options.theme ?? "auto";
+  if (!REPORT_THEMES.includes(theme)) {
+    console.error(
+      `\u2716 arcane: --theme must be one of ${REPORT_THEMES.join(" | ")}, got "${options.theme}".`
+    );
+    process.exitCode = 1;
+    return;
+  }
   if (await isShallowRepository(targetDir)) {
     console.warn(
       "\u26A0 arcane: shallow clone -- the report's version span and cast are derived from git history and will be omitted. Run `git fetch --unshallow` for a complete report."
@@ -7895,13 +7991,19 @@ async function runReport(targetDir, options = {}, assetsDir) {
     templatePath,
     mode: "write",
     programs,
-    outDir: options.out
+    outDir: options.out,
+    theme
   });
   for (const u of result.unwritten) {
     console.warn(
       `\u26A0 arcane: ${u.slug} has ${u.ids.length} epic(s) with no \`**Report:**\` line in PLAN.md -- they render as "unwritten": ${u.ids.join(", ")}.`
     );
   }
+  for (const slug of result.themeSkipped) {
+    console.warn(
+      `\u26A0 arcane: ${slug} -- --theme ${theme} requested, but this template has no theme toggle to set (older vendored template?); left unchanged.`
+    );
+  }
   if (result.repaired.length === 0) {
     console.log(`Show report: ${programs.length} program(s) already up to date.`);
     return;
@@ -8054,6 +8156,9 @@ program.command("report").description(
 ).option("--plan <path>", "Report on one PLAN.md instead of auto-discovering docs/plans/*/PLAN.md").option("--out <dir>", "Write show-report.{json,html} into this directory instead of next to the plan").option(
   "--refresh",
   "Accepted for forward compatibility; external-data snapshotting is not implemented yet, so this regenerates from local sources"
+).option(
+  "--theme <mode>",
+  "Force the in-page toggle's initial state: auto | light | dark (default auto, byte-for-byte unchanged). Pair with --out for a one-off themed copy so it doesn't overwrite the tracked show-report.html"
 ).action(async (opts) => {
   await runReport(process.cwd(), opts, ASSETS_DIR);
 });