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

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

@@ -1,6 +1,5 @@
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
 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';
@@ -63,8 +62,7 @@ export class SourceSyncBlockStoreManager {
     _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", (Ids, value) => {
@@ -137,10 +135,7 @@ export class SourceSyncBlockStoreManager {
       } = syncBlockNode.attrs;
       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;
       }
       const cachedBlock = this.syncBlockCache.get(resourceId);
 
@@ -282,10 +277,9 @@ export class SourceSyncBlockStoreManager {
       } else {
         var _this$saveExperience3;
         (_this$saveExperience3 = this.saveExperience) === null || _this$saveExperience3 === void 0 ? void 0 : _this$saveExperience3.failure();
-        const attributionEnabled = fg('platform_editor_blocks_patch_3');
         writeResults.filter(result => !result.resourceId || result.error).forEach(result => {
           var _this$fireAnalyticsEv3;
-          (_this$fireAnalyticsEv3 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv3 === void 0 ? void 0 : _this$fireAnalyticsEv3.call(this, updateErrorPayload(result.error || 'Failed to write data', result.resourceId, getSourceProductFromResourceIdSafe(result.resourceId), buildErrorAttribution(attributionEnabled, result.error, result.statusCode)));
+          (_this$fireAnalyticsEv3 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv3 === void 0 ? void 0 : _this$fireAnalyticsEv3.call(this, updateErrorPayload(result.error || 'Failed to write data', result.resourceId, getSourceProductFromResourceIdSafe(result.resourceId), buildErrorAttribution(result.error, result.statusCode)));
         });
         return false;
       }
@@ -296,7 +290,7 @@ export class SourceSyncBlockStoreManager {
       });
       // 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$fireAnalyticsEv4 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv4 === void 0 ? void 0 : _this$fireAnalyticsEv4.call(this, updateErrorPayload(error.message, undefined, undefined, buildErrorAttribution(fg('platform_editor_blocks_patch_3'))));
+      (_this$fireAnalyticsEv4 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv4 === void 0 ? void 0 : _this$fireAnalyticsEv4.call(this, updateErrorPayload(error.message, undefined, undefined, buildErrorAttribution()));
       return false;
     } finally {
       var _this$flushCompletion;
@@ -368,15 +362,12 @@ export class SourceSyncBlockStoreManager {
       (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 ? void 0 : _this$fireAnalyticsEv5.call(this, createErrorPayload('creation complete callback missing', resourceId, getSourceProductFromResourceIdSafe(resourceId)));
     }
     if (success) {
-      var _this$fireAnalyticsEv6;
+      var _this$fireAnalyticsEv6, _this$fireAnalyticsEv7, _this$syncBlockCache$2;
       const sourceProduct = getSourceProductFromResourceIdSafe(resourceId);
       (_this$fireAnalyticsEv6 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv6 === void 0 ? void 0 : _this$fireAnalyticsEv6.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$fireAnalyticsEv7, _this$syncBlockCache$2;
-        (_this$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 ? void 0 : _this$fireAnalyticsEv7.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$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 ? void 0 : _this$fireAnalyticsEv7.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$fireAnalyticsEv8;
       // Delete the node from cache if fail to create so it's not flushed to BE
@@ -395,16 +386,12 @@ export class SourceSyncBlockStoreManager {
    * 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`.
    */
   maybeEmitFirstContentAdded(resourceId, blockInstanceId) {
     var _this$fireAnalyticsEv9;
     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.
@@ -501,7 +488,7 @@ export class SourceSyncBlockStoreManager {
           (_this$createExperienc3 = this.createExperience) === null || _this$createExperienc3 === void 0 ? void 0 : _this$createExperienc3.failure({
             reason: result.error || 'Failed to create bodied sync block'
           });
-          (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 ? void 0 : _this$fireAnalyticsEv0.call(this, createErrorPayload(result.error || 'Failed to create bodied sync block', resourceId, getSourceProductFromResourceIdSafe(resourceId), buildErrorAttribution(fg('platform_editor_blocks_patch_3'), result.error, result.statusCode)));
+          (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 ? void 0 : _this$fireAnalyticsEv0.call(this, createErrorPayload(result.error || 'Failed to create bodied sync block', resourceId, getSourceProductFromResourceIdSafe(resourceId), buildErrorAttribution(result.error, result.statusCode)));
         }
       }).catch(error => {
         var _this$createExperienc4, _this$fireAnalyticsEv1;
@@ -542,19 +529,14 @@ export class SourceSyncBlockStoreManager {
   }
 
   /**
-   * 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.
    */
   emitDeleteSuccess(resourceId, reason, mechanism) {
-    var _this$syncBlockCache$3, _this$fireAnalyticsEv12;
+    var _this$syncBlockCache$3, _this$fireAnalyticsEv11;
     const sourceProduct = getSourceProductFromResourceIdSafe(resourceId);
-    if (!fg('platform_editor_blocks_patch_4')) {
-      var _this$fireAnalyticsEv11;
-      (_this$fireAnalyticsEv11 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv11 === void 0 ? void 0 : _this$fireAnalyticsEv11.call(this, deleteSuccessPayload(resourceId, sourceProduct));
-      return;
-    }
     const now = Date.now();
     const lastEmittedAt = this.recentDeleteEmissions.get(resourceId);
     if (lastEmittedAt !== undefined && now - lastEmittedAt < DELETE_DEDUPE_WINDOW_MS) {
@@ -567,7 +549,7 @@ export class SourceSyncBlockStoreManager {
       deletionReason: reason,
       mechanism
     };
-    (_this$fireAnalyticsEv12 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv12 === void 0 ? void 0 : _this$fireAnalyticsEv12.call(this, deleteSuccessPayload(resourceId, sourceProduct, enrichment));
+    (_this$fireAnalyticsEv11 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv11 === void 0 ? void 0 : _this$fireAnalyticsEv11.call(this, deleteSuccessPayload(resourceId, sourceProduct, enrichment));
   }
   async delete(syncBlockIds, onDelete, onDeleteCompleted, reason, mechanism) {
     try {
@@ -603,13 +585,12 @@ export class SourceSyncBlockStoreManager {
           this.setPendingDeletion(Ids, false);
         };
         (_this$deleteExperienc3 = this.deleteExperience) === null || _this$deleteExperienc3 === void 0 ? void 0 : _this$deleteExperienc3.failure();
-        const attributionEnabled = fg('platform_editor_blocks_patch_3');
         results.forEach(result => {
           if (result.success) {
             this.emitDeleteSuccess(result.resourceId, reason, mechanism);
           } else {
-            var _this$fireAnalyticsEv13;
-            (_this$fireAnalyticsEv13 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv13 === void 0 ? void 0 : _this$fireAnalyticsEv13.call(this, deleteErrorPayload(result.error || 'Failed to delete synced block', result.resourceId, getSourceProductFromResourceIdSafe(result.resourceId), buildErrorAttribution(attributionEnabled, result.error, result.statusCode)));
+            var _this$fireAnalyticsEv12;
+            (_this$fireAnalyticsEv12 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv12 === void 0 ? void 0 : _this$fireAnalyticsEv12.call(this, deleteErrorPayload(result.error || 'Failed to delete synced block', result.resourceId, getSourceProductFromResourceIdSafe(result.resourceId), buildErrorAttribution(result.error, result.statusCode)));
           }
         });
       }
@@ -617,12 +598,12 @@ export class SourceSyncBlockStoreManager {
       return isDeleteSuccessful;
     } catch (error) {
       // Thrown (non-result) failure — no structured SyncBlockError/status is available,
-      // so the attribution `reason` falls back to `unknown` when the gate is on.
-      const attribution = buildErrorAttribution(fg('platform_editor_blocks_patch_3'));
+      // so the attribution `reason` falls back to `unknown`.
+      const attribution = buildErrorAttribution();
       syncBlockIds.forEach(Ids => {
-        var _this$fireAnalyticsEv14;
+        var _this$fireAnalyticsEv13;
         this.setPendingDeletion(Ids, false);
-        (_this$fireAnalyticsEv14 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv14 === void 0 ? void 0 : _this$fireAnalyticsEv14.call(this, deleteErrorPayload(error.message, Ids.resourceId, getSourceProductFromResourceIdSafe(Ids.resourceId), attribution));
+        (_this$fireAnalyticsEv13 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv13 === void 0 ? void 0 : _this$fireAnalyticsEv13.call(this, deleteErrorPayload(error.message, Ids.resourceId, getSourceProductFromResourceIdSafe(Ids.resourceId), attribution));
       });
       logException(error, {
         location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
@@ -782,11 +763,11 @@ export class SourceSyncBlockStoreManager {
         return sourceInfo;
       });
     } catch (error) {
-      var _this$fireAnalyticsEv15;
+      var _this$fireAnalyticsEv14;
       logException(error, {
         location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
       });
-      (_this$fireAnalyticsEv15 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv15 === void 0 ? void 0 : _this$fireAnalyticsEv15.call(this, getSourceInfoErrorPayload(error.message));
+      (_this$fireAnalyticsEv14 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv14 === void 0 ? void 0 : _this$fireAnalyticsEv14.call(this, getSourceInfoErrorPayload(error.message));
       return Promise.resolve(undefined);
     }
   }
@@ -797,11 +778,11 @@ export class SourceSyncBlockStoreManager {
       }
       return this.dataProvider.fetchReferences(resourceId, true);
     } catch (error) {
-      var _this$fireAnalyticsEv16;
+      var _this$fireAnalyticsEv15;
       logException(error, {
         location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
       });
-      (_this$fireAnalyticsEv16 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv16 === void 0 ? void 0 : _this$fireAnalyticsEv16.call(this, fetchReferencesErrorPayload(error.message, resourceId, getSourceProductFromResourceIdSafe(resourceId)));
+      (_this$fireAnalyticsEv15 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv15 === void 0 ? void 0 : _this$fireAnalyticsEv15.call(this, fetchReferencesErrorPayload(error.message, resourceId, getSourceProductFromResourceIdSafe(resourceId)));
       return Promise.resolve({
         error: SyncBlockError.Errored
       });

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

@@ -1,7 +1,6 @@
 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';
@@ -25,14 +24,9 @@ export class SyncBlockBatchFetcher {
       }
 
       // 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;
       }
       const resourceIds = Array.from(this.pendingFetchRequests);
       const syncBlockNodes = resourceIds.map(resId => {
@@ -49,8 +43,8 @@ export class SyncBlockBatchFetcher {
         // 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(resId => this.pendingFetchRequests.add(resId));
           if (!this.isDestroyed) {
             this.scheduledBatchFetch();
@@ -60,7 +54,7 @@ export class SyncBlockBatchFetcher {
         logException(error, {
           location: 'editor-synced-block-provider/syncBlockBatchFetcher/batchedFetchSyncBlocks'
         });
-        const attribution = buildFetchErrorAttribution(fg('platform_editor_blocks_patch_3'), error.message);
+        const attribution = buildFetchErrorAttribution(error.message);
         resourceIds.forEach(resId => {
           var _this$deps$getFireAna;
           (_this$deps$getFireAna = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna === void 0 ? void 0 : _this$deps$getFireAna(fetchErrorPayload(error.message, resId, getSourceProductFromResourceIdSafe(resId), attribution));

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

@@ -23,7 +23,7 @@ export class SyncBlockProviderFactoryManager {
       logException(error, {
         location: 'editor-synced-block-provider/syncBlockProviderFactoryManager'
       });
-      (_this$deps$getFireAna = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna === void 0 ? 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 ? void 0 : _this$deps$getFireAna(fetchErrorPayload(error.message, resourceId, getSourceProductFromResourceIdSafe(resourceId), buildFetchErrorAttribution(error.message)));
       return undefined;
     }
     let providerFactory = this.providerFactories.get(resourceId);
@@ -97,7 +97,7 @@ export class SyncBlockProviderFactoryManager {
       logException(error, {
         location: 'editor-synced-block-provider/syncBlockProviderFactoryManager'
       });
-      (_this$deps$getFireAna2 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna2 === void 0 ? 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 ? void 0 : _this$deps$getFireAna2(fetchErrorPayload(error.message, resourceId, getSourceProductFromResourceIdSafe(resourceId), buildFetchErrorAttribution(error.message)));
     }
   }
   getSSRProviders(resourceId) {
@@ -173,7 +173,7 @@ export class 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 ? 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;
     }
     const parentInfo = dataProvider.retrieveSyncBlockParentInfo(syncBlock.data.sourceAri, syncBlock.data.product);

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

@@ -1,6 +1,5 @@
 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';
@@ -209,7 +208,6 @@ export const useMemoizedSyncBlockStoreManager = (dataProvider, fireAnalyticsEven
     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
@@ -221,9 +219,7 @@ export const useMemoizedSyncBlockStoreManager = (dataProvider, fireAnalyticsEven
   // changes, triggering the cleanup for the old instance.
   useEffect(() => {
     return () => {
-      if (fg('platform_synced_block_patch_14')) {
-        syncBlockStoreManager.destroy();
-      }
+      syncBlockStoreManager.destroy();
     };
   }, [syncBlockStoreManager]);
   return syncBlockStoreManager;

package/dist/es2019/store-manager/syncBlockSubscriptionManager.js

@@ -2,7 +2,6 @@ import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import { bind } from 'bind-event-listener';
 import { getDocument } from '@atlaskit/browser-apis';
 import { logException } from '@atlaskit/editor-common/monitoring';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { buildFetchErrorAttribution, fetchErrorPayload, fetchSuccessPayload } from '../utils/errorHandling';
 import { resolveSyncBlockInstance } from '../utils/resolveSyncBlockInstance';
 import { getSourceProductFromResourceIdSafe } from '../utils/utils';
@@ -17,19 +16,15 @@ export class SyncBlockSubscriptionManager {
   // EDITOR-7861: higher ceiling lets transient WS-gateway drops self-heal
   // before a terminal failure is surfaced.
   getMaxRetryAttempts() {
-    return fg('platform_editor_blocks_patch_3') ? SyncBlockSubscriptionManager.MAX_RETRY_ATTEMPTS_HARDENED : SyncBlockSubscriptionManager.MAX_RETRY_ATTEMPTS;
+    return SyncBlockSubscriptionManager.MAX_RETRY_ATTEMPTS_HARDENED;
   }
 
-  // Backoff delay for the given attempt.
-  // Gate OFF: pure exponential (1s, 2s, 4s, 8s, 16s).
-  // Gate ON (EDITOR-7861): exponential capped at MAX_RETRY_DELAY_MS with equal
-  // jitter (capped/2 + random*capped/2) — de-synchronises simultaneous
-  // reconnects while guaranteeing a non-zero delay (full jitter could hit 0).
+  // Backoff delay for the given attempt (EDITOR-7861): exponential capped at
+  // MAX_RETRY_DELAY_MS with equal jitter (capped/2 + random*capped/2) —
+  // de-synchronises simultaneous reconnects while guaranteeing a non-zero delay
+  // (full jitter could hit 0).
   getReconnectionDelay(attempts) {
     const exponential = SyncBlockSubscriptionManager.INITIAL_RETRY_DELAY_MS * Math.pow(SyncBlockSubscriptionManager.RETRY_BACKOFF_MULTIPLIER, attempts);
-    if (!fg('platform_editor_blocks_patch_3')) {
-      return exponential;
-    }
     const half = Math.min(exponential, SyncBlockSubscriptionManager.MAX_RETRY_DELAY_MS) / 2;
     return Math.round(half + Math.random() * half);
   }
@@ -39,12 +34,7 @@ export class SyncBlockSubscriptionManager {
     _defineProperty(this, "graphqlSubscriptions", new Map());
     _defineProperty(this, "subscriptionChangeListeners", new Set());
     _defineProperty(this, "useRealTimeSubscriptions", false);
-    // Track pending cache deletions to handle block moves (unmount/remount)
-    // When a block is moved, the old component unmounts before the new one mounts,
-    // causing the cache to be deleted prematurely. We delay deletion to allow
-    // the new component to subscribe and cancel the pending deletion.
-    _defineProperty(this, "pendingCacheDeletions", new Map());
-    // backoff cap (gate ON)
+    // backoff cap
     _defineProperty(this, "retryAttempts", new Map());
     _defineProperty(this, "pendingRetries", new Map());
     // Resources whose reconnection exhausted while the tab was hidden: parked here and
@@ -105,7 +95,7 @@ export class SyncBlockSubscriptionManager {
         logException(error, {
           location: 'editor-synced-block-provider/syncBlockSubscriptionManager/notifySubscriptionChangeListeners'
         });
-        (_this$deps$getFireAna = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna === void 0 ? void 0 : _this$deps$getFireAna(fetchErrorPayload(error.message, undefined, undefined, buildFetchErrorAttribution(fg('platform_editor_blocks_patch_3'), error.message)));
+        (_this$deps$getFireAna = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna === void 0 ? void 0 : _this$deps$getFireAna(fetchErrorPayload(error.message, undefined, undefined, buildFetchErrorAttribution(error.message)));
       }
     });
   }
@@ -125,16 +115,7 @@ export class SyncBlockSubscriptionManager {
     // This handles the case where a block is moved - the old component unmounts
     // (scheduling deletion) but the new component mounts and subscribes before
     // the deletion timeout fires.
-    //
-    // Under the flag, cache deletion is owned by the store manager.
-    // With the flag off, the legacy 1s timer path is preserved.
-    const pendingDeletion = this.pendingCacheDeletions.get(resourceId);
-    if (fg('platform_synced_block_patch_14')) {
-      this.deps.cancelPendingCacheDeletion(resourceId);
-    } else if (pendingDeletion) {
-      clearTimeout(pendingDeletion);
-      this.pendingCacheDeletions.delete(resourceId);
-    }
+    this.deps.cancelPendingCacheDeletion(resourceId);
 
     // add to subscriptions map
     const resourceSubscriptions = this.subscriptions.get(resourceId) || {};
@@ -178,30 +159,9 @@ export class SyncBlockSubscriptionManager {
           // Notify listeners that subscription was removed
           this.notifySubscriptionChangeListeners();
 
-          // Under the flag, delegate cache deletion to the store manager
+          // Delegate cache deletion to the store manager
           // which uses a 30s grace period with guard re-checks.
-          if (fg('platform_synced_block_patch_14')) {
-            this.deps.scheduleCacheDeletion(resourceId);
-          } else {
-            // Legacy path (unchanged): delay cache deletion to handle
-            // block moves (unmount/remount). When a block is moved, the
-            // old component unmounts before the new one mounts. By
-            // delaying deletion, we give the new component time to
-            // subscribe and cancel this pending deletion, preserving
-            // the cached data.
-            // TODO: EDITOR-4152 - Rework this logic (superseded by
-            // `platform_synced_block_patch_14`).
-            const deletionTimeout = setTimeout(() => {
-              const hasSubscribers = this.subscriptions.has(resourceId);
-
-              // Only delete if still no subscribers (wasn't re-subscribed)
-              if (!hasSubscribers) {
-                this.deps.deleteFromCache(resourceId);
-              }
-              this.pendingCacheDeletions.delete(resourceId);
-            }, 1000);
-            this.pendingCacheDeletions.set(resourceId, deletionTimeout);
-          }
+          this.deps.scheduleCacheDeletion(resourceId);
         } else {
           this.subscriptions.set(resourceId, resourceSubscriptions);
         }
@@ -278,17 +238,10 @@ export class SyncBlockSubscriptionManager {
       logException(error, {
         location: 'editor-synced-block-provider/syncBlockSubscriptionManager/graphql-subscription'
       });
-      // EDITOR-7861: a single socket drop is usually transient and
-      // recovers on reconnect, so under the gate we don't fire a
-      // user-facing error here — it's only surfaced on exhaustion (see
-      // scheduleReconnection). Gate OFF keeps the legacy fire-on-drop.
-      // This branch only runs when the gate is OFF, so buildFetchErrorAttribution
-      // would return undefined; the structured attribution (EDITOR-7862) is therefore
-      // applied at the gate-ON exhaustion site in scheduleReconnection instead.
-      if (!fg('platform_editor_blocks_patch_3')) {
-        var _this$deps$getFireAna2;
-        (_this$deps$getFireAna2 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna2 === void 0 ? void 0 : _this$deps$getFireAna2(fetchErrorPayload(error.message, resourceId, getSourceProductFromResourceIdSafe(resourceId)));
-      }
+      // EDITOR-7861: a single socket drop is usually transient and recovers
+      // on reconnect, so we don't fire a user-facing error here — it's only
+      // surfaced on exhaustion, where the structured attribution
+      // (EDITOR-7862) is applied too. See scheduleReconnection.
       this.handleSubscriptionTerminated(resourceId);
     }, () => {
       this.handleSubscriptionTerminated(resourceId);
@@ -324,26 +277,25 @@ export class SyncBlockSubscriptionManager {
     const attempts = (_this$retryAttempts$g = this.retryAttempts.get(resourceId)) !== null && _this$retryAttempts$g !== void 0 ? _this$retryAttempts$g : 0;
     const maxAttempts = this.getMaxRetryAttempts();
     if (attempts >= maxAttempts) {
-      var _this$deps$getFireAna4;
+      var _this$deps$getFireAna3;
       // Exhausted all attempts — the only place a WS drop surfaces as a
-      // fetch error under the gate (EDITOR-7861).
+      // fetch error (EDITOR-7861).
       const errorMessage = `Subscription reconnection failed after ${attempts} attempts`;
 
       // Tab hidden at exhaustion: don't surface a terminal failure (user isn't
       // looking, and most exhaustions self-recover once foregrounded). Park + re-arm
       // on wake, emitting a benign `deferred` signal so suppression stays auditable.
-      const shouldDefer = fg('platform_editor_blocks_patch_3') && this.isDocumentHidden();
-      if (shouldDefer) {
-        var _this$deps$getFireAna3;
+      if (this.isDocumentHidden()) {
+        var _this$deps$getFireAna2;
         this.deferredExhausted.add(resourceId);
         this.registerWakeListeners();
-        (_this$deps$getFireAna3 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna3 === void 0 ? void 0 : _this$deps$getFireAna3(fetchErrorPayload(errorMessage, resourceId, getSourceProductFromResourceIdSafe(resourceId), buildFetchErrorAttribution(true, errorMessage, undefined, /* deferred */true)));
+        (_this$deps$getFireAna2 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna2 === void 0 ? void 0 : _this$deps$getFireAna2(fetchErrorPayload(errorMessage, resourceId, getSourceProductFromResourceIdSafe(resourceId), buildFetchErrorAttribution(errorMessage, undefined, /* deferred */true)));
         return;
       }
       logException(new Error(errorMessage), {
         location: 'editor-synced-block-provider/syncBlockSubscriptionManager/max-retries-exhausted'
       });
-      (_this$deps$getFireAna4 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna4 === void 0 ? void 0 : _this$deps$getFireAna4(fetchErrorPayload(errorMessage, resourceId, getSourceProductFromResourceIdSafe(resourceId), buildFetchErrorAttribution(fg('platform_editor_blocks_patch_3'), errorMessage)));
+      (_this$deps$getFireAna3 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna3 === void 0 ? void 0 : _this$deps$getFireAna3(fetchErrorPayload(errorMessage, resourceId, getSourceProductFromResourceIdSafe(resourceId), buildFetchErrorAttribution(errorMessage)));
       return;
     }
     const delay = this.getReconnectionDelay(attempts);
@@ -502,12 +454,6 @@ export class SyncBlockSubscriptionManager {
     this.titleSubscriptions.clear();
     this.subscriptionChangeListeners.clear();
     this.useRealTimeSubscriptions = false;
-
-    // Clear any pending cache deletions
-    for (const timeout of this.pendingCacheDeletions.values()) {
-      clearTimeout(timeout);
-    }
-    this.pendingCacheDeletions.clear();
   }
   shouldUseRealTime() {
     return this.useRealTimeSubscriptions;
@@ -531,28 +477,26 @@ export class SyncBlockSubscriptionManager {
       const callbacks = this.subscriptions.get(syncBlockInstance.resourceId);
       const localIds = callbacks ? Object.keys(callbacks) : [];
       localIds.forEach(localId => {
-        var _this$deps$getFireAna5, _syncBlockInstance$da, _syncBlockInstance$da2;
-        (_this$deps$getFireAna5 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna5 === void 0 ? void 0 : _this$deps$getFireAna5(fetchSuccessPayload(syncBlockInstance.resourceId, localId, (_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)));
+        var _this$deps$getFireAna4, _syncBlockInstance$da, _syncBlockInstance$da2;
+        (_this$deps$getFireAna4 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna4 === void 0 ? void 0 : _this$deps$getFireAna4(fetchSuccessPayload(syncBlockInstance.resourceId, localId, (_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)));
       });
       this.deps.fetchSyncBlockSourceInfo(resolved.resourceId);
     } else {
-      var _syncBlockInstance$er, _syncBlockInstance$er2, _this$deps$getFireAna6, _syncBlockInstance$da3, _syncBlockInstance$da4, _syncBlockInstance$er3, _syncBlockInstance$er4, _syncBlockInstance$er5;
+      var _syncBlockInstance$er, _syncBlockInstance$er2, _this$deps$getFireAna5, _syncBlockInstance$da3, _syncBlockInstance$da4, _syncBlockInstance$er3, _syncBlockInstance$er4, _syncBlockInstance$er5;
       const errorMessage = ((_syncBlockInstance$er = syncBlockInstance.error) === null || _syncBlockInstance$er === void 0 ? void 0 : _syncBlockInstance$er.reason) || ((_syncBlockInstance$er2 = syncBlockInstance.error) === null || _syncBlockInstance$er2 === void 0 ? void 0 : _syncBlockInstance$er2.type);
 
       // Prefer the structured `type` (a `SyncBlockError` enum value) for classification
       // and fall back to the free-text `reason` so source-state/permission strings are
       // still bucketed (EDITOR-7862). The emitted free-text `error` attribute is unchanged.
-      (_this$deps$getFireAna6 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna6 === void 0 ? void 0 : _this$deps$getFireAna6(fetchErrorPayload(errorMessage, syncBlockInstance.resourceId, (_syncBlockInstance$da3 = (_syncBlockInstance$da4 = syncBlockInstance.data) === null || _syncBlockInstance$da4 === void 0 ? void 0 : _syncBlockInstance$da4.product) !== null && _syncBlockInstance$da3 !== void 0 ? _syncBlockInstance$da3 : getSourceProductFromResourceIdSafe(syncBlockInstance.resourceId), 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), (_syncBlockInstance$er5 = syncBlockInstance.error) === null || _syncBlockInstance$er5 === void 0 ? void 0 : _syncBlockInstance$er5.statusCode)));
+      (_this$deps$getFireAna5 = this.deps.getFireAnalyticsEvent()) === null || _this$deps$getFireAna5 === void 0 ? void 0 : _this$deps$getFireAna5(fetchErrorPayload(errorMessage, syncBlockInstance.resourceId, (_syncBlockInstance$da3 = (_syncBlockInstance$da4 = syncBlockInstance.data) === null || _syncBlockInstance$da4 === void 0 ? void 0 : _syncBlockInstance$da4.product) !== null && _syncBlockInstance$da3 !== void 0 ? _syncBlockInstance$da3 : getSourceProductFromResourceIdSafe(syncBlockInstance.resourceId), 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), (_syncBlockInstance$er5 = syncBlockInstance.error) === null || _syncBlockInstance$er5 === void 0 ? void 0 : _syncBlockInstance$er5.statusCode)));
     }
   }
 }
 // Reconnection with exponential backoff.
 _defineProperty(SyncBlockSubscriptionManager, "INITIAL_RETRY_DELAY_MS", 1000);
 _defineProperty(SyncBlockSubscriptionManager, "RETRY_BACKOFF_MULTIPLIER", 2);
-_defineProperty(SyncBlockSubscriptionManager, "MAX_RETRY_ATTEMPTS", 5);
-// legacy (gate OFF)
 _defineProperty(SyncBlockSubscriptionManager, "MAX_RETRY_ATTEMPTS_HARDENED", 8);
-// gate ON (EDITOR-7861)
+// EDITOR-7861
 _defineProperty(SyncBlockSubscriptionManager, "MAX_RETRY_DELAY_MS", 30000);
 // Coalesce wake-event bursts into one re-arm sweep to avoid a reconnection storm.
 _defineProperty(SyncBlockSubscriptionManager, "WAKE_DEBOUNCE_MS", 1000);

package/dist/es2019/utils/errorHandling.js

@@ -60,17 +60,9 @@ export const classifyErrorReason = error => {
 
 /**
  * Builds the {@link ErrorAttributionAttributes} for a failed synced-block operation from
- * the raw result `error` field and optional backend `statusCode`. Returns `undefined`
- * when the `platform_editor_blocks_patch_3` gate is OFF, so the new `reason`/`statusCode`
- * attributes are only emitted once the gate is rolled out (EDITOR-7796).
- *
- * `gateEnabled` is injected by the caller (the store managers evaluate `fg(...)`) so this
- * helper stays pure and trivially unit-testable for both gate states.
+ * the raw result `error` field and optional backend `statusCode` (EDITOR-7796).
  */
-export const buildErrorAttribution = (gateEnabled, error, statusCode) => {
-  if (!gateEnabled) {
-    return undefined;
-  }
+export const buildErrorAttribution = (error, statusCode) => {
   return {
     reason: classifyErrorReason(error),
     ...(statusCode !== undefined && {
@@ -181,17 +173,10 @@ export const classifyFetchErrorReason = error => {
 /**
  * Builds the {@link FetchErrorAttributionAttributes} for a failed fetch/subscribe
  * synced-block operation from the raw `error` field and optional backend `statusCode`.
- * Returns `undefined` when the `platform_editor_blocks_patch_3` gate is OFF, so the new
- * `reason`/`statusCode`/`benign` attributes are only emitted once the gate is rolled out
- * (EDITOR-7862). The existing free-text `error` attribute is always left unchanged.
- *
- * `gateEnabled` is injected by the caller (the store managers evaluate `fg(...)`) so this
- * helper stays pure and trivially unit-testable for both gate states.
+ * Emits the `reason`/`statusCode`/`benign` attributes (EDITOR-7862). The existing
+ * free-text `error` attribute is always left unchanged.
  */
-export const buildFetchErrorAttribution = (gateEnabled, error, statusCode, deferred) => {
-  if (!gateEnabled) {
-    return undefined;
-  }
+export const buildFetchErrorAttribution = (error, statusCode, deferred) => {
   const reason = classifyFetchErrorReason(error);
   return {
     reason,
@@ -261,8 +246,8 @@ export function getErrorPayload(actionSubjectId, error, resourceId, sourceProduc
 }
 export const fetchErrorPayload = (error, resourceId, sourceProduct, attribution) =>
 // Branch on attribution presence so each call resolves to a concrete overload: with
-// attribution it hits the fetch overload (wider `reason`); without it (gate OFF) it
-// hits the no-attribution overload. Both produce a fetch event regardless.
+// attribution it hits the fetch overload (wider `reason`); without it, the
+// no-attribution overload. Both produce a fetch event regardless.
 attribution ? getErrorPayload(ACTION_SUBJECT_ID.SYNCED_BLOCK_FETCH, error, resourceId, sourceProduct, attribution) : getErrorPayload(ACTION_SUBJECT_ID.SYNCED_BLOCK_FETCH, error, resourceId, sourceProduct);
 export const getSourceInfoErrorPayload = (error, resourceId, sourceProduct) => getErrorPayload(ACTION_SUBJECT_ID.SYNCED_BLOCK_GET_SOURCE_INFO, error, resourceId, sourceProduct);
 export const updateErrorPayload = (error, resourceId, sourceProduct, attribution) => getErrorPayload(ACTION_SUBJECT_ID.SYNCED_BLOCK_UPDATE, error, resourceId, sourceProduct, attribution);
@@ -272,8 +257,7 @@ export const deleteErrorPayload = (error, resourceId, sourceProduct, attribution
 export const updateCacheErrorPayload = (error, resourceId, sourceProduct) => getErrorPayload(ACTION_SUBJECT_ID.SYNCED_BLOCK_UPDATE_CACHE, error, resourceId, sourceProduct);
 /**
  * Payload for `SYNCED_BLOCK_SOURCE_INFO_ORPHANED`. Fired when source-info
- * resolves into a cache that has already been deleted — should be unreachable
- * under `platform_synced_block_patch_14`.
+ * resolves into a cache that has already been deleted — should be unreachable.
  */
 export const sourceInfoOrphanedPayload = (resourceId, sourceProduct, context) => ({
   action: ACTION.ERROR,
@@ -349,14 +333,13 @@ export const createSuccessPayload = (resourceId, sourceProduct) => {
 
 /**
  * Optional enrichment for the `syncedBlockCreate` success event. All fields
- * optional so gate-off/legacy payloads are unchanged. `inputMethod`: creating
+ * optional so legacy payloads are unchanged. `inputMethod`: creating
  * surface (enum, PII-safe). `createdEmpty`: true from an empty selection, false
  * when content was converted.
  */
 
 /**
- * Operational `syncedBlockCreate` success event, behind
- * `platform_editor_blocks_patch_4`, with the `blockInstanceId` join key and,
+ * Operational `syncedBlockCreate` success event with the `blockInstanceId` join key and,
  * when available, the `inputMethod` + `createdEmpty` creation-type signals.
  */
 export const createSuccessOperationalPayload = (resourceId, blockInstanceId, sourceProduct, enrichment) => ({
@@ -382,8 +365,7 @@ export const createSuccessOperationalPayload = (resourceId, blockInstanceId, sou
 });
 
 /**
- * Operational first-content-added event, behind
- * `platform_editor_blocks_patch_4`. Fired once when a block created empty first
+ * Operational first-content-added event. Fired once when a block created empty first
  * gains user content. Join keys only (`resourceId` + `blockInstanceId`), no user
  * content (PII-safe).
  */
@@ -419,9 +401,8 @@ export const updateSuccessPayload = (resourceId, hasReference, sourceProduct) =>
 });
 
 /**
- * Optional enrichment for the `syncedBlockDelete` success event behind
- * `platform_editor_blocks_patch_4`. All fields optional so the gate-off payload
- * is unchanged; `blockInstanceId` is the bare-uuid join key.
+ * Optional enrichment for the `syncedBlockDelete` success event. All fields are
+ * optional for extensibility; `blockInstanceId` is the bare-uuid join key.
  */
 
 export const deleteSuccessPayload = (resourceId, sourceProduct, enrichment) => ({

package/dist/esm/common/types.js

@@ -49,14 +49,13 @@ export var SyncBlockError = /*#__PURE__*/function (SyncBlockError) {
  * 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.
  */
 
-/** Legacy message — kept identical for gate-off and historical events. */
+/** Legacy message — kept identical for historical events. */
 export var PROVIDER_NOT_READY_MESSAGE = 'Data provider not set';
 
 /**