@atlaskit/editor-plugin-show-diff 16.0.12 → 16.0.14

package/CHANGELOG.md

@@ -1,5 +1,35 @@
 # @atlaskit/editor-plugin-show-diff
 
+## 16.0.14
+
+### Patch Changes
+
+- [`6667b6ae8e77c`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/6667b6ae8e77c) -
+  Ensure contributor tags are rendered for shown attributed diff content, including whitespace-only
+  changes and deletion widgets, without tagging structurally empty paragraphs.
+
+## 16.0.13
+
+### Patch Changes
+
+- [`7d1a41f1575e5`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/7d1a41f1575e5) -
+  Reserve red for deletions in show-diff attribution: an actor hashing to a red palette slot now
+  takes the next colour instead. Gated on confluence_ncs_step_diffing_version_history and
+  platform_editor_show_diff_color_scheme_refactor.
+- [`19e336d422584`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/19e336d422584) -
+  Fix whole-row table replacements rendering no diff for the new content.
+  `createChangedRowDecorationWidgets` only handled row deletions, so a `replaceNode` on a `tableRow`
+  (as AI suggested edits emits) left the reviewer with just the removed side — or an empty row in
+  the clean view.
+
+  Also give the changed-row widget an indicator anchor, so the indicator bar spans both the proposed
+  and replaced rows instead of only the row being changed. This affects row deletions too, whose
+  deleted-row bar previously had no anchor to resolve against.
+
+  Both behind `platform_editor_ai_show_diff_patch_1`.
+
+- Updated dependencies
+
 ## 16.0.12
 
 ### Patch Changes

package/dist/cjs/pm-plugins/decorations/colorSchemes/attributions.js

@@ -25,15 +25,40 @@ function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length)
 
 /** Maps the shared 18-slot telepointer palette onto the ten bolder diff colours. */
 var HASHED_PARTICIPANT_COLOR_SCHEMES = [].concat((0, _toConsumableArray2.default)(_types.PARTICIPANT_COLOR_SCHEMES), ['blue', 'red', 'orange', 'yellow', 'green', 'teal', 'purple', 'magenta']);
+
+/**
+ * Red means "deleted" in both public schemes, so it never identifies an actor. Reserving it here
+ * covers every path into an attribution colour; `createAttributionColorMap` is the only caller, and
+ * it is gated on `confluence_ncs_step_diffing_version_history` plus
+ * `platform_editor_show_diff_color_scheme_refactor`.
+ */
+var RESERVED_ATTRIBUTION_COLORS = new Set(['red']);
+var isAssignable = function isAssignable(color) {
+  return !RESERVED_ATTRIBUTION_COLORS.has(color);
+};
+
+/** Projects a telepointer slot onto its diff hue, advancing past a hue reserved for deletion. */
+var getSlotColor = function getSlotColor(index) {
+  for (var offset = 0; offset < HASHED_PARTICIPANT_COLOR_SCHEMES.length; offset++) {
+    var color = HASHED_PARTICIPANT_COLOR_SCHEMES[(index + offset) % HASHED_PARTICIPANT_COLOR_SCHEMES.length];
+    if (color && isAssignable(color)) {
+      return color;
+    }
+  }
+  return undefined;
+};
 var getAvailableColor = function getAvailableColor(hashedColor, allocatedColors) {
+  var _PARTICIPANT_COLOR_SC;
   var startIndex = _types.PARTICIPANT_COLOR_SCHEMES.indexOf(hashedColor);
   for (var offset = 0; offset < _types.PARTICIPANT_COLOR_SCHEMES.length; offset++) {
     var color = _types.PARTICIPANT_COLOR_SCHEMES[(startIndex + offset) % _types.PARTICIPANT_COLOR_SCHEMES.length];
-    if (color && !allocatedColors.has(color)) {
+    if (color && isAssignable(color) && !allocatedColors.has(color)) {
       return color;
     }
   }
-  return hashedColor;
+
+  // Every assignable hue is taken, so a duplicate is unavoidable — but never the reserved hue.
+  return isAssignable(hashedColor) ? hashedColor : (_PARTICIPANT_COLOR_SC = _types.PARTICIPANT_COLOR_SCHEMES.find(isAssignable)) !== null && _PARTICIPANT_COLOR_SC !== void 0 ? _PARTICIPANT_COLOR_SC : hashedColor;
 };
 
 /**
@@ -133,7 +158,8 @@ var createAttributionColorMap = exports.createAttributionColorMap = function cre
   var colors = new Map();
   var allocatedColors = new Set();
 
-  // Reserve shared brand colours before hashing other actors, regardless of step order.
+  // Reserve shared brand colours before hashing other actors, regardless of step order. A brand
+  // slot is pinned deliberately, so it is taken verbatim rather than through `getSlotColor`.
   var _iterator2 = _createForOfIteratorHelper(stepAttributions),
     _step2;
   try {
@@ -168,7 +194,7 @@ var createAttributionColorMap = exports.createAttributionColorMap = function cre
       }
       var _getParticipantColor2 = (0, _utils.getParticipantColor)(_identity.colorSeed, _attribution === null || _attribution === void 0 ? void 0 : _attribution.agentType),
         _index = _getParticipantColor2.index;
-      var hashedColor = HASHED_PARTICIPANT_COLOR_SCHEMES[_index];
+      var hashedColor = getSlotColor(_index);
       if (!hashedColor) {
         continue;
       }

package/dist/cjs/pm-plugins/decorations/createChangedRowDecorationWidgets.js

@@ -12,10 +12,12 @@ var _utils = require("@atlaskit/editor-prosemirror/utils");
 var _view = require("@atlaskit/editor-prosemirror/view");
 var _tableMap = require("@atlaskit/editor-tables/table-map");
 var _isExperimentEnabled = require("@atlaskit/platform-feature-experiments/is-experiment-enabled");
+var _fg = require("@atlaskit/platform-feature-flags/fg");
 var _expValEquals = require("@atlaskit/tmp-editor-statsig/exp-val-equals");
 var _isExtendedEnabled = require("../isExtendedEnabled");
 var _factory = require("./colorSchemes/factory");
 var _schemes = require("./colorSchemes/schemes");
+var _createAnchorDecorationWidgets = require("./createAnchorDecorationWidgets");
 var _createChangedRowDecorationWidgetsStyles = require("./createChangedRowDecorationWidgets.styles.legacy");
 var _decorationKeys = require("./decorationKeys");
 var _findSafeInsertPos = require("./utils/findSafeInsertPos");
@@ -54,8 +56,14 @@ var extractChangedRows = function extractChangedRows(_ref) {
     return changedRows;
   }
   var newTableMap = _tableMap.TableMap.get(tableNew.node);
+  // A row DELETION makes the table shorter. An in-place whole-row REPLACEMENT (`replaceNode` on a
+  // `tableRow`, which AI suggested edits emits) leaves the row count untouched — and this helper
+  // was written for deletions only, so it used to discard every replacement and the changed row's
+  // content was never rendered anywhere. The reviewer was left with just the removed side.
+  var isRowDeletion = oldTableMap.height > newTableMap.height;
+  var handleRowReplacement = oldTableMap.height === newTableMap.height && (0, _fg.fg)('platform_editor_ai_show_diff_patch_1');
   // If no rows were changed, return empty
-  if (oldTableMap.height <= newTableMap.height ||
+  if (!(isRowDeletion || handleRowReplacement) ||
   // For now ignore if there are column deletions as well
   oldTableMap.width !== newTableMap.width) {
     return changedRows;
@@ -99,6 +107,15 @@ var extractChangedRows = function extractChangedRows(_ref) {
 
   // Filter changes that never truly got deleted
   return changedRows.filter(function (changedRow) {
+    // A replacement leaves every sibling row in place, so asking "does this row still exist
+    // ANYWHERE in the new table?" can drop a genuinely rewritten row that happens to match a
+    // sibling. Compare it against the row at the same index instead. Deletions must keep the
+    // whole-table `some()`: their indices shift, so an index comparison would let every
+    // surviving row through and draw a widget for each one.
+    if (handleRowReplacement) {
+      var counterpart = tableNew.node.maybeChild(changedRow.rowIndex);
+      return !counterpart || !(0, _document.areNodesEqualIgnoreAttrs)(counterpart, changedRow.rowNode);
+    }
     return !tableNew.node.children.some(function (newRow) {
       return (0, _document.areNodesEqualIgnoreAttrs)(newRow, changedRow.rowNode);
     });
@@ -142,7 +159,10 @@ var createChangedRowDOM = function createChangedRowDOM(rowNode, cellEdgeAttrs, n
   if (!(0, _isExtendedEnabled.isExtendedEnabled)(diffType) || !isInserted) {
     tr.setAttribute('style', (0, _isExperimentEnabled.isExperimentEnabled)('platform_editor_show_diff_color_scheme_refactor') ? (0, _factory.buildDeletedRowStyle)(colors) : (0, _createChangedRowDecorationWidgetsStyles.resolveDeletedRowStyleLegacy)((0, _schemes.getLegacyColorScheme)(colorScheme)));
   }
-  tr.setAttribute('data-testid', 'show-diff-deleted-row');
+  // Mirrors the strikethrough condition above: under the extended experience an `isInserted` row
+  // is ADDED content, so it must not claim the "deleted" testid — page models match that as
+  // removed content (same reasoning as `createTableCellContentWidgets`).
+  tr.setAttribute('data-testid', (0, _isExtendedEnabled.isExtendedEnabled)(diffType) && isInserted && (0, _fg.fg)('platform_editor_ai_show_diff_patch_1') ? 'show-diff-changed-row' : 'show-diff-deleted-row');
 
   // Serialize each cell in the row
   var cellIndex = 0;
@@ -222,7 +242,9 @@ var createChangedRowDecorationWidgets = exports.createChangedRowDecorationWidget
     colorScheme = _ref3.colorScheme,
     _ref3$isInserted = _ref3.isInserted,
     isInserted = _ref3$isInserted === void 0 ? false : _ref3$isInserted,
-    diffType = _ref3.diffType;
+    diffType = _ref3.diffType,
+    _ref3$showIndicators = _ref3.showIndicators,
+    showIndicators = _ref3$showIndicators === void 0 ? false : _ref3$showIndicators;
   // First, expand the changes to include complete deleted rows
   var changedRows = expandDiffForChangedRows({
     changes: changes.filter(function (change) {
@@ -232,7 +254,7 @@ var createChangedRowDecorationWidgets = exports.createChangedRowDecorationWidget
     newDoc: newDoc,
     diffType: diffType
   });
-  return changedRows.map(function (changedRow) {
+  return changedRows.flatMap(function (changedRow) {
     var rowDOM = createChangedRowDOM(changedRow.rowNode, changedRow.cellEdgeAttrs, nodeViewSerializer, colorScheme, isInserted, diffType);
 
     // Find safe insertion position for the deleted row
@@ -240,13 +262,37 @@ var createChangedRowDecorationWidgets = exports.createChangedRowDecorationWidget
     // -1 to find the first safe position from the table
     originalDoc.slice(changedRow.fromA, changedRow.toA));
     var diffId = crypto.randomUUID();
-    return _view.Decoration.widget(safeInsertPos, rowDOM, _objectSpread({}, (0, _decorationKeys.buildDiffDecorationSpec)({
+    var decorations = [];
+
+    // `IndicatorBarContentComponent` renders a bar for every widget descriptor and anchors its
+    // top and bottom to `anchor-<diffId>`. Without an element carrying that anchor name the bar
+    // is still created but can never resolve a position, so it silently does not render and the
+    // bar covers only the row being changed. `createNodeChangedDecorationWidget` sets the same
+    // property on its own widget DOM for non-table content; this path returns before reaching it.
+    if (showIndicators && (0, _isExtendedEnabled.isExtendedEnabled)(diffType) && (0, _fg.fg)('platform_editor_ai_show_diff_patch_1')) {
+      rowDOM.style.setProperty('anchor-name', "--".concat((0, _decorationKeys.buildAnchorDecorationKey)({
+        diffId: diffId
+      })));
+
+      // A table's content can extend past the doc margin, so the bar also needs a left anchor
+      // measured against the table itself or it is clipped once the table is resized.
+      var leftAnchor = (0, _createAnchorDecorationWidgets.createLeftAnchorWidget)({
+        doc: newDoc,
+        from: safeInsertPos,
+        diffId: diffId
+      });
+      if (leftAnchor) {
+        decorations.push(leftAnchor);
+      }
+    }
+    decorations.push(_view.Decoration.widget(safeInsertPos, rowDOM, _objectSpread({}, (0, _decorationKeys.buildDiffDecorationSpec)({
       colorScheme: colorScheme,
       decorationType: 'widget',
       diffId: diffId,
       isActive: false,
       isInserted: isInserted,
       diffType: diffType
-    })));
+    }))));
+    return decorations;
   });
 };

package/dist/cjs/pm-plugins/decorations/createInlineChangedDecoration.js

@@ -12,7 +12,7 @@ var _view = require("@atlaskit/editor-prosemirror/view");
 var _isExperimentEnabled = require("@atlaskit/platform-feature-experiments/is-experiment-enabled");
 var _buildContributorTagDom = require("../../ui/ContributorTag/buildContributorTagDom");
 var _isExtendedEnabled = require("../isExtendedEnabled");
-var _hasVisibleContent = require("../utils/hasVisibleContent");
+var _isEmptyParagraphSlice = require("../utils/isEmptyParagraphSlice");
 var _factory = require("./colorSchemes/factory");
 var _schemes = require("./colorSchemes/schemes");
 var _createAnchorDecorationWidgets = require("./createAnchorDecorationWidgets");
@@ -163,11 +163,9 @@ var createInlineChangedDecoration = exports.createInlineChangedDecoration = func
   // tag host's key, remounting every tag mid-step. Random ids are kept when tags are off.
   var diffId = showContributorTags ? "inline-".concat(change.fromB, "-").concat(change.toB) : crypto.randomUUID();
 
-  // Whether this highlight hosts a tag. A range of nothing but whitespace has nothing to caption,
-  // so it stays unattributed: the spec is the only route attribution takes to
-  // `extractContributorTags`, so dropping the key there is what keeps a tag from being resolved for
-  // a host that is never emitted (EDITOR-8855).
-  var canTagChange = showContributorTags && !!doc && (0, _isExtendedEnabled.isExtendedEnabled)(diffType) && (0, _createContributorTagWidget.isContributorTagWidgetEnabled)() && (0, _hasVisibleContent.hasVisibleContent)(doc.slice(change.fromB, change.toB).content);
+  // Match the rendered shape: whitespace text is eligible, while an empty paragraph is structural
+  // content with no tag host.
+  var canTagChange = showContributorTags && !!doc && (0, _isExtendedEnabled.isExtendedEnabled)(diffType) && (0, _createContributorTagWidget.isContributorTagWidgetEnabled)() && !(0, _isEmptyParagraphSlice.isEmptyParagraphSlice)(doc.slice(change.fromB, change.toB));
   if (shouldHideDeleted) {
     return [_view.Decoration.inline(change.fromB, change.toB, {
       style: displayNoneStyle

package/dist/cjs/pm-plugins/decorations/createNodeChangedDecorationWidget.js

@@ -9,7 +9,7 @@ var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/de
 var _view = require("@atlaskit/editor-prosemirror/view");
 var _fg = require("@atlaskit/platform-feature-flags/fg");
 var _isExtendedEnabled = require("../isExtendedEnabled");
-var _hasVisibleContent = require("../utils/hasVisibleContent");
+var _isEmptyParagraphSlice = require("../utils/isEmptyParagraphSlice");
 var _createAnchorDecorationWidgets = require("./createAnchorDecorationWidgets");
 var _createChangedRowDecorationWidgets = require("./createChangedRowDecorationWidgets");
 var _createContributorTagWidget = require("./createContributorTagWidget");
@@ -101,7 +101,7 @@ var createTableCellContentWidgets = function createTableCellContentWidgets(_ref)
  * that is not in the current document.
  */
 var createNodeChangedDecorationWidget = exports.createNodeChangedDecorationWidget = function createNodeChangedDecorationWidget(_ref2) {
-  var _slice$content, _slice$content2, _slice$content3, _$safeInsertPos$nodeB, _slice$content$firstC, _newDoc$firstChild, _slice$content$firstC2;
+  var _$safeInsertPos$nodeB, _slice$content$firstC, _newDoc$firstChild, _slice$content$firstC2;
   var attributionKey = _ref2.attributionKey,
     change = _ref2.change,
     doc = _ref2.doc,
@@ -124,7 +124,7 @@ var createNodeChangedDecorationWidget = exports.createNodeChangedDecorationWidge
     reveal = _ref2.reveal,
     tagMountContext = _ref2.tagMountContext;
   var slice = doc.slice(change.fromA, change.toA);
-  var shouldSkipDeletedEmptyParagraphDecoration = !isInserted && (slice === null || slice === void 0 || (_slice$content = slice.content) === null || _slice$content === void 0 ? void 0 : _slice$content.childCount) === 1 && (slice === null || slice === void 0 || (_slice$content2 = slice.content) === null || _slice$content2 === void 0 || (_slice$content2 = _slice$content2.firstChild) === null || _slice$content2 === void 0 ? void 0 : _slice$content2.type.name) === 'paragraph' && (slice === null || slice === void 0 || (_slice$content3 = slice.content) === null || _slice$content3 === void 0 || (_slice$content3 = _slice$content3.firstChild) === null || _slice$content3 === void 0 ? void 0 : _slice$content3.content.size) === 0;
+  var shouldSkipDeletedEmptyParagraphDecoration = !isInserted && (0, _isEmptyParagraphSlice.isEmptyParagraphSlice)(slice);
   // Widget decoration used for deletions as the content is not in the document
   // and we want to display the deleted content with a style.
   // For `placeBelow`, anchor at the END of the new content (change.toB) so the deleted
@@ -172,7 +172,10 @@ var createNodeChangedDecorationWidget = exports.createNodeChangedDecorationWidge
       nodeViewSerializer: nodeViewSerializer,
       colorScheme: colorScheme,
       isInserted: isInserted,
-      diffType: diffType
+      diffType: diffType,
+      // Needed for the row's own indicator anchor; this path returns before the
+      // `anchor-name` assignment further down.
+      showIndicators: showIndicators
     });
   }
   var serializer = nodeViewSerializer;
@@ -223,10 +226,9 @@ var createNodeChangedDecorationWidget = exports.createNodeChangedDecorationWidge
   // Derived from the deleted range so it survives a recalculation — see the same reasoning in
   // `createInlineChangedDecoration`.
   var diffId = showContributorTags ? "widget-".concat(change.fromA, "-").concat(change.toA) : crypto.randomUUID();
-  // Whether this widget hosts a tag. Removing nothing but whitespace leaves no visible deleted
-  // content to caption, so the change stays unattributed — matching the inline half's rule in
-  // `createInlineChangedDecoration` (EDITOR-8855).
-  var canTagWidget = showContributorTags && (0, _hasVisibleContent.hasVisibleContent)(slice.content);
+  // Match the rendering predicate above: an empty paragraph has no widget to tag, while whitespace
+  // text in a rendered widget remains eligible.
+  var canTagWidget = showContributorTags && !shouldSkipDeletedEmptyParagraphDecoration;
   var decorations = [];
   var replacementNode = newDoc.nodeAt(change.fromB);
   var firstReplacedNode = slice.content.firstChild;

package/dist/cjs/pm-plugins/utils/isEmptyParagraphSlice.js

@@ -0,0 +1,11 @@
+"use strict";
+
+Object.defineProperty(exports, "__esModule", {
+  value: true
+});
+exports.isEmptyParagraphSlice = void 0;
+/** Whether a slice contains exactly one structurally empty paragraph. */
+var isEmptyParagraphSlice = exports.isEmptyParagraphSlice = function isEmptyParagraphSlice(slice) {
+  var _slice$content$firstC;
+  return (slice === null || slice === void 0 ? void 0 : slice.content.childCount) === 1 && ((_slice$content$firstC = slice.content.firstChild) === null || _slice$content$firstC === void 0 ? void 0 : _slice$content$firstC.type.name) === 'paragraph' && slice.content.firstChild.content.size === 0;
+};

package/dist/cjs/ui/IndicatorBar/IndicatorBar.js

@@ -1,4 +1,4 @@
-/* IndicatorBar.tsx generated by @compiled/babel-plugin v3.0.2 */
+/* IndicatorBar.tsx generated by @compiled/babel-plugin v3.0.1 */
 "use strict";
 
 var _typeof = require("@babel/runtime/helpers/typeof");

package/dist/cjs/ui/IndicatorBar/IndicatorBarContentComponent.js

@@ -1,4 +1,4 @@
-/* IndicatorBarContentComponent.tsx generated by @compiled/babel-plugin v3.0.2 */
+/* IndicatorBarContentComponent.tsx generated by @compiled/babel-plugin v3.0.1 */
 "use strict";
 
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");

package/dist/es2019/pm-plugins/decorations/colorSchemes/attributions.js

@@ -9,15 +9,38 @@ import { PARTICIPANT_COLOR_SCHEMES } from './types';
 
 /** Maps the shared 18-slot telepointer palette onto the ten bolder diff colours. */
 const HASHED_PARTICIPANT_COLOR_SCHEMES = [...PARTICIPANT_COLOR_SCHEMES, 'blue', 'red', 'orange', 'yellow', 'green', 'teal', 'purple', 'magenta'];
+
+/**
+ * Red means "deleted" in both public schemes, so it never identifies an actor. Reserving it here
+ * covers every path into an attribution colour; `createAttributionColorMap` is the only caller, and
+ * it is gated on `confluence_ncs_step_diffing_version_history` plus
+ * `platform_editor_show_diff_color_scheme_refactor`.
+ */
+const RESERVED_ATTRIBUTION_COLORS = new Set(['red']);
+const isAssignable = color => !RESERVED_ATTRIBUTION_COLORS.has(color);
+
+/** Projects a telepointer slot onto its diff hue, advancing past a hue reserved for deletion. */
+const getSlotColor = index => {
+  for (let offset = 0; offset < HASHED_PARTICIPANT_COLOR_SCHEMES.length; offset++) {
+    const color = HASHED_PARTICIPANT_COLOR_SCHEMES[(index + offset) % HASHED_PARTICIPANT_COLOR_SCHEMES.length];
+    if (color && isAssignable(color)) {
+      return color;
+    }
+  }
+  return undefined;
+};
 const getAvailableColor = (hashedColor, allocatedColors) => {
+  var _PARTICIPANT_COLOR_SC;
   const startIndex = PARTICIPANT_COLOR_SCHEMES.indexOf(hashedColor);
   for (let offset = 0; offset < PARTICIPANT_COLOR_SCHEMES.length; offset++) {
     const color = PARTICIPANT_COLOR_SCHEMES[(startIndex + offset) % PARTICIPANT_COLOR_SCHEMES.length];
-    if (color && !allocatedColors.has(color)) {
+    if (color && isAssignable(color) && !allocatedColors.has(color)) {
       return color;
     }
   }
-  return hashedColor;
+
+  // Every assignable hue is taken, so a duplicate is unavoidable — but never the reserved hue.
+  return isAssignable(hashedColor) ? hashedColor : (_PARTICIPANT_COLOR_SC = PARTICIPANT_COLOR_SCHEMES.find(isAssignable)) !== null && _PARTICIPANT_COLOR_SC !== void 0 ? _PARTICIPANT_COLOR_SC : hashedColor;
 };
 
 /**
@@ -104,7 +127,8 @@ export const createAttributionColorMap = stepAttributions => {
   const colors = new Map();
   const allocatedColors = new Set();
 
-  // Reserve shared brand colours before hashing other actors, regardless of step order.
+  // Reserve shared brand colours before hashing other actors, regardless of step order. A brand
+  // slot is pinned deliberately, so it is taken verbatim rather than through `getSlotColor`.
   for (const attribution of stepAttributions) {
     const identity = getAttributionIdentity(attribution);
     if (!identity) {
@@ -128,7 +152,7 @@ export const createAttributionColorMap = stepAttributions => {
     const {
       index
     } = getParticipantColor(identity.colorSeed, attribution === null || attribution === void 0 ? void 0 : attribution.agentType);
-    const hashedColor = HASHED_PARTICIPANT_COLOR_SCHEMES[index];
+    const hashedColor = getSlotColor(index);
     if (!hashedColor) {
       continue;
     }

package/dist/es2019/pm-plugins/decorations/createChangedRowDecorationWidgets.js

@@ -3,12 +3,14 @@ import { findParentNodeClosestToPos } from '@atlaskit/editor-prosemirror/utils';
 import { Decoration } from '@atlaskit/editor-prosemirror/view';
 import { TableMap } from '@atlaskit/editor-tables/table-map';
 import { isExperimentEnabled } from '@atlaskit/platform-feature-experiments/is-experiment-enabled';
+import { fg } from '@atlaskit/platform-feature-flags/fg';
 import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 import { isExtendedEnabled } from '../isExtendedEnabled';
 import { buildAddedCellOverlayRoundedStyle, buildAddedCellOverlayStyle, buildDeletedCellOverlayRoundedStyle, buildDeletedCellOverlayStyle, buildDeletedRowStyle } from './colorSchemes/factory';
 import { colorSchemeRegistry, getLegacyColorScheme } from './colorSchemes/schemes';
+import { createLeftAnchorWidget } from './createAnchorDecorationWidgets';
 import { resolveCellOverlayStyleLegacy, resolveDeletedRowStyleLegacy } from './createChangedRowDecorationWidgets.styles.legacy';
-import { buildDiffDecorationSpec } from './decorationKeys';
+import { buildAnchorDecorationKey, buildDiffDecorationSpec } from './decorationKeys';
 import { findSafeInsertPos } from './utils/findSafeInsertPos';
 import { applyCellEdgeAttrs, getRowCellEdgeAttrs, getTableTopAndBottomCellEdgeAttrs } from './utils/tableCellEdgeAttrs';
 /**
@@ -37,8 +39,14 @@ const extractChangedRows = ({
     return changedRows;
   }
   const newTableMap = TableMap.get(tableNew.node);
+  // A row DELETION makes the table shorter. An in-place whole-row REPLACEMENT (`replaceNode` on a
+  // `tableRow`, which AI suggested edits emits) leaves the row count untouched — and this helper
+  // was written for deletions only, so it used to discard every replacement and the changed row's
+  // content was never rendered anywhere. The reviewer was left with just the removed side.
+  const isRowDeletion = oldTableMap.height > newTableMap.height;
+  const handleRowReplacement = oldTableMap.height === newTableMap.height && fg('platform_editor_ai_show_diff_patch_1');
   // If no rows were changed, return empty
-  if (oldTableMap.height <= newTableMap.height ||
+  if (!(isRowDeletion || handleRowReplacement) ||
   // For now ignore if there are column deletions as well
   oldTableMap.width !== newTableMap.width) {
     return changedRows;
@@ -80,6 +88,15 @@ const extractChangedRows = ({
 
   // Filter changes that never truly got deleted
   return changedRows.filter(changedRow => {
+    // A replacement leaves every sibling row in place, so asking "does this row still exist
+    // ANYWHERE in the new table?" can drop a genuinely rewritten row that happens to match a
+    // sibling. Compare it against the row at the same index instead. Deletions must keep the
+    // whole-table `some()`: their indices shift, so an index comparison would let every
+    // surviving row through and draw a widget for each one.
+    if (handleRowReplacement) {
+      const counterpart = tableNew.node.maybeChild(changedRow.rowIndex);
+      return !counterpart || !areNodesEqualIgnoreAttrs(counterpart, changedRow.rowNode);
+    }
     return !tableNew.node.children.some(newRow => areNodesEqualIgnoreAttrs(newRow, changedRow.rowNode));
   });
 };
@@ -121,7 +138,10 @@ const createChangedRowDOM = (rowNode, cellEdgeAttrs, nodeViewSerializer, colorSc
   if (!isExtendedEnabled(diffType) || !isInserted) {
     tr.setAttribute('style', isExperimentEnabled('platform_editor_show_diff_color_scheme_refactor') ? buildDeletedRowStyle(colors) : resolveDeletedRowStyleLegacy(getLegacyColorScheme(colorScheme)));
   }
-  tr.setAttribute('data-testid', 'show-diff-deleted-row');
+  // Mirrors the strikethrough condition above: under the extended experience an `isInserted` row
+  // is ADDED content, so it must not claim the "deleted" testid — page models match that as
+  // removed content (same reasoning as `createTableCellContentWidgets`).
+  tr.setAttribute('data-testid', isExtendedEnabled(diffType) && isInserted && fg('platform_editor_ai_show_diff_patch_1') ? 'show-diff-changed-row' : 'show-diff-deleted-row');
 
   // Serialize each cell in the row
   let cellIndex = 0;
@@ -192,7 +212,8 @@ export const createChangedRowDecorationWidgets = ({
   nodeViewSerializer,
   colorScheme,
   isInserted = false,
-  diffType
+  diffType,
+  showIndicators = false
 }) => {
   // First, expand the changes to include complete deleted rows
   const changedRows = expandDiffForChangedRows({
@@ -201,7 +222,7 @@ export const createChangedRowDecorationWidgets = ({
     newDoc,
     diffType
   });
-  return changedRows.map(changedRow => {
+  return changedRows.flatMap(changedRow => {
     const rowDOM = createChangedRowDOM(changedRow.rowNode, changedRow.cellEdgeAttrs, nodeViewSerializer, colorScheme, isInserted, diffType);
 
     // Find safe insertion position for the deleted row
@@ -209,7 +230,30 @@ export const createChangedRowDecorationWidgets = ({
     // -1 to find the first safe position from the table
     originalDoc.slice(changedRow.fromA, changedRow.toA));
     const diffId = crypto.randomUUID();
-    return Decoration.widget(safeInsertPos, rowDOM, {
+    const decorations = [];
+
+    // `IndicatorBarContentComponent` renders a bar for every widget descriptor and anchors its
+    // top and bottom to `anchor-<diffId>`. Without an element carrying that anchor name the bar
+    // is still created but can never resolve a position, so it silently does not render and the
+    // bar covers only the row being changed. `createNodeChangedDecorationWidget` sets the same
+    // property on its own widget DOM for non-table content; this path returns before reaching it.
+    if (showIndicators && isExtendedEnabled(diffType) && fg('platform_editor_ai_show_diff_patch_1')) {
+      rowDOM.style.setProperty('anchor-name', `--${buildAnchorDecorationKey({
+        diffId
+      })}`);
+
+      // A table's content can extend past the doc margin, so the bar also needs a left anchor
+      // measured against the table itself or it is clipped once the table is resized.
+      const leftAnchor = createLeftAnchorWidget({
+        doc: newDoc,
+        from: safeInsertPos,
+        diffId
+      });
+      if (leftAnchor) {
+        decorations.push(leftAnchor);
+      }
+    }
+    decorations.push(Decoration.widget(safeInsertPos, rowDOM, {
       ...buildDiffDecorationSpec({
         colorScheme,
         decorationType: 'widget',
@@ -218,6 +262,7 @@ export const createChangedRowDecorationWidgets = ({
         isInserted,
         diffType
       })
-    });
+    }));
+    return decorations;
   });
 };

package/dist/es2019/pm-plugins/decorations/createInlineChangedDecoration.js

@@ -3,7 +3,7 @@ import { Decoration } from '@atlaskit/editor-prosemirror/view';
 import { isExperimentEnabled } from '@atlaskit/platform-feature-experiments/is-experiment-enabled';
 import { CONTRIBUTOR_TAG_Z_INDEX } from '../../ui/ContributorTag/buildContributorTagDom';
 import { isExtendedEnabled } from '../isExtendedEnabled';
-import { hasVisibleContent } from '../utils/hasVisibleContent';
+import { isEmptyParagraphSlice } from '../utils/isEmptyParagraphSlice';
 import { buildAtomicInlineChangedCSSVariables, buildDeletedInlineContentStyleExtended, buildDeletedInlineStyle, buildDeletedInlineStyleStandard, buildInsertStyle, buildInsertStyleActive, buildInsertStyleExtended, buildInsertStyleExtendedActive, buildInsertStyleExtendedNoUnderline, buildInsertStyleExtendedNoUnderlineActive } from './colorSchemes/factory';
 import { colorSchemeRegistry, getLegacyColorScheme } from './colorSchemes/schemes';
 import { createInlineIndicatorAnchorWidgets } from './createAnchorDecorationWidgets';
@@ -138,11 +138,9 @@ export const createInlineChangedDecoration = ({
   // tag host's key, remounting every tag mid-step. Random ids are kept when tags are off.
   const diffId = showContributorTags ? `inline-${change.fromB}-${change.toB}` : crypto.randomUUID();
 
-  // Whether this highlight hosts a tag. A range of nothing but whitespace has nothing to caption,
-  // so it stays unattributed: the spec is the only route attribution takes to
-  // `extractContributorTags`, so dropping the key there is what keeps a tag from being resolved for
-  // a host that is never emitted (EDITOR-8855).
-  const canTagChange = showContributorTags && !!doc && isExtendedEnabled(diffType) && isContributorTagWidgetEnabled() && hasVisibleContent(doc.slice(change.fromB, change.toB).content);
+  // Match the rendered shape: whitespace text is eligible, while an empty paragraph is structural
+  // content with no tag host.
+  const canTagChange = showContributorTags && !!doc && isExtendedEnabled(diffType) && isContributorTagWidgetEnabled() && !isEmptyParagraphSlice(doc.slice(change.fromB, change.toB));
   if (shouldHideDeleted) {
     return [Decoration.inline(change.fromB, change.toB, {
       style: displayNoneStyle

package/dist/es2019/pm-plugins/decorations/createNodeChangedDecorationWidget.js

@@ -1,7 +1,7 @@
 import { Decoration } from '@atlaskit/editor-prosemirror/view';
 import { fg } from '@atlaskit/platform-feature-flags/fg';
 import { isExtendedEnabled } from '../isExtendedEnabled';
-import { hasVisibleContent } from '../utils/hasVisibleContent';
+import { isEmptyParagraphSlice } from '../utils/isEmptyParagraphSlice';
 import { createLeftAnchorWidget } from './createAnchorDecorationWidgets';
 import { createChangedRowDecorationWidgets } from './createChangedRowDecorationWidgets';
 import { createContributorTagHost, unmountContributorTag } from './createContributorTagWidget';
@@ -114,9 +114,9 @@ export const createNodeChangedDecorationWidget = ({
   reveal,
   tagMountContext
 }) => {
-  var _slice$content, _slice$content2, _slice$content2$first, _slice$content3, _slice$content3$first, _$safeInsertPos$nodeB, _slice$content$firstC, _newDoc$firstChild, _slice$content$firstC2;
+  var _$safeInsertPos$nodeB, _slice$content$firstC, _newDoc$firstChild, _slice$content$firstC2;
   const slice = doc.slice(change.fromA, change.toA);
-  const shouldSkipDeletedEmptyParagraphDecoration = !isInserted && (slice === null || slice === void 0 ? void 0 : (_slice$content = slice.content) === null || _slice$content === void 0 ? void 0 : _slice$content.childCount) === 1 && (slice === null || slice === void 0 ? void 0 : (_slice$content2 = slice.content) === null || _slice$content2 === void 0 ? void 0 : (_slice$content2$first = _slice$content2.firstChild) === null || _slice$content2$first === void 0 ? void 0 : _slice$content2$first.type.name) === 'paragraph' && (slice === null || slice === void 0 ? void 0 : (_slice$content3 = slice.content) === null || _slice$content3 === void 0 ? void 0 : (_slice$content3$first = _slice$content3.firstChild) === null || _slice$content3$first === void 0 ? void 0 : _slice$content3$first.content.size) === 0;
+  const shouldSkipDeletedEmptyParagraphDecoration = !isInserted && isEmptyParagraphSlice(slice);
   // Widget decoration used for deletions as the content is not in the document
   // and we want to display the deleted content with a style.
   // For `placeBelow`, anchor at the END of the new content (change.toB) so the deleted
@@ -156,7 +156,10 @@ export const createNodeChangedDecorationWidget = ({
       nodeViewSerializer,
       colorScheme,
       isInserted,
-      diffType
+      diffType,
+      // Needed for the row's own indicator anchor; this path returns before the
+      // `anchor-name` assignment further down.
+      showIndicators
     });
   }
   const serializer = nodeViewSerializer;
@@ -207,10 +210,9 @@ export const createNodeChangedDecorationWidget = ({
   // Derived from the deleted range so it survives a recalculation — see the same reasoning in
   // `createInlineChangedDecoration`.
   const diffId = showContributorTags ? `widget-${change.fromA}-${change.toA}` : crypto.randomUUID();
-  // Whether this widget hosts a tag. Removing nothing but whitespace leaves no visible deleted
-  // content to caption, so the change stays unattributed — matching the inline half's rule in
-  // `createInlineChangedDecoration` (EDITOR-8855).
-  const canTagWidget = showContributorTags && hasVisibleContent(slice.content);
+  // Match the rendering predicate above: an empty paragraph has no widget to tag, while whitespace
+  // text in a rendered widget remains eligible.
+  const canTagWidget = showContributorTags && !shouldSkipDeletedEmptyParagraphDecoration;
   const decorations = [];
   const replacementNode = newDoc.nodeAt(change.fromB);
   const firstReplacedNode = slice.content.firstChild;

package/dist/es2019/pm-plugins/utils/isEmptyParagraphSlice.js

@@ -0,0 +1,5 @@
+/** Whether a slice contains exactly one structurally empty paragraph. */
+export const isEmptyParagraphSlice = slice => {
+  var _slice$content$firstC;
+  return (slice === null || slice === void 0 ? void 0 : slice.content.childCount) === 1 && ((_slice$content$firstC = slice.content.firstChild) === null || _slice$content$firstC === void 0 ? void 0 : _slice$content$firstC.type.name) === 'paragraph' && slice.content.firstChild.content.size === 0;
+};

package/dist/es2019/ui/IndicatorBar/IndicatorBar.js

@@ -1,4 +1,4 @@
-/* IndicatorBar.tsx generated by @compiled/babel-plugin v3.0.2 */
+/* IndicatorBar.tsx generated by @compiled/babel-plugin v3.0.1 */
 import "./IndicatorBar.compiled.css";
 import * as React from 'react';
 import { ax, ix } from "@compiled/react/runtime";

package/dist/es2019/ui/IndicatorBar/IndicatorBarContentComponent.js

@@ -1,4 +1,4 @@
-/* IndicatorBarContentComponent.tsx generated by @compiled/babel-plugin v3.0.2 */
+/* IndicatorBarContentComponent.tsx generated by @compiled/babel-plugin v3.0.1 */
 import { ax, ix } from "@compiled/react/runtime";
 import React from 'react';
 import { useSharedPluginStateWithSelector } from '@atlaskit/editor-common/hooks';

package/dist/esm/pm-plugins/decorations/colorSchemes/attributions.js

@@ -19,15 +19,40 @@ import { PARTICIPANT_COLOR_SCHEMES } from './types';
 
 /** Maps the shared 18-slot telepointer palette onto the ten bolder diff colours. */
 var HASHED_PARTICIPANT_COLOR_SCHEMES = [].concat(_toConsumableArray(PARTICIPANT_COLOR_SCHEMES), ['blue', 'red', 'orange', 'yellow', 'green', 'teal', 'purple', 'magenta']);
+
+/**
+ * Red means "deleted" in both public schemes, so it never identifies an actor. Reserving it here
+ * covers every path into an attribution colour; `createAttributionColorMap` is the only caller, and
+ * it is gated on `confluence_ncs_step_diffing_version_history` plus
+ * `platform_editor_show_diff_color_scheme_refactor`.
+ */
+var RESERVED_ATTRIBUTION_COLORS = new Set(['red']);
+var isAssignable = function isAssignable(color) {
+  return !RESERVED_ATTRIBUTION_COLORS.has(color);
+};
+
+/** Projects a telepointer slot onto its diff hue, advancing past a hue reserved for deletion. */
+var getSlotColor = function getSlotColor(index) {
+  for (var offset = 0; offset < HASHED_PARTICIPANT_COLOR_SCHEMES.length; offset++) {
+    var color = HASHED_PARTICIPANT_COLOR_SCHEMES[(index + offset) % HASHED_PARTICIPANT_COLOR_SCHEMES.length];
+    if (color && isAssignable(color)) {
+      return color;
+    }
+  }
+  return undefined;
+};
 var getAvailableColor = function getAvailableColor(hashedColor, allocatedColors) {
+  var _PARTICIPANT_COLOR_SC;
   var startIndex = PARTICIPANT_COLOR_SCHEMES.indexOf(hashedColor);
   for (var offset = 0; offset < PARTICIPANT_COLOR_SCHEMES.length; offset++) {
     var color = PARTICIPANT_COLOR_SCHEMES[(startIndex + offset) % PARTICIPANT_COLOR_SCHEMES.length];
-    if (color && !allocatedColors.has(color)) {
+    if (color && isAssignable(color) && !allocatedColors.has(color)) {
       return color;
     }
   }
-  return hashedColor;
+
+  // Every assignable hue is taken, so a duplicate is unavoidable — but never the reserved hue.
+  return isAssignable(hashedColor) ? hashedColor : (_PARTICIPANT_COLOR_SC = PARTICIPANT_COLOR_SCHEMES.find(isAssignable)) !== null && _PARTICIPANT_COLOR_SC !== void 0 ? _PARTICIPANT_COLOR_SC : hashedColor;
 };
 
 /**
@@ -127,7 +152,8 @@ export var createAttributionColorMap = function createAttributionColorMap(stepAt
   var colors = new Map();
   var allocatedColors = new Set();
 
-  // Reserve shared brand colours before hashing other actors, regardless of step order.
+  // Reserve shared brand colours before hashing other actors, regardless of step order. A brand
+  // slot is pinned deliberately, so it is taken verbatim rather than through `getSlotColor`.
   var _iterator2 = _createForOfIteratorHelper(stepAttributions),
     _step2;
   try {
@@ -162,7 +188,7 @@ export var createAttributionColorMap = function createAttributionColorMap(stepAt
       }
       var _getParticipantColor2 = getParticipantColor(_identity.colorSeed, _attribution === null || _attribution === void 0 ? void 0 : _attribution.agentType),
         _index = _getParticipantColor2.index;
-      var hashedColor = HASHED_PARTICIPANT_COLOR_SCHEMES[_index];
+      var hashedColor = getSlotColor(_index);
       if (!hashedColor) {
         continue;
       }

package/dist/esm/pm-plugins/decorations/createChangedRowDecorationWidgets.js

@@ -10,12 +10,14 @@ import { findParentNodeClosestToPos } from '@atlaskit/editor-prosemirror/utils';
 import { Decoration } from '@atlaskit/editor-prosemirror/view';
 import { TableMap } from '@atlaskit/editor-tables/table-map';
 import { isExperimentEnabled } from '@atlaskit/platform-feature-experiments/is-experiment-enabled';
+import { fg } from '@atlaskit/platform-feature-flags/fg';
 import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 import { isExtendedEnabled } from '../isExtendedEnabled';
 import { buildAddedCellOverlayRoundedStyle, buildAddedCellOverlayStyle, buildDeletedCellOverlayRoundedStyle, buildDeletedCellOverlayStyle, buildDeletedRowStyle } from './colorSchemes/factory';
 import { colorSchemeRegistry, getLegacyColorScheme } from './colorSchemes/schemes';
+import { createLeftAnchorWidget } from './createAnchorDecorationWidgets';
 import { resolveCellOverlayStyleLegacy, resolveDeletedRowStyleLegacy } from './createChangedRowDecorationWidgets.styles.legacy';
-import { buildDiffDecorationSpec } from './decorationKeys';
+import { buildAnchorDecorationKey, buildDiffDecorationSpec } from './decorationKeys';
 import { findSafeInsertPos } from './utils/findSafeInsertPos';
 import { applyCellEdgeAttrs, getRowCellEdgeAttrs, getTableTopAndBottomCellEdgeAttrs } from './utils/tableCellEdgeAttrs';
 /**
@@ -47,8 +49,14 @@ var extractChangedRows = function extractChangedRows(_ref) {
     return changedRows;
   }
   var newTableMap = TableMap.get(tableNew.node);
+  // A row DELETION makes the table shorter. An in-place whole-row REPLACEMENT (`replaceNode` on a
+  // `tableRow`, which AI suggested edits emits) leaves the row count untouched — and this helper
+  // was written for deletions only, so it used to discard every replacement and the changed row's
+  // content was never rendered anywhere. The reviewer was left with just the removed side.
+  var isRowDeletion = oldTableMap.height > newTableMap.height;
+  var handleRowReplacement = oldTableMap.height === newTableMap.height && fg('platform_editor_ai_show_diff_patch_1');
   // If no rows were changed, return empty
-  if (oldTableMap.height <= newTableMap.height ||
+  if (!(isRowDeletion || handleRowReplacement) ||
   // For now ignore if there are column deletions as well
   oldTableMap.width !== newTableMap.width) {
     return changedRows;
@@ -92,6 +100,15 @@ var extractChangedRows = function extractChangedRows(_ref) {
 
   // Filter changes that never truly got deleted
   return changedRows.filter(function (changedRow) {
+    // A replacement leaves every sibling row in place, so asking "does this row still exist
+    // ANYWHERE in the new table?" can drop a genuinely rewritten row that happens to match a
+    // sibling. Compare it against the row at the same index instead. Deletions must keep the
+    // whole-table `some()`: their indices shift, so an index comparison would let every
+    // surviving row through and draw a widget for each one.
+    if (handleRowReplacement) {
+      var counterpart = tableNew.node.maybeChild(changedRow.rowIndex);
+      return !counterpart || !areNodesEqualIgnoreAttrs(counterpart, changedRow.rowNode);
+    }
     return !tableNew.node.children.some(function (newRow) {
       return areNodesEqualIgnoreAttrs(newRow, changedRow.rowNode);
     });
@@ -135,7 +152,10 @@ var createChangedRowDOM = function createChangedRowDOM(rowNode, cellEdgeAttrs, n
   if (!isExtendedEnabled(diffType) || !isInserted) {
     tr.setAttribute('style', isExperimentEnabled('platform_editor_show_diff_color_scheme_refactor') ? buildDeletedRowStyle(colors) : resolveDeletedRowStyleLegacy(getLegacyColorScheme(colorScheme)));
   }
-  tr.setAttribute('data-testid', 'show-diff-deleted-row');
+  // Mirrors the strikethrough condition above: under the extended experience an `isInserted` row
+  // is ADDED content, so it must not claim the "deleted" testid — page models match that as
+  // removed content (same reasoning as `createTableCellContentWidgets`).
+  tr.setAttribute('data-testid', isExtendedEnabled(diffType) && isInserted && fg('platform_editor_ai_show_diff_patch_1') ? 'show-diff-changed-row' : 'show-diff-deleted-row');
 
   // Serialize each cell in the row
   var cellIndex = 0;
@@ -215,7 +235,9 @@ export var createChangedRowDecorationWidgets = function createChangedRowDecorati
     colorScheme = _ref3.colorScheme,
     _ref3$isInserted = _ref3.isInserted,
     isInserted = _ref3$isInserted === void 0 ? false : _ref3$isInserted,
-    diffType = _ref3.diffType;
+    diffType = _ref3.diffType,
+    _ref3$showIndicators = _ref3.showIndicators,
+    showIndicators = _ref3$showIndicators === void 0 ? false : _ref3$showIndicators;
   // First, expand the changes to include complete deleted rows
   var changedRows = expandDiffForChangedRows({
     changes: changes.filter(function (change) {
@@ -225,7 +247,7 @@ export var createChangedRowDecorationWidgets = function createChangedRowDecorati
     newDoc: newDoc,
     diffType: diffType
   });
-  return changedRows.map(function (changedRow) {
+  return changedRows.flatMap(function (changedRow) {
     var rowDOM = createChangedRowDOM(changedRow.rowNode, changedRow.cellEdgeAttrs, nodeViewSerializer, colorScheme, isInserted, diffType);
 
     // Find safe insertion position for the deleted row
@@ -233,13 +255,37 @@ export var createChangedRowDecorationWidgets = function createChangedRowDecorati
     // -1 to find the first safe position from the table
     originalDoc.slice(changedRow.fromA, changedRow.toA));
     var diffId = crypto.randomUUID();
-    return Decoration.widget(safeInsertPos, rowDOM, _objectSpread({}, buildDiffDecorationSpec({
+    var decorations = [];
+
+    // `IndicatorBarContentComponent` renders a bar for every widget descriptor and anchors its
+    // top and bottom to `anchor-<diffId>`. Without an element carrying that anchor name the bar
+    // is still created but can never resolve a position, so it silently does not render and the
+    // bar covers only the row being changed. `createNodeChangedDecorationWidget` sets the same
+    // property on its own widget DOM for non-table content; this path returns before reaching it.
+    if (showIndicators && isExtendedEnabled(diffType) && fg('platform_editor_ai_show_diff_patch_1')) {
+      rowDOM.style.setProperty('anchor-name', "--".concat(buildAnchorDecorationKey({
+        diffId: diffId
+      })));
+
+      // A table's content can extend past the doc margin, so the bar also needs a left anchor
+      // measured against the table itself or it is clipped once the table is resized.
+      var leftAnchor = createLeftAnchorWidget({
+        doc: newDoc,
+        from: safeInsertPos,
+        diffId: diffId
+      });
+      if (leftAnchor) {
+        decorations.push(leftAnchor);
+      }
+    }
+    decorations.push(Decoration.widget(safeInsertPos, rowDOM, _objectSpread({}, buildDiffDecorationSpec({
       colorScheme: colorScheme,
       decorationType: 'widget',
       diffId: diffId,
       isActive: false,
       isInserted: isInserted,
       diffType: diffType
-    })));
+    }))));
+    return decorations;
   });
 };

package/dist/esm/pm-plugins/decorations/createInlineChangedDecoration.js

@@ -7,7 +7,7 @@ import { Decoration } from '@atlaskit/editor-prosemirror/view';
 import { isExperimentEnabled } from '@atlaskit/platform-feature-experiments/is-experiment-enabled';
 import { CONTRIBUTOR_TAG_Z_INDEX } from '../../ui/ContributorTag/buildContributorTagDom';
 import { isExtendedEnabled } from '../isExtendedEnabled';
-import { hasVisibleContent } from '../utils/hasVisibleContent';
+import { isEmptyParagraphSlice } from '../utils/isEmptyParagraphSlice';
 import { buildAtomicInlineChangedCSSVariables, buildDeletedInlineContentStyleExtended, buildDeletedInlineStyle, buildDeletedInlineStyleStandard, buildInsertStyle, buildInsertStyleActive, buildInsertStyleExtended, buildInsertStyleExtendedActive, buildInsertStyleExtendedNoUnderline, buildInsertStyleExtendedNoUnderlineActive } from './colorSchemes/factory';
 import { colorSchemeRegistry, getLegacyColorScheme } from './colorSchemes/schemes';
 import { createInlineIndicatorAnchorWidgets } from './createAnchorDecorationWidgets';
@@ -156,11 +156,9 @@ export var createInlineChangedDecoration = function createInlineChangedDecoratio
   // tag host's key, remounting every tag mid-step. Random ids are kept when tags are off.
   var diffId = showContributorTags ? "inline-".concat(change.fromB, "-").concat(change.toB) : crypto.randomUUID();
 
-  // Whether this highlight hosts a tag. A range of nothing but whitespace has nothing to caption,
-  // so it stays unattributed: the spec is the only route attribution takes to
-  // `extractContributorTags`, so dropping the key there is what keeps a tag from being resolved for
-  // a host that is never emitted (EDITOR-8855).
-  var canTagChange = showContributorTags && !!doc && isExtendedEnabled(diffType) && isContributorTagWidgetEnabled() && hasVisibleContent(doc.slice(change.fromB, change.toB).content);
+  // Match the rendered shape: whitespace text is eligible, while an empty paragraph is structural
+  // content with no tag host.
+  var canTagChange = showContributorTags && !!doc && isExtendedEnabled(diffType) && isContributorTagWidgetEnabled() && !isEmptyParagraphSlice(doc.slice(change.fromB, change.toB));
   if (shouldHideDeleted) {
     return [Decoration.inline(change.fromB, change.toB, {
       style: displayNoneStyle

package/dist/esm/pm-plugins/decorations/createNodeChangedDecorationWidget.js

@@ -4,7 +4,7 @@ function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t =
 import { Decoration } from '@atlaskit/editor-prosemirror/view';
 import { fg } from '@atlaskit/platform-feature-flags/fg';
 import { isExtendedEnabled } from '../isExtendedEnabled';
-import { hasVisibleContent } from '../utils/hasVisibleContent';
+import { isEmptyParagraphSlice } from '../utils/isEmptyParagraphSlice';
 import { createLeftAnchorWidget } from './createAnchorDecorationWidgets';
 import { createChangedRowDecorationWidgets } from './createChangedRowDecorationWidgets';
 import { createContributorTagHost, unmountContributorTag } from './createContributorTagWidget';
@@ -94,7 +94,7 @@ var createTableCellContentWidgets = function createTableCellContentWidgets(_ref)
  * that is not in the current document.
  */
 export var createNodeChangedDecorationWidget = function createNodeChangedDecorationWidget(_ref2) {
-  var _slice$content, _slice$content2, _slice$content3, _$safeInsertPos$nodeB, _slice$content$firstC, _newDoc$firstChild, _slice$content$firstC2;
+  var _$safeInsertPos$nodeB, _slice$content$firstC, _newDoc$firstChild, _slice$content$firstC2;
   var attributionKey = _ref2.attributionKey,
     change = _ref2.change,
     doc = _ref2.doc,
@@ -117,7 +117,7 @@ export var createNodeChangedDecorationWidget = function createNodeChangedDecorat
     reveal = _ref2.reveal,
     tagMountContext = _ref2.tagMountContext;
   var slice = doc.slice(change.fromA, change.toA);
-  var shouldSkipDeletedEmptyParagraphDecoration = !isInserted && (slice === null || slice === void 0 || (_slice$content = slice.content) === null || _slice$content === void 0 ? void 0 : _slice$content.childCount) === 1 && (slice === null || slice === void 0 || (_slice$content2 = slice.content) === null || _slice$content2 === void 0 || (_slice$content2 = _slice$content2.firstChild) === null || _slice$content2 === void 0 ? void 0 : _slice$content2.type.name) === 'paragraph' && (slice === null || slice === void 0 || (_slice$content3 = slice.content) === null || _slice$content3 === void 0 || (_slice$content3 = _slice$content3.firstChild) === null || _slice$content3 === void 0 ? void 0 : _slice$content3.content.size) === 0;
+  var shouldSkipDeletedEmptyParagraphDecoration = !isInserted && isEmptyParagraphSlice(slice);
   // Widget decoration used for deletions as the content is not in the document
   // and we want to display the deleted content with a style.
   // For `placeBelow`, anchor at the END of the new content (change.toB) so the deleted
@@ -165,7 +165,10 @@ export var createNodeChangedDecorationWidget = function createNodeChangedDecorat
       nodeViewSerializer: nodeViewSerializer,
       colorScheme: colorScheme,
       isInserted: isInserted,
-      diffType: diffType
+      diffType: diffType,
+      // Needed for the row's own indicator anchor; this path returns before the
+      // `anchor-name` assignment further down.
+      showIndicators: showIndicators
     });
   }
   var serializer = nodeViewSerializer;
@@ -216,10 +219,9 @@ export var createNodeChangedDecorationWidget = function createNodeChangedDecorat
   // Derived from the deleted range so it survives a recalculation — see the same reasoning in
   // `createInlineChangedDecoration`.
   var diffId = showContributorTags ? "widget-".concat(change.fromA, "-").concat(change.toA) : crypto.randomUUID();
-  // Whether this widget hosts a tag. Removing nothing but whitespace leaves no visible deleted
-  // content to caption, so the change stays unattributed — matching the inline half's rule in
-  // `createInlineChangedDecoration` (EDITOR-8855).
-  var canTagWidget = showContributorTags && hasVisibleContent(slice.content);
+  // Match the rendering predicate above: an empty paragraph has no widget to tag, while whitespace
+  // text in a rendered widget remains eligible.
+  var canTagWidget = showContributorTags && !shouldSkipDeletedEmptyParagraphDecoration;
   var decorations = [];
   var replacementNode = newDoc.nodeAt(change.fromB);
   var firstReplacedNode = slice.content.firstChild;

package/dist/esm/pm-plugins/utils/isEmptyParagraphSlice.js

@@ -0,0 +1,5 @@
+/** Whether a slice contains exactly one structurally empty paragraph. */
+export var isEmptyParagraphSlice = function isEmptyParagraphSlice(slice) {
+  var _slice$content$firstC;
+  return (slice === null || slice === void 0 ? void 0 : slice.content.childCount) === 1 && ((_slice$content$firstC = slice.content.firstChild) === null || _slice$content$firstC === void 0 ? void 0 : _slice$content$firstC.type.name) === 'paragraph' && slice.content.firstChild.content.size === 0;
+};

package/dist/esm/ui/IndicatorBar/IndicatorBar.js

@@ -1,4 +1,4 @@
-/* IndicatorBar.tsx generated by @compiled/babel-plugin v3.0.2 */
+/* IndicatorBar.tsx generated by @compiled/babel-plugin v3.0.1 */
 import "./IndicatorBar.compiled.css";
 import * as React from 'react';
 import { ax, ix } from "@compiled/react/runtime";

package/dist/esm/ui/IndicatorBar/IndicatorBarContentComponent.js

@@ -1,4 +1,4 @@
-/* IndicatorBarContentComponent.tsx generated by @compiled/babel-plugin v3.0.2 */
+/* IndicatorBarContentComponent.tsx generated by @compiled/babel-plugin v3.0.1 */
 import { ax, ix } from "@compiled/react/runtime";
 import React from 'react';
 import { useSharedPluginStateWithSelector } from '@atlaskit/editor-common/hooks';

package/dist/types/pm-plugins/decorations/createChangedRowDecorationWidgets.d.ts

@@ -8,7 +8,7 @@ type SimpleChange = Pick<Change, 'fromA' | 'toA' | 'fromB' | 'deleted'>;
 /**
  * Main function to handle deleted rows - computes diff and creates decorations
  */
-export declare const createChangedRowDecorationWidgets: ({ changes, originalDoc, newDoc, nodeViewSerializer, colorScheme, isInserted, diffType, }: {
+export declare const createChangedRowDecorationWidgets: ({ changes, originalDoc, newDoc, nodeViewSerializer, colorScheme, isInserted, diffType, showIndicators, }: {
     changes: SimpleChange[];
     colorScheme?: ColorScheme;
     diffType?: DiffType;
@@ -16,5 +16,6 @@ export declare const createChangedRowDecorationWidgets: ({ changes, originalDoc,
     newDoc: PMNode;
     nodeViewSerializer: NodeViewSerializer;
     originalDoc: PMNode;
+    showIndicators?: boolean;
 }) => Decoration[];
 export {};

package/dist/types/pm-plugins/utils/isEmptyParagraphSlice.d.ts

@@ -0,0 +1,3 @@
+import type { Slice } from '@atlaskit/editor-prosemirror/model';
+/** Whether a slice contains exactly one structurally empty paragraph. */
+export declare const isEmptyParagraphSlice: (slice: Slice | undefined) => boolean;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-show-diff",
-	"version": "16.0.12",
+	"version": "16.0.14",
 	"description": "ShowDiff plugin for @atlaskit/editor-core",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -64,7 +64,7 @@
 		"react-intl": "^7.0.0"
 	},
 	"peerDependencies": {
-		"@atlaskit/editor-common": "^122.7.0",
+		"@atlaskit/editor-common": "^122.8.0",
 		"react": "^18.2.0 || ^19.2.0",
 		"react-dom": "^18.2.0 || ^19.2.0",
 		"react-intl": "^5.25.1 || ^6.0.0 || ^7.0.0"

package/dist/cjs/pm-plugins/utils/hasVisibleContent.js

@@ -1,37 +0,0 @@
-"use strict";
-
-var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.hasVisibleContent = void 0;
-var _toConsumableArray2 = _interopRequireDefault(require("@babel/runtime/helpers/toConsumableArray"));
-var _charsByOffset = require("./charsByOffset");
-/**
- * Whether `fragment` holds anything a reader can see.
- *
- * Text counts once it carries a single non-whitespace character; a leaf node (mention, emoji, date,
- * media, rule, hardBreak, …) counts in its own right, exactly as `resolveTagAnchorPos` treats one.
- * Everything else is descended into, so a paragraph is visible when its content is.
- *
- * A change whose content is nothing but whitespace — a space typed between two words, a run of
- * trailing spaces, an added blank paragraph — has no visible content for a contributor tag to
- * caption, so it carries no tag (EDITOR-8855).
- */
-var _hasVisibleContent = exports.hasVisibleContent = function hasVisibleContent(fragment) {
-  var visible = false;
-  fragment.forEach(function (node) {
-    if (visible) {
-      return;
-    }
-    if (node.isText) {
-      var _node$text;
-      visible = (0, _toConsumableArray2.default)((_node$text = node.text) !== null && _node$text !== void 0 ? _node$text : '').some(function (char) {
-        return !(0, _charsByOffset.isWhitespaceChar)(char);
-      });
-      return;
-    }
-    visible = node.isLeaf || _hasVisibleContent(node.content);
-  });
-  return visible;
-};

package/dist/es2019/pm-plugins/utils/hasVisibleContent.js

@@ -1,28 +0,0 @@
-import { isWhitespaceChar } from './charsByOffset';
-
-/**
- * Whether `fragment` holds anything a reader can see.
- *
- * Text counts once it carries a single non-whitespace character; a leaf node (mention, emoji, date,
- * media, rule, hardBreak, …) counts in its own right, exactly as `resolveTagAnchorPos` treats one.
- * Everything else is descended into, so a paragraph is visible when its content is.
- *
- * A change whose content is nothing but whitespace — a space typed between two words, a run of
- * trailing spaces, an added blank paragraph — has no visible content for a contributor tag to
- * caption, so it carries no tag (EDITOR-8855).
- */
-export const hasVisibleContent = fragment => {
-  let visible = false;
-  fragment.forEach(node => {
-    if (visible) {
-      return;
-    }
-    if (node.isText) {
-      var _node$text;
-      visible = [...((_node$text = node.text) !== null && _node$text !== void 0 ? _node$text : '')].some(char => !isWhitespaceChar(char));
-      return;
-    }
-    visible = node.isLeaf || hasVisibleContent(node.content);
-  });
-  return visible;
-};

package/dist/esm/pm-plugins/utils/hasVisibleContent.js

@@ -1,32 +0,0 @@
-import _toConsumableArray from "@babel/runtime/helpers/toConsumableArray";
-import { isWhitespaceChar } from './charsByOffset';
-
-/**
- * Whether `fragment` holds anything a reader can see.
- *
- * Text counts once it carries a single non-whitespace character; a leaf node (mention, emoji, date,
- * media, rule, hardBreak, …) counts in its own right, exactly as `resolveTagAnchorPos` treats one.
- * Everything else is descended into, so a paragraph is visible when its content is.
- *
- * A change whose content is nothing but whitespace — a space typed between two words, a run of
- * trailing spaces, an added blank paragraph — has no visible content for a contributor tag to
- * caption, so it carries no tag (EDITOR-8855).
- */
-var _hasVisibleContent = function hasVisibleContent(fragment) {
-  var visible = false;
-  fragment.forEach(function (node) {
-    if (visible) {
-      return;
-    }
-    if (node.isText) {
-      var _node$text;
-      visible = _toConsumableArray((_node$text = node.text) !== null && _node$text !== void 0 ? _node$text : '').some(function (char) {
-        return !isWhitespaceChar(char);
-      });
-      return;
-    }
-    visible = node.isLeaf || _hasVisibleContent(node.content);
-  });
-  return visible;
-};
-export { _hasVisibleContent as hasVisibleContent };

package/dist/types/pm-plugins/utils/hasVisibleContent.d.ts

@@ -1,13 +0,0 @@
-import type { Fragment } from '@atlaskit/editor-prosemirror/model';
-/**
- * Whether `fragment` holds anything a reader can see.
- *
- * Text counts once it carries a single non-whitespace character; a leaf node (mention, emoji, date,
- * media, rule, hardBreak, …) counts in its own right, exactly as `resolveTagAnchorPos` treats one.
- * Everything else is descended into, so a paragraph is visible when its content is.
- *
- * A change whose content is nothing but whitespace — a space typed between two words, a run of
- * trailing spaces, an added blank paragraph — has no visible content for a contributor tag to
- * caption, so it carries no tag (EDITOR-8855).
- */
-export declare const hasVisibleContent: (fragment: Fragment) => boolean;