polfan-server-js-client 0.2.101 → 0.2.103

package/.idea/workspace.xml

@@ -4,10 +4,9 @@
     <option name="autoReloadType" value="SELECTIVE" />
   </component>
   <component name="ChangeListManager">
-    <list default="true" id="831dae43-0da1-47fd-a5f7-33dd5eec2992" name="Changes" comment="Adjust session data changes">
-      <change afterPath="$PROJECT_DIR$/src/types/src/schemes/commands/UnfollowTopic.ts" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/src/types" beforeDir="false" afterPath="$PROJECT_DIR$/src/types" afterDir="false" />
-      <change beforePath="$PROJECT_DIR$/src/types/src/index.ts" beforeDir="false" afterPath="$PROJECT_DIR$/src/types/src/index.ts" afterDir="false" />
+    <list default="true" id="831dae43-0da1-47fd-a5f7-33dd5eec2992" name="Changes" comment="Topic unfollowing">
+      <change beforePath="$PROJECT_DIR$/package.json" beforeDir="false" afterPath="$PROJECT_DIR$/package.json" afterDir="false" />
+      <change beforePath="$PROJECT_DIR$/src/WebApiChatClient.ts" beforeDir="false" afterPath="$PROJECT_DIR$/src/WebApiChatClient.ts" afterDir="false" />
     </list>
     <option name="SHOW_DIALOG" value="false" />
     <option name="HIGHLIGHT_CONFLICTS" value="true" />
@@ -30,7 +29,7 @@
         <entry key="$PROJECT_DIR$/src/types" value="562c7beff95ea1eaf5c236df978f50a614a146af" />
       </map>
     </option>
-    <option name="RECENT_GIT_ROOT_PATH" value="$PROJECT_DIR$/src/types" />
+    <option name="RECENT_GIT_ROOT_PATH" value="$PROJECT_DIR$" />
     <option name="ROOT_SYNC" value="DONT_SYNC" />
   </component>
   <component name="GitHubPullRequestSearchHistory">{
@@ -71,7 +70,7 @@
     "RunOnceActivity.typescript.service.memoryLimit.init": "true",
     "codeWithMe.voiceChat.enabledByDefault": "false",
     "com.intellij.ml.llm.matterhorn.ej.ui.settings.DefaultModelSelectionForGA.v1": "true",
-    "git-widget-placeholder": "main",
+    "git-widget-placeholder": "master",
     "git.auto.fetch.suggestion.counter": "1",
     "ignore.virus.scanning.warn.message": "true",
     "junie.onboarding.icon.badge.shown": "true",
@@ -210,23 +209,7 @@
       <workItem from="1782664203095" duration="5539000" />
       <workItem from="1782909199211" duration="2404000" />
       <workItem from="1783939298935" duration="21428000" />
-      <workItem from="1784102649250" duration="4695000" />
-    </task>
-    <task id="LOCAL-00052" summary="Changes in chat protocol and schemes">
-      <option name="closed" value="true" />
-      <created>1754249755333</created>
-      <option name="number" value="00052" />
-      <option name="presentableId" value="LOCAL-00052" />
-      <option name="project" value="LOCAL" />
-      <updated>1754249755333</updated>
-    </task>
-    <task id="LOCAL-00053" summary="New member count field in RoomSummary">
-      <option name="closed" value="true" />
-      <created>1754319172537</created>
-      <option name="number" value="00053" />
-      <option name="presentableId" value="LOCAL-00053" />
-      <option name="project" value="LOCAL" />
-      <updated>1754319172537</updated>
+      <workItem from="1784102649250" duration="10048000" />
     </task>
     <task id="LOCAL-00054" summary="Version bump">
       <option name="closed" value="true" />
@@ -604,7 +587,23 @@
       <option name="project" value="LOCAL" />
       <updated>1784225321435</updated>
     </task>
-    <option name="localTasksCounter" value="101" />
+    <task id="LOCAL-00101" summary="Topic unfollowing">
+      <option name="closed" value="true" />
+      <created>1784229730189</created>
+      <option name="number" value="00101" />
+      <option name="presentableId" value="LOCAL-00101" />
+      <option name="project" value="LOCAL" />
+      <updated>1784229730189</updated>
+    </task>
+    <task id="LOCAL-00102" summary="Topic unfollowing">
+      <option name="closed" value="true" />
+      <created>1784236719408</created>
+      <option name="number" value="00102" />
+      <option name="presentableId" value="LOCAL-00102" />
+      <option name="project" value="LOCAL" />
+      <updated>1784236719408</updated>
+    </task>
+    <option name="localTasksCounter" value="103" />
     <servers />
   </component>
   <component name="TypeScriptGeneratedFilesManager">
@@ -649,7 +648,6 @@
     </option>
   </component>
   <component name="VcsManagerConfiguration">
-    <MESSAGE value="Fix URL parameter handling in GET and DELETE requests" />
     <MESSAGE value="Improve hasOldest behavior when resetToLatest used" />
     <MESSAGE value="Align permissions to server changes" />
     <MESSAGE value="Allow to define ManageRoom permission on room level" />
@@ -674,7 +672,8 @@
     <MESSAGE value="Adjust method names" />
     <MESSAGE value="Fix roles checking in spaceless rooms" />
     <MESSAGE value="Adjust session data changes" />
-    <option name="LAST_COMMIT_MESSAGE" value="Adjust session data changes" />
+    <MESSAGE value="Topic unfollowing" />
+    <option name="LAST_COMMIT_MESSAGE" value="Topic unfollowing" />
   </component>
   <component name="XSLT-Support.FileAssociations.UIState">
     <expand />

package/build/index.cjs.js

@@ -1906,6 +1906,9 @@ var FollowedTopicsManager = /*#__PURE__*/function (_EventTarget) {
     _this.tracker.client.on('TopicFollowed', function (ev) {
       return _this.handleTopicFollowed(ev);
     });
+    _this.tracker.client.on('TopicUnfollowed', function (ev) {
+      return _this.handleTopicUnfollowed(ev);
+    });
     _this.tracker.client.on('NewMessage', function (ev) {
       return _this.handleNewMessage(ev);
     });
@@ -2345,6 +2348,13 @@ var FollowedTopicsManager = /*#__PURE__*/function (_EventTarget) {
       this.setFollowedTopicsArray([ev.followedTopic.location.roomId], [ev.followedTopic]);
       this.invalidateUnreadSummaries(ev.followedTopic.location.roomId, ev.followedTopic.location.topicId);
     }
+  }, {
+    key: "handleTopicUnfollowed",
+    value: function handleTopicUnfollowed(ev) {
+      var _this$followedTopics$3;
+      (_this$followedTopics$3 = this.followedTopics.get(ev.location.roomId)) === null || _this$followedTopics$3 === void 0 || _this$followedTopics$3["delete"](ev.location.topicId);
+      this.invalidateUnreadSummaries(ev.location.roomId, ev.location.topicId);
+    }
   }, {
     key: "handleRoomDeleted",
     value: function handleRoomDeleted(ev) {
@@ -2399,8 +2409,8 @@ var FollowedTopicsManager = /*#__PURE__*/function (_EventTarget) {
   }, {
     key: "handleTopicDeleted",
     value: function handleTopicDeleted(ev) {
-      var _this$followedTopics$3;
-      (_this$followedTopics$3 = this.followedTopics.get(ev.location.roomId)) === null || _this$followedTopics$3 === void 0 || _this$followedTopics$3["delete"](ev.location.topicId);
+      var _this$followedTopics$4;
+      (_this$followedTopics$4 = this.followedTopics.get(ev.location.roomId)) === null || _this$followedTopics$4 === void 0 || _this$followedTopics$4["delete"](ev.location.topicId);
       this.invalidateUnreadSummaries(ev.location.roomId, ev.location.topicId);
     }
 
@@ -5218,7 +5228,7 @@ var WebApiChatClient = /*#__PURE__*/function (_AbstractChatClient) {
     WebApiChatClient_classCallCheck(this, WebApiChatClient);
     _this = WebApiChatClient_callSuper(this, WebApiChatClient);
     WebApiChatClient_defineProperty(_this, "Event", WebApiChatClientEvent);
-    WebApiChatClient_defineProperty(_this, "sendStack", void 0);
+    WebApiChatClient_defineProperty(_this, "sendStack", []);
     _this.options = options;
     return _this;
   }
@@ -5227,17 +5237,18 @@ var WebApiChatClient = /*#__PURE__*/function (_AbstractChatClient) {
     key: "send",
     value: function () {
       var _send = WebApiChatClient_asyncToGenerator(/*#__PURE__*/WebApiChatClient_regenerator().m(function _callee(commandType, commandData) {
-        var envelope;
+        var envelope, item;
         return WebApiChatClient_regenerator().w(function (_context) {
           while (1) switch (_context.n) {
             case 0:
               envelope = this.createEnvelope(commandType, commandData);
-              this.sendStack.push({
+              item = {
                 data: envelope,
                 attempts: 0,
                 lastTimeoutId: null
-              });
-              this.makeApiCall(this.sendStack.length - 1);
+              };
+              this.sendStack.push(item);
+              this.makeApiCall(item);
               return _context.a(2, this.createPromiseFromCommandEnvelope(envelope));
           }
         }, _callee, this);
@@ -5251,36 +5262,53 @@ var WebApiChatClient = /*#__PURE__*/function (_AbstractChatClient) {
     key: "destroy",
     value: function destroy() {
       var _this2 = this;
-      // Cancel all awaiting requests
-      this.sendStack.forEach(function (item) {
+      // Cancel all awaiting requests, rejecting their promises so no caller
+      // is left awaiting forever
+      var pending = this.sendStack;
+      this.sendStack = [];
+      pending.forEach(function (item) {
         if (item.lastTimeoutId) {
           clearTimeout(item.lastTimeoutId);
         }
-        _this2.awaitingResponse["delete"](item.data.ref);
+        _this2.handleEnvelopeSendError(item.data, new Error('Client destroyed'));
       });
-      this.sendStack = [];
       this.emit(this.Event.destroy, false);
     }
   }, {
     key: "onMessage",
     value: function () {
-      var _onMessage = WebApiChatClient_asyncToGenerator(/*#__PURE__*/WebApiChatClient_regenerator().m(function _callee2(reqId, response) {
-        var envelope;
+      var _onMessage = WebApiChatClient_asyncToGenerator(/*#__PURE__*/WebApiChatClient_regenerator().m(function _callee2(item, response) {
+        var envelope, _t;
         return WebApiChatClient_regenerator().w(function (_context2) {
-          while (1) switch (_context2.n) {
+          while (1) switch (_context2.p = _context2.n) {
             case 0:
-              this.sendStack.splice(reqId, 1);
+              _context2.p = 0;
               _context2.n = 1;
               return response.json();
             case 1:
               envelope = _context2.v;
+              _context2.n = 3;
+              break;
+            case 2:
+              _context2.p = 2;
+              _t = _context2.v;
+              // Non-envelope response (e.g. a proxy error page) - failed attempt
+              this.onError(item, "envelope ".concat(item.data.ref, " (HTTP ").concat(response.status, ", unparsable body)"));
+              return _context2.a(2);
+            case 3:
+              if (this.removeFromStack(item)) {
+                _context2.n = 4;
+                break;
+              }
+              return _context2.a(2);
+            case 4:
               this.handleIncomingEnvelope(envelope);
               this.emit(envelope.type, envelope.data);
               this.emit(this.Event.message, envelope);
-            case 2:
+            case 5:
               return _context2.a(2);
           }
-        }, _callee2, this);
+        }, _callee2, this, [[0, 2]]);
       }));
       function onMessage(_x3, _x4) {
         return _onMessage.apply(this, arguments);
@@ -5289,43 +5317,60 @@ var WebApiChatClient = /*#__PURE__*/function (_AbstractChatClient) {
     }()
   }, {
     key: "onError",
-    value: function onError(reqId, body) {
+    value: function onError(item, body) {
       var _this$options$attempt,
         _this3 = this,
         _this$options$attempt2;
-      if (this.sendStack[reqId].attempts >= ((_this$options$attempt = this.options.attemptsToSend) !== null && _this$options$attempt !== void 0 ? _this$options$attempt : 10)) {
-        this.sendStack.splice(reqId, 1);
-        this.handleEnvelopeSendError(this.sendStack[reqId].data, new Error("Cannot send ".concat(body, "; aborting after reaching the maximum connection errors")));
+      if (!this.sendStack.includes(item)) {
+        return; // Destroyed in the meantime
+      }
+      if (item.attempts >= ((_this$options$attempt = this.options.attemptsToSend) !== null && _this$options$attempt !== void 0 ? _this$options$attempt : 10)) {
+        this.removeFromStack(item);
+        this.handleEnvelopeSendError(item.data, new Error("Cannot send ".concat(body, "; aborting after reaching the maximum connection errors")));
         return;
       }
-      this.sendStack[reqId].lastTimeoutId = setTimeout(function () {
-        return _this3.makeApiCall(reqId);
+      item.lastTimeoutId = setTimeout(function () {
+        return _this3.makeApiCall(item);
       }, (_this$options$attempt2 = this.options.attemptDelayMs) !== null && _this$options$attempt2 !== void 0 ? _this$options$attempt2 : 3000);
     }
   }, {
     key: "makeApiCall",
-    value: function makeApiCall(reqId) {
+    value: function makeApiCall(item) {
       var _this$options$queryPa,
         _this4 = this;
-      this.sendStack[reqId].attempts++;
-      var bodyJson = JSON.stringify(this.sendStack[reqId].data);
+      if (!this.sendStack.includes(item)) {
+        return; // Destroyed before this (re)try fired
+      }
+      item.attempts++;
+      item.lastTimeoutId = null;
+      var bodyJson = JSON.stringify(item.data);
       var headers = {
         'Content-Type': 'application/json',
         Accept: 'application/json'
       };
       headers.Authorization = "Bearer ".concat(this.options.token);
-      var params = new URLSearchParams((_this$options$queryPa = this.options.queryParams) !== null && _this$options$queryPa !== void 0 ? _this$options$queryPa : {});
-      var url = "".concat(this.options.url).concat(params ? '?' + params : '');
+      var query = new URLSearchParams((_this$options$queryPa = this.options.queryParams) !== null && _this$options$queryPa !== void 0 ? _this$options$queryPa : {}).toString();
+      var url = query ? "".concat(this.options.url, "?").concat(query) : this.options.url;
       fetch(url, {
         headers: headers,
         body: bodyJson,
         method: 'POST'
       }).then(function (response) {
-        return _this4.onMessage(reqId, response);
+        return _this4.onMessage(item, response);
       })["catch"](function () {
-        return _this4.onError(reqId, bodyJson);
+        return _this4.onError(item, bodyJson);
       });
     }
+  }, {
+    key: "removeFromStack",
+    value: function removeFromStack(item) {
+      var index = this.sendStack.indexOf(item);
+      if (index === -1) {
+        return false;
+      }
+      this.sendStack.splice(index, 1);
+      return true;
+    }
   }]);
 }(AbstractChatClient);
 ;// ./src/AbstractRestClient.ts