@atlaskit/editor-plugin-collab-edit 31.0.2 → 31.0.4

package/CHANGELOG.md

@@ -1,5 +1,20 @@
 # @atlaskit/editor-plugin-collab-edit
 
+## 31.0.4
+
+### Patch Changes
+
+- [`ffa18bdae0ee3`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/ffa18bdae0ee3) -
+  Clean up feature gate `platform_editor_ai_collapse_streaming_steps`
+- Updated dependencies
+
+## 31.0.3
+
+### Patch Changes
+
+- [`4cdaf1f75d6ad`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/4cdaf1f75d6ad) -
+  Use const for stable local bindings
+
 ## 31.0.2
 
 ### Patch Changes

package/dist/cjs/collabEditPlugin.js

@@ -216,7 +216,7 @@ var collabEditPlugin = exports.collabEditPlugin = function collabEditPlugin(_ref
         // Analytics steps are filtered first so they cannot break up an otherwise adjacent
         // run of streaming frames.
         var filtered = (0, _filterAnalytics.filterAnalyticsSteps)(steps);
-        var transformed = (0, _fg.fg)('platform_editor_ai_collapse_streaming_steps') ? (0, _collapseStreamingSteps.collapseStreamingSteps)(filtered) : filtered;
+        var transformed = (0, _collapseStreamingSteps.collapseStreamingSteps)(filtered);
         return (0, _mergeUnconfirmed.mergeUnconfirmedSteps)(transformed, api);
       };
       var plugins = [].concat((0, _toConsumableArray2.default)(useNativePlugin ? [{
@@ -279,15 +279,9 @@ var collabEditPlugin = exports.collabEditPlugin = function collabEditPlugin(_ref
       // `props.transactions` is what `applyTransaction` produced: the dispatched transaction
       // followed by everything `appendTransaction` added. Streaming frames are built in an
       // appendTransaction, so their hold metadata is never on `originalTransaction`.
-      // Evaluated once and used for both the observation and the send decision. If the gate
-      // is turned off mid-stream, an already-active hold must stop applying immediately
-      // rather than persisting until the backstop expires and blocking every send.
-      var collapseStreamingStepsEnabled = (0, _fg.fg)('platform_editor_ai_collapse_streaming_steps');
-      if (collapseStreamingStepsEnabled) {
-        props.transactions.forEach(function (transaction) {
-          return sendHold.observe(transaction);
-        });
-      }
+      props.transactions.forEach(function (transaction) {
+        return sendHold.observe(transaction);
+      });
       var addErrorAnalytics = (0, _analytics.addSynchronyErrorAnalytics)(props.newEditorState, props.newEditorState.tr, featureFlags, api === null || api === void 0 || (_api$analytics2 = api.analytics) === null || _api$analytics2 === void 0 ? void 0 : _api$analytics2.actions);
       var isEmptyDoc = (0, _utils.isEmptyDocument)(props.newEditorState.doc);
       var viewMode = api === null || api === void 0 || (_api$editorViewMode = api.editorViewMode) === null || _api$editorViewMode === void 0 || (_api$editorViewMode = _api$editorViewMode.sharedState.currentState()) === null || _api$editorViewMode === void 0 ? void 0 : _api$editorViewMode.mode;
@@ -299,7 +293,7 @@ var collabEditPlugin = exports.collabEditPlugin = function collabEditPlugin(_ref
         useNativePlugin: (_options$useNativePlu = options.useNativePlugin) !== null && _options$useNativePlu !== void 0 ? _options$useNativePlu : false,
         hideTelecursorOnLoad: !isEmptyDoc && ((_options$hideTelecurs = options.hideTelecursorOnLoad) !== null && _options$hideTelecurs !== void 0 ? _options$hideTelecurs : false),
         viewMode: viewMode,
-        isSendHeld: collapseStreamingStepsEnabled && sendHold.isHeld()
+        isSendHeld: sendHold.isHeld()
       }), addErrorAnalytics);
       if ((0, _fg.fg)('platform_editor_collab_organic_change_reporting')) {
         (0, _monitorOrganicChanges.monitorOrganic)(_objectSpread(_objectSpread({

package/dist/cjs/pm-plugins/main/plugin-state.js

@@ -87,11 +87,9 @@ var PluginState = exports.PluginState = /*#__PURE__*/function () {
     key: "apply",
     value: function apply(tr) {
       var _this = this;
-      // Ignored via go/ees005
-      // eslint-disable-next-line prefer-const
       var participants = this.participants,
-        sid = this.sid,
-        isReady = this.isReady;
+        sid = this.sid;
+      var isReady = this.isReady;
       var presenceData = tr.getMeta('presence');
       var telepointerData = tr.getMeta('telepointer');
       var nudgeTelepointerData = tr.getMeta('nudgeTelepointer');

package/dist/es2019/collabEditPlugin.js

@@ -173,7 +173,7 @@ export const collabEditPlugin = ({
         // Analytics steps are filtered first so they cannot break up an otherwise adjacent
         // run of streaming frames.
         const filtered = filterAnalyticsSteps(steps);
-        const transformed = fg('platform_editor_ai_collapse_streaming_steps') ? collapseStreamingSteps(filtered) : filtered;
+        const transformed = collapseStreamingSteps(filtered);
         return mergeUnconfirmedSteps(transformed, api);
       };
       const plugins = [...(useNativePlugin ? [{
@@ -229,13 +229,7 @@ export const collabEditPlugin = ({
       // `props.transactions` is what `applyTransaction` produced: the dispatched transaction
       // followed by everything `appendTransaction` added. Streaming frames are built in an
       // appendTransaction, so their hold metadata is never on `originalTransaction`.
-      // Evaluated once and used for both the observation and the send decision. If the gate
-      // is turned off mid-stream, an already-active hold must stop applying immediately
-      // rather than persisting until the backstop expires and blocking every send.
-      const collapseStreamingStepsEnabled = fg('platform_editor_ai_collapse_streaming_steps');
-      if (collapseStreamingStepsEnabled) {
-        props.transactions.forEach(transaction => sendHold.observe(transaction));
-      }
+      props.transactions.forEach(transaction => sendHold.observe(transaction));
       const addErrorAnalytics = addSynchronyErrorAnalytics(props.newEditorState, props.newEditorState.tr, featureFlags, api === null || api === void 0 ? void 0 : (_api$analytics2 = api.analytics) === null || _api$analytics2 === void 0 ? void 0 : _api$analytics2.actions);
       const isEmptyDoc = isEmptyDocument(props.newEditorState.doc);
       const viewMode = api === null || api === void 0 ? void 0 : (_api$editorViewMode = api.editorViewMode) === null || _api$editorViewMode === void 0 ? void 0 : (_api$editorViewMode$s = _api$editorViewMode.sharedState.currentState()) === null || _api$editorViewMode$s === void 0 ? void 0 : _api$editorViewMode$s.mode;
@@ -247,7 +241,7 @@ export const collabEditPlugin = ({
         useNativePlugin: (_options$useNativePlu = options.useNativePlugin) !== null && _options$useNativePlu !== void 0 ? _options$useNativePlu : false,
         hideTelecursorOnLoad: !isEmptyDoc && ((_options$hideTelecurs = options.hideTelecursorOnLoad) !== null && _options$hideTelecurs !== void 0 ? _options$hideTelecurs : false),
         viewMode,
-        isSendHeld: collapseStreamingStepsEnabled && sendHold.isHeld()
+        isSendHeld: sendHold.isHeld()
       }), addErrorAnalytics);
       if (fg('platform_editor_collab_organic_change_reporting')) {
         monitorOrganic({

package/dist/es2019/pm-plugins/main/plugin-state.js

@@ -57,11 +57,11 @@ export class PluginState {
     return (_participant$presence = participant === null || participant === void 0 ? void 0 : participant.presenceId) !== null && _participant$presence !== void 0 ? _participant$presence : sessionId;
   }
   apply(tr) {
-    // Ignored via go/ees005
-    // eslint-disable-next-line prefer-const
     let {
       participants,
-      sid,
+      sid
+    } = this;
+    const {
       isReady
     } = this;
     const presenceData = tr.getMeta('presence');

package/dist/esm/collabEditPlugin.js

@@ -209,7 +209,7 @@ export var collabEditPlugin = function collabEditPlugin(_ref4) {
         // Analytics steps are filtered first so they cannot break up an otherwise adjacent
         // run of streaming frames.
         var filtered = filterAnalyticsSteps(steps);
-        var transformed = fg('platform_editor_ai_collapse_streaming_steps') ? collapseStreamingSteps(filtered) : filtered;
+        var transformed = collapseStreamingSteps(filtered);
         return mergeUnconfirmedSteps(transformed, api);
       };
       var plugins = [].concat(_toConsumableArray(useNativePlugin ? [{
@@ -272,15 +272,9 @@ export var collabEditPlugin = function collabEditPlugin(_ref4) {
       // `props.transactions` is what `applyTransaction` produced: the dispatched transaction
       // followed by everything `appendTransaction` added. Streaming frames are built in an
       // appendTransaction, so their hold metadata is never on `originalTransaction`.
-      // Evaluated once and used for both the observation and the send decision. If the gate
-      // is turned off mid-stream, an already-active hold must stop applying immediately
-      // rather than persisting until the backstop expires and blocking every send.
-      var collapseStreamingStepsEnabled = fg('platform_editor_ai_collapse_streaming_steps');
-      if (collapseStreamingStepsEnabled) {
-        props.transactions.forEach(function (transaction) {
-          return sendHold.observe(transaction);
-        });
-      }
+      props.transactions.forEach(function (transaction) {
+        return sendHold.observe(transaction);
+      });
       var addErrorAnalytics = addSynchronyErrorAnalytics(props.newEditorState, props.newEditorState.tr, featureFlags, api === null || api === void 0 || (_api$analytics2 = api.analytics) === null || _api$analytics2 === void 0 ? void 0 : _api$analytics2.actions);
       var isEmptyDoc = isEmptyDocument(props.newEditorState.doc);
       var viewMode = api === null || api === void 0 || (_api$editorViewMode = api.editorViewMode) === null || _api$editorViewMode === void 0 || (_api$editorViewMode = _api$editorViewMode.sharedState.currentState()) === null || _api$editorViewMode === void 0 ? void 0 : _api$editorViewMode.mode;
@@ -292,7 +286,7 @@ export var collabEditPlugin = function collabEditPlugin(_ref4) {
         useNativePlugin: (_options$useNativePlu = options.useNativePlugin) !== null && _options$useNativePlu !== void 0 ? _options$useNativePlu : false,
         hideTelecursorOnLoad: !isEmptyDoc && ((_options$hideTelecurs = options.hideTelecursorOnLoad) !== null && _options$hideTelecurs !== void 0 ? _options$hideTelecurs : false),
         viewMode: viewMode,
-        isSendHeld: collapseStreamingStepsEnabled && sendHold.isHeld()
+        isSendHeld: sendHold.isHeld()
       }), addErrorAnalytics);
       if (fg('platform_editor_collab_organic_change_reporting')) {
         monitorOrganic(_objectSpread(_objectSpread({

package/dist/esm/pm-plugins/main/plugin-state.js

@@ -81,11 +81,9 @@ export var PluginState = /*#__PURE__*/function () {
     key: "apply",
     value: function apply(tr) {
       var _this = this;
-      // Ignored via go/ees005
-      // eslint-disable-next-line prefer-const
       var participants = this.participants,
-        sid = this.sid,
-        isReady = this.isReady;
+        sid = this.sid;
+      var isReady = this.isReady;
       var presenceData = tr.getMeta('presence');
       var telepointerData = tr.getMeta('telepointer');
       var nudgeTelepointerData = tr.getMeta('nudgeTelepointer');

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-collab-edit",
-	"version": "31.0.2",
+	"version": "31.0.4",
 	"description": "Collab Edit plugin for @atlaskit/editor-core",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -37,20 +37,15 @@
 		"memoize-one": "^6.0.0"
 	},
 	"peerDependencies": {
-		"@atlaskit/editor-common": "^134.1.0",
+		"@atlaskit/editor-common": "^134.2.0",
 		"react": "^18.2.0 || ^19.2.0",
 		"react-dom": "^18.2.0 || ^19.2.0"
 	},
 	"devDependencies": {
-		"@testing-library/react": "^16.3.0",
 		"react": "^19.2.0",
 		"react-dom": "^19.2.0"
 	},
 	"platform-feature-flags": {
-		"platform_editor_ai_collapse_streaming_steps": {
-			"type": "boolean",
-			"referenceOnly": true
-		},
 		"confluence_ncs_step_diffing_version_history": {
 			"type": "boolean",
 			"referenceOnly": true