@qodalis/cli 2.0.0-beta.8 → 2.0.0-beta.9

package/public-api.js

@@ -411,7 +411,7 @@ var CliBackgroundServiceRegistry_TOKEN = "cli-background-service-registry";
 var CliPackageManagerService_TOKEN = "cli-package-manager-service";
 
 // src/lib/version.ts
-var LIBRARY_VERSION = "2.0.0-beta.8";
+var LIBRARY_VERSION = "2.0.0-beta.9";
 var API_VERSION = 2;
 var CliAliasCommandProcessor = class {
   constructor() {
@@ -1337,6 +1337,13 @@ var CliEvalCommandProcessor = class {
     };
   }
   async processCommand(command, context) {
+    if (!context.options?.allowEval) {
+      context.writer.writeError(
+        "eval is disabled by default for security. Set allowEval: true in CliOptions to enable it."
+      );
+      context.process.exit(1, { silent: true });
+      return;
+    }
     try {
       const output = eval(command.value ?? "");
       if (Array.isArray(output)) {
@@ -2698,6 +2705,78 @@ var SYSTEM_OPTIONS = [
     options: [
       { label: "English", value: "en" }
     ]
+  },
+  {
+    key: "cursorBlink",
+    label: "Cursor Blink",
+    description: "Whether the terminal cursor blinks",
+    type: "boolean",
+    defaultValue: true
+  },
+  {
+    key: "cursorStyle",
+    label: "Cursor Style",
+    description: "Terminal cursor shape",
+    type: "select",
+    defaultValue: "block",
+    options: [
+      { label: "Block", value: "block" },
+      { label: "Underline", value: "underline" },
+      { label: "Bar", value: "bar" }
+    ]
+  },
+  {
+    key: "scrollback",
+    label: "Scrollback",
+    description: "Number of scrollback lines to retain",
+    type: "number",
+    defaultValue: 1e3,
+    validator: (value) => {
+      if (value < 0 || value > 1e5) {
+        return { valid: false, message: "Scrollback must be between 0 and 100,000" };
+      }
+      return { valid: true };
+    }
+  },
+  {
+    key: "fontSize",
+    label: "Font Size",
+    description: "Terminal font size in pixels",
+    type: "number",
+    defaultValue: 20,
+    validator: (value) => {
+      if (value < 8 || value > 40) {
+        return { valid: false, message: "Font size must be between 8 and 40" };
+      }
+      return { valid: true };
+    }
+  },
+  {
+    key: "fontFamily",
+    label: "Font Family",
+    description: "Terminal font family",
+    type: "string",
+    defaultValue: "monospace"
+  },
+  {
+    key: "smoothScrollDuration",
+    label: "Smooth Scroll",
+    description: "Smooth scroll duration in milliseconds (0 to disable)",
+    type: "number",
+    defaultValue: 0,
+    validator: (value) => {
+      if (value < 0 || value > 1e3) {
+        return { valid: false, message: "Smooth scroll duration must be between 0 and 1,000 ms" };
+      }
+      return { valid: true };
+    }
+  },
+  {
+    key: "greeting",
+    label: "Greeting",
+    description: "Show animated time-based greeting on startup",
+    type: "boolean",
+    defaultValue: true
   }
 ];
 var LOG_LEVEL_MAP = {
@@ -2996,6 +3075,26 @@ var CliConfigureCommandProcessor = class _CliConfigureCommandProcessor {
       } catch {
       }
     }
+    if (context2.terminal?.options) {
+      if (settings["cursorBlink"] !== void 0) {
+        context2.terminal.options.cursorBlink = settings["cursorBlink"];
+      }
+      if (settings["cursorStyle"]) {
+        context2.terminal.options.cursorStyle = settings["cursorStyle"];
+      }
+      if (settings["scrollback"] !== void 0) {
+        context2.terminal.options.scrollback = settings["scrollback"];
+      }
+      if (settings["fontSize"] !== void 0) {
+        context2.terminal.options.fontSize = settings["fontSize"];
+      }
+      if (settings["fontFamily"]) {
+        context2.terminal.options.fontFamily = settings["fontFamily"];
+      }
+      if (settings["smoothScrollDuration"] !== void 0) {
+        context2.terminal.options.smoothScrollDuration = settings["smoothScrollDuration"];
+      }
+    }
   }
   // ── Child processor builders ────────────────────────────────────────
   buildListProcessor() {
@@ -3097,6 +3196,7 @@ var CliConfigureCommandProcessor = class _CliConfigureCommandProcessor {
       command: "set",
       description: "Set a configuration value (e.g. configure set system.logLevel Debug)",
       valueRequired: true,
+      acceptsRawInput: true,
       processCommand: async (command2, context2) => {
         const raw = command2.value;
         const spaceIndex = raw.indexOf(" ");
@@ -3240,12 +3340,15 @@ var CliConfigureCommandProcessor = class _CliConfigureCommandProcessor {
         }
         return { value: num };
       }
-      case "boolean":
+      case "boolean": {
+        const lower = rawValue.toLowerCase();
+        if (["true", "1", "yes"].includes(lower)) return { value: true };
+        if (["false", "0", "no"].includes(lower)) return { value: false };
         return {
-          value: ["true", "1", "yes"].includes(
-            rawValue.toLowerCase()
-          )
+          value: null,
+          error: translator.t("cli.configure.invalid_boolean", 'Invalid boolean value: "{value}". Use true or false.', { value: rawValue })
         };
+      }
       case "select":
         if (option.options) {
           const match = option.options.find(
@@ -4430,6 +4533,13 @@ var ScriptLoaderService = class {
       xhr.send();
     });
   }
+  /**
+   * Inject an inline script into the page.
+   *
+   * **Security:** This runs `code` in the host page context with full
+   * DOM access. Only call with trusted content (e.g. verified UMD bundles
+   * loaded via the package manager). Never pass user-supplied strings.
+   */
   injectBodyScript(code) {
     const script = document.createElement("script");
     script.text = code;
@@ -4497,6 +4607,10 @@ var CliPackageManagerService = class {
   constructor() {
     this.storageKey = "cli-packages";
     this.QODALIS_COMMAND_PREFIX = "@qodalis/cli-";
+    /** In-memory cache; `null` means "not yet loaded from storage". */
+    this.cache = null;
+    /** Whether the one-time localStorage migration has been checked. */
+    this.migrationDone = false;
   }
   /**
    * Set the key-value store instance for persistence.
@@ -4505,17 +4619,27 @@ var CliPackageManagerService = class {
     this.store = store;
   }
   /**
-   * Retrieves the list of packages
-   * @returns {Package[]} Array of packages
+   * Retrieves the list of packages.
+   * Uses an in-memory cache after the first load.
    */
   async getPackages() {
-    const oldPackages = localStorage.getItem("cliPackages");
-    if (oldPackages) {
-      localStorage.removeItem("cliPackages");
-      await this.store.set(this.storageKey, JSON.parse(oldPackages));
+    if (this.cache !== null) {
+      return this.cache;
+    }
+    if (!this.migrationDone) {
+      this.migrationDone = true;
+      try {
+        const oldPackages = localStorage.getItem("cliPackages");
+        if (oldPackages) {
+          localStorage.removeItem("cliPackages");
+          await this.store.set(this.storageKey, JSON.parse(oldPackages));
+        }
+      } catch {
+      }
     }
     const packages = await this.store.get(this.storageKey);
-    return packages ? packages : [];
+    this.cache = packages ?? [];
+    return this.cache;
   }
   /**
    * Retrieves a package by name from the list.
@@ -4559,7 +4683,7 @@ var CliPackageManagerService = class {
     if (packages.length === updatedPackages.length) {
       throw new Error(`Package with name "${packageName}" not found.`);
     }
-    this.savePackages(updatedPackages);
+    await this.savePackages(updatedPackages);
     return packageToRemove;
   }
   /**
@@ -4575,6 +4699,7 @@ var CliPackageManagerService = class {
     await this.savePackages(packages);
   }
   async savePackages(packages) {
+    this.cache = packages;
     await this.store.set(this.storageKey, packages);
   }
 };
@@ -5651,9 +5776,10 @@ var CliPackagesCommandProcessor = class {
 
 // src/lib/server/cli-server-connection.ts
 var CliServerConnection = class {
-  constructor(_config, _backgroundServices) {
+  constructor(_config, _backgroundServices, _logger) {
     this._config = _config;
     this._backgroundServices = _backgroundServices;
+    this._logger = _logger;
     this._connected = false;
     this._commands = [];
     this._capabilities = null;
@@ -5677,7 +5803,8 @@ var CliServerConnection = class {
       this._connected = true;
       this._capabilities = await this.fetchCapabilities();
       this.connectEventSocket();
-    } catch {
+    } catch (e) {
+      this._logger?.warn(`Failed to connect to server "${this._config.name}": ${e}`);
       this._connected = false;
       this._commands = [];
       this._capabilities = null;
@@ -5705,7 +5832,8 @@ var CliServerConnection = class {
       const response = await this.httpFetch(url);
       if (!response.ok) return null;
       return response.json();
-    } catch {
+    } catch (e) {
+      this._logger?.debug(`Failed to fetch capabilities for "${this._config.name}": ${e}`);
       return null;
     }
   }
@@ -5735,7 +5863,8 @@ var CliServerConnection = class {
       const url = `${this.normalizeUrl(this._config.url)}/api/v1/qcli/version`;
       const response = await this.httpFetch(url);
       return response.ok;
-    } catch {
+    } catch (e) {
+      this._logger?.debug(`Ping failed for "${this._config.name}": ${e}`);
       return false;
     }
   }
@@ -5783,10 +5912,12 @@ var CliServerConnection = class {
           this.closeEventSocketDirect();
         }
       });
-      this._backgroundServices.start(serviceName).catch(() => {
+      this._backgroundServices.start(serviceName).catch((e) => {
+        this._logger?.debug(`Background event service failed for "${this._config.name}", falling back to direct: ${e}`);
         this.connectEventSocketDirect();
       });
-    } catch {
+    } catch (e) {
+      this._logger?.debug(`Failed to register event service for "${this._config.name}", falling back to direct: ${e}`);
       this.connectEventSocketDirect();
     }
   }
@@ -5811,7 +5942,8 @@ var CliServerConnection = class {
       };
       this._eventSocket.onerror = () => {
       };
-    } catch {
+    } catch (e) {
+      this._logger?.debug(`WebSocket not available for "${this._config.name}": ${e}`);
     }
   }
   handleServerDisconnect() {
@@ -6098,12 +6230,13 @@ var CliServerManager = class {
     this.unregisterBareAliases();
     this.registerBareAliases();
   }
-  async connectAll(servers, logger, backgroundServices) {
+  async connectAll(servers, logger, backgroundServices, cliLogger) {
     this._logger = logger;
+    this._cliLogger = cliLogger;
     this._backgroundServices = backgroundServices;
     for (const config of servers) {
       if (config.enabled === false) continue;
-      const connection = new CliServerConnection(config, backgroundServices);
+      const connection = new CliServerConnection(config, backgroundServices, cliLogger);
       this.connections.set(config.name, connection);
       connection.onDisconnect = () => {
         this.handleDisconnect(config.name);
@@ -10547,22 +10680,31 @@ var CliCommandExecutionContext = class {
 var ServiceLogBuffer = class {
   constructor(maxSize = 1e3) {
     this.maxSize = maxSize;
-    this.entries = [];
+    this.head = 0;
+    this.size = 0;
+    this.buffer = new Array(maxSize);
   }
   add(message, level = "info") {
-    this.entries.push({ timestamp: /* @__PURE__ */ new Date(), level, message });
-    if (this.entries.length > this.maxSize) {
-      this.entries.shift();
+    const entry = { timestamp: /* @__PURE__ */ new Date(), level, message };
+    this.buffer[this.head] = entry;
+    this.head = (this.head + 1) % this.maxSize;
+    if (this.size < this.maxSize) {
+      this.size++;
     }
   }
   get(limit) {
-    if (limit === void 0) {
-      return [...this.entries];
+    const count = limit !== void 0 ? Math.min(limit, this.size) : this.size;
+    const result = new Array(count);
+    const start = (this.head - count + this.maxSize) % this.maxSize;
+    for (let i = 0; i < count; i++) {
+      result[i] = this.buffer[(start + i) % this.maxSize];
     }
-    return this.entries.slice(-limit);
+    return result;
   }
   clear() {
-    this.entries = [];
+    this.buffer = new Array(this.maxSize);
+    this.head = 0;
+    this.size = 0;
   }
 };
 
@@ -10708,18 +10850,19 @@ var CliWorkerServiceRunner = class {
   }
   async stop() {
     if (!this.worker) return;
+    const worker = this.worker;
     const stopMsg = { type: "stop" };
-    this.worker.postMessage(stopMsg);
+    worker.postMessage(stopMsg);
     await Promise.race([
       new Promise((resolve) => {
-        const originalOnMessage = this.worker.onmessage;
-        this.worker.onmessage = (ev) => {
-          if (ev.data.type === "status" && ev.data.status === "stopped") {
-            resolve();
-          }
+        const originalOnMessage = worker.onmessage;
+        worker.onmessage = (ev) => {
           if (originalOnMessage) {
             originalOnMessage(ev);
           }
+          if (ev.data.type === "status" && ev.data.status === "stopped") {
+            resolve();
+          }
         };
       }),
       new Promise((resolve) => setTimeout(resolve, 5e3))
@@ -10727,8 +10870,12 @@ var CliWorkerServiceRunner = class {
     this.terminate();
   }
   terminate() {
-    this.worker?.terminate();
-    this.worker = null;
+    if (this.worker) {
+      this.worker.onmessage = null;
+      this.worker.onerror = null;
+      this.worker.terminate();
+      this.worker = null;
+    }
   }
 };
 
@@ -10856,7 +11003,7 @@ var CliBackgroundServiceRegistry = class {
     }
     await this.start(name);
   }
-  startJob(name, task, _options) {
+  async startJob(name, task, _options) {
     const jobService = {
       name,
       type: "job",
@@ -11060,10 +11207,6 @@ var CliBackgroundServiceRegistry = class {
     }
   }
 };
-var ANSI_RE = /\x1b\[[0-9;]*m/g;
-function visibleLength(text) {
-  return text.replace(ANSI_RE, "").length;
-}
 var CliTerminalWriter = class {
   constructor(terminal) {
     this.terminal = terminal;
@@ -11162,15 +11305,48 @@ var CliTerminalWriter = class {
       this.writeln(row.map((item) => item.padEnd(colWidth)).join(""));
     }
   }
+  writeLink(text, url) {
+    this.writeln(`\x1B]8;;${url}\x07${text}\x1B]8;;\x07`);
+  }
+  writeBox(content, options) {
+    const lines = Array.isArray(content) ? content : [content];
+    const pad2 = options?.padding ?? 1;
+    const color = options?.borderColor;
+    const wrap = (s) => color ? this.wrapInColor(s, color) : s;
+    const maxContentWidth = Math.max(
+      ...lines.map((l) => cliCore.visibleLength(l)),
+      options?.title ? cliCore.visibleLength(options.title) + 2 : 0
+    );
+    const innerWidth = maxContentWidth + pad2 * 2;
+    if (options?.title) {
+      const title = ` ${options.title} `;
+      const remaining = innerWidth - cliCore.visibleLength(title);
+      const left = Math.floor(remaining / 2);
+      const right = remaining - left;
+      this.writeln(wrap("\u250C" + "\u2500".repeat(left) + title + "\u2500".repeat(right) + "\u2510"));
+    } else {
+      this.writeln(wrap("\u250C" + "\u2500".repeat(innerWidth) + "\u2510"));
+    }
+    const padStr = " ".repeat(pad2);
+    for (const line of lines) {
+      const rightPad = innerWidth - pad2 * 2 - cliCore.visibleLength(line);
+      this.writeln(wrap("\u2502") + padStr + line + " ".repeat(Math.max(0, rightPad)) + padStr + wrap("\u2502"));
+    }
+    this.writeln(wrap("\u2514" + "\u2500".repeat(innerWidth) + "\u2518"));
+  }
+  writeIndented(text, level = 1) {
+    const indent = "  ".repeat(level);
+    this.writeln(`${indent}${text}`);
+  }
   writeTable(headers, rows) {
     const colWidths = headers.map(
       (header, colIndex) => Math.max(
-        visibleLength(header),
-        ...rows.map((row) => visibleLength(row[colIndex] ?? ""))
+        cliCore.visibleLength(header),
+        ...rows.map((row) => cliCore.visibleLength(row[colIndex] ?? ""))
       )
     );
     const padText = (text, width) => {
-      const visible = visibleLength(text ?? "");
+      const visible = cliCore.visibleLength(text ?? "");
       const padding = Math.max(0, width - visible);
       return (text ?? "") + " ".repeat(padding);
     };
@@ -13936,8 +14112,8 @@ var CliExecutionContext = class {
     };
     this.services = deps.services;
     this.translator = deps.translator;
-    const stateStoreManager = deps.stateStoreManager;
-    this.state = stateStoreManager.getStateStore("shared");
+    this.stateStoreManager = deps.stateStoreManager;
+    this.state = this.stateStoreManager.getStateStore("shared");
     this.options = cliOptions;
     this.writer = new CliTerminalWriter(terminal);
     const spinner = new CliTerminalSpinner(terminal);
@@ -14007,7 +14183,7 @@ var CliExecutionContext = class {
     });
     const termEl = this.terminal.element;
     if (termEl) {
-      window.addEventListener("keydown", (event) => {
+      this.windowKeydownListener = (event) => {
         if (!termEl.contains(document.activeElement)) return;
         if (event.ctrlKey && !event.altKey && !event.metaKey) {
           const key = event.key.toLowerCase();
@@ -14015,7 +14191,8 @@ var CliExecutionContext = class {
             event.preventDefault();
           }
         }
-      }, true);
+      };
+      window.addEventListener("keydown", this.windowKeydownListener, true);
     }
   }
   setCurrentLine(line) {
@@ -14216,6 +14393,11 @@ var CliExecutionContext = class {
     this.clearAllManagedTimers();
     this.resizeDisposable?.dispose();
     this.resizeDisposable = null;
+    if (this.windowKeydownListener) {
+      window.removeEventListener("keydown", this.windowKeydownListener, true);
+      this.windowKeydownListener = void 0;
+    }
+    this.stateStoreManager.dispose();
   }
   // -- Mode stack management --
   get currentMode() {
@@ -14242,12 +14424,6 @@ var CliExecutionContext = class {
     }
   }
 };
-
-// src/lib/services/capturing-terminal-writer.ts
-var ANSI_RE2 = /\x1b\[[0-9;]*m/g;
-function stripAnsi(text) {
-  return text.replace(ANSI_RE2, "");
-}
 var CapturingTerminalWriter = class {
   constructor(inner) {
     this.inner = inner;
@@ -14262,12 +14438,12 @@ var CapturingTerminalWriter = class {
   }
   // -- stdout-equivalent methods (captured) --------------------------------
   write(text) {
-    this._lines.push(stripAnsi(text));
+    this._lines.push(cliCore.stripAnsi(text));
     this.inner.write(text);
   }
   writeln(text) {
     if (text !== void 0 && text !== "") {
-      this._lines.push(stripAnsi(text));
+      this._lines.push(cliCore.stripAnsi(text));
     }
     this.inner.writeln(text);
   }
@@ -14290,11 +14466,11 @@ var CapturingTerminalWriter = class {
     this.inner.writeInfo(message);
   }
   writeWarning(message) {
-    this._stderrLines.push(stripAnsi(message));
+    this._stderrLines.push(cliCore.stripAnsi(message));
     this.inner.writeWarning(message);
   }
   writeError(message) {
-    this._stderrLines.push(stripAnsi(message));
+    this._stderrLines.push(cliCore.stripAnsi(message));
     this.inner.writeError(message);
   }
   // -- pass-through methods ------------------------------------------------
@@ -14319,6 +14495,19 @@ var CapturingTerminalWriter = class {
   writeColumns(items, options) {
     this.inner.writeColumns(items, options);
   }
+  writeLink(text, url) {
+    this._lines.push(cliCore.stripAnsi(text));
+    this.inner.writeLink(text, url);
+  }
+  writeBox(content, options) {
+    const lines = Array.isArray(content) ? content : [content];
+    lines.forEach((l) => this._lines.push(cliCore.stripAnsi(l)));
+    this.inner.writeBox(content, options);
+  }
+  writeIndented(text, level) {
+    this._lines.push(cliCore.stripAnsi(text));
+    this.inner.writeIndented(text, level);
+  }
   // -- captured data accessors ---------------------------------------------
   /**
    * Returns true if any stdout-equivalent output was captured.
@@ -14450,6 +14639,7 @@ var CliCommandExecutor = class {
     let lastExitSuccess = true;
     let shouldRunNext = true;
     let pipelineData = void 0;
+    let lastCapturingWriter;
     for (let i = 0; i < parts.length; i++) {
       const part = parts[i];
       if (part.type === "&&") {
@@ -14500,7 +14690,7 @@ var CliCommandExecutor = class {
           continue;
         }
         if (shouldRunNext) {
-          await this.appendStderrToFile(nextPart.value, context2);
+          await this.appendStderrToFile(nextPart.value, context2, lastCapturingWriter);
         }
         continue;
       } else if (part.type === "2>") {
@@ -14512,7 +14702,7 @@ var CliCommandExecutor = class {
           continue;
         }
         if (shouldRunNext) {
-          await this.writeStderrToFile(nextPart.value, context2);
+          await this.writeStderrToFile(nextPart.value, context2, lastCapturingWriter);
         }
         continue;
       }
@@ -14520,7 +14710,7 @@ var CliCommandExecutor = class {
         continue;
       }
       try {
-        await this.executeSingleCommand(
+        lastCapturingWriter = await this.executeSingleCommand(
           part.value,
           pipelineData,
           rootContext
@@ -14732,7 +14922,6 @@ var CliCommandExecutor = class {
       commandContext.writer
     );
     commandContext.writer = capturingWriter;
-    this.lastCapturingWriter = capturingWriter;
     try {
       const hooks = processor.hooks ?? [];
       for (const hook of hooks.filter((h) => h.when === "before")) {
@@ -14758,6 +14947,7 @@ var CliCommandExecutor = class {
       if (!process.exited) {
         process.end();
       }
+      return capturingWriter;
     } catch (e) {
       context2.spinner?.hide();
       if (e instanceof ProcessExitedError) {
@@ -14785,6 +14975,7 @@ var CliCommandExecutor = class {
     } finally {
       abortSub.unsubscribe();
     }
+    return capturingWriter;
   }
   completeProcess(processEntry, context2, exitCode) {
     if (!processEntry) return;
@@ -14802,8 +14993,8 @@ var CliCommandExecutor = class {
     } catch {
     }
   }
-  async writeStderrToFile(filePath, context2) {
-    const stderr = this.lastCapturingWriter?.getCapturedStderr();
+  async writeStderrToFile(filePath, context2, capturingWriter) {
+    const stderr = capturingWriter?.getCapturedStderr();
     if (!stderr) return;
     const FS_TOKEN2 = "cli-file-system-service";
     let fs;
@@ -14827,8 +15018,8 @@ var CliCommandExecutor = class {
       context2.writer.writeError(`2> failed: ${e.message || e}`);
     }
   }
-  async appendStderrToFile(filePath, context2) {
-    const stderr = this.lastCapturingWriter?.getCapturedStderr();
+  async appendStderrToFile(filePath, context2, capturingWriter) {
+    const stderr = capturingWriter?.getCapturedStderr();
     if (!stderr) return;
     const FS_TOKEN2 = "cli-file-system-service";
     let fs;
@@ -15338,10 +15529,27 @@ var welcomeModule = {
     }
     recordWelcomeMessageDisplay();
     context2.showPrompt();
-    await context2.textAnimator?.showText(getGreetingBasedOnTime(), {
-      speed: 60,
-      removeAfterTyping: true
-    });
+    let showGreeting = true;
+    try {
+      const storeManager = context2.services.get(
+        CliStateStoreManager_TOKEN
+      );
+      const configureStore = storeManager.getStateStore(
+        cliCore.CLI_CONFIGURE_STORE_NAME
+      );
+      await configureStore.initialize();
+      const configState = configureStore.getState();
+      if (configState?.["system"]?.["greeting"] === false) {
+        showGreeting = false;
+      }
+    } catch {
+    }
+    if (showGreeting) {
+      await context2.textAnimator?.showText(getGreetingBasedOnTime(), {
+        speed: 60,
+        removeAfterTyping: true
+      });
+    }
   }
 };
 function shouldDisplayWelcomeMessage(showOption) {
@@ -15377,6 +15585,7 @@ var CliBoot = class {
     this.initialized = false;
     this.initializing = false;
     this.bootedModules = /* @__PURE__ */ new Set();
+    this.initializedProcessors = /* @__PURE__ */ new Set();
     this.moduleRegistry = new cliCore.CliModuleRegistry();
   }
   /**
@@ -15393,6 +15602,7 @@ var CliBoot = class {
       return;
     }
     this.initializing = true;
+    this.moduleRegistry.onWarn = (msg) => context2.logger.warn(msg);
     cliCore.initializeBrowserEnvironment({
       context: context2,
       registry: this.moduleRegistry
@@ -15445,7 +15655,7 @@ var CliBoot = class {
       try {
         await module.onInit(context2);
       } catch (e) {
-        console.error(
+        context2.logger.error(
           `Error in onInit for module "${module.name}":`,
           e
         );
@@ -15468,7 +15678,7 @@ var CliBoot = class {
             });
           }
         } catch (e) {
-          console.error(
+          context2.logger.error(
             `Setup failed for module "${module.name}":`,
             e
           );
@@ -15558,14 +15768,16 @@ var CliBoot = class {
     });
   }
   async bootShared(context2) {
-    await this.initializeProcessorsInternal(
-      context2,
-      this.registry.processors
+    const uninitialized = this.registry.processors.filter(
+      (p) => !this.initializedProcessors.has(p)
     );
-    await cliCore.delay(300);
+    if (uninitialized.length > 0) {
+      await this.initializeProcessorsInternal(context2, uninitialized);
+    }
   }
   async initializeProcessorsInternal(context2, processors, parent) {
     for (const p of processors) {
+      if (this.initializedProcessors.has(p)) continue;
       try {
         p.parent = parent;
         if (p.initialize) {
@@ -15576,6 +15788,7 @@ var CliBoot = class {
           await processorContext.state.initialize();
           await p.initialize(processorContext);
         }
+        this.initializedProcessors.add(p);
         if (p.processors && p.processors.length > 0) {
           await this.initializeProcessorsInternal(
             context2,
@@ -15584,7 +15797,7 @@ var CliBoot = class {
           );
         }
       } catch (e) {
-        console.error(
+        context2.logger.error(
           `Error initializing processor "${p.command}":`,
           e
         );
@@ -15702,9 +15915,10 @@ var CliKeyValueStore = class {
         this.db = event.target.result;
         resolve();
       };
-      request.onerror = (event) => {
-        console.error("Error initializing IndexedDB:", event);
-        reject();
+      request.onerror = () => {
+        reject(new Error(
+          `Failed to open IndexedDB "${this.dbName}": ${request.error?.message ?? "unknown error"}`
+        ));
       };
     });
   }
@@ -15725,13 +15939,15 @@ var CliKeyValueStore = class {
         request.onsuccess = () => {
           resolve(request.result);
         };
-        request.onerror = (event) => {
-          console.error("Error getting value:", event);
-          reject(void 0);
+        request.onerror = () => {
+          reject(new Error(
+            `IndexedDB get("${key}") failed: ${request.error?.message ?? "unknown error"}`
+          ));
         };
       } catch (e) {
-        console.error("Error getting value:", e);
-        reject(void 0);
+        reject(new Error(
+          `IndexedDB get("${key}") failed: ${e instanceof Error ? e.message : e}`
+        ));
       }
     });
   }
@@ -15750,9 +15966,10 @@ var CliKeyValueStore = class {
       const store = transaction.objectStore(this.storeName);
       const request = store.put(value, key);
       request.onsuccess = () => resolve();
-      request.onerror = (event) => {
-        console.error("Error setting value:", event);
-        reject();
+      request.onerror = () => {
+        reject(new Error(
+          `IndexedDB set("${key}") failed: ${request.error?.message ?? "unknown error"}`
+        ));
       };
     });
   }
@@ -15770,9 +15987,10 @@ var CliKeyValueStore = class {
       const store = transaction.objectStore(this.storeName);
       const request = store.delete(key);
       request.onsuccess = () => resolve();
-      request.onerror = (event) => {
-        console.error("Error removing value:", event);
-        reject();
+      request.onerror = () => {
+        reject(new Error(
+          `IndexedDB remove("${key}") failed: ${request.error?.message ?? "unknown error"}`
+        ));
       };
     });
   }
@@ -15789,9 +16007,10 @@ var CliKeyValueStore = class {
       const store = transaction.objectStore(this.storeName);
       const request = store.clear();
       request.onsuccess = () => resolve();
-      request.onerror = (event) => {
-        console.error("Error clearing store:", event);
-        reject();
+      request.onerror = () => {
+        reject(new Error(
+          `IndexedDB clear() failed: ${request.error?.message ?? "unknown error"}`
+        ));
       };
     });
   }
@@ -15845,6 +16064,13 @@ var CliStateStore = class {
       );
     }
   }
+  /**
+   * Complete the BehaviorSubject so all subscribers are notified
+   * and no further emissions occur. Called during engine teardown.
+   */
+  dispose() {
+    this.state$.complete();
+  }
 };
 
 // src/lib/state/cli-state-store-manager.ts
@@ -15880,6 +16106,18 @@ var CliStateStoreManager = class {
     });
     return entries;
   }
+  /**
+   * Dispose all stores (completes their BehaviorSubjects).
+   * Called during engine teardown.
+   */
+  dispose() {
+    this.stores.forEach((store) => {
+      if (store.dispose) {
+        store.dispose();
+      }
+    });
+    this.stores.clear();
+  }
 };
 
 // src/lib/editor/syntax/registry.ts
@@ -16923,7 +17161,7 @@ function createServerModule() {
 }
 
 // src/lib/engine/cli-engine.ts
-var CliEngine = class {
+var CliEngine = class _CliEngine {
   constructor(container, options) {
     this.container = container;
     this.options = options;
@@ -17070,9 +17308,9 @@ var CliEngine = class {
     ]);
     if (this.options?.servers && this.options.servers.length > 0) {
       await serverManager.connectAll(this.options.servers, {
-        warn: (msg) => console.warn(msg),
-        info: (msg) => console.log(msg)
-      }, this.executionContext.backgroundServices);
+        warn: (msg) => this.executionContext.logger.warn(msg),
+        info: (msg) => this.executionContext.logger.info(msg)
+      }, this.executionContext.backgroundServices, this.executionContext.logger);
     }
     const serverModule = createServerModule();
     const allModules = this.options?.snapshot ? [serverModule, ...this.userModules] : [welcomeModule, serverModule, ...this.userModules];
@@ -17106,7 +17344,7 @@ var CliEngine = class {
         try {
           await module.onAfterBoot(this.executionContext);
         } catch (e) {
-          console.error(
+          this.executionContext.logger.error(
             `Error in onAfterBoot for module "${module.name}":`,
             e
           );
@@ -17129,7 +17367,7 @@ var CliEngine = class {
           try {
             module.onDestroy(this.executionContext);
           } catch (e) {
-            console.error(`Error in onDestroy for module "${module.name}":`, e);
+            this.executionContext.logger.error(`Error in onDestroy for module "${module.name}":`, e);
           }
         }
       }
@@ -17301,17 +17539,29 @@ var CliEngine = class {
       this.executionContext?.handleTerminalResize();
     });
   }
+  static {
+    /** Maximum time (ms) to wait for container layout before rejecting. */
+    this.LAYOUT_TIMEOUT_MS = 1e4;
+  }
   /**
    * Wait until the container element has non-zero dimensions.
    * xterm.js requires the host element to be laid out before open() is called.
+   * Rejects after {@link LAYOUT_TIMEOUT_MS} if the container stays hidden.
    */
   waitForLayout() {
-    return new Promise((resolve) => {
+    return new Promise((resolve, reject) => {
+      const deadline = Date.now() + _CliEngine.LAYOUT_TIMEOUT_MS;
       const check = () => {
         if (this.container.offsetWidth > 0 && this.container.offsetHeight > 0) {
           resolve();
           return;
         }
+        if (Date.now() >= deadline) {
+          reject(new Error(
+            `CliEngine.start(): container has zero dimensions after ${_CliEngine.LAYOUT_TIMEOUT_MS}ms. Ensure the container element is visible and laid out before calling start().`
+          ));
+          return;
+        }
         requestAnimationFrame(check);
       };
       check();
@@ -17543,6 +17793,18 @@ function createTestWriter() {
       for (const item of items) {
         stdout.push(item);
       }
+    },
+    writeLink(text, _url) {
+      stdout.push(text);
+    },
+    writeBox(content, _options) {
+      const lines = Array.isArray(content) ? content : [content];
+      for (const line of lines) {
+        stdout.push(line);
+      }
+    },
+    writeIndented(text, level = 1) {
+      stdout.push("  ".repeat(level) + text);
     }
   };
 }