@atlaskit/editor-plugin-show-diff 13.0.7 → 13.0.9

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @atlaskit/editor-plugin-show-diff
 
+## 13.0.9
+
+### Patch Changes
+
+- [`0123de9acf8b1`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/0123de9acf8b1) -
+  [CCI-18588] Fix multi-second browser freeze when opening the AI Review moment on a large generated
+  table. For big inserted tables the diff previously emitted one decoration per cell (O(cells)) plus
+  redundant inner-block decorations, forcing a full synchronous table relayout. It now skips the
+  per-cell inline and inner-block (table/row/paragraph) decorations while keeping the cell overlay
+  highlight. Gated behind `platform_editor_ai_new_aifc_editor_experience`; behaviour is unchanged
+  for small tables and when the experiment is off.
+
+## 13.0.8
+
+### Patch Changes
+
+- [`d4fbe7dee465e`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/d4fbe7dee465e) -
+  Consolidate diff decoration scrolling into a single internal helper.
+- Updated dependencies
+
 ## 13.0.7
 
 ### Patch Changes

package/dist/cjs/pm-plugins/calculateDiff/calculateDiffDecorations.js

@@ -95,6 +95,26 @@ var leafTextblockRanges = function leafTextblockRanges(doc, from, to) {
   });
   return ranges;
 };
+
+// A large inserted table is treated with a coarse decoration path (see below)
+// once it has more leaf textblocks (~cells) than this threshold. Small tables
+// keep the precise per-cell path.
+var LARGE_TABLE_LEAF_THRESHOLD = 40;
+var isLargeInsertedTableRange = function isLargeInsertedTableRange(doc, from, to) {
+  var containsTable = false;
+  var leafCount = 0;
+  doc.nodesBetween(from, to, function (node) {
+    if (node.type.name === 'table') {
+      containsTable = true;
+    }
+    if (node.isTextblock) {
+      leafCount += 1;
+      return false;
+    }
+    return true;
+  });
+  return containsTable && leafCount > LARGE_TABLE_LEAF_THRESHOLD;
+};
 var calculateNodesForBlockDecoration = function calculateNodesForBlockDecoration(_ref2) {
   var doc = _ref2.doc,
     from = _ref2.from,
@@ -107,10 +127,20 @@ var calculateNodesForBlockDecoration = function calculateNodesForBlockDecoration
     shouldHideDeleted = _ref2$shouldHideDelet === void 0 ? false : _ref2$shouldHideDelet,
     _ref2$showIndicators = _ref2.showIndicators,
     showIndicators = _ref2$showIndicators === void 0 ? false : _ref2$showIndicators,
-    diffType = _ref2.diffType;
+    diffType = _ref2.diffType,
+    _ref2$coarseTableCell = _ref2.coarseTableCellsOnly,
+    coarseTableCellsOnly = _ref2$coarseTableCell === void 0 ? false : _ref2$coarseTableCell;
   var decorations = [];
   // Iterate over the document nodes within the range
   doc.nodesBetween(from, to, function (node, pos) {
+    // Coarse path: only cell/header overlays are visible; table/row/paragraph
+    // decorations are redundant and cause expensive per-cell re-render.
+    if (coarseTableCellsOnly) {
+      var name = node.type.name;
+      if (name !== 'tableCell' && name !== 'tableHeader') {
+        return;
+      }
+    }
     if (node.isBlock && (!(0, _isExtendedEnabled.isExtendedEnabled)(diffType) || pos + node.nodeSize <= to)) {
       var nodeEnd = pos + node.nodeSize;
       var isActive = activeIndexPos && pos === activeIndexPos.from && nodeEnd === activeIndexPos.to;
@@ -290,12 +320,18 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref4
       // emit ONE inline decoration per leaf text-bearing block within the range — the text
       // gets highlighted, and structural gaps never do.
       var isSmartNodeLevel = diffType === 'smart' && (0, _platformFeatureFlags.fg)('platform_editor_ai_smart_diff') && (0, _helpers.smartChangeLevel)(change) === 'node';
+      // For a large inserted table, skip the per-cell inline decorations that
+      // freeze the browser on re-render. Only active in the AIFC experience.
+      var useCoarseTableDecoration = isSmartNodeLevel && (0, _expValEquals.expValEquals)('platform_editor_ai_new_aifc_editor_experience', 'isEnabled', true) && isLargeInsertedTableRange(tr.doc, change.fromB, change.toB);
       // Whether the deleted-content widget will actually be rendered for this
       // change. Used to decide if indicator anchor positions should be adjusted
       // inward — when the widget is present the anchor must stay at the block
       // boundary to keep the indicator bar continuous with the deleted content.
       var willRenderDeletedWidget = change.deleted.length > 0 && !((0, _isExtendedEnabled.isExtendedEnabled)(diffType) && !isInverted && hideDeletedDiffs && change.inserted.length > 0);
-      if (isSmartNodeLevel) {
+      if (useCoarseTableDecoration) {
+        // Skip the O(cells) inline decorations; the cell overlays (added below)
+        // provide the highlight without the expensive content re-render.
+      } else if (isSmartNodeLevel) {
         var _iterator2 = _createForOfIteratorHelper(leafTextblockRanges(tr.doc, change.fromB, change.toB)),
           _step2;
         try {
@@ -354,7 +390,8 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref4
       }), {}, {
         activeIndexPos: activeIndexPos,
         intl: intl,
-        diffType: diffType
+        diffType: diffType,
+        coarseTableCellsOnly: useCoarseTableDecoration
       }))));
     }
     if (change.deleted.length > 0) {

package/dist/cjs/pm-plugins/main.js

@@ -187,7 +187,7 @@ var createPlugin = exports.createPlugin = function createPlugin(config, getIntl,
           if (pluginState !== null && pluginState !== void 0 && pluginState.scrollIntoView && (0, _isExtendedEnabled.isExtendedEnabled)(pluginState === null || pluginState === void 0 ? void 0 : pluginState.diffType)) {
             var _cancelPendingScrollT;
             (_cancelPendingScrollT = cancelPendingScrollToDecoration) === null || _cancelPendingScrollT === void 0 || _cancelPendingScrollT();
-            cancelPendingScrollToDecoration = (0, _scrollToDiff.scrollToFirstDecoration)(view, (0, _getScrollableDecorations.getScrollableDecorations)(pluginState.decorations, view.state.doc, pluginState === null || pluginState === void 0 ? void 0 : pluginState.diffType));
+            cancelPendingScrollToDecoration = (0, _scrollToDiff.scrollToDecoration)(view, (0, _getScrollableDecorations.getScrollableDecorations)(pluginState.decorations, view.state.doc, pluginState === null || pluginState === void 0 ? void 0 : pluginState.diffType));
 
             // Reset the flag so we don't scroll again on subsequent updates
             view.dispatch(view.state.tr.setMeta(showDiffPluginKey, {
@@ -208,7 +208,7 @@ var createPlugin = exports.createPlugin = function createPlugin(config, getIntl,
               // avoid a circular dependency; expand picks up the meta if loaded (no-op if not).
               api === null || api === void 0 || api.core.actions.execute((0, _expand.toggleExpandRange)(activeDecoration.from, activeDecoration.to, true));
             }
-            cancelPendingScrollToDecoration = (0, _scrollToDiff.scrollToActiveDecoration)(view, scrollableDecorations, pluginState.activeIndex);
+            cancelPendingScrollToDecoration = (0, _scrollToDiff.scrollToDecoration)(view, scrollableDecorations, pluginState.activeIndex);
           }
         },
         destroy: function destroy() {

package/dist/cjs/pm-plugins/scrollToDiff.js

@@ -3,7 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
-exports.scrollToFirstDecoration = exports.scrollToActiveDecoration = void 0;
+exports.scrollToDecoration = void 0;
 var _decorationKeys = require("./decorations/decorationKeys");
 /**
  * Extra space above the scrolled-to element so it does not sit flush under the
@@ -40,29 +40,13 @@ function scrollToSelection(node) {
 }
 
 /**
- * Schedules scrolling to the first diff decoration after the next frame.
- * Unlike `scrollToActiveDecoration`, this does not require an active index —
- * it simply scrolls to bring the first decoration into view.
- *
- * The caller must pass the list of scrollable decorations as produced by
- * `getScrollableDecorations`, which is filtered (non-scrollable decorations
- * removed) and sorted by document position. This guarantees "first" means the
- * topmost scrollable diff in the document rather than an arbitrary decoration
- * from the raw `DecorationSet` ordering. This is delegated to
- * `scrollToActiveDecoration` at index 0 so both scroll paths behave identically.
- *
- * @returns A function that cancels the scheduled `requestAnimationFrame` if it has not run yet.
- */
-var scrollToFirstDecoration = exports.scrollToFirstDecoration = function scrollToFirstDecoration(view, scrollableDecorations) {
-  return scrollToActiveDecoration(view, scrollableDecorations, 0);
-};
-
-/**
- * Schedules scrolling to the decoration at the given index after the next frame.
+ * Schedules scrolling to the decoration at the given index after the next frame. Defaults to the
+ * first decoration when no index is provided.
  *
  * @returns A function that cancels the scheduled `requestAnimationFrame` if it has not run yet.
  */
-var scrollToActiveDecoration = exports.scrollToActiveDecoration = function scrollToActiveDecoration(view, decorations, activeIndex) {
+var scrollToDecoration = exports.scrollToDecoration = function scrollToDecoration(view, decorations) {
+  var activeIndex = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : 0;
   var decoration = decorations[activeIndex];
   if (!decoration) {
     return function () {};

package/dist/es2019/pm-plugins/calculateDiff/calculateDiffDecorations.js

@@ -81,6 +81,26 @@ const leafTextblockRanges = (doc, from, to) => {
   });
   return ranges;
 };
+
+// A large inserted table is treated with a coarse decoration path (see below)
+// once it has more leaf textblocks (~cells) than this threshold. Small tables
+// keep the precise per-cell path.
+const LARGE_TABLE_LEAF_THRESHOLD = 40;
+const isLargeInsertedTableRange = (doc, from, to) => {
+  let containsTable = false;
+  let leafCount = 0;
+  doc.nodesBetween(from, to, node => {
+    if (node.type.name === 'table') {
+      containsTable = true;
+    }
+    if (node.isTextblock) {
+      leafCount += 1;
+      return false;
+    }
+    return true;
+  });
+  return containsTable && leafCount > LARGE_TABLE_LEAF_THRESHOLD;
+};
 const calculateNodesForBlockDecoration = ({
   doc,
   from,
@@ -90,11 +110,20 @@ const calculateNodesForBlockDecoration = ({
   activeIndexPos,
   shouldHideDeleted = false,
   showIndicators = false,
-  diffType
+  diffType,
+  coarseTableCellsOnly = false
 }) => {
   const decorations = [];
   // Iterate over the document nodes within the range
   doc.nodesBetween(from, to, (node, pos) => {
+    // Coarse path: only cell/header overlays are visible; table/row/paragraph
+    // decorations are redundant and cause expensive per-cell re-render.
+    if (coarseTableCellsOnly) {
+      const name = node.type.name;
+      if (name !== 'tableCell' && name !== 'tableHeader') {
+        return;
+      }
+    }
     if (node.isBlock && (!isExtendedEnabled(diffType) || pos + node.nodeSize <= to)) {
       const nodeEnd = pos + node.nodeSize;
       const isActive = activeIndexPos && pos === activeIndexPos.from && nodeEnd === activeIndexPos.to;
@@ -264,12 +293,18 @@ const calculateDiffDecorationsInner = ({
       // emit ONE inline decoration per leaf text-bearing block within the range — the text
       // gets highlighted, and structural gaps never do.
       const isSmartNodeLevel = diffType === 'smart' && fg('platform_editor_ai_smart_diff') && smartChangeLevel(change) === 'node';
+      // For a large inserted table, skip the per-cell inline decorations that
+      // freeze the browser on re-render. Only active in the AIFC experience.
+      const useCoarseTableDecoration = isSmartNodeLevel && expValEquals('platform_editor_ai_new_aifc_editor_experience', 'isEnabled', true) && isLargeInsertedTableRange(tr.doc, change.fromB, change.toB);
       // Whether the deleted-content widget will actually be rendered for this
       // change. Used to decide if indicator anchor positions should be adjusted
       // inward — when the widget is present the anchor must stay at the block
       // boundary to keep the indicator bar continuous with the deleted content.
       const willRenderDeletedWidget = change.deleted.length > 0 && !(isExtendedEnabled(diffType) && !isInverted && hideDeletedDiffs && change.inserted.length > 0);
-      if (isSmartNodeLevel) {
+      if (useCoarseTableDecoration) {
+        // Skip the O(cells) inline decorations; the cell overlays (added below)
+        // provide the highlight without the expensive content re-render.
+      } else if (isSmartNodeLevel) {
         for (const [from, to] of leafTextblockRanges(tr.doc, change.fromB, change.toB)) {
           decorations.push(...createInlineChangedDecoration({
             change: {
@@ -319,7 +354,8 @@ const calculateDiffDecorationsInner = ({
         }),
         activeIndexPos,
         intl,
-        diffType
+        diffType,
+        coarseTableCellsOnly: useCoarseTableDecoration
       }));
     }
     if (change.deleted.length > 0) {

package/dist/es2019/pm-plugins/main.js

@@ -9,7 +9,7 @@ import { enforceCustomStepRegisters } from './enforceCustomStepRegisters';
 import { getScrollableDecorations } from './getScrollableDecorations';
 import { isExtendedEnabled } from './isExtendedEnabled';
 import { NodeViewSerializer } from './NodeViewSerializer';
-import { scrollToActiveDecoration, scrollToFirstDecoration } from './scrollToDiff';
+import { scrollToDecoration } from './scrollToDiff';
 export const showDiffPluginKey = new PluginKey('showDiffPlugin');
 export const createPlugin = (config, getIntl, api, onEditorView) => {
   enforceCustomStepRegisters();
@@ -194,7 +194,7 @@ export const createPlugin = (config, getIntl, api, onEditorView) => {
           if (pluginState !== null && pluginState !== void 0 && pluginState.scrollIntoView && isExtendedEnabled(pluginState === null || pluginState === void 0 ? void 0 : pluginState.diffType)) {
             var _cancelPendingScrollT;
             (_cancelPendingScrollT = cancelPendingScrollToDecoration) === null || _cancelPendingScrollT === void 0 ? void 0 : _cancelPendingScrollT();
-            cancelPendingScrollToDecoration = scrollToFirstDecoration(view, getScrollableDecorations(pluginState.decorations, view.state.doc, pluginState === null || pluginState === void 0 ? void 0 : pluginState.diffType));
+            cancelPendingScrollToDecoration = scrollToDecoration(view, getScrollableDecorations(pluginState.decorations, view.state.doc, pluginState === null || pluginState === void 0 ? void 0 : pluginState.diffType));
 
             // Reset the flag so we don't scroll again on subsequent updates
             view.dispatch(view.state.tr.setMeta(showDiffPluginKey, {
@@ -215,7 +215,7 @@ export const createPlugin = (config, getIntl, api, onEditorView) => {
               // avoid a circular dependency; expand picks up the meta if loaded (no-op if not).
               api === null || api === void 0 ? void 0 : api.core.actions.execute(toggleExpandRange(activeDecoration.from, activeDecoration.to, true));
             }
-            cancelPendingScrollToDecoration = scrollToActiveDecoration(view, scrollableDecorations, pluginState.activeIndex);
+            cancelPendingScrollToDecoration = scrollToDecoration(view, scrollableDecorations, pluginState.activeIndex);
           }
         },
         destroy() {

package/dist/es2019/pm-plugins/scrollToDiff.js

@@ -35,27 +35,12 @@ function scrollToSelection(node) {
 }
 
 /**
- * Schedules scrolling to the first diff decoration after the next frame.
- * Unlike `scrollToActiveDecoration`, this does not require an active index —
- * it simply scrolls to bring the first decoration into view.
- *
- * The caller must pass the list of scrollable decorations as produced by
- * `getScrollableDecorations`, which is filtered (non-scrollable decorations
- * removed) and sorted by document position. This guarantees "first" means the
- * topmost scrollable diff in the document rather than an arbitrary decoration
- * from the raw `DecorationSet` ordering. This is delegated to
- * `scrollToActiveDecoration` at index 0 so both scroll paths behave identically.
- *
- * @returns A function that cancels the scheduled `requestAnimationFrame` if it has not run yet.
- */
-export const scrollToFirstDecoration = (view, scrollableDecorations) => scrollToActiveDecoration(view, scrollableDecorations, 0);
-
-/**
- * Schedules scrolling to the decoration at the given index after the next frame.
+ * Schedules scrolling to the decoration at the given index after the next frame. Defaults to the
+ * first decoration when no index is provided.
  *
  * @returns A function that cancels the scheduled `requestAnimationFrame` if it has not run yet.
  */
-export const scrollToActiveDecoration = (view, decorations, activeIndex) => {
+export const scrollToDecoration = (view, decorations, activeIndex = 0) => {
   const decoration = decorations[activeIndex];
   if (!decoration) {
     return () => {};

package/dist/esm/pm-plugins/calculateDiff/calculateDiffDecorations.js

@@ -88,6 +88,26 @@ var leafTextblockRanges = function leafTextblockRanges(doc, from, to) {
   });
   return ranges;
 };
+
+// A large inserted table is treated with a coarse decoration path (see below)
+// once it has more leaf textblocks (~cells) than this threshold. Small tables
+// keep the precise per-cell path.
+var LARGE_TABLE_LEAF_THRESHOLD = 40;
+var isLargeInsertedTableRange = function isLargeInsertedTableRange(doc, from, to) {
+  var containsTable = false;
+  var leafCount = 0;
+  doc.nodesBetween(from, to, function (node) {
+    if (node.type.name === 'table') {
+      containsTable = true;
+    }
+    if (node.isTextblock) {
+      leafCount += 1;
+      return false;
+    }
+    return true;
+  });
+  return containsTable && leafCount > LARGE_TABLE_LEAF_THRESHOLD;
+};
 var calculateNodesForBlockDecoration = function calculateNodesForBlockDecoration(_ref2) {
   var doc = _ref2.doc,
     from = _ref2.from,
@@ -100,10 +120,20 @@ var calculateNodesForBlockDecoration = function calculateNodesForBlockDecoration
     shouldHideDeleted = _ref2$shouldHideDelet === void 0 ? false : _ref2$shouldHideDelet,
     _ref2$showIndicators = _ref2.showIndicators,
     showIndicators = _ref2$showIndicators === void 0 ? false : _ref2$showIndicators,
-    diffType = _ref2.diffType;
+    diffType = _ref2.diffType,
+    _ref2$coarseTableCell = _ref2.coarseTableCellsOnly,
+    coarseTableCellsOnly = _ref2$coarseTableCell === void 0 ? false : _ref2$coarseTableCell;
   var decorations = [];
   // Iterate over the document nodes within the range
   doc.nodesBetween(from, to, function (node, pos) {
+    // Coarse path: only cell/header overlays are visible; table/row/paragraph
+    // decorations are redundant and cause expensive per-cell re-render.
+    if (coarseTableCellsOnly) {
+      var name = node.type.name;
+      if (name !== 'tableCell' && name !== 'tableHeader') {
+        return;
+      }
+    }
     if (node.isBlock && (!isExtendedEnabled(diffType) || pos + node.nodeSize <= to)) {
       var nodeEnd = pos + node.nodeSize;
       var isActive = activeIndexPos && pos === activeIndexPos.from && nodeEnd === activeIndexPos.to;
@@ -283,12 +313,18 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref4
       // emit ONE inline decoration per leaf text-bearing block within the range — the text
       // gets highlighted, and structural gaps never do.
       var isSmartNodeLevel = diffType === 'smart' && fg('platform_editor_ai_smart_diff') && smartChangeLevel(change) === 'node';
+      // For a large inserted table, skip the per-cell inline decorations that
+      // freeze the browser on re-render. Only active in the AIFC experience.
+      var useCoarseTableDecoration = isSmartNodeLevel && expValEquals('platform_editor_ai_new_aifc_editor_experience', 'isEnabled', true) && isLargeInsertedTableRange(tr.doc, change.fromB, change.toB);
       // Whether the deleted-content widget will actually be rendered for this
       // change. Used to decide if indicator anchor positions should be adjusted
       // inward — when the widget is present the anchor must stay at the block
       // boundary to keep the indicator bar continuous with the deleted content.
       var willRenderDeletedWidget = change.deleted.length > 0 && !(isExtendedEnabled(diffType) && !isInverted && hideDeletedDiffs && change.inserted.length > 0);
-      if (isSmartNodeLevel) {
+      if (useCoarseTableDecoration) {
+        // Skip the O(cells) inline decorations; the cell overlays (added below)
+        // provide the highlight without the expensive content re-render.
+      } else if (isSmartNodeLevel) {
         var _iterator2 = _createForOfIteratorHelper(leafTextblockRanges(tr.doc, change.fromB, change.toB)),
           _step2;
         try {
@@ -347,7 +383,8 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref4
       }), {}, {
         activeIndexPos: activeIndexPos,
         intl: intl,
-        diffType: diffType
+        diffType: diffType,
+        coarseTableCellsOnly: useCoarseTableDecoration
       }))));
     }
     if (change.deleted.length > 0) {

package/dist/esm/pm-plugins/main.js

@@ -12,7 +12,7 @@ import { enforceCustomStepRegisters } from './enforceCustomStepRegisters';
 import { getScrollableDecorations } from './getScrollableDecorations';
 import { isExtendedEnabled } from './isExtendedEnabled';
 import { NodeViewSerializer } from './NodeViewSerializer';
-import { scrollToActiveDecoration, scrollToFirstDecoration } from './scrollToDiff';
+import { scrollToDecoration } from './scrollToDiff';
 export var showDiffPluginKey = new PluginKey('showDiffPlugin');
 export var createPlugin = function createPlugin(config, getIntl, api, onEditorView) {
   enforceCustomStepRegisters();
@@ -180,7 +180,7 @@ export var createPlugin = function createPlugin(config, getIntl, api, onEditorVi
           if (pluginState !== null && pluginState !== void 0 && pluginState.scrollIntoView && isExtendedEnabled(pluginState === null || pluginState === void 0 ? void 0 : pluginState.diffType)) {
             var _cancelPendingScrollT;
             (_cancelPendingScrollT = cancelPendingScrollToDecoration) === null || _cancelPendingScrollT === void 0 || _cancelPendingScrollT();
-            cancelPendingScrollToDecoration = scrollToFirstDecoration(view, getScrollableDecorations(pluginState.decorations, view.state.doc, pluginState === null || pluginState === void 0 ? void 0 : pluginState.diffType));
+            cancelPendingScrollToDecoration = scrollToDecoration(view, getScrollableDecorations(pluginState.decorations, view.state.doc, pluginState === null || pluginState === void 0 ? void 0 : pluginState.diffType));
 
             // Reset the flag so we don't scroll again on subsequent updates
             view.dispatch(view.state.tr.setMeta(showDiffPluginKey, {
@@ -201,7 +201,7 @@ export var createPlugin = function createPlugin(config, getIntl, api, onEditorVi
               // avoid a circular dependency; expand picks up the meta if loaded (no-op if not).
               api === null || api === void 0 || api.core.actions.execute(toggleExpandRange(activeDecoration.from, activeDecoration.to, true));
             }
-            cancelPendingScrollToDecoration = scrollToActiveDecoration(view, scrollableDecorations, pluginState.activeIndex);
+            cancelPendingScrollToDecoration = scrollToDecoration(view, scrollableDecorations, pluginState.activeIndex);
           }
         },
         destroy: function destroy() {

package/dist/esm/pm-plugins/scrollToDiff.js

@@ -35,29 +35,13 @@ function scrollToSelection(node) {
 }
 
 /**
- * Schedules scrolling to the first diff decoration after the next frame.
- * Unlike `scrollToActiveDecoration`, this does not require an active index —
- * it simply scrolls to bring the first decoration into view.
- *
- * The caller must pass the list of scrollable decorations as produced by
- * `getScrollableDecorations`, which is filtered (non-scrollable decorations
- * removed) and sorted by document position. This guarantees "first" means the
- * topmost scrollable diff in the document rather than an arbitrary decoration
- * from the raw `DecorationSet` ordering. This is delegated to
- * `scrollToActiveDecoration` at index 0 so both scroll paths behave identically.
- *
- * @returns A function that cancels the scheduled `requestAnimationFrame` if it has not run yet.
- */
-export var scrollToFirstDecoration = function scrollToFirstDecoration(view, scrollableDecorations) {
-  return scrollToActiveDecoration(view, scrollableDecorations, 0);
-};
-
-/**
- * Schedules scrolling to the decoration at the given index after the next frame.
+ * Schedules scrolling to the decoration at the given index after the next frame. Defaults to the
+ * first decoration when no index is provided.
  *
  * @returns A function that cancels the scheduled `requestAnimationFrame` if it has not run yet.
  */
-export var scrollToActiveDecoration = function scrollToActiveDecoration(view, decorations, activeIndex) {
+export var scrollToDecoration = function scrollToDecoration(view, decorations) {
+  var activeIndex = arguments.length > 2 && arguments[2] !== undefined ? arguments[2] : 0;
   var decoration = decorations[activeIndex];
   if (!decoration) {
     return function () {};

package/dist/types/pm-plugins/scrollToDiff.d.ts

@@ -1,22 +1,8 @@
 import type { EditorView, Decoration } from '@atlaskit/editor-prosemirror/view';
 /**
- * Schedules scrolling to the first diff decoration after the next frame.
- * Unlike `scrollToActiveDecoration`, this does not require an active index —
- * it simply scrolls to bring the first decoration into view.
- *
- * The caller must pass the list of scrollable decorations as produced by
- * `getScrollableDecorations`, which is filtered (non-scrollable decorations
- * removed) and sorted by document position. This guarantees "first" means the
- * topmost scrollable diff in the document rather than an arbitrary decoration
- * from the raw `DecorationSet` ordering. This is delegated to
- * `scrollToActiveDecoration` at index 0 so both scroll paths behave identically.
- *
- * @returns A function that cancels the scheduled `requestAnimationFrame` if it has not run yet.
- */
-export declare const scrollToFirstDecoration: (view: EditorView, scrollableDecorations: Decoration[]) => (() => void);
-/**
- * Schedules scrolling to the decoration at the given index after the next frame.
+ * Schedules scrolling to the decoration at the given index after the next frame. Defaults to the
+ * first decoration when no index is provided.
  *
  * @returns A function that cancels the scheduled `requestAnimationFrame` if it has not run yet.
  */
-export declare const scrollToActiveDecoration: (view: EditorView, decorations: Decoration[], activeIndex: number) => (() => void);
+export declare const scrollToDecoration: (view: EditorView, decorations: Decoration[], activeIndex?: number) => (() => void);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-show-diff",
-	"version": "13.0.7",
+	"version": "13.0.9",
 	"description": "ShowDiff plugin for @atlaskit/editor-core",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -36,8 +36,8 @@
 		"prosemirror-changeset": "^2.3.1"
 	},
 	"devDependencies": {
-		"@atlaskit/adf-utils": "^20.6.0",
-		"@atlaskit/button": "^25.0.0",
+		"@atlaskit/adf-utils": "^20.7.0",
+		"@atlaskit/button": "^25.1.0",
 		"@atlaskit/css": "^1.0.0",
 		"@atlaskit/dropdown-menu": "^18.0.0",
 		"@atlaskit/editor-core": "^224.1.0",
@@ -53,7 +53,7 @@
 		"react-intl": "^7.0.0"
 	},
 	"peerDependencies": {
-		"@atlaskit/editor-common": "^119.7.0",
+		"@atlaskit/editor-common": "^119.9.0",
 		"react": "^18.2.0 || ^19.2.0"
 	},
 	"techstack": {