vscode-jsonrpc - npm Package Compare versions

Comparing version 2.3.2-next.2 to 2.3.2-next.3

lib/main.d.ts

@@ -28,3 +28,3 @@ import { Message, RequestMessage, RequestType, ResponseError, ErrorCodes, NotificationMessage, NotificationType } from './messages';
 export interface Tracer {
-    log(message: string): void;
+    log(message: string, data?: string): void;
 }
@@ -31,0 +31,0 @@ export interface MessageConnection {

lib/main.js

@@ -203,3 +203,4 @@ /* --------------------------------------------------------------------------------------------
                 if (is.defined(responseMessage.error)) {
-                    responsePromise.reject(responseMessage.error);
+                    var error = responseMessage.error;
+                    responsePromise.reject(new messages_1.ResponseError(error.code, error.message, error.data));
                 }
@@ -278,50 +279,54 @@ else if (is.defined(responseMessage.result)) {
     function traceRequest(message) {
-        tracer.log("[" + (new Date().toLocaleTimeString()) + "] Sending request '" + message.method + " - (" + message.id + ")'.");
+        var data = undefined;
         if (trace === Trace.Verbose && message.params) {
-            tracer.log("Params: " + JSON.stringify(message.params, null, 4) + "\n\n");
+            data = "Params: " + JSON.stringify(message.params, null, 4) + "\n\n";
         }
+        tracer.log("Sending request '" + message.method + " - (" + message.id + ")'.", data);
     }
     function traceSendNotification(message) {
-        tracer.log("[" + (new Date().toLocaleTimeString()) + "] Sending notification '" + message.method + "'.");
+        var data = undefined;
         if (trace === Trace.Verbose) {
             if (message.params) {
-                tracer.log("Params: " + JSON.stringify(message.params, null, 4) + "\n\n");
+                data = "Params: " + JSON.stringify(message.params, null, 4) + "\n\n";
             }
             else {
-                tracer.log('No parameters provided.\n\n');
+                data = 'No parameters provided.\n\n';
             }
         }
+        tracer.log("Sending notification '" + message.method + "'.", data);
     }
     function traceReceivedNotification(message) {
-        tracer.log("[" + (new Date().toLocaleTimeString()) + "] Received notification '" + message.method + "'.");
+        var data = undefined;
         if (trace === Trace.Verbose) {
             if (message.params) {
-                tracer.log("Params: " + JSON.stringify(message.params, null, 4) + "\n\n");
+                data = "Params: " + JSON.stringify(message.params, null, 4) + "\n\n";
             }
             else {
-                tracer.log('No parameters provided.\n\n');
+                data = 'No parameters provided.\n\n';
             }
         }
+        tracer.log("Received notification '" + message.method + "'.", data);
     }
     function traceResponse(message, responsePromise) {
-        if (responsePromise) {
-            var error = message.error ? " Request failed: " + message.error.message + " (" + message.error.code + ")." : '';
-            tracer.log("[" + (new Date().toLocaleTimeString()) + "] Recevied response '" + responsePromise.method + " - (" + message.id + ")' in " + (Date.now() - responsePromise.timerStart) + "ms." + error);
-        }
-        else {
-            tracer.log("[" + (new Date().toLocaleTimeString()) + "] Recevied response " + message.id + " without active response promise.");
-        }
+        var data = undefined;
         if (trace === Trace.Verbose) {
             if (message.error && message.error.data) {
-                tracer.log("Error data: " + JSON.stringify(message.error.data, null, 4) + "\n\n");
+                data = "Error data: " + JSON.stringify(message.error.data, null, 4) + "\n\n";
             }
             else {
                 if (message.result) {
-                    tracer.log("Result: " + JSON.stringify(message.result, null, 4) + "\n\n");
+                    data = "Result: " + JSON.stringify(message.result, null, 4) + "\n\n";
                 }
-                else {
-                    tracer.log('No result returned.\n\n');
+                else if (is.undefined(message.error)) {
+                    data = 'No result returned.\n\n';
                 }
             }
         }
+        if (responsePromise) {
+            var error = message.error ? " Request failed: " + message.error.message + " (" + message.error.code + ")." : '';
+            tracer.log("Recevied response '" + responsePromise.method + " - (" + message.id + ")' in " + (Date.now() - responsePromise.timerStart) + "ms." + error, data);
+        }
+        else {
+            tracer.log("Recevied response " + message.id + " without active response promise.", data);
+        }
     }
@@ -328,0 +333,0 @@ var callback = function (message) {

package.json

 {
 	"name": "vscode-jsonrpc",
 	"description": "A json rpc implementation over streams",
-	"version": "2.3.2-next.2",
+	"version": "2.3.2-next.3",
 	"author": "Microsoft Corporation",
@@ -6,0 +6,0 @@ "license": "MIT",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

63611

increased by0.07%

Lines of code

1522

increased by0.33%

No dependency changes