muonroi-cli 1.8.0 → 1.8.1

package/dist/src/utils/install-manager.test.js

@@ -1,7 +1,15 @@
+import { exec } from "child_process";
 import fs from "fs";
 import os from "os";
 import path from "path";
-import { afterEach, describe, expect, it } from "vitest";
+import { afterEach, beforeEach, describe, expect, it, vi } from "vitest";
+vi.mock("child_process", async () => {
+    const actual = await vi.importActual("child_process");
+    return {
+        ...actual,
+        exec: vi.fn(),
+    };
+});
 import { buildScriptUninstallPlan, getInstallMetadataPath, getReleaseTargetForPlatform, getScriptInstallContext, getScriptInstallDir, loadScriptInstallMetadata, parseChecksumsFile, saveScriptInstallMetadata, } from "./install-manager.js";
 let tempDirs = [];
 afterEach(() => {
@@ -124,4 +132,38 @@ describe("buildScriptUninstallPlan", () => {
         expect(plan?.removePaths).toContain(path.join(installDir, currentTarget.binaryName));
     });
 });
+describe("runManagedUpdate", () => {
+    beforeEach(() => {
+        vi.stubGlobal("fetch", vi.fn());
+    });
+    afterEach(() => {
+        vi.restoreAllMocks();
+    });
+    it("handles dev-link update when newer version is available", async () => {
+        vi.mocked(exec).mockImplementation(((cmd, callback) => {
+            callback(null, "hash\trefs/tags/v2.0.0\n", "");
+            return {};
+        }));
+        const { runManagedUpdate } = await import("./install-manager.js");
+        const result = await runManagedUpdate("1.0.0");
+        expect(result.success).toBe(true);
+        expect(result.output).toContain("A new version of muonroi-cli is available!");
+        expect(result.output).toContain("Current version: v1.0.0");
+        expect(result.output).toContain("Latest version: v2.0.0");
+        expect(result.output).toContain("git -C");
+        expect(result.output).toContain("pull && bun install && bun run build");
+    });
+    it("handles dev-link when already up to date", async () => {
+        vi.mocked(exec).mockImplementation(((cmd, callback) => {
+            callback(null, "hash\trefs/tags/v1.0.0\n", "");
+            return {};
+        }));
+        const { runManagedUpdate } = await import("./install-manager.js");
+        const result = await runManagedUpdate("1.0.0");
+        expect(result.success).toBe(true);
+        expect(result.output).toContain("You are already up to date!");
+        expect(result.output).toContain("Current version: v1.0.0");
+        expect(result.output).toContain("Latest version: v1.0.0");
+    });
+});
 //# sourceMappingURL=install-manager.test.js.map

package/dist/src/utils/logger.d.ts

@@ -0,0 +1,28 @@
+export type LogLevel = "debug" | "info" | "warn" | "error";
+export type LogNamespace = "cli" | "ui" | "orchestrator" | "storage" | "ee" | "mcp" | "pil" | "router";
+export interface LogContext {
+    elapsedMs?: number;
+    error?: Error | unknown;
+    [key: string]: unknown;
+}
+/**
+ * Checks if the given log level is enabled based on the current process level weight.
+ */
+export declare function isLogLevelEnabled(level: LogLevel): boolean;
+/**
+ * Redacts common patterns of API keys and credential strings from log messages.
+ */
+export declare function redactSecrets(str: string): string;
+/**
+ * Recursively redacts sensitive fields from context objects.
+ */
+export declare function redactObject(obj: unknown): unknown;
+/**
+ * Structured unified logging system.
+ */
+export declare const logger: {
+    debug(ns: LogNamespace, msg: string, ctx?: LogContext): void;
+    info(ns: LogNamespace, msg: string, ctx?: LogContext): void;
+    warn(ns: LogNamespace, msg: string, ctx?: LogContext): void;
+    error(ns: LogNamespace, msg: string, ctx?: LogContext): void;
+};

package/dist/src/utils/logger.js

@@ -0,0 +1,171 @@
+import * as fs from "fs";
+import * as path from "path";
+import * as os from "os";
+const LEVEL_WEIGHTS = {
+    debug: 0,
+    info: 1,
+    warn: 2,
+    error: 3,
+};
+// Default log level is 'info' unless explicitly configured.
+const CURRENT_WEIGHT = (() => {
+    const envLevel = process.env.MUONROI_LOG_LEVEL?.toLowerCase();
+    if (envLevel === "debug")
+        return LEVEL_WEIGHTS.debug;
+    if (envLevel === "info")
+        return LEVEL_WEIGHTS.info;
+    if (envLevel === "warn")
+        return LEVEL_WEIGHTS.warn;
+    if (envLevel === "error")
+        return LEVEL_WEIGHTS.error;
+    return LEVEL_WEIGHTS.info;
+})();
+/**
+ * Checks if the given log level is enabled based on the current process level weight.
+ */
+export function isLogLevelEnabled(level) {
+    return LEVEL_WEIGHTS[level] >= CURRENT_WEIGHT;
+}
+/**
+ * Redacts common patterns of API keys and credential strings from log messages.
+ */
+export function redactSecrets(str) {
+    return str
+        .replace(/\bsk-[A-Za-z0-9-_]{20,}\b/g, "[REDACTED_API_KEY]")
+        .replace(/\bxai-[A-Za-z0-9-_]{20,}\b/g, "[REDACTED_API_KEY]")
+        .replace(/\bAIzaSy[A-Za-z0-9-_]{30,}\b/g, "[REDACTED_API_KEY]");
+}
+/**
+ * Recursively redacts sensitive fields from context objects.
+ */
+export function redactObject(obj) {
+    if (obj === null || obj === undefined)
+        return obj;
+    if (typeof obj === "string") {
+        return redactSecrets(obj);
+    }
+    if (Array.isArray(obj)) {
+        return obj.map(redactObject);
+    }
+    if (typeof obj === "object") {
+        const res = {};
+        for (const [k, v] of Object.entries(obj)) {
+            const lowerK = k.toLowerCase();
+            if (lowerK.includes("key") ||
+                lowerK.includes("secret") ||
+                lowerK.includes("token") ||
+                lowerK.includes("password") ||
+                lowerK.includes("auth")) {
+                res[k] = "[REDACTED]";
+            }
+            else {
+                res[k] = redactObject(v);
+            }
+        }
+        return res;
+    }
+    return obj;
+}
+/**
+ * Returns true if the interactive TUI is active, preventing console writes.
+ */
+function isTuiActive() {
+    try {
+        return globalThis.__muonroiTuiActive === true;
+    }
+    catch {
+        return false;
+    }
+}
+/**
+ * Writes logs safely to ~/.muonroi-cli/debug.log.
+ */
+function appendToFile(level, ns, msg, ctx) {
+    try {
+        const dir = path.join(os.homedir(), ".muonroi-cli");
+        if (!fs.existsSync(dir)) {
+            fs.mkdirSync(dir, { recursive: true });
+        }
+        const logPath = path.join(dir, "debug.log");
+        const ts = new Date().toISOString();
+        const redactedCtx = ctx ? redactObject(ctx) : null;
+        const ctxStr = redactedCtx ? ` ${JSON.stringify(redactedCtx)}` : "";
+        const logLine = `[${ts}] [${level.toUpperCase()}] [${ns.toUpperCase()}] ${redactSecrets(msg)}${ctxStr}\n`;
+        fs.appendFileSync(logPath, logLine, "utf8");
+    }
+    catch {
+        /* fail-open */
+    }
+}
+/**
+ * Formats console log lines with colors.
+ */
+function formatConsole(level, ns, msg, ctx) {
+    const ts = new Date().toISOString().split("T")[1].slice(0, -1); // HH:MM:SS.mmm
+    const levelStr = level.toUpperCase();
+    const nsStr = ns.toUpperCase();
+    const cleanMsg = redactSecrets(msg);
+    const redactedCtx = ctx ? redactObject(ctx) : null;
+    const ctxStr = redactedCtx ? ` ${JSON.stringify(redactedCtx)}` : "";
+    // Apply colors for developer convenience in terminal logs (non-TUI)
+    let color = "\x1b[0m"; // Reset
+    if (level === "debug")
+        color = "\x1b[90m"; // Gray
+    if (level === "info")
+        color = "\x1b[32m"; // Green
+    if (level === "warn")
+        color = "\x1b[33m"; // Yellow
+    if (level === "error")
+        color = "\x1b[31m"; // Red
+    return `${color}[${ts}] [${levelStr}] [${nsStr}] ${cleanMsg}${ctxStr}\x1b[0m`;
+}
+/**
+ * Structured unified logging system.
+ */
+export const logger = {
+    debug(ns, msg, ctx) {
+        if (!isLogLevelEnabled("debug"))
+            return;
+        if (isTuiActive()) {
+            appendToFile("debug", ns, msg, ctx);
+        }
+        else {
+            // eslint-disable-next-line no-console
+            console.log(formatConsole("debug", ns, msg, ctx));
+        }
+    },
+    info(ns, msg, ctx) {
+        if (!isLogLevelEnabled("info"))
+            return;
+        if (isTuiActive()) {
+            appendToFile("info", ns, msg, ctx);
+        }
+        else {
+            // eslint-disable-next-line no-console
+            console.log(formatConsole("info", ns, msg, ctx));
+        }
+    },
+    warn(ns, msg, ctx) {
+        if (!isLogLevelEnabled("warn"))
+            return;
+        if (isTuiActive()) {
+            appendToFile("warn", ns, msg, ctx);
+        }
+        else {
+            // eslint-disable-next-line no-console
+            console.warn(formatConsole("warn", ns, msg, ctx));
+        }
+    },
+    error(ns, msg, ctx) {
+        if (!isLogLevelEnabled("error"))
+            return;
+        if (isTuiActive()) {
+            appendToFile("error", ns, msg, ctx);
+        }
+        else {
+            // eslint-disable-next-line no-console
+            console.error(formatConsole("error", ns, msg, ctx));
+        }
+    },
+};
+//# sourceMappingURL=logger.js.map

package/dist/src/utils/settings.d.ts

@@ -75,6 +75,14 @@ export interface UserSettings {
      * defaultModel field stays as the hard pin for legacy paths.
      */
     defaultProvider?: ProviderId;
+    /** When true, the agent prioritizes task completion over strict runaway safety caps. */
+    agentFirst?: boolean;
+    /** Custom soft limit on the number of tool execution steps before pausing. */
+    maxToolRounds?: number;
+    /** Custom hard limit on the number of tool execution steps per turn. */
+    hardMaxToolRounds?: number;
+    /** Custom limit on the number of LLM call round-trips allowed in a single turn. */
+    maxLlmCallsPerTurn?: number;
     /** Shell used by the bash tool. On Windows, defaults to Git Bash when present. */
     shell?: ShellSettings;
     lsp?: LspSettings;
@@ -343,7 +351,7 @@ export declare function getTopLevelCompactKeepLast(): number;
  * higher default so single-tool turns are unaffected. Env override:
  * MUONROI_TOP_LEVEL_TOOL_BUDGET_CHARS.
  */
-export declare function getTopLevelToolBudgetChars(): number;
+export declare function getTopLevelToolBudgetChars(maxRounds?: number): number;
 export declare function getRoleModel(role: ModelRole): string | undefined;
 export declare function getRoleModels(): Partial<Record<ModelRole, string>>;
 export declare function getCouncilRounds(): number;

package/dist/src/utils/settings.js

@@ -4,6 +4,7 @@ import * as path from "path";
 import { getEffectiveReasoningEffort, getFirstCatalogModel, getFirstCatalogProvider, getModelByTier, getModelIds, getModelInfo, MODELS, normalizeModelId, } from "../models/registry.js";
 import { apiBaseFor, PROVIDER_ENDPOINTS } from "../providers/endpoints.js";
 import { ALL_PROVIDER_IDS } from "../providers/types.js";
+import { logger } from "./logger.js";
 import { normalizeShellSettings } from "./shell.js";
 export function getCatalogDefaultModel() {
     const provider = getDefaultProvider();
@@ -79,18 +80,58 @@ function ensureDir(dir) {
     }
 }
 function readJson(filePath) {
-    try {
-        if (!fs.existsSync(filePath))
-            return null;
-        return JSON.parse(fs.readFileSync(filePath, "utf-8"));
-    }
-    catch {
-        return null;
+    const RETRIES = 5;
+    let lastErr;
+    for (let attempt = 0; attempt < RETRIES; attempt++) {
+        try {
+            if (!fs.existsSync(filePath))
+                return null;
+            return JSON.parse(fs.readFileSync(filePath, "utf-8"));
+        }
+        catch (err) {
+            lastErr = err;
+            const code = err?.code;
+            if (code === "EBUSY" || code === "EPERM" || err instanceof SyntaxError) {
+                logger.warn("cli", `Lock contention or syntax error reading ${path.basename(filePath)}, retrying (attempt ${attempt + 1}/${RETRIES})`, { error: err });
+                // Spin wait briefly to let the lock release or the write complete
+                const end = Date.now() + 20 * (attempt + 1);
+                while (Date.now() < end) {
+                    // busy wait
+                }
+            }
+            else {
+                return null;
+            }
+        }
     }
+    return null;
 }
 function writeJson(filePath, data) {
     ensureDir(path.dirname(filePath));
-    fs.writeFileSync(filePath, JSON.stringify(data, null, 2), { mode: 0o600 });
+    const serialized = JSON.stringify(data, null, 2);
+    const RETRIES = 5;
+    let lastErr;
+    for (let attempt = 0; attempt < RETRIES; attempt++) {
+        try {
+            fs.writeFileSync(filePath, serialized, { mode: 0o600 });
+            return;
+        }
+        catch (err) {
+            lastErr = err;
+            const code = err?.code;
+            if (code === "EBUSY" || code === "EPERM") {
+                logger.warn("cli", `Lock contention writing ${path.basename(filePath)}, retrying (attempt ${attempt + 1}/${RETRIES})`, { error: err });
+                const end = Date.now() + 20 * (attempt + 1);
+                while (Date.now() < end) {
+                    // busy wait
+                }
+            }
+            else {
+                throw err;
+            }
+        }
+    }
+    throw lastErr;
 }
 /**
  * Ensure the CLI's own project-local footprint (`.muonroi-cli/`) is gitignored
@@ -559,13 +600,13 @@ export function getSubAgentBudgetChars() {
     const envRaw = process.env.MUONROI_SUB_AGENT_BUDGET_CHARS;
     if (envRaw) {
         const n = Number(envRaw);
-        if (Number.isFinite(n) && n >= 20_000 && n <= 600_000)
+        if (Number.isFinite(n) && n >= 20_000 && n <= 5_000_000)
             return Math.floor(n);
     }
     const val = loadUserSettings().subAgentBudgetChars;
-    if (typeof val === "number" && val >= 20_000 && val <= 600_000)
+    if (typeof val === "number" && val >= 20_000 && val <= 5_000_000)
         return Math.floor(val);
-    return 120_000;
+    return 240_000;
 }
 /**
  * Stall watchdog timeout (ms) for streaming model calls: if the provider sends
@@ -694,24 +735,20 @@ export function getTopLevelCompactKeepLast() {
  * higher default so single-tool turns are unaffected. Env override:
  * MUONROI_TOP_LEVEL_TOOL_BUDGET_CHARS.
  */
-export function getTopLevelToolBudgetChars() {
+export function getTopLevelToolBudgetChars(maxRounds) {
     const envRaw = process.env.MUONROI_TOP_LEVEL_TOOL_BUDGET_CHARS;
     if (envRaw) {
         const n = Number(envRaw);
-        if (Number.isFinite(n) && n >= 50_000 && n <= 1_500_000)
+        if (Number.isFinite(n) && n >= 50_000 && n <= 10_000_000)
             return Math.floor(n);
     }
     const val = loadUserSettings().topLevelToolBudgetChars;
-    if (typeof val === "number" && val >= 50_000 && val <= 1_500_000)
+    if (typeof val === "number" && val >= 50_000 && val <= 10_000_000)
         return Math.floor(val);
-    // Phase C5 symmetry — lowered from 400_000 to 200_000 chars. Evidence from
-    // session f1eef338c784: top-level tool loop ran 49 turns consuming 3.4M
-    // input tokens before the budget cap (set at 400K chars) meaningfully
-    // constrained tool outputs. At 200K chars the 50% tier fires at 100K chars
-    // (~3 tool turns), and the 80% tier at 160K chars (~5 turns); small tasks
-    // (1-3 turns) are unaffected. The sub-agent budget is 120K, so 200K maintains
-    // a ~1.7x ratio reflecting the broader top-level conversation context.
-    return 200_000;
+    // Dynamically scale default based on maxRounds relative to default base (40)
+    const baseRounds = 40;
+    const scale = maxRounds && maxRounds > baseRounds ? maxRounds / baseRounds : 1;
+    return Math.floor(400_000 * scale);
 }
 export function getRoleModel(role) {
     return loadUserSettings().roleModels?.[role];

package/dist/src/utils/side-question.js

@@ -1,7 +1,7 @@
 import { generateText } from "ai";
 import { resolveModelRuntime } from "../providers/runtime.js";
-const SIDE_QUESTION_SYSTEM = `You are a helpful coding assistant answering a quick side question. The user is in the middle of a coding session and needs a fast, concise answer. Keep your response short and focused — this is a side question, not the main task.
-
+const SIDE_QUESTION_SYSTEM = `You are a helpful coding assistant answering a quick side question. The user is in the middle of a coding session and needs a fast, concise answer. Keep your response short and focused — this is a side question, not the main task.
+
 If conversation context is provided below, use it to give a more relevant answer.`;
 export async function runSideQuestion(question, provider, modelId, conversationContext, signal) {
     const runtime = resolveModelRuntime(provider, modelId);

package/dist/src/utils/skills.js

@@ -163,9 +163,9 @@ export function discoverSkills(projectRoot) {
     _skillsCache = { skills, cachedAt: now, cwd: projectRoot };
     return skills;
 }
-const SKILLS_INSTRUCTIONS = `AGENT SKILLS (optional):
-The following <available_skills> list specialized workflows. Use them when they might help the user's request — not only on exact keyword matches.
-If a skill's description fits the task or could improve consistency, read that skill's instructions first using read_file with the path from <location>, then follow the SKILL.md body.
+const SKILLS_INSTRUCTIONS = `AGENT SKILLS (optional):
+The following <available_skills> list specialized workflows. Use them when they might help the user's request — not only on exact keyword matches.
+If a skill's description fits the task or could improve consistency, read that skill's instructions first using read_file with the path from <location>, then follow the SKILL.md body.
 Paths inside a skill (scripts/, references/, assets/) are relative to the skill directory (the folder containing SKILL.md); prefer absolute paths in tool calls.`;
 /** OpenCode-style XML catalog plus activation instructions for read_file. Returns null if no skills. */
 export function formatSkillsForPrompt(skills) {

package/dist/src/utils/update-checker.test.js

@@ -23,7 +23,11 @@ describe("checkForUpdate", () => {
         expect(result.hasUpdate).toBe(true);
         expect(result.latestVersion).toBe("2.0.0");
         expect(result.currentVersion).toBe("1.0.0");
-        expect(mockFetch).toHaveBeenCalledWith(RELEASE_URL, expect.objectContaining({ headers: { Accept: "application/vnd.github+json" } }));
+        expect(mockFetch).toHaveBeenCalledWith(RELEASE_URL, expect.objectContaining({
+            headers: expect.objectContaining({
+                Accept: "application/vnd.github+json",
+            }),
+        }));
     });
     it("returns hasUpdate=false when current version matches latest", async () => {
         vi.stubGlobal("fetch", vi.fn().mockResolvedValue({

package/dist/src/verify/__tests__/coverage-parsers.test.js

@@ -1,38 +1,38 @@
 import { describe, expect, it } from "vitest";
 import { extractCoverageFromOutput, parseBunCoverage, parseJestCoverage, parsePytestCoverage, parseVitestCoverage, } from "../coverage-parsers.js";
-const BUN_OUTPUT = `
-[0.12ms] 11 tests passed
-[0.00ms] 0 tests failed
-
------------------------|---------|---------|---------|---------|-----------------------
-File                   | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
------------------------|---------|---------|---------|---------|-----------------------
-All files              |   85.50 |   70.00 |   90.00 |   85.50 |
- index.ts              |   85.50 |   70.00 |   90.00 |   85.50 | 10-15
------------------------|---------|---------|---------|---------|-----------------------
+const BUN_OUTPUT = `
+[0.12ms] 11 tests passed
+[0.00ms] 0 tests failed
+
+-----------------------|---------|---------|---------|---------|-----------------------
+File                   | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
+-----------------------|---------|---------|---------|---------|-----------------------
+All files              |   85.50 |   70.00 |   90.00 |   85.50 |
+ index.ts              |   85.50 |   70.00 |   90.00 |   85.50 | 10-15
+-----------------------|---------|---------|---------|---------|-----------------------
 `;
-const VITEST_OUTPUT = `
-  Files                    | % Stmts | % Branch | % Funcs | % Lines | Uncovered Lines
- --------------------------|---------|---------|---------|---------|-----------------
-  All files                |   92.31 |   85.71 |     100 |   92.31 | 
-   reality-anchor.ts       |     100 |     100 |     100 |     100 | 
-   verify-result.ts        |   83.33 |      75 |     100 |   83.33 | 12-15
- --------------------------|---------|---------|---------|---------|-----------------
+const VITEST_OUTPUT = `
+  Files                    | % Stmts | % Branch | % Funcs | % Lines | Uncovered Lines
+ --------------------------|---------|---------|---------|---------|-----------------
+  All files                |   92.31 |   85.71 |     100 |   92.31 | 
+   reality-anchor.ts       |     100 |     100 |     100 |     100 | 
+   verify-result.ts        |   83.33 |      75 |     100 |   83.33 | 12-15
+ --------------------------|---------|---------|---------|---------|-----------------
 `;
-const JEST_OUTPUT = `
-----------|---------|----------|---------|---------|-------------------
-File      | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
-----------|---------|----------|---------|---------|-------------------
-All files |   75.42 |    62.15 |   80.52 |   75.42 |                   
-----------|---------|----------|---------|---------|-------------------
+const JEST_OUTPUT = `
+----------|---------|----------|---------|---------|-------------------
+File      | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s 
+----------|---------|----------|---------|---------|-------------------
+All files |   75.42 |    62.15 |   80.52 |   75.42 |                   
+----------|---------|----------|---------|---------|-------------------
 `;
-const PYTEST_OUTPUT = `
-Name                      Stmts   Miss  Cover
----------------------------------------------
-muonroi/__init__.py           0      0   100%
-muonroi/cli.py               42      8    81%
----------------------------------------------
-TOTAL                        42      8    81%
+const PYTEST_OUTPUT = `
+Name                      Stmts   Miss  Cover
+---------------------------------------------
+muonroi/__init__.py           0      0   100%
+muonroi/cli.py               42      8    81%
+---------------------------------------------
+TOTAL                        42      8    81%
 `;
 describe("Coverage Parsers", () => {
     it("should parse bun coverage", () => {

package/package.json

@@ -3,7 +3,7 @@
   "workspaces": [
     "packages/*"
   ],
-  "version": "1.8.0",
+  "version": "1.8.1",
   "description": "BYOK AI coding agent with multi-model council debate, role-based routing, and auto-compact.",
   "repository": {
     "type": "git",