sync-rpc 1.3.0 → 1.3.4

package/lib/__tests__/index.test.js

@@ -22,3 +22,22 @@ rpc.FUNCTION_PRIORITY.forEach((fn, i) => {
     }
   });
 });
+rpc.FUNCTION_PRIORITY.forEach((fn, i) => {
+  let longMessage = '';
+  for (let i = 0; i < 100000; i++) {
+    longMessage += 'My Long Message Content';
+  }
+  test('profile large ' + fn.name, () => {
+    try {
+      rpc.configuration.fastestFunction = fn;
+      const start = Date.now();
+      for (let i = 0; i < 10; i++) {
+        expect(client(longMessage)).toBe(`sent ${longMessage} to My Server`);
+      }
+      const end = Date.now();
+      console.log('large ' + fn.name + ': ' + (end - start));
+    } catch (ex) {
+      console.log('large ' + fn.name + ' fails');
+    }
+  });
+});

package/lib/index.js

@@ -48,18 +48,18 @@ function findPort() {
   const findPortResult = spawnSync(process.execPath, [
     require.resolve('./find-port'),
   ]);
-  if (findPortResult.status !== 0) {
-    throw new Error(
-      findPortResult.stderr.toString() ||
-        'find port exited with code ' + findPortResult.status
-    );
-  }
   if (findPortResult.error) {
     if (typeof findPortResult.error === 'string') {
       throw new Error(findPortResult.error);
     }
     throw findPortResult.error;
   }
+  if (findPortResult.status !== 0) {
+    throw new Error(
+      findPortResult.stderr.toString() ||
+        'find port exited with code ' + findPortResult.status
+    );
+  }
   const portString = findPortResult.stdout.toString('utf8').trim();
   if (!/^[0-9]+$/.test(portString)) {
     throw new Error('Invalid port number string returned: ' + portString);
@@ -91,7 +91,12 @@ function nativeNC(port, input) {
 }
 
 function nodeNC(port, input) {
-  return spawnSync(process.execPath, ['-e', nodeNetCatSrc(port, input)]);
+  const src = nodeNetCatSrc(port, input);
+  if (src.length < 1000) {
+    return spawnSync(process.execPath, ['-e', src]);
+  } else {
+    return spawnSync(process.execPath, [], {input: src});
+  }
 }
 
 function test(fn, port) {
@@ -117,25 +122,25 @@ function sendMessage(input) {
   try {
     return JSON.parse(res.stdout.toString('utf8'));
   } catch (ex) {
+    if (res.error) {
+      if (typeof res.error === 'string') res.error = new Error(res.error);
+      throw res.error;
+    }
     if (res.status !== 0) {
       throw new Error(
         configuration.fastestFunction.name +
           ' failed:\n' +
-          res.stdout.toString() +
+          (res.stdout && res.stdout.toString()) +
           '\n' +
-          res.stderr.toString()
+          (res.stderr && res.stderr.toString())
       );
     }
-    if (res.error) {
-      if (typeof res.error === 'string') res.error = new Error(res.error);
-      throw res.error;
-    }
     throw new Error(
       configuration.fastestFunction.name +
         ' failed:\n' +
-        res.stdout.toString() +
+        (res.stdout && res.stdout).toString() +
         '\n' +
-        res.stderr.toString()
+        (res.stderr && res.stderr).toString()
     );
   }
 }

package/lib/worker.js

@@ -9,10 +9,10 @@ const modules = [];
 
 const NULL_PROMISE = Promise.resolve(null);
 const server = net.createServer({allowHalfOpen: true}, c => {
-  let responed = false;
+  let responded = false;
   function respond(data) {
     if (responded) return;
-    responed = true;
+    responded = true;
     c.end(JSON.stringify(data));
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sync-rpc",
-  "version": "1.3.0",
+  "version": "1.3.4",
   "main": "lib/index.js",
   "description": "Run asynchronous commands synchronously by putting them in a separate process",
   "keywords": [],