botversion-sdk 2.1.0 → 2.2.1

package/bin/scan-backend.js

@@ -1,4 +1,4 @@
-#!/usr/bin/env node
+#!/usr/bin/env node
 "use strict";
 
 var path = require("path");
@@ -29,7 +29,7 @@ var BotVersionClient = require("../client");
 
 var apiKey = process.env.BOTVERSION_API_KEY;
 var platformUrl =
-  process.env.BOTVERSION_PLATFORM_URL || "https://console.botversion.com";
+  process.env.BOTVERSION_PLATFORM_URL || "http://localhost:3000";
 
 if (!apiKey) {
   console.error(

package/bin/scan-frontend.js

@@ -1,4 +1,4 @@
-#!/usr/bin/env node
+#!/usr/bin/env node
 "use strict";
 
 var path = require("path");
@@ -30,7 +30,7 @@ var sdk = require("../index");
 
 var apiKey = process.env.BOTVERSION_API_KEY;
 var platformUrl =
-  process.env.BOTVERSION_PLATFORM_URL || "https://console.botversion.com";
+  process.env.BOTVERSION_PLATFORM_URL || "http://localhost:3000";
 
 if (!apiKey) {
   console.error(

package/client.js

@@ -11,7 +11,7 @@ const url = require("url");
  */
 function BotVersionClient(options) {
   this.apiKey = options.apiKey;
-  this.platformUrl = options.platformUrl || "https://console.botversion.com";
+  this.platformUrl = options.platformUrl || "http://localhost:3000";
   this.debug = options.debug || false;
   this.timeout = options.timeout || 30000;
 
@@ -19,6 +19,8 @@ function BotVersionClient(options) {
   this._queue = [];
   this._flushTimer = null;
   this._flushDelay = options.flushDelay || 3000;
+  // Tracks in-flight updates per endpoint so they are sent one after another
+  this._inflight = {};
   var self = this;
   process.on("beforeExit", function () {
     if (self._queue.length > 0) self._flush();
@@ -74,13 +76,10 @@ BotVersionClient.prototype._flush = function () {
   var toSend = self._queue.slice();
   self._queue = [];
 
-  self
-    ._post("/api/sdk/register-endpoints", {
-      workspaceKey: self.apiKey,
-      endpoints: toSend,
-    })
-    .then(function (data) {})
-    .catch(function (err) {});
+  self._post("/api/sdk/register-endpoints", {
+    workspaceKey: self.apiKey,
+    endpoints: toSend,
+  });
 };
 
 /**
@@ -89,14 +88,38 @@ BotVersionClient.prototype._flush = function () {
 BotVersionClient.prototype.updateEndpoint = function (endpoint) {
   var self = this;
 
-  return self._post("/api/sdk/update-endpoint", {
+  var payload = {
     workspaceKey: self.apiKey,
     method: endpoint.method,
     path: endpoint.path,
     requestBody: endpoint.requestBody || null,
     responseBody: endpoint.responseBody || null,
     detectedBy: endpoint.detectedBy || "runtime",
-  });
+  };
+
+  // Updates for the same endpoint go out one after another, so a later update
+  // (e.g. its reply shape) can never race the first registration of that endpoint.
+  var key = String(endpoint.method).toUpperCase() + ":" + endpoint.path;
+  var previous = self._inflight[key] || Promise.resolve();
+  var send = function () {
+    return self._post("/api/sdk/update-endpoint", payload).then(
+      function (res) {
+        return res;
+      },
+      function (err) {
+        throw err;
+      },
+    );
+  };
+  var current = previous.then(send, send);
+  self._inflight[key] = current;
+
+  var cleanup = function () {
+    if (self._inflight[key] === current) delete self._inflight[key];
+  };
+  current.then(cleanup, cleanup);
+
+  return current;
 };
 
 BotVersionClient.prototype.registerRoutePatterns = function (

package/index.js

@@ -226,6 +226,9 @@ function runFullScan(
   if (shouldScanBackend) {
     var endpoints = runBackendScanner(detectedBackend, app, options, cwd);
     result.endpointCount = endpoints.length;
+    endpoints.filter(function (e) {
+      return e.method === "GET" || e.method === "DELETE";
+    });
 
     if (endpoints.length > 0) {
       client.registerEndpoints(endpoints);
@@ -421,7 +424,7 @@ var BotVersion = {
 
     this._client = new BotVersionClient({
       apiKey: options.apiKey,
-      platformUrl: options.platformUrl || "https://console.botversion.com",
+      platformUrl: options.platformUrl || "http://localhost:3000",
       debug: options.debug || false,
       timeout: options.timeout || 30000,
     });