@atlaskit/collab-provider 8.8.1 → 8.9.0

package/dist/esm/provider/index.js

@@ -15,14 +15,13 @@ import { Channel } from '../channel';
 import { createLogger } from '../helpers/utils';
 import AnalyticsHelper from '../analytics/analytics-helper';
 import { telepointerCallback } from '../participants/telepointers-helper';
-import { ParticipantsService } from '../participants/participants-service';
 import { DestroyError, GetCurrentStateError, GetFinalAcknowledgedStateError, ProviderInitialisationError, SendTransactionError, SetEditorWidthError, SetMetadataError, SetTitleError, NCS_ERROR_CODE } from '../errors/error-types';
 import { MetadataService } from '../metadata/metadata-service';
 import { DocumentService } from '../document/document-service';
+import { NamespaceService } from '../namespace/namespace-service';
+import { ParticipantsService } from '../participants/participants-service';
 import { errorCodeMapper } from '../errors/error-code-mapper';
 var logger = createLogger('Provider', 'black');
-var SEND_PRESENCE_INTERVAL = 150 * 1000; // 150 seconds
-
 var OUT_OF_SYNC_PERIOD = 3 * 1000; // 3 seconds
 
 export var MAX_STEP_REJECTED_ERROR = 15;
@@ -35,10 +34,25 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
     _this = _super.call(this);
     _defineProperty(_assertThisInitialized(_this), "isChannelInitialized", false);
     _defineProperty(_assertThisInitialized(_this), "isProviderInitialized", false);
-    // To keep track of the namespace event changes from the server.
-    _defineProperty(_assertThisInitialized(_this), "isNamespaceLocked", false);
+    // isPreinitializating acts as a feature flag to determine when the provider has been initialized early
+    // and also contains the initial draft
+    _defineProperty(_assertThisInitialized(_this), "isPreinitializing", false);
+    /**
+     * Wrapper for this.emit, it binds scope for callbacks and waits for initialising of the editor before emitting events.
+     * Waiting for the collab provider to become connected to the editor ensures the editor doesn't miss any events.
+     * @param evt - Event name to emit to subscribers
+     * @param data - Event data to emit to subscribers
+     */
     _defineProperty(_assertThisInitialized(_this), "emitCallback", function (evt, data) {
-      return _this.emit(evt, data);
+      // When the provider is initialized early, we want the editor state promise to resolve before emitting events
+      // to ensure that it is ready to listen to the events fired by NCS
+      if (_this.isPreinitializing) {
+        _this.getStatePromise.then(function () {
+          return _this.emit(evt, data);
+        });
+      } else {
+        _this.emit(evt, data);
+      }
     });
     _defineProperty(_assertThisInitialized(_this), "initializeChannel", function () {
       _this.emit('connecting', {
@@ -49,7 +63,7 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
         var sid = _ref.sid,
           initialized = _ref.initialized;
         _this.sessionId = sid;
-        _this.emit('connected', {
+        _this.emitCallback('connected', {
           sid: sid,
           initial: !initialized
         });
@@ -75,7 +89,7 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
         Date.now() - _this.disconnectedAt >= OUT_OF_SYNC_PERIOD) {
           _this.documentService.throttledCatchup();
         }
-        _this.startInactiveRemover();
+        _this.participantsService.startInactiveRemover(_this.sessionId);
         _this.disconnectedAt = undefined;
       }).on('init', function (_ref2) {
         var doc = _ref2.doc,
@@ -88,7 +102,10 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
           metadata: metadata
         });
         _this.metadataService.updateMetadata(metadata);
-      }).on('restore', _this.documentService.onRestore).on('steps:added', _this.documentService.onStepsAdded).on('metadata:changed', _this.metadataService.onMetadataChanged).on('participant:telepointer', _this.onParticipantTelepointer.bind(_assertThisInitialized(_this))).on('presence:joined', _this.onPresenceJoined.bind(_assertThisInitialized(_this))).on('presence', _this.onPresence.bind(_assertThisInitialized(_this))).on('participant:left', _this.onParticipantLeft.bind(_assertThisInitialized(_this))).on('participant:updated', _this.onParticipantUpdated.bind(_assertThisInitialized(_this))).on('disconnect', _this.onDisconnected.bind(_assertThisInitialized(_this))).on('error', _this.onErrorHandled).on('status', _this.onNamespaceStatusChanged.bind(_assertThisInitialized(_this))).connect(shouldInitialize);
+        _this.isProviderInitialized = true;
+      }).on('restore', _this.documentService.onRestore).on('steps:added', _this.documentService.onStepsAdded).on('metadata:changed', _this.metadataService.onMetadataChanged).on('participant:telepointer', function (payload) {
+        return _this.participantsService.onParticipantTelepointer(payload, _this.sessionId);
+      }).on('presence:joined', _this.participantsService.onPresenceJoined).on('presence', _this.participantsService.onPresence).on('participant:left', _this.participantsService.onParticipantLeft).on('participant:updated', _this.participantsService.onParticipantUpdated).on('disconnect', _this.onDisconnected.bind(_assertThisInitialized(_this))).on('error', _this.onErrorHandled).on('status', _this.namespaceService.onNamespaceStatusChanged).connect(shouldInitialize);
     });
     /**
      * @param {InternalError} error The error to handle
@@ -108,90 +125,15 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
         if (mappedError) {
           var _this$analyticsHelper2;
           (_this$analyticsHelper2 = _this.analyticsHelper) === null || _this$analyticsHelper2 === void 0 ? void 0 : _this$analyticsHelper2.sendErrorEvent(mappedError, 'Error emitted');
-          _this.emit('error', mappedError);
+          _this.emitCallback('error', mappedError);
         }
       }
     });
-    _defineProperty(_assertThisInitialized(_this), "sendPresence", function () {
-      try {
-        if (_this.presenceUpdateTimeout) {
-          clearTimeout(_this.presenceUpdateTimeout);
-        }
-        _this.channel.broadcast('participant:updated', {
-          sessionId: _this.sessionId,
-          userId: _this.userId,
-          clientId: _this.clientId
-        });
-        _this.presenceUpdateTimeout = window.setTimeout(function () {
-          return _this.sendPresence();
-        }, SEND_PRESENCE_INTERVAL);
-      } catch (error) {
-        var _this$analyticsHelper3;
-        // We don't want to throw errors for Presence features as they tend to self-restore
-        (_this$analyticsHelper3 = _this.analyticsHelper) === null || _this$analyticsHelper3 === void 0 ? void 0 : _this$analyticsHelper3.sendErrorEvent(error, 'Error while sending presence');
-      }
-    });
-    /**
-     * Called when a participant joins the session.
-     *
-     * We keep track of participants internally in this class, and emit the `presence` event to update
-     * the active avatars in the editor.
-     * This method will be triggered from backend to notify all participants to exchange presence
-     */
-    _defineProperty(_assertThisInitialized(_this), "onPresenceJoined", function (_ref3) {
-      var sessionId = _ref3.sessionId;
-      try {
-        logger('Participant joined with session: ', sessionId);
-        // This expose existing users to the newly joined user
-        _this.sendPresence();
-      } catch (error) {
-        var _this$analyticsHelper4;
-        // We don't want to throw errors for Presence features as they tend to self-restore
-        (_this$analyticsHelper4 = _this.analyticsHelper) === null || _this$analyticsHelper4 === void 0 ? void 0 : _this$analyticsHelper4.sendErrorEvent(error, 'Error while joining presence');
-      }
-    });
-    _defineProperty(_assertThisInitialized(_this), "onPresence", function (_ref4) {
-      var userId = _ref4.userId;
-      try {
-        logger('onPresence userId: ', userId);
-        _this.userId = userId;
-        _this.sendPresence();
-        _this.channel.sendPresenceJoined();
-      } catch (error) {
-        var _this$analyticsHelper5;
-        // We don't want to throw errors for Presence features as they tend to self-restore
-        (_this$analyticsHelper5 = _this.analyticsHelper) === null || _this$analyticsHelper5 === void 0 ? void 0 : _this$analyticsHelper5.sendErrorEvent(error, 'Error while receiving presence');
-      }
-    });
-    /**
-     * Called when a participant leaves the session.
-     *
-     * We emit the `presence` event to update the active avatars in the editor.
-     */
-    _defineProperty(_assertThisInitialized(_this), "onParticipantLeft", function (data) {
-      return _this.participantsService.participantLeft(data, _this.emitCallback);
-    });
-    _defineProperty(_assertThisInitialized(_this), "startInactiveRemover", function () {
-      return _this.participantsService.removeInactiveParticipants(_this.sessionId, _this.emitCallback);
-    });
-    /**
-     * Called when we receive an update event from another participant.
-     */
-    _defineProperty(_assertThisInitialized(_this), "onParticipantUpdated", function (data) {
-      return _this.participantsService.updateParticipant(data, _this.config.getUser, _this.emitCallback);
-    });
-    /**
-     * Called when we receive a telepointer update from another
-     * participant.
-     */
-    _defineProperty(_assertThisInitialized(_this), "onParticipantTelepointer", function (data) {
-      return _this.participantsService.participantTelepointer(data, _this.sessionId, _this.config.getUser, _this.emitCallback);
-    });
     // Note: this gets triggered on page reload for Firefox (not other browsers) because of closeOnBeforeunload: false
-    _defineProperty(_assertThisInitialized(_this), "onDisconnected", function (_ref5) {
-      var reason = _ref5.reason;
+    _defineProperty(_assertThisInitialized(_this), "onDisconnected", function (_ref3) {
+      var reason = _ref3.reason;
       _this.disconnectedAt = Date.now();
-      _this.participantsService.disconnect(reason, _this.sessionId, _this.emitCallback);
+      _this.participantsService.disconnect(reason, _this.sessionId);
     });
     /**
      * Returns the documents metadata
@@ -205,7 +147,7 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
      * @throws {GetCurrentStateError} Something went wrong while returning the current state
      */
     _defineProperty(_assertThisInitialized(_this), "getCurrentState", /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee() {
-      var _this$analyticsHelper6;
+      var _this$analyticsHelper3;
       return _regeneratorRuntime.wrap(function _callee$(_context) {
         while (1) switch (_context.prev = _context.next) {
           case 0:
@@ -214,7 +156,7 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
           case 4:
             _context.prev = 4;
             _context.t0 = _context["catch"](0);
-            (_this$analyticsHelper6 = _this.analyticsHelper) === null || _this$analyticsHelper6 === void 0 ? void 0 : _this$analyticsHelper6.sendErrorEvent(_context.t0, 'Error while returning ADF version of current draft document');
+            (_this$analyticsHelper3 = _this.analyticsHelper) === null || _this$analyticsHelper3 === void 0 ? void 0 : _this$analyticsHelper3.sendErrorEvent(_context.t0, 'Error while returning ADF version of current draft document');
             throw new GetCurrentStateError('Error while returning the current state of the draft document', _context.t0);
           case 8:
           case "end":
@@ -228,7 +170,7 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
      * @throws {GetFinalAcknowledgedStateError} Something went wrong while returning the acknowledged state
      */
     _defineProperty(_assertThisInitialized(_this), "getFinalAcknowledgedState", /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee2() {
-      var _this$analyticsHelper7;
+      var _this$analyticsHelper4;
       return _regeneratorRuntime.wrap(function _callee2$(_context2) {
         while (1) switch (_context2.prev = _context2.next) {
           case 0:
@@ -237,7 +179,7 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
           case 4:
             _context2.prev = 4;
             _context2.t0 = _context2["catch"](0);
-            (_this$analyticsHelper7 = _this.analyticsHelper) === null || _this$analyticsHelper7 === void 0 ? void 0 : _this$analyticsHelper7.sendErrorEvent(_context2.t0, 'Error while returning ADF version of the final draft document');
+            (_this$analyticsHelper4 = _this.analyticsHelper) === null || _this$analyticsHelper4 === void 0 ? void 0 : _this$analyticsHelper4.sendErrorEvent(_context2.t0, 'Error while returning ADF version of the final draft document');
             throw new GetFinalAcknowledgedStateError('Error while returning the final acknowledged state of the draft document', _context2.t0);
           case 8:
           case "end":
@@ -248,51 +190,6 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
     _defineProperty(_assertThisInitialized(_this), "getUnconfirmedSteps", function () {
       return _this.documentService.getUnconfirmedSteps();
     });
-    /**
-     * ESS-2916 namespace status event- lock/unlock
-     */
-    _defineProperty(_assertThisInitialized(_this), "onNamespaceStatusChanged", /*#__PURE__*/function () {
-      var _ref8 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee3(data) {
-        var isLocked, waitTimeInMs, timestamp, start;
-        return _regeneratorRuntime.wrap(function _callee3$(_context3) {
-          while (1) switch (_context3.prev = _context3.next) {
-            case 0:
-              isLocked = data.isLocked, waitTimeInMs = data.waitTimeInMs, timestamp = data.timestamp;
-              start = Date.now();
-              logger("Received a namespace status changed event ", {
-                data: data
-              });
-              if (!(isLocked && waitTimeInMs)) {
-                _context3.next = 8;
-                break;
-              }
-              _this.isNamespaceLocked = true;
-              logger("Received a namespace status change event ", {
-                isLocked: isLocked
-              });
-
-              // To protect the collab editing process from locked out due to BE
-              setTimeout(function () {
-                logger("The namespace lock has expired", {
-                  waitTime: Date.now() - start,
-                  timestamp: timestamp
-                });
-                _this.isNamespaceLocked = false;
-              }, waitTimeInMs);
-              return _context3.abrupt("return");
-            case 8:
-              _this.isNamespaceLocked = false;
-              logger("The page lock has expired");
-            case 10:
-            case "end":
-              return _context3.stop();
-          }
-        }, _callee3);
-      }));
-      return function (_x) {
-        return _ref8.apply(this, arguments);
-      };
-    }());
     /**
      * Used when the provider is disconnected or destroyed to prevent perpetual timers from continuously running
      */
@@ -306,11 +203,16 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
     _this.isChannelInitialized = false;
     _this.initialDraft = _this.config.initialDraft;
     _this.isProviderInitialized = false;
-    _this.participantsService = new ParticipantsService(_this.analyticsHelper);
+    _this.isPreinitializing = false;
+    _this.participantsService = new ParticipantsService(_this.analyticsHelper, undefined, _this.emitCallback, _this.config.getUser, _this.channel.broadcast, _this.channel.sendPresenceJoined);
     _this.metadataService = new MetadataService(_this.emitCallback, _this.channel.sendMetadata);
     _this.documentService = new DocumentService(_this.participantsService, _this.analyticsHelper, _this.channel.fetchCatchup, _this.emitCallback, _this.channel.broadcast, function () {
       return _this.userId;
     }, _this.onErrorHandled, _this.metadataService);
+    _this.getStatePromise = new Promise(function (resolve) {
+      _this.getStatePromiseResolve = resolve;
+    });
+    _this.namespaceService = new NamespaceService();
     return _this;
   }
   _createClass(Provider, [{
@@ -329,38 +231,54 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
     }
 
     /**
-     * Initialisation logic, called by the editor in the collab-edit plugin
-     * @param {Object} parameters ...
-     * @param {Function} parameters.getState Function that returns the editor state, used to retrieve collab-edit properties and to interact with prosemirror-collab
-     * @param {SyncUpErrorFunction} parameters.onSyncUpError (Optional) Function that gets called when the sync of steps fails after retrying 30 times, used by Editor to log to analytics
+     * Initialisation logic, called by the editor in the collab-edit plugin.
+     *
+     * When getState is nullish and a initialDraft is provided the collab provider is in a state of pre-initialization,
+     * the provider starts to enable the connection to NCS, but the provider will not emit any events until this function
+     * is called again with a getState function, indicating that the editor is loaded and ready to receive the emits.
+     *
+     * @param {Object} options ...
+     * @param {Function} options.getState Function that returns the editor state, used to retrieve collab-edit properties and to interact with prosemirror-collab
+     * @param {SyncUpErrorFunction} options.onSyncUpError (Optional) Function that gets called when the sync of steps fails after retrying 30 times, used by Editor to log to analytics
      * @throws {ProviderInitialisationError} Something went wrong during provider initialisation
      */
   }, {
     key: "setup",
-    value: function setup(_ref9) {
-      var getState = _ref9.getState,
-        onSyncUpError = _ref9.onSyncUpError;
+    value: function setup(_ref6) {
+      var getState = _ref6.getState,
+        onSyncUpError = _ref6.onSyncUpError;
       this.checkForCookies();
       try {
-        var collabPlugin = getState().plugins.find(function (p) {
-          return p.key === 'collab$';
-        });
-        if (collabPlugin === undefined) {
-          throw new ProviderInitialisationError('Collab provider attempted to initialise, but Editor state is missing collab plugin');
+        // if setup is called with no state and the initial draft is already provided
+        // set a flag to mark early provider setup
+        if (!getState && this.initialDraft) {
+          this.isPreinitializing = true;
+        }
+        if (getState) {
+          // if provider has already been initialized earlier, resolve the state once it is available
+          if (this.isPreinitializing) {
+            this.getStatePromiseResolve();
+          }
+          var collabPlugin = getState().plugins.find(function (p) {
+            return p.key === 'collab$';
+          });
+          if (collabPlugin === undefined) {
+            throw new ProviderInitialisationError('Collab provider attempted to initialise, but Editor state is missing collab plugin');
+          }
+          this.clientId = collabPlugin.spec.config.clientID;
+          this.documentService.setup({
+            getState: getState,
+            onSyncUpError: onSyncUpError,
+            clientId: this.clientId
+          });
         }
-        this.clientId = collabPlugin.spec.config.clientID;
-        this.documentService.setup({
-          getState: getState,
-          onSyncUpError: onSyncUpError,
-          clientId: this.clientId
-        });
         if (!this.isChannelInitialized) {
           this.initializeChannel();
           this.isChannelInitialized = true;
         }
       } catch (initError) {
-        var _this$analyticsHelper8;
-        (_this$analyticsHelper8 = this.analyticsHelper) === null || _this$analyticsHelper8 === void 0 ? void 0 : _this$analyticsHelper8.sendErrorEvent(initError, 'Error while initialising the provider');
+        var _this$analyticsHelper5;
+        (_this$analyticsHelper5 = this.analyticsHelper) === null || _this$analyticsHelper5 === void 0 ? void 0 : _this$analyticsHelper5.sendErrorEvent(initError, 'Error while initialising the provider');
         // Throw error so consumers are aware the initialisation failed when initialising themselves
         throw new ProviderInitialisationError('Provider initialisation error', initError);
       }
@@ -370,9 +288,9 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
     key: "checkForCookies",
     value: function checkForCookies() {
       if (!global.navigator.cookieEnabled) {
-        var _this$analyticsHelper9;
+        var _this$analyticsHelper6;
         var initError = new ProviderInitialisationError('Cookies are not enabled. Please enable cookies to use collaborative editing.');
-        (_this$analyticsHelper9 = this.analyticsHelper) === null || _this$analyticsHelper9 === void 0 ? void 0 : _this$analyticsHelper9.sendErrorEvent(initError, 'Error while initialising the provider - cookies disabled');
+        (_this$analyticsHelper6 = this.analyticsHelper) === null || _this$analyticsHelper6 === void 0 ? void 0 : _this$analyticsHelper6.sendErrorEvent(initError, 'Error while initialising the provider - cookies disabled');
         throw new ProviderInitialisationError('Provider initialisation error - cookies disabled', initError);
       }
     }
@@ -389,14 +307,14 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
     value: function send(_tr, _oldState, newState) {
       try {
         // Don't send steps while the document is locked (eg. when restoring the document)
-        if (this.isNamespaceLocked) {
+        if (this.namespaceService.getIsNamespaceLocked()) {
           logger('The document is temporary locked');
           return;
         }
         this.documentService.send(_tr, _oldState, newState);
       } catch (error) {
-        var _this$analyticsHelper10;
-        (_this$analyticsHelper10 = this.analyticsHelper) === null || _this$analyticsHelper10 === void 0 ? void 0 : _this$analyticsHelper10.sendErrorEvent(error, 'Error while sending steps for a transaction');
+        var _this$analyticsHelper7;
+        (_this$analyticsHelper7 = this.analyticsHelper) === null || _this$analyticsHelper7 === void 0 ? void 0 : _this$analyticsHelper7.sendErrorEvent(error, 'Error while sending steps for a transaction');
         throw new SendTransactionError('Error while sending steps for a transaction', error);
       }
     }
@@ -424,9 +342,9 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
           this.channel.broadcast('participant:telepointer', payload, callback);
         }
       } catch (error) {
-        var _this$analyticsHelper11;
+        var _this$analyticsHelper8;
         // We don't want to throw errors for Presence features as they tend to self-restore
-        (_this$analyticsHelper11 = this.analyticsHelper) === null || _this$analyticsHelper11 === void 0 ? void 0 : _this$analyticsHelper11.sendErrorEvent(error, 'Error while sending message - telepointer');
+        (_this$analyticsHelper8 = this.analyticsHelper) === null || _this$analyticsHelper8 === void 0 ? void 0 : _this$analyticsHelper8.sendErrorEvent(error, 'Error while sending message - telepointer');
       }
     }
   }, {
@@ -465,8 +383,8 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
         _get(_getPrototypeOf(Provider.prototype), "unsubscribeAll", this).call(this);
         this.channel.disconnect();
       } catch (error) {
-        var _this$analyticsHelper12;
-        (_this$analyticsHelper12 = this.analyticsHelper) === null || _this$analyticsHelper12 === void 0 ? void 0 : _this$analyticsHelper12.sendErrorEvent(error, 'Error while shutting down the collab provider');
+        var _this$analyticsHelper9;
+        (_this$analyticsHelper9 = this.analyticsHelper) === null || _this$analyticsHelper9 === void 0 ? void 0 : _this$analyticsHelper9.sendErrorEvent(error, 'Error while shutting down the collab provider');
         throw new DestroyError('Error while shutting down the collab provider', error);
       }
       this.clearTimers();
@@ -486,8 +404,8 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
       try {
         this.metadataService.setTitle(title, broadcast);
       } catch (error) {
-        var _this$analyticsHelper13;
-        (_this$analyticsHelper13 = this.analyticsHelper) === null || _this$analyticsHelper13 === void 0 ? void 0 : _this$analyticsHelper13.sendErrorEvent(error, 'Error while setting title');
+        var _this$analyticsHelper10;
+        (_this$analyticsHelper10 = this.analyticsHelper) === null || _this$analyticsHelper10 === void 0 ? void 0 : _this$analyticsHelper10.sendErrorEvent(error, 'Error while setting title');
         throw new SetTitleError('Error while setting title', error);
       }
     }
@@ -505,8 +423,8 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
       try {
         this.metadataService.setEditorWidth(editorWidth, broadcast);
       } catch (error) {
-        var _this$analyticsHelper14;
-        (_this$analyticsHelper14 = this.analyticsHelper) === null || _this$analyticsHelper14 === void 0 ? void 0 : _this$analyticsHelper14.sendErrorEvent(error, 'Error while setting editor width');
+        var _this$analyticsHelper11;
+        (_this$analyticsHelper11 = this.analyticsHelper) === null || _this$analyticsHelper11 === void 0 ? void 0 : _this$analyticsHelper11.sendErrorEvent(error, 'Error while setting editor width');
         throw new SetEditorWidthError('Error while setting editor width', error);
       }
     }
@@ -522,8 +440,8 @@ export var Provider = /*#__PURE__*/function (_Emitter) {
       try {
         this.metadataService.setMetadata(metadata);
       } catch (error) {
-        var _this$analyticsHelper15;
-        (_this$analyticsHelper15 = this.analyticsHelper) === null || _this$analyticsHelper15 === void 0 ? void 0 : _this$analyticsHelper15.sendErrorEvent(error, 'Error while setting metadata');
+        var _this$analyticsHelper12;
+        (_this$analyticsHelper12 = this.analyticsHelper) === null || _this$analyticsHelper12 === void 0 ? void 0 : _this$analyticsHelper12.sendErrorEvent(error, 'Error while setting metadata');
         throw new SetMetadataError('Error while setting metadata', error);
       }
     }

package/dist/esm/version-wrapper.js

@@ -1,5 +1,5 @@
 export var name = "@atlaskit/collab-provider";
-export var version = "8.8.1";
+export var version = "8.9.0";
 export var nextMajorVersion = function nextMajorVersion() {
   return [Number(version.split('.')[0]) + 1, 0, 0].join('.');
 };

package/dist/esm/version.json

@@ -1,5 +1,5 @@
 {
 	"name": "@atlaskit/collab-provider",
-	"version": "8.8.1",
+	"version": "8.9.0",
 	"sideEffects": false
 }

package/dist/types/channel.d.ts

@@ -34,7 +34,7 @@ export declare class Channel extends Emitter<ChannelEvent> {
      * @throws {NotInitializedError} Channel not initialized
      * @throws {NotConnectedError} Channel not connected
      */
-    broadcast: <K extends keyof ChannelEvent>(type: K, data: Omit<ChannelEvent[K], "timestamp">, callback?: Function | undefined) => void;
+    broadcast: <K extends keyof ChannelEvent>(type: K, data: Omit<ChannelEvent[K], "timestamp">, callback?: Function) => void;
     /**
      * Send metadata to the back-end service over the channel
      * @throws {NotInitializedError} Channel not initialized