@atlaskit/editor-plugin-synced-block 8.3.2 → 8.3.4

package/dist/cjs/ui/DeleteConfirmationModal.js

@@ -20,36 +20,12 @@ var _hooks = require("@atlaskit/editor-common/hooks");
 var _messages = require("@atlaskit/editor-common/messages");
 var _editorPluginConnectivity = require("@atlaskit/editor-plugin-connectivity");
 var _modalDialog = _interopRequireWildcard(require("@atlaskit/modal-dialog"));
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _compiled = require("@atlaskit/primitives/compiled");
 var _spinner = _interopRequireDefault(require("@atlaskit/spinner"));
 var _main = require("../pm-plugins/main");
 function _interopRequireWildcard(e, t) { if ("function" == typeof WeakMap) var r = new WeakMap(), n = new WeakMap(); return (_interopRequireWildcard = function _interopRequireWildcard(e, t) { if (!t && e && e.__esModule) return e; var o, i, f = { __proto__: null, default: e }; if (null === e || "object" != _typeof(e) && "function" != typeof e) return f; if (o = t ? n : r) { if (o.has(e)) return o.get(e); o.set(e, f); } for (var _t in e) "default" !== _t && {}.hasOwnProperty.call(e, _t) && ((i = (o = Object.defineProperty) && Object.getOwnPropertyDescriptor(e, _t)) && (i.get || i.set) ? o(f, _t, i) : f[_t] = e[_t]); return f; })(e, t); }
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
-var modalContentMapOld = {
-  'source-block-deleted': {
-    titleMultiple: _messages.syncBlockMessages.deleteConfirmationModalTitleMultiple,
-    titleSingle: _messages.syncBlockMessages.deletionConfirmationModalTitleSingle,
-    descriptionSingle: _messages.syncBlockMessages.deletionConfirmationModalDescriptionNoRef,
-    descriptionMultiple: _messages.syncBlockMessages.deletionConfirmationModalDescription,
-    confirmButtonLabel: _messages.syncBlockMessages.deleteConfirmationModalDeleteButton
-  },
-  'source-block-unpublished': {
-    titleMultiple: _messages.syncBlockMessages.deleteConfirmationModalTitleMultiple,
-    titleSingle: _messages.syncBlockMessages.deletionConfirmationModalTitleSingle,
-    descriptionSingle: _messages.syncBlockMessages.deletionConfirmationModalDescriptionNoRef,
-    descriptionMultiple: _messages.syncBlockMessages.deletionConfirmationModalDescription,
-    confirmButtonLabel: _messages.syncBlockMessages.deleteConfirmationModalDeleteButton
-  },
-  'source-block-unsynced': {
-    titleMultiple: _messages.syncBlockMessages.unsyncConfirmationModalTitle,
-    titleSingle: _messages.syncBlockMessages.unsyncConfirmationModalTitle,
-    descriptionSingle: _messages.syncBlockMessages.unsyncConfirmModalDescriptionSingle,
-    descriptionMultiple: _messages.syncBlockMessages.unsyncConfirmModalDescriptionMultiple,
-    confirmButtonLabel: _messages.syncBlockMessages.deleteConfirmationModalUnsyncButton
-  }
-};
 var modalContentMap = {
   'source-block-deleted': {
     titleMultiple: _messages.syncBlockMessages.deleteConfirmationModalTitleMultiple,
@@ -111,10 +87,9 @@ var DeleteConfirmationModal = exports.DeleteConfirmationModal = function DeleteC
     formatMessage = _useIntl.formatMessage;
   var resolverRef = _react.default.useRef(undefined);
 
-  // When platform_synced_block_patch_9 is on and a source block with no references is deleted,
-  // the modal is never shown but onDeleteCompleted still sets bodiedSyncBlockDeletionStatus to
-  // 'completed'. This ref signals the useEffect to silently reset the status without trying to
-  // close the modal (which was never open).
+  // When a source block with no references is deleted, the modal is never shown but
+  // onDeleteCompleted still sets bodiedSyncBlockDeletionStatus to 'completed'. This ref signals
+  // the useEffect to silently reset the status without trying to close the modal (which was never open).
   var skipModalOnCompletedRef = _react.default.useRef(false);
   var handleClick = (0, _react.useCallback)(function (confirm) {
     return function () {
@@ -206,24 +181,20 @@ var DeleteConfirmationModal = exports.DeleteConfirmationModal = function DeleteC
       return _regenerator.default.wrap(function _callee3$(_context3) {
         while (1) switch (_context3.prev = _context3.next) {
           case 0:
-            if (!(0, _platformFeatureFlags.fg)('platform_synced_block_patch_9')) {
-              _context3.next = 15;
-              break;
-            }
-            _context3.prev = 1;
-            _context3.next = 4;
+            _context3.prev = 0;
+            _context3.next = 3;
             return fetchReferenceCountRef.current(syncBlockIds);
-          case 4:
+          case 3:
             count = _context3.sent;
-            _context3.next = 10;
+            _context3.next = 9;
             break;
-          case 7:
-            _context3.prev = 7;
-            _context3.t0 = _context3["catch"](1);
+          case 6:
+            _context3.prev = 6;
+            _context3.t0 = _context3["catch"](0);
             count = 1;
-          case 10:
+          case 9:
             if (!(count === 0)) {
-              _context3.next = 14;
+              _context3.next = 13;
               break;
             }
             // No references — auto-confirm without showing the modal.
@@ -240,9 +211,8 @@ var DeleteConfirmationModal = exports.DeleteConfirmationModal = function DeleteC
               } : {}));
             });
             return _context3.abrupt("return", true);
-          case 14:
+          case 13:
             setReferenceCount(count);
-          case 15:
             setIsOpen(true);
             setSyncBlockIds(syncBlockIds);
             if (deleteReason) {
@@ -261,11 +231,11 @@ var DeleteConfirmationModal = exports.DeleteConfirmationModal = function DeleteC
               });
             }
             return _context3.abrupt("return", confirmedPromise);
-          case 21:
+          case 20:
           case "end":
             return _context3.stop();
         }
-      }, _callee3, null, [[1, 7]]);
+      }, _callee3, null, [[0, 6]]);
     }));
     return function (_x3, _x4) {
       return _ref5.apply(this, arguments);
@@ -282,7 +252,7 @@ var DeleteConfirmationModal = exports.DeleteConfirmationModal = function DeleteC
     };
   }, [syncBlockStoreManager, confirmationCallback]);
   (0, _react.useEffect)(function () {
-    if (skipModalOnCompletedRef.current && (0, _platformFeatureFlags.fg)('platform_synced_block_patch_9')) {
+    if (skipModalOnCompletedRef.current) {
       if (bodiedSyncBlockDeletionStatus === 'completed') {
         var _api$core6;
         // Deletion was auto-confirmed without showing the modal (no references).
@@ -317,44 +287,10 @@ var DeleteConfirmationModal = exports.DeleteConfirmationModal = function DeleteC
       });
     }
   }, [api === null || api === void 0 || (_api$core8 = api.core) === null || _api$core8 === void 0 ? void 0 : _api$core8.actions, bodiedSyncBlockDeletionStatus, isOpen]);
-  (0, _react.useEffect)(function () {
-    // When the flag is off, fetch references after the modal opens (original behaviour).
-    // When the flag is on, references are fetched and set before the modal opens in
-    // confirmationCallback, so this useEffect is skipped to avoid a duplicate fetch.
-    if (isOpen && syncBlockIds !== undefined && !(0, _platformFeatureFlags.fg)('platform_synced_block_patch_9')) {
-      var fetchReferences = /*#__PURE__*/function () {
-        var _ref0 = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee4() {
-          var totalCount;
-          return _regenerator.default.wrap(function _callee4$(_context4) {
-            while (1) switch (_context4.prev = _context4.next) {
-              case 0:
-                _context4.prev = 0;
-                _context4.next = 3;
-                return fetchReferenceCountRef.current(syncBlockIds);
-              case 3:
-                totalCount = _context4.sent;
-                setReferenceCount(totalCount);
-                _context4.next = 10;
-                break;
-              case 7:
-                _context4.prev = 7;
-                _context4.t0 = _context4["catch"](0);
-                setReferenceCount(0);
-              case 10:
-              case "end":
-                return _context4.stop();
-            }
-          }, _callee4, null, [[0, 7]]);
-        }));
-        return function fetchReferences() {
-          return _ref0.apply(this, arguments);
-        };
-      }();
 
-      // eslint-disable-next-line @atlassian/perf-linting/no-chain-state-updates -- Ignored via go/ees017 (to be fixed)
-      fetchReferences();
-    }
-  }, [isOpen, syncBlockIds]);
+  // References are fetched and set before the modal opens in
+  // confirmationCallback, so no additional fetch is needed here.
+
   return /*#__PURE__*/_react.default.createElement(_modalDialog.ModalTransition, null, isOpen && /*#__PURE__*/_react.default.createElement(_modalDialog.default, {
     onClose: handleClick(false),
     testId: "sync-block-delete-confirmation",
@@ -364,7 +300,7 @@ var DeleteConfirmationModal = exports.DeleteConfirmationModal = function DeleteC
   }, /*#__PURE__*/_react.default.createElement(_spinner.default, {
     size: "large"
   })) : /*#__PURE__*/_react.default.createElement(ModalContent, {
-    content: (0, _platformFeatureFlags.fg)('platform_synced_block_patch_8') ? modalContentMap[deleteReason] : modalContentMapOld[deleteReason],
+    content: modalContentMap[deleteReason],
     referenceCount: referenceCount,
     handleClick: handleClick,
     formatMessage: formatMessage,
@@ -374,15 +310,15 @@ var DeleteConfirmationModal = exports.DeleteConfirmationModal = function DeleteC
     sourceCount: (syncBlockIds === null || syncBlockIds === void 0 ? void 0 : syncBlockIds.length) || 0
   }))));
 };
-var ModalContent = function ModalContent(_ref1) {
-  var content = _ref1.content,
-    referenceCount = _ref1.referenceCount,
-    handleClick = _ref1.handleClick,
-    formatMessage = _ref1.formatMessage,
-    isDeleting = _ref1.isDeleting,
-    isDisabled = _ref1.isDisabled,
-    deleteReason = _ref1.deleteReason,
-    sourceCount = _ref1.sourceCount;
+var ModalContent = function ModalContent(_ref0) {
+  var content = _ref0.content,
+    referenceCount = _ref0.referenceCount,
+    handleClick = _ref0.handleClick,
+    formatMessage = _ref0.formatMessage,
+    isDeleting = _ref0.isDeleting,
+    isDisabled = _ref0.isDisabled,
+    deleteReason = _ref0.deleteReason,
+    sourceCount = _ref0.sourceCount;
   var titleMultiple = content.titleMultiple,
     titleSingle = content.titleSingle,
     descriptionSingle = content.descriptionSingle,

package/dist/cjs/ui/SyncBlockLabel.js

@@ -13,7 +13,6 @@ var _messages = require("@atlaskit/editor-common/messages");
 var _syncBlock = require("@atlaskit/editor-common/sync-block");
 var _blockSynced = _interopRequireDefault(require("@atlaskit/icon-lab/core/block-synced"));
 var _compiled = require("@atlaskit/primitives/compiled");
-var _experiments = require("@atlaskit/tmp-editor-statsig/experiments");
 var _tooltip = _interopRequireDefault(require("@atlaskit/tooltip"));
 var _visuallyHidden = _interopRequireDefault(require("@atlaskit/visually-hidden"));
 var _time = require("./utils/time");
@@ -80,9 +79,7 @@ var SyncBlockLabelComponent = function SyncBlockLabelComponent(_ref) {
     // eslint-disable-next-line @atlaskit/ui-styling-standard/no-classname-prop
     ,
     className: _syncBlock.SyncBlockLabelSharedCssClassName.labelClassName,
-    "aria-describedby": (isSource || isUnsyncedBlock) && (0, _experiments.editorExperiment)('platform_synced_block_patch_6', true, {
-      exposure: true
-    }) ? undefined : ariaDescribedById
+    "aria-describedby": isSource || isUnsyncedBlock ? undefined : ariaDescribedById
   }, /*#__PURE__*/_react.default.createElement(_blockSynced.default, {
     color: "var(--ds-icon-subtle, #505258)",
     size: "small",

package/dist/cjs/ui/SyncedLocationDropdown.js

@@ -278,7 +278,7 @@ var processReferenceData = exports.processReferenceData = function processRefere
       var references = _step.value;
       if (references.length > 1) {
         references.forEach(function (reference, index) {
-          return reference.title = "".concat(reference.title === '' && reference.hasAccess && (0, _platformFeatureFlags.fg)('platform_synced_block_patch_8') ? formatMessage(_messages.syncBlockMessages.syncedLocationDropdownUntitledPage) : reference.title, ": ").concat(formatMessage(_messages.syncBlockMessages.syncedLocationDropdownTitleBlockIndex, {
+          return reference.title = "".concat(reference.title === '' && reference.hasAccess ? formatMessage(_messages.syncBlockMessages.syncedLocationDropdownUntitledPage) : reference.title, ": ").concat(formatMessage(_messages.syncBlockMessages.syncedLocationDropdownTitleBlockIndex, {
             index: index + 1
           }));
         });
@@ -428,7 +428,7 @@ var DropdownContent = function DropdownContent(_ref7) {
               count: "".concat(referenceData.length > 99 ? '99+' : referenceData.length)
             })
           }, referenceData.map(function (reference) {
-            var title = reference.title === '' && reference.hasAccess && (0, _platformFeatureFlags.fg)('platform_synced_block_patch_8') ? formatMessage(_messages.syncBlockMessages.syncedLocationDropdownUntitledPage) : reference.title || reference.url || '';
+            var title = reference.title === '' && reference.hasAccess ? formatMessage(_messages.syncBlockMessages.syncedLocationDropdownUntitledPage) : reference.title || reference.url || '';
             return /*#__PURE__*/React.createElement("div", {
               key: reference.title,
               className: (0, _runtime.ax)(["_2ll012x7"])

package/dist/cjs/ui/floating-toolbar.js

@@ -18,7 +18,6 @@ var _copy = _interopRequireDefault(require("@atlaskit/icon/core/copy"));
 var _delete = _interopRequireDefault(require("@atlaskit/icon/core/delete"));
 var _edit = _interopRequireDefault(require("@atlaskit/icon/core/edit"));
 var _linkBroken = _interopRequireDefault(require("@atlaskit/icon/core/link-broken"));
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _editorCommands = require("../editor-commands");
 var _utils2 = require("../pm-plugins/utils/utils");
 var _types = require("../types");
@@ -87,7 +86,7 @@ var getToolbarConfig = exports.getToolbarConfig = function getToolbarConfig(stat
           var _syncBlockInstance$da;
           return /*#__PURE__*/_react.default.createElement(_ui.FloatingToolbarButton, {
             areAnyNewToolbarFlagsEnabled: true,
-            disabled: (0, _platformFeatureFlags.fg)('platform_synced_block_patch_8') ? (syncBlockInstance === null || syncBlockInstance === void 0 || (_syncBlockInstance$da = syncBlockInstance.data) === null || _syncBlockInstance$da === void 0 ? void 0 : _syncBlockInstance$da.status) === 'unpublished' : false,
+            disabled: (syncBlockInstance === null || syncBlockInstance === void 0 || (_syncBlockInstance$da = syncBlockInstance.data) === null || _syncBlockInstance$da === void 0 ? void 0 : _syncBlockInstance$da.status) === 'unpublished',
             icon: /*#__PURE__*/_react.default.createElement(_linkBroken.default, {
               label: ""
             }),

package/dist/cjs/ui/quick-insert.js

@@ -10,7 +10,6 @@ exports.getQuickInsertConfig = void 0;
 require("./quick-insert.compiled.css");
 var React = _interopRequireWildcard(require("react"));
 var _runtime = require("@compiled/react/runtime");
-var _toConsumableArray2 = _interopRequireDefault(require("@babel/runtime/helpers/toConsumableArray"));
 var _messages = require("@atlaskit/editor-common/messages");
 var _quickInsert = require("@atlaskit/editor-common/quick-insert");
 var _lozenge = _interopRequireDefault(require("@atlaskit/lozenge"));
@@ -29,8 +28,8 @@ var getQuickInsertConfig = exports.getQuickInsertConfig = function getQuickInser
       id: 'syncBlock',
       title: formatMessage(_messages.blockTypeMessages.syncedBlock),
       description: formatMessage(_messages.blockTypeMessages.syncedBlockDescription),
-      priority: (0, _platformFeatureFlags.fg)('platform_synced_block_patch_10') ? 400 : 800,
-      keywords: ['synced', 'block', 'synced-block', 'sync', 'sync-block', 'auto', 'update', 'excerpt', 'connect'].concat((0, _toConsumableArray2.default)((0, _platformFeatureFlags.fg)('platform_synced_block_patch_10') ? ['create'] : [])),
+      priority: 400,
+      keywords: ['synced', 'block', 'synced-block', 'sync', 'sync-block', 'auto', 'update', 'excerpt', 'connect', 'create'],
       isDisabledOffline: true,
       keyshortcut: '',
       lozenge: /*#__PURE__*/React.createElement("span", {

package/dist/es2019/nodeviews/bodiedSyncedBlock.js

@@ -5,7 +5,6 @@ import ReactNodeView from '@atlaskit/editor-common/react-node-view';
 import { BodiedSyncBlockSharedCssClassName } from '@atlaskit/editor-common/sync-block';
 import { isOfflineMode } from '@atlaskit/editor-plugin-connectivity';
 import { DOMSerializer } from '@atlaskit/editor-prosemirror/model';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { BodiedSyncBlockWrapper } from '../ui/BodiedSyncBlockWrapper';
 import { SyncBlockLabel } from '../ui/SyncBlockLabel';
 const toDOMOld = () => ['div', {
@@ -138,12 +137,11 @@ const toDOM = node => ['div', {
   contenteditable: 'true'
 }, 0]];
 export class BodiedSyncBlock {
-  constructor(node, view, getPos, api, nodeViewPortalProviderAPI, syncBlockStore) {
+  constructor(node, view, getPos, api, nodeViewPortalProviderAPI) {
     this.node = node;
     this.view = view;
     this.getPos = getPos;
     this.api = api;
-    this.syncBlockStore = syncBlockStore;
     this.nodeViewPortalProviderAPI = nodeViewPortalProviderAPI;
     const {
       dom,
@@ -170,12 +168,8 @@ export class BodiedSyncBlock {
     this.handleConnectivityModeChange();
     this.handleViewModeChange();
 
-    // update sync block data on initial creation
-    // When fg is ON, cache is populated in state.init() and updated in appendTransaction
-    if (!fg('platform_synced_block_update_refactor')) {
-      var _this$syncedBlockStor;
-      (_this$syncedBlockStor = this.syncedBlockStore) === null || _this$syncedBlockStor === void 0 ? void 0 : _this$syncedBlockStor.sourceManager.updateSyncBlockData(node, false);
-    }
+    // Cache is populated in state.init() and updated in appendTransaction,
+    // so no additional updateSyncBlockData call is needed here.
   }
   updateContentEditable({
     nextConnectivityMode,
@@ -213,22 +207,13 @@ export class BodiedSyncBlock {
       });
     }
   }
-  get syncedBlockStore() {
-    var _this$api$syncedBlock2, _this$api10, _this$api10$syncedBlo, _this$api10$syncedBlo2;
-    return (_this$api$syncedBlock2 = (_this$api10 = this.api) === null || _this$api10 === void 0 ? void 0 : (_this$api10$syncedBlo = _this$api10.syncedBlock.sharedState) === null || _this$api10$syncedBlo === void 0 ? void 0 : (_this$api10$syncedBlo2 = _this$api10$syncedBlo.currentState()) === null || _this$api10$syncedBlo2 === void 0 ? void 0 : _this$api10$syncedBlo2.syncBlockStore) !== null && _this$api$syncedBlock2 !== void 0 ? _this$api$syncedBlock2 : this.syncBlockStore;
-  }
   update(node) {
     if (this.node.type !== node.type) {
       return false;
     }
-    if (node !== this.node) {
-      // When fg is ON, cache updates are handled in appendTransaction where we can
-      // filter out non-user changes (remote collab, table auto-scale, etc.)
-      if (!fg('platform_synced_block_update_refactor')) {
-        var _this$syncedBlockStor2;
-        (_this$syncedBlockStor2 = this.syncedBlockStore) === null || _this$syncedBlockStor2 === void 0 ? void 0 : _this$syncedBlockStor2.sourceManager.updateSyncBlockData(node, false);
-      }
-    }
+
+    // Cache updates are handled in appendTransaction where we can
+    // filter out non-user changes (remote collab, table auto-scale, etc.)
     this.node = node;
     return true;
   }
@@ -248,12 +233,11 @@ export class BodiedSyncBlock {
 export const bodiedSyncBlockNodeView = props => {
   const {
     api,
-    syncBlockStore,
     pmPluginFactoryParams: {
       nodeViewPortalProviderAPI
     }
   } = props;
   return (node, view, getPos) => {
-    return new BodiedSyncBlock(node, view, getPos, api, nodeViewPortalProviderAPI, syncBlockStore);
+    return new BodiedSyncBlock(node, view, getPos, api, nodeViewPortalProviderAPI);
   };
 };

package/dist/es2019/nodeviews/syncedBlock.js

@@ -5,7 +5,6 @@ import { ErrorBoundary } from '@atlaskit/editor-common/error-boundary';
 import ReactNodeView from '@atlaskit/editor-common/react-node-view';
 import { SyncBlockSharedCssClassName, SyncBlockActionsProvider } from '@atlaskit/editor-common/sync-block';
 import { expValEqualsNoExposure } from '@atlaskit/tmp-editor-statsig/exp-val-equals-no-exposure';
-import { editorExperiment } from '@atlaskit/tmp-editor-statsig/experiments';
 import { removeSyncedBlockAtPos } from '../editor-commands';
 import { SyncBlockRendererWrapper } from '../ui/SyncBlockRendererWrapper';
 export class SyncBlock extends ReactNodeView {
@@ -42,9 +41,7 @@ export class SyncBlock extends ReactNodeView {
     return currentNode.attrs.localId === newNode.attrs.localId && currentNode.attrs.resourceId === newNode.attrs.resourceId;
   }
   update(node, decorations, innerDecorations) {
-    return super.update(node, decorations, innerDecorations, editorExperiment('platform_synced_block_patch_6', true, {
-      exposure: true
-    }) ? this.validUpdate : undefined);
+    return super.update(node, decorations, innerDecorations, this.validUpdate);
   }
   render({
     getPos

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

@@ -337,18 +337,14 @@ export const createPlugin = (options, pmPluginFactoryParams, syncBlockStore, api
 
           // Populate source sync block cache from initial document
           // When fg is ON, this replaces the constructor call in the nodeview
-          if (fg('platform_synced_block_update_refactor')) {
-            instance.doc.forEach(node => {
-              if (syncBlockStore.sourceManager.isSourceBlock(node)) {
-                syncBlockStore.sourceManager.updateSyncBlockData(node, false);
-              }
-            });
-
-            // Fetch statuses from the backend so we can identify unpublished blocks on cancel
-            if (fg('platform_synced_block_patch_10')) {
-              syncBlockStore.sourceManager.fetchAndCacheStatuses();
+          instance.doc.forEach(node => {
+            if (syncBlockStore.sourceManager.isSourceBlock(node)) {
+              syncBlockStore.sourceManager.updateSyncBlockData(node, false);
             }
-          }
+          });
+
+          // Fetch statuses from the backend so we can identify unpublished blocks on cancel
+          syncBlockStore.sourceManager.fetchAndCacheStatuses();
         }
 
         // Read initial shared-state signals for status decorations
@@ -666,7 +662,7 @@ export const createPlugin = (options, pmPluginFactoryParams, syncBlockStore, api
         }
       }
       const viewMode = api === null || api === void 0 ? void 0 : (_api$editorViewMode4 = api.editorViewMode) === null || _api$editorViewMode4 === void 0 ? void 0 : (_api$editorViewMode4$ = _api$editorViewMode4.sharedState.currentState()) === null || _api$editorViewMode4$ === void 0 ? void 0 : _api$editorViewMode4$.mode;
-      if (viewMode === 'view' && fg('platform_synced_block_patch_8')) {
+      if (viewMode === 'view') {
         return true;
       }
       const isOffline = isOfflineMode(api === null || api === void 0 ? void 0 : (_api$connectivity5 = api.connectivity) === null || _api$connectivity5 === void 0 ? void 0 : (_api$connectivity5$sh = _api$connectivity5.sharedState.currentState()) === null || _api$connectivity5$sh === void 0 ? void 0 : _api$connectivity5$sh.mode);
@@ -684,31 +680,30 @@ export const createPlugin = (options, pmPluginFactoryParams, syncBlockStore, api
           }
         });
       }
-      if (fg('platform_synced_block_update_refactor')) {
-        // if doc changed and it's a remote transaction, check if any synced block were added,
-        // and if so, for source synced blocks, ensure we update the cache with them
-        // and for reference synced blocks, ensure we fetch the data from the server
-        if (tr.docChanged && tr.getMeta('isRemote')) {
-          const {
-            added
-          } = trackSyncBlocks(node => syncBlockStore.isSyncBlock(node), tr, state);
-          const sourceSyncBlockNodes = added.filter(nodeInfo => nodeInfo.node && syncBlockStore.sourceManager.isSourceBlock(nodeInfo.node));
-          const referenceSyncBlockNodes = added.filter(nodeInfo => nodeInfo.node && syncBlockStore.referenceManager.isReferenceBlock(nodeInfo.node));
-          sourceSyncBlockNodes.forEach(nodeInfo => {
-            var _nodeInfo$attrs2;
-            if ((_nodeInfo$attrs2 = nodeInfo.attrs) !== null && _nodeInfo$attrs2 !== void 0 && _nodeInfo$attrs2.resourceId && nodeInfo.node) {
-              syncBlockStore.sourceManager.updateSyncBlockData(nodeInfo.node, tr.getMeta('isRemote'));
-            }
-          });
 
-          // Fetch statuses for remotely-added source sync blocks
-          // so we can identify unpublished blocks on cancel
-          if (sourceSyncBlockNodes.length > 0 && fg('platform_synced_block_patch_10')) {
-            syncBlockStore.sourceManager.fetchAndCacheStatuses();
+      // if doc changed and it's a remote transaction, check if any synced block were added,
+      // and if so, for source synced blocks, ensure we update the cache with them
+      // and for reference synced blocks, ensure we fetch the data from the server
+      if (tr.docChanged && tr.getMeta('isRemote')) {
+        const {
+          added
+        } = trackSyncBlocks(node => syncBlockStore.isSyncBlock(node), tr, state);
+        const sourceSyncBlockNodes = added.filter(nodeInfo => nodeInfo.node && syncBlockStore.sourceManager.isSourceBlock(nodeInfo.node));
+        const referenceSyncBlockNodes = added.filter(nodeInfo => nodeInfo.node && syncBlockStore.referenceManager.isReferenceBlock(nodeInfo.node));
+        sourceSyncBlockNodes.forEach(nodeInfo => {
+          var _nodeInfo$attrs2;
+          if ((_nodeInfo$attrs2 = nodeInfo.attrs) !== null && _nodeInfo$attrs2 !== void 0 && _nodeInfo$attrs2.resourceId && nodeInfo.node) {
+            syncBlockStore.sourceManager.updateSyncBlockData(nodeInfo.node, tr.getMeta('isRemote'));
           }
-          const syncBlockNodes = referenceSyncBlockNodes.map(nodeInfo => nodeInfo.node).filter(node => node !== undefined);
-          syncBlockStore.referenceManager.fetchSyncBlocksData(convertPMNodesToSyncBlockNodes(syncBlockNodes));
+        });
+
+        // Fetch statuses for remotely-added source sync blocks
+        // so we can identify unpublished blocks on cancel
+        if (sourceSyncBlockNodes.length > 0) {
+          syncBlockStore.sourceManager.fetchAndCacheStatuses();
         }
+        const syncBlockNodes = referenceSyncBlockNodes.map(nodeInfo => nodeInfo.node).filter(node => node !== undefined);
+        syncBlockStore.referenceManager.fetchSyncBlocksData(convertPMNodesToSyncBlockNodes(syncBlockNodes));
       }
       if (!tr.docChanged || Boolean(tr.getMeta('isRemote')) || !isOffline && isConfirmedSyncBlockDeletion) {
         return true;
@@ -751,22 +746,21 @@ export const createPlugin = (options, pmPluginFactoryParams, syncBlockStore, api
         }
       }
       const viewMode = api === null || api === void 0 ? void 0 : (_api$editorViewMode5 = api.editorViewMode) === null || _api$editorViewMode5 === void 0 ? void 0 : (_api$editorViewMode5$ = _api$editorViewMode5.sharedState.currentState()) === null || _api$editorViewMode5$ === void 0 ? void 0 : _api$editorViewMode5$.mode;
-      if (viewMode === 'view' && fg('platform_synced_block_patch_8')) {
+      if (viewMode === 'view') {
         return null;
       }
 
-      // Update source sync block cache for user-initiated changes only
-      // When fg is ON, cache updates are handled here instead of in the nodeview update()
-      if (fg('platform_synced_block_update_refactor')) {
-        const isUserChange = tr => tr.docChanged && !isDirtyTransaction(tr) && !tr.getMeta('isRemote');
-        const hasSourceBlockEdit = trs.some(tr => isUserChange(tr) && hasEditInSyncBlock(tr, oldState));
-        if (hasSourceBlockEdit) {
-          newState.doc.forEach(node => {
-            if (syncBlockStore.sourceManager.isSourceBlock(node)) {
-              syncBlockStore.sourceManager.updateSyncBlockData(node, false);
-            }
-          });
-        }
+      // Update source sync block cache for user-initiated changes only.
+      // Cache updates are handled here instead of in the nodeview update() so we
+      // can filter out non-user changes (remote collab, dirty programmatic txns).
+      const isUserChange = tr => tr.docChanged && !isDirtyTransaction(tr) && !tr.getMeta('isRemote');
+      const hasSourceBlockEdit = trs.some(tr => isUserChange(tr) && hasEditInSyncBlock(tr, oldState));
+      if (hasSourceBlockEdit) {
+        newState.doc.forEach(node => {
+          if (syncBlockStore.sourceManager.isSourceBlock(node)) {
+            syncBlockStore.sourceManager.updateSyncBlockData(node, false);
+          }
+        });
       }
       trs.filter(tr => tr.docChanged).forEach(tr => {
         if (confirmationTransactionRef.current) {
@@ -800,7 +794,7 @@ export const createPlugin = (options, pmPluginFactoryParams, syncBlockStore, api
       // first occurrence and delete subsequent duplicates entirely (including their
       // contents), since a document must not contain two source sync blocks with the
       // same resourceId.
-      if (trs.some(tr => tr.docChanged && !tr.getMeta('isRemote')) && fg('platform_synced_block_patch_8')) {
+      if (trs.some(tr => tr.docChanged && !tr.getMeta('isRemote'))) {
         // Quick check: only walk the full document when at least one
         // transaction inserted a source synced block. This avoids an
         // expensive descendants() traversal on every local edit.

package/dist/es2019/pm-plugins/menu-and-toolbar-experiences.js

@@ -3,7 +3,6 @@ import { ACTION, ACTION_SUBJECT_ID } from '@atlaskit/editor-common/analytics';
 import { Experience, EXPERIENCE_ID, ExperienceCheckDomMutation, ExperienceCheckTimeout, getNodeQuery, getPopupContainerFromEditorView, popupWithNestedElement, getSelectionAncestorDOM } from '@atlaskit/editor-common/experiences';
 import { SafePlugin } from '@atlaskit/editor-common/safe-plugin';
 import { PluginKey } from '@atlaskit/editor-prosemirror/state';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 import { SYNCED_BLOCK_BUTTON_TEST_ID } from '../types';
 import { syncedBlockPluginKey } from './main';
@@ -11,7 +10,6 @@ const TIMEOUT_DURATION = 30000;
 const pluginKey = new PluginKey('syncedBlockMenuAndToolbarExperience');
 const SYNCED_BLOCK_BUTTON_TEST_IDS = Object.values(SYNCED_BLOCK_BUTTON_TEST_ID);
 const syncedBlockButtonIds = new Set(SYNCED_BLOCK_BUTTON_TEST_IDS);
-let targetEl;
 export const getMenuAndToolbarExperiencesPlugin = ({
   refs,
   dispatchAnalyticsEvent
@@ -139,7 +137,7 @@ export const getMenuAndToolbarExperiencesPlugin = ({
           if (!typeaheadPopup || !(typeaheadPopup instanceof HTMLElement)) {
             return;
           }
-          const targetElement = fg('platform_synced_block_fix_experience_tracking') ? typeaheadPopup.querySelector('[role="option"][aria-selected="true"]') : typeaheadPopup.querySelector('[role="option"]');
+          const targetElement = typeaheadPopup.querySelector('[role="option"][aria-selected="true"]');
           if (!targetElement || !(targetElement instanceof HTMLElement)) {
             return;
           }
@@ -297,16 +295,6 @@ const handleButtonClick = ({
 const isEnterKey = key => {
   return key === 'Enter';
 };
-const getTarget = containerElement => {
-  if (!targetEl) {
-    const element = containerElement === null || containerElement === void 0 ? void 0 : containerElement.querySelector('.ProseMirror');
-    if (!element || !(element instanceof HTMLElement)) {
-      return null;
-    }
-    targetEl = element;
-  }
-  return targetEl;
-};
 const syncedBlockAddedToDomCheck = (refs, editorViewRef) => new ExperienceCheckDomMutation({
   onDomMutation: ({
     mutations
@@ -321,20 +309,20 @@ const syncedBlockAddedToDomCheck = (refs, editorViewRef) => new ExperienceCheckD
   observeConfig: () => {
     var _editorViewRef$curren2;
     return [{
-      target: fg('platform_synced_block_fix_experience_tracking') ? editorViewRef === null || editorViewRef === void 0 ? void 0 : (_editorViewRef$curren2 = editorViewRef.current) === null || _editorViewRef$curren2 === void 0 ? void 0 : _editorViewRef$curren2.dom : getTarget(refs.containerElement),
+      target: editorViewRef === null || editorViewRef === void 0 ? void 0 : (_editorViewRef$curren2 = editorViewRef.current) === null || _editorViewRef$curren2 === void 0 ? void 0 : _editorViewRef$curren2.dom,
       options: {
         childList: true
       }
     },
     // When wrapping a node with breakout mark with sync block, breakout dom is reused
     // hence we need to observe subtree to catch sync block mutation
-    ...(fg('platform_synced_block_fix_experience_tracking') ? [{
+    {
       target: getSelectionAncestorDOM(editorViewRef === null || editorViewRef === void 0 ? void 0 : editorViewRef.current),
       options: {
         childList: true,
         subtree: true
       }
-    }] : [])];
+    }];
   }
 });
 const isBodiedSyncBlockAddedInMutation = ({
@@ -358,17 +346,17 @@ const referenceSyncBlockRemovedFromDomCheck = (refs, editorViewRef) => new Exper
   observeConfig: () => {
     var _editorViewRef$curren3;
     return [{
-      target: fg('platform_synced_block_fix_experience_tracking') ? editorViewRef === null || editorViewRef === void 0 ? void 0 : (_editorViewRef$curren3 = editorViewRef.current) === null || _editorViewRef$curren3 === void 0 ? void 0 : _editorViewRef$curren3.dom : getTarget(refs.containerElement),
+      target: editorViewRef === null || editorViewRef === void 0 ? void 0 : (_editorViewRef$curren3 = editorViewRef.current) === null || _editorViewRef$curren3 === void 0 ? void 0 : _editorViewRef$curren3.dom,
       options: {
         childList: true
       }
-    }, ...(fg('platform_synced_block_fix_experience_tracking') ? [{
+    }, {
       target: getSelectionAncestorDOM(editorViewRef === null || editorViewRef === void 0 ? void 0 : editorViewRef.current),
       options: {
         childList: true,
         subtree: true
       }
-    }] : [])];
+    }];
   }
 });
 const isSyncBlockRemovedInMutation = ({

package/dist/es2019/pm-plugins/utils/utils.js

@@ -2,8 +2,6 @@ import { expandSelectionToBlockRange } from '@atlaskit/editor-common/selection';
 import { Fragment } from '@atlaskit/editor-prosemirror/model';
 import { ReplaceAroundStep, ReplaceStep } from '@atlaskit/editor-prosemirror/transform';
 import { findParentNodeOfType, findParentNodeOfTypeClosestToPos, findSelectedNodeOfType } from '@atlaskit/editor-prosemirror/utils';
-import { editorExperiment } from '@atlaskit/tmp-editor-statsig/experiments';
-
 /**
  * Defers a callback to the next microtask (when gated) or next macrotask via setTimeout(0).
  * Used to avoid re-entrant ProseMirror dispatch cycles.
@@ -96,9 +94,7 @@ const fragmentContainsExtension = fragment => {
     if (found) {
       return;
     }
-    if (editorExperiment('platform_synced_block_patch_6', true, {
-      exposure: true
-    }) ? EXTENSION_NODES.has(node.type.name) : node.type.name === 'inlineExtension') {
+    if (EXTENSION_NODES.has(node.type.name)) {
       found = true;
     } else if (node.content.size) {
       if (fragmentContainsExtension(node.content)) {
@@ -159,17 +155,13 @@ export const wasExtensionInsertedInBodiedSyncBlock = (tr, state) => {
     if (resourceId !== undefined) {
       return false;
     }
-    if (editorExperiment('platform_synced_block_patch_6', true, {
-      exposure: true
-    }) ? EXTENSION_NODES.has(node.type.name) : node.type.name === 'inlineExtension') {
+    if (EXTENSION_NODES.has(node.type.name)) {
       const $pos = tr.doc.resolve(pos);
       const parent = findParentNodeOfTypeClosestToPos($pos, bodiedSyncBlock);
       if (parent !== null && parent !== void 0 && parent.node.attrs.resourceId) {
         const mappedPos = tr.mapping.invert().map(pos);
         const nodeBefore = state.doc.nodeAt(mappedPos);
-        if (!nodeBefore || (editorExperiment('platform_synced_block_patch_6', true, {
-          exposure: true
-        }) ? EXTENSION_NODES.has(nodeBefore.type.name) : nodeBefore.type.name !== 'inlineExtension')) {
+        if (!nodeBefore || EXTENSION_NODES.has(nodeBefore.type.name)) {
           resourceId = parent.node.attrs.resourceId;
           return false;
         }