@atlaskit/editor-synced-block-provider 3.25.0 → 3.26.1

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

@@ -171,7 +171,7 @@ export class ReferenceSyncBlockStoreManager {
   getInitialSyncBlockData(resourceId) {
     var _this$dataProvider2, _this$dataProvider2$g;
     const syncBlockNode = createSyncBlockNode('', resourceId);
-    if (isSSR() && fg('platform_synced_block_dogfooding') || fg('platform_synced_block_patch_1')) {
+    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
@@ -317,16 +317,13 @@ export class ReferenceSyncBlockStoreManager {
   }
   fetchSyncBlockSourceInfo(resourceId) {
     try {
+      var _this$fetchSourceInfo, _existingSyncBlock$da;
       if (!resourceId || !this.dataProvider) {
         throw new Error('Data provider or resourceId not set');
       }
-      if (fg('platform_synced_block_dogfooding')) {
-        const existingRequest = this.syncBlockSourceInfoRequests.get(resourceId);
-        if (existingRequest) {
-          return existingRequest;
-        }
-      } else if (this.syncBlockSourceInfoRequestsOld.get(resourceId)) {
-        return Promise.resolve(undefined);
+      const existingRequest = this.syncBlockSourceInfoRequests.get(resourceId);
+      if (existingRequest) {
+        return existingRequest;
       }
       const existingSyncBlock = this.getFromCache(resourceId);
       if (!existingSyncBlock) {
@@ -343,79 +340,64 @@ export class ReferenceSyncBlockStoreManager {
       } = existingSyncBlock.data || {};
       // skip if source URL and title are already present
       if (sourceURL && sourceTitle) {
-        if (fg('platform_synced_block_dogfooding')) {
-          return Promise.resolve({
-            title: sourceTitle,
-            url: sourceURL,
-            subType: sourceSubType,
-            sourceAri: sourceAri || '',
-            onSameDocument,
-            productType: product
-          });
-        } else {
-          return Promise.resolve(undefined);
-        }
+        return Promise.resolve({
+          title: sourceTitle,
+          url: sourceURL,
+          subType: sourceSubType,
+          sourceAri: sourceAri || '',
+          onSameDocument,
+          productType: product
+        });
       }
       if (!sourceAri || !product || !blockInstanceId) {
         var _this$fireAnalyticsEv8;
         (_this$fireAnalyticsEv8 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv8 === void 0 ? void 0 : _this$fireAnalyticsEv8.call(this, getSourceInfoErrorPayload('SourceAri, product or blockInstanceId missing', resourceId));
         return Promise.resolve(undefined);
       }
-      if (fg('platform_synced_block_dogfooding')) {
-        var _this$fetchSourceInfo;
-        (_this$fetchSourceInfo = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo === void 0 ? void 0 : _this$fetchSourceInfo.start({});
-      }
-      const sourceInfoPromise = this.dataProvider.fetchSyncBlockSourceInfo(blockInstanceId, sourceAri, product, this.fireAnalyticsEvent).then(sourceInfo => {
+      (_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 sourceInfoPromise = this.dataProvider.fetchSyncBlockSourceInfo(blockInstanceId, sourceAri, product, this.fireAnalyticsEvent, true,
+      // hasAccess
+      'edit',
+      // urlType
+      isUnpublished).then(sourceInfo => {
         if (!sourceInfo) {
-          if (fg('platform_synced_block_dogfooding')) {
-            var _this$fetchSourceInfo2, _this$fireAnalyticsEv9;
-            (_this$fetchSourceInfo2 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo2 === void 0 ? void 0 : _this$fetchSourceInfo2.failure({
-              reason: 'No source info returned'
-            });
-            (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 ? void 0 : _this$fireAnalyticsEv9.call(this, getSourceInfoErrorPayload('No source info returned', resourceId));
-          }
+          var _this$fetchSourceInfo2, _this$fireAnalyticsEv9;
+          (_this$fetchSourceInfo2 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo2 === void 0 ? void 0 : _this$fetchSourceInfo2.failure({
+            reason: 'No source info returned'
+          });
+          (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 ? void 0 : _this$fireAnalyticsEv9.call(this, getSourceInfoErrorPayload('No source info returned', resourceId));
           return undefined;
         }
         this.updateCacheWithSourceInfo(resourceId, sourceInfo);
         if (sourceInfo.title) {
           this.updateSourceTitleSubscriptions(resourceId, sourceInfo.title);
         }
-        if (fg('platform_synced_block_dogfooding')) {
-          if (sourceInfo.title && sourceInfo.url) {
-            var _this$fetchSourceInfo3;
-            (_this$fetchSourceInfo3 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo3 === void 0 ? void 0 : _this$fetchSourceInfo3.success();
-          } else {
-            var _this$fetchSourceInfo4, _this$fireAnalyticsEv0;
-            (_this$fetchSourceInfo4 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo4 === void 0 ? void 0 : _this$fetchSourceInfo4.failure({
-              reason: 'Missing title or url'
-            });
-            (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 ? void 0 : _this$fireAnalyticsEv0.call(this, getSourceInfoErrorPayload('Missing title or url', resourceId));
-          }
-          return sourceInfo;
-        }
-      }).catch(error => {
-        var _this$fireAnalyticsEv1;
-        if (fg('platform_synced_block_dogfooding')) {
-          var _this$fetchSourceInfo5;
-          (_this$fetchSourceInfo5 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo5 === void 0 ? void 0 : _this$fetchSourceInfo5.failure({
-            reason: error.message
+        if (sourceInfo.title && sourceInfo.url) {
+          var _this$fetchSourceInfo3;
+          (_this$fetchSourceInfo3 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo3 === void 0 ? void 0 : _this$fetchSourceInfo3.success();
+        } else {
+          var _this$fetchSourceInfo4, _this$fireAnalyticsEv0;
+          (_this$fetchSourceInfo4 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo4 === void 0 ? void 0 : _this$fetchSourceInfo4.failure({
+            reason: 'Missing title or url'
           });
+          (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 ? void 0 : _this$fireAnalyticsEv0.call(this, getSourceInfoErrorPayload('Missing title or url', resourceId));
         }
+        return sourceInfo;
+      }).catch(error => {
+        var _this$fetchSourceInfo5, _this$fireAnalyticsEv1;
+        (_this$fetchSourceInfo5 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo5 === void 0 ? void 0 : _this$fetchSourceInfo5.failure({
+          reason: error.message
+        });
         (_this$fireAnalyticsEv1 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv1 === void 0 ? void 0 : _this$fireAnalyticsEv1.call(this, getSourceInfoErrorPayload(error.message, resourceId));
         return undefined;
       }).finally(() => {
-        if (fg('platform_synced_block_dogfooding')) {
-          this.syncBlockSourceInfoRequests.delete(resourceId);
-        } else {
-          this.syncBlockSourceInfoRequestsOld.delete(resourceId);
-        }
+        this.syncBlockSourceInfoRequests.delete(resourceId);
       });
-      if (fg('platform_synced_block_dogfooding')) {
-        this.syncBlockSourceInfoRequests.set(resourceId, sourceInfoPromise);
-        return sourceInfoPromise;
-      } else {
-        this.syncBlockSourceInfoRequestsOld.set(resourceId, true);
-      }
+      this.syncBlockSourceInfoRequests.set(resourceId, sourceInfoPromise);
+      return sourceInfoPromise;
     } catch (error) {
       var _this$fireAnalyticsEv10;
       logException(error, {
@@ -432,6 +414,7 @@ export class ReferenceSyncBlockStoreManager {
    * @returns The fetched sync block data results
    */
   async fetchSyncBlocksData(syncBlockNodes) {
+    var _this$fetchExperience;
     if (syncBlockNodes.length === 0) {
       return;
     }
@@ -458,10 +441,7 @@ export class ReferenceSyncBlockStoreManager {
     nodesToFetch.forEach(node => {
       this.syncBlockFetchDataRequests.set(node.attrs.resourceId, true);
     });
-    if (fg('platform_synced_block_dogfooding')) {
-      var _this$fetchExperience;
-      (_this$fetchExperience = this.fetchExperience) === null || _this$fetchExperience === void 0 ? void 0 : _this$fetchExperience.start({});
-    }
+    (_this$fetchExperience = this.fetchExperience) === null || _this$fetchExperience === void 0 ? void 0 : _this$fetchExperience.start({});
     const data = await this.dataProvider.fetchNodesData(nodesToFetch).finally(() => {
       nodesToFetch.forEach(node => {
         this.syncBlockFetchDataRequests.delete(node.attrs.resourceId);
@@ -482,7 +462,7 @@ export class ReferenceSyncBlockStoreManager {
 
       // Check if this is a newly added unpublished sync block and notify
       // Only trigger for sync blocks that were just added (not refreshed or loaded on page init)
-      if (!syncBlockInstance.error && ((_resolvedSyncBlockIns = resolvedSyncBlockInstance.data) === null || _resolvedSyncBlockIns === void 0 ? void 0 : _resolvedSyncBlockIns.status) === 'unpublished' && this.newlyAddedSyncBlocks.has(syncBlockInstance.resourceId) && this.onUnpublishedSyncBlockDetected && fg('platform_synced_block_dogfooding')) {
+      if (!syncBlockInstance.error && ((_resolvedSyncBlockIns = resolvedSyncBlockInstance.data) === null || _resolvedSyncBlockIns === void 0 ? void 0 : _resolvedSyncBlockIns.status) === 'unpublished' && this.newlyAddedSyncBlocks.has(syncBlockInstance.resourceId) && this.onUnpublishedSyncBlockDetected) {
         // Remove from newly added set after checking to prevent duplicate flags
         this.newlyAddedSyncBlocks.delete(syncBlockInstance.resourceId);
         this.onUnpublishedSyncBlockDetected(resolvedSyncBlockInstance.resourceId);
@@ -499,7 +479,7 @@ export class ReferenceSyncBlockStoreManager {
           hasUnexpectedError = true;
         }
         return;
-      } else if (fg('platform_synced_block_dogfooding')) {
+      } else {
         const callbacks = this.subscriptions.get(syncBlockInstance.resourceId);
         const localIds = callbacks ? Object.keys(callbacks) : [];
         localIds.forEach(localId => {
@@ -509,21 +489,19 @@ export class ReferenceSyncBlockStoreManager {
       }
       this.fetchSyncBlockSourceInfo(resolvedSyncBlockInstance.resourceId);
     });
-    if (fg('platform_synced_block_dogfooding')) {
-      if (hasUnexpectedError) {
-        var _this$fetchExperience2;
-        (_this$fetchExperience2 = this.fetchExperience) === null || _this$fetchExperience2 === void 0 ? void 0 : _this$fetchExperience2.failure({
-          reason: 'Unexpected error during fetch'
-        });
-      } else if (hasExpectedError) {
-        var _this$fetchExperience3;
-        (_this$fetchExperience3 = this.fetchExperience) === null || _this$fetchExperience3 === void 0 ? void 0 : _this$fetchExperience3.abort({
-          reason: 'Expected error: NotFound or PermissionDenied'
-        });
-      } else {
-        var _this$fetchExperience4;
-        (_this$fetchExperience4 = this.fetchExperience) === null || _this$fetchExperience4 === void 0 ? void 0 : _this$fetchExperience4.success();
-      }
+    if (hasUnexpectedError) {
+      var _this$fetchExperience2;
+      (_this$fetchExperience2 = this.fetchExperience) === null || _this$fetchExperience2 === void 0 ? void 0 : _this$fetchExperience2.failure({
+        reason: 'Unexpected error during fetch'
+      });
+    } else if (hasExpectedError) {
+      var _this$fetchExperience3;
+      (_this$fetchExperience3 = this.fetchExperience) === null || _this$fetchExperience3 === void 0 ? void 0 : _this$fetchExperience3.abort({
+        reason: 'Expected error: NotFound or PermissionDenied'
+      });
+    } else {
+      var _this$fetchExperience4;
+      (_this$fetchExperience4 = this.fetchExperience) === null || _this$fetchExperience4 === void 0 ? void 0 : _this$fetchExperience4.success();
     }
   }
   updateCacheWithSourceInfo(resourceId, sourceInfo) {
@@ -628,7 +606,7 @@ export class ReferenceSyncBlockStoreManager {
 
     // 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() && fg('platform_synced_block_dogfooding') // in SSR, prefer data provider cache
+    (_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);
     if (cachedData) {
       callback(cachedData);
@@ -804,9 +782,6 @@ export class ReferenceSyncBlockStoreManager {
     return providerFactory;
   }
   getSSRProviders(resourceId) {
-    if (!fg('platform_synced_block_dogfooding')) {
-      return null;
-    }
     if (!this.dataProvider) {
       return null;
     }
@@ -912,6 +887,7 @@ export class ReferenceSyncBlockStoreManager {
     }
     let success = true;
     try {
+      var _this$saveExperience;
       const blocks = [];
 
       // Collect all reference synced blocks on the current document
@@ -923,14 +899,6 @@ export class ReferenceSyncBlockStoreManager {
           });
         });
       });
-      if (!fg('platform_synced_block_dogfooding')) {
-        // It's possible that the last reference block on the document was just deleted,
-        // we still want to write to BE to update reference count
-        if (blocks.length === 0) {
-          this.isCacheDirty = false;
-          return true;
-        }
-      }
       if (!this.dataProvider) {
         throw new Error('Data provider not set');
       }
@@ -941,40 +909,31 @@ export class ReferenceSyncBlockStoreManager {
       // is still making changes, the new changes might not be saved if they all happen
       // exactly at a time when the updateReferenceData is being executed asynchronously.
       this.isCacheDirty = false;
-      if (fg('platform_synced_block_dogfooding')) {
-        var _this$saveExperience;
-        (_this$saveExperience = this.saveExperience) === null || _this$saveExperience === void 0 ? void 0 : _this$saveExperience.start();
-      }
+      (_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$fireAnalyticsEv19;
+        var _this$saveExperience2, _this$fireAnalyticsEv19;
         success = false;
-        if (fg('platform_synced_block_dogfooding')) {
-          var _this$saveExperience2;
-          (_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$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'));
       }
     } catch (error) {
-      var _this$fireAnalyticsEv20;
+      var _this$saveExperience3, _this$fireAnalyticsEv20;
       success = false;
       logException(error, {
         location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
       });
-      if (fg('platform_synced_block_dogfooding')) {
-        var _this$saveExperience3;
-        (_this$saveExperience3 = this.saveExperience) === null || _this$saveExperience3 === void 0 ? void 0 : _this$saveExperience3.failure({
-          reason: error.message
-        });
-      }
+      (_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));
     } finally {
       if (!success) {
         // set isCacheDirty back to true for cases where it failed to update the reference synced blocks on the BE
         this.isCacheDirty = true;
-      } else if (fg('platform_synced_block_dogfooding')) {
+      } else {
         var _this$saveExperience4;
         (_this$saveExperience4 = this.saveExperience) === null || _this$saveExperience4 === void 0 ? void 0 : _this$saveExperience4.success();
       }

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';
@@ -71,6 +70,7 @@ export class SourceSyncBlockStoreManager {
    */
   async flush() {
     try {
+      var _this$saveExperience;
       const bodiedSyncBlockNodes = [];
       const bodiedSyncBlockData = [];
       Array.from(this.syncBlockCache.values()).forEach(syncBlockData => {
@@ -100,10 +100,7 @@ export class SourceSyncBlockStoreManager {
       if (!this.dataProvider) {
         throw new Error('Data provider not set');
       }
-      if (fg('platform_synced_block_dogfooding')) {
-        var _this$saveExperience;
-        (_this$saveExperience = this.saveExperience) === null || _this$saveExperience === void 0 ? void 0 : _this$saveExperience.start({});
-      }
+      (_this$saveExperience = this.saveExperience) === null || _this$saveExperience === void 0 ? void 0 : _this$saveExperience.start({});
       const writeResults = await this.dataProvider.writeNodesData(bodiedSyncBlockNodes, bodiedSyncBlockData);
       writeResults.forEach(result => {
         // set isDirty to true for cases where it failed to save the sync block to the BE
@@ -115,22 +112,18 @@ export class SourceSyncBlockStoreManager {
         }
       });
       if (writeResults.every(result => result.resourceId && !result.error)) {
-        if (fg('platform_synced_block_dogfooding')) {
-          var _this$saveExperience2;
-          (_this$saveExperience2 = this.saveExperience) === null || _this$saveExperience2 === void 0 ? void 0 : _this$saveExperience2.success();
-          writeResults.forEach(result => {
-            if (result.resourceId && !result.error) {
-              var _this$fireAnalyticsEv2;
-              (_this$fireAnalyticsEv2 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv2 === void 0 ? void 0 : _this$fireAnalyticsEv2.call(this, updateSuccessPayload(result.resourceId, false));
-            }
-          });
-        }
+        var _this$saveExperience2;
+        (_this$saveExperience2 = this.saveExperience) === null || _this$saveExperience2 === void 0 ? void 0 : _this$saveExperience2.success();
+        writeResults.forEach(result => {
+          if (result.resourceId && !result.error) {
+            var _this$fireAnalyticsEv2;
+            (_this$fireAnalyticsEv2 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv2 === void 0 ? void 0 : _this$fireAnalyticsEv2.call(this, updateSuccessPayload(result.resourceId, false));
+          }
+        });
         return true;
       } else {
-        if (fg('platform_synced_block_dogfooding')) {
-          var _this$saveExperience3;
-          (_this$saveExperience3 = this.saveExperience) === null || _this$saveExperience3 === void 0 ? void 0 : _this$saveExperience3.failure();
-        }
+        var _this$saveExperience3;
+        (_this$saveExperience3 = this.saveExperience) === null || _this$saveExperience3 === void 0 ? void 0 : _this$saveExperience3.failure();
         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));
@@ -163,12 +156,10 @@ export class SourceSyncBlockStoreManager {
    */
   commitPendingCreation(success) {
     if (success && this.creationCallback) {
+      var _this$fireAnalyticsEv5;
       this.creationCallback();
-      if (fg('platform_synced_block_dogfooding')) {
-        var _this$fireAnalyticsEv5;
-        (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 ? void 0 : _this$fireAnalyticsEv5.call(this, createSuccessPayload(this.pendingResourceId || ''));
-      }
-    } else if (success && !this.creationCallback && fg('platform_synced_block_dogfooding')) {
+      (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 ? void 0 : _this$fireAnalyticsEv5.call(this, createSuccessPayload(this.pendingResourceId || ''));
+    } else if (success && !this.creationCallback) {
       var _this$fireAnalyticsEv6;
       (_this$fireAnalyticsEv6 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv6 === void 0 ? void 0 : _this$fireAnalyticsEv6.call(this, createErrorPayload('creation callback missing', this.pendingResourceId));
     }
@@ -232,11 +223,9 @@ export class SourceSyncBlockStoreManager {
       }).then(result => {
         const resourceId = result.resourceId;
         if (resourceId) {
+          var _this$createExperienc2;
           this.commitPendingCreation(true);
-          if (fg('platform_synced_block_dogfooding')) {
-            var _this$createExperienc2;
-            (_this$createExperienc2 = this.createExperience) === null || _this$createExperienc2 === void 0 ? void 0 : _this$createExperienc2.success();
-          }
+          (_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
@@ -244,28 +233,22 @@ export class SourceSyncBlockStoreManager {
             this.updateSyncBlockData(nodeData);
           }
         } else {
-          var _this$fireAnalyticsEv7;
+          var _this$createExperienc3, _this$fireAnalyticsEv7;
           this.commitPendingCreation(false);
-          if (fg('platform_synced_block_dogfooding')) {
-            var _this$createExperienc3;
-            (_this$createExperienc3 = this.createExperience) === null || _this$createExperienc3 === void 0 ? void 0 : _this$createExperienc3.failure({
-              reason: result.error || 'Failed to create bodied sync block'
-            });
-          }
+          (_this$createExperienc3 = this.createExperience) === null || _this$createExperienc3 === void 0 ? void 0 : _this$createExperienc3.failure({
+            reason: result.error || 'Failed to create bodied sync block'
+          });
           (_this$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 ? void 0 : _this$fireAnalyticsEv7.call(this, createErrorPayload(result.error || 'Failed to create bodied sync block', resourceId));
         }
       }).catch(error => {
-        var _this$fireAnalyticsEv8;
+        var _this$createExperienc4, _this$fireAnalyticsEv8;
         this.commitPendingCreation(false);
         logException(error, {
           location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
         });
-        if (fg('platform_synced_block_dogfooding')) {
-          var _this$createExperienc4;
-          (_this$createExperienc4 = this.createExperience) === null || _this$createExperienc4 === void 0 ? void 0 : _this$createExperienc4.failure({
-            reason: error.message
-          });
-        }
+        (_this$createExperienc4 = this.createExperience) === null || _this$createExperienc4 === void 0 ? void 0 : _this$createExperienc4.failure({
+          reason: error.message
+        });
         (_this$fireAnalyticsEv8 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv8 === void 0 ? void 0 : _this$fireAnalyticsEv8.call(this, createErrorPayload(error.message, resourceId));
       });
       this.registerPendingCreation(resourceId);
@@ -282,62 +265,51 @@ export class SourceSyncBlockStoreManager {
   }
   async delete(syncBlockIds, onDelete, onDeleteCompleted, reason) {
     try {
+      var _this$deleteExperienc;
       if (!this.dataProvider) {
         throw new Error('Data provider not set');
       }
       syncBlockIds.forEach(Ids => {
         this.setPendingDeletion(Ids, true);
       });
-      if (fg('platform_synced_block_dogfooding')) {
-        var _this$deleteExperienc;
-        (_this$deleteExperienc = this.deleteExperience) === null || _this$deleteExperienc === void 0 ? void 0 : _this$deleteExperienc.start({});
-      }
+      (_this$deleteExperienc = this.deleteExperience) === null || _this$deleteExperienc === void 0 ? void 0 : _this$deleteExperienc.start({});
       const results = await this.dataProvider.deleteNodesData(syncBlockIds.map(attrs => attrs.resourceId), reason);
       let callback;
       const isDeleteSuccessful = results.every(result => result.success);
       onDeleteCompleted(isDeleteSuccessful);
       if (isDeleteSuccessful) {
+        var _this$deleteExperienc2;
         onDelete();
         callback = Ids => this.syncBlockCache.delete(Ids.resourceId);
         this.clearPendingDeletion();
-        if (fg('platform_synced_block_dogfooding')) {
-          var _this$deleteExperienc2;
-          (_this$deleteExperienc2 = this.deleteExperience) === null || _this$deleteExperienc2 === void 0 ? void 0 : _this$deleteExperienc2.success();
-          results.forEach(result => {
-            var _this$fireAnalyticsEv0;
-            (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 ? void 0 : _this$fireAnalyticsEv0.call(this, deleteSuccessPayload(result.resourceId));
-          });
-        }
+        (_this$deleteExperienc2 = this.deleteExperience) === null || _this$deleteExperienc2 === void 0 ? void 0 : _this$deleteExperienc2.success();
+        results.forEach(result => {
+          var _this$fireAnalyticsEv0;
+          (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 ? void 0 : _this$fireAnalyticsEv0.call(this, deleteSuccessPayload(result.resourceId));
+        });
       } else {
+        var _this$deleteExperienc3;
         callback = Ids => {
           this.setPendingDeletion(Ids, false);
         };
-        if (fg('platform_synced_block_dogfooding')) {
-          var _this$deleteExperienc3;
-          (_this$deleteExperienc3 = this.deleteExperience) === null || _this$deleteExperienc3 === void 0 ? void 0 : _this$deleteExperienc3.failure();
-          results.forEach(result => {
-            if (result.success) {
-              var _this$fireAnalyticsEv1;
-              (_this$fireAnalyticsEv1 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv1 === void 0 ? void 0 : _this$fireAnalyticsEv1.call(this, deleteSuccessPayload(result.resourceId));
-            } else {
-              var _this$fireAnalyticsEv10;
-              (_this$fireAnalyticsEv10 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv10 === void 0 ? void 0 : _this$fireAnalyticsEv10.call(this, deleteErrorPayload(result.error || 'Failed to delete synced block', result.resourceId));
-            }
-          });
-        } else {
-          results.filter(result => result.resourceId === undefined).forEach(result => {
-            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'));
-          });
-        }
+        (_this$deleteExperienc3 = this.deleteExperience) === null || _this$deleteExperienc3 === void 0 ? void 0 : _this$deleteExperienc3.failure();
+        results.forEach(result => {
+          if (result.success) {
+            var _this$fireAnalyticsEv1;
+            (_this$fireAnalyticsEv1 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv1 === void 0 ? void 0 : _this$fireAnalyticsEv1.call(this, deleteSuccessPayload(result.resourceId));
+          } else {
+            var _this$fireAnalyticsEv10;
+            (_this$fireAnalyticsEv10 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv10 === void 0 ? void 0 : _this$fireAnalyticsEv10.call(this, deleteErrorPayload(result.error || 'Failed to delete synced block', result.resourceId));
+          }
+        });
       }
       syncBlockIds.forEach(callback);
       return isDeleteSuccessful;
     } catch (error) {
       syncBlockIds.forEach(Ids => {
-        var _this$fireAnalyticsEv12;
+        var _this$fireAnalyticsEv11;
         this.setPendingDeletion(Ids, false);
-        (_this$fireAnalyticsEv12 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv12 === void 0 ? void 0 : _this$fireAnalyticsEv12.call(this, deleteErrorPayload(error.message, Ids.resourceId));
+        (_this$fireAnalyticsEv11 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv11 === void 0 ? void 0 : _this$fireAnalyticsEv11.call(this, deleteErrorPayload(error.message, Ids.resourceId));
       });
       logException(error, {
         location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
@@ -400,33 +372,29 @@ export class SourceSyncBlockStoreManager {
   }
   getSyncBlockSourceInfo(localId) {
     try {
+      var _this$fetchSourceInfo;
       if (!this.dataProvider) {
         throw new Error('Data provider not set');
       }
-      if (fg('platform_synced_block_dogfooding')) {
-        var _this$fetchSourceInfo;
-        (_this$fetchSourceInfo = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo === void 0 ? void 0 : _this$fetchSourceInfo.start();
-      }
+      (_this$fetchSourceInfo = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo === void 0 ? void 0 : _this$fetchSourceInfo.start();
       return this.dataProvider.fetchSyncBlockSourceInfo(localId, undefined, undefined, this.fireAnalyticsEvent).then(sourceInfo => {
-        if (fg('platform_synced_block_dogfooding')) {
-          if (!sourceInfo) {
-            var _this$fetchSourceInfo2;
-            (_this$fetchSourceInfo2 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo2 === void 0 ? void 0 : _this$fetchSourceInfo2.failure({
-              reason: 'No source info returned'
-            });
-          } else {
-            var _this$fetchSourceInfo3;
-            (_this$fetchSourceInfo3 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo3 === void 0 ? void 0 : _this$fetchSourceInfo3.success();
-          }
+        if (!sourceInfo) {
+          var _this$fetchSourceInfo2;
+          (_this$fetchSourceInfo2 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo2 === void 0 ? void 0 : _this$fetchSourceInfo2.failure({
+            reason: 'No source info returned'
+          });
+        } else {
+          var _this$fetchSourceInfo3;
+          (_this$fetchSourceInfo3 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo3 === void 0 ? void 0 : _this$fetchSourceInfo3.success();
         }
         return sourceInfo;
       });
     } catch (error) {
-      var _this$fireAnalyticsEv13;
+      var _this$fireAnalyticsEv12;
       logException(error, {
         location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
       });
-      (_this$fireAnalyticsEv13 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv13 === void 0 ? void 0 : _this$fireAnalyticsEv13.call(this, getSourceInfoErrorPayload(error.message));
+      (_this$fireAnalyticsEv12 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv12 === void 0 ? void 0 : _this$fireAnalyticsEv12.call(this, getSourceInfoErrorPayload(error.message));
       return Promise.resolve(undefined);
     }
   }
@@ -437,11 +405,11 @@ export class SourceSyncBlockStoreManager {
       }
       return this.dataProvider.fetchReferences(resourceId, true);
     } catch (error) {
-      var _this$fireAnalyticsEv14;
+      var _this$fireAnalyticsEv13;
       logException(error, {
         location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
       });
-      (_this$fireAnalyticsEv14 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv14 === void 0 ? void 0 : _this$fireAnalyticsEv14.call(this, fetchReferencesErrorPayload(error.message));
+      (_this$fireAnalyticsEv13 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv13 === void 0 ? void 0 : _this$fireAnalyticsEv13.call(this, fetchReferencesErrorPayload(error.message));
       return Promise.resolve({
         error: SyncBlockError.Errored
       });

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

@@ -105,10 +105,8 @@ export class SyncBlockStoreManager {
     this.fireAnalyticsEvent = fireAnalyticsEvent;
     this.referenceSyncBlockStoreManager.setFireAnalyticsEvent(fireAnalyticsEvent);
     this.sourceSyncBlockStoreManager.setFireAnalyticsEvent(fireAnalyticsEvent);
-    if (fg('platform_synced_block_dogfooding')) {
-      this.fetchReferencesExperience = getFetchReferencesExperience(fireAnalyticsEvent);
-      this.fetchSourceInfoExperience = getFetchSourceInfoExperience(fireAnalyticsEvent);
-    }
+    this.fetchReferencesExperience = getFetchReferencesExperience(fireAnalyticsEvent);
+    this.fetchSourceInfoExperience = getFetchSourceInfoExperience(fireAnalyticsEvent);
   }
   get referenceManager() {
     return this.referenceSyncBlockStoreManager;

package/dist/es2019/utils/errorHandling.js

@@ -1,5 +1,4 @@
 import { ACTION, ACTION_SUBJECT, EVENT_TYPE, ACTION_SUBJECT_ID } from '@atlaskit/editor-common/analytics';
-import { fg } from '@atlaskit/platform-feature-flags';
 export const stringifyError = error => {
   try {
     return JSON.stringify(error);
@@ -12,13 +11,11 @@ export const getErrorPayload = (actionSubjectId, error, resourceId) => ({
   actionSubject: ACTION_SUBJECT.SYNCED_BLOCK,
   actionSubjectId,
   eventType: EVENT_TYPE.OPERATIONAL,
-  attributes: fg('platform_synced_block_dogfooding') ? {
+  attributes: {
     error,
     ...(resourceId && {
       resourceId
     })
-  } : {
-    error
   }
 });
 export const fetchErrorPayload = (error, resourceId) => getErrorPayload(ACTION_SUBJECT_ID.SYNCED_BLOCK_FETCH, error, resourceId);