@dexto/core 1.1.10 → 1.2.0

package/dist/logger/logger.cjs

@@ -165,26 +165,54 @@ class Logger {
   }
   // General logging methods with optional color parameter
   error(message, meta, color) {
-    this.logger.error(message, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.error(message, meta);
+    } else {
+      this.logger.error(message, { ...meta, color });
+    }
   }
   warn(message, meta, color) {
-    this.logger.warn(message, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.warn(message, meta);
+    } else {
+      this.logger.warn(message, { ...meta, color });
+    }
   }
   info(message, meta, color) {
-    this.logger.info(message, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.info(message, meta);
+    } else {
+      this.logger.info(message, { ...meta, color });
+    }
   }
   http(message, meta, color) {
-    this.logger.http(message, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.http(message, meta);
+    } else {
+      this.logger.http(message, { ...meta, color });
+    }
   }
   verbose(message, meta, color) {
-    this.logger.verbose(message, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.verbose(message, meta);
+    } else {
+      this.logger.verbose(message, { ...meta, color });
+    }
   }
   debug(message, meta, color) {
     const formattedMessage = typeof message === "string" ? message : JSON.stringify(message, null, 2);
-    this.logger.debug(formattedMessage, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.debug(formattedMessage, meta);
+    } else {
+      this.logger.debug(formattedMessage, { ...meta, color });
+    }
   }
   silly(message, meta, color) {
-    this.logger.silly(message, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.silly(message, meta);
+    } else {
+      this.logger.silly(message, { ...meta, color });
+    }
   }
   // Display AI response in a box
   displayAIResponse(response) {

package/dist/logger/logger.d.ts

@@ -1,11 +1,53 @@
+/**
+ * TODO(logger-browser-safety): Make logging environment-safe without breaking consumers
+ *
+ * Problem
+ * - The core logger is Node-only (winston + fs/path + file IO). If any module that depends on this
+ *   logger is included in a browser bundle (directly or via an overly broad root export), Web UI builds
+ *   can fail. Today our Web UI only imports types and a small util, so bundlers tree‑shake away the
+ *   Node logger, but this is fragile if future UI imports include runtime modules.
+ *
+ * Constraints/Goals
+ * - Keep file-based logging as the default for Node (excellent DX/UX).
+ * - Keep logs visible in the browser (console-based), never a no‑op.
+ * - Avoid touching "a lot of files" or changing call sites for consumers.
+ * - Keep `@dexto/core` ergonomic; browser consumers should not have to learn a separate API.
+ *
+ * Plan (incremental)
+ * 1) Introduce an environment-aware logging boundary:
+ *    - Create a browser-safe logger implementation (console-based) that matches the logger API.
+ *    - Expose logging via a subpath `@dexto/core/logger` and add conditional exports so that:
+ *      - Browser resolves to the console logger
+ *      - Node resolves to this file-based logger (current Winston version)
+ *    - Alternative: use the `browser` field in package.json to alias the Node logger file to a
+ *      browser-safe implementation at bundle time. Subpath conditional exports are preferred for clarity.
+ *
+ * 2) Keep the root export of `@dexto/core` browser-safe by default:
+ *    - Continue to expose types and UI-safe helpers from the root.
+ *    - Keep Node-only modules (storage/config/agent-registry/etc.) on explicit subpaths so browser
+ *      builds do not pull them accidentally.
+ *
+ * 3) Optional API ergonomics:
+ *    - Provide a top-level `Dexto` object (runtime orchestrator) that accepts a `logger` in its config
+ *      and propagates it to sub-services (MCP manager, storage, etc.). This allows injection without
+ *      consumers needing to import Node-only loggers.
+ *
+ * 4) UI safety now (tactical):
+ *    - Ensure the Web UI imports types with `import type { ... } from '@dexto/core'` and uses API calls
+ *      for runtime. This avoids bundling Node code until the logger split is implemented.
+ *
+ * Verification & Guardrails
+ * - Add a CI check that building a minimal Next/Vite app that imports root `@dexto/core` types succeeds.
+ * - Mark side-effect status appropriately and keep top-level Node-only side effects out of root paths.
+ */
 type ChalkColor = 'black' | 'red' | 'green' | 'yellow' | 'blue' | 'magenta' | 'cyan' | 'white' | 'gray' | 'grey' | 'blackBright' | 'redBright' | 'greenBright' | 'yellowBright' | 'blueBright' | 'magentaBright' | 'cyanBright' | 'whiteBright';
-interface LoggerOptions {
+export interface LoggerOptions {
     level?: string;
     silent?: boolean;
     logToConsole?: boolean;
     customLogPath?: string;
 }
-declare class Logger {
+export declare class Logger {
     private logger;
     private isSilent;
     private logFilePath;
@@ -47,6 +89,6 @@ declare class Logger {
     }): void;
     displayError(message: string, error?: Error): void;
 }
-declare const logger: Logger;
-
-export { Logger, type LoggerOptions, logger };
+export declare const logger: Logger;
+export {};
+//# sourceMappingURL=logger.d.ts.map

package/dist/logger/logger.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"logger.d.ts","sourceRoot":"","sources":["../../src/logger/logger.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAyCG;AAyCH,KAAK,UAAU,GACT,OAAO,GACP,KAAK,GACL,OAAO,GACP,QAAQ,GACR,MAAM,GACN,SAAS,GACT,MAAM,GACN,OAAO,GACP,MAAM,GACN,MAAM,GACN,aAAa,GACb,WAAW,GACX,aAAa,GACb,cAAc,GACd,YAAY,GACZ,eAAe,GACf,YAAY,GACZ,aAAa,CAAC;AA2CpB,MAAM,WAAW,aAAa;IAC1B,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,MAAM,CAAC,EAAE,OAAO,CAAC;IACjB,YAAY,CAAC,EAAE,OAAO,CAAC;IACvB,aAAa,CAAC,EAAE,MAAM,CAAC;CAC1B;AAWD,qBAAa,MAAM;IACf,OAAO,CAAC,MAAM,CAAiB;IAC/B,OAAO,CAAC,QAAQ,CAAkB;IAClC,OAAO,CAAC,WAAW,CAAuB;IAC1C,OAAO,CAAC,YAAY,CAAkB;gBAE1B,OAAO,GAAE,aAAkB;IAsBvC,OAAO,CAAC,oBAAoB;IAc5B,OAAO,CAAC,gBAAgB;IA4DxB,KAAK,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,GAAG,EAAE,KAAK,CAAC,EAAE,UAAU;IASrD,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,GAAG,EAAE,KAAK,CAAC,EAAE,UAAU;IAQpD,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,GAAG,EAAE,KAAK,CAAC,EAAE,UAAU;IAQpD,IAAI,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,GAAG,EAAE,KAAK,CAAC,EAAE,UAAU;IAQpD,OAAO,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,GAAG,EAAE,KAAK,CAAC,EAAE,UAAU;IAQvD,KAAK,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,EAAE,IAAI,CAAC,EAAE,GAAG,EAAE,KAAK,CAAC,EAAE,UAAU;IAU9D,KAAK,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,GAAG,EAAE,KAAK,CAAC,EAAE,UAAU;IASrD,iBAAiB,CAAC,QAAQ,EAAE,GAAG;IAkB/B,QAAQ,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,GAAG;IAUpC,UAAU,CAAC,MAAM,EAAE,GAAG;IA4EtB,QAAQ,CAAC,KAAK,EAAE,MAAM;IAatB,cAAc,IAAI,MAAM,GAAG,IAAI;IAK/B,QAAQ,IAAI,MAAM;IAKlB,kBAAkB,CAAC,IAAI,EAAE;QACrB,KAAK,CAAC,EAAE,MAAM,CAAC;QACf,QAAQ,CAAC,EAAE,MAAM,CAAC;QAClB,gBAAgB,CAAC,EAAE;YAAE,KAAK,EAAE,MAAM,CAAC;YAAC,KAAK,EAAE,MAAM,EAAE,CAAA;SAAE,CAAC;QACtD,iBAAiB,CAAC,EAAE;YAAE,CAAC,GAAG,EAAE,MAAM,GAAG,MAAM,CAAA;SAAE,CAAC;QAC9C,SAAS,CAAC,EAAE;YAAE,KAAK,EAAE,MAAM,CAAC;YAAC,GAAG,EAAE,MAAM,CAAC;YAAC,QAAQ,EAAE,MAAM,CAAA;SAAE,CAAC;QAC7D,SAAS,CAAC,EAAE,MAAM,CAAC;QACnB,QAAQ,CAAC,EAAE,MAAM,CAAC;QAClB,OAAO,CAAC,EAAE,MAAM,CAAC;KACpB;IAoDD,YAAY,CAAC,OAAO,EAAE,MAAM,EAAE,KAAK,CAAC,EAAE,KAAK;CAkB9C;AAGD,eAAO,MAAM,MAAM,QAAe,CAAC"}

package/dist/logger/logger.js

@@ -1,3 +1,4 @@
+import "../chunk-C6A6W6XS.js";
 import * as winston from "winston";
 import chalk from "chalk";
 import boxen from "boxen";
@@ -131,26 +132,54 @@ class Logger {
   }
   // General logging methods with optional color parameter
   error(message, meta, color) {
-    this.logger.error(message, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.error(message, meta);
+    } else {
+      this.logger.error(message, { ...meta, color });
+    }
   }
   warn(message, meta, color) {
-    this.logger.warn(message, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.warn(message, meta);
+    } else {
+      this.logger.warn(message, { ...meta, color });
+    }
   }
   info(message, meta, color) {
-    this.logger.info(message, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.info(message, meta);
+    } else {
+      this.logger.info(message, { ...meta, color });
+    }
   }
   http(message, meta, color) {
-    this.logger.http(message, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.http(message, meta);
+    } else {
+      this.logger.http(message, { ...meta, color });
+    }
   }
   verbose(message, meta, color) {
-    this.logger.verbose(message, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.verbose(message, meta);
+    } else {
+      this.logger.verbose(message, { ...meta, color });
+    }
   }
   debug(message, meta, color) {
     const formattedMessage = typeof message === "string" ? message : JSON.stringify(message, null, 2);
-    this.logger.debug(formattedMessage, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.debug(formattedMessage, meta);
+    } else {
+      this.logger.debug(formattedMessage, { ...meta, color });
+    }
   }
   silly(message, meta, color) {
-    this.logger.silly(message, { ...meta, color });
+    if (meta instanceof Error) {
+      this.logger.silly(message, meta);
+    } else {
+      this.logger.silly(message, { ...meta, color });
+    }
   }
   // Display AI response in a box
   displayAIResponse(response) {

package/dist/mcp/error-codes.cjs

@@ -27,6 +27,7 @@ var MCPErrorCode = /* @__PURE__ */ ((MCPErrorCode2) => {
   MCPErrorCode2["DUPLICATE_NAME"] = "mcp_duplicate_name";
   MCPErrorCode2["CONNECTION_FAILED"] = "mcp_connection_failed";
   MCPErrorCode2["PROTOCOL_ERROR"] = "mcp_protocol_error";
+  MCPErrorCode2["SERVER_NOT_FOUND"] = "mcp_server_not_found";
   MCPErrorCode2["TOOL_NOT_FOUND"] = "mcp_tool_not_found";
   MCPErrorCode2["PROMPT_NOT_FOUND"] = "mcp_prompt_not_found";
   MCPErrorCode2["RESOURCE_NOT_FOUND"] = "mcp_resource_not_found";

package/dist/mcp/error-codes.d.ts

@@ -2,15 +2,15 @@
  * MCP-specific error codes
  * Includes server configuration, connection, and protocol errors
  */
-declare enum MCPErrorCode {
+export declare enum MCPErrorCode {
     SCHEMA_VALIDATION = "mcp_schema_validation",
     COMMAND_MISSING = "mcp_command_missing",
     DUPLICATE_NAME = "mcp_duplicate_name",
     CONNECTION_FAILED = "mcp_connection_failed",
     PROTOCOL_ERROR = "mcp_protocol_error",
+    SERVER_NOT_FOUND = "mcp_server_not_found",
     TOOL_NOT_FOUND = "mcp_tool_not_found",
     PROMPT_NOT_FOUND = "mcp_prompt_not_found",
     RESOURCE_NOT_FOUND = "mcp_resource_not_found"
 }
-
-export { MCPErrorCode };
+//# sourceMappingURL=error-codes.d.ts.map

package/dist/mcp/error-codes.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"error-codes.d.ts","sourceRoot":"","sources":["../../src/mcp/error-codes.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,oBAAY,YAAY;IAEpB,iBAAiB,0BAA0B;IAC3C,eAAe,wBAAwB;IACvC,cAAc,uBAAuB;IAGrC,iBAAiB,0BAA0B;IAG3C,cAAc,uBAAuB;IAGrC,gBAAgB,yBAAyB;IACzC,cAAc,uBAAuB;IACrC,gBAAgB,yBAAyB;IACzC,kBAAkB,2BAA2B;CAChD"}

package/dist/mcp/error-codes.js

@@ -1,9 +1,11 @@
+import "../chunk-C6A6W6XS.js";
 var MCPErrorCode = /* @__PURE__ */ ((MCPErrorCode2) => {
   MCPErrorCode2["SCHEMA_VALIDATION"] = "mcp_schema_validation";
   MCPErrorCode2["COMMAND_MISSING"] = "mcp_command_missing";
   MCPErrorCode2["DUPLICATE_NAME"] = "mcp_duplicate_name";
   MCPErrorCode2["CONNECTION_FAILED"] = "mcp_connection_failed";
   MCPErrorCode2["PROTOCOL_ERROR"] = "mcp_protocol_error";
+  MCPErrorCode2["SERVER_NOT_FOUND"] = "mcp_server_not_found";
   MCPErrorCode2["TOOL_NOT_FOUND"] = "mcp_tool_not_found";
   MCPErrorCode2["PROMPT_NOT_FOUND"] = "mcp_prompt_not_found";
   MCPErrorCode2["RESOURCE_NOT_FOUND"] = "mcp_resource_not_found";

package/dist/mcp/errors.cjs

@@ -64,6 +64,18 @@ class MCPError {
       "Use a unique name for each MCP server"
     );
   }
+  /**
+   * MCP server not found
+   */
+  static serverNotFound(serverName, reason) {
+    return new import_DextoRuntimeError.DextoRuntimeError(
+      import_error_codes.MCPErrorCode.SERVER_NOT_FOUND,
+      import_types.ErrorScope.MCP,
+      import_types.ErrorType.NOT_FOUND,
+      `MCP server '${serverName}' not found${reason ? `: ${reason}` : ""}`,
+      { serverName, reason }
+    );
+  }
   /**
    * MCP tool not found
    */

package/dist/mcp/errors.d.ts

@@ -1,23 +1,9 @@
 import { DextoRuntimeError } from '../errors/DextoRuntimeError.js';
-import '../errors/DextoBaseError.js';
-import '../errors/types.js';
-import '../agent/error-codes.js';
-import '../config/error-codes.js';
-import '../context/error-codes.js';
-import '../llm/error-codes.js';
-import './error-codes.js';
-import '../session/error-codes.js';
-import '../storage/error-codes.js';
-import '../systemPrompt/error-codes.js';
-import '../tools/error-codes.js';
-import '../preferences/error-codes.js';
-import '../agent/registry/error-codes.js';
-
 /**
  * MCP-specific error factory
  * Creates properly typed errors for MCP operations
  */
-declare class MCPError {
+export declare class MCPError {
     /**
      * MCP server connection failed
      */
@@ -36,6 +22,13 @@ declare class MCPError {
         name: string;
         existingName: string;
     }>;
+    /**
+     * MCP server not found
+     */
+    static serverNotFound(serverName: string, reason?: string): DextoRuntimeError<{
+        serverName: string;
+        reason: string | undefined;
+    }>;
     /**
      * MCP tool not found
      */
@@ -68,5 +61,4 @@ declare class MCPError {
         reason: string;
     }>;
 }
-
-export { MCPError };
+//# sourceMappingURL=errors.d.ts.map

package/dist/mcp/errors.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"errors.d.ts","sourceRoot":"","sources":["../../src/mcp/errors.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,MAAM,mCAAmC,CAAC;AAItE;;;GAGG;AACH,qBAAa,QAAQ;IACjB;;OAEG;IACH,MAAM,CAAC,gBAAgB,CAAC,UAAU,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM;;;;IAW1D;;OAEG;IACH,MAAM,CAAC,aAAa,CAAC,OAAO,EAAE,MAAM,EAAE,OAAO,CAAC,EAAE,OAAO;IAWvD;;OAEG;IACH,MAAM,CAAC,aAAa,CAAC,IAAI,EAAE,MAAM,EAAE,YAAY,EAAE,MAAM;;;;IAWvD;;OAEG;IACH,MAAM,CAAC,cAAc,CAAC,UAAU,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,MAAM;;;;IAUzD;;OAEG;IACH,MAAM,CAAC,YAAY,CAAC,QAAQ,EAAE,MAAM;;;IAUpC;;OAEG;IACH,MAAM,CAAC,cAAc,CAAC,UAAU,EAAE,MAAM;;;IAUxC;;OAEG;IACH,MAAM,CAAC,gBAAgB,CAAC,WAAW,EAAE,MAAM;;;IAU3C;;OAEG;IACH,MAAM,CAAC,kBAAkB,CAAC,OAAO,CAAC,EAAE,MAAM;;;IAU1C;;OAEG;IACH,MAAM,CAAC,iBAAiB,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM;;;;CAS5D"}

package/dist/mcp/errors.js

@@ -1,3 +1,4 @@
+import "../chunk-C6A6W6XS.js";
 import { DextoRuntimeError } from "../errors/DextoRuntimeError.js";
 import { ErrorScope, ErrorType } from "../errors/types.js";
 import { MCPErrorCode } from "./error-codes.js";
@@ -41,6 +42,18 @@ class MCPError {
       "Use a unique name for each MCP server"
     );
   }
+  /**
+   * MCP server not found
+   */
+  static serverNotFound(serverName, reason) {
+    return new DextoRuntimeError(
+      MCPErrorCode.SERVER_NOT_FOUND,
+      ErrorScope.MCP,
+      ErrorType.NOT_FOUND,
+      `MCP server '${serverName}' not found${reason ? `: ${reason}` : ""}`,
+      { serverName, reason }
+    );
+  }
   /**
    * MCP tool not found
    */

package/dist/mcp/index.d.ts

@@ -1,26 +1,8 @@
-export { MCPManager } from './manager.js';
-export { MCPClient } from './mcp-client.js';
-export { IMCPClient } from './types.js';
-export { HttpServerConfig, HttpServerConfigSchema, McpServerConfig, McpServerConfigSchema, ServerConfigs, ServerConfigsSchema, SseServerConfig, SseServerConfigSchema, StdioServerConfig, StdioServerConfigSchema, ValidatedHttpServerConfig, ValidatedMcpServerConfig, ValidatedServerConfigs, ValidatedSseServerConfig, ValidatedStdioServerConfig } from './schemas.js';
-export { MCPError } from './errors.js';
-export { MCPErrorCode } from './error-codes.js';
-export { McpServerContext, resolveAndValidateMcpServerConfig } from './resolver.js';
-import '../tools/types.js';
-import 'json-schema';
-import 'zod';
-import '@modelcontextprotocol/sdk/types.js';
-import '@modelcontextprotocol/sdk/client/index.js';
-import '../errors/DextoRuntimeError.js';
-import '../errors/DextoBaseError.js';
-import '../errors/types.js';
-import '../agent/error-codes.js';
-import '../config/error-codes.js';
-import '../context/error-codes.js';
-import '../llm/error-codes.js';
-import '../session/error-codes.js';
-import '../storage/error-codes.js';
-import '../systemPrompt/error-codes.js';
-import '../tools/error-codes.js';
-import '../preferences/error-codes.js';
-import '../agent/registry/error-codes.js';
-import '../utils/result.js';
+export * from './manager.js';
+export * from './mcp-client.js';
+export * from './types.js';
+export * from './schemas.js';
+export * from './errors.js';
+export * from './error-codes.js';
+export * from './resolver.js';
+//# sourceMappingURL=index.d.ts.map

package/dist/mcp/index.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/mcp/index.ts"],"names":[],"mappings":"AAAA,cAAc,cAAc,CAAC;AAC7B,cAAc,iBAAiB,CAAC;AAChC,cAAc,YAAY,CAAC;AAC3B,cAAc,cAAc,CAAC;AAC7B,cAAc,aAAa,CAAC;AAC5B,cAAc,kBAAkB,CAAC;AACjC,cAAc,eAAe,CAAC"}