@kb-labs/mcp-app 2.107.0 → 2.110.0

package/dist/bin.cjs

@@ -1717,6 +1717,89 @@ var init_dist2 = __esm({
     ]);
   }
 });
+
+// ../../../core/retry/dist/index.js
+function asRecord(value) {
+  return typeof value === "object" && value !== null ? value : void 0;
+}
+function getStatus(value) {
+  const response = asRecord(value?.response);
+  const status = value?.status ?? value?.statusCode ?? response?.status;
+  return typeof status === "number" ? status : void 0;
+}
+function getCode(value) {
+  return typeof value?.code === "string" ? value.code : void 0;
+}
+function makeFailure(input, kind, source, context, code, transient = false, retrySafety = "never") {
+  const record2 = asRecord(input);
+  const message2 = input instanceof Error ? input.message : String(record2?.message ?? input ?? "Unknown failure");
+  const retryAfterMs = typeof record2?.retryAfterMs === "number" ? record2.retryAfterMs : void 0;
+  return {
+    message: message2,
+    code: code ?? getCode(record2) ?? "UNKNOWN_ERROR",
+    kind,
+    source,
+    transient,
+    retrySafety,
+    phase: context.phase,
+    retryAfterMs,
+    details: asRecord(record2?.details)
+  };
+}
+function classifyFailure(input, context = {}) {
+  const record2 = asRecord(input);
+  const source = context.source ?? record2?.source ?? "execution";
+  const explicitKind = record2?.kind;
+  const explicitCode = getCode(record2);
+  const status = getStatus(record2);
+  const code = explicitCode?.toUpperCase();
+  if (explicitKind && ["command", "network", "timeout", "rate_limit", "server", "validation", "configuration", "authentication", "authorization", "not_found", "cancelled", "infrastructure", "unknown"].includes(explicitKind)) {
+    const transient = ["network", "timeout", "rate_limit", "server"].includes(explicitKind);
+    const safe = explicitKind === "network" && (code === "ECONNREFUSED" || code === "NETWORK_UNAVAILABLE");
+    return makeFailure(input, explicitKind, source, context, code, transient, safe ? "safe" : transient ? "requires_idempotency" : "never");
+  }
+  if (code && NETWORK_CODES.has(code)) {
+    return makeFailure(input, code === "ETIMEDOUT" ? "timeout" : "network", source, context, code, true, context.phase === "dispatch" ? "safe" : "requires_idempotency");
+  }
+  if (status !== void 0) {
+    if (status === 429) {
+      return makeFailure(input, "rate_limit", source, context, "HTTP_429", true, "safe");
+    }
+    if (status >= 500 || status === 408 || status === 425) {
+      return makeFailure(input, status === 408 ? "timeout" : "server", source, context, `HTTP_${status}`, true, "requires_idempotency");
+    }
+    if (status >= 400) {
+      return makeFailure(input, "validation", source, context, `HTTP_${status}`, false, "never");
+    }
+  }
+  const message2 = input instanceof Error ? input.message.toLowerCase() : String(record2?.message ?? input ?? "").toLowerCase();
+  const name = typeof record2?.name === "string" ? record2.name.toLowerCase() : "";
+  if (message2.includes("429") || message2.includes("rate limit") || message2.includes("too many requests") || message2.includes("quota exceeded") || name.includes("ratelimit")) {
+    return makeFailure(input, "rate_limit", source, context, "RATE_LIMIT", true, "safe");
+  }
+  if (message2.includes("500") || message2.includes("502") || message2.includes("503") || message2.includes("504") || message2.includes("service unavailable") || message2.includes("bad gateway")) {
+    return makeFailure(input, "server", source, context, "SERVER_ERROR", true, "requires_idempotency");
+  }
+  if (message2.includes("400") || message2.includes("401") || message2.includes("403") || message2.includes("404") || message2.includes("bad request") || message2.includes("unauthorized") || message2.includes("forbidden") || message2.includes("not found")) {
+    return makeFailure(input, "validation", source, context, "CLIENT_ERROR");
+  }
+  if (message2.includes("timeout") || message2.includes("timed out") || message2.includes("etimedout") || message2.includes("deadline exceeded") || name.includes("timeout") || code?.includes("TIMEOUT")) {
+    return makeFailure(input, "timeout", source, context, "TIMEOUT", true, "requires_idempotency");
+  }
+  if (message2.includes("unauthorized") || message2.includes("forbidden")) {
+    return makeFailure(input, "authentication", source, context, "AUTHENTICATION_FAILED");
+  }
+  if (message2.includes("econn") || message2.includes("enotfound") || message2.includes("network") || message2.includes("socket") || message2.includes("dns") || message2.includes("host not connected") || name.includes("fetch") || name.includes("network")) {
+    return makeFailure(input, "network", source, context, "NETWORK_ERROR", true, "requires_idempotency");
+  }
+  return makeFailure(input, source === "command" ? "command" : "unknown", source, context, code ?? "UNKNOWN_ERROR");
+}
+var NETWORK_CODES;
+var init_dist3 = __esm({
+  "../../../core/retry/dist/index.js"() {
+    NETWORK_CODES = /* @__PURE__ */ new Set(["ECONNRESET", "ECONNREFUSED", "ETIMEDOUT", "ENOTFOUND", "EAI_AGAIN", "EPIPE", "ECONNABORTED"]);
+  }
+});
 function getRateLimitConfig(configOrPreset) {
   if (!configOrPreset) {
     return RATE_LIMIT_PRESETS["openai-tier-2"];
@@ -1737,58 +1820,21 @@ function estimateBatchTokens(texts) {
   return texts.reduce((sum, text) => sum + estimateTokens(text), 0);
 }
 function classifyError(error2) {
-  if (!error2) {
-    return "unknown";
+  const kind = classifyFailure(error2, { source: "transport" }).kind;
+  if (kind === "rate_limit") {
+    return "rate_limit";
   }
-  if (error2 instanceof Error) {
-    const message2 = error2.message.toLowerCase();
-    const name = error2.name.toLowerCase();
-    if (message2.includes("429") || message2.includes("rate limit") || message2.includes("too many requests") || message2.includes("quota exceeded") || name.includes("ratelimit")) {
-      return "rate_limit";
-    }
-    if (message2.includes("timeout") || message2.includes("timed out") || message2.includes("etimedout") || message2.includes("deadline exceeded") || name.includes("timeout")) {
-      return "timeout";
-    }
-    if (message2.includes("econnrefused") || message2.includes("econnreset") || message2.includes("enotfound") || message2.includes("network") || message2.includes("socket") || message2.includes("dns") || name.includes("fetch") || name.includes("network")) {
-      return "network";
-    }
-    if (message2.includes("500") || message2.includes("502") || message2.includes("503") || message2.includes("504") || message2.includes("internal server") || message2.includes("bad gateway") || message2.includes("service unavailable") || message2.includes("gateway timeout")) {
-      return "server_error";
-    }
-    if (message2.includes("400") || message2.includes("401") || message2.includes("403") || message2.includes("404") || message2.includes("bad request") || message2.includes("unauthorized") || message2.includes("forbidden") || message2.includes("not found")) {
-      return "client_error";
-    }
+  if (kind === "timeout") {
+    return "timeout";
   }
-  if (typeof error2 === "object" && error2 !== null) {
-    const obj = error2;
-    const status = obj.status ?? obj.statusCode ?? obj.code;
-    if (typeof status === "number") {
-      if (status === 429) {
-        return "rate_limit";
-      }
-      if (status >= 500) {
-        return "server_error";
-      }
-      if (status >= 400) {
-        return "client_error";
-      }
-    }
-    const code = obj.code;
-    if (typeof code === "string") {
-      const lowerCode = code.toLowerCase();
-      if (lowerCode.includes("timeout")) {
-        return "timeout";
-      }
-      if (lowerCode.includes("econnrefused")) {
-        return "network";
-      }
-      if (lowerCode.includes("econnreset")) {
-        return "network";
-      }
-      if (lowerCode.includes("enotfound")) {
-        return "network";
-      }
-    }
+  if (kind === "network") {
+    return "network";
+  }
+  if (kind === "server") {
+    return "server_error";
+  }
+  if (kind === "validation" || kind === "authentication" || kind === "authorization" || kind === "not_found") {
+    return "client_error";
   }
   return "unknown";
 }
@@ -1868,8 +1914,9 @@ function createQueuedVectorStore(broker, vectorStore) {
   return new QueuedVectorStore(broker, vectorStore);
 }
 var DEFAULT_RETRY_CONFIG, DEFAULT_RATE_LIMIT_CONFIG, PriorityQueue, RATE_LIMIT_PRESETS, ResourceBroker, InMemoryRateLimitBackend, QueuedLLM, QueuedEmbeddings, QueuedVectorStore;
-var init_dist3 = __esm({
+var init_dist4 = __esm({
   "../../../core/resource-broker/dist/index.js"() {
+    init_dist3();
     DEFAULT_RETRY_CONFIG = {
       maxRetries: 3,
       baseDelay: 1e3,
@@ -14598,14 +14645,14 @@ var require_scope = __commonJS({
           return (0, code_1._)`${scopeName}${name.scopePath}`;
         });
       }
-      scopeCode(values = this._values, usedValues, getCode) {
+      scopeCode(values = this._values, usedValues, getCode2) {
         return this._reduceValues(values, (name) => {
           if (name.value === void 0)
             throw new Error(`CodeGen: name "${name}" has no value`);
           return name.value.code;
-        }, usedValues, getCode);
+        }, usedValues, getCode2);
       }
-      _reduceValues(values, valueCode, usedValues = {}, getCode) {
+      _reduceValues(values, valueCode, usedValues = {}, getCode2) {
         let code = code_1.nil;
         for (const prefix in values) {
           const vs = values[prefix];
@@ -14620,7 +14667,7 @@ var require_scope = __commonJS({
             if (c) {
               const def = this.opts.es5 ? exports$1.varKinds.var : exports$1.varKinds.const;
               code = (0, code_1._)`${code}${def} ${name} = ${c};${this.opts._n}`;
-            } else if (c = getCode === null || getCode === void 0 ? void 0 : getCode(name)) {
+            } else if (c = getCode2 === null || getCode2 === void 0 ? void 0 : getCode2(name)) {
               code = (0, code_1._)`${code}${c}${this.opts._n}`;
             } else {
               throw new ValueError(name);
@@ -21474,7 +21521,7 @@ async function loadOverlays(projectRoot) {
   return { overlays, diagnostics };
 }
 var import_ajv, import_ajv_formats, __defProp2, __getOwnPropNames2, __esm2, __export2, KbError, init_kb_error, fs_atomic_exports, init_fs_atomic, LRUCache, FSCache, GlobSchema, ProductOverridesSchema, ProductsMapSchema, ScopeV2Schema, ProfileMetaSchema, ProfileV2Schema, ResolvedScopeSchema, BundleProfileSourceSchema, ProfileTraceSchema, ScopeSelectionSchema, ajv, schemaMap, OVERLAY_DIRECTIVE_KEY;
-var init_dist4 = __esm({
+var init_dist5 = __esm({
   "../../../core/config/dist/index.js"() {
     __toESM(require_dist());
     init_zod();
@@ -21785,34 +21832,10 @@ function serializeError(error2) {
   return serialize(error2);
 }
 function isRetryableError2(error2) {
-  if (error2 instanceof TimeoutError) {
-    return true;
-  }
   if (error2 instanceof CircuitOpenError) {
     return false;
   }
-  const code = error2.code;
-  if (code) {
-    const retryableCodes = [
-      "ECONNRESET",
-      // Connection reset
-      "ECONNREFUSED",
-      // Connection refused
-      "ETIMEDOUT",
-      // Operation timed out
-      "ENOTFOUND",
-      // DNS lookup failed
-      "EAI_AGAIN"
-      // DNS temporary failure
-    ];
-    return retryableCodes.includes(code);
-  }
-  const e = error2;
-  const status = e.status ?? e.statusCode;
-  if (status) {
-    return status === 503 || status === 429;
-  }
-  return false;
+  return classifyFailure(error2, { source: "transport", phase: "dispatch" }).transient;
 }
 function getOperationTimeout(adapter, method) {
   const exactKey = `${adapter}.${method}`;
@@ -21893,9 +21916,10 @@ function createProxyPlatform(options) {
   };
 }
 var BulkTransferHelper, DEFAULT_SOCKET_PATH, UnixSocketServer, IPCServer, ChildIPCServer, TransportError, TimeoutError, CircuitOpenError, OPERATION_TIMEOUTS, IPCTransport, UnixSocketTransport, RemoteAdapter, CacheProxy, LLMProxy, EmbeddingsProxy, VectorStoreProxy, StorageProxy, DocumentDatabaseProxy, stripSignal, KVStoreProxy, stripSignal2, ConfigProxy, EventBusProxy, LoggerProxy;
-var init_dist5 = __esm({
+var init_dist6 = __esm({
   "../../../core/ipc/dist/index.js"() {
     init_serializable();
+    init_dist3();
     BulkTransferHelper = class _BulkTransferHelper {
       /** Map of temp file IDs to file paths (for cleanup) */
       static tempFiles = /* @__PURE__ */ new Map();
@@ -23746,7 +23770,7 @@ function getProviderFromEntry(entry) {
   return "default";
 }
 var TierResolver, CapabilityResolver, LLMRouter;
-var init_dist6 = __esm({
+var init_dist7 = __esm({
   "../../../core/llm-router/dist/index.js"() {
     init_dist2();
     TierResolver = class {
@@ -26354,7 +26378,7 @@ function makeAssemblyHook(getLogger2) {
   );
 }
 function isCommandResult(value) {
-  return typeof value === "object" && value !== null && "exitCode" in value && typeof value.exitCode === "number";
+  return typeof value === "object" && value !== null && "ok" in value && typeof value.ok === "boolean";
 }
 function toStandardMeta(executionMeta, descriptor) {
   return {
@@ -26372,12 +26396,15 @@ function wrapCliResult(runResult, descriptor) {
   const { data, executionMeta } = runResult;
   const standardMeta = toStandardMeta(executionMeta, descriptor);
   if (isCommandResult(data)) {
+    const ok = data.ok;
+    const exitCode = ok ? 0 : 1;
     const mergedMeta = {
       ...data.meta,
       ...standardMeta
     };
     return {
-      exitCode: data.exitCode,
+      exitCode,
+      ok,
       result: data.result,
       meta: mergedMeta
     };
@@ -26385,22 +26412,24 @@ function wrapCliResult(runResult, descriptor) {
   if (data === void 0 || data === null) {
     return {
       exitCode: 0,
+      ok: true,
       meta: standardMeta
     };
   }
   return {
     exitCode: 0,
+    ok: true,
     result: data,
     meta: standardMeta
   };
 }
 var PluginError, PermissionError, TimeoutError2, AbortError, PLATFORM_CONTEXT_KEY2, platformContext2, RUNTIME_CONTEXT_KEY2, runtimeContext2, HARDCODED_DENIED_PATTERNS, ALWAYS_ALLOWED, BLOCKED_COMMANDS, ANALYTICS_EVENT, EVENT_BUS_TOPIC, NAMESPACE_SEPARATOR, KV_NAMESPACE_SEPARATOR, validateName, wrapLogger, wrapLlm, wrapEmbeddings, wrapVectorStore, wrapCache, wrapStorage, wrapNotifier, ADAPTER_REGISTRY, PIPELINE_SLOTS, SLOT_ORDER, NOOP_LIFECYCLE, ANSI_RE;
-var init_dist7 = __esm({
+var init_dist8 = __esm({
   "../../../core/plugin-runtime/dist/index.js"() {
     init_dist2();
-    init_dist3();
+    init_dist4();
+    init_dist7();
     init_dist6();
-    init_dist5();
     PluginError = class _PluginError extends Error {
       /**
        * Error code for programmatic handling
@@ -27055,7 +27084,7 @@ function createDefaultIPCServerFactory() {
   return async (platform2, executionId) => {
     const platformContainer = ensurePlatformContainer(platform2);
     if (process.platform === "win32") {
-      const { UnixSocketServer: UnixSocketServer22, createSocketPath: createSocketPath2 } = await Promise.resolve().then(() => (init_dist5(), dist_exports));
+      const { UnixSocketServer: UnixSocketServer22, createSocketPath: createSocketPath2 } = await Promise.resolve().then(() => (init_dist6(), dist_exports));
       const socketPath = createSocketPath2(`subprocess-${executionId}`);
       const authToken = crypto4.randomBytes(32).toString("hex");
       const serverConfig = { socketPath };
@@ -27063,7 +27092,7 @@ function createDefaultIPCServerFactory() {
       const server = new UnixSocketServer22(platformContainer, serverConfig);
       return new UnixSocketIPCServer(server, socketPath, authToken);
     } else {
-      const { UnixSocketServer: UnixSocketServer22, createSocketPath: createSocketPath2 } = await Promise.resolve().then(() => (init_dist5(), dist_exports));
+      const { UnixSocketServer: UnixSocketServer22, createSocketPath: createSocketPath2 } = await Promise.resolve().then(() => (init_dist6(), dist_exports));
       const socketPath = createSocketPath2(`subprocess-${executionId}`);
       const authToken = crypto4.randomBytes(32).toString("hex");
       const serverConfig = { socketPath };
@@ -27254,12 +27283,12 @@ function buildRoutingBackend(localBackend, remoteFactory, provisionEnvironment,
   };
 }
 var LocalWorkspaceManager, localWorkspaceManager, KNOWN_ERROR_CODES, ExecutionLayerError, TimeoutError3, AbortError2, HandlerContractError, HandlerNotFoundError, WorkspaceError, PermissionDeniedError, ValidationError, QueueFullError, AcquireTimeoutError, WorkerCrashedError, WorkerUnhealthyError, InProcessBackend, UnixSocketIPCServer, SubprocessBackend, UnixSocketPlatformTransportFactory, PoolStatsTracker, DEFAULT_STARTUP_TIMEOUT, DEFAULT_HEALTH_CHECK_TIMEOUT, Worker, PoolLifecycleManager, PoolQueueManager, PoolExecutor, WorkerPool, __dirname$1, WorkerPoolBackend, RemoteBackend, SubprocessRunnerAdapter, PROTOCOL_VERSION, DEFAULT_WORKER_POOL_CONFIG, IPCPlatformTransportFactory;
-var init_dist8 = __esm({
+var init_dist9 = __esm({
   "../../../core/plugin-execution-factory/dist/index.js"() {
     init_dist();
-    init_dist7();
+    init_dist8();
     init_dist2();
-    init_dist5();
+    init_dist6();
     LocalWorkspaceManager = class {
       async lease(config2, ctx) {
         return {
@@ -29271,7 +29300,7 @@ function sleep2(ms) {
   });
 }
 var DEFAULT_BUFFER_POLICY, AdaptiveBuffer, CALL_TIMEOUT_MS, HostCallDispatcher, globalDispatcher, GatewayDispatchTransport, HOSTS_COLLECTION, TOKENS_COLLECTION, docToDescriptor, HostStore, GatewayHostResolver;
-var init_dist9 = __esm({
+var init_dist10 = __esm({
   "../../gateway/core/dist/index.js"() {
     DEFAULT_BUFFER_POLICY = {
       minTTL: 3e4,
@@ -78129,7 +78158,7 @@ function extractEntityKinds(manifest) {
 // ../../../core/runtime/dist/index.js
 init_serializable();
 init_dist2();
-init_dist3();
+init_dist4();
 
 // ../../../node_modules/.pnpm/ulid@2.4.0/node_modules/ulid/dist/index.esm.js
 function createError(message2) {
@@ -78224,7 +78253,7 @@ function isDisposable(value) {
 }
 
 // ../../../core/runtime/dist/index.js
-init_dist4();
+init_dist5();
 var CSI = "\x1B[";
 var RESET = "\x1B[0m";
 var createColor = (...codes) => (text2) => `${CSI}${codes.join(";")}m${text2}${RESET}`;
@@ -83523,8 +83552,8 @@ async function initPlatform(config2 = {}, cwd = process.cwd(), uiProvider, platf
       );
       try {
         if (execution.mode === "container" && execution.container) {
-          const { createIsolatedExecutionBackend: createIsolatedExecutionBackend2 } = await Promise.resolve().then(() => (init_dist8(), dist_exports2));
-          const { GatewayDispatchTransport: GatewayDispatchTransport2 } = await Promise.resolve().then(() => (init_dist9(), dist_exports3));
+          const { createIsolatedExecutionBackend: createIsolatedExecutionBackend2 } = await Promise.resolve().then(() => (init_dist9(), dist_exports2));
+          const { GatewayDispatchTransport: GatewayDispatchTransport2 } = await Promise.resolve().then(() => (init_dist10(), dist_exports3));
           const containerCfg = execution.container;
           const strictIsolation = {
             buildTransport(ctx) {
@@ -83618,8 +83647,8 @@ async function initPlatform(config2 = {}, cwd = process.cwd(), uiProvider, platf
             dispatchUrl: containerCfg.gatewayDispatchUrl
           });
         } else if (execution.workspaceAgent?.enabled && execution.workspaceAgent.gatewayUrl) {
-          const { createIsolatedExecutionBackend: createIsolatedExecutionBackend2 } = await Promise.resolve().then(() => (init_dist8(), dist_exports2));
-          const { GatewayHostResolver: GatewayHostResolver2, GatewayDispatchTransport: GatewayDispatchTransport2 } = await Promise.resolve().then(() => (init_dist9(), dist_exports3));
+          const { createIsolatedExecutionBackend: createIsolatedExecutionBackend2 } = await Promise.resolve().then(() => (init_dist9(), dist_exports2));
+          const { GatewayHostResolver: GatewayHostResolver2, GatewayDispatchTransport: GatewayDispatchTransport2 } = await Promise.resolve().then(() => (init_dist10(), dist_exports3));
           const agentCfg = execution.workspaceAgent;
           const hostResolver = new GatewayHostResolver2({
             gatewayUrl: agentCfg.gatewayUrl,
@@ -83667,7 +83696,7 @@ async function initPlatform(config2 = {}, cwd = process.cwd(), uiProvider, platf
             fallback: agentCfg.fallback ?? "local"
           });
         } else {
-          const { createExecutionBackend: createExecutionBackend2 } = await Promise.resolve().then(() => (init_dist8(), dist_exports2));
+          const { createExecutionBackend: createExecutionBackend2 } = await Promise.resolve().then(() => (init_dist9(), dist_exports2));
           const factoryMode = execution.mode ?? "auto";
           const executionBackend = createExecutionBackend2({
             platform,
@@ -84522,10 +84551,10 @@ init_container();
 init_adapter_status();
 
 // src/bootstrap.ts
-init_dist7();
+init_dist8();
 
 // ../../../core/policy/dist/index.js
-init_dist4();
+init_dist5();
 var import_ajv2 = __toESM(require_ajv());
 var import_ajv_formats2 = __toESM(require_dist2());
 var DEFAULT_POLICY = {
@@ -107514,7 +107543,7 @@ function defineSystemCommand(config2) {
 // ../../../cli/runtime/dist/index.js
 __toESM(require_dist());
 __toESM(require_cli_table3());
-init_dist7();
+init_dist8();
 __toESM(require_colorette());
 var __require2 = /* @__PURE__ */ ((x) => typeof __require !== "undefined" ? __require : typeof Proxy !== "undefined" ? new Proxy(x, {
   get: (a, b) => (typeof __require !== "undefined" ? __require : a)[b]
@@ -109477,7 +109506,7 @@ defineSystemCommand({
       });
       ctx.platform?.logger?.info("Health check completed", { status: snapshot.status });
       return {
-        ok: snapshot.status === "healthy",
+        ok: true,
         status: snapshot.status === "healthy" ? "success" : "warning",
         healthStatus: snapshot.status || "unhealthy",
         snapshot
@@ -109501,11 +109530,11 @@ defineSystemCommand({
       const jsonOutput = result.snapshot || {
         schema: "kb.health/1",
         status: result.healthStatus,
-        error: result.error
+        error: !result.ok ? result.error : void 0
       };
       ctx.ui?.json?.(jsonOutput);
     } else {
-      if (result.error) {
+      if (!result.ok) {
         ctx.ui.error("System Health", {
           sections: [
             {
@@ -109970,12 +109999,12 @@ async function runTrace(ctx) {
     }
   }
   const failing = stages.find((s) => s.status === "error") ?? stages.find((s) => s.status === "warning");
-  return {
-    ok: !stages.some((s) => s.status === "error" || s.status === "warning"),
+  const result = {
     command: ctx.segments.join(" "),
     stages,
     verdict: failing ? { rootCause: failing.code, remediation: failing.remediation } : { rootCause: "NONE" }
   };
+  return failing ? { ok: false, error: failing.message, ...result } : { ok: true, ...result };
 }
 defineSystemCommand({
   name: "diag",
@@ -110163,7 +110192,7 @@ defineSystemCommand({
       errors: diagnostics.filter((d) => d.status === "error").length
     };
     ctx.platform?.logger?.info("Diag command completed", summary);
-    return { ok: summary.errors === 0, diagnostics, summary };
+    return summary.errors === 0 ? { ok: true, diagnostics, summary } : { ok: false, error: `${summary.errors} diagnostic error(s) found`, diagnostics, summary };
   },
   // eslint-disable-next-line sonarjs/cognitive-complexity
   formatter(result, ctx, flags) {
@@ -110251,7 +110280,10 @@ defineSystemCommand({
     }
     const report = registry22.getDiagnostics();
     await registry22.dispose();
-    return { ok: report.summary.errors === 0, report };
+    if (report.summary.errors === 0) {
+      return { ok: true, report };
+    }
+    return { ok: false, error: `${report.summary.errors} diagnostic error(s) found`, report };
   },
   formatter(result, ctx, flags) {
     if (flags.json) {
@@ -110634,7 +110666,8 @@ defineSystemCommand({
       return;
     }
     if (!result.ok) {
-      ctx.ui.error("Log Diagnose", { sections: [{ header: "Error", items: [result.error ?? "Unknown"] }] });
+      const error2 = typeof result.error === "string" ? result.error : "Unknown";
+      ctx.ui.error("Log Diagnose", { sections: [{ header: "Error", items: [error2] }] });
       return;
     }
     const { summary, topErrors, bySource, period } = result;
@@ -110760,7 +110793,8 @@ defineSystemCommand({
       return;
     }
     if (!result.ok) {
-      ctx.ui.error("Log Context", { sections: [{ header: "Error", items: [result.error ?? "Unknown"] }] });
+      const error2 = typeof result.error === "string" ? result.error : "Unknown";
+      ctx.ui.error("Log Context", { sections: [{ header: "Error", items: [error2] }] });
       return;
     }
     const { timeline, correlationKey, byLevel } = data;
@@ -110941,7 +110975,8 @@ defineSystemCommand({
       return;
     }
     if (!result.ok) {
-      ctx.ui.error("Log Summarize", { sections: [{ header: "Error", items: [result.error ?? "Unknown"] }] });
+      const error2 = typeof result.error === "string" ? result.error : "Unknown";
+      ctx.ui.error("Log Summarize", { sections: [{ header: "Error", items: [error2] }] });
       return;
     }
     ctx.ui.write(`Question: ${result.question}
@@ -111010,7 +111045,8 @@ defineSystemCommand({
       return;
     }
     if (!result.ok) {
-      ctx.ui.error("Log Query", { sections: [{ header: "Error", items: [result.error ?? "Unknown"] }] });
+      const error2 = typeof result.error === "string" ? result.error : "Unknown";
+      ctx.ui.error("Log Query", { sections: [{ header: "Error", items: [error2] }] });
       return;
     }
     const raw = result._raw;
@@ -111072,7 +111108,8 @@ defineSystemCommand({
       return;
     }
     if (!result.ok) {
-      ctx.ui.error("Log Search", { sections: [{ header: "Error", items: [result.error ?? "Unknown"] }] });
+      const error2 = typeof result.error === "string" ? result.error : "Unknown";
+      ctx.ui.error("Log Search", { sections: [{ header: "Error", items: [error2] }] });
       return;
     }
     const raw = result._raw;
@@ -111138,7 +111175,8 @@ defineSystemCommand({
       return;
     }
     if (!result.ok) {
-      ctx.ui.error("Log Get", { sections: [{ header: "Error", items: [result.error ?? "Unknown"] }] });
+      const error2 = typeof result.error === "string" ? result.error : "Unknown";
+      ctx.ui.error("Log Get", { sections: [{ header: "Error", items: [error2] }] });
       return;
     }
     const rawLog = result._rawLog;
@@ -111232,7 +111270,8 @@ defineSystemCommand({
       return;
     }
     if (!result.ok) {
-      ctx.ui.error("Log Stats", { sections: [{ header: "Error", items: [result.error ?? "Unknown error"] }] });
+      const error2 = typeof result.error === "string" ? result.error : "Unknown error";
+      ctx.ui.error("Log Stats", { sections: [{ header: "Error", items: [error2] }] });
       return;
     }
     const sections = [];
@@ -111743,11 +111782,10 @@ defineSystemCommand({
       installed: result.installed.length,
       errors: result.errors.length
     });
-    return {
-      ok: result.ok,
-      status: result.ok ? "success" : "error",
-      sync: result
-    };
+    if (result.ok) {
+      return { ok: true, status: "success", sync: result };
+    }
+    return { ok: false, error: result.errors.join("; ") || "Platform sync failed", status: "error", sync: result };
   },
   // eslint-disable-next-line sonarjs/cognitive-complexity
   formatter(result, ctx, flags) {
@@ -111758,7 +111796,7 @@ defineSystemCommand({
     const sync2 = result.sync;
     if (!sync2) {
       ctx.ui.error("Platform Sync", {
-        sections: [{ header: "Error", items: [result.error ?? "Unknown error"] }]
+        sections: [{ header: "Error", items: ["Unknown error"] }]
       });
       return;
     }