modality-kit 0.12.13 → 0.12.15

package/dist/index.js

@@ -5180,7 +5180,17 @@ class JSONRPCUtils {
   }
   static deserialize(data) {
     try {
-      return JSON.parse(data);
+      let messageStr;
+      if (typeof data !== "string") {
+        if (Buffer.isBuffer(data)) {
+          messageStr = data.toString();
+        } else {
+          return data;
+        }
+      } else {
+        messageStr = data;
+      }
+      return JSON.parse(messageStr);
     } catch {
       return null;
     }
@@ -5489,6 +5499,10 @@ class ERROR_METHOD_NOT_FOUND extends ErrorCode {
   code = -32601 /* METHOD_NOT_FOUND */;
 }
 
+class ERROR_PARSE_ERROR extends ErrorCode {
+  code = -32700 /* PARSE_ERROR */;
+}
+
 class JSONRPCManager extends JSONRPCCall {
   methods = new Map;
   config;
@@ -5508,27 +5522,29 @@ class JSONRPCManager extends JSONRPCCall {
     try {
       switch (validation.messageType) {
         case "request":
-          return await this.processRequest(validation.message, options);
+          return await this.processMethod(validation.message, options);
         case "notification":
-          return this.processNotification(validation.message, options);
+          this.processNotification(validation.message, options);
+          break;
         case "response":
           const response = validation.message;
           logger2.info("Received response message, handling internally", [
             response,
             response.id
           ]);
-          return this.handleResponse(response);
+          this.handleResponse(response);
+          break;
         case "batch":
           return await this.processBatchRequest(validation.message, options);
         default:
           return JSONRPCUtils.createErrorResponse(JSONRPCUtils.createError(-32600 /* INVALID_REQUEST */, "Unknown message type"), validation.message.id || null);
       }
     } catch (error) {
-      console.error("Error processing JSON-RPC message:", error);
+      logger2.error("Error processing JSON-RPC message:", error);
       return JSONRPCUtils.createErrorResponse(this.config.errorHandler(error), validation.message.id || null);
     }
   }
-  async processRequest(request, options = {}) {
+  async processMethod(request, options = {}) {
     const context = {
       ...options,
       request,
@@ -5548,7 +5564,7 @@ class JSONRPCManager extends JSONRPCCall {
       }
       return JSONRPCUtils.createSuccessResponse(result, request.id);
     } catch (error) {
-      console.error(`Error executing method '${request.method}':`, error);
+      logger2.error(`Error executing method '${request.method}':`, error);
       const jsonrpcError = this.config.errorHandler(error, context);
       if (this.eventHandlers.onMethodError) {
         this.eventHandlers.onMethodError(request.method, jsonrpcError, context);
@@ -5565,15 +5581,14 @@ class JSONRPCManager extends JSONRPCCall {
     try {
       const methodConfig = this.methods.get(notification.method);
       if (!methodConfig) {
-        throw new ERROR_METHOD_NOT_FOUND(`Notification method '${notification.method}' not found`);
+        throw new ERROR_METHOD_NOT_FOUND(`Notification method '${notification.method}' not handled.`);
       }
       if (this.eventHandlers.onMethodCall) {
         this.eventHandlers.onMethodCall(notification.method, notification.params, context);
       }
       return await methodConfig.handler(notification.params, context);
     } catch (error) {
-      console.error(`Error executing notification '${notification.method}':`, error);
-      throw error;
+      logger2.info(`Notification get exception: '${notification.method}':`, error);
     }
   }
   async processBatchRequest(batchRequest, options = {}) {
@@ -5583,7 +5598,7 @@ class JSONRPCManager extends JSONRPCCall {
     }
     const promises = batchRequest.map((item) => {
       if (JSONRPCUtils.isRequest(item)) {
-        return this.processRequest(item, options);
+        return this.processMethod(item, options);
       } else if (JSONRPCUtils.isNotification(item)) {
         return this.processNotification(item, options);
       }
@@ -5605,10 +5620,6 @@ class JSONRPCManager extends JSONRPCCall {
       errorType
     });
   }
-  sendMessage(message, options) {
-    console.warn("JSONRPCManager.sendMessage not implemented - message not sent:");
-    console.dir({ message, options }, { depth: null, color: true });
-  }
   sendNotification(method, params, options) {
     const notification = JSONRPCUtils.createNotification(method, params);
     return this.sendMessage(notification, options);
@@ -5622,7 +5633,7 @@ class JSONRPCManager extends JSONRPCCall {
   unregisterMethod(methodName) {
     const removed = this.methods.delete(methodName);
     if (removed) {
-      console.log(`Unregistered JSON-RPC method: ${methodName}`);
+      logger2.info(`Unregistered JSON-RPC method: ${methodName}`);
     }
     return removed;
   }
@@ -5636,12 +5647,9 @@ class JSONRPCManager extends JSONRPCCall {
   }
   async validateMessage(data, options = {}) {
     try {
-      const messageStr = typeof data === "string" ? data : data.toString();
-      const message = JSONRPCUtils.deserialize(messageStr);
+      const message = JSONRPCUtils.deserialize(data);
       if (!message) {
-        console.error("Failed to parse JSON-RPC message:", messageStr);
-        const errorResponse = JSONRPCUtils.createErrorResponse(JSONRPCUtils.createError(-32700 /* PARSE_ERROR */, "Parse error"), null);
-        return this.sendMessage(errorResponse, options);
+        throw new ERROR_PARSE_ERROR("Failed to parse JSON-RPC message");
       }
       const validation = JSONRPCUtils.validateMessage(message);
       if (!validation.valid) {
@@ -5654,7 +5662,7 @@ class JSONRPCManager extends JSONRPCCall {
       }
     } catch (err) {
       const error = err;
-      console.error("Error handling validateMessage:", error);
+      logger2.error("Error handling validateMessage:", error);
       const errorResponse = JSONRPCUtils.createErrorResponse(this.config.errorHandler(error), null);
       return this.sendMessage(errorResponse, options);
     }

package/dist/types/JSONRPCUtils.d.ts

@@ -70,5 +70,5 @@ export declare class JSONRPCUtils {
     /**
      * Deserialize a JSON-RPC message from string
      */
-    static deserialize(data: string): JSONRPCMessage | null;
+    static deserialize(data: string | Buffer | Record<string, any>): JSONRPCMessage | null;
 }

package/dist/types/jsonrpc-manager.d.ts

@@ -57,7 +57,7 @@ export declare class ERROR_METHOD_NOT_FOUND extends ErrorCode {
 /**
  * Central JSON-RPC Manager class
  */
-export declare class JSONRPCManager<TContext> extends JSONRPCCall {
+export declare abstract class JSONRPCManager<TContext> extends JSONRPCCall {
     private methods;
     private config;
     private eventHandlers;
@@ -69,7 +69,7 @@ export declare class JSONRPCManager<TContext> extends JSONRPCCall {
     /**
      * Process a JSON-RPC request
      */
-    private processRequest;
+    private processMethod;
     /**
      * Process a JSON-RPC notification
      */
@@ -85,7 +85,7 @@ export declare class JSONRPCManager<TContext> extends JSONRPCCall {
     /**
      * Send a message (to be overridden by implementation)
      */
-    protected sendMessage(message: JSONRPCMessage, options?: TContext): any;
+    protected abstract sendMessage(message: JSONRPCMessage, options?: TContext): any;
     /**
      * Send a JSON-RPC notification (no response expected)
      */
@@ -113,7 +113,7 @@ export declare class JSONRPCManager<TContext> extends JSONRPCCall {
     /**
      * Process incoming WebSocket message
      */
-    validateMessage(data: string | Buffer, options?: any): Promise<void>;
+    validateMessage(data: string | Buffer | Record<string, any>, options?: any): Promise<void>;
     /**
      * Get manager statistics
      */

package/package.json

@@ -1,5 +1,5 @@
 {
-  "version": "0.12.13",
+  "version": "0.12.15",
   "name": "modality-kit",
   "repository": {
     "type": "git",
@@ -28,7 +28,7 @@
     "build:clean": "find ./dist -name '*.*' | xargs rm -rf",
     "build:types": "bun tsc -p ./",
     "build:src": "bun build src/index.ts --outdir dist",
-    "build": "bun run build:clean && bun run build:src && bun run build:types",
+    "build": "bun run build:clean && bun run build:types && bun run build:src",
     "dev": "bunx concurrently 'bun --watch tsc  -p ./' 'bun build:src -- --watch --sourcemap=inline'",
     "test": "bun test",
     "prepublishOnly": "npm run build && npm run test"