@atlaskit/editor-plugin-show-diff 19.0.0 → 19.0.2

package/CHANGELOG.md

@@ -1,5 +1,31 @@
 # @atlaskit/editor-plugin-show-diff
 
+## 19.0.2
+
+### Patch Changes
+
+- [`202069bab1e36`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/202069bab1e36) -
+  Clean up the shipped `platform_editor_enghealth_a11y_jan_fixes` experiment. Toolbar dropdown items
+  now always render with `role="menuitem"` and without `aria-pressed`, the block menu always has
+  `role="menu"`, and show-diff deleted content always uses the a11y-fixed styles.
+- [`5a9dc1baf2bde`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/5a9dc1baf2bde) -
+  EDITOR-9232 Clamp out-of-range suggestion slice override measurements behind
+  platform_editor_ai_show_diff_patch_2
+- [`8e881a291b8c9`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/8e881a291b8c9) -
+  Keep existing parent list markers unhighlighted for nested additions behind
+  platform_editor_ai_review_moment.
+- Updated dependencies
+
+## 19.0.1
+
+### Patch Changes
+
+- [`27d5e74498554`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/27d5e74498554) -
+  Clean up feature gate `platform_editor_reduce_diff_attr_sensitivity`
+- [`6fc3ffdbe2bf3`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/6fc3ffdbe2bf3) -
+  Use Rovo's registered brand colour in its fixed participant palette slot.
+- Updated dependencies
+
 ## 19.0.0
 
 ### Minor Changes

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

@@ -253,6 +253,9 @@ var calculateNodesForBlockDecoration = function calculateNodesForBlockDecoration
   // table/row/paragraph decorations there are redundant and cause expensive per-cell
   // re-render. Blocks outside the table keep their normal decorations.
   var coarseTables = coarseTableCellsOnly ? tableRanges(doc, from, to) : [];
+  // nodesBetween visits ancestors that start before the changed range.
+  // Only fully contained blocks should mark existing list bullets as new.
+  var requireContainedBlocks = (0, _unsafeExpValNoExposure.UNSAFE_expValNoExposure)('platform_editor_ai_review_moment', 'isEnabled', false) === true;
   // Iterate over the document nodes within the range
   doc.nodesBetween(from, to, function (node, pos) {
     if (coarseTableCellsOnly && isInsideTable(coarseTables, pos)) {
@@ -261,7 +264,7 @@ var calculateNodesForBlockDecoration = function calculateNodesForBlockDecoration
         return;
       }
     }
-    if (node.isBlock && (!(0, _isExtendedEnabled.isExtendedEnabled)(diffType) || pos + node.nodeSize <= to)) {
+    if (node.isBlock && (!(0, _isExtendedEnabled.isExtendedEnabled)(diffType) || (!requireContainedBlocks || pos >= from) && pos + node.nodeSize <= to)) {
       var _nodeEnd = pos + node.nodeSize;
       var _isActive = isRangeActive(activeIndexPos, pos, _nodeEnd);
       decorations.push.apply(decorations, (0, _toConsumableArray2.default)((0, _createBlockChangedDecoration.createBlockChangedDecoration)({
@@ -419,14 +422,14 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
         steppedDoc = result.doc;
       }
     }
-    // Gate the complete attribution fix: mapped positions and step-time nodes are used together.
-    // The fallback deliberately preserves the legacy raw-position lookups for rollout safety.
+    // Mapped positions and step-time nodes are used together so an attr change is attributed to the
+    // node it actually happened on, even when later steps move it.
   } catch (err) {
     _iterator2.e(err);
   } finally {
     _iterator2.f();
   }
-  var attrStepContexts = (0, _fg.fg)('platform_editor_reduce_diff_attr_sensitivity') ? pendingAttrSteps.flatMap(function (_ref1) {
+  var attrStepContexts = pendingAttrSteps.flatMap(function (_ref1) {
     var afterNode = _ref1.afterNode,
       attributionKey = _ref1.attributionKey,
       beforeNode = _ref1.beforeNode,
@@ -447,17 +450,6 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
     }), {}, {
       step: step
     })];
-  }) : pendingAttrSteps.map(function (_ref10) {
-    var attributionKey = _ref10.attributionKey,
-      step = _ref10.step;
-    return {
-      attributionKey: attributionKey,
-      afterNode: tr.doc.nodeAt(step.pos),
-      beforeNode: originalDoc.nodeAt(step.pos),
-      finalPos: step.pos,
-      originalPos: step.pos,
-      step: step
-    };
   });
 
   // Rather than using .eq() we use a custom function that only checks for structural
@@ -800,13 +792,13 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
   // smart path. Only add the legacy step-range decoration when no equivalent changeset range was
   // produced; otherwise the same content receives two inline decorations (and two navigation
   // changes), usually one attribution colour and one default purple decoration.
-  (0, _getMarkChangeRanges.getMarkChangeRanges)(simplifiedSteps, simplifiedStepAttributions, (0, _fg.fg)('confluence_ncs_step_diffing_version_history') ? stepMaps : undefined).filter(function (_ref11) {
+  (0, _getMarkChangeRanges.getMarkChangeRanges)(simplifiedSteps, simplifiedStepAttributions, (0, _fg.fg)('confluence_ncs_step_diffing_version_history') ? stepMaps : undefined).filter(function (_ref10) {
+    var fromB = _ref10.fromB,
+      toB = _ref10.toB;
+    return !isHandledByTable(fromB, toB);
+  }).filter(function (_ref11) {
     var fromB = _ref11.fromB,
       toB = _ref11.toB;
-    return !isHandledByTable(fromB, toB);
-  }).filter(function (_ref12) {
-    var fromB = _ref12.fromB,
-      toB = _ref12.toB;
     return !(0, _fg.fg)('confluence_ncs_step_diffing_version_history') || !changes.some(function (change) {
       return change.fromB === fromB && change.toB === toB;
     });
@@ -828,13 +820,13 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
       tagMountContext: tagMountContext
     })));
   });
-  (0, _getAttrChangeRanges.getAttrChangeRanges)(tr.doc, attrStepContexts, originalDoc).filter(function (_ref13) {
+  (0, _getAttrChangeRanges.getAttrChangeRanges)(tr.doc, attrStepContexts, originalDoc).filter(function (_ref12) {
+    var fromB = _ref12.fromB,
+      toB = _ref12.toB;
+    return !isHandledByTable(fromB, toB);
+  }).filter(function (_ref13) {
     var fromB = _ref13.fromB,
       toB = _ref13.toB;
-    return !isHandledByTable(fromB, toB);
-  }).filter(function (_ref14) {
-    var fromB = _ref14.fromB,
-      toB = _ref14.toB;
     return !(0, _fg.fg)('confluence_ncs_step_diffing_version_history') || !changes.some(function (change) {
       return change.fromB === fromB && change.toB === toB;
     });
@@ -944,49 +936,49 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
 var calculateDiffDecorations = exports.calculateDiffDecorations = (0, _memoizeOne.default)(calculateDiffDecorationsInner,
 // Cache results unless relevant inputs change. `tagMountContext` is deliberately absent: it is
 // built once per plugin instance, so it can never be the reason a diff needs recalculating.
-function (_ref15, _ref16) {
-  var _ref20;
+function (_ref14, _ref15) {
+  var _ref19;
+  var _ref16 = (0, _slicedToArray2.default)(_ref14, 1),
+    _ref16$ = _ref16[0],
+    pluginState = _ref16$.pluginState,
+    state = _ref16$.state,
+    colorScheme = _ref16$.colorScheme,
+    intl = _ref16$.intl,
+    activeIndexPos = _ref16$.activeIndexPos,
+    isInverted = _ref16$.isInverted,
+    diffType = _ref16$.diffType,
+    hideDeletedDiffs = _ref16$.hideDeletedDiffs,
+    hideAddedDiffsUnderline = _ref16$.hideAddedDiffsUnderline,
+    showIndicators = _ref16$.showIndicators,
+    smartThresholds = _ref16$.smartThresholds,
+    deletedDiffPlacement = _ref16$.deletedDiffPlacement,
+    inlineDeletedDiffPlacement = _ref16$.inlineDeletedDiffPlacement,
+    reveal = _ref16$.reveal;
   var _ref17 = (0, _slicedToArray2.default)(_ref15, 1),
     _ref17$ = _ref17[0],
-    pluginState = _ref17$.pluginState,
-    state = _ref17$.state,
-    colorScheme = _ref17$.colorScheme,
-    intl = _ref17$.intl,
-    activeIndexPos = _ref17$.activeIndexPos,
-    isInverted = _ref17$.isInverted,
-    diffType = _ref17$.diffType,
-    hideDeletedDiffs = _ref17$.hideDeletedDiffs,
-    hideAddedDiffsUnderline = _ref17$.hideAddedDiffsUnderline,
-    showIndicators = _ref17$.showIndicators,
-    smartThresholds = _ref17$.smartThresholds,
-    deletedDiffPlacement = _ref17$.deletedDiffPlacement,
-    inlineDeletedDiffPlacement = _ref17$.inlineDeletedDiffPlacement,
-    reveal = _ref17$.reveal;
-  var _ref18 = (0, _slicedToArray2.default)(_ref16, 1),
-    _ref18$ = _ref18[0],
-    lastPluginState = _ref18$.pluginState,
-    lastState = _ref18$.state,
-    lastColorScheme = _ref18$.colorScheme,
-    lastIntl = _ref18$.intl,
-    lastActiveIndexPos = _ref18$.activeIndexPos,
-    lastIsInverted = _ref18$.isInverted,
-    lastDiffType = _ref18$.diffType,
-    lastHideDeletedDiffs = _ref18$.hideDeletedDiffs,
-    lastHideAddedDiffsUnderline = _ref18$.hideAddedDiffsUnderline,
-    lastShowIndicators = _ref18$.showIndicators,
-    lastSmartThresholds = _ref18$.smartThresholds,
-    lastDeletedDiffPlacement = _ref18$.deletedDiffPlacement,
-    lastInlineDeletedDiffPlacement = _ref18$.inlineDeletedDiffPlacement,
-    lastReveal = _ref18$.reveal;
+    lastPluginState = _ref17$.pluginState,
+    lastState = _ref17$.state,
+    lastColorScheme = _ref17$.colorScheme,
+    lastIntl = _ref17$.intl,
+    lastActiveIndexPos = _ref17$.activeIndexPos,
+    lastIsInverted = _ref17$.isInverted,
+    lastDiffType = _ref17$.diffType,
+    lastHideDeletedDiffs = _ref17$.hideDeletedDiffs,
+    lastHideAddedDiffsUnderline = _ref17$.hideAddedDiffsUnderline,
+    lastShowIndicators = _ref17$.showIndicators,
+    lastSmartThresholds = _ref17$.smartThresholds,
+    lastDeletedDiffPlacement = _ref17$.deletedDiffPlacement,
+    lastInlineDeletedDiffPlacement = _ref17$.inlineDeletedDiffPlacement,
+    lastReveal = _ref17$.reveal;
   var originalDocIsSame = lastPluginState.originalDoc && pluginState.originalDoc && pluginState.originalDoc.eq(lastPluginState.originalDoc);
   if ((0, _isExtendedEnabled.isExtendedEnabled)(diffType)) {
-    var _ref19;
-    return (_ref19 = colorScheme === lastColorScheme && intl.locale === lastIntl.locale && isInverted === lastIsInverted && diffType === lastDiffType && (0, _isEqual.default)(activeIndexPos, lastActiveIndexPos) && originalDocIsSame && (0, _isEqual.default)(pluginState.steps, lastPluginState.steps) && (0, _isEqual.default)(pluginState.stepAttributions, lastPluginState.stepAttributions) &&
+    var _ref18;
+    return (_ref18 = colorScheme === lastColorScheme && intl.locale === lastIntl.locale && isInverted === lastIsInverted && diffType === lastDiffType && (0, _isEqual.default)(activeIndexPos, lastActiveIndexPos) && originalDocIsSame && (0, _isEqual.default)(pluginState.steps, lastPluginState.steps) && (0, _isEqual.default)(pluginState.stepAttributions, lastPluginState.stepAttributions) &&
     // Contributors can arrive after the first calculation; without this, tags never appear.
     (0, _isEqual.default)(pluginState.contributors, lastPluginState.contributors) && state.doc.eq(lastState.doc) && hideDeletedDiffs === lastHideDeletedDiffs && hideAddedDiffsUnderline === lastHideAddedDiffsUnderline && showIndicators === lastShowIndicators && (0, _isEqual.default)(smartThresholds, lastSmartThresholds) && deletedDiffPlacement === lastDeletedDiffPlacement && inlineDeletedDiffPlacement === lastInlineDeletedDiffPlacement &&
     // Turning the reveal on or off changes the emitted styles, so a cached result from the
     // other setting would leave the diff painted in the wrong resting state.
-    (0, _isEqual.default)(reveal, lastReveal)) !== null && _ref19 !== void 0 ? _ref19 : false;
+    (0, _isEqual.default)(reveal, lastReveal)) !== null && _ref18 !== void 0 ? _ref18 : false;
   }
-  return (_ref20 = originalDocIsSame && (0, _isEqual.default)(pluginState.steps, lastPluginState.steps) && (0, _isEqual.default)(pluginState.stepAttributions, lastPluginState.stepAttributions) && (0, _isEqual.default)(pluginState.contributors, lastPluginState.contributors) && state.doc.eq(lastState.doc) && colorScheme === lastColorScheme && intl.locale === lastIntl.locale && (0, _isEqual.default)(activeIndexPos, lastActiveIndexPos) && hideDeletedDiffs === lastHideDeletedDiffs) !== null && _ref20 !== void 0 ? _ref20 : false;
+  return (_ref19 = originalDocIsSame && (0, _isEqual.default)(pluginState.steps, lastPluginState.steps) && (0, _isEqual.default)(pluginState.stepAttributions, lastPluginState.stepAttributions) && (0, _isEqual.default)(pluginState.contributors, lastPluginState.contributors) && state.doc.eq(lastState.doc) && colorScheme === lastColorScheme && intl.locale === lastIntl.locale && (0, _isEqual.default)(activeIndexPos, lastActiveIndexPos) && hideDeletedDiffs === lastHideDeletedDiffs) !== null && _ref19 !== void 0 ? _ref19 : false;
 });

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

@@ -106,7 +106,6 @@ var getAgentBrandForColor = function getAgentBrandForColor(attribution, contribu
   var agentBrand = contributors === null || contributors === void 0 || (_contributors = contributors[(_getAttributionKey = getAttributionKey(attribution)) !== null && _getAttributionKey !== void 0 ? _getAttributionKey : '']) === null || _contributors === void 0 ? void 0 : _contributors.agentKind;
   return isDiffAgentBrand(agentBrand) ? agentBrand : attribution === null || attribution === void 0 ? void 0 : attribution.agentType;
 };
-
 /**
  * Public contributor list to internal key-addressed record. A contributor whose attribution carries
  * no identity is dropped, since it could never match a step. Later entries win.

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

@@ -1183,8 +1183,7 @@ function buildDeletedInlineNodeCSSVariables(colors, isActive) {
  *
  * The three `stateful`-only variables are omitted for `static` (standard) so editor-core's fallback
  * wins, which is what standard rendered before:
- * - `--diff-delete-opacity` — the fallback is 0.6 or 0.8 depending on
- *   `platform_editor_enghealth_a11y_jan_fixes`, a split the scheme must not flatten.
+ * - `--diff-delete-opacity` — standard falls through to editor-core's 0.8 fallback.
  * - `--diff-delete-ring-width` — traditional's deleted media ring tracks the inherited marker ring
  *   width instead of a fixed 1px.
  * - `--diff-delete-text-decoration-color` — standard's blockquote strike must stay `currentColor`.

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

@@ -131,7 +131,10 @@ var getEdgeCasesForSlice = function getEdgeCasesForSlice(doc, from, slice, measu
   var targets = [];
   // edge cases can only be top level nodes, so we only care about the top level nodes in the slice
   slice.content.forEach(function (node, offset) {
-    var edgeCase = edgeCases(doc, from + offset, node);
+    // the node can be outside the doc if we have a sliceOverride which adds additional new content
+    // but we still need to the edgeCases to resolve a node in the doc to attach the widget to
+    // and we use the provided measureElement to make sure it is in the right position
+    var edgeCase = edgeCases(doc, Math.min(from + offset, doc.content.size - 1), node);
     if (edgeCase) {
       targets.push(_objectSpread(_objectSpread({}, edgeCase), {}, {
         measureElement: measureElement

package/dist/cjs/pm-plugins/decorations/utils/getAttrChangeRanges.js

@@ -11,7 +11,6 @@ var _omit = _interopRequireDefault(require("lodash/omit"));
 var _setAttrs = require("@atlaskit/adf-schema/steps/set-attrs");
 var _nodeTypeUtils = require("@atlaskit/editor-common/utils/node-type-utils");
 var _transform = require("@atlaskit/editor-prosemirror/transform");
-var _fg = require("@atlaskit/platform-feature-flags/fg");
 var _diffableAttrs = require("./diffableAttrs");
 function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
 function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { (0, _defineProperty2.default)(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
@@ -75,10 +74,10 @@ var getAttrChangeRanges = exports.getAttrChangeRanges = function getAttrChangeRa
       return undefined;
     }
     var stepAttrs = getStepAttrs(step);
-    // SetAttrsStep contains the complete replacement attrs, not only the attrs that changed.
-    // Keep the legacy payload-based checks until the rollout gate is enabled, and fall back
-    // to them if either step-time node is unavailable.
-    var attrsToCheck = (0, _fg.fg)('platform_editor_reduce_diff_attr_sensitivity') && beforeNode && afterNode ? stepAttrs.filter(function (attrName) {
+    // SetAttrsStep contains the complete replacement attrs, not only the attrs that changed,
+    // so narrow it to the attrs whose values actually differ. Fall back to the full step
+    // payload if either step-time node is unavailable.
+    var attrsToCheck = beforeNode && afterNode ? stepAttrs.filter(function (attrName) {
       return !(0, _isEqual.default)(beforeNode.attrs[attrName], afterNode.attrs[attrName]);
     }) : stepAttrs;
     var $pos = doc.resolve(finalPos);
@@ -139,12 +138,12 @@ var getAttrChangeRanges = exports.getAttrChangeRanges = function getAttrChangeRa
       });
     }
 
-    // Extension nodes: highlight changes to user-visible configuration. When enabled,
-    // transient extension metadata is ignored by the comparison below.
+    // Extension nodes: highlight changes to user-visible configuration. Transient extension
+    // metadata is ignored by the comparison below.
     if (nodeAtPos && extensionNodeNames.includes(nodeAtPos.type.name) && attrsToCheck.some(function (v) {
       return (0, _diffableAttrs.isDiffableAttr)(nodeAtPos.type.name, v);
     })) {
-      if ((0, _fg.fg)('platform_editor_reduce_diff_attr_sensitivity') && beforeNode && afterNode && beforeNode.type === afterNode.type && haveSameRelevantExtensionAttrs(beforeNode, afterNode)) {
+      if (beforeNode && afterNode && beforeNode.type === afterNode.type && haveSameRelevantExtensionAttrs(beforeNode, afterNode)) {
         return undefined;
       }
       var isInline = nodeAtPos.type.name === 'inlineExtension';

package/dist/cjs/pm-plugins/decorations/utils/wrapBlockNodeViewStyles.js

@@ -8,7 +8,6 @@ exports.resolveRemovedLozengeStyle = exports.resolveNestedInsertedNodeStyle = ex
 var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 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");
@@ -54,7 +53,7 @@ var getChangedContentStyleNext = function getChangedContentStyleNext(colorScheme
   if ((0, _isExtendedEnabled.isExtendedEnabled)(diffType) && isInserted) {
     return (0, _factory.buildInsertedInlineStyle)(colors, isActive, hideAddedDiffsUnderline);
   }
-  var base = (0, _factory.buildDeletedInlineContentStyle)(colors, isActive ? 'active' : (0, _expValEquals.expValEquals)('platform_editor_enghealth_a11y_jan_fixes', 'isEnabled', true) ? 'new' : 'default');
+  var base = (0, _factory.buildDeletedInlineContentStyle)(colors, isActive ? 'active' : 'new');
 
   // Text-like blocks (heading, lists, blockquote) render their deleted content through this
   // getter rather than `getDeletedContentStyle`, so without this they never receive the

package/dist/cjs/pm-plugins/decorations/utils/wrapBlockNodeViewStyles.legacy.js

@@ -6,7 +6,6 @@ Object.defineProperty(exports, "__esModule", {
 exports.resolveRemovedLozengeStyleLegacy = exports.resolveNestedInsertedNodeStyleLegacy = exports.resolveCellOverlayStyleLegacy = exports.hasRestingDeletedRingLegacy = exports.getInsertedContentStyleLegacy = exports.getDeletedContentStyleUnboundedLegacy = exports.getDeletedContentStyleLegacy = exports.getChangedNodeStyleLegacy = exports.getChangedContentStyleLegacy = void 0;
 var _lazyNodeView = require("@atlaskit/editor-common/lazy-node-view");
 var _fg = require("@atlaskit/platform-feature-flags/fg");
-var _expValEquals = require("@atlaskit/tmp-editor-statsig/exp-val-equals");
 var _isExtendedEnabled = require("../../isExtendedEnabled");
 var _standard = require("../colorSchemes/standard");
 var _traditional = require("../colorSchemes/traditional");
@@ -101,7 +100,7 @@ var getChangedContentStyleLegacy = exports.getChangedContentStyleLegacy = functi
   if (isActive) {
     return (0, _standard.getStandardDeletedContentStyleActive)();
   }
-  return (0, _expValEquals.expValEquals)('platform_editor_enghealth_a11y_jan_fixes', 'isEnabled', true) ? (0, _standard.getStandardDeletedContentStyleNew)() : (0, _standard.getStandardDeletedContentStyle)();
+  return (0, _standard.getStandardDeletedContentStyleNew)();
 };
 
 /** Pre-refactor node-specific style for a changed block node. */

package/dist/cjs/ui/ContributorTag/contributorTagController.js

@@ -38,7 +38,12 @@ var chatgptBrandColor = (0, _getThirdPartyAgentColor.getThirdPartyAgentColor)({
 });
 
 /** Brand accents for agent kinds with no ADS token match. New agents opt in by adding an entry. */
-var AGENT_KIND_ACCENT_OVERRIDES = _objectSpread({}, chatgptBrandColor ? {
+var AGENT_KIND_ACCENT_OVERRIDES = _objectSpread({
+  rovo: {
+    background: "var(--ds-background-neutral-bold, #292A2E)",
+    text: "var(--ds-text-inverse, #FFFFFF)"
+  }
+}, chatgptBrandColor ? {
   chatgpt: {
     background: chatgptBrandColor.bold,
     text: chatgptBrandColor.boldText

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

@@ -221,6 +221,9 @@ const calculateNodesForBlockDecoration = ({
   // table/row/paragraph decorations there are redundant and cause expensive per-cell
   // re-render. Blocks outside the table keep their normal decorations.
   const coarseTables = coarseTableCellsOnly ? tableRanges(doc, from, to) : [];
+  // nodesBetween visits ancestors that start before the changed range.
+  // Only fully contained blocks should mark existing list bullets as new.
+  const requireContainedBlocks = UNSAFE_expValNoExposure('platform_editor_ai_review_moment', 'isEnabled', false) === true;
   // Iterate over the document nodes within the range
   doc.nodesBetween(from, to, (node, pos) => {
     if (coarseTableCellsOnly && isInsideTable(coarseTables, pos)) {
@@ -229,7 +232,7 @@ const calculateNodesForBlockDecoration = ({
         return;
       }
     }
-    if (node.isBlock && (!isExtendedEnabled(diffType) || pos + node.nodeSize <= to)) {
+    if (node.isBlock && (!isExtendedEnabled(diffType) || (!requireContainedBlocks || pos >= from) && pos + node.nodeSize <= to)) {
       const nodeEnd = pos + node.nodeSize;
       const isActive = isRangeActive(activeIndexPos, pos, nodeEnd);
       decorations.push(...createBlockChangedDecoration({
@@ -377,9 +380,9 @@ const calculateDiffDecorationsInner = ({
       steppedDoc = result.doc;
     }
   }
-  // Gate the complete attribution fix: mapped positions and step-time nodes are used together.
-  // The fallback deliberately preserves the legacy raw-position lookups for rollout safety.
-  const attrStepContexts = fg('platform_editor_reduce_diff_attr_sensitivity') ? pendingAttrSteps.flatMap(({
+  // Mapped positions and step-time nodes are used together so an attr change is attributed to the
+  // node it actually happened on, even when later steps move it.
+  const attrStepContexts = pendingAttrSteps.flatMap(({
     afterNode,
     attributionKey,
     beforeNode,
@@ -401,17 +404,7 @@ const calculateDiffDecorationsInner = ({
       }),
       step
     }];
-  }) : pendingAttrSteps.map(({
-    attributionKey,
-    step
-  }) => ({
-    attributionKey,
-    afterNode: tr.doc.nodeAt(step.pos),
-    beforeNode: originalDoc.nodeAt(step.pos),
-    finalPos: step.pos,
-    originalPos: step.pos,
-    step
-  }));
+  });
 
   // Rather than using .eq() we use a custom function that only checks for structural
   // changes and ignores differences in attributes which don't affect decoration positions

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

@@ -86,7 +86,6 @@ const getAgentBrandForColor = (attribution, contributors) => {
   const agentBrand = contributors === null || contributors === void 0 ? void 0 : (_contributors = contributors[(_getAttributionKey = getAttributionKey(attribution)) !== null && _getAttributionKey !== void 0 ? _getAttributionKey : '']) === null || _contributors === void 0 ? void 0 : _contributors.agentKind;
   return isDiffAgentBrand(agentBrand) ? agentBrand : attribution === null || attribution === void 0 ? void 0 : attribution.agentType;
 };
-
 /**
  * Public contributor list to internal key-addressed record. A contributor whose attribution carries
  * no identity is dropped, since it could never match a step. Later entries win.

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

@@ -1106,8 +1106,7 @@ export function buildDeletedInlineNodeCSSVariables(colors, isActive) {
  *
  * The three `stateful`-only variables are omitted for `static` (standard) so editor-core's fallback
  * wins, which is what standard rendered before:
- * - `--diff-delete-opacity` — the fallback is 0.6 or 0.8 depending on
- *   `platform_editor_enghealth_a11y_jan_fixes`, a split the scheme must not flatten.
+ * - `--diff-delete-opacity` — standard falls through to editor-core's 0.8 fallback.
  * - `--diff-delete-ring-width` — traditional's deleted media ring tracks the inherited marker ring
  *   width instead of a fixed 1px.
  * - `--diff-delete-text-decoration-color` — standard's blockquote strike must stay `currentColor`.

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

@@ -118,7 +118,10 @@ const getEdgeCasesForSlice = (doc, from, slice, measureElement) => {
   const targets = [];
   // edge cases can only be top level nodes, so we only care about the top level nodes in the slice
   slice.content.forEach((node, offset) => {
-    const edgeCase = edgeCases(doc, from + offset, node);
+    // the node can be outside the doc if we have a sliceOverride which adds additional new content
+    // but we still need to the edgeCases to resolve a node in the doc to attach the widget to
+    // and we use the provided measureElement to make sure it is in the right position
+    const edgeCase = edgeCases(doc, Math.min(from + offset, doc.content.size - 1), node);
     if (edgeCase) {
       targets.push({
         ...edgeCase,

package/dist/es2019/pm-plugins/decorations/utils/getAttrChangeRanges.js

@@ -3,7 +3,6 @@ import omit from 'lodash/omit';
 import { SetAttrsStep } from '@atlaskit/adf-schema/steps/set-attrs';
 import { getBaseNodeTypeName } from '@atlaskit/editor-common/utils/node-type-utils';
 import { AttrStep } from '@atlaskit/editor-prosemirror/transform';
-import { fg } from '@atlaskit/platform-feature-flags/fg';
 import { getRequiredIncludedDiffableAttrs, isDiffableAttr } from './diffableAttrs';
 const filterUndefined = x => !!x;
 
@@ -59,10 +58,10 @@ export const getAttrChangeRanges = (doc, attrStepContexts, originalDoc) => {
       return undefined;
     }
     const stepAttrs = getStepAttrs(step);
-    // SetAttrsStep contains the complete replacement attrs, not only the attrs that changed.
-    // Keep the legacy payload-based checks until the rollout gate is enabled, and fall back
-    // to them if either step-time node is unavailable.
-    const attrsToCheck = fg('platform_editor_reduce_diff_attr_sensitivity') && beforeNode && afterNode ? stepAttrs.filter(attrName => !isEqual(beforeNode.attrs[attrName], afterNode.attrs[attrName])) : stepAttrs;
+    // SetAttrsStep contains the complete replacement attrs, not only the attrs that changed,
+    // so narrow it to the attrs whose values actually differ. Fall back to the full step
+    // payload if either step-time node is unavailable.
+    const attrsToCheck = beforeNode && afterNode ? stepAttrs.filter(attrName => !isEqual(beforeNode.attrs[attrName], afterNode.attrs[attrName])) : stepAttrs;
     const $pos = doc.resolve(finalPos);
     const nodeAtPos = doc.nodeAt(finalPos);
     const originalNodeAtPos = originalPos === undefined ? null : originalDoc.nodeAt(originalPos);
@@ -120,10 +119,10 @@ export const getAttrChangeRanges = (doc, attrStepContexts, originalDoc) => {
       };
     }
 
-    // Extension nodes: highlight changes to user-visible configuration. When enabled,
-    // transient extension metadata is ignored by the comparison below.
+    // Extension nodes: highlight changes to user-visible configuration. Transient extension
+    // metadata is ignored by the comparison below.
     if (nodeAtPos && extensionNodeNames.includes(nodeAtPos.type.name) && attrsToCheck.some(v => isDiffableAttr(nodeAtPos.type.name, v))) {
-      if (fg('platform_editor_reduce_diff_attr_sensitivity') && beforeNode && afterNode && beforeNode.type === afterNode.type && haveSameRelevantExtensionAttrs(beforeNode, afterNode)) {
+      if (beforeNode && afterNode && beforeNode.type === afterNode.type && haveSameRelevantExtensionAttrs(beforeNode, afterNode)) {
         return undefined;
       }
       const isInline = nodeAtPos.type.name === 'inlineExtension';

package/dist/es2019/pm-plugins/decorations/utils/wrapBlockNodeViewStyles.js

@@ -12,7 +12,6 @@
  */
 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, buildAddedCellOverlayStyleNew, buildDeletedCellOverlayRoundedStyle, buildDeletedCellOverlayStyle, buildDeletedInlineContentStyle, buildDeletedInlineContentStyleExtended, buildDeletedLozengeActiveStyle, buildDeletedLozengeStyle, buildDeletedNodeCSSVariables, buildDeletedQuoteNodeWithLozengeStyle, buildDeletedStrikethroughLine, buildDeletedWrappedBlockOutline, buildInsertedBlockNodeStyle, buildInsertedInlineStyle, buildInsertStyleInBlockExtended, buildInsertStyleInBlockExtendedNoUnderline, buildInsertStyleNode } from '../colorSchemes/factory';
 import { colorSchemeRegistry, getLegacyColorScheme, standardScheme } from '../colorSchemes/schemes';
@@ -39,7 +38,7 @@ const getChangedContentStyleNext = (colorScheme, isActive = false, isInserted =
   if (isExtendedEnabled(diffType) && isInserted) {
     return buildInsertedInlineStyle(colors, isActive, hideAddedDiffsUnderline);
   }
-  const base = buildDeletedInlineContentStyle(colors, isActive ? 'active' : expValEquals('platform_editor_enghealth_a11y_jan_fixes', 'isEnabled', true) ? 'new' : 'default');
+  const base = buildDeletedInlineContentStyle(colors, isActive ? 'active' : 'new');
 
   // Text-like blocks (heading, lists, blockquote) render their deleted content through this
   // getter rather than `getDeletedContentStyle`, so without this they never receive the

package/dist/es2019/pm-plugins/decorations/utils/wrapBlockNodeViewStyles.legacy.js

@@ -10,9 +10,8 @@
  */
 import { convertToInlineCss } from '@atlaskit/editor-common/lazy-node-view';
 import { fg } from '@atlaskit/platform-feature-flags/fg';
-import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 import { isExtendedEnabled } from '../../isExtendedEnabled';
-import { deletedBlockOutline, deletedBlockOutlineActive, deletedBlockOutlineRounded, deletedBlockOutlineRoundedActive, deletedContentStyleUnbounded, deletedInlineContentStyleExtended, deletedStyleQuoteNodeWithLozenge, deletedStyleQuoteNodeWithLozengeActive, editingContentStyleInBlockExtended, editingContentStyleInBlockExtendedNoUnderline, editingStyleExtended, editingStyleExtendedNoUnderline, editingStyleActiveExtended, editingStyleActiveExtendedNoUnderline, editingStyleNode, getStandardDeletedContentStyle, getStandardDeletedContentStyleActive, getStandardDeletedContentStyleNew, addedCellOverlayStyle, addedCellOverlayRoundedStyle, deletedCellOverlayStyle, deletedCellOverlayRoundedStyle } from '../colorSchemes/standard';
+import { deletedBlockOutline, deletedBlockOutlineActive, deletedBlockOutlineRounded, deletedBlockOutlineRoundedActive, deletedContentStyleUnbounded, deletedInlineContentStyleExtended, deletedStyleQuoteNodeWithLozenge, deletedStyleQuoteNodeWithLozengeActive, editingContentStyleInBlockExtended, editingContentStyleInBlockExtendedNoUnderline, editingStyleExtended, editingStyleExtendedNoUnderline, editingStyleActiveExtended, editingStyleActiveExtendedNoUnderline, editingStyleNode, getStandardDeletedContentStyleActive, getStandardDeletedContentStyleNew, addedCellOverlayStyle, addedCellOverlayRoundedStyle, deletedCellOverlayStyle, deletedCellOverlayRoundedStyle } from '../colorSchemes/standard';
 import { deletedTraditionalBlockOutlineActive, deletedTraditionalBlockOutlineNew, deletedTraditionalBlockOutlineRoundedActive, deletedTraditionalBlockOutlineRoundedNew, deletedTraditionalContentStyleUnbounded, deletedTraditionalContentStyleUnboundedActive, getDeletedTraditionalInlineStyle, deletedTraditionalStyleQuoteNode, deletedTraditionalStyleQuoteNodeActive, traditionalInsertStyle, traditionalInsertStyleActive, traditionalStyleNodeActive, traditionalStyleNodeNew, traditionalAddedCellOverlayRoundedStyle, traditionalAddedCellOverlayStyleNew, deletedTraditionalCellOverlayStyle, deletedTraditionalCellOverlayRoundedStyle } from '../colorSchemes/traditional';
 const lozengeStyle = convertToInlineCss({
   display: 'inline-flex',
@@ -90,7 +89,7 @@ export const getChangedContentStyleLegacy = (colorScheme, isActive = false, isIn
   if (isActive) {
     return getStandardDeletedContentStyleActive();
   }
-  return expValEquals('platform_editor_enghealth_a11y_jan_fixes', 'isEnabled', true) ? getStandardDeletedContentStyleNew() : getStandardDeletedContentStyle();
+  return getStandardDeletedContentStyleNew();
 };
 
 /** Pre-refactor node-specific style for a changed block node. */

package/dist/es2019/ui/ContributorTag/contributorTagController.js

@@ -28,6 +28,10 @@ const chatgptBrandColor = getThirdPartyAgentColor({
 
 /** Brand accents for agent kinds with no ADS token match. New agents opt in by adding an entry. */
 const AGENT_KIND_ACCENT_OVERRIDES = {
+  rovo: {
+    background: "var(--ds-background-neutral-bold, #292A2E)",
+    text: "var(--ds-text-inverse, #FFFFFF)"
+  },
   ...(chatgptBrandColor ? {
     chatgpt: {
       background: chatgptBrandColor.bold,

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

@@ -246,6 +246,9 @@ var calculateNodesForBlockDecoration = function calculateNodesForBlockDecoration
   // table/row/paragraph decorations there are redundant and cause expensive per-cell
   // re-render. Blocks outside the table keep their normal decorations.
   var coarseTables = coarseTableCellsOnly ? tableRanges(doc, from, to) : [];
+  // nodesBetween visits ancestors that start before the changed range.
+  // Only fully contained blocks should mark existing list bullets as new.
+  var requireContainedBlocks = UNSAFE_expValNoExposure('platform_editor_ai_review_moment', 'isEnabled', false) === true;
   // Iterate over the document nodes within the range
   doc.nodesBetween(from, to, function (node, pos) {
     if (coarseTableCellsOnly && isInsideTable(coarseTables, pos)) {
@@ -254,7 +257,7 @@ var calculateNodesForBlockDecoration = function calculateNodesForBlockDecoration
         return;
       }
     }
-    if (node.isBlock && (!isExtendedEnabled(diffType) || pos + node.nodeSize <= to)) {
+    if (node.isBlock && (!isExtendedEnabled(diffType) || (!requireContainedBlocks || pos >= from) && pos + node.nodeSize <= to)) {
       var _nodeEnd = pos + node.nodeSize;
       var _isActive = isRangeActive(activeIndexPos, pos, _nodeEnd);
       decorations.push.apply(decorations, _toConsumableArray(createBlockChangedDecoration({
@@ -412,14 +415,14 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
         steppedDoc = result.doc;
       }
     }
-    // Gate the complete attribution fix: mapped positions and step-time nodes are used together.
-    // The fallback deliberately preserves the legacy raw-position lookups for rollout safety.
+    // Mapped positions and step-time nodes are used together so an attr change is attributed to the
+    // node it actually happened on, even when later steps move it.
   } catch (err) {
     _iterator2.e(err);
   } finally {
     _iterator2.f();
   }
-  var attrStepContexts = fg('platform_editor_reduce_diff_attr_sensitivity') ? pendingAttrSteps.flatMap(function (_ref1) {
+  var attrStepContexts = pendingAttrSteps.flatMap(function (_ref1) {
     var afterNode = _ref1.afterNode,
       attributionKey = _ref1.attributionKey,
       beforeNode = _ref1.beforeNode,
@@ -440,17 +443,6 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
     }), {}, {
       step: step
     })];
-  }) : pendingAttrSteps.map(function (_ref10) {
-    var attributionKey = _ref10.attributionKey,
-      step = _ref10.step;
-    return {
-      attributionKey: attributionKey,
-      afterNode: tr.doc.nodeAt(step.pos),
-      beforeNode: originalDoc.nodeAt(step.pos),
-      finalPos: step.pos,
-      originalPos: step.pos,
-      step: step
-    };
   });
 
   // Rather than using .eq() we use a custom function that only checks for structural
@@ -793,13 +785,13 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
   // smart path. Only add the legacy step-range decoration when no equivalent changeset range was
   // produced; otherwise the same content receives two inline decorations (and two navigation
   // changes), usually one attribution colour and one default purple decoration.
-  getMarkChangeRanges(simplifiedSteps, simplifiedStepAttributions, fg('confluence_ncs_step_diffing_version_history') ? stepMaps : undefined).filter(function (_ref11) {
+  getMarkChangeRanges(simplifiedSteps, simplifiedStepAttributions, fg('confluence_ncs_step_diffing_version_history') ? stepMaps : undefined).filter(function (_ref10) {
+    var fromB = _ref10.fromB,
+      toB = _ref10.toB;
+    return !isHandledByTable(fromB, toB);
+  }).filter(function (_ref11) {
     var fromB = _ref11.fromB,
       toB = _ref11.toB;
-    return !isHandledByTable(fromB, toB);
-  }).filter(function (_ref12) {
-    var fromB = _ref12.fromB,
-      toB = _ref12.toB;
     return !fg('confluence_ncs_step_diffing_version_history') || !changes.some(function (change) {
       return change.fromB === fromB && change.toB === toB;
     });
@@ -821,13 +813,13 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
       tagMountContext: tagMountContext
     })));
   });
-  getAttrChangeRanges(tr.doc, attrStepContexts, originalDoc).filter(function (_ref13) {
+  getAttrChangeRanges(tr.doc, attrStepContexts, originalDoc).filter(function (_ref12) {
+    var fromB = _ref12.fromB,
+      toB = _ref12.toB;
+    return !isHandledByTable(fromB, toB);
+  }).filter(function (_ref13) {
     var fromB = _ref13.fromB,
       toB = _ref13.toB;
-    return !isHandledByTable(fromB, toB);
-  }).filter(function (_ref14) {
-    var fromB = _ref14.fromB,
-      toB = _ref14.toB;
     return !fg('confluence_ncs_step_diffing_version_history') || !changes.some(function (change) {
       return change.fromB === fromB && change.toB === toB;
     });
@@ -937,49 +929,49 @@ var calculateDiffDecorationsInner = function calculateDiffDecorationsInner(_ref8
 export var calculateDiffDecorations = memoizeOne(calculateDiffDecorationsInner,
 // Cache results unless relevant inputs change. `tagMountContext` is deliberately absent: it is
 // built once per plugin instance, so it can never be the reason a diff needs recalculating.
-function (_ref15, _ref16) {
-  var _ref20;
+function (_ref14, _ref15) {
+  var _ref19;
+  var _ref16 = _slicedToArray(_ref14, 1),
+    _ref16$ = _ref16[0],
+    pluginState = _ref16$.pluginState,
+    state = _ref16$.state,
+    colorScheme = _ref16$.colorScheme,
+    intl = _ref16$.intl,
+    activeIndexPos = _ref16$.activeIndexPos,
+    isInverted = _ref16$.isInverted,
+    diffType = _ref16$.diffType,
+    hideDeletedDiffs = _ref16$.hideDeletedDiffs,
+    hideAddedDiffsUnderline = _ref16$.hideAddedDiffsUnderline,
+    showIndicators = _ref16$.showIndicators,
+    smartThresholds = _ref16$.smartThresholds,
+    deletedDiffPlacement = _ref16$.deletedDiffPlacement,
+    inlineDeletedDiffPlacement = _ref16$.inlineDeletedDiffPlacement,
+    reveal = _ref16$.reveal;
   var _ref17 = _slicedToArray(_ref15, 1),
     _ref17$ = _ref17[0],
-    pluginState = _ref17$.pluginState,
-    state = _ref17$.state,
-    colorScheme = _ref17$.colorScheme,
-    intl = _ref17$.intl,
-    activeIndexPos = _ref17$.activeIndexPos,
-    isInverted = _ref17$.isInverted,
-    diffType = _ref17$.diffType,
-    hideDeletedDiffs = _ref17$.hideDeletedDiffs,
-    hideAddedDiffsUnderline = _ref17$.hideAddedDiffsUnderline,
-    showIndicators = _ref17$.showIndicators,
-    smartThresholds = _ref17$.smartThresholds,
-    deletedDiffPlacement = _ref17$.deletedDiffPlacement,
-    inlineDeletedDiffPlacement = _ref17$.inlineDeletedDiffPlacement,
-    reveal = _ref17$.reveal;
-  var _ref18 = _slicedToArray(_ref16, 1),
-    _ref18$ = _ref18[0],
-    lastPluginState = _ref18$.pluginState,
-    lastState = _ref18$.state,
-    lastColorScheme = _ref18$.colorScheme,
-    lastIntl = _ref18$.intl,
-    lastActiveIndexPos = _ref18$.activeIndexPos,
-    lastIsInverted = _ref18$.isInverted,
-    lastDiffType = _ref18$.diffType,
-    lastHideDeletedDiffs = _ref18$.hideDeletedDiffs,
-    lastHideAddedDiffsUnderline = _ref18$.hideAddedDiffsUnderline,
-    lastShowIndicators = _ref18$.showIndicators,
-    lastSmartThresholds = _ref18$.smartThresholds,
-    lastDeletedDiffPlacement = _ref18$.deletedDiffPlacement,
-    lastInlineDeletedDiffPlacement = _ref18$.inlineDeletedDiffPlacement,
-    lastReveal = _ref18$.reveal;
+    lastPluginState = _ref17$.pluginState,
+    lastState = _ref17$.state,
+    lastColorScheme = _ref17$.colorScheme,
+    lastIntl = _ref17$.intl,
+    lastActiveIndexPos = _ref17$.activeIndexPos,
+    lastIsInverted = _ref17$.isInverted,
+    lastDiffType = _ref17$.diffType,
+    lastHideDeletedDiffs = _ref17$.hideDeletedDiffs,
+    lastHideAddedDiffsUnderline = _ref17$.hideAddedDiffsUnderline,
+    lastShowIndicators = _ref17$.showIndicators,
+    lastSmartThresholds = _ref17$.smartThresholds,
+    lastDeletedDiffPlacement = _ref17$.deletedDiffPlacement,
+    lastInlineDeletedDiffPlacement = _ref17$.inlineDeletedDiffPlacement,
+    lastReveal = _ref17$.reveal;
   var originalDocIsSame = lastPluginState.originalDoc && pluginState.originalDoc && pluginState.originalDoc.eq(lastPluginState.originalDoc);
   if (isExtendedEnabled(diffType)) {
-    var _ref19;
-    return (_ref19 = colorScheme === lastColorScheme && intl.locale === lastIntl.locale && isInverted === lastIsInverted && diffType === lastDiffType && isEqual(activeIndexPos, lastActiveIndexPos) && originalDocIsSame && isEqual(pluginState.steps, lastPluginState.steps) && isEqual(pluginState.stepAttributions, lastPluginState.stepAttributions) &&
+    var _ref18;
+    return (_ref18 = colorScheme === lastColorScheme && intl.locale === lastIntl.locale && isInverted === lastIsInverted && diffType === lastDiffType && isEqual(activeIndexPos, lastActiveIndexPos) && originalDocIsSame && isEqual(pluginState.steps, lastPluginState.steps) && isEqual(pluginState.stepAttributions, lastPluginState.stepAttributions) &&
     // Contributors can arrive after the first calculation; without this, tags never appear.
     isEqual(pluginState.contributors, lastPluginState.contributors) && state.doc.eq(lastState.doc) && hideDeletedDiffs === lastHideDeletedDiffs && hideAddedDiffsUnderline === lastHideAddedDiffsUnderline && showIndicators === lastShowIndicators && isEqual(smartThresholds, lastSmartThresholds) && deletedDiffPlacement === lastDeletedDiffPlacement && inlineDeletedDiffPlacement === lastInlineDeletedDiffPlacement &&
     // Turning the reveal on or off changes the emitted styles, so a cached result from the
     // other setting would leave the diff painted in the wrong resting state.
-    isEqual(reveal, lastReveal)) !== null && _ref19 !== void 0 ? _ref19 : false;
+    isEqual(reveal, lastReveal)) !== null && _ref18 !== void 0 ? _ref18 : false;
   }
-  return (_ref20 = originalDocIsSame && isEqual(pluginState.steps, lastPluginState.steps) && isEqual(pluginState.stepAttributions, lastPluginState.stepAttributions) && isEqual(pluginState.contributors, lastPluginState.contributors) && state.doc.eq(lastState.doc) && colorScheme === lastColorScheme && intl.locale === lastIntl.locale && isEqual(activeIndexPos, lastActiveIndexPos) && hideDeletedDiffs === lastHideDeletedDiffs) !== null && _ref20 !== void 0 ? _ref20 : false;
+  return (_ref19 = originalDocIsSame && isEqual(pluginState.steps, lastPluginState.steps) && isEqual(pluginState.stepAttributions, lastPluginState.stepAttributions) && isEqual(pluginState.contributors, lastPluginState.contributors) && state.doc.eq(lastState.doc) && colorScheme === lastColorScheme && intl.locale === lastIntl.locale && isEqual(activeIndexPos, lastActiveIndexPos) && hideDeletedDiffs === lastHideDeletedDiffs) !== null && _ref19 !== void 0 ? _ref19 : false;
 });

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

@@ -100,7 +100,6 @@ var getAgentBrandForColor = function getAgentBrandForColor(attribution, contribu
   var agentBrand = contributors === null || contributors === void 0 || (_contributors = contributors[(_getAttributionKey = getAttributionKey(attribution)) !== null && _getAttributionKey !== void 0 ? _getAttributionKey : '']) === null || _contributors === void 0 ? void 0 : _contributors.agentKind;
   return isDiffAgentBrand(agentBrand) ? agentBrand : attribution === null || attribution === void 0 ? void 0 : attribution.agentType;
 };
-
 /**
  * Public contributor list to internal key-addressed record. A contributor whose attribution carries
  * no identity is dropped, since it could never match a step. Later entries win.

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

@@ -1107,8 +1107,7 @@ export function buildDeletedInlineNodeCSSVariables(colors, isActive) {
  *
  * The three `stateful`-only variables are omitted for `static` (standard) so editor-core's fallback
  * wins, which is what standard rendered before:
- * - `--diff-delete-opacity` — the fallback is 0.6 or 0.8 depending on
- *   `platform_editor_enghealth_a11y_jan_fixes`, a split the scheme must not flatten.
+ * - `--diff-delete-opacity` — standard falls through to editor-core's 0.8 fallback.
  * - `--diff-delete-ring-width` — traditional's deleted media ring tracks the inherited marker ring
  *   width instead of a fixed 1px.
  * - `--diff-delete-text-decoration-color` — standard's blockquote strike must stay `currentColor`.

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

@@ -125,7 +125,10 @@ var getEdgeCasesForSlice = function getEdgeCasesForSlice(doc, from, slice, measu
   var targets = [];
   // edge cases can only be top level nodes, so we only care about the top level nodes in the slice
   slice.content.forEach(function (node, offset) {
-    var edgeCase = edgeCases(doc, from + offset, node);
+    // the node can be outside the doc if we have a sliceOverride which adds additional new content
+    // but we still need to the edgeCases to resolve a node in the doc to attach the widget to
+    // and we use the provided measureElement to make sure it is in the right position
+    var edgeCase = edgeCases(doc, Math.min(from + offset, doc.content.size - 1), node);
     if (edgeCase) {
       targets.push(_objectSpread(_objectSpread({}, edgeCase), {}, {
         measureElement: measureElement

package/dist/esm/pm-plugins/decorations/utils/getAttrChangeRanges.js

@@ -6,7 +6,6 @@ import omit from 'lodash/omit';
 import { SetAttrsStep } from '@atlaskit/adf-schema/steps/set-attrs';
 import { getBaseNodeTypeName } from '@atlaskit/editor-common/utils/node-type-utils';
 import { AttrStep } from '@atlaskit/editor-prosemirror/transform';
-import { fg } from '@atlaskit/platform-feature-flags/fg';
 import { getRequiredIncludedDiffableAttrs, isDiffableAttr } from './diffableAttrs';
 var filterUndefined = function filterUndefined(x) {
   return !!x;
@@ -68,10 +67,10 @@ export var getAttrChangeRanges = function getAttrChangeRanges(doc, attrStepConte
       return undefined;
     }
     var stepAttrs = getStepAttrs(step);
-    // SetAttrsStep contains the complete replacement attrs, not only the attrs that changed.
-    // Keep the legacy payload-based checks until the rollout gate is enabled, and fall back
-    // to them if either step-time node is unavailable.
-    var attrsToCheck = fg('platform_editor_reduce_diff_attr_sensitivity') && beforeNode && afterNode ? stepAttrs.filter(function (attrName) {
+    // SetAttrsStep contains the complete replacement attrs, not only the attrs that changed,
+    // so narrow it to the attrs whose values actually differ. Fall back to the full step
+    // payload if either step-time node is unavailable.
+    var attrsToCheck = beforeNode && afterNode ? stepAttrs.filter(function (attrName) {
       return !isEqual(beforeNode.attrs[attrName], afterNode.attrs[attrName]);
     }) : stepAttrs;
     var $pos = doc.resolve(finalPos);
@@ -132,12 +131,12 @@ export var getAttrChangeRanges = function getAttrChangeRanges(doc, attrStepConte
       });
     }
 
-    // Extension nodes: highlight changes to user-visible configuration. When enabled,
-    // transient extension metadata is ignored by the comparison below.
+    // Extension nodes: highlight changes to user-visible configuration. Transient extension
+    // metadata is ignored by the comparison below.
     if (nodeAtPos && extensionNodeNames.includes(nodeAtPos.type.name) && attrsToCheck.some(function (v) {
       return isDiffableAttr(nodeAtPos.type.name, v);
     })) {
-      if (fg('platform_editor_reduce_diff_attr_sensitivity') && beforeNode && afterNode && beforeNode.type === afterNode.type && haveSameRelevantExtensionAttrs(beforeNode, afterNode)) {
+      if (beforeNode && afterNode && beforeNode.type === afterNode.type && haveSameRelevantExtensionAttrs(beforeNode, afterNode)) {
         return undefined;
       }
       var isInline = nodeAtPos.type.name === 'inlineExtension';

package/dist/esm/pm-plugins/decorations/utils/wrapBlockNodeViewStyles.js

@@ -15,7 +15,6 @@ function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t =
  */
 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, buildAddedCellOverlayStyleNew, buildDeletedCellOverlayRoundedStyle, buildDeletedCellOverlayStyle, buildDeletedInlineContentStyle, buildDeletedInlineContentStyleExtended, buildDeletedLozengeActiveStyle, buildDeletedLozengeStyle, buildDeletedNodeCSSVariables, buildDeletedQuoteNodeWithLozengeStyle, buildDeletedStrikethroughLine, buildDeletedWrappedBlockOutline, buildInsertedBlockNodeStyle, buildInsertedInlineStyle, buildInsertStyleInBlockExtended, buildInsertStyleInBlockExtendedNoUnderline, buildInsertStyleNode } from '../colorSchemes/factory';
 import { colorSchemeRegistry, getLegacyColorScheme, standardScheme } from '../colorSchemes/schemes';
@@ -48,7 +47,7 @@ var getChangedContentStyleNext = function getChangedContentStyleNext(colorScheme
   if (isExtendedEnabled(diffType) && isInserted) {
     return buildInsertedInlineStyle(colors, isActive, hideAddedDiffsUnderline);
   }
-  var base = buildDeletedInlineContentStyle(colors, isActive ? 'active' : expValEquals('platform_editor_enghealth_a11y_jan_fixes', 'isEnabled', true) ? 'new' : 'default');
+  var base = buildDeletedInlineContentStyle(colors, isActive ? 'active' : 'new');
 
   // Text-like blocks (heading, lists, blockquote) render their deleted content through this
   // getter rather than `getDeletedContentStyle`, so without this they never receive the

package/dist/esm/pm-plugins/decorations/utils/wrapBlockNodeViewStyles.legacy.js

@@ -10,9 +10,8 @@
  */
 import { convertToInlineCss } from '@atlaskit/editor-common/lazy-node-view';
 import { fg } from '@atlaskit/platform-feature-flags/fg';
-import { expValEquals } from '@atlaskit/tmp-editor-statsig/exp-val-equals';
 import { isExtendedEnabled } from '../../isExtendedEnabled';
-import { deletedBlockOutline, deletedBlockOutlineActive, deletedBlockOutlineRounded, deletedBlockOutlineRoundedActive, deletedContentStyleUnbounded, deletedInlineContentStyleExtended, deletedStyleQuoteNodeWithLozenge, deletedStyleQuoteNodeWithLozengeActive, editingContentStyleInBlockExtended, editingContentStyleInBlockExtendedNoUnderline, editingStyleExtended, editingStyleExtendedNoUnderline, editingStyleActiveExtended, editingStyleActiveExtendedNoUnderline, editingStyleNode, getStandardDeletedContentStyle, getStandardDeletedContentStyleActive, getStandardDeletedContentStyleNew, addedCellOverlayStyle, addedCellOverlayRoundedStyle, deletedCellOverlayStyle, deletedCellOverlayRoundedStyle } from '../colorSchemes/standard';
+import { deletedBlockOutline, deletedBlockOutlineActive, deletedBlockOutlineRounded, deletedBlockOutlineRoundedActive, deletedContentStyleUnbounded, deletedInlineContentStyleExtended, deletedStyleQuoteNodeWithLozenge, deletedStyleQuoteNodeWithLozengeActive, editingContentStyleInBlockExtended, editingContentStyleInBlockExtendedNoUnderline, editingStyleExtended, editingStyleExtendedNoUnderline, editingStyleActiveExtended, editingStyleActiveExtendedNoUnderline, editingStyleNode, getStandardDeletedContentStyleActive, getStandardDeletedContentStyleNew, addedCellOverlayStyle, addedCellOverlayRoundedStyle, deletedCellOverlayStyle, deletedCellOverlayRoundedStyle } from '../colorSchemes/standard';
 import { deletedTraditionalBlockOutlineActive, deletedTraditionalBlockOutlineNew, deletedTraditionalBlockOutlineRoundedActive, deletedTraditionalBlockOutlineRoundedNew, deletedTraditionalContentStyleUnbounded, deletedTraditionalContentStyleUnboundedActive, getDeletedTraditionalInlineStyle, deletedTraditionalStyleQuoteNode, deletedTraditionalStyleQuoteNodeActive, traditionalInsertStyle, traditionalInsertStyleActive, traditionalStyleNodeActive, traditionalStyleNodeNew, traditionalAddedCellOverlayRoundedStyle, traditionalAddedCellOverlayStyleNew, deletedTraditionalCellOverlayStyle, deletedTraditionalCellOverlayRoundedStyle } from '../colorSchemes/traditional';
 var lozengeStyle = convertToInlineCss({
   display: 'inline-flex',
@@ -94,7 +93,7 @@ export var getChangedContentStyleLegacy = function getChangedContentStyleLegacy(
   if (isActive) {
     return getStandardDeletedContentStyleActive();
   }
-  return expValEquals('platform_editor_enghealth_a11y_jan_fixes', 'isEnabled', true) ? getStandardDeletedContentStyleNew() : getStandardDeletedContentStyle();
+  return getStandardDeletedContentStyleNew();
 };
 
 /** Pre-refactor node-specific style for a changed block node. */

package/dist/esm/ui/ContributorTag/contributorTagController.js

@@ -31,7 +31,12 @@ var chatgptBrandColor = getThirdPartyAgentColor({
 });
 
 /** Brand accents for agent kinds with no ADS token match. New agents opt in by adding an entry. */
-var AGENT_KIND_ACCENT_OVERRIDES = _objectSpread({}, chatgptBrandColor ? {
+var AGENT_KIND_ACCENT_OVERRIDES = _objectSpread({
+  rovo: {
+    background: "var(--ds-background-neutral-bold, #292A2E)",
+    text: "var(--ds-text-inverse, #FFFFFF)"
+  }
+}, chatgptBrandColor ? {
   chatgpt: {
     background: chatgptBrandColor.bold,
     text: chatgptBrandColor.boldText

package/dist/types/pm-plugins/decorations/colorSchemes/factory.d.ts

@@ -238,8 +238,7 @@ export declare function buildDeletedInlineNodeCSSVariables(colors: DiffColorSche
  *
  * The three `stateful`-only variables are omitted for `static` (standard) so editor-core's fallback
  * wins, which is what standard rendered before:
- * - `--diff-delete-opacity` — the fallback is 0.6 or 0.8 depending on
- *   `platform_editor_enghealth_a11y_jan_fixes`, a split the scheme must not flatten.
+ * - `--diff-delete-opacity` — standard falls through to editor-core's 0.8 fallback.
  * - `--diff-delete-ring-width` — traditional's deleted media ring tracks the inherited marker ring
  *   width instead of a fixed 1px.
  * - `--diff-delete-text-decoration-color` — standard's blockquote strike must stay `currentColor`.

package/dist/types/pm-plugins/decorations/utils/getAttrChangeRanges.d.ts

@@ -5,8 +5,8 @@ import type { Step as ProseMirrorStep } from '@atlaskit/editor-prosemirror/trans
 export type InlineAttrChangeNodeName = 'date' | 'emoji' | 'mention' | 'status';
 type AttrChangeStep = AttrStep | SetAttrsStep;
 export type AttrStepContext = {
-    attributionKey?: string;
     afterNode: PMNode | null;
+    attributionKey?: string;
     beforeNode: PMNode | null;
     finalPos: number;
     originalPos?: number;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-show-diff",
-	"version": "19.0.0",
+	"version": "19.0.2",
 	"description": "ShowDiff plugin for @atlaskit/editor-core",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -39,7 +39,7 @@
 		"@atlaskit/platform-feature-experiments": "^0.3.0",
 		"@atlaskit/platform-feature-flags": "^2.2.0",
 		"@atlaskit/primitives": "^22.5.0",
-		"@atlaskit/tmp-editor-statsig": "^205.0.0",
+		"@atlaskit/tmp-editor-statsig": "^207.0.0",
 		"@atlaskit/tokens": "^20.0.0",
 		"@babel/runtime": "^7.0.0",
 		"@compiled/react": "^1.0.2",
@@ -64,14 +64,14 @@
 		"@atlassian/confluence-presets": "workspace:^",
 		"@atlassian/content-reconciliation": "^0.1.3506",
 		"@atlassian/editor-ai-injected-editors": "^35.0.0",
-		"@atlassian/editor-plugin-ai": "^75.0.0",
+		"@atlassian/editor-plugin-ai": "^75.1.0",
 		"@atlassian/structured-docs-types": "workspace:^",
 		"react": "^19.2.0",
 		"react-dom": "^19.2.0",
 		"react-intl": "^7.0.0"
 	},
 	"peerDependencies": {
-		"@atlaskit/editor-common": "^125.0.0",
+		"@atlaskit/editor-common": "^125.2.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"
@@ -99,7 +99,7 @@
 		"confluence_ncs_step_diffing_version_history": {
 			"type": "boolean"
 		},
-		"platform_editor_reduce_diff_attr_sensitivity": {
+		"platform_editor_diff_inline_mark_changes": {
 			"type": "boolean"
 		}
 	}

package/tsconfig.json

@@ -1,5 +1,5 @@
 {
-	"extends": "./tsconfig.dev.json",
+	"extends": "./tsconfig.app.json",
 	"compilerOptions": {
 		"disableReferencedProjectLoad": true,
 		"disableSolutionSearching": true