@atlaskit/editor-synced-block-provider 25.0.1 → 25.0.3

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # @atlaskit/editor-synced-block-provider
 
+## 25.0.3
+
+### Patch Changes
+
+- [`004b7c3c33232`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/004b7c3c33232) -
+  Fire a per-request `syncedBlockGetSourceInfo` success event, tagged with `sourceLocation`, when a
+  reference synced block resolves its source info, and tag the missing-source-ARI error with
+  `sourceLocation`. Stop adding `sourceLocation` metadata to the shared source-info experience
+  events. Gated by the `platform_editor_blocks_patch_12` experiment (with
+  `editor_synced_blocks_jira_custom_rich_text`); with the experiment off, events are unchanged.
+- Updated dependencies
+
+## 25.0.2
+
+### Patch Changes
+
+- Updated dependencies
+
 ## 25.0.1
 
 ### Patch Changes

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

@@ -435,17 +435,20 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
         // `product` from cached data is preferred when available; fall back to parsing
         // the resourceId.
         var sourceProduct = product !== null && product !== void 0 ? product : (0, _utils.getSourceProductFromResourceIdSafe)(resourceId);
+
+        // With the experiment off, events and experience calls are identical to master.
+        var isPerRequestSourceInfo = (0, _isExperimentEnabled.isExperimentEnabled)('platform_editor_blocks_patch_12');
         if (!sourceAri || !product || !blockInstanceId) {
           var _this$fireAnalyticsEv3;
-          (_this$fireAnalyticsEv3 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv3 === void 0 || _this$fireAnalyticsEv3.call(this, (0, _errorHandling.getSourceInfoErrorPayload)('SourceAri, product or blockInstanceId missing', resourceId, sourceProduct));
+          (_this$fireAnalyticsEv3 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv3 === void 0 || _this$fireAnalyticsEv3.call(this, (0, _errorHandling.getSourceInfoErrorPayload)('SourceAri, product or blockInstanceId missing', resourceId, sourceProduct, isPerRequestSourceInfo ? (0, _fieldAwareLocations.getSourceLocation)(sourceAri) : undefined));
           return Promise.resolve(undefined);
         }
         var sourceLocation = (0, _fieldAwareLocations.getSourceLocation)(sourceAri);
-        // Undefined with `platform_editor_blocks_patch_13` off, so experience calls below keep
-        // exactly their previous arguments (no `metadata` key, `success()` without arguments).
-        var experienceMetadata = sourceLocation && {
+        // The shared experience cannot count requests, so with the experiment on the
+        // per-request event below carries `sourceLocation` instead of the experience.
+        var experienceMetadata = !isPerRequestSourceInfo && sourceLocation ? {
           sourceLocation: sourceLocation
-        };
+        } : undefined;
         var withExperienceMetadata = function withExperienceMetadata(options) {
           return experienceMetadata ? _objectSpread(_objectSpread({}, options), {}, {
             metadata: experienceMetadata
@@ -476,20 +479,25 @@ var ReferenceSyncBlockStoreManager = exports.ReferenceSyncBlockStoreManager = /*
               var _this2$fetchSourceInf3;
               (_this2$fetchSourceInf3 = _this2.fetchSourceInfoExperience) === null || _this2$fetchSourceInf3 === void 0 || _this2$fetchSourceInf3.success();
             }
+            // Pairs with the error event above as the per-request denominator.
+            if (isPerRequestSourceInfo && sourceLocation) {
+              var _this2$fireAnalyticsE2;
+              (_this2$fireAnalyticsE2 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE2 === void 0 || _this2$fireAnalyticsE2.call(_this2, (0, _errorHandling.getSourceInfoSuccessPayload)(resourceId, blockInstanceId, sourceProduct, sourceLocation));
+            }
           } else {
-            var _this2$fetchSourceInf4, _this2$fireAnalyticsE2;
+            var _this2$fetchSourceInf4, _this2$fireAnalyticsE3;
             (_this2$fetchSourceInf4 = _this2.fetchSourceInfoExperience) === null || _this2$fetchSourceInf4 === void 0 || _this2$fetchSourceInf4.failure(withExperienceMetadata({
               reason: 'Missing title or url'
             }));
-            (_this2$fireAnalyticsE2 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE2 === void 0 || _this2$fireAnalyticsE2.call(_this2, (0, _errorHandling.getSourceInfoErrorPayload)('Missing title or url', resourceId, sourceProduct, sourceLocation));
+            (_this2$fireAnalyticsE3 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE3 === void 0 || _this2$fireAnalyticsE3.call(_this2, (0, _errorHandling.getSourceInfoErrorPayload)('Missing title or url', resourceId, sourceProduct, sourceLocation));
           }
           return sourceInfo;
         }).catch(function (error) {
-          var _this2$fetchSourceInf5, _this2$fireAnalyticsE3;
+          var _this2$fetchSourceInf5, _this2$fireAnalyticsE4;
           (_this2$fetchSourceInf5 = _this2.fetchSourceInfoExperience) === null || _this2$fetchSourceInf5 === void 0 || _this2$fetchSourceInf5.failure(withExperienceMetadata({
             reason: error.message
           }));
-          (_this2$fireAnalyticsE3 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE3 === void 0 || _this2$fireAnalyticsE3.call(_this2, (0, _errorHandling.getSourceInfoErrorPayload)(error.message, resourceId, sourceProduct, sourceLocation));
+          (_this2$fireAnalyticsE4 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE4 === void 0 || _this2$fireAnalyticsE4.call(_this2, (0, _errorHandling.getSourceInfoErrorPayload)(error.message, resourceId, sourceProduct, sourceLocation));
           return undefined;
         }).finally(function () {
           _this2.syncBlockSourceInfoRequests.delete(resourceId);

package/dist/cjs/utils/errorHandling.js

@@ -6,7 +6,7 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.fetchSuccessPayload = exports.fetchReferencesErrorPayload = exports.fetchErrorPayload = exports.deleteSuccessPayload = exports.deleteErrorPayload = exports.createSuccessPayload = exports.createSuccessOperationalPayload = exports.createErrorPayload = exports.classifyFetchErrorReason = exports.classifyErrorReason = exports.cacheDeletionForcedPayload = exports.buildFetchErrorAttribution = exports.buildErrorAttribution = exports.addContentSuccessPayload = exports.FETCH_BENIGN_REASONS = void 0;
 exports.getErrorPayload = getErrorPayload;
-exports.updateSuccessPayload = exports.updateReferenceErrorPayload = exports.updateErrorPayload = exports.updateCacheErrorPayload = exports.stringifyError = exports.sourceInfoOrphanedPayload = exports.getSourceInfoErrorPayload = exports.getPiiSafeOriginalError = void 0;
+exports.updateSuccessPayload = exports.updateReferenceErrorPayload = exports.updateErrorPayload = exports.updateCacheErrorPayload = exports.stringifyError = exports.sourceInfoOrphanedPayload = exports.getSourceInfoSuccessPayload = exports.getSourceInfoErrorPayload = exports.getPiiSafeOriginalError = void 0;
 var _slicedToArray2 = _interopRequireDefault(require("@babel/runtime/helpers/slicedToArray"));
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _enums = require("@atlaskit/editor-common/analytics/types/enums");
@@ -333,6 +333,28 @@ var fetchSuccessPayload = exports.fetchSuccessPayload = function fetchSuccessPay
     })
   };
 };
+
+/**
+ * Per-request `syncedBlockGetSourceInfo` success event. Pairs with
+ * `getSourceInfoErrorPayload` so a success rate can be computed per request.
+ */
+var getSourceInfoSuccessPayload = exports.getSourceInfoSuccessPayload = function getSourceInfoSuccessPayload(resourceId, blockInstanceId, sourceProduct, sourceLocation) {
+  return {
+    action: _enums.ACTION.FETCHED,
+    actionSubject: _enums.ACTION_SUBJECT.SYNCED_BLOCK,
+    actionSubjectId: _enums.ACTION_SUBJECT_ID.SYNCED_BLOCK_GET_SOURCE_INFO,
+    eventType: _enums.EVENT_TYPE.OPERATIONAL,
+    attributes: _objectSpread(_objectSpread(_objectSpread({
+      resourceId: resourceId
+    }, blockInstanceId && {
+      blockInstanceId: blockInstanceId
+    }), sourceProduct && {
+      sourceProduct: sourceProduct
+    }), sourceLocation && {
+      sourceLocation: sourceLocation
+    })
+  };
+};
 var createSuccessPayload = exports.createSuccessPayload = function createSuccessPayload(resourceId, sourceProduct, sourceLocation) {
   return {
     action: _enums.ACTION.INSERTED,

package/dist/cjs/utils/fieldAwareLocations.js

@@ -5,7 +5,6 @@ Object.defineProperty(exports, "__esModule", {
 });
 exports.isFieldAwareJiraLocation = exports.getSourceLocation = exports.getFieldAwareLocationScope = void 0;
 var _isExperimentEnabled = require("@atlaskit/platform-feature-experiments/is-experiment-enabled");
-var _fg = require("@atlaskit/platform-feature-flags/fg");
 var _ari = require("../clients/block-service/ari");
 var _ari2 = require("../clients/jira/ari");
 /**
@@ -34,14 +33,8 @@ var getFieldAwareLocationScope = exports.getFieldAwareLocationScope = function g
 /**
  * Analytics `sourceLocation` for a source ARI: `undefined` for non-Jira sources and for the
  * control cohort, so events outside the experiment keep their current shape.
- *
- * The `platform_editor_blocks_patch_13` gate runs first: with it off this returns `undefined`
- * without the product check or the experiment read, so no new exposure is logged.
  */
 var getSourceLocation = exports.getSourceLocation = function getSourceLocation(sourceAri) {
-  if (!(0, _fg.fg)('platform_editor_blocks_patch_13')) {
-    return undefined;
-  }
   if (!sourceAri || !isFieldAwareJiraLocation((0, _ari.getProductFromSourceAri)(sourceAri))) {
     return undefined;
   }

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

@@ -3,7 +3,7 @@ import isEqual from 'lodash/isEqual';
 import { logException } from '@atlaskit/editor-common/monitoring/error';
 import { isExperimentEnabled } from '@atlaskit/platform-feature-experiments/is-experiment-enabled';
 import { isProviderNotReadyError, ProviderNotReadyError, SyncBlockError } from '../common/types';
-import { buildErrorAttribution, buildFetchErrorAttribution, cacheDeletionForcedPayload, fetchErrorPayload, fetchSuccessPayload, getSourceInfoErrorPayload, sourceInfoOrphanedPayload, updateReferenceErrorPayload } from '../utils/errorHandling';
+import { buildErrorAttribution, buildFetchErrorAttribution, cacheDeletionForcedPayload, fetchErrorPayload, fetchSuccessPayload, getSourceInfoErrorPayload, getSourceInfoSuccessPayload, sourceInfoOrphanedPayload, updateReferenceErrorPayload } from '../utils/errorHandling';
 import { getFetchExperience, getFetchSourceInfoExperience, getSaveReferenceExperience } from '../utils/experienceTracking';
 import { getSourceLocation } from '../utils/fieldAwareLocations';
 import { resolveSyncBlockInstance } from '../utils/resolveSyncBlockInstance';
@@ -343,17 +343,20 @@ export class ReferenceSyncBlockStoreManager {
       // `product` from cached data is preferred when available; fall back to parsing
       // the resourceId.
       const sourceProduct = product !== null && product !== void 0 ? product : getSourceProductFromResourceIdSafe(resourceId);
+
+      // With the experiment off, events and experience calls are identical to master.
+      const isPerRequestSourceInfo = isExperimentEnabled('platform_editor_blocks_patch_12');
       if (!sourceAri || !product || !blockInstanceId) {
         var _this$fireAnalyticsEv3;
-        (_this$fireAnalyticsEv3 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv3 === void 0 ? void 0 : _this$fireAnalyticsEv3.call(this, getSourceInfoErrorPayload('SourceAri, product or blockInstanceId missing', resourceId, sourceProduct));
+        (_this$fireAnalyticsEv3 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv3 === void 0 ? void 0 : _this$fireAnalyticsEv3.call(this, getSourceInfoErrorPayload('SourceAri, product or blockInstanceId missing', resourceId, sourceProduct, isPerRequestSourceInfo ? getSourceLocation(sourceAri) : undefined));
         return Promise.resolve(undefined);
       }
       const sourceLocation = getSourceLocation(sourceAri);
-      // Undefined with `platform_editor_blocks_patch_13` off, so experience calls below keep
-      // exactly their previous arguments (no `metadata` key, `success()` without arguments).
-      const experienceMetadata = sourceLocation && {
+      // The shared experience cannot count requests, so with the experiment on the
+      // per-request event below carries `sourceLocation` instead of the experience.
+      const experienceMetadata = !isPerRequestSourceInfo && sourceLocation ? {
         sourceLocation
-      };
+      } : undefined;
       const withExperienceMetadata = options => experienceMetadata ? {
         ...options,
         metadata: experienceMetadata
@@ -383,20 +386,25 @@ export class ReferenceSyncBlockStoreManager {
             var _this$fetchSourceInfo4;
             (_this$fetchSourceInfo4 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo4 === void 0 ? void 0 : _this$fetchSourceInfo4.success();
           }
+          // Pairs with the error event above as the per-request denominator.
+          if (isPerRequestSourceInfo && sourceLocation) {
+            var _this$fireAnalyticsEv5;
+            (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 ? void 0 : _this$fireAnalyticsEv5.call(this, getSourceInfoSuccessPayload(resourceId, blockInstanceId, sourceProduct, sourceLocation));
+          }
         } else {
-          var _this$fetchSourceInfo5, _this$fireAnalyticsEv5;
+          var _this$fetchSourceInfo5, _this$fireAnalyticsEv6;
           (_this$fetchSourceInfo5 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo5 === void 0 ? void 0 : _this$fetchSourceInfo5.failure(withExperienceMetadata({
             reason: 'Missing title or url'
           }));
-          (_this$fireAnalyticsEv5 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv5 === void 0 ? void 0 : _this$fireAnalyticsEv5.call(this, getSourceInfoErrorPayload('Missing title or url', resourceId, sourceProduct, sourceLocation));
+          (_this$fireAnalyticsEv6 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv6 === void 0 ? void 0 : _this$fireAnalyticsEv6.call(this, getSourceInfoErrorPayload('Missing title or url', resourceId, sourceProduct, sourceLocation));
         }
         return sourceInfo;
       }).catch(error => {
-        var _this$fetchSourceInfo6, _this$fireAnalyticsEv6;
+        var _this$fetchSourceInfo6, _this$fireAnalyticsEv7;
         (_this$fetchSourceInfo6 = this.fetchSourceInfoExperience) === null || _this$fetchSourceInfo6 === void 0 ? void 0 : _this$fetchSourceInfo6.failure(withExperienceMetadata({
           reason: error.message
         }));
-        (_this$fireAnalyticsEv6 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv6 === void 0 ? void 0 : _this$fireAnalyticsEv6.call(this, getSourceInfoErrorPayload(error.message, resourceId, sourceProduct, sourceLocation));
+        (_this$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 ? void 0 : _this$fireAnalyticsEv7.call(this, getSourceInfoErrorPayload(error.message, resourceId, sourceProduct, sourceLocation));
         return undefined;
       }).finally(() => {
         this.syncBlockSourceInfoRequests.delete(resourceId);
@@ -404,11 +412,11 @@ export class ReferenceSyncBlockStoreManager {
       this.syncBlockSourceInfoRequests.set(resourceId, sourceInfoPromise);
       return sourceInfoPromise;
     } catch (error) {
-      var _this$fireAnalyticsEv7;
+      var _this$fireAnalyticsEv8;
       logException(error, {
         location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
       });
-      (_this$fireAnalyticsEv7 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv7 === void 0 ? void 0 : _this$fireAnalyticsEv7.call(this, getSourceInfoErrorPayload(error.message, resourceId, getSourceProductFromResourceIdSafe(resourceId)));
+      (_this$fireAnalyticsEv8 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv8 === void 0 ? void 0 : _this$fireAnalyticsEv8.call(this, getSourceInfoErrorPayload(error.message, resourceId, getSourceProductFromResourceIdSafe(resourceId)));
     }
     return Promise.resolve(undefined);
   }
@@ -536,12 +544,12 @@ export class ReferenceSyncBlockStoreManager {
     data.forEach(syncBlockInstance => {
       var _resolvedSyncBlockIns;
       if (!syncBlockInstance.resourceId) {
-        var _syncBlockInstance$er, _syncBlockInstance$er2, _this$fireAnalyticsEv8, _syncBlockInstance$er3, _syncBlockInstance$er4, _syncBlockInstance$er5;
+        var _syncBlockInstance$er, _syncBlockInstance$er2, _this$fireAnalyticsEv9, _syncBlockInstance$er3, _syncBlockInstance$er4, _syncBlockInstance$er5;
         const payload = ((_syncBlockInstance$er = syncBlockInstance.error) === null || _syncBlockInstance$er === void 0 ? void 0 : _syncBlockInstance$er.reason) || ((_syncBlockInstance$er2 = syncBlockInstance.error) === null || _syncBlockInstance$er2 === void 0 ? void 0 : _syncBlockInstance$er2.type) || 'Returned sync block instance does not have resource id';
         // No resourceId means we cannot derive a sourceProduct here; intentionally omit.
         // Classify on the structured `type` first, falling back to the free-text
         // `reason`/payload (EDITOR-7862).
-        (_this$fireAnalyticsEv8 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv8 === void 0 ? void 0 : _this$fireAnalyticsEv8.call(this, fetchErrorPayload(payload, undefined, undefined, buildFetchErrorAttribution(((_syncBlockInstance$er3 = syncBlockInstance.error) === null || _syncBlockInstance$er3 === void 0 ? void 0 : _syncBlockInstance$er3.type) || ((_syncBlockInstance$er4 = syncBlockInstance.error) === null || _syncBlockInstance$er4 === void 0 ? void 0 : _syncBlockInstance$er4.reason) || payload, (_syncBlockInstance$er5 = syncBlockInstance.error) === null || _syncBlockInstance$er5 === void 0 ? void 0 : _syncBlockInstance$er5.statusCode)));
+        (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 ? void 0 : _this$fireAnalyticsEv9.call(this, fetchErrorPayload(payload, undefined, undefined, buildFetchErrorAttribution(((_syncBlockInstance$er3 = syncBlockInstance.error) === null || _syncBlockInstance$er3 === void 0 ? void 0 : _syncBlockInstance$er3.type) || ((_syncBlockInstance$er4 = syncBlockInstance.error) === null || _syncBlockInstance$er4 === void 0 ? void 0 : _syncBlockInstance$er4.reason) || payload, (_syncBlockInstance$er5 = syncBlockInstance.error) === null || _syncBlockInstance$er5 === void 0 ? void 0 : _syncBlockInstance$er5.statusCode)));
         return;
       }
       const existingSyncBlock = this.getFromCache(syncBlockInstance.resourceId);
@@ -574,11 +582,11 @@ export class ReferenceSyncBlockStoreManager {
         // inflating error-rate metrics with expected transient failures
         const isRetryingEntityNotFound = syncBlockInstance.error.type === SyncBlockError.EntityNotFound && ((_this$entityNotFoundR = this.entityNotFoundRetryCount.get(syncBlockInstance.resourceId)) !== null && _this$entityNotFoundR !== void 0 ? _this$entityNotFoundR : 0) < ENTITY_NOT_FOUND_MAX_RETRIES;
         if (!isRetryingEntityNotFound) {
-          var _this$fireAnalyticsEv9, _syncBlockInstance$da, _syncBlockInstance$da2;
+          var _this$fireAnalyticsEv0, _syncBlockInstance$da, _syncBlockInstance$da2;
           // Classify on the structured `type` (a `SyncBlockError` enum value) first,
           // falling back to the free-text `reason` so source-state/permission strings
           // are still bucketed (EDITOR-7862). The emitted `error` attribute is unchanged.
-          (_this$fireAnalyticsEv9 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv9 === void 0 ? void 0 : _this$fireAnalyticsEv9.call(this, fetchErrorPayload(syncBlockInstance.error.reason || syncBlockInstance.error.type, syncBlockInstance.resourceId, (_syncBlockInstance$da = (_syncBlockInstance$da2 = syncBlockInstance.data) === null || _syncBlockInstance$da2 === void 0 ? void 0 : _syncBlockInstance$da2.product) !== null && _syncBlockInstance$da !== void 0 ? _syncBlockInstance$da : getSourceProductFromResourceIdSafe(syncBlockInstance.resourceId), buildFetchErrorAttribution(syncBlockInstance.error.type || syncBlockInstance.error.reason, syncBlockInstance.error.statusCode)));
+          (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 ? void 0 : _this$fireAnalyticsEv0.call(this, fetchErrorPayload(syncBlockInstance.error.reason || syncBlockInstance.error.type, syncBlockInstance.resourceId, (_syncBlockInstance$da = (_syncBlockInstance$da2 = syncBlockInstance.data) === null || _syncBlockInstance$da2 === void 0 ? void 0 : _syncBlockInstance$da2.product) !== null && _syncBlockInstance$da !== void 0 ? _syncBlockInstance$da : getSourceProductFromResourceIdSafe(syncBlockInstance.resourceId), buildFetchErrorAttribution(syncBlockInstance.error.type || syncBlockInstance.error.reason, syncBlockInstance.error.statusCode)));
         }
         if (syncBlockInstance.error.type === SyncBlockError.NotFound || syncBlockInstance.error.type === SyncBlockError.Forbidden) {
           hasExpectedError = true;
@@ -598,8 +606,8 @@ export class ReferenceSyncBlockStoreManager {
       const callbacks = this._subscriptionManager.getSubscriptions().get(syncBlockInstance.resourceId);
       const localIds = callbacks ? Object.keys(callbacks) : [];
       localIds.forEach(localId => {
-        var _this$fireAnalyticsEv0, _syncBlockInstance$da3, _syncBlockInstance$da4;
-        (_this$fireAnalyticsEv0 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv0 === void 0 ? void 0 : _this$fireAnalyticsEv0.call(this, fetchSuccessPayload(syncBlockInstance.resourceId, localId, (_syncBlockInstance$da3 = (_syncBlockInstance$da4 = syncBlockInstance.data) === null || _syncBlockInstance$da4 === void 0 ? void 0 : _syncBlockInstance$da4.product) !== null && _syncBlockInstance$da3 !== void 0 ? _syncBlockInstance$da3 : getSourceProductFromResourceIdSafe(syncBlockInstance.resourceId)));
+        var _this$fireAnalyticsEv1, _syncBlockInstance$da3, _syncBlockInstance$da4;
+        (_this$fireAnalyticsEv1 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv1 === void 0 ? void 0 : _this$fireAnalyticsEv1.call(this, fetchSuccessPayload(syncBlockInstance.resourceId, localId, (_syncBlockInstance$da3 = (_syncBlockInstance$da4 = syncBlockInstance.data) === null || _syncBlockInstance$da4 === void 0 ? void 0 : _syncBlockInstance$da4.product) !== null && _syncBlockInstance$da3 !== void 0 ? _syncBlockInstance$da3 : getSourceProductFromResourceIdSafe(syncBlockInstance.resourceId)));
       });
       this.fetchSyncBlockSourceInfo(resolvedSyncBlockInstance.resourceId);
     });
@@ -613,8 +621,8 @@ export class ReferenceSyncBlockStoreManager {
     // If the cache entry was deleted while the source-info request was
     // in flight, fire an analytics event so the race is observable.
     if (!existingSyncBlock) {
-      var _this$fireAnalyticsEv1;
-      (_this$fireAnalyticsEv1 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv1 === void 0 ? void 0 : _this$fireAnalyticsEv1.call(this, sourceInfoOrphanedPayload(resourceId, getSourceProductFromResourceIdSafe(resourceId), {
+      var _this$fireAnalyticsEv10;
+      (_this$fireAnalyticsEv10 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv10 === void 0 ? void 0 : _this$fireAnalyticsEv10.call(this, sourceInfoOrphanedPayload(resourceId, getSourceProductFromResourceIdSafe(resourceId), {
         hasPendingDeletion: this.pendingCacheDeletions.has(resourceId),
         hasSubscribers: this._subscriptionManager.getSubscriptions().has(resourceId)
       }));
@@ -753,7 +761,7 @@ export class ReferenceSyncBlockStoreManager {
     }
     const currentCount = (_this$cacheDeletionRe = this.cacheDeletionRescheduleCounts.get(resourceId)) !== null && _this$cacheDeletionRe !== void 0 ? _this$cacheDeletionRe : 0;
     if (currentCount >= CACHE_DELETION_MAX_RESCHEDULES) {
-      var _this$fireAnalyticsEv10;
+      var _this$fireAnalyticsEv11;
       // Stuck guard — force deletion to prevent unbounded memory growth and
       // fire analytics so the stuck state is visible in production telemetry.
       //
@@ -763,7 +771,7 @@ export class ReferenceSyncBlockStoreManager {
       // continue to render with stale data until their next re-render
       // triggers a new batch fetch — typically within ~1 frame. We accept
       // this brief stale window in exchange for bounded memory growth.
-      (_this$fireAnalyticsEv10 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv10 === void 0 ? void 0 : _this$fireAnalyticsEv10.call(this, cacheDeletionForcedPayload(currentCount, resourceId, getSourceProductFromResourceIdSafe(resourceId)));
+      (_this$fireAnalyticsEv11 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv11 === void 0 ? void 0 : _this$fireAnalyticsEv11.call(this, cacheDeletionForcedPayload(currentCount, resourceId, getSourceProductFromResourceIdSafe(resourceId)));
       logException(new Error(`Cache deletion forced after ${currentCount} reschedules — stuck in-flight guard`), {
         location: 'editor-synced-block-provider/referenceSyncBlockStoreManager/cache-deletion-forced'
       });
@@ -891,7 +899,7 @@ export class ReferenceSyncBlockStoreManager {
       }
       return this.subscribeToSyncBlock(resourceId, localId, callback);
     } catch (error) {
-      var _this$fireAnalyticsEv11;
+      var _this$fireAnalyticsEv12;
       // EDITOR-7860: benign not-ready/torn-down case — suppress both the
       // exception-tracker log and the analytics event (checked first so the
       // benign case stays fully silent).
@@ -901,7 +909,7 @@ export class ReferenceSyncBlockStoreManager {
       logException(error, {
         location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
       });
-      (_this$fireAnalyticsEv11 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv11 === void 0 ? void 0 : _this$fireAnalyticsEv11.call(this, fetchErrorPayload(error.message, undefined, undefined, buildFetchErrorAttribution(error.message)));
+      (_this$fireAnalyticsEv12 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv12 === void 0 ? void 0 : _this$fireAnalyticsEv12.call(this, fetchErrorPayload(error.message, undefined, undefined, buildFetchErrorAttribution(error.message)));
       return () => {};
     }
   }
@@ -1005,15 +1013,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$fireAnalyticsEv12;
+        var _this$saveExperience2, _this$fireAnalyticsEv13;
         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$fireAnalyticsEv12 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv12 === void 0 ? void 0 : _this$fireAnalyticsEv12.call(this, updateReferenceErrorPayload(updateResult.error || 'Failed to update reference synced blocks on the document', undefined, undefined, buildErrorAttribution(updateResult.error, updateResult.statusCode)));
+        (_this$fireAnalyticsEv13 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv13 === void 0 ? void 0 : _this$fireAnalyticsEv13.call(this, updateReferenceErrorPayload(updateResult.error || 'Failed to update reference synced blocks on the document', undefined, undefined, buildErrorAttribution(updateResult.error, updateResult.statusCode)));
       }
     } catch (error) {
-      var _this$saveExperience3, _this$fireAnalyticsEv13;
+      var _this$saveExperience3, _this$fireAnalyticsEv14;
       success = false;
       logException(error, {
         location: 'editor-synced-block-provider/referenceSyncBlockStoreManager'
@@ -1024,7 +1032,7 @@ export class ReferenceSyncBlockStoreManager {
       // No `resourceId` available in this catch — sourceProduct is intentionally omitted.
       // No structured SyncBlockError/status here, so the attribution `reason` falls back
       // to `unknown`.
-      (_this$fireAnalyticsEv13 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv13 === void 0 ? void 0 : _this$fireAnalyticsEv13.call(this, updateReferenceErrorPayload(error.message, undefined, undefined, buildErrorAttribution()));
+      (_this$fireAnalyticsEv14 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv14 === void 0 ? void 0 : _this$fireAnalyticsEv14.call(this, updateReferenceErrorPayload(error.message, undefined, undefined, buildErrorAttribution()));
     } finally {
       if (!success) {
         // set isCacheDirty back to true for cases where it failed to update the reference synced blocks on the BE

package/dist/es2019/utils/errorHandling.js

@@ -319,6 +319,29 @@ export const fetchSuccessPayload = (resourceId, blockInstanceId, sourceProduct)
     })
   }
 });
+
+/**
+ * Per-request `syncedBlockGetSourceInfo` success event. Pairs with
+ * `getSourceInfoErrorPayload` so a success rate can be computed per request.
+ */
+export const getSourceInfoSuccessPayload = (resourceId, blockInstanceId, sourceProduct, sourceLocation) => ({
+  action: ACTION.FETCHED,
+  actionSubject: ACTION_SUBJECT.SYNCED_BLOCK,
+  actionSubjectId: ACTION_SUBJECT_ID.SYNCED_BLOCK_GET_SOURCE_INFO,
+  eventType: EVENT_TYPE.OPERATIONAL,
+  attributes: {
+    resourceId,
+    ...(blockInstanceId && {
+      blockInstanceId
+    }),
+    ...(sourceProduct && {
+      sourceProduct
+    }),
+    ...(sourceLocation && {
+      sourceLocation
+    })
+  }
+});
 export const createSuccessPayload = (resourceId, sourceProduct, sourceLocation) => {
   return {
     action: ACTION.INSERTED,

package/dist/es2019/utils/fieldAwareLocations.js

@@ -1,5 +1,4 @@
 import { isExperimentEnabled } from '@atlaskit/platform-feature-experiments/is-experiment-enabled';
-import { fg } from '@atlaskit/platform-feature-flags/fg';
 import { getLocationScope, getProductFromSourceAri } from '../clients/block-service/ari';
 import { JIRA_DESCRIPTION_FIELD_ID, parseJiraFieldLocation } from '../clients/jira/ari';
 /**
@@ -25,14 +24,8 @@ export const getFieldAwareLocationScope = ({
 /**
  * Analytics `sourceLocation` for a source ARI: `undefined` for non-Jira sources and for the
  * control cohort, so events outside the experiment keep their current shape.
- *
- * The `platform_editor_blocks_patch_13` gate runs first: with it off this returns `undefined`
- * without the product check or the experiment read, so no new exposure is logged.
  */
 export const getSourceLocation = sourceAri => {
-  if (!fg('platform_editor_blocks_patch_13')) {
-    return undefined;
-  }
   if (!sourceAri || !isFieldAwareJiraLocation(getProductFromSourceAri(sourceAri))) {
     return undefined;
   }

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

@@ -14,7 +14,7 @@ import isEqual from 'lodash/isEqual';
 import { logException } from '@atlaskit/editor-common/monitoring/error';
 import { isExperimentEnabled } from '@atlaskit/platform-feature-experiments/is-experiment-enabled';
 import { isProviderNotReadyError, ProviderNotReadyError, SyncBlockError } from '../common/types';
-import { buildErrorAttribution, buildFetchErrorAttribution, cacheDeletionForcedPayload, fetchErrorPayload, fetchSuccessPayload, getSourceInfoErrorPayload, sourceInfoOrphanedPayload, updateReferenceErrorPayload } from '../utils/errorHandling';
+import { buildErrorAttribution, buildFetchErrorAttribution, cacheDeletionForcedPayload, fetchErrorPayload, fetchSuccessPayload, getSourceInfoErrorPayload, getSourceInfoSuccessPayload, sourceInfoOrphanedPayload, updateReferenceErrorPayload } from '../utils/errorHandling';
 import { getFetchExperience, getFetchSourceInfoExperience, getSaveReferenceExperience } from '../utils/experienceTracking';
 import { getSourceLocation } from '../utils/fieldAwareLocations';
 import { resolveSyncBlockInstance } from '../utils/resolveSyncBlockInstance';
@@ -428,17 +428,20 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
         // `product` from cached data is preferred when available; fall back to parsing
         // the resourceId.
         var sourceProduct = product !== null && product !== void 0 ? product : getSourceProductFromResourceIdSafe(resourceId);
+
+        // With the experiment off, events and experience calls are identical to master.
+        var isPerRequestSourceInfo = isExperimentEnabled('platform_editor_blocks_patch_12');
         if (!sourceAri || !product || !blockInstanceId) {
           var _this$fireAnalyticsEv3;
-          (_this$fireAnalyticsEv3 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv3 === void 0 || _this$fireAnalyticsEv3.call(this, getSourceInfoErrorPayload('SourceAri, product or blockInstanceId missing', resourceId, sourceProduct));
+          (_this$fireAnalyticsEv3 = this.fireAnalyticsEvent) === null || _this$fireAnalyticsEv3 === void 0 || _this$fireAnalyticsEv3.call(this, getSourceInfoErrorPayload('SourceAri, product or blockInstanceId missing', resourceId, sourceProduct, isPerRequestSourceInfo ? getSourceLocation(sourceAri) : undefined));
           return Promise.resolve(undefined);
         }
         var sourceLocation = getSourceLocation(sourceAri);
-        // Undefined with `platform_editor_blocks_patch_13` off, so experience calls below keep
-        // exactly their previous arguments (no `metadata` key, `success()` without arguments).
-        var experienceMetadata = sourceLocation && {
+        // The shared experience cannot count requests, so with the experiment on the
+        // per-request event below carries `sourceLocation` instead of the experience.
+        var experienceMetadata = !isPerRequestSourceInfo && sourceLocation ? {
           sourceLocation: sourceLocation
-        };
+        } : undefined;
         var withExperienceMetadata = function withExperienceMetadata(options) {
           return experienceMetadata ? _objectSpread(_objectSpread({}, options), {}, {
             metadata: experienceMetadata
@@ -469,20 +472,25 @@ export var ReferenceSyncBlockStoreManager = /*#__PURE__*/function () {
               var _this2$fetchSourceInf3;
               (_this2$fetchSourceInf3 = _this2.fetchSourceInfoExperience) === null || _this2$fetchSourceInf3 === void 0 || _this2$fetchSourceInf3.success();
             }
+            // Pairs with the error event above as the per-request denominator.
+            if (isPerRequestSourceInfo && sourceLocation) {
+              var _this2$fireAnalyticsE2;
+              (_this2$fireAnalyticsE2 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE2 === void 0 || _this2$fireAnalyticsE2.call(_this2, getSourceInfoSuccessPayload(resourceId, blockInstanceId, sourceProduct, sourceLocation));
+            }
           } else {
-            var _this2$fetchSourceInf4, _this2$fireAnalyticsE2;
+            var _this2$fetchSourceInf4, _this2$fireAnalyticsE3;
             (_this2$fetchSourceInf4 = _this2.fetchSourceInfoExperience) === null || _this2$fetchSourceInf4 === void 0 || _this2$fetchSourceInf4.failure(withExperienceMetadata({
               reason: 'Missing title or url'
             }));
-            (_this2$fireAnalyticsE2 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE2 === void 0 || _this2$fireAnalyticsE2.call(_this2, getSourceInfoErrorPayload('Missing title or url', resourceId, sourceProduct, sourceLocation));
+            (_this2$fireAnalyticsE3 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE3 === void 0 || _this2$fireAnalyticsE3.call(_this2, getSourceInfoErrorPayload('Missing title or url', resourceId, sourceProduct, sourceLocation));
           }
           return sourceInfo;
         }).catch(function (error) {
-          var _this2$fetchSourceInf5, _this2$fireAnalyticsE3;
+          var _this2$fetchSourceInf5, _this2$fireAnalyticsE4;
           (_this2$fetchSourceInf5 = _this2.fetchSourceInfoExperience) === null || _this2$fetchSourceInf5 === void 0 || _this2$fetchSourceInf5.failure(withExperienceMetadata({
             reason: error.message
           }));
-          (_this2$fireAnalyticsE3 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE3 === void 0 || _this2$fireAnalyticsE3.call(_this2, getSourceInfoErrorPayload(error.message, resourceId, sourceProduct, sourceLocation));
+          (_this2$fireAnalyticsE4 = _this2.fireAnalyticsEvent) === null || _this2$fireAnalyticsE4 === void 0 || _this2$fireAnalyticsE4.call(_this2, getSourceInfoErrorPayload(error.message, resourceId, sourceProduct, sourceLocation));
           return undefined;
         }).finally(function () {
           _this2.syncBlockSourceInfoRequests.delete(resourceId);

package/dist/esm/utils/errorHandling.js

@@ -324,6 +324,28 @@ export var fetchSuccessPayload = function fetchSuccessPayload(resourceId, blockI
     })
   };
 };
+
+/**
+ * Per-request `syncedBlockGetSourceInfo` success event. Pairs with
+ * `getSourceInfoErrorPayload` so a success rate can be computed per request.
+ */
+export var getSourceInfoSuccessPayload = function getSourceInfoSuccessPayload(resourceId, blockInstanceId, sourceProduct, sourceLocation) {
+  return {
+    action: ACTION.FETCHED,
+    actionSubject: ACTION_SUBJECT.SYNCED_BLOCK,
+    actionSubjectId: ACTION_SUBJECT_ID.SYNCED_BLOCK_GET_SOURCE_INFO,
+    eventType: EVENT_TYPE.OPERATIONAL,
+    attributes: _objectSpread(_objectSpread(_objectSpread({
+      resourceId: resourceId
+    }, blockInstanceId && {
+      blockInstanceId: blockInstanceId
+    }), sourceProduct && {
+      sourceProduct: sourceProduct
+    }), sourceLocation && {
+      sourceLocation: sourceLocation
+    })
+  };
+};
 export var createSuccessPayload = function createSuccessPayload(resourceId, sourceProduct, sourceLocation) {
   return {
     action: ACTION.INSERTED,

package/dist/esm/utils/fieldAwareLocations.js

@@ -1,5 +1,4 @@
 import { isExperimentEnabled } from '@atlaskit/platform-feature-experiments/is-experiment-enabled';
-import { fg } from '@atlaskit/platform-feature-flags/fg';
 import { getLocationScope, getProductFromSourceAri } from '../clients/block-service/ari';
 import { JIRA_DESCRIPTION_FIELD_ID, parseJiraFieldLocation } from '../clients/jira/ari';
 /**
@@ -28,14 +27,8 @@ export var getFieldAwareLocationScope = function getFieldAwareLocationScope(_ref
 /**
  * Analytics `sourceLocation` for a source ARI: `undefined` for non-Jira sources and for the
  * control cohort, so events outside the experiment keep their current shape.
- *
- * The `platform_editor_blocks_patch_13` gate runs first: with it off this returns `undefined`
- * without the product check or the experiment read, so no new exposure is logged.
  */
 export var getSourceLocation = function getSourceLocation(sourceAri) {
-  if (!fg('platform_editor_blocks_patch_13')) {
-    return undefined;
-  }
   if (!sourceAri || !isFieldAwareJiraLocation(getProductFromSourceAri(sourceAri))) {
     return undefined;
   }

package/dist/types/utils/errorHandling.d.ts

@@ -175,6 +175,11 @@ export declare const sourceInfoOrphanedPayload: (resourceId?: string, sourceProd
 export declare const cacheDeletionForcedPayload: (rescheduleCount: number, resourceId?: string, sourceProduct?: string) => RendererSyncBlockEventPayload;
 export declare const fetchReferencesErrorPayload: (error: string, resourceId?: string, sourceProduct?: string) => SyncBlockEventPayload;
 export declare const fetchSuccessPayload: (resourceId: string, blockInstanceId?: string, sourceProduct?: string) => RendererSyncBlockEventPayload;
+/**
+ * Per-request `syncedBlockGetSourceInfo` success event. Pairs with
+ * `getSourceInfoErrorPayload` so a success rate can be computed per request.
+ */
+export declare const getSourceInfoSuccessPayload: (resourceId: string, blockInstanceId?: string, sourceProduct?: string, sourceLocation?: SyncBlockSourceLocation) => RendererSyncBlockEventPayload;
 export declare const createSuccessPayload: (resourceId: string, sourceProduct?: string, sourceLocation?: SyncBlockSourceLocation) => SyncBlockEventPayload;
 /**
  * Optional enrichment for the `syncedBlockCreate` success event. All fields

package/dist/types/utils/fieldAwareLocations.d.ts

@@ -17,8 +17,5 @@ export declare const getFieldAwareLocationScope: ({ documentAri, hostAri, produc
 /**
  * Analytics `sourceLocation` for a source ARI: `undefined` for non-Jira sources and for the
  * control cohort, so events outside the experiment keep their current shape.
- *
- * The `platform_editor_blocks_patch_13` gate runs first: with it off this returns `undefined`
- * without the product check or the experiment read, so no new exposure is logged.
  */
 export declare const getSourceLocation: (sourceAri: string | undefined) => SyncBlockSourceLocation | undefined;

package/package.json

@@ -26,7 +26,7 @@
 		"@atlaskit/platform-feature-experiments": "^0.3.0",
 		"@atlaskit/platform-feature-flags": "^2.2.0",
 		"@atlaskit/task-decision": "^21.12.0",
-		"@atlaskit/tmp-editor-statsig": "^243.0.0",
+		"@atlaskit/tmp-editor-statsig": "^244.0.0",
 		"@babel/runtime": "^7.0.0",
 		"@compiled/react": "^1.0.2",
 		"bind-event-listener": "^3.0.0",
@@ -36,7 +36,7 @@
 		"uuid": "^11.1.1"
 	},
 	"peerDependencies": {
-		"@atlaskit/editor-common": "^133.1.0",
+		"@atlaskit/editor-common": "^133.2.0",
 		"react": "^18.2.0 || ^19.2.0"
 	},
 	"devDependencies": {
@@ -45,16 +45,11 @@
 		"react-dom": "^19.2.0"
 	},
 	"name": "@atlaskit/editor-synced-block-provider",
-	"version": "25.0.1",
+	"version": "25.0.3",
 	"description": "Synced Block Provider for @atlaskit/editor-plugin-synced-block",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
 	"publishConfig": {
 		"registry": "https://registry.npmjs.org/"
-	},
-	"platform-feature-flags": {
-		"platform_editor_blocks_patch_13": {
-			"type": "boolean"
-		}
 	}
 }