jsonrpc-client-websocket 1.4.2 → 1.5.1

package/lib/deferred-promise.js

@@ -1,7 +1,4 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.DeferredPromise = void 0;
-class DeferredPromise {
+export class DeferredPromise {
     constructor() {
         this.promise = new Promise((resolve, reject) => {
             this.deferResolve = resolve;
@@ -18,4 +15,3 @@ class DeferredPromise {
         this.deferReject(error);
     }
 }
-exports.DeferredPromise = DeferredPromise;

package/lib/index.js

@@ -1,14 +1,2 @@
-"use strict";
-var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    Object.defineProperty(o, k2, { enumerable: true, get: function() { return m[k]; } });
-}) : (function(o, m, k, k2) {
-    if (k2 === undefined) k2 = k;
-    o[k2] = m[k];
-}));
-var __exportStar = (this && this.__exportStar) || function(m, exports) {
-    for (var p in m) if (p !== "default" && !Object.prototype.hasOwnProperty.call(exports, p)) __createBinding(exports, m, p);
-};
-Object.defineProperty(exports, "__esModule", { value: true });
-__exportStar(require("./jsonrpc-websocket"), exports);
-__exportStar(require("./jsonrpc.model"), exports);
+export * from './jsonrpc-websocket';
+export * from './jsonrpc.model';

package/lib/jsonrpc-websocket.d.ts

@@ -22,8 +22,8 @@ export declare class JsonRpcWebsocket {
     close(): Promise<CloseEvent>;
     on(methodName: string, callback: (...args: any[]) => any): void;
     off(methodName: string): void;
-    call(method: string, params?: any): Promise<JsonRpcResponse>;
-    notify(method: string, params?: any): void;
+    call(method: string, params?: unknown): Promise<JsonRpcResponse>;
+    notify(method: string, params?: unknown): void;
     private createWebsocket;
     private respondOk;
     private respondError;

package/lib/jsonrpc-websocket.js

@@ -1,17 +1,14 @@
-"use strict";
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.JsonRpcWebsocket = exports.WebsocketReadyStates = void 0;
-const getParameterNames = require("get-parameter-names");
-const deferred_promise_1 = require("./deferred-promise");
-const jsonrpc_model_1 = require("./jsonrpc.model");
-var WebsocketReadyStates;
+import * as getParameterNames from 'get-parameter-names';
+import { DeferredPromise } from './deferred-promise';
+import { JsonRpcErrorCodes } from './jsonrpc.model';
+export var WebsocketReadyStates;
 (function (WebsocketReadyStates) {
     WebsocketReadyStates[WebsocketReadyStates["CONNECTING"] = 0] = "CONNECTING";
     WebsocketReadyStates[WebsocketReadyStates["OPEN"] = 1] = "OPEN";
     WebsocketReadyStates[WebsocketReadyStates["CLOSING"] = 2] = "CLOSING";
     WebsocketReadyStates[WebsocketReadyStates["CLOSED"] = 3] = "CLOSED";
-})(WebsocketReadyStates = exports.WebsocketReadyStates || (exports.WebsocketReadyStates = {}));
-class JsonRpcWebsocket {
+})(WebsocketReadyStates || (WebsocketReadyStates = {}));
+export class JsonRpcWebsocket {
     constructor(url, requestTimeoutMs, onError) {
         this.url = url;
         this.requestTimeoutMs = requestTimeoutMs;
@@ -45,10 +42,9 @@ class JsonRpcWebsocket {
     off(methodName) {
         delete this.rpcMethods[methodName.toLowerCase()]; // case-insensitive!
     }
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/explicit-module-boundary-types
     call(method, params) {
         if (!this.websocket || this.state !== WebsocketReadyStates.OPEN) {
-            return Promise.reject({ code: jsonrpc_model_1.JsonRpcErrorCodes.INTERNAL_ERROR, message: 'The websocket is not opened' });
+            return Promise.reject({ code: JsonRpcErrorCodes.INTERNAL_ERROR, message: 'The websocket is not opened' });
         }
         const request = {
             id: this.getRequestId(),
@@ -61,11 +57,10 @@ class JsonRpcWebsocket {
         }
         catch (e) {
             // istanbul ignore next
-            return Promise.reject({ code: jsonrpc_model_1.JsonRpcErrorCodes.INTERNAL_ERROR, message: `Internal error. ${e}` });
+            return Promise.reject({ code: JsonRpcErrorCodes.INTERNAL_ERROR, message: `Internal error. ${e}` });
         }
         return this.createPendingRequest(request);
     }
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/explicit-module-boundary-types
     notify(method, params) {
         if (!this.websocket || this.state !== WebsocketReadyStates.OPEN) {
             throw new Error('The websocket is not opened');
@@ -87,8 +82,8 @@ class JsonRpcWebsocket {
         // See https://developer.mozilla.org/en-US/docs/Web/API/CloseEvent for close event codes
         // this.websocket = new WebSocket(this.url, ['jsonrpc-2.0']);
         this.websocket = new WebSocket(this.url);
-        const openDeferredPromise = new deferred_promise_1.DeferredPromise();
-        this.closeDeferredPromise = new deferred_promise_1.DeferredPromise();
+        const openDeferredPromise = new DeferredPromise();
+        this.closeDeferredPromise = new DeferredPromise();
         this.websocket.onopen = (event) => {
             openDeferredPromise.resolve(event);
         };
@@ -111,14 +106,12 @@ class JsonRpcWebsocket {
         this.websocket.onmessage = (message) => this.handleMessage(message.data);
         return openDeferredPromise.asPromise();
     }
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     respondOk(id, result) {
         this.respond(id, result);
     }
     respondError(id, error) {
         this.respond(id, undefined, error);
     }
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     respond(id, result, error) {
         // istanbul ignore if
         if (!this.websocket || this.state !== WebsocketReadyStates.OPEN) {
@@ -132,7 +125,7 @@ class JsonRpcWebsocket {
             jsonrpc: this.jsonRpcVersion,
             error: error,
             id: id,
-            result: result
+            result: result,
         };
         try {
             this.websocket.send(JSON.stringify(response));
@@ -148,14 +141,14 @@ class JsonRpcWebsocket {
             data = JSON.parse(msg);
         }
         catch (e) /* istanbul ignore next */ {
-            this.handleError(jsonrpc_model_1.JsonRpcErrorCodes.PARSE_ERROR, `Invalid JSON was received. ${e}`);
+            this.handleError(JsonRpcErrorCodes.PARSE_ERROR, `Invalid JSON was received. ${e}`);
             return;
         }
         const isResponse = !!data && this.hasProperty(data, 'id') && (this.hasProperty(data, 'result') || this.hasProperty(data, 'error'));
         const isRequest = !!data && this.hasProperty(data, 'method');
         const requestId = isRequest && data.id ? data.id : undefined;
         if (!data.jsonrpc || data.jsonrpc !== this.jsonRpcVersion) {
-            this.handleError(jsonrpc_model_1.JsonRpcErrorCodes.INVALID_REQUEST, `Invalid JSON RPC protocol version. Expecting ${this.jsonRpcVersion}, but got ${data.jsonrpc}`, requestId);
+            this.handleError(JsonRpcErrorCodes.INVALID_REQUEST, `Invalid JSON RPC protocol version. Expecting ${this.jsonRpcVersion}, but got ${data.jsonrpc}`, requestId);
             return;
         }
         if (isResponse) {
@@ -165,13 +158,13 @@ class JsonRpcWebsocket {
             this.handleRequest(data);
         }
         else {
-            this.handleError(jsonrpc_model_1.JsonRpcErrorCodes.INVALID_REQUEST, `Received unknown data: ${JSON.stringify(data)}`, requestId);
+            this.handleError(JsonRpcErrorCodes.INVALID_REQUEST, `Received unknown data: ${JSON.stringify(data)}`, requestId);
         }
     }
     handleRequest(request) {
         const method = this.rpcMethods[request.method.toLowerCase()]; // case-insensitive!
         if (!method) {
-            this.handleError(jsonrpc_model_1.JsonRpcErrorCodes.METHOD_NOT_FOUND, `Method '${request.method}' was not found`, request.id);
+            this.handleError(JsonRpcErrorCodes.METHOD_NOT_FOUND, `Method '${request.method}' was not found`, request.id);
             return;
         }
         let requestParams = [];
@@ -179,12 +172,17 @@ class JsonRpcWebsocket {
             requestParams = this.getRequestParams(method, request);
         }
         catch (error) {
-            this.handleError(jsonrpc_model_1.JsonRpcErrorCodes.INVALID_PARAMS, error.message, request.id);
+            this.handleError(JsonRpcErrorCodes.INVALID_PARAMS, error.message, request.id);
             return;
         }
-        const result = method(...requestParams);
-        if (request.id) {
-            this.respondOk(request.id, result);
+        try {
+            const result = method(...requestParams);
+            if (request.id) {
+                this.respondOk(request.id, result);
+            }
+        }
+        catch (error) {
+            this.handleError(JsonRpcErrorCodes.REQUEST_FAILED, `Method '${request.method}' has thrown: '${error.message}'`, request.id);
         }
     }
     // eslint-disable-next-line @typescript-eslint/no-explicit-any
@@ -202,7 +200,7 @@ class JsonRpcWebsocket {
                 if (method.length !== Object.keys(request.params).length) {
                     throw new Error(`Invalid parameters. Method '${request.method}' expects parameters [${parameterNames}], but got [${Object.keys(request.params)}]`);
                 }
-                parameterNames.forEach(paramName => {
+                parameterNames.forEach((paramName) => {
                     const paramValue = request.params[paramName];
                     if (paramValue === undefined) {
                         throw new Error(`Invalid parameters. Method '${request.method}' expects parameters [${parameterNames}], but got [${Object.keys(request.params)}]`);
@@ -220,7 +218,7 @@ class JsonRpcWebsocket {
         const activeRequest = this.pendingRequests[response.id];
         if (activeRequest === undefined) {
             this.callOnError({
-                code: jsonrpc_model_1.JsonRpcErrorCodes.INTERNAL_ERROR,
+                code: JsonRpcErrorCodes.INTERNAL_ERROR,
                 message: `Received a response with id ${response.id}, which does not match any requests made by this client`,
             });
             return;
@@ -229,7 +227,7 @@ class JsonRpcWebsocket {
         if (this.hasProperty(response, 'result') && this.hasProperty(response, 'error')) {
             const errorResponse = {
                 error: {
-                    code: jsonrpc_model_1.JsonRpcErrorCodes.INVALID_RESPONSE,
+                    code: JsonRpcErrorCodes.INVALID_RESPONSE,
                     message: `Invalid response. Either result or error must be set, but not both. ${JSON.stringify(response)}`,
                 },
                 id: activeRequest.request.id,
@@ -253,7 +251,7 @@ class JsonRpcWebsocket {
         }
     }
     createPendingRequest(request) {
-        const response = new deferred_promise_1.DeferredPromise();
+        const response = new DeferredPromise();
         this.pendingRequests[request.id] = {
             request,
             response,
@@ -270,7 +268,7 @@ class JsonRpcWebsocket {
             }
             const response = {
                 error: {
-                    code: jsonrpc_model_1.JsonRpcErrorCodes.REQUEST_TIMEOUT,
+                    code: JsonRpcErrorCodes.REQUEST_TIMEOUT,
                     message: `Request ${activeRequest.request.id} exceeded the maximum time of ${this.requestTimeoutMs}ms and was aborted`,
                 },
                 id: activeRequest.request.id,
@@ -288,9 +286,7 @@ class JsonRpcWebsocket {
     getRequestId() {
         return ++this.requestId;
     }
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     hasProperty(object, propertyName) {
         return !!Object.prototype.hasOwnProperty.call(object, propertyName);
     }
 }
-exports.JsonRpcWebsocket = JsonRpcWebsocket;

package/lib/jsonrpc.model.d.ts

@@ -5,7 +5,8 @@ export declare enum JsonRpcErrorCodes {
     INVALID_PARAMS = -32602,
     INTERNAL_ERROR = -32603,
     INVALID_RESPONSE = -32001,
-    REQUEST_TIMEOUT = -32002
+    REQUEST_TIMEOUT = -32002,
+    REQUEST_FAILED = -32003
 }
 export declare class JsonRpcError {
     code: number;

package/lib/jsonrpc.model.js

@@ -1,9 +1,6 @@
-"use strict";
 /* tslint:disable:max-classes-per-file */
-Object.defineProperty(exports, "__esModule", { value: true });
-exports.JsonRpcRequest = exports.JsonRpcResponse = exports.JsonRpcError = exports.JsonRpcErrorCodes = void 0;
 // See https://www.jsonrpc.org/specification for the protocol details
-var JsonRpcErrorCodes;
+export var JsonRpcErrorCodes;
 (function (JsonRpcErrorCodes) {
     JsonRpcErrorCodes[JsonRpcErrorCodes["PARSE_ERROR"] = -32700] = "PARSE_ERROR";
     JsonRpcErrorCodes[JsonRpcErrorCodes["INVALID_REQUEST"] = -32600] = "INVALID_REQUEST";
@@ -13,13 +10,11 @@ var JsonRpcErrorCodes;
     // App specific
     JsonRpcErrorCodes[JsonRpcErrorCodes["INVALID_RESPONSE"] = -32001] = "INVALID_RESPONSE";
     JsonRpcErrorCodes[JsonRpcErrorCodes["REQUEST_TIMEOUT"] = -32002] = "REQUEST_TIMEOUT";
-})(JsonRpcErrorCodes = exports.JsonRpcErrorCodes || (exports.JsonRpcErrorCodes = {}));
-class JsonRpcError {
+    JsonRpcErrorCodes[JsonRpcErrorCodes["REQUEST_FAILED"] = -32003] = "REQUEST_FAILED";
+})(JsonRpcErrorCodes || (JsonRpcErrorCodes = {}));
+export class JsonRpcError {
 }
-exports.JsonRpcError = JsonRpcError;
-class JsonRpcResponse {
+export class JsonRpcResponse {
 }
-exports.JsonRpcResponse = JsonRpcResponse;
-class JsonRpcRequest {
+export class JsonRpcRequest {
 }
-exports.JsonRpcRequest = JsonRpcRequest;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "jsonrpc-client-websocket",
-  "version": "1.4.2",
+  "version": "1.5.1",
   "description": "A simple JSON RPC 2.0 websocket client",
   "main": "lib/index.js",
   "types": "lib/index.d.ts",
@@ -8,16 +8,17 @@
     "lib/**/*"
   ],
   "scripts": {
-    "test": "jest --config jestconfig.json",
+    "test": "jest",
     "coverage": "jest --config jestconfig.json --coverage",
     "build": "tsc",
-    "format": "prettier --write \"src/**/*.ts\" \"src/**/*.js\"",
+    "format": "prettier --write \"src/**/*.ts\"",
     "lint": "tsc --noEmit && eslint . --ext .js,.jsx,.ts,.tsx --fix",
     "prepublishOnly": "npm test && npm run lint",
     "prepare": "npm run build",
     "preversion": "npm run lint",
     "version": "npm run format && git add -A src",
-    "postversion": "git push && git push --tags"
+    "postversion": "git push && git push --tags",
+    "updateAll": "npm-check-updates --upgrade"
   },
   "repository": {
     "type": "git",
@@ -37,18 +38,20 @@
   "author": "Fabricio Bastian",
   "license": "MIT",
   "devDependencies": {
-    "@types/jest": "^26.0.14",
-    "@typescript-eslint/eslint-plugin": "^4.3.0",
-    "@typescript-eslint/parser": "^4.3.0",
-    "eslint": "^7.10.0",
-    "eslint-config-prettier": "^6.12.0",
-    "eslint-plugin-jest": "^24.0.2",
-    "jest": "^26.4.2",
-    "jest-websocket-mock": "^2.2.0",
-    "mock-socket": "^9.0.3",
-    "prettier": "^2.1.2",
-    "ts-jest": "^26.4.1",
-    "typescript": "^4.0.3"
+    "@types/jest": "^27.0.3",
+    "@typescript-eslint/eslint-plugin": "^5.4.0",
+    "@typescript-eslint/parser": "^5.4.0",
+    "eslint": "^8.3.0",
+    "eslint-config-prettier": "^8.3.0",
+    "eslint-plugin-jest": "^25.3.0",
+    "jest": "^27.3.1",
+    "jest-websocket-mock": "^2.2.1",
+    "mock-socket": "^9.0.8",
+    "npm-check-updates": "^12.0.2",
+    "prettier": "^2.5.0",
+    "ts-jest": "^27.0.7",
+    "ts-node": "^10.4.0",
+    "typescript": "^4.5.2"
   },
   "dependencies": {
     "get-parameter-names": "^0.3.0"