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

package/dist/esm/clients/confluence/sourceInfo.js

@@ -242,7 +242,7 @@ export var fetchConfluencePageInfo = /*#__PURE__*/function () {
     return _regeneratorRuntime.wrap(function _callee5$(_context5) {
       while (1) switch (_context5.prev = _context5.next) {
         case 0:
-          if (!(isUnpublished && fg('platform_synced_block_patch_1'))) {
+          if (!isUnpublished) {
             _context5.next = 5;
             break;
           }

package/dist/esm/providers/block-service/blockServiceAPI.js

@@ -11,7 +11,6 @@ function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length)
 import _regeneratorRuntime from "@babel/runtime/regenerator";
 /* eslint-disable require-unicode-regexp  */
 import { useMemo } from 'react';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { generateBlockAri, generateBlockAriFromReference } from '../../clients/block-service/ari';
 import { batchRetrieveSyncedBlocks, BlockError, createSyncedBlock, deleteSyncedBlock, getReferenceSyncedBlocks, getReferenceSyncedBlocksByBlockAri, getSyncedBlockContent, updateReferenceSyncedBlockOnDocument, updateSyncedBlock } from '../../clients/block-service/blockService';
 import { subscribeToBlockUpdates as subscribeToBlockUpdatesWS } from '../../clients/block-service/blockSubscription';
@@ -237,7 +236,6 @@ var _batchFetchData = /*#__PURE__*/function () {
           _context2.prev = 5;
           _context2.next = 8;
           return batchRetrieveSyncedBlocks({
-            documentAri: parentAri,
             blockIdentifiers: blockIdentifiers
           });
         case 8:
@@ -432,8 +430,7 @@ var BlockServiceADFFetchProvider = /*#__PURE__*/function () {
               _context3.prev = 1;
               _context3.next = 4;
               return getSyncedBlockContent({
-                blockAri: blockAri,
-                documentAri: this.parentAri
+                blockAri: blockAri
               });
             case 4:
               blockContentResponse = _context3.sent;
@@ -635,15 +632,13 @@ var BlockServiceADFWriteProvider = /*#__PURE__*/function () {
       parentAri = _ref5.parentAri,
       parentId = _ref5.parentId,
       product = _ref5.product,
-      getVersion = _ref5.getVersion,
-      isParentUnpublished = _ref5.isParentUnpublished;
+      getVersion = _ref5.getVersion;
     _classCallCheck(this, BlockServiceADFWriteProvider);
     this.cloudId = cloudId;
     this.parentAri = parentAri;
     this.parentId = parentId;
     this.product = product;
     this.getVersion = getVersion;
-    this.isParentUnpublished = isParentUnpublished;
   }
 
   // it will first try to update and if it can't (404) then it will try to create
@@ -728,7 +723,6 @@ var BlockServiceADFWriteProvider = /*#__PURE__*/function () {
     key: "createData",
     value: function () {
       var _createData = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee7(data) {
-        var _this$isParentUnpubli;
         var resourceId, blockAri, stepVersion, status;
         return _regeneratorRuntime.wrap(function _callee7$(_context7) {
           while (1) switch (_context7.prev = _context7.next) {
@@ -762,7 +756,7 @@ var BlockServiceADFWriteProvider = /*#__PURE__*/function () {
               _context7.t0 = undefined;
             case 11:
               stepVersion = _context7.t0;
-              status = fg('platform_synced_block_patch_1') ? 'unpublished' : (_this$isParentUnpubli = this.isParentUnpublished) !== null && _this$isParentUnpubli !== void 0 && _this$isParentUnpubli.call(this) ? 'unpublished' : data.status || 'active';
+              status = 'unpublished';
               _context7.prev = 13;
               _context7.next = 16;
               return createSyncedBlock({
@@ -959,8 +953,7 @@ var createBlockServiceAPIProviders = function createBlockServiceAPIProviders(_re
     parentAri = _ref6.parentAri,
     parentId = _ref6.parentId,
     product = _ref6.product,
-    getVersion = _ref6.getVersion,
-    isParentUnpublished = _ref6.isParentUnpublished;
+    getVersion = _ref6.getVersion;
   return {
     fetchProvider: new BlockServiceADFFetchProvider({
       cloudId: cloudId,
@@ -971,8 +964,7 @@ var createBlockServiceAPIProviders = function createBlockServiceAPIProviders(_re
       parentAri: parentAri,
       parentId: parentId,
       product: product,
-      getVersion: getVersion,
-      isParentUnpublished: isParentUnpublished
+      getVersion: getVersion
     })
   };
 };
@@ -981,18 +973,16 @@ export var useMemoizedBlockServiceAPIProviders = function useMemoizedBlockServic
     parentAri = _ref7.parentAri,
     parentId = _ref7.parentId,
     product = _ref7.product,
-    getVersion = _ref7.getVersion,
-    isParentUnpublished = _ref7.isParentUnpublished;
+    getVersion = _ref7.getVersion;
   return useMemo(function () {
     return createBlockServiceAPIProviders({
       cloudId: cloudId,
       parentAri: parentAri,
       parentId: parentId,
       product: product,
-      getVersion: getVersion,
-      isParentUnpublished: isParentUnpublished
+      getVersion: getVersion
     });
-  }, [cloudId, parentAri, parentId, product, getVersion, isParentUnpublished]);
+  }, [cloudId, parentAri, parentId, product, getVersion]);
 };
 var createBlockServiceFetchOnlyAPIProvider = function createBlockServiceFetchOnlyAPIProvider(_ref8) {
   var cloudId = _ref8.cloudId,

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

@@ -11,7 +11,6 @@ function _unsupportedIterableToArray(r, a) { if (r) { if ("string" == typeof r)
 function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length); for (var e = 0, n = Array(a); e < a; e++) n[e] = r[e]; return n; }
 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';
@@ -66,12 +65,10 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
         });
       });
     }));
-    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();
@@ -214,16 +211,9 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
   }, {
     key: "getInitialSyncBlockData",
     value: function getInitialSyncBlockData(resourceId) {
-      var _this$dataProvider2;
+      var _this$dataProvider;
       var syncBlockNode = createSyncBlockNode('', resourceId);
-      if (isSSR() || fg('platform_synced_block_patch_1')) {
-        var _this$dataProvider;
-        // 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 || (_this$dataProvider = _this$dataProvider.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider === void 0 ? void 0 : _this$dataProvider.data;
-      }
-      return this.getFromCache(resourceId) || ((_this$dataProvider2 = this.dataProvider) === null || _this$dataProvider2 === void 0 || (_this$dataProvider2 = _this$dataProvider2.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider2 === void 0 ? void 0 : _this$dataProvider2.data);
+      return (_this$dataProvider = this.dataProvider) === null || _this$dataProvider === void 0 || (_this$dataProvider = _this$dataProvider.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider === void 0 ? void 0 : _this$dataProvider.data;
     }
 
     /**
@@ -328,13 +318,13 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
   }, {
     key: "setupGraphQLSubscription",
     value: function setupGraphQLSubscription(resourceId) {
-      var _this$dataProvider3,
+      var _this$dataProvider2,
         _this4 = this;
       // 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;
       }
       var unsubscribe = this.dataProvider.subscribeToBlockUpdates(resourceId, function (syncBlockInstance) {
@@ -496,9 +486,7 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
           return Promise.resolve(undefined);
         }
         (_this$fetchSourceInfo = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo === void 0 || _this$fetchSourceInfo.start({});
-
-        // Only use unpublished endpoint if feature flag is enabled
-        var isUnpublished = fg('platform_synced_block_patch_1') && ((_existingSyncBlock$da = existingSyncBlock.data) === null || _existingSyncBlock$da === void 0 ? void 0 : _existingSyncBlock$da.status) === 'unpublished';
+        var isUnpublished = ((_existingSyncBlock$da = existingSyncBlock.data) === null || _existingSyncBlock$da === void 0 ? void 0 : _existingSyncBlock$da.status) === 'unpublished';
         var sourceInfoPromise = this.dataProvider.fetchSyncBlockSourceInfo(blockInstanceId, sourceAri, product, this.fireAnalyticsEvent, true,
         // hasAccess
         'edit',
@@ -779,16 +767,11 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
     value: function updateCache(syncBlock) {
       var resourceId = syncBlock.resourceId;
       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 || _this$dataProvider4.updateCache(_defineProperty({}, resourceId, syncBlock), {
-            strategy: 'merge',
-            source: 'network'
-          });
-        } else {
-          this.syncBlockCache.set(resourceId, syncBlock);
-        }
+        var _this$dataProvider3;
+        (_this$dataProvider3 = this.dataProvider) === null || _this$dataProvider3 === void 0 || _this$dataProvider3.updateCache(_defineProperty({}, resourceId, syncBlock), {
+          strategy: 'merge',
+          source: 'network'
+        });
         var callbacks = this.subscriptions.get(resourceId);
         if (callbacks) {
           Object.values(callbacks).forEach(function (callback) {
@@ -810,25 +793,15 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
   }, {
     key: "getFromCache",
     value: function getFromCache(resourceId) {
-      if (fg('platform_synced_block_patch_1')) {
-        var _this$dataProvider5;
-        // Use the cache in dataProvider
-        var syncBlockNode = createSyncBlockNode('', resourceId);
-        return (_this$dataProvider5 = this.dataProvider) === null || _this$dataProvider5 === void 0 || (_this$dataProvider5 = _this$dataProvider5.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider5 === void 0 ? void 0 : _this$dataProvider5.data;
-      }
-      return this.syncBlockCache.get(resourceId);
+      var _this$dataProvider4;
+      var syncBlockNode = createSyncBlockNode('', resourceId);
+      return (_this$dataProvider4 = this.dataProvider) === null || _this$dataProvider4 === void 0 || (_this$dataProvider4 = _this$dataProvider4.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider4 === void 0 ? void 0 : _this$dataProvider4.data;
     }
   }, {
     key: "deleteFromCache",
     value: function 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 || _this$dataProvider6.removeFromCache([resourceId]);
-      }
+      var _this$dataProvider5;
+      (_this$dataProvider5 = this.dataProvider) === null || _this$dataProvider5 === void 0 || _this$dataProvider5.removeFromCache([resourceId]);
       this.providerFactories.delete(resourceId);
     }
   }, {
@@ -850,9 +823,7 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
   }, {
     key: "subscribeToSyncBlock",
     value: function subscribeToSyncBlock(resourceId, localId, callback) {
-      var _this$dataProvider7,
-        _this$dataProvider8,
-        _this$dataProvider9,
+      var _this$dataProvider6,
         _this0 = this;
       // Cancel any pending cache deletion for this resourceId.
       // This handles the case where a block is moved - the old component unmounts
@@ -879,23 +850,11 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
       var syncBlockNode = createSyncBlockNode(localId, resourceId);
 
       // call the callback immediately if we have cached data
-      var 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 || (_this$dataProvider7 = _this$dataProvider7.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider7 === void 0 ? void 0 : _this$dataProvider7.data : isSSR() // in SSR, prefer data provider cache
-      ? ((_this$dataProvider8 = this.dataProvider) === null || _this$dataProvider8 === void 0 || (_this$dataProvider8 = _this$dataProvider8.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider8 === void 0 ? void 0 : _this$dataProvider8.data) || this.getFromCache(resourceId) : this.getFromCache(resourceId) || ((_this$dataProvider9 = this.dataProvider) === null || _this$dataProvider9 === void 0 || (_this$dataProvider9 = _this$dataProvider9.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider9 === void 0 ? void 0 : _this$dataProvider9.data);
+      var cachedData = (_this$dataProvider6 = this.dataProvider) === null || _this$dataProvider6 === void 0 || (_this$dataProvider6 = _this$dataProvider6.getNodeDataFromCache(syncBlockNode)) === null || _this$dataProvider6 === void 0 ? void 0 : _this$dataProvider6.data;
       if (cachedData) {
         callback(cachedData);
       } else {
-        if (fg('platform_synced_block_patch_1')) {
-          this.debouncedBatchedFetchSyncBlocks(resourceId);
-        } else {
-          this.fetchSyncBlocksData([syncBlockNode]).catch(function (error) {
-            var _this0$fireAnalyticsE;
-            logException(error, {
-              location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
-            });
-            (_this0$fireAnalyticsE = _this0.fireAnalyticsEvent) === null || _this0$fireAnalyticsE === void 0 || _this0$fireAnalyticsE.call(_this0, fetchErrorPayload(error.message, resourceId));
-          });
-        }
+        this.debouncedBatchedFetchSyncBlocks(resourceId);
       }
 
       // Set up GraphQL subscription if real-time subscriptions are enabled
@@ -1345,7 +1304,7 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
   }, {
     key: "destroy",
     value: function destroy() {
-      var _this$saveExperience5, _this$fetchExperience0, _this$fetchSourceInfo2;
+      var _this$dataProvider7, _this$saveExperience5, _this$fetchExperience0, _this$fetchSourceInfo2;
       // Cancel any queued flush to prevent it from running after destroy
       if (this.queuedFlushTimeout) {
         clearTimeout(this.queuedFlushTimeout);
@@ -1354,20 +1313,13 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
 
       // 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 || _this$dataProvider0.resetCache();
-      } else {
-        this.syncBlockCache.clear();
-      }
+      (_this$dataProvider7 = this.dataProvider) === null || _this$dataProvider7 === 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/esm/store-manager/sourceSyncBlockStoreManager.js

@@ -6,7 +6,6 @@ import _regeneratorRuntime from "@babel/runtime/regenerator";
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 import { logException } from '@atlaskit/editor-common/monitoring';
-import { fg } from '@atlaskit/platform-feature-flags';
 import { SyncBlockError } from '../common/types';
 import { updateErrorPayload, createErrorPayload, deleteErrorPayload, updateCacheErrorPayload, getSourceInfoErrorPayload, updateSuccessPayload, createSuccessPayload, deleteSuccessPayload, fetchReferencesErrorPayload } from '../utils/errorHandling';
 import { getCreateSourceExperience, getDeleteSourceExperience, getSaveSourceExperience, getFetchSourceInfoExperience } from '../utils/experienceTracking';
@@ -99,7 +98,7 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
                 // - 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 && !(_this2.isPendingCreation(syncBlockData.resourceId) && fg('platform_synced_block_patch_1'))) {
+                if (!syncBlockData.pendingDeletion && syncBlockData.isDirty && !_this2.isPendingCreation(syncBlockData.resourceId)) {
                   bodiedSyncBlockNodes.push({
                     type: 'bodiedSyncBlock',
                     attrs: {
@@ -113,10 +112,10 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
                   // 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)
-                  var dataToFlush = fg('platform_synced_block_patch_1') ? _objectSpread(_objectSpread({}, syncBlockData), {}, {
+                  // When flushing, set status to 'active' so the block is published
+                  var dataToFlush = _objectSpread(_objectSpread({}, syncBlockData), {}, {
                     status: 'active'
-                  }) : syncBlockData;
+                  });
                   bodiedSyncBlockData.push(dataToFlush);
                 }
               });
@@ -190,28 +189,13 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
         return _flush.apply(this, arguments);
       }
       return flush;
-    }() // Remove this method and pendingResourceId when cleaning up platform_synced_block_patch_1
-    )
-  }, {
-    key: "registerPendingCreation",
-    value: function registerPendingCreation(resourceId) {
-      this.pendingResourceId = resourceId;
-    }
+    }())
   }, {
     key: "isPendingCreation",
     value: function 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
-     */
-  }, {
-    key: "registerCreationCallback",
-    value: function registerCreationCallback(callback) {
-      this.creationCallback = callback;
-    }
-
     /**
      *  Fires callback to insert node (if creation is successful) and clears pending creation data
      * @param success
@@ -219,47 +203,23 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
   }, {
     key: "commitPendingCreation",
     value: function commitPendingCreation(success, resourceId) {
-      if (fg('platform_synced_block_patch_1')) {
-        var onCompletion = this.creationCompletionCallbacks.get(resourceId);
-        if (onCompletion) {
-          this.creationCompletionCallbacks.delete(resourceId);
-          onCompletion(success);
-        } else {
-          var _this$fireAnalyticsEv3;
-          (_this$fireAnalyticsEv3 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv3 === void 0 || _this$fireAnalyticsEv3.call(this, createErrorPayload('creation complete callback missing', resourceId));
-        }
-        if (success) {
-          var _this$fireAnalyticsEv4;
-          (_this$fireAnalyticsEv4 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv4 === void 0 || _this$fireAnalyticsEv4.call(this, createSuccessPayload(resourceId || ''));
-        } else {
-          var _this$fireAnalyticsEv5;
-          // Delete the node from cache if fail to create so it's not flushed to BE
-          this.syncBlockCache.delete(resourceId || '');
-          (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 || _this$fireAnalyticsEv5.call(this, createErrorPayload('Fail to create bodied sync block', resourceId));
-        }
+      var onCompletion = this.creationCompletionCallbacks.get(resourceId);
+      if (onCompletion) {
+        this.creationCompletionCallbacks.delete(resourceId);
+        onCompletion(success);
       } else {
-        if (success && this.creationCallback) {
-          var _this$fireAnalyticsEv6;
-          this.creationCallback();
-          (_this$fireAnalyticsEv6 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv6 === void 0 || _this$fireAnalyticsEv6.call(this, createSuccessPayload(this.pendingResourceId || ''));
-        } else if (success && !this.creationCallback) {
-          var _this$fireAnalyticsEv7;
-          (_this$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 || _this$fireAnalyticsEv7.call(this, createErrorPayload('creation callback missing', this.pendingResourceId));
-        }
-        this.pendingResourceId = undefined;
+        var _this$fireAnalyticsEv3;
+        (_this$fireAnalyticsEv3 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv3 === void 0 || _this$fireAnalyticsEv3.call(this, createErrorPayload('creation complete callback missing', resourceId));
+      }
+      if (success) {
+        var _this$fireAnalyticsEv4;
+        (_this$fireAnalyticsEv4 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv4 === void 0 || _this$fireAnalyticsEv4.call(this, createSuccessPayload(resourceId || ''));
+      } else {
+        var _this$fireAnalyticsEv5;
+        // Delete the node from cache if fail to create so it's not flushed to BE
+        this.syncBlockCache.delete(resourceId || '');
+        (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 || _this$fireAnalyticsEv5.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
-     */
-  }, {
-    key: "hasPendingCreation",
-    value: function hasPendingCreation() {
-      return fg('platform_synced_block_patch_1') ? this.creationCompletionCallbacks.size > 0 : !!this.pendingResourceId;
     }
   }, {
     key: "registerConfirmationCallback",
@@ -309,9 +269,7 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
         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 || _this$createExperienc.start({});
         this.dataProvider.createNodeData({
           content: [],
@@ -319,18 +277,10 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
           resourceId: resourceId
         }).then(function (result) {
           var resourceId = result.resourceId || '';
-          if (resourceId && (!fg('platform_synced_block_patch_1') || !result.error)) {
+          if (resourceId && !result.error) {
             var _this4$createExperien;
             _this4.commitPendingCreation(true, resourceId);
             (_this4$createExperien = _this4.createExperience) === null || _this4$createExperien === void 0 || _this4$createExperien.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) {
-                _this4.updateSyncBlockData(nodeData);
-              }
-            }
           } else {
             var _this4$createExperien2, _this4$fireAnalyticsE;
             _this4.commitPendingCreation(false, resourceId);
@@ -350,18 +300,15 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
           });
           (_this4$fireAnalyticsE2 = _this4.fireAnalyticsEvent) === null || _this4$fireAnalyticsE2 === void 0 || _this4$fireAnalyticsE2.call(_this4, createErrorPayload(error.message, resourceId));
         });
-        if (!fg('platform_synced_block_patch_1')) {
-          this.registerPendingCreation(resourceId);
-        }
       } catch (error) {
-        var _this$fireAnalyticsEv8;
-        if (fg('platform_synced_block_patch_1') ? this.isPendingCreation(resourceId) : this.hasPendingCreation()) {
+        var _this$fireAnalyticsEv6;
+        if (this.isPendingCreation(resourceId)) {
           this.commitPendingCreation(false, resourceId);
         }
         logException(error, {
           location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
         });
-        (_this$fireAnalyticsEv8 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv8 === void 0 || _this$fireAnalyticsEv8.call(this, createErrorPayload(error.message));
+        (_this$fireAnalyticsEv6 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv6 === void 0 || _this$fireAnalyticsEv6.call(this, createErrorPayload(error.message));
       }
     }
   }, {
@@ -572,11 +519,11 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
           return sourceInfo;
         });
       } catch (error) {
-        var _this$fireAnalyticsEv9;
+        var _this$fireAnalyticsEv7;
         logException(error, {
           location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
         });
-        (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 || _this$fireAnalyticsEv9.call(this, getSourceInfoErrorPayload(error.message));
+        (_this$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 || _this$fireAnalyticsEv7.call(this, getSourceInfoErrorPayload(error.message));
         return Promise.resolve(undefined);
       }
     }
@@ -589,11 +536,11 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
         }
         return this.dataProvider.fetchReferences(resourceId, true);
       } catch (error) {
-        var _this$fireAnalyticsEv0;
+        var _this$fireAnalyticsEv8;
         logException(error, {
           location: 'editor-synced-block-provider/sourceSyncBlockStoreManager'
         });
-        (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 || _this$fireAnalyticsEv0.call(this, fetchReferencesErrorPayload(error.message));
+        (_this$fireAnalyticsEv8 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv8 === void 0 || _this$fireAnalyticsEv8.call(this, fetchReferencesErrorPayload(error.message));
         return Promise.resolve({
           error: SyncBlockError.Errored
         });
@@ -605,9 +552,7 @@ export var SourceSyncBlockStoreManager = /*#__PURE__*/function () {
       var _this$saveExperience4, _this$createExperienc2, _this$deleteExperienc4, _this$fetchSourceInfo2;
       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 || _this$saveExperience4.abort({
         reason: 'editorDestroyed'

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

@@ -7,7 +7,6 @@ function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbol
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 import { 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';
@@ -28,7 +27,7 @@ export var SyncBlockStoreManager = /*#__PURE__*/function () {
     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);
   }
   return _createClass(SyncBlockStoreManager, [{
     key: "fetchReferencesSourceInfo",

package/dist/types/clients/block-service/blockService.d.ts

@@ -67,7 +67,6 @@ export declare const isBlockContentResponse: (response: BlockContentResponse | E
 export declare const getReferenceSyncedBlocks: (documentAri: string) => Promise<ReferenceSyncedBlockResponse>;
 export type GetSyncedBlockContentRequest = {
     blockAri: string;
-    documentAri?: string;
 };
 export type DeleteSyncedBlockRequest = {
     blockAri: string;
@@ -99,7 +98,6 @@ type UpdateReferenceSyncedBlockOnDocumentRequest = {
 };
 export type BatchRetrieveSyncedBlocksRequest = {
     blockIdentifiers: BlockIdentifier[];
-    documentAri: string;
 };
 type BlockIdentifier = {
     blockAri: string;
@@ -125,13 +123,12 @@ export declare const getSyncedBlockContent: ({ blockAri, }: GetSyncedBlockConten
 /**
  * Batch retrieves multiple synced blocks by their ARIs.
  *
- * Calls the Block Service API endpoint: `POST /v1/block/batch-retrieve`
- * or GraphQL query `blockService_batchRetrieveBlocks` when feature flag is enabled
+ * Calls the Block Service GraphQL API: `blockService_batchRetrieveBlocks`
  *
- * @param blockAris - Array of block ARIs to retrieve
+ * @param blockIdentifiers - Array of block identifiers to retrieve
  * @returns A promise containing arrays of successfully fetched blocks and any errors encountered
  */
-export declare const batchRetrieveSyncedBlocks: ({ blockIdentifiers, documentAri, }: BatchRetrieveSyncedBlocksRequest) => Promise<BatchRetrieveSyncedBlocksResponse>;
+export declare const batchRetrieveSyncedBlocks: ({ blockIdentifiers, }: BatchRetrieveSyncedBlocksRequest) => Promise<BatchRetrieveSyncedBlocksResponse>;
 export declare const deleteSyncedBlock: ({ blockAri, deleteReason, }: DeleteSyncedBlockRequest) => Promise<void>;
 export declare const updateSyncedBlock: ({ blockAri, content, stepVersion, status, }: UpdateSyncedBlockRequest) => Promise<void>;
 export declare const createSyncedBlock: ({ blockAri, blockInstanceId, sourceAri, product, content, stepVersion, status, }: CreateSyncedBlockRequest) => Promise<BlockContentResponse>;

package/dist/types/providers/block-service/blockServiceAPI.d.ts

@@ -64,7 +64,6 @@ declare class BlockServiceADFFetchProvider implements ADFFetchProvider {
 interface BlockServiceADFWriteProviderProps {
     cloudId: string;
     getVersion?: () => Promise<number | undefined>;
-    isParentUnpublished?: () => boolean;
     parentAri: string | undefined;
     parentId?: string;
     product: SyncBlockProduct;
@@ -76,10 +75,9 @@ declare class BlockServiceADFWriteProvider implements ADFWriteProvider {
     private cloudId;
     private parentId?;
     private getVersion?;
-    private isParentUnpublished?;
     product: SyncBlockProduct;
     parentAri: string | undefined;
-    constructor({ cloudId, parentAri, parentId, product, getVersion, isParentUnpublished, }: BlockServiceADFWriteProviderProps);
+    constructor({ cloudId, parentAri, parentId, product, getVersion, }: BlockServiceADFWriteProviderProps);
     writeData(data: SyncBlockData): Promise<WriteSyncBlockResult>;
     createData(data: SyncBlockData): Promise<WriteSyncBlockResult>;
     deleteData(resourceId: string, deleteReason: string | undefined): Promise<DeleteSyncBlockResult>;
@@ -90,12 +88,11 @@ declare class BlockServiceADFWriteProvider implements ADFWriteProvider {
 interface BlockServiceAPIProvidersProps {
     cloudId: string;
     getVersion?: () => Promise<number | undefined>;
-    isParentUnpublished?: () => boolean;
     parentAri: string | undefined;
     parentId?: string;
     product: SyncBlockProduct;
 }
-export declare const useMemoizedBlockServiceAPIProviders: ({ cloudId, parentAri, parentId, product, getVersion, isParentUnpublished, }: BlockServiceAPIProvidersProps) => {
+export declare const useMemoizedBlockServiceAPIProviders: ({ cloudId, parentAri, parentId, product, getVersion, }: BlockServiceAPIProvidersProps) => {
     fetchProvider: BlockServiceADFFetchProvider;
     writeProvider: BlockServiceADFWriteProvider;
 };

package/dist/types/store-manager/referenceSyncBlockStoreManager.d.ts

@@ -6,14 +6,12 @@ import { type ResourceId, type SyncBlockNode, type SyncBlockPrefetchData } from
 import type { SyncBlockInstance, SubscriptionCallback, SyncBlockDataProvider, TitleSubscriptionCallback, SyncBlockSourceInfo } from '../providers/types';
 export declare class ReferenceSyncBlockStoreManager {
     private dataProvider?;
-    private syncBlockCache;
     private isCacheDirty;
     private subscriptions;
     private titleSubscriptions;
     private providerFactories;
     private fireAnalyticsEvent?;
     private syncBlockFetchDataRequests;
-    private syncBlockSourceInfoRequestsOld;
     private syncBlockSourceInfoRequests;
     private isRefreshingSubscriptions;
     private pendingCacheDeletions;