@atlaskit/editor-synced-block-provider 3.28.0 → 3.28.1

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

@@ -1,7 +1,6 @@
 import _defineProperty from "@babel/runtime/helpers/defineProperty";
 import isEqual from 'lodash/isEqual';
 import rafSchedule from 'raf-schd';
-import { isSSR } from '@atlaskit/editor-common/core-utils';
 import { logException } from '@atlaskit/editor-common/monitoring';
 import { ProviderFactory } from '@atlaskit/editor-common/provider-factory';
 import { fg } from '@atlaskit/platform-feature-flags';
@@ -54,12 +53,10 @@ export class ReferenceSyncBlockStoreManager {
         });
       });
     }));
-    this.syncBlockCache = new Map();
     this.subscriptions = new Map();
     this.titleSubscriptions = new Map();
     this.dataProvider = dataProvider;
     this.syncBlockFetchDataRequests = new Map();
-    this.syncBlockSourceInfoRequestsOld = new Map();
     this.syncBlockSourceInfoRequests = new Map();
     this.providerFactories = new Map();
     this.pendingCacheDeletions = new Map();
@@ -176,16 +173,9 @@ export class ReferenceSyncBlockStoreManager {
     this.fireAnalyticsEvent = fireAnalyticsEvent;
   }
   getInitialSyncBlockData(resourceId) {
-    var _this$dataProvider2, _this$dataProvider2$g;
+    var _this$dataProvider, _this$dataProvider$ge;
     const syncBlockNode = createSyncBlockNode('', resourceId);
-    if (isSSR() || fg('platform_synced_block_patch_1')) {
-      var _this$dataProvider, _this$dataProvider$ge;
-      // In SSR, prefer data from data provider cache
-      // should not take from store manager cache as it may be in incomplete state
-      // will be unified to the same cache later.
-      return (_this$dataProvider = this.dataProvider) === null || _this$dataProvider === void 0 ? void 0 : (_this$dataProvider$ge = _this$dataProvider.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider$ge === void 0 ? void 0 : _this$dataProvider$ge.data;
-    }
-    return this.getFromCache(resourceId) || ((_this$dataProvider2 = this.dataProvider) === null || _this$dataProvider2 === void 0 ? void 0 : (_this$dataProvider2$g = _this$dataProvider2.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider2$g === void 0 ? void 0 : _this$dataProvider2$g.data);
+    return (_this$dataProvider = this.dataProvider) === null || _this$dataProvider === void 0 ? void 0 : (_this$dataProvider$ge = _this$dataProvider.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider$ge === void 0 ? void 0 : _this$dataProvider$ge.data;
   }
 
   /**
@@ -228,12 +218,12 @@ export class ReferenceSyncBlockStoreManager {
    * @param resourceId - The resource ID of the block to subscribe to
    */
   setupGraphQLSubscription(resourceId) {
-    var _this$dataProvider3;
+    var _this$dataProvider2;
     // Don't set up duplicate subscriptions
     if (this.graphqlSubscriptions.has(resourceId)) {
       return;
     }
-    if (!((_this$dataProvider3 = this.dataProvider) !== null && _this$dataProvider3 !== void 0 && _this$dataProvider3.subscribeToBlockUpdates)) {
+    if (!((_this$dataProvider2 = this.dataProvider) !== null && _this$dataProvider2 !== void 0 && _this$dataProvider2.subscribeToBlockUpdates)) {
       return;
     }
     const unsubscribe = this.dataProvider.subscribeToBlockUpdates(resourceId, syncBlockInstance => {
@@ -362,9 +352,7 @@ export class ReferenceSyncBlockStoreManager {
         return Promise.resolve(undefined);
       }
       (_this$fetchSourceInfo = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo === void 0 ? void 0 : _this$fetchSourceInfo.start({});
-
-      // Only use unpublished endpoint if feature flag is enabled
-      const isUnpublished = fg('platform_synced_block_patch_1') && ((_existingSyncBlock$da = existingSyncBlock.data) === null || _existingSyncBlock$da === void 0 ? void 0 : _existingSyncBlock$da.status) === 'unpublished';
+      const isUnpublished = ((_existingSyncBlock$da = existingSyncBlock.data) === null || _existingSyncBlock$da === void 0 ? void 0 : _existingSyncBlock$da.status) === 'unpublished';
       const sourceInfoPromise = this.dataProvider.fetchSyncBlockSourceInfo(blockInstanceId, sourceAri, product, this.fireAnalyticsEvent, true,
       // hasAccess
       'edit',
@@ -597,18 +585,13 @@ export class ReferenceSyncBlockStoreManager {
       resourceId
     } = syncBlock;
     if (resourceId) {
-      if (fg('platform_synced_block_patch_1')) {
-        var _this$dataProvider4;
-        // Use the cache in dataProvider
-        (_this$dataProvider4 = this.dataProvider) === null || _this$dataProvider4 === void 0 ? void 0 : _this$dataProvider4.updateCache({
-          [resourceId]: syncBlock
-        }, {
-          strategy: 'merge',
-          source: 'network'
-        });
-      } else {
-        this.syncBlockCache.set(resourceId, syncBlock);
-      }
+      var _this$dataProvider3;
+      (_this$dataProvider3 = this.dataProvider) === null || _this$dataProvider3 === void 0 ? void 0 : _this$dataProvider3.updateCache({
+        [resourceId]: syncBlock
+      }, {
+        strategy: 'merge',
+        source: 'network'
+      });
       const callbacks = this.subscriptions.get(resourceId);
       if (callbacks) {
         Object.values(callbacks).forEach(callback => {
@@ -626,23 +609,13 @@ export class ReferenceSyncBlockStoreManager {
     }
   }
   getFromCache(resourceId) {
-    if (fg('platform_synced_block_patch_1')) {
-      var _this$dataProvider5, _this$dataProvider5$g;
-      // Use the cache in dataProvider
-      const syncBlockNode = createSyncBlockNode('', resourceId);
-      return (_this$dataProvider5 = this.dataProvider) === null || _this$dataProvider5 === void 0 ? void 0 : (_this$dataProvider5$g = _this$dataProvider5.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider5$g === void 0 ? void 0 : _this$dataProvider5$g.data;
-    }
-    return this.syncBlockCache.get(resourceId);
+    var _this$dataProvider4, _this$dataProvider4$g;
+    const syncBlockNode = createSyncBlockNode('', resourceId);
+    return (_this$dataProvider4 = this.dataProvider) === null || _this$dataProvider4 === void 0 ? void 0 : (_this$dataProvider4$g = _this$dataProvider4.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider4$g === void 0 ? void 0 : _this$dataProvider4$g.data;
   }
   deleteFromCache(resourceId) {
-    if (fg('platform_synced_block_patch_1')) {
-      var _this$dataProvider6;
-      // For dataProvider cache, we update with empty/deleted state
-      // The cache is managed per-node basis via resetCache if needed
-      // For now, we don't explicitly delete from dataProvider cache
-      // as the cache is meant to persist for cache-first-then-network strategy
-      (_this$dataProvider6 = this.dataProvider) === null || _this$dataProvider6 === void 0 ? void 0 : _this$dataProvider6.removeFromCache([resourceId]);
-    }
+    var _this$dataProvider5;
+    (_this$dataProvider5 = this.dataProvider) === null || _this$dataProvider5 === void 0 ? void 0 : _this$dataProvider5.removeFromCache([resourceId]);
     this.providerFactories.delete(resourceId);
   }
   debouncedBatchedFetchSyncBlocks(resourceId) {
@@ -660,7 +633,7 @@ export class ReferenceSyncBlockStoreManager {
     }
   }
   subscribeToSyncBlock(resourceId, localId, callback) {
-    var _this$dataProvider7, _this$dataProvider7$g, _this$dataProvider8, _this$dataProvider8$g, _this$dataProvider9, _this$dataProvider9$g;
+    var _this$dataProvider6, _this$dataProvider6$g;
     // Cancel any pending cache deletion for this resourceId.
     // This handles the case where a block is moved - the old component unmounts
     // (scheduling deletion) but the new component mounts and subscribes before
@@ -689,23 +662,11 @@ export class ReferenceSyncBlockStoreManager {
     const syncBlockNode = createSyncBlockNode(localId, resourceId);
 
     // call the callback immediately if we have cached data
-    const cachedData = fg('platform_synced_block_patch_1') ? // When feature flag is enabled, use dataProvider cache only
-    (_this$dataProvider7 = this.dataProvider) === null || _this$dataProvider7 === void 0 ? void 0 : (_this$dataProvider7$g = _this$dataProvider7.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider7$g === void 0 ? void 0 : _this$dataProvider7$g.data : isSSR() // in SSR, prefer data provider cache
-    ? ((_this$dataProvider8 = this.dataProvider) === null || _this$dataProvider8 === void 0 ? void 0 : (_this$dataProvider8$g = _this$dataProvider8.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider8$g === void 0 ? void 0 : _this$dataProvider8$g.data) || this.getFromCache(resourceId) : this.getFromCache(resourceId) || ((_this$dataProvider9 = this.dataProvider) === null || _this$dataProvider9 === void 0 ? void 0 : (_this$dataProvider9$g = _this$dataProvider9.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider9$g === void 0 ? void 0 : _this$dataProvider9$g.data);
+    const cachedData = (_this$dataProvider6 = this.dataProvider) === null || _this$dataProvider6 === void 0 ? void 0 : (_this$dataProvider6$g = _this$dataProvider6.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider6$g === void 0 ? void 0 : _this$dataProvider6$g.data;
     if (cachedData) {
       callback(cachedData);
     } else {
-      if (fg('platform_synced_block_patch_1')) {
-        this.debouncedBatchedFetchSyncBlocks(resourceId);
-      } else {
-        this.fetchSyncBlocksData([syncBlockNode]).catch(error => {
-          var _this$fireAnalyticsEv14;
-          logException(error, {
-            location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
-          });
-          (_this$fireAnalyticsEv14 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv14 === void 0 ? void 0 : _this$fireAnalyticsEv14.call(this, fetchErrorPayload(error.message, resourceId));
-        });
-      }
+      this.debouncedBatchedFetchSyncBlocks(resourceId);
     }
 
     // Set up GraphQL subscription if real-time subscriptions are enabled
@@ -797,11 +758,11 @@ export class ReferenceSyncBlockStoreManager {
       }
       return this.subscribeToSyncBlock(resourceId, localId, callback);
     } catch (error) {
-      var _this$fireAnalyticsEv15;
+      var _this$fireAnalyticsEv14;
       logException(error, {
         location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
       });
-      (_this$fireAnalyticsEv15 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv15 === void 0 ? void 0 : _this$fireAnalyticsEv15.call(this, fetchErrorPayload(error.message));
+      (_this$fireAnalyticsEv14 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv14 === void 0 ? void 0 : _this$fireAnalyticsEv14.call(this, fetchErrorPayload(error.message));
       return () => {};
     }
   }
@@ -821,12 +782,12 @@ export class ReferenceSyncBlockStoreManager {
   }
   getProviderFactory(resourceId) {
     if (!this.dataProvider) {
-      var _this$fireAnalyticsEv16;
+      var _this$fireAnalyticsEv15;
       const error = new Error('Data provider not set');
       logException(error, {
         location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
       });
-      (_this$fireAnalyticsEv16 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv16 === void 0 ? void 0 : _this$fireAnalyticsEv16.call(this, fetchErrorPayload(error.message));
+      (_this$fireAnalyticsEv15 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv15 === void 0 ? void 0 : _this$fireAnalyticsEv15.call(this, fetchErrorPayload(error.message));
       return undefined;
     }
     const {
@@ -856,11 +817,11 @@ export class ReferenceSyncBlockStoreManager {
       try {
         this.retrieveDynamicProviders(resourceId, providerFactory, providerCreator);
       } catch (error) {
-        var _this$fireAnalyticsEv17;
+        var _this$fireAnalyticsEv16;
         logException(error, {
           location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
         });
-        (_this$fireAnalyticsEv17 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv17 === void 0 ? void 0 : _this$fireAnalyticsEv17.call(this, fetchErrorPayload(error.message, resourceId));
+        (_this$fireAnalyticsEv16 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv16 === void 0 ? void 0 : _this$fireAnalyticsEv16.call(this, fetchErrorPayload(error.message, resourceId));
       }
     }
     return providerFactory;
@@ -916,8 +877,8 @@ export class ReferenceSyncBlockStoreManager {
       return;
     }
     if (!syncBlock.data.sourceAri || !syncBlock.data.product) {
-      var _this$fireAnalyticsEv18;
-      (_this$fireAnalyticsEv18 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv18 === void 0 ? void 0 : _this$fireAnalyticsEv18.call(this, fetchErrorPayload('Sync block source ari or product not found'));
+      var _this$fireAnalyticsEv17;
+      (_this$fireAnalyticsEv17 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv17 === void 0 ? void 0 : _this$fireAnalyticsEv17.call(this, fetchErrorPayload('Sync block source ari or product not found'));
       return;
     }
     const parentInfo = this.dataProvider.retrieveSyncBlockParentInfo((_syncBlock$data2 = syncBlock.data) === null || _syncBlock$data2 === void 0 ? void 0 : _syncBlock$data2.sourceAri, (_syncBlock$data3 = syncBlock.data) === null || _syncBlock$data3 === void 0 ? void 0 : _syncBlock$data3.product);
@@ -1035,15 +996,15 @@ export class ReferenceSyncBlockStoreManager {
       (_this$saveExperience = this.saveExperience) === null || _this$saveExperience === void 0 ? void 0 : _this$saveExperience.start();
       const updateResult = await this.dataProvider.updateReferenceData(blocks);
       if (!updateResult.success) {
-        var _this$saveExperience2, _this$fireAnalyticsEv19;
+        var _this$saveExperience2, _this$fireAnalyticsEv18;
         success = false;
         (_this$saveExperience2 = this.saveExperience) === null || _this$saveExperience2 === void 0 ? void 0 : _this$saveExperience2.failure({
           reason: updateResult.error || 'Failed to update reference synced blocks on the document'
         });
-        (_this$fireAnalyticsEv19 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv19 === void 0 ? void 0 : _this$fireAnalyticsEv19.call(this, updateReferenceErrorPayload(updateResult.error || 'Failed to update reference synced blocks on the document'));
+        (_this$fireAnalyticsEv18 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv18 === void 0 ? void 0 : _this$fireAnalyticsEv18.call(this, updateReferenceErrorPayload(updateResult.error || 'Failed to update reference synced blocks on the document'));
       }
     } catch (error) {
-      var _this$saveExperience3, _this$fireAnalyticsEv20;
+      var _this$saveExperience3, _this$fireAnalyticsEv19;
       success = false;
       logException(error, {
         location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
@@ -1051,7 +1012,7 @@ export class ReferenceSyncBlockStoreManager {
       (_this$saveExperience3 = this.saveExperience) === null || _this$saveExperience3 === void 0 ? void 0 : _this$saveExperience3.failure({
         reason: error.message
       });
-      (_this$fireAnalyticsEv20 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv20 === void 0 ? void 0 : _this$fireAnalyticsEv20.call(this, updateReferenceErrorPayload(error.message));
+      (_this$fireAnalyticsEv19 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv19 === void 0 ? void 0 : _this$fireAnalyticsEv19.call(this, updateReferenceErrorPayload(error.message));
     } finally {
       if (!success) {
         // set isCacheDirty back to true for cases where it failed to update the reference synced blocks on the BE
@@ -1082,7 +1043,7 @@ export class ReferenceSyncBlockStoreManager {
     return success;
   }
   destroy() {
-    var _this$saveExperience5, _this$fetchExperience0, _this$fetchSourceInfo6;
+    var _this$dataProvider7, _this$saveExperience5, _this$fetchExperience0, _this$fetchSourceInfo6;
     // Cancel any queued flush to prevent it from running after destroy
     if (this.queuedFlushTimeout) {
       clearTimeout(this.queuedFlushTimeout);
@@ -1091,20 +1052,13 @@ export class ReferenceSyncBlockStoreManager {
 
     // Clean up all GraphQL subscriptions first
     this.cleanupAllGraphQLSubscriptions();
-    if (fg('platform_synced_block_patch_1')) {
-      var _this$dataProvider0;
-      // Reset cache in dataProvider
-      (_this$dataProvider0 = this.dataProvider) === null || _this$dataProvider0 === void 0 ? void 0 : _this$dataProvider0.resetCache();
-    } else {
-      this.syncBlockCache.clear();
-    }
+    (_this$dataProvider7 = this.dataProvider) === null || _this$dataProvider7 === void 0 ? void 0 : _this$dataProvider7.resetCache();
     this.scheduledBatchFetch.cancel();
     this.pendingFetchRequests.clear();
     this.dataProvider = undefined;
     this.subscriptions.clear();
     this.titleSubscriptions.clear();
     this.syncBlockFetchDataRequests.clear();
-    this.syncBlockSourceInfoRequestsOld.clear();
     this.syncBlockSourceInfoRequests.clear();
     this.providerFactories.clear();
     this.isRefreshingSubscriptions = false;

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, deleteSuccessPayload, fetchReferencesErrorPayload } from '../utils/errorHandling';
 import { getCreateSourceExperience, getDeleteSourceExperience, getSaveSourceExperience, getFetchSourceInfoExperience } from '../utils/experienceTracking';
@@ -80,7 +79,7 @@ export class SourceSyncBlockStoreManager {
         // - are waiting to be deleted to avoid nodes being re-created
         // - haven't been updated since we last flushed
         // - are still pending BE creation
-        if (!syncBlockData.pendingDeletion && syncBlockData.isDirty && !(this.isPendingCreation(syncBlockData.resourceId) && fg('platform_synced_block_patch_1'))) {
+        if (!syncBlockData.pendingDeletion && syncBlockData.isDirty && !this.isPendingCreation(syncBlockData.resourceId)) {
           bodiedSyncBlockNodes.push({
             type: 'bodiedSyncBlock',
             attrs: {
@@ -94,11 +93,11 @@ export class SourceSyncBlockStoreManager {
           // is still making changes, the new changes might not be saved if they all happen
           // exactly at a time when the writeNodesData is being executed asynchronously.
           syncBlockData.isDirty = false;
-          // When flushing, set status to 'active' so the block is published (when feature gate is on)
-          const dataToFlush = fg('platform_synced_block_patch_1') ? {
+          // When flushing, set status to 'active' so the block is published
+          const dataToFlush = {
             ...syncBlockData,
             status: 'active'
-          } : syncBlockData;
+          };
           bodiedSyncBlockData.push(dataToFlush);
         }
       });
@@ -147,66 +146,32 @@ export class SourceSyncBlockStoreManager {
       return false;
     }
   }
-
-  // Remove this method and pendingResourceId when cleaning up platform_synced_block_patch_1
-  registerPendingCreation(resourceId) {
-    this.pendingResourceId = resourceId;
-  }
   isPendingCreation(resourceId) {
     return this.creationCompletionCallbacks.has(resourceId);
   }
 
-  /**
-   * Register callback function (which inserts node, handles focus etc) to be used later when creation to backend succeed
-   */
-  registerCreationCallback(callback) {
-    this.creationCallback = callback;
-  }
-
   /**
    *  Fires callback to insert node (if creation is successful) and clears pending creation data
    * @param success
    */
   commitPendingCreation(success, resourceId) {
-    if (fg('platform_synced_block_patch_1')) {
-      const onCompletion = this.creationCompletionCallbacks.get(resourceId);
-      if (onCompletion) {
-        this.creationCompletionCallbacks.delete(resourceId);
-        onCompletion(success);
-      } else {
-        var _this$fireAnalyticsEv5;
-        (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 ? void 0 : _this$fireAnalyticsEv5.call(this, createErrorPayload('creation complete callback missing', resourceId));
-      }
-      if (success) {
-        var _this$fireAnalyticsEv6;
-        (_this$fireAnalyticsEv6 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv6 === void 0 ? void 0 : _this$fireAnalyticsEv6.call(this, createSuccessPayload(resourceId || ''));
-      } else {
-        var _this$fireAnalyticsEv7;
-        // Delete the node from cache if fail to create so it's not flushed to BE
-        this.syncBlockCache.delete(resourceId || '');
-        (_this$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 ? void 0 : _this$fireAnalyticsEv7.call(this, createErrorPayload('Fail to create bodied sync block', resourceId));
-      }
+    const onCompletion = this.creationCompletionCallbacks.get(resourceId);
+    if (onCompletion) {
+      this.creationCompletionCallbacks.delete(resourceId);
+      onCompletion(success);
     } else {
-      if (success && this.creationCallback) {
-        var _this$fireAnalyticsEv8;
-        this.creationCallback();
-        (_this$fireAnalyticsEv8 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv8 === void 0 ? void 0 : _this$fireAnalyticsEv8.call(this, createSuccessPayload(this.pendingResourceId || ''));
-      } else if (success && !this.creationCallback) {
-        var _this$fireAnalyticsEv9;
-        (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 ? void 0 : _this$fireAnalyticsEv9.call(this, createErrorPayload('creation callback missing', this.pendingResourceId));
-      }
-      this.pendingResourceId = undefined;
+      var _this$fireAnalyticsEv5;
+      (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 ? void 0 : _this$fireAnalyticsEv5.call(this, createErrorPayload('creation complete callback missing', resourceId));
+    }
+    if (success) {
+      var _this$fireAnalyticsEv6;
+      (_this$fireAnalyticsEv6 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv6 === void 0 ? void 0 : _this$fireAnalyticsEv6.call(this, createSuccessPayload(resourceId || ''));
+    } else {
+      var _this$fireAnalyticsEv7;
+      // Delete the node from cache if fail to create so it's not flushed to BE
+      this.syncBlockCache.delete(resourceId || '');
+      (_this$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 ? void 0 : _this$fireAnalyticsEv7.call(this, createErrorPayload('Fail to create bodied sync block', resourceId));
     }
-    this.creationCallback = undefined;
-  }
-
-  /**
-   *
-   * @returns true if waiting for the result of saving new bodiedSyncBlock to backend
-   * Remove this method when cleaning up platform_synced_block_patch_1
-   */
-  hasPendingCreation() {
-    return fg('platform_synced_block_patch_1') ? this.creationCompletionCallbacks.size > 0 : !!this.pendingResourceId;
   }
   registerConfirmationCallback(callback) {
     this.confirmationCallback = callback;
@@ -249,9 +214,7 @@ export class SourceSyncBlockStoreManager {
       if (!this.dataProvider) {
         throw new Error('Data provider not set');
       }
-      if (fg('platform_synced_block_patch_1')) {
-        this.creationCompletionCallbacks.set(resourceId, onCompletion);
-      }
+      this.creationCompletionCallbacks.set(resourceId, onCompletion);
       (_this$createExperienc = this.createExperience) === null || _this$createExperienc === void 0 ? void 0 : _this$createExperienc.start({});
       this.dataProvider.createNodeData({
         content: [],
@@ -259,28 +222,20 @@ export class SourceSyncBlockStoreManager {
         resourceId
       }).then(result => {
         const resourceId = result.resourceId || '';
-        if (resourceId && (!fg('platform_synced_block_patch_1') || !result.error)) {
+        if (resourceId && !result.error) {
           var _this$createExperienc2;
           this.commitPendingCreation(true, resourceId);
           (_this$createExperienc2 = this.createExperience) === null || _this$createExperienc2 === void 0 ? void 0 : _this$createExperienc2.success();
-
-          // Update the sync block data with the node data if it is provided
-          // to avoid any race conditions where the data could be missed during a render operation
-          if (!fg('platform_synced_block_patch_1')) {
-            if (nodeData) {
-              this.updateSyncBlockData(nodeData);
-            }
-          }
         } else {
-          var _this$createExperienc3, _this$fireAnalyticsEv0;
+          var _this$createExperienc3, _this$fireAnalyticsEv8;
           this.commitPendingCreation(false, resourceId);
           (_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));
+          (_this$fireAnalyticsEv8 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv8 === void 0 ? void 0 : _this$fireAnalyticsEv8.call(this, createErrorPayload(result.error || 'Failed to create bodied sync block', resourceId));
         }
       }).catch(error => {
-        var _this$createExperienc4, _this$fireAnalyticsEv1;
+        var _this$createExperienc4, _this$fireAnalyticsEv9;
         this.commitPendingCreation(false, resourceId);
         logException(error, {
           location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
@@ -288,20 +243,17 @@ export class SourceSyncBlockStoreManager {
         (_this$createExperienc4 = this.createExperience) === null || _this$createExperienc4 === void 0 ? void 0 : _this$createExperienc4.failure({
           reason: error.message
         });
-        (_this$fireAnalyticsEv1 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv1 === void 0 ? void 0 : _this$fireAnalyticsEv1.call(this, createErrorPayload(error.message, resourceId));
+        (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 ? void 0 : _this$fireAnalyticsEv9.call(this, createErrorPayload(error.message, resourceId));
       });
-      if (!fg('platform_synced_block_patch_1')) {
-        this.registerPendingCreation(resourceId);
-      }
     } catch (error) {
-      var _this$fireAnalyticsEv10;
-      if (fg('platform_synced_block_patch_1') ? this.isPendingCreation(resourceId) : this.hasPendingCreation()) {
+      var _this$fireAnalyticsEv0;
+      if (this.isPendingCreation(resourceId)) {
         this.commitPendingCreation(false, resourceId);
       }
       logException(error, {
         location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
       });
-      (_this$fireAnalyticsEv10 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv10 === void 0 ? void 0 : _this$fireAnalyticsEv10.call(this, createErrorPayload(error.message));
+      (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 ? void 0 : _this$fireAnalyticsEv0.call(this, createErrorPayload(error.message));
     }
   }
   async delete(syncBlockIds, onDelete, onDeleteCompleted, reason) {
@@ -325,8 +277,8 @@ export class SourceSyncBlockStoreManager {
         this.clearPendingDeletion();
         (_this$deleteExperienc2 = this.deleteExperience) === null || _this$deleteExperienc2 === void 0 ? void 0 : _this$deleteExperienc2.success();
         results.forEach(result => {
-          var _this$fireAnalyticsEv11;
-          (_this$fireAnalyticsEv11 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv11 === void 0 ? void 0 : _this$fireAnalyticsEv11.call(this, deleteSuccessPayload(result.resourceId));
+          var _this$fireAnalyticsEv1;
+          (_this$fireAnalyticsEv1 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv1 === void 0 ? void 0 : _this$fireAnalyticsEv1.call(this, deleteSuccessPayload(result.resourceId));
         });
       } else {
         var _this$deleteExperienc3;
@@ -336,11 +288,11 @@ export class SourceSyncBlockStoreManager {
         (_this$deleteExperienc3 = this.deleteExperience) === null || _this$deleteExperienc3 === void 0 ? void 0 : _this$deleteExperienc3.failure();
         results.forEach(result => {
           if (result.success) {
-            var _this$fireAnalyticsEv12;
-            (_this$fireAnalyticsEv12 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv12 === void 0 ? void 0 : _this$fireAnalyticsEv12.call(this, deleteSuccessPayload(result.resourceId));
+            var _this$fireAnalyticsEv10;
+            (_this$fireAnalyticsEv10 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv10 === void 0 ? void 0 : _this$fireAnalyticsEv10.call(this, deleteSuccessPayload(result.resourceId));
           } 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));
+            var _this$fireAnalyticsEv11;
+            (_this$fireAnalyticsEv11 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv11 === void 0 ? void 0 : _this$fireAnalyticsEv11.call(this, deleteErrorPayload(result.error || 'Failed to delete synced block', result.resourceId));
           }
         });
       }
@@ -348,9 +300,9 @@ export class SourceSyncBlockStoreManager {
       return isDeleteSuccessful;
     } catch (error) {
       syncBlockIds.forEach(Ids => {
-        var _this$fireAnalyticsEv14;
+        var _this$fireAnalyticsEv12;
         this.setPendingDeletion(Ids, false);
-        (_this$fireAnalyticsEv14 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv14 === void 0 ? void 0 : _this$fireAnalyticsEv14.call(this, deleteErrorPayload(error.message, Ids.resourceId));
+        (_this$fireAnalyticsEv12 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv12 === void 0 ? void 0 : _this$fireAnalyticsEv12.call(this, deleteErrorPayload(error.message, Ids.resourceId));
       });
       logException(error, {
         location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
@@ -431,11 +383,11 @@ export class SourceSyncBlockStoreManager {
         return sourceInfo;
       });
     } catch (error) {
-      var _this$fireAnalyticsEv15;
+      var _this$fireAnalyticsEv13;
       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$fireAnalyticsEv13 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv13 === void 0 ? void 0 : _this$fireAnalyticsEv13.call(this, getSourceInfoErrorPayload(error.message));
       return Promise.resolve(undefined);
     }
   }
@@ -446,11 +398,11 @@ export class SourceSyncBlockStoreManager {
       }
       return this.dataProvider.fetchReferences(resourceId, true);
     } catch (error) {
-      var _this$fireAnalyticsEv16;
+      var _this$fireAnalyticsEv14;
       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));
+      (_this$fireAnalyticsEv14 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv14 === void 0 ? void 0 : _this$fireAnalyticsEv14.call(this, fetchReferencesErrorPayload(error.message));
       return Promise.resolve({
         error: SyncBlockError.Errored
       });
@@ -460,9 +412,7 @@ export class SourceSyncBlockStoreManager {
     var _this$saveExperience4, _this$createExperienc5, _this$deleteExperienc4, _this$fetchSourceInfo4;
     this.syncBlockCache.clear();
     this.confirmationCallback = undefined;
-    this.pendingResourceId = undefined;
     this.creationCompletionCallbacks.clear();
-    this.creationCallback = undefined;
     this.dataProvider = undefined;
     (_this$saveExperience4 = this.saveExperience) === null || _this$saveExperience4 === void 0 ? void 0 : _this$saveExperience4.abort({
       reason: 'editorDestroyed'

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

@@ -1,6 +1,5 @@
 import { useMemo } 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';
@@ -20,7 +19,7 @@ export class SyncBlockStoreManager {
     this.sourceSyncBlockStoreManager = new SourceSyncBlockStoreManager(dataProvider);
     this.referenceSyncBlockStoreManager = new ReferenceSyncBlockStoreManager(dataProvider);
     this.dataProvider = dataProvider;
-    this.referenceSyncBlockStoreManager.setRealTimeSubscriptionsEnabled(fg('platform_synced_block_patch_1'));
+    this.referenceSyncBlockStoreManager.setRealTimeSubscriptionsEnabled(true);
   }
   async fetchReferencesSourceInfo(resourceId, blockInstanceId, isSourceSyncBlock) {
     try {