@atlaskit/editor-synced-block-provider 10.0.2 → 10.0.4

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # @atlaskit/editor-synced-block-provider
 
+## 10.0.4
+
+### Patch Changes
+
+- [`0b4753c0a8c07`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/0b4753c0a8c07) -
+  Clean up feature gates from completed rollout
+- Updated dependencies
+
+## 10.0.3
+
+### Patch Changes
+
+- Updated dependencies
+
 ## 10.0.2
 
 ### Patch Changes

package/dist/cjs/common/types.js

@@ -54,8 +54,7 @@ var SyncBlockError = exports.SyncBlockError = /*#__PURE__*/function (SyncBlockEr
  * asynchronously and `destroy()` nulls it on orphaned managers, so queued/
  * in-flight ops throw `Data provider not set` — previously mis-logged as a real
  * error. These let throw and catch sites agree on one non-string-matched signal
- * so the residual false errors are suppressed. Gated by
- * `platform_editor_blocks_patch_3`.
+ * so the residual false errors are suppressed.
  *
  * NB: these intentionally live here rather than in a dedicated module to avoid
  * adding a downstream file to consuming Jira packages' Thunderstone complexity.

package/dist/cjs/hooks/useFetchSyncBlockData.js

@@ -47,10 +47,10 @@ var useFetchSyncBlockData = exports.useFetchSyncBlockData = function useFetchSyn
   // On Jira the data provider is wired asynchronously, so the manager can be
   // constructed with `dataProvider === undefined`. Fetching/subscribing in that
   // window throws `Data provider not set`, logged as a false fetch error
-  // (EDITOR-7860). Gate on readiness; once the provider resolves a new manager
+  // (EDITOR-7860). Check readiness; once the provider resolves a new manager
   // instance is created so `referenceManager` changes identity and the effect
   // below re-runs, subscribing exactly once.
-  var isDataProviderReady = (0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3') ? (_manager$referenceMan2 = (_manager$referenceMan3 = (_manager$referenceMan4 = manager.referenceManager).hasDataProvider) === null || _manager$referenceMan3 === void 0 ? void 0 : _manager$referenceMan3.call(_manager$referenceMan4)) !== null && _manager$referenceMan2 !== void 0 ? _manager$referenceMan2 : false : true;
+  var isDataProviderReady = (_manager$referenceMan2 = (_manager$referenceMan3 = (_manager$referenceMan4 = manager.referenceManager).hasDataProvider) === null || _manager$referenceMan3 === void 0 ? void 0 : _manager$referenceMan3.call(_manager$referenceMan4)) !== null && _manager$referenceMan2 !== void 0 ? _manager$referenceMan2 : false;
   var reloadData = (0, _react.useCallback)( /*#__PURE__*/(0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee() {
     var syncBlockNode, _t;
     return _regenerator.default.wrap(function (_context) {
@@ -91,7 +91,7 @@ var useFetchSyncBlockData = exports.useFetchSyncBlockData = function useFetchSyn
         case 5:
           _context.prev = 5;
           _t = _context["catch"](2);
-          if (!((0, _types.isProviderNotReadyError)(_t) && (0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'))) {
+          if (!(0, _types.isProviderNotReadyError)(_t)) {
             _context.next = 6;
             break;
           }
@@ -105,7 +105,7 @@ var useFetchSyncBlockData = exports.useFetchSyncBlockData = function useFetchSyn
           (0, _monitoring.logException)(_t, {
             location: 'editor-synced-block-provider/useFetchSyncBlockData'
           });
-          fireAnalyticsEvent === null || fireAnalyticsEvent === void 0 || fireAnalyticsEvent((0, _errorHandling.fetchErrorPayload)(_t.message, resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(resourceId), (0, _errorHandling.buildFetchErrorAttribution)((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'), _t.message)));
+          fireAnalyticsEvent === null || fireAnalyticsEvent === void 0 || fireAnalyticsEvent((0, _errorHandling.fetchErrorPayload)(_t.message, resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(resourceId), (0, _errorHandling.buildFetchErrorAttribution)(_t.message)));
 
           // Thread the PII-safe original message/name so the renderer can de-opaque
           // this otherwise-bare `errored` failure. `originalMessage` carries the

package/dist/cjs/store-manager/referenceSyncBlockStoreManager.js

@@ -13,7 +13,6 @@ var _createClass2 = _interopRequireDefault(require("@babel/runtime/helpers/creat
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _isEqual = _interopRequireDefault(require("lodash/isEqual"));
 var _monitoring = require("@atlaskit/editor-common/monitoring");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _types = require("../common/types");
 var _errorHandling = require("../utils/errorHandling");
 var _experienceTracking = require("../utils/experienceTracking");
@@ -33,7 +32,7 @@ var ENTITY_NOT_FOUND_MAX_RETRIES = 3;
 var ENTITY_NOT_FOUND_INITIAL_DELAY_MS = 2000;
 
 // Grace period before a cache entry is removed after the last subscriber
-// unsubscribes (gated by `platform_synced_block_patch_14`). Guards are
+// unsubscribes. Guards are
 // re-checked at fire time; if any are positive, the timer is rescheduled.
 var CACHE_DELETION_GRACE_PERIOD_MS = 30000;
 // Max reschedules before force-deleting with an analytics event (~5 min).
@@ -63,7 +62,7 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
     (0, _defineProperty2.default)(this, "entityNotFoundRetryCount", new Map());
     (0, _defineProperty2.default)(this, "entityNotFoundRetryTimers", new Map());
     // Pending cache deletion timers keyed by resourceId (gated by
-    // `platform_synced_block_patch_14`). Cancelled when a subscriber re-attaches.
+    // Cancelled when a subscriber re-attaches.
     (0, _defineProperty2.default)(this, "pendingCacheDeletions", new Map());
     // Reschedule counter per resource — reset on actual deletion or re-subscribe.
     (0, _defineProperty2.default)(this, "cacheDeletionRescheduleCounts", new Map());
@@ -100,7 +99,7 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
         _this.isCacheDirty = true;
       },
       // Delegate cache lifecycle to the store manager so guards can be
-      // checked atomically (gated by `platform_synced_block_patch_14`).
+      // checked atomically.
       scheduleCacheDeletion: function scheduleCacheDeletion(rid) {
         return _this.scheduleCacheDeletion(rid);
       },
@@ -548,28 +547,22 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
               return _context2.abrupt("return");
             case 2:
               if (this.dataProvider) {
-                _context2.next = 4;
-                break;
-              }
-              if (!(0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3')) {
                 _context2.next = 3;
                 break;
               }
               throw new _types.ProviderNotReadyError();
             case 3:
-              throw new Error('Data provider not set');
-            case 4:
               nodesToFetch.forEach(function (node) {
                 _this4.syncBlockFetchDataRequests.set(node.attrs.resourceId, true);
               });
               (_this$fetchExperience6 = this.fetchExperience) === null || _this$fetchExperience6 === void 0 || _this$fetchExperience6.start({});
-              _context2.next = 5;
+              _context2.next = 4;
               return this.dataProvider.fetchNodesData(nodesToFetch).finally(function () {
                 nodesToFetch.forEach(function (node) {
                   _this4.syncBlockFetchDataRequests.delete(node.attrs.resourceId);
                 });
               });
-            case 5:
+            case 4:
               data = _context2.sent;
               _this$processFetchedD2 = this.processFetchedData(data), hasUnexpectedError = _this$processFetchedD2.hasUnexpectedError, hasExpectedError = _this$processFetchedD2.hasExpectedError;
               if (hasUnexpectedError) {
@@ -583,7 +576,7 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
               } else {
                 (_this$fetchExperience9 = this.fetchExperience) === null || _this$fetchExperience9 === void 0 || _this$fetchExperience9.success();
               }
-            case 6:
+            case 5:
             case "end":
               return _context2.stop();
           }
@@ -614,7 +607,7 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
           // No resourceId means we cannot derive a sourceProduct here; intentionally omit.
           // Classify on the structured `type` first, falling back to the free-text
           // `reason`/payload (EDITOR-7862).
-          (_this5$fireAnalyticsE = _this5.fireAnalyticsEvent) === null || _this5$fireAnalyticsE === void 0 || _this5$fireAnalyticsE.call(_this5, (0, _errorHandling.fetchErrorPayload)(payload, undefined, undefined, (0, _errorHandling.buildFetchErrorAttribution)((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'), ((_syncBlockInstance$er3 = syncBlockInstance.error) === null || _syncBlockInstance$er3 === void 0 ? void 0 : _syncBlockInstance$er3.type) || ((_syncBlockInstance$er4 = syncBlockInstance.error) === null || _syncBlockInstance$er4 === void 0 ? void 0 : _syncBlockInstance$er4.reason) || payload, (_syncBlockInstance$er5 = syncBlockInstance.error) === null || _syncBlockInstance$er5 === void 0 ? void 0 : _syncBlockInstance$er5.statusCode)));
+          (_this5$fireAnalyticsE = _this5.fireAnalyticsEvent) === null || _this5$fireAnalyticsE === void 0 || _this5$fireAnalyticsE.call(_this5, (0, _errorHandling.fetchErrorPayload)(payload, undefined, undefined, (0, _errorHandling.buildFetchErrorAttribution)(((_syncBlockInstance$er3 = syncBlockInstance.error) === null || _syncBlockInstance$er3 === void 0 ? void 0 : _syncBlockInstance$er3.type) || ((_syncBlockInstance$er4 = syncBlockInstance.error) === null || _syncBlockInstance$er4 === void 0 ? void 0 : _syncBlockInstance$er4.reason) || payload, (_syncBlockInstance$er5 = syncBlockInstance.error) === null || _syncBlockInstance$er5 === void 0 ? void 0 : _syncBlockInstance$er5.statusCode)));
           return;
         }
         var existingSyncBlock = _this5.getFromCache(syncBlockInstance.resourceId);
@@ -645,13 +638,13 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
           var _this5$entityNotFound;
           // Skip error analytics when EntityNotFound will be retried, to avoid
           // inflating error-rate metrics with expected transient failures
-          var isRetryingEntityNotFound = syncBlockInstance.error.type === _types.SyncBlockError.EntityNotFound && ((_this5$entityNotFound = _this5.entityNotFoundRetryCount.get(syncBlockInstance.resourceId)) !== null && _this5$entityNotFound !== void 0 ? _this5$entityNotFound : 0) < ENTITY_NOT_FOUND_MAX_RETRIES && (0, _platformFeatureFlags.fg)('platform_synced_block_patch_13');
+          var isRetryingEntityNotFound = syncBlockInstance.error.type === _types.SyncBlockError.EntityNotFound && ((_this5$entityNotFound = _this5.entityNotFoundRetryCount.get(syncBlockInstance.resourceId)) !== null && _this5$entityNotFound !== void 0 ? _this5$entityNotFound : 0) < ENTITY_NOT_FOUND_MAX_RETRIES;
           if (!isRetryingEntityNotFound) {
             var _this5$fireAnalyticsE2, _syncBlockInstance$da, _syncBlockInstance$da2;
             // Classify on the structured `type` (a `SyncBlockError` enum value) first,
             // falling back to the free-text `reason` so source-state/permission strings
             // are still bucketed (EDITOR-7862). The emitted `error` attribute is unchanged.
-            (_this5$fireAnalyticsE2 = _this5.fireAnalyticsEvent) === null || _this5$fireAnalyticsE2 === void 0 || _this5$fireAnalyticsE2.call(_this5, (0, _errorHandling.fetchErrorPayload)(syncBlockInstance.error.reason || syncBlockInstance.error.type, syncBlockInstance.resourceId, (_syncBlockInstance$da = (_syncBlockInstance$da2 = syncBlockInstance.data) === null || _syncBlockInstance$da2 === void 0 ? void 0 : _syncBlockInstance$da2.product) !== null && _syncBlockInstance$da !== void 0 ? _syncBlockInstance$da : (0, _utils.getSourceProductFromResourceIdSafe)(syncBlockInstance.resourceId), (0, _errorHandling.buildFetchErrorAttribution)((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'), syncBlockInstance.error.type || syncBlockInstance.error.reason, syncBlockInstance.error.statusCode)));
+            (_this5$fireAnalyticsE2 = _this5.fireAnalyticsEvent) === null || _this5$fireAnalyticsE2 === void 0 || _this5$fireAnalyticsE2.call(_this5, (0, _errorHandling.fetchErrorPayload)(syncBlockInstance.error.reason || syncBlockInstance.error.type, syncBlockInstance.resourceId, (_syncBlockInstance$da = (_syncBlockInstance$da2 = syncBlockInstance.data) === null || _syncBlockInstance$da2 === void 0 ? void 0 : _syncBlockInstance$da2.product) !== null && _syncBlockInstance$da !== void 0 ? _syncBlockInstance$da : (0, _utils.getSourceProductFromResourceIdSafe)(syncBlockInstance.resourceId), (0, _errorHandling.buildFetchErrorAttribution)(syncBlockInstance.error.type || syncBlockInstance.error.reason, syncBlockInstance.error.statusCode)));
           }
           if (syncBlockInstance.error.type === _types.SyncBlockError.NotFound || syncBlockInstance.error.type === _types.SyncBlockError.Forbidden) {
             hasExpectedError = true;
@@ -659,9 +652,7 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
             // Schedule a retry for EntityNotFound — the source block may be in
             // the process of being created by a collaborator (race condition
             // between NCS propagation and Block Service createBlock call).
-            if ((0, _platformFeatureFlags.fg)('platform_synced_block_patch_13')) {
-              _this5.scheduleEntityNotFoundRetry(syncBlockInstance.resourceId);
-            }
+            _this5.scheduleEntityNotFoundRetry(syncBlockInstance.resourceId);
             if (!isRetryingEntityNotFound) {
               hasUnexpectedError = true;
             }
@@ -689,7 +680,7 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
       var existingSyncBlock = this.getFromCache(resourceId);
       // If the cache entry was deleted while the source-info request was
       // in flight, fire an analytics event so the race is observable.
-      if (!existingSyncBlock && (0, _platformFeatureFlags.fg)('platform_synced_block_patch_14')) {
+      if (!existingSyncBlock) {
         var _this$fireAnalyticsEv5;
         (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 || _this$fireAnalyticsEv5.call(this, (0, _errorHandling.sourceInfoOrphanedPayload)(resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(resourceId), {
           hasPendingDeletion: this.pendingCacheDeletions.has(resourceId),
@@ -742,16 +733,14 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
       this._providerFactoryManager.deleteFactory(resourceId);
       // Evict in-flight source-info promise and reset reschedule counter
       // so a stale resolution can't silently merge into a re-fetched entry.
-      if ((0, _platformFeatureFlags.fg)('platform_synced_block_patch_14')) {
-        this.syncBlockSourceInfoRequests.delete(resourceId);
-        this.cacheDeletionRescheduleCounts.delete(resourceId);
-      }
+      this.syncBlockSourceInfoRequests.delete(resourceId);
+      this.cacheDeletionRescheduleCounts.delete(resourceId);
     }
 
     /**
      * Returns true if the cache entry for `resourceId` is safe to delete:
      * no active subscribers, no in-flight source-info request, and no
-     * queued/in-flight batch fetch (gated by `platform_synced_block_patch_14`).
+     * queued/in-flight batch fetch.
      */
   }, {
     key: "canDeleteCache",
@@ -770,7 +759,7 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
 
     /**
      * Schedules cache deletion for `resourceId` after the grace period
-     * (gated by `platform_synced_block_patch_14`). Called when the last
+     * Called when the last
      * subscriber unsubscribes. Guards are re-checked at fire time; if any
      * are positive the timer is rescheduled up to MAX_RESCHEDULES times.
      */
@@ -778,9 +767,6 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
     key: "scheduleCacheDeletion",
     value: function scheduleCacheDeletion(resourceId) {
       var _this6 = this;
-      if (!(0, _platformFeatureFlags.fg)('platform_synced_block_patch_14')) {
-        return;
-      }
       if (this.isDestroyed) {
         return;
       }
@@ -808,15 +794,12 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
 
     /**
      * Cancels any pending cache deletion timer for `resourceId` and resets the
-     * reschedule counter (gated by `platform_synced_block_patch_14`). Called
+     * reschedule counter. Called
      * when a new subscriber arrives.
      */
   }, {
     key: "cancelPendingCacheDeletion",
     value: function cancelPendingCacheDeletion(resourceId) {
-      if (!(0, _platformFeatureFlags.fg)('platform_synced_block_patch_14')) {
-        return;
-      }
       var existing = this.pendingCacheDeletions.get(resourceId);
       if (existing) {
         clearTimeout(existing);
@@ -968,14 +951,14 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
         var _this$fireAnalyticsEv7;
         // EDITOR-7860: benign not-ready/torn-down case — suppress both the
         // exception-tracker log and the analytics event (checked first so the
-        // benign case stays fully silent). Gate-off behaviour is unchanged.
-        if ((0, _types.isProviderNotReadyError)(error) && (0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3')) {
+        // benign case stays fully silent).
+        if ((0, _types.isProviderNotReadyError)(error)) {
           return function () {};
         }
         (0, _monitoring.logException)(error, {
           location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
         });
-        (_this$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 || _this$fireAnalyticsEv7.call(this, (0, _errorHandling.fetchErrorPayload)(error.message, undefined, undefined, (0, _errorHandling.buildFetchErrorAttribution)((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'), error.message)));
+        (_this$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 || _this$fireAnalyticsEv7.call(this, (0, _errorHandling.fetchErrorPayload)(error.message, undefined, undefined, (0, _errorHandling.buildFetchErrorAttribution)(error.message)));
         return function () {};
       }
     }
@@ -1138,7 +1121,7 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
                 (_this$saveExperience2 = this.saveExperience) === null || _this$saveExperience2 === void 0 || _this$saveExperience2.failure({
                   reason: updateResult.error || 'Failed to update reference synced blocks on the document'
                 });
-                (_this$fireAnalyticsEv8 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv8 === void 0 || _this$fireAnalyticsEv8.call(this, (0, _errorHandling.updateReferenceErrorPayload)(updateResult.error || 'Failed to update reference synced blocks on the document', undefined, undefined, (0, _errorHandling.buildErrorAttribution)((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'), updateResult.error, updateResult.statusCode)));
+                (_this$fireAnalyticsEv8 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv8 === void 0 || _this$fireAnalyticsEv8.call(this, (0, _errorHandling.updateReferenceErrorPayload)(updateResult.error || 'Failed to update reference synced blocks on the document', undefined, undefined, (0, _errorHandling.buildErrorAttribution)(updateResult.error, updateResult.statusCode)));
               }
               _context4.next = 17;
               break;
@@ -1155,7 +1138,7 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
               // No `resourceId` available in this catch — sourceProduct is intentionally omitted.
               // No structured SyncBlockError/status here, so the attribution `reason` falls back
               // to `unknown` when the gate is on.
-              (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 || _this$fireAnalyticsEv9.call(this, (0, _errorHandling.updateReferenceErrorPayload)(_t3.message, undefined, undefined, (0, _errorHandling.buildErrorAttribution)((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'))));
+              (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 || _this$fireAnalyticsEv9.call(this, (0, _errorHandling.updateReferenceErrorPayload)(_t3.message, undefined, undefined, (0, _errorHandling.buildErrorAttribution)()));
             case 17:
               _context4.prev = 17;
               if (!success) {
@@ -1237,17 +1220,9 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
       });
       this.fireAnalyticsEvent = undefined;
 
-      // Under `platform_synced_block_patch_14`, `destroy()` is now wired to
-      // React component unmount via `useMemoizedSyncBlockStoreManager`.
-      // Clearing the module-level singleton on unmount would wipe SSR session
-      // cache data that a sibling/successor manager (e.g. the editor
-      // instance that mounts immediately after the renderer unmounts during
-      // the view-mode transition) is about to read.
-      // Let entries age out naturally instead — the in-memory cache is
-      // naturally bounded by `maxSize` (LRU) and cleared on hard navigation.
-      if (!(0, _platformFeatureFlags.fg)('platform_synced_block_patch_14')) {
-        _syncBlockInMemorySessionCache.syncBlockInMemorySessionCache.clear();
-      }
+      // `destroy()` is wired to React component unmount via
+      // `useMemoizedSyncBlockStoreManager`. Let the in-memory session cache age
+      // out naturally instead of clearing it during a view-mode transition.
     }
   }]);
 }();

package/dist/cjs/store-manager/sourceSyncBlockStoreManager.js

@@ -12,7 +12,6 @@ var _classCallCheck2 = _interopRequireDefault(require("@babel/runtime/helpers/cl
 var _createClass2 = _interopRequireDefault(require("@babel/runtime/helpers/createClass"));
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _monitoring = require("@atlaskit/editor-common/monitoring");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _types = require("../common/types");
 var _errorHandling = require("../utils/errorHandling");
 var _experienceTracking = require("../utils/experienceTracking");
@@ -82,8 +81,7 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
     (0, _defineProperty2.default)(this, "awaitingFirstContent", new Set());
     /**
      * resourceId -> timestamp (ms) of the last `syncedBlockDelete` emission, used
-     * to suppress duplicates within {@link DELETE_DEDUPE_WINDOW_MS}. Only consulted
-     * behind `platform_editor_blocks_patch_4`.
+     * to suppress duplicates within {@link DELETE_DEDUPE_WINDOW_MS}.
      */
     (0, _defineProperty2.default)(this, "recentDeleteEmissions", new Map());
     (0, _defineProperty2.default)(this, "setPendingDeletion", function (Ids, value) {
@@ -169,10 +167,7 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
           resourceId = _syncBlockNode$attrs.resourceId;
         if (!localId || !resourceId) {
           // Identifiers not populated yet: benign timing case, skip as a no-op.
-          if ((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_4')) {
-            return false;
-          }
-          throw new Error('Local ID or resource ID is not set');
+          return false;
         }
         var cachedBlock = this.syncBlockCache.get(resourceId);
 
@@ -215,7 +210,7 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
     value: (function () {
       var _flush = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee() {
         var _this2 = this;
-        var _this$saveExperience, timedOut, timeoutId, timeout, _iterator, _step, resourceId, bodiedSyncBlockNodes, bodiedSyncBlockData, writeResults, _this$saveExperience2, _this$saveExperience3, attributionEnabled, _this$fireAnalyticsEv2, _this$flushCompletion, _t;
+        var _this$saveExperience, timedOut, timeoutId, timeout, _iterator, _step, resourceId, bodiedSyncBlockNodes, bodiedSyncBlockData, writeResults, _this$saveExperience2, _this$saveExperience3, _this$fireAnalyticsEv2, _this$flushCompletion, _t;
         return _regenerator.default.wrap(function (_context) {
           while (1) switch (_context.prev = _context.next) {
             case 0:
@@ -334,12 +329,11 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
               return _context.abrupt("return", true);
             case 7:
               (_this$saveExperience3 = this.saveExperience) === null || _this$saveExperience3 === void 0 || _this$saveExperience3.failure();
-              attributionEnabled = (0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3');
               writeResults.filter(function (result) {
                 return !result.resourceId || result.error;
               }).forEach(function (result) {
                 var _this2$fireAnalyticsE2;
-                (_this2$fireAnalyticsE2 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE2 === void 0 || _this2$fireAnalyticsE2.call(_this2, (0, _errorHandling.updateErrorPayload)(result.error || 'Failed to write data', result.resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(result.resourceId), (0, _errorHandling.buildErrorAttribution)(attributionEnabled, result.error, result.statusCode)));
+                (_this2$fireAnalyticsE2 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE2 === void 0 || _this2$fireAnalyticsE2.call(_this2, (0, _errorHandling.updateErrorPayload)(result.error || 'Failed to write data', result.resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(result.resourceId), (0, _errorHandling.buildErrorAttribution)(result.error, result.statusCode)));
               });
               return _context.abrupt("return", false);
             case 8:
@@ -353,7 +347,7 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
               });
               // Top-level flush failure is not tied to a single resourceId. There is no structured
               // SyncBlockError/status here, so the attribution `reason` falls back to `unknown`.
-              (_this$fireAnalyticsEv2 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv2 === void 0 || _this$fireAnalyticsEv2.call(this, (0, _errorHandling.updateErrorPayload)(_t.message, undefined, undefined, (0, _errorHandling.buildErrorAttribution)((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'))));
+              (_this$fireAnalyticsEv2 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv2 === void 0 || _this$fireAnalyticsEv2.call(this, (0, _errorHandling.updateErrorPayload)(_t.message, undefined, undefined, (0, _errorHandling.buildErrorAttribution)()));
               return _context.abrupt("return", false);
             case 10:
               _context.prev = 10;
@@ -447,15 +441,12 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
         (_this$fireAnalyticsEv3 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv3 === void 0 || _this$fireAnalyticsEv3.call(this, (0, _errorHandling.createErrorPayload)('creation complete callback missing', resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(resourceId)));
       }
       if (success) {
-        var _this$fireAnalyticsEv4;
+        var _this$fireAnalyticsEv4, _this$fireAnalyticsEv5, _this$syncBlockCache$2;
         var sourceProduct = (0, _utils.getSourceProductFromResourceIdSafe)(resourceId);
         (_this$fireAnalyticsEv4 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv4 === void 0 || _this$fireAnalyticsEv4.call(this, (0, _errorHandling.createSuccessPayload)(resourceId || '', sourceProduct));
         // Operational create-success event with the join key + creation-type
         // signals (inputMethod / createdEmpty) captured at the command layer.
-        if ((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_4')) {
-          var _this$fireAnalyticsEv5, _this$syncBlockCache$2;
-          (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 || _this$fireAnalyticsEv5.call(this, (0, _errorHandling.createSuccessOperationalPayload)(resourceId || '', (_this$syncBlockCache$2 = this.syncBlockCache.get(resourceId)) === null || _this$syncBlockCache$2 === void 0 ? void 0 : _this$syncBlockCache$2.blockInstanceId, sourceProduct, this.creationEnrichment.get(resourceId)));
-        }
+        (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 || _this$fireAnalyticsEv5.call(this, (0, _errorHandling.createSuccessOperationalPayload)(resourceId || '', (_this$syncBlockCache$2 = this.syncBlockCache.get(resourceId)) === null || _this$syncBlockCache$2 === void 0 ? void 0 : _this$syncBlockCache$2.blockInstanceId, sourceProduct, this.creationEnrichment.get(resourceId)));
       } else {
         var _this$fireAnalyticsEv6;
         // Delete the node from cache if fail to create so it's not flushed to BE
@@ -474,7 +465,6 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
      * Fire the first-content-added event once when a block created empty this
      * session first gains content. The caller detects the in-block edit; this owns
      * dedupe + emission (fires at most once per block, only for empty→content).
-     * Gated behind `platform_editor_blocks_patch_4`.
      */
   }, {
     key: "maybeEmitFirstContentAdded",
@@ -483,9 +473,6 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
       if (this.viewMode === 'view') {
         return;
       }
-      if (!(0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_4')) {
-        return;
-      }
       // `delete` returns false when the id was not tracked (already emitted, or
       // the block was not created empty this session), so this both dedupes and
       // scopes emission to genuine first-content transitions in one check.
@@ -589,7 +576,7 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
             (_this4$createExperien2 = _this4.createExperience) === null || _this4$createExperien2 === void 0 || _this4$createExperien2.failure({
               reason: result.error || 'Failed to create bodied sync block'
             });
-            (_this4$fireAnalyticsE = _this4.fireAnalyticsEvent) === null || _this4$fireAnalyticsE === void 0 || _this4$fireAnalyticsE.call(_this4, (0, _errorHandling.createErrorPayload)(result.error || 'Failed to create bodied sync block', resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(resourceId), (0, _errorHandling.buildErrorAttribution)((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'), result.error, result.statusCode)));
+            (_this4$fireAnalyticsE = _this4.fireAnalyticsEvent) === null || _this4$fireAnalyticsE === void 0 || _this4$fireAnalyticsE.call(_this4, (0, _errorHandling.createErrorPayload)(result.error || 'Failed to create bodied sync block', resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(resourceId), (0, _errorHandling.buildErrorAttribution)(result.error, result.statusCode)));
           }
         }).catch(function (error) {
           var _this4$createExperien3, _this4$fireAnalyticsE2;
@@ -652,13 +639,8 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
   }, {
     key: "emitDeleteSuccess",
     value: function emitDeleteSuccess(resourceId, reason, mechanism) {
-      var _this$syncBlockCache$3, _this$fireAnalyticsEv0;
+      var _this$syncBlockCache$3, _this$fireAnalyticsEv9;
       var sourceProduct = (0, _utils.getSourceProductFromResourceIdSafe)(resourceId);
-      if (!(0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_4')) {
-        var _this$fireAnalyticsEv9;
-        (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 || _this$fireAnalyticsEv9.call(this, (0, _errorHandling.deleteSuccessPayload)(resourceId, sourceProduct));
-        return;
-      }
       var now = Date.now();
       var lastEmittedAt = this.recentDeleteEmissions.get(resourceId);
       if (lastEmittedAt !== undefined && now - lastEmittedAt < DELETE_DEDUPE_WINDOW_MS) {
@@ -671,14 +653,14 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
         deletionReason: reason,
         mechanism: mechanism
       };
-      (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 || _this$fireAnalyticsEv0.call(this, (0, _errorHandling.deleteSuccessPayload)(resourceId, sourceProduct, enrichment));
+      (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 || _this$fireAnalyticsEv9.call(this, (0, _errorHandling.deleteSuccessPayload)(resourceId, sourceProduct, enrichment));
     }
   }, {
     key: "delete",
     value: function () {
       var _delete2 = (0, _asyncToGenerator2.default)( /*#__PURE__*/_regenerator.default.mark(function _callee2(syncBlockIds, onDelete, onDeleteCompleted, reason, mechanism) {
         var _this5 = this;
-        var _this$deleteExperienc, results, callback, isDeleteSuccessful, _this$deleteExperienc2, _this$deleteExperienc3, attributionEnabled, attribution, _t2;
+        var _this$deleteExperienc, results, callback, isDeleteSuccessful, _this$deleteExperienc2, _this$deleteExperienc3, attribution, _t2;
         return _regenerator.default.wrap(function (_context2) {
           while (1) switch (_context2.prev = _context2.next) {
             case 0:
@@ -726,13 +708,12 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
                   _this5.setPendingDeletion(Ids, false);
                 };
                 (_this$deleteExperienc3 = this.deleteExperience) === null || _this$deleteExperienc3 === void 0 || _this$deleteExperienc3.failure();
-                attributionEnabled = (0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3');
                 results.forEach(function (result) {
                   if (result.success) {
                     _this5.emitDeleteSuccess(result.resourceId, reason, mechanism);
                   } else {
                     var _this5$fireAnalyticsE;
-                    (_this5$fireAnalyticsE = _this5.fireAnalyticsEvent) === null || _this5$fireAnalyticsE === void 0 || _this5$fireAnalyticsE.call(_this5, (0, _errorHandling.deleteErrorPayload)(result.error || 'Failed to delete synced block', result.resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(result.resourceId), (0, _errorHandling.buildErrorAttribution)(attributionEnabled, result.error, result.statusCode)));
+                    (_this5$fireAnalyticsE = _this5.fireAnalyticsEvent) === null || _this5$fireAnalyticsE === void 0 || _this5$fireAnalyticsE.call(_this5, (0, _errorHandling.deleteErrorPayload)(result.error || 'Failed to delete synced block', result.resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(result.resourceId), (0, _errorHandling.buildErrorAttribution)(result.error, result.statusCode)));
                   }
                 });
               }
@@ -743,7 +724,7 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
               _t2 = _context2["catch"](0);
               // Thrown (non-result) failure — no structured SyncBlockError/status is available,
               // so the attribution `reason` falls back to `unknown` when the gate is on.
-              attribution = (0, _errorHandling.buildErrorAttribution)((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'));
+              attribution = (0, _errorHandling.buildErrorAttribution)();
               syncBlockIds.forEach(function (Ids) {
                 var _this5$fireAnalyticsE2;
                 _this5.setPendingDeletion(Ids, false);
@@ -1020,11 +1001,11 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
           return sourceInfo;
         });
       } catch (error) {
-        var _this$fireAnalyticsEv1;
+        var _this$fireAnalyticsEv0;
         (0, _monitoring.logException)(error, {
           location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
         });
-        (_this$fireAnalyticsEv1 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv1 === void 0 || _this$fireAnalyticsEv1.call(this, (0, _errorHandling.getSourceInfoErrorPayload)(error.message));
+        (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 || _this$fireAnalyticsEv0.call(this, (0, _errorHandling.getSourceInfoErrorPayload)(error.message));
         return Promise.resolve(undefined);
       }
     }
@@ -1037,11 +1018,11 @@ var SourceSyncBlockStoreManager = exports.SourceSyncBlockStoreManager = /*#__PUR
         }
         return this.dataProvider.fetchReferences(resourceId, true);
       } catch (error) {
-        var _this$fireAnalyticsEv10;
+        var _this$fireAnalyticsEv1;
         (0, _monitoring.logException)(error, {
           location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
         });
-        (_this$fireAnalyticsEv10 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv10 === void 0 || _this$fireAnalyticsEv10.call(this, (0, _errorHandling.fetchReferencesErrorPayload)(error.message, resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(resourceId)));
+        (_this$fireAnalyticsEv1 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv1 === void 0 || _this$fireAnalyticsEv1.call(this, (0, _errorHandling.fetchReferencesErrorPayload)(error.message, resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(resourceId)));
         return Promise.resolve({
           error: _types.SyncBlockError.Errored
         });

package/dist/cjs/store-manager/syncBlockBatchFetcher.js

@@ -10,7 +10,6 @@ var _createClass2 = _interopRequireDefault(require("@babel/runtime/helpers/creat
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _rafSchd = _interopRequireDefault(require("raf-schd"));
 var _monitoring = require("@atlaskit/editor-common/monitoring");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _types = require("../common/types");
 var _errorHandling = require("../utils/errorHandling");
 var _utils = require("../utils/utils");
@@ -36,14 +35,9 @@ var SyncBlockBatchFetcher = exports.SyncBlockBatchFetcher = /*#__PURE__*/functio
       }
 
       // EDITOR-7860: not ready — skip and leave resourceIds queued for the
-      // next batch once the provider resolves. Gate-off is unchanged (the
-      // readiness check is nested under the gate so it is not consulted when
-      // the gate is off, and `fg()` stays a standalone condition so gate
-      // exposure is still tracked — satisfies @atlaskit/platform/no-preconditioning).
-      if ((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3')) {
-        if (_this.deps.isProviderReady && !_this.deps.isProviderReady()) {
-          return;
-        }
+      // next batch once the provider resolves.
+      if (_this.deps.isProviderReady && !_this.deps.isProviderReady()) {
+        return;
       }
       var resourceIds = Array.from(_this.pendingFetchRequests);
       var syncBlockNodes = resourceIds.map(function (resId) {
@@ -62,8 +56,8 @@ var SyncBlockBatchFetcher = exports.SyncBlockBatchFetcher = /*#__PURE__*/functio
         // nothing (no analytics, no exception-tracker noise). Checked before
         // `logException` so the benign case stays silent. Re-schedule so the
         // re-queued IDs are retried on the next frame even if no further
-        // `queueFetch()` arrives. Gate-off behaviour is unchanged.
-        if ((0, _types.isProviderNotReadyError)(error) && (0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3')) {
+        // `queueFetch()` arrives.
+        if ((0, _types.isProviderNotReadyError)(error)) {
           resourceIds.forEach(function (resId) {
             return _this.pendingFetchRequests.add(resId);
           });
@@ -75,7 +69,7 @@ var SyncBlockBatchFetcher = exports.SyncBlockBatchFetcher = /*#__PURE__*/functio
         (0, _monitoring.logException)(error, {
           location: 'editor-synced-block-provider/syncBlockBatchFetcher/batchedFetchSyncBlocks'
         });
-        var attribution = (0, _errorHandling.buildFetchErrorAttribution)((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'), error.message);
+        var attribution = (0, _errorHandling.buildFetchErrorAttribution)(error.message);
         resourceIds.forEach(function (resId) {
           var _this$deps$getFireAna;
           (_this$deps$getFireAna = _this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna === void 0 || _this$deps$getFireAna((0, _errorHandling.fetchErrorPayload)(error.message, resId, (0, _utils.getSourceProductFromResourceIdSafe)(resId), attribution));

package/dist/cjs/store-manager/syncBlockProviderFactoryManager.js

@@ -35,7 +35,7 @@ var SyncBlockProviderFactoryManager = exports.SyncBlockProviderFactoryManager =
         (0, _monitoring.logException)(error, {
           location: 'editor-synced-block-provider/syncBlockProviderFactoryManager'
         });
-        (_this$deps$getFireAna = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna === void 0 || _this$deps$getFireAna((0, _errorHandling.fetchErrorPayload)(error.message, resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(resourceId), (0, _errorHandling.buildFetchErrorAttribution)((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'), error.message)));
+        (_this$deps$getFireAna = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna === void 0 || _this$deps$getFireAna((0, _errorHandling.fetchErrorPayload)(error.message, resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(resourceId), (0, _errorHandling.buildFetchErrorAttribution)(error.message)));
         return undefined;
       }
       var providerFactory = this.providerFactories.get(resourceId);
@@ -111,7 +111,7 @@ var SyncBlockProviderFactoryManager = exports.SyncBlockProviderFactoryManager =
         (0, _monitoring.logException)(error, {
           location: 'editor-synced-block-provider/syncBlockProviderFactoryManager'
         });
-        (_this$deps$getFireAna2 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna2 === void 0 || _this$deps$getFireAna2((0, _errorHandling.fetchErrorPayload)(error.message, resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(resourceId), (0, _errorHandling.buildFetchErrorAttribution)((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'), error.message)));
+        (_this$deps$getFireAna2 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna2 === void 0 || _this$deps$getFireAna2((0, _errorHandling.fetchErrorPayload)(error.message, resourceId, (0, _utils.getSourceProductFromResourceIdSafe)(resourceId), (0, _errorHandling.buildFetchErrorAttribution)(error.message)));
       }
     }
   }, {
@@ -194,7 +194,7 @@ var SyncBlockProviderFactoryManager = exports.SyncBlockProviderFactoryManager =
       if (!syncBlock.data.sourceAri || !syncBlock.data.product) {
         var _this$deps$getFireAna3, _syncBlock$data$produ;
         (_this$deps$getFireAna3 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna3 === void 0 || _this$deps$getFireAna3((0, _errorHandling.fetchErrorPayload)('Sync block source ari or product not found', resourceId, // Prefer cached product when available; fall back to parsing resourceId.
-        (_syncBlock$data$produ = syncBlock.data.product) !== null && _syncBlock$data$produ !== void 0 ? _syncBlock$data$produ : (0, _utils.getSourceProductFromResourceIdSafe)(resourceId), (0, _errorHandling.buildFetchErrorAttribution)((0, _platformFeatureFlags.fg)('platform_editor_blocks_patch_3'), 'Sync block source ari or product not found')));
+        (_syncBlock$data$produ = syncBlock.data.product) !== null && _syncBlock$data$produ !== void 0 ? _syncBlock$data$produ : (0, _utils.getSourceProductFromResourceIdSafe)(resourceId), (0, _errorHandling.buildFetchErrorAttribution)('Sync block source ari or product not found')));
         return;
       }
       var parentInfo = dataProvider.retrieveSyncBlockParentInfo(syncBlock.data.sourceAri, syncBlock.data.product);

package/dist/cjs/store-manager/syncBlockStoreManager.js

@@ -12,7 +12,6 @@ var _classCallCheck2 = _interopRequireDefault(require("@babel/runtime/helpers/cl
 var _createClass2 = _interopRequireDefault(require("@babel/runtime/helpers/createClass"));
 var _react = require("react");
 var _monitoring = require("@atlaskit/editor-common/monitoring");
-var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
 var _ari = require("../clients/block-service/ari");
 var _types = require("../common/types");
 var _errorHandling = require("../utils/errorHandling");
@@ -308,7 +307,6 @@ var useMemoizedSyncBlockStoreManager = exports.useMemoizedSyncBlockStoreManager
     syncBlockStoreManager.setFireAnalyticsEvent(fireAnalyticsEvent);
   }
 
-  // Gated by platform_synced_block_patch_14:
   // Destroy the SyncBlockStoreManager when:
   //   (a) the component unmounts — manager is fully cleaned up, or
   //   (b) dataProvider changes — the old manager (now orphaned by the
@@ -320,9 +318,7 @@ var useMemoizedSyncBlockStoreManager = exports.useMemoizedSyncBlockStoreManager
   // changes, triggering the cleanup for the old instance.
   (0, _react.useEffect)(function () {
     return function () {
-      if ((0, _platformFeatureFlags.fg)('platform_synced_block_patch_14')) {
-        syncBlockStoreManager.destroy();
-      }
+      syncBlockStoreManager.destroy();
     };
   }, [syncBlockStoreManager]);
   return syncBlockStoreManager;