@atlaskit/editor-synced-block-provider 10.0.3 → 10.0.5

package/dist/esm/hooks/useFetchSyncBlockData.js

@@ -40,10 +40,10 @@ export var useFetchSyncBlockData = function useFetchSyncBlockData(manager, resou
   // 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 = 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 = useCallback( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee() {
     var syncBlockNode, _t;
     return _regeneratorRuntime.wrap(function (_context) {
@@ -84,7 +84,7 @@ export var useFetchSyncBlockData = function useFetchSyncBlockData(manager, resou
         case 5:
           _context.prev = 5;
           _t = _context["catch"](2);
-          if (!(isProviderNotReadyError(_t) && fg('platform_editor_blocks_patch_3'))) {
+          if (!isProviderNotReadyError(_t)) {
             _context.next = 6;
             break;
           }
@@ -98,7 +98,7 @@ export var useFetchSyncBlockData = function useFetchSyncBlockData(manager, resou
           logException(_t, {
             location: 'editor-synced-block-provider/useFetchSyncBlockData'
           });
-          fireAnalyticsEvent === null || fireAnalyticsEvent === void 0 || fireAnalyticsEvent(fetchErrorPayload(_t.message, resourceId, getSourceProductFromResourceIdSafe(resourceId), buildFetchErrorAttribution(fg('platform_editor_blocks_patch_3'), _t.message)));
+          fireAnalyticsEvent === null || fireAnalyticsEvent === void 0 || fireAnalyticsEvent(fetchErrorPayload(_t.message, resourceId, getSourceProductFromResourceIdSafe(resourceId), 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/esm/store-manager/referenceSyncBlockStoreManager.js

@@ -11,7 +11,6 @@ function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbol
 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) { _defineProperty(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; }
 import isEqual from 'lodash/isEqual';
 import { logException } from '@atlaskit/editor-common/monitoring';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { isProviderNotReadyError, ProviderNotReadyError, SyncBlockError } from '../common/types';
 import { buildErrorAttribution, buildFetchErrorAttribution, cacheDeletionForcedPayload, fetchErrorPayload, fetchSuccessPayload, getSourceInfoErrorPayload, sourceInfoOrphanedPayload, updateReferenceErrorPayload } from '../utils/errorHandling';
 import { getFetchExperience, getFetchSourceInfoExperience, getSaveReferenceExperience } from '../utils/experienceTracking';
@@ -26,7 +25,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).
@@ -55,8 +54,7 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
     // Track retry attempts for EntityNotFound errors (block may be in the process of being created)
     _defineProperty(this, "entityNotFoundRetryCount", new Map());
     _defineProperty(this, "entityNotFoundRetryTimers", new Map());
-    // Pending cache deletion timers keyed by resourceId (gated by
-    // `platform_synced_block_patch_14`). Cancelled when a subscriber re-attaches.
+    // Pending cache deletion timers keyed by resourceId. Cancelled when a subscriber re-attaches.
     _defineProperty(this, "pendingCacheDeletions", new Map());
     // Reschedule counter per resource — reset on actual deletion or re-subscribe.
     _defineProperty(this, "cacheDeletionRescheduleCounts", new Map());
@@ -77,9 +75,6 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
       updateCache: function updateCache(inst) {
         return _this.updateCache(inst);
       },
-      deleteFromCache: function deleteFromCache(rid) {
-        return _this.deleteFromCache(rid);
-      },
       debouncedBatchedFetchSyncBlocks: function debouncedBatchedFetchSyncBlocks(rid) {
         return _this.debouncedBatchedFetchSyncBlocks(rid);
       },
@@ -93,7 +88,7 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
         _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);
       },
@@ -541,28 +536,22 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
               return _context2.abrupt("return");
             case 2:
               if (this.dataProvider) {
-                _context2.next = 4;
-                break;
-              }
-              if (!fg('platform_editor_blocks_patch_3')) {
                 _context2.next = 3;
                 break;
               }
               throw new 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) {
@@ -576,7 +565,7 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
               } else {
                 (_this$fetchExperience9 = this.fetchExperience) === null || _this$fetchExperience9 === void 0 || _this$fetchExperience9.success();
               }
-            case 6:
+            case 5:
             case "end":
               return _context2.stop();
           }
@@ -607,7 +596,7 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
           // 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, fetchErrorPayload(payload, undefined, undefined, buildFetchErrorAttribution(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, fetchErrorPayload(payload, undefined, undefined, 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);
@@ -638,13 +627,13 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
           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 === SyncBlockError.EntityNotFound && ((_this5$entityNotFound = _this5.entityNotFoundRetryCount.get(syncBlockInstance.resourceId)) !== null && _this5$entityNotFound !== void 0 ? _this5$entityNotFound : 0) < ENTITY_NOT_FOUND_MAX_RETRIES && fg('platform_synced_block_patch_13');
+          var isRetryingEntityNotFound = syncBlockInstance.error.type === 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, 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 : getSourceProductFromResourceIdSafe(syncBlockInstance.resourceId), buildFetchErrorAttribution(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, 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 : getSourceProductFromResourceIdSafe(syncBlockInstance.resourceId), buildFetchErrorAttribution(syncBlockInstance.error.type || syncBlockInstance.error.reason, syncBlockInstance.error.statusCode)));
           }
           if (syncBlockInstance.error.type === SyncBlockError.NotFound || syncBlockInstance.error.type === SyncBlockError.Forbidden) {
             hasExpectedError = true;
@@ -652,9 +641,7 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
             // 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 (fg('platform_synced_block_patch_13')) {
-              _this5.scheduleEntityNotFoundRetry(syncBlockInstance.resourceId);
-            }
+            _this5.scheduleEntityNotFoundRetry(syncBlockInstance.resourceId);
             if (!isRetryingEntityNotFound) {
               hasUnexpectedError = true;
             }
@@ -682,7 +669,7 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
       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 && fg('platform_synced_block_patch_14')) {
+      if (!existingSyncBlock) {
         var _this$fireAnalyticsEv5;
         (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 || _this$fireAnalyticsEv5.call(this, sourceInfoOrphanedPayload(resourceId, getSourceProductFromResourceIdSafe(resourceId), {
           hasPendingDeletion: this.pendingCacheDeletions.has(resourceId),
@@ -735,16 +722,14 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
       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 (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",
@@ -763,7 +748,7 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
 
     /**
      * 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.
      */
@@ -771,9 +756,6 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
     key: "scheduleCacheDeletion",
     value: function scheduleCacheDeletion(resourceId) {
       var _this6 = this;
-      if (!fg('platform_synced_block_patch_14')) {
-        return;
-      }
       if (this.isDestroyed) {
         return;
       }
@@ -801,15 +783,12 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
 
     /**
      * 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 (!fg('platform_synced_block_patch_14')) {
-        return;
-      }
       var existing = this.pendingCacheDeletions.get(resourceId);
       if (existing) {
         clearTimeout(existing);
@@ -831,7 +810,7 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
     value: function onCacheDeletionTimerFire(resourceId) {
       var _this$cacheDeletionRe;
       if (this.canDeleteCache(resourceId)) {
-        // `deleteFromCache` resets the reschedule counter under the flag.
+        // `deleteFromCache` resets the reschedule counter.
         this.deleteFromCache(resourceId);
         return;
       }
@@ -851,7 +830,7 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
         logException(new Error("Cache deletion forced after ".concat(currentCount, " reschedules \u2014 stuck in-flight guard")), {
           location: 'editor-synced-block-provider/referenceSyncBlockStoreManager/cache-deletion-forced'
         });
-        // `deleteFromCache` resets the reschedule counter under the flag.
+        // `deleteFromCache` resets the reschedule counter.
         this.deleteFromCache(resourceId);
         // If subscribers still exist, kick off a fresh fetch so they get
         // fresh data on the next batch tick instead of holding stale data
@@ -961,14 +940,14 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
         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 (isProviderNotReadyError(error) && fg('platform_editor_blocks_patch_3')) {
+        // benign case stays fully silent).
+        if (isProviderNotReadyError(error)) {
           return function () {};
         }
         logException(error, {
           location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
         });
-        (_this$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 || _this$fireAnalyticsEv7.call(this, fetchErrorPayload(error.message, undefined, undefined, buildFetchErrorAttribution(fg('platform_editor_blocks_patch_3'), error.message)));
+        (_this$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 || _this$fireAnalyticsEv7.call(this, fetchErrorPayload(error.message, undefined, undefined, buildFetchErrorAttribution(error.message)));
         return function () {};
       }
     }
@@ -1131,7 +1110,7 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
                 (_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, updateReferenceErrorPayload(updateResult.error || 'Failed to update reference synced blocks on the document', undefined, undefined, buildErrorAttribution(fg('platform_editor_blocks_patch_3'), updateResult.error, updateResult.statusCode)));
+                (_this$fireAnalyticsEv8 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv8 === void 0 || _this$fireAnalyticsEv8.call(this, updateReferenceErrorPayload(updateResult.error || 'Failed to update reference synced blocks on the document', undefined, undefined, buildErrorAttribution(updateResult.error, updateResult.statusCode)));
               }
               _context4.next = 17;
               break;
@@ -1147,8 +1126,8 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
               });
               // 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, updateReferenceErrorPayload(_t3.message, undefined, undefined, buildErrorAttribution(fg('platform_editor_blocks_patch_3'))));
+              // to `unknown`.
+              (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 || _this$fireAnalyticsEv9.call(this, updateReferenceErrorPayload(_t3.message, undefined, undefined, buildErrorAttribution()));
             case 17:
               _context4.prev = 17;
               if (!success) {
@@ -1230,17 +1209,9 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
       });
       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 (!fg('platform_synced_block_patch_14')) {
-        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/esm/store-manager/sourceSyncBlockStoreManager.js

@@ -10,7 +10,6 @@ function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length)
 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) { _defineProperty(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; }
 import { logException } from '@atlaskit/editor-common/monitoring';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { SyncBlockError } from '../common/types';
 import { updateErrorPayload, createErrorPayload, deleteErrorPayload, updateCacheErrorPayload, getSourceInfoErrorPayload, updateSuccessPayload, createSuccessPayload, createSuccessOperationalPayload, addContentSuccessPayload, deleteSuccessPayload, fetchReferencesErrorPayload, buildErrorAttribution } from '../utils/errorHandling';
 import { getCreateSourceExperience, getDeleteSourceExperience, getSaveSourceExperience, getFetchSourceInfoExperience } from '../utils/experienceTracking';
@@ -75,8 +74,7 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
     _defineProperty(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}.
      */
     _defineProperty(this, "recentDeleteEmissions", new Map());
     _defineProperty(this, "setPendingDeletion", function (Ids, value) {
@@ -162,10 +160,7 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
           resourceId = _syncBlockNode$attrs.resourceId;
         if (!localId || !resourceId) {
           // Identifiers not populated yet: benign timing case, skip as a no-op.
-          if (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);
 
@@ -208,7 +203,7 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
     value: (function () {
       var _flush = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.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 _regeneratorRuntime.wrap(function (_context) {
           while (1) switch (_context.prev = _context.next) {
             case 0:
@@ -327,12 +322,11 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
               return _context.abrupt("return", true);
             case 7:
               (_this$saveExperience3 = this.saveExperience) === null || _this$saveExperience3 === void 0 || _this$saveExperience3.failure();
-              attributionEnabled = 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, updateErrorPayload(result.error || 'Failed to write data', result.resourceId, getSourceProductFromResourceIdSafe(result.resourceId), buildErrorAttribution(attributionEnabled, result.error, result.statusCode)));
+                (_this2$fireAnalyticsE2 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE2 === void 0 || _this2$fireAnalyticsE2.call(_this2, updateErrorPayload(result.error || 'Failed to write data', result.resourceId, getSourceProductFromResourceIdSafe(result.resourceId), buildErrorAttribution(result.error, result.statusCode)));
               });
               return _context.abrupt("return", false);
             case 8:
@@ -346,7 +340,7 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
               });
               // 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, updateErrorPayload(_t.message, undefined, undefined, buildErrorAttribution(fg('platform_editor_blocks_patch_3'))));
+              (_this$fireAnalyticsEv2 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv2 === void 0 || _this$fireAnalyticsEv2.call(this, updateErrorPayload(_t.message, undefined, undefined, buildErrorAttribution()));
               return _context.abrupt("return", false);
             case 10:
               _context.prev = 10;
@@ -440,15 +434,12 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
         (_this$fireAnalyticsEv3 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv3 === void 0 || _this$fireAnalyticsEv3.call(this, createErrorPayload('creation complete callback missing', resourceId, getSourceProductFromResourceIdSafe(resourceId)));
       }
       if (success) {
-        var _this$fireAnalyticsEv4;
+        var _this$fireAnalyticsEv4, _this$fireAnalyticsEv5, _this$syncBlockCache$2;
         var sourceProduct = getSourceProductFromResourceIdSafe(resourceId);
         (_this$fireAnalyticsEv4 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv4 === void 0 || _this$fireAnalyticsEv4.call(this, createSuccessPayload(resourceId || '', sourceProduct));
         // Operational create-success event with the join key + creation-type
         // signals (inputMethod / createdEmpty) captured at the command layer.
-        if (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, 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, 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
@@ -467,7 +458,6 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
      * 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",
@@ -476,9 +466,6 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
       if (this.viewMode === 'view') {
         return;
       }
-      if (!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.
@@ -582,7 +569,7 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
             (_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, createErrorPayload(result.error || 'Failed to create bodied sync block', resourceId, getSourceProductFromResourceIdSafe(resourceId), buildErrorAttribution(fg('platform_editor_blocks_patch_3'), result.error, result.statusCode)));
+            (_this4$fireAnalyticsE = _this4.fireAnalyticsEvent) === null || _this4$fireAnalyticsE === void 0 || _this4$fireAnalyticsE.call(_this4, createErrorPayload(result.error || 'Failed to create bodied sync block', resourceId, getSourceProductFromResourceIdSafe(resourceId), buildErrorAttribution(result.error, result.statusCode)));
           }
         }).catch(function (error) {
           var _this4$createExperien3, _this4$fireAnalyticsE2;
@@ -637,21 +624,16 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
     }
 
     /**
-     * Emit the `syncedBlockDelete` success event. Gate off: legacy payload. Gate
-     * on: attaches `deletionReason`, `mechanism` and `blockInstanceId`, and
-     * suppresses repeat emissions within {@link DELETE_DEDUPE_WINDOW_MS}. Must run
-     * while the cache entry still exists so `blockInstanceId` is available.
+     * Emit the `syncedBlockDelete` success event. Attaches `deletionReason`,
+     * `mechanism` and `blockInstanceId`, and suppresses repeat emissions within
+     * {@link DELETE_DEDUPE_WINDOW_MS}. Must run while the cache entry still exists
+     * so `blockInstanceId` is available.
      */
   }, {
     key: "emitDeleteSuccess",
     value: function emitDeleteSuccess(resourceId, reason, mechanism) {
-      var _this$syncBlockCache$3, _this$fireAnalyticsEv0;
+      var _this$syncBlockCache$3, _this$fireAnalyticsEv9;
       var sourceProduct = getSourceProductFromResourceIdSafe(resourceId);
-      if (!fg('platform_editor_blocks_patch_4')) {
-        var _this$fireAnalyticsEv9;
-        (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 || _this$fireAnalyticsEv9.call(this, deleteSuccessPayload(resourceId, sourceProduct));
-        return;
-      }
       var now = Date.now();
       var lastEmittedAt = this.recentDeleteEmissions.get(resourceId);
       if (lastEmittedAt !== undefined && now - lastEmittedAt < DELETE_DEDUPE_WINDOW_MS) {
@@ -664,14 +646,14 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
         deletionReason: reason,
         mechanism: mechanism
       };
-      (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 || _this$fireAnalyticsEv0.call(this, deleteSuccessPayload(resourceId, sourceProduct, enrichment));
+      (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 || _this$fireAnalyticsEv9.call(this, deleteSuccessPayload(resourceId, sourceProduct, enrichment));
     }
   }, {
     key: "delete",
     value: function () {
       var _delete2 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.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 _regeneratorRuntime.wrap(function (_context2) {
           while (1) switch (_context2.prev = _context2.next) {
             case 0:
@@ -719,13 +701,12 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
                   _this5.setPendingDeletion(Ids, false);
                 };
                 (_this$deleteExperienc3 = this.deleteExperience) === null || _this$deleteExperienc3 === void 0 || _this$deleteExperienc3.failure();
-                attributionEnabled = 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, deleteErrorPayload(result.error || 'Failed to delete synced block', result.resourceId, getSourceProductFromResourceIdSafe(result.resourceId), buildErrorAttribution(attributionEnabled, result.error, result.statusCode)));
+                    (_this5$fireAnalyticsE = _this5.fireAnalyticsEvent) === null || _this5$fireAnalyticsE === void 0 || _this5$fireAnalyticsE.call(_this5, deleteErrorPayload(result.error || 'Failed to delete synced block', result.resourceId, getSourceProductFromResourceIdSafe(result.resourceId), buildErrorAttribution(result.error, result.statusCode)));
                   }
                 });
               }
@@ -735,8 +716,8 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
               _context2.prev = 4;
               _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 = buildErrorAttribution(fg('platform_editor_blocks_patch_3'));
+              // so the attribution `reason` falls back to `unknown`.
+              attribution = buildErrorAttribution();
               syncBlockIds.forEach(function (Ids) {
                 var _this5$fireAnalyticsE2;
                 _this5.setPendingDeletion(Ids, false);
@@ -1013,11 +994,11 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
           return sourceInfo;
         });
       } catch (error) {
-        var _this$fireAnalyticsEv1;
+        var _this$fireAnalyticsEv0;
         logException(error, {
           location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
         });
-        (_this$fireAnalyticsEv1 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv1 === void 0 || _this$fireAnalyticsEv1.call(this, getSourceInfoErrorPayload(error.message));
+        (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 || _this$fireAnalyticsEv0.call(this, getSourceInfoErrorPayload(error.message));
         return Promise.resolve(undefined);
       }
     }
@@ -1030,11 +1011,11 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
         }
         return this.dataProvider.fetchReferences(resourceId, true);
       } catch (error) {
-        var _this$fireAnalyticsEv10;
+        var _this$fireAnalyticsEv1;
         logException(error, {
           location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
         });
-        (_this$fireAnalyticsEv10 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv10 === void 0 || _this$fireAnalyticsEv10.call(this, fetchReferencesErrorPayload(error.message, resourceId, getSourceProductFromResourceIdSafe(resourceId)));
+        (_this$fireAnalyticsEv1 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv1 === void 0 || _this$fireAnalyticsEv1.call(this, fetchReferencesErrorPayload(error.message, resourceId, getSourceProductFromResourceIdSafe(resourceId)));
         return Promise.resolve({
           error: SyncBlockError.Errored
         });

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

@@ -3,7 +3,6 @@ import _createClass from "@babel/runtime/helpers/createClass";
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import rafSchedule from 'raf-schd';
 import { logException } from '@atlaskit/editor-common/monitoring';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { isProviderNotReadyError } from '../common/types';
 import { buildFetchErrorAttribution, fetchErrorPayload } from '../utils/errorHandling';
 import { createSyncBlockNode, getSourceProductFromResourceIdSafe } from '../utils/utils';
@@ -29,14 +28,9 @@ export var SyncBlockBatchFetcher = /*#__PURE__*/function () {
       }
 
       // 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 (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) {
@@ -55,8 +49,8 @@ export var SyncBlockBatchFetcher = /*#__PURE__*/function () {
         // 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 (isProviderNotReadyError(error) && fg('platform_editor_blocks_patch_3')) {
+        // `queueFetch()` arrives.
+        if (isProviderNotReadyError(error)) {
           resourceIds.forEach(function (resId) {
             return _this.pendingFetchRequests.add(resId);
           });
@@ -68,7 +62,7 @@ export var SyncBlockBatchFetcher = /*#__PURE__*/function () {
         logException(error, {
           location: 'editor-synced-block-provider/syncBlockBatchFetcher/batchedFetchSyncBlocks'
         });
-        var attribution = buildFetchErrorAttribution(fg('platform_editor_blocks_patch_3'), error.message);
+        var attribution = 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(fetchErrorPayload(error.message, resId, getSourceProductFromResourceIdSafe(resId), attribution));

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

@@ -28,7 +28,7 @@ export var SyncBlockProviderFactoryManager = /*#__PURE__*/function () {
         logException(error, {
           location: 'editor-synced-block-provider/syncBlockProviderFactoryManager'
         });
-        (_this$deps$getFireAna = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna === void 0 || _this$deps$getFireAna(fetchErrorPayload(error.message, resourceId, getSourceProductFromResourceIdSafe(resourceId), buildFetchErrorAttribution(fg('platform_editor_blocks_patch_3'), error.message)));
+        (_this$deps$getFireAna = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna === void 0 || _this$deps$getFireAna(fetchErrorPayload(error.message, resourceId, getSourceProductFromResourceIdSafe(resourceId), buildFetchErrorAttribution(error.message)));
         return undefined;
       }
       var providerFactory = this.providerFactories.get(resourceId);
@@ -104,7 +104,7 @@ export var SyncBlockProviderFactoryManager = /*#__PURE__*/function () {
         logException(error, {
           location: 'editor-synced-block-provider/syncBlockProviderFactoryManager'
         });
-        (_this$deps$getFireAna2 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna2 === void 0 || _this$deps$getFireAna2(fetchErrorPayload(error.message, resourceId, getSourceProductFromResourceIdSafe(resourceId), buildFetchErrorAttribution(fg('platform_editor_blocks_patch_3'), error.message)));
+        (_this$deps$getFireAna2 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna2 === void 0 || _this$deps$getFireAna2(fetchErrorPayload(error.message, resourceId, getSourceProductFromResourceIdSafe(resourceId), buildFetchErrorAttribution(error.message)));
       }
     }
   }, {
@@ -187,7 +187,7 @@ export var SyncBlockProviderFactoryManager = /*#__PURE__*/function () {
       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(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 : getSourceProductFromResourceIdSafe(resourceId), buildFetchErrorAttribution(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 : getSourceProductFromResourceIdSafe(resourceId), buildFetchErrorAttribution('Sync block source ari or product not found')));
         return;
       }
       var parentInfo = dataProvider.retrieveSyncBlockParentInfo(syncBlock.data.sourceAri, syncBlock.data.product);

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

@@ -7,7 +7,6 @@ function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbol
 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) { _defineProperty(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; }
 import { useEffect, useMemo, useRef } from 'react';
 import { logException } from '@atlaskit/editor-common/monitoring';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { getProductFromSourceAri } from '../clients/block-service/ari';
 import { SyncBlockError } from '../common/types';
 import { fetchReferencesErrorPayload } from '../utils/errorHandling';
@@ -303,7 +302,6 @@ export var useMemoizedSyncBlockStoreManager = function useMemoizedSyncBlockStore
     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
@@ -315,9 +313,7 @@ export var useMemoizedSyncBlockStoreManager = function useMemoizedSyncBlockStore
   // changes, triggering the cleanup for the old instance.
   useEffect(function () {
     return function () {
-      if (fg('platform_synced_block_patch_14')) {
-        syncBlockStoreManager.destroy();
-      }
+      syncBlockStoreManager.destroy();
     };
   }, [syncBlockStoreManager]);
   return syncBlockStoreManager;