@atlaskit/editor-plugin-local-id 1.0.2 → 1.0.4

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @atlaskit/editor-plugin-local-id
 
+## 1.0.4
+
+### Patch Changes
+
+- [`84d49ebfb038b`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/84d49ebfb038b) -
+  EDITOR-1426 Updated addLocalIdToNode to use setNodeAttribute which creates an attr step instead of
+  a replace step as localId attribute changes should not result in a document change.
+- Updated dependencies
+
+## 1.0.3
+
+### Patch Changes
+
+- [`655a927604c69`](https://bitbucket.org/atlassian/atlassian-frontend-monorepo/commits/655a927604c69) -
+  Ignore media group node for local id as it is not in the schema for now
+
 ## 1.0.2
 
 ### Patch Changes

package/afm-cc/tsconfig.json

@@ -2,9 +2,9 @@
 	"extends": "../../../../tsconfig.entry-points.confluence.json",
 	"compilerOptions": {
 		"target": "es5",
-		"composite": true,
 		"outDir": "../../../../../confluence/tsDist/@atlaskit__editor-plugin-local-id",
-		"rootDir": "../"
+		"rootDir": "../",
+		"composite": true
 	},
 	"include": [
 		"../src/**/*.ts",
@@ -24,4 +24,4 @@
 			"path": "../../editor-common/afm-cc/tsconfig.json"
 		}
 	]
-}
+}

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

@@ -1,18 +1,14 @@
 "use strict";
 
-var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
 exports.localIdPluginKey = exports.createPlugin = exports.addLocalIdToNode = void 0;
-var _defineProperty2 = _interopRequireDefault(require("@babel/runtime/helpers/defineProperty"));
 var _adfSchema = require("@atlaskit/adf-schema");
 var _safePlugin = require("@atlaskit/editor-common/safe-plugin");
 var _utils = require("@atlaskit/editor-common/utils");
 var _state = require("@atlaskit/editor-prosemirror/state");
 var _platformFeatureFlags = require("@atlaskit/platform-feature-flags");
-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; }
 function _createForOfIteratorHelper(r, e) { var t = "undefined" != typeof Symbol && r[Symbol.iterator] || r["@@iterator"]; if (!t) { if (Array.isArray(r) || (t = _unsupportedIterableToArray(r)) || e && r && "number" == typeof r.length) { t && (r = t); var _n = 0, F = function F() {}; return { s: F, n: function n() { return _n >= r.length ? { done: !0 } : { done: !1, value: r[_n++] }; }, e: function e(r) { throw r; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var o, a = !0, u = !1; return { s: function s() { t = t.call(r); }, n: function n() { var r = t.next(); return a = r.done, r; }, e: function e(r) { u = !0, o = r; }, f: function f() { try { a || null == t.return || t.return(); } finally { if (u) throw o; } } }; }
 function _unsupportedIterableToArray(r, a) { if (r) { if ("string" == typeof r) return _arrayLikeToArray(r, a); var t = {}.toString.call(r).slice(8, -1); return "Object" === t && r.constructor && (t = r.constructor.name), "Map" === t || "Set" === t ? Array.from(r) : "Arguments" === t || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(t) ? _arrayLikeToArray(r, a) : void 0; } }
 function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length); for (var e = 0, n = Array(a); e < a; e++) n[e] = r[e]; return n; }
@@ -41,13 +37,16 @@ var createPlugin = exports.createPlugin = function createPlugin() {
         var localIdWasAdded = false;
         var _editorView$state$sch = editorView.state.schema.nodes,
           text = _editorView$state$sch.text,
-          hardBreak = _editorView$state$sch.hardBreak;
-        var ignoredNodeTypes = [text.name, hardBreak.name];
+          hardBreak = _editorView$state$sch.hardBreak,
+          mediaGroup = _editorView$state$sch.mediaGroup;
+        // Media group is ignored for now
+        // https://bitbucket.org/atlassian/adf-schema/src/fb2236147a0c2bc9c8efbdb75fd8f8c411df44ba/packages/adf-schema/src/next-schema/nodes/mediaGroup.ts#lines-12
+        var ignoredNodeTypes = mediaGroup ? [text.name, hardBreak.name, mediaGroup.name] : [text.name, hardBreak.name];
         editorView.state.doc.descendants(function (node, pos) {
           // Skip text nodes, hard breaks and nodes that already have local IDs
           if (!ignoredNodeTypes.includes(node.type.name) && !node.attrs.localId) {
             localIdWasAdded = true;
-            addLocalIdToNode(node, pos, tr);
+            addLocalIdToNode(pos, tr);
           }
           return true; // Continue traversing
         });
@@ -71,11 +70,15 @@ var createPlugin = exports.createPlugin = function createPlugin() {
       var tr = newState.tr;
       var _newState$schema$node = newState.schema.nodes,
         text = _newState$schema$node.text,
-        hardBreak = _newState$schema$node.hardBreak;
-      var ignoredNodeTypes = [text === null || text === void 0 ? void 0 : text.name, hardBreak === null || hardBreak === void 0 ? void 0 : hardBreak.name];
+        hardBreak = _newState$schema$node.hardBreak,
+        mediaGroup = _newState$schema$node.mediaGroup;
+      // Media group is ignored for now
+      // https://bitbucket.org/atlassian/adf-schema/src/fb2236147a0c2bc9c8efbdb75fd8f8c411df44ba/packages/adf-schema/src/next-schema/nodes/mediaGroup.ts#lines-12
+      var ignoredNodeTypes = [text === null || text === void 0 ? void 0 : text.name, hardBreak === null || hardBreak === void 0 ? void 0 : hardBreak.name, mediaGroup === null || mediaGroup === void 0 ? void 0 : mediaGroup.name];
       var addedNodes = new Set();
       var addedNodePos = new Map();
       var localIds = new Set();
+
       // Process only the nodes added in the transactions
       transactions.forEach(function (transaction) {
         if (!transaction.docChanged) {
@@ -101,7 +104,7 @@ var createPlugin = exports.createPlugin = function createPlugin() {
                 addedNodePos.set(node, pos);
               } else {
                 if (!(node !== null && node !== void 0 && node.attrs.localId)) {
-                  addLocalIdToNode(node, pos, tr);
+                  addLocalIdToNode(pos, tr);
                 }
               }
               return true;
@@ -125,7 +128,7 @@ var createPlugin = exports.createPlugin = function createPlugin() {
             if (!node.attrs.localId || localIds.has(node.attrs.localId)) {
               var pos = addedNodePos.get(node);
               if (pos !== undefined) {
-                addLocalIdToNode(node, pos, tr);
+                addLocalIdToNode(pos, tr);
                 modified = true;
               }
             }
@@ -144,15 +147,10 @@ var createPlugin = exports.createPlugin = function createPlugin() {
  * Adds a local ID to a ProseMirror node
  *
  * This utility function updates a node's attributes to include a unique local ID.
- * It preserves all existing attributes and marks while adding the new localId.
  *
- * @param node - The ProseMirror node to add a local ID to
  * @param pos - The position of the node in the document
  * @param tr - The transaction to apply the change to
- * @returns The updated transaction with the node markup change
  */
-var addLocalIdToNode = exports.addLocalIdToNode = function addLocalIdToNode(node, pos, tr) {
-  tr.setNodeMarkup(pos, node.type, _objectSpread(_objectSpread({}, node.attrs), {}, {
-    localId: _adfSchema.uuid.generate()
-  }), node.marks);
+var addLocalIdToNode = exports.addLocalIdToNode = function addLocalIdToNode(pos, tr) {
+  tr.setNodeAttribute(pos, 'localId', _adfSchema.uuid.generate());
 };

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

@@ -28,14 +28,17 @@ export const createPlugin = () => {
         let localIdWasAdded = false;
         const {
           text,
-          hardBreak
+          hardBreak,
+          mediaGroup
         } = editorView.state.schema.nodes;
-        const ignoredNodeTypes = [text.name, hardBreak.name];
+        // Media group is ignored for now
+        // https://bitbucket.org/atlassian/adf-schema/src/fb2236147a0c2bc9c8efbdb75fd8f8c411df44ba/packages/adf-schema/src/next-schema/nodes/mediaGroup.ts#lines-12
+        const ignoredNodeTypes = mediaGroup ? [text.name, hardBreak.name, mediaGroup.name] : [text.name, hardBreak.name];
         editorView.state.doc.descendants((node, pos) => {
           // Skip text nodes, hard breaks and nodes that already have local IDs
           if (!ignoredNodeTypes.includes(node.type.name) && !node.attrs.localId) {
             localIdWasAdded = true;
-            addLocalIdToNode(node, pos, tr);
+            addLocalIdToNode(pos, tr);
           }
           return true; // Continue traversing
         });
@@ -59,12 +62,16 @@ export const createPlugin = () => {
       const tr = newState.tr;
       const {
         text,
-        hardBreak
+        hardBreak,
+        mediaGroup
       } = newState.schema.nodes;
-      const ignoredNodeTypes = [text === null || text === void 0 ? void 0 : text.name, hardBreak === null || hardBreak === void 0 ? void 0 : hardBreak.name];
+      // Media group is ignored for now
+      // https://bitbucket.org/atlassian/adf-schema/src/fb2236147a0c2bc9c8efbdb75fd8f8c411df44ba/packages/adf-schema/src/next-schema/nodes/mediaGroup.ts#lines-12
+      const ignoredNodeTypes = [text === null || text === void 0 ? void 0 : text.name, hardBreak === null || hardBreak === void 0 ? void 0 : hardBreak.name, mediaGroup === null || mediaGroup === void 0 ? void 0 : mediaGroup.name];
       const addedNodes = new Set();
       const addedNodePos = new Map();
       const localIds = new Set();
+
       // Process only the nodes added in the transactions
       transactions.forEach(transaction => {
         if (!transaction.docChanged) {
@@ -90,7 +97,7 @@ export const createPlugin = () => {
                 addedNodePos.set(node, pos);
               } else {
                 if (!(node !== null && node !== void 0 && node.attrs.localId)) {
-                  addLocalIdToNode(node, pos, tr);
+                  addLocalIdToNode(pos, tr);
                 }
               }
               return true;
@@ -110,7 +117,7 @@ export const createPlugin = () => {
           if (!node.attrs.localId || localIds.has(node.attrs.localId)) {
             const pos = addedNodePos.get(node);
             if (pos !== undefined) {
-              addLocalIdToNode(node, pos, tr);
+              addLocalIdToNode(pos, tr);
               modified = true;
             }
           }
@@ -124,16 +131,10 @@ export const createPlugin = () => {
  * Adds a local ID to a ProseMirror node
  *
  * This utility function updates a node's attributes to include a unique local ID.
- * It preserves all existing attributes and marks while adding the new localId.
  *
- * @param node - The ProseMirror node to add a local ID to
  * @param pos - The position of the node in the document
  * @param tr - The transaction to apply the change to
- * @returns The updated transaction with the node markup change
  */
-export const addLocalIdToNode = (node, pos, tr) => {
-  tr.setNodeMarkup(pos, node.type, {
-    ...node.attrs,
-    localId: uuid.generate()
-  }, node.marks);
+export const addLocalIdToNode = (pos, tr) => {
+  tr.setNodeAttribute(pos, 'localId', uuid.generate());
 };

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

@@ -1,6 +1,3 @@
-import _defineProperty from "@babel/runtime/helpers/defineProperty";
-function ownKeys(e, r) { var t = Object.keys(e); if (Object.getOwnPropertySymbols) { var o = Object.getOwnPropertySymbols(e); r && (o = o.filter(function (r) { return Object.getOwnPropertyDescriptor(e, r).enumerable; })), t.push.apply(t, o); } return t; }
-function _objectSpread(e) { for (var r = 1; r < arguments.length; r++) { var t = null != arguments[r] ? arguments[r] : {}; r % 2 ? ownKeys(Object(t), !0).forEach(function (r) { _defineProperty(e, r, t[r]); }) : Object.getOwnPropertyDescriptors ? Object.defineProperties(e, Object.getOwnPropertyDescriptors(t)) : ownKeys(Object(t)).forEach(function (r) { Object.defineProperty(e, r, Object.getOwnPropertyDescriptor(t, r)); }); } return e; }
 function _createForOfIteratorHelper(r, e) { var t = "undefined" != typeof Symbol && r[Symbol.iterator] || r["@@iterator"]; if (!t) { if (Array.isArray(r) || (t = _unsupportedIterableToArray(r)) || e && r && "number" == typeof r.length) { t && (r = t); var _n = 0, F = function F() {}; return { s: F, n: function n() { return _n >= r.length ? { done: !0 } : { done: !1, value: r[_n++] }; }, e: function e(r) { throw r; }, f: F }; } throw new TypeError("Invalid attempt to iterate non-iterable instance.\nIn order to be iterable, non-array objects must have a [Symbol.iterator]() method."); } var o, a = !0, u = !1; return { s: function s() { t = t.call(r); }, n: function n() { var r = t.next(); return a = r.done, r; }, e: function e(r) { u = !0, o = r; }, f: function f() { try { a || null == t.return || t.return(); } finally { if (u) throw o; } } }; }
 function _unsupportedIterableToArray(r, a) { if (r) { if ("string" == typeof r) return _arrayLikeToArray(r, a); var t = {}.toString.call(r).slice(8, -1); return "Object" === t && r.constructor && (t = r.constructor.name), "Map" === t || "Set" === t ? Array.from(r) : "Arguments" === t || /^(?:Ui|I)nt(?:8|16|32)(?:Clamped)?Array$/.test(t) ? _arrayLikeToArray(r, a) : void 0; } }
 function _arrayLikeToArray(r, a) { (null == a || a > r.length) && (a = r.length); for (var e = 0, n = Array(a); e < a; e++) n[e] = r[e]; return n; }
@@ -34,13 +31,16 @@ export var createPlugin = function createPlugin() {
         var localIdWasAdded = false;
         var _editorView$state$sch = editorView.state.schema.nodes,
           text = _editorView$state$sch.text,
-          hardBreak = _editorView$state$sch.hardBreak;
-        var ignoredNodeTypes = [text.name, hardBreak.name];
+          hardBreak = _editorView$state$sch.hardBreak,
+          mediaGroup = _editorView$state$sch.mediaGroup;
+        // Media group is ignored for now
+        // https://bitbucket.org/atlassian/adf-schema/src/fb2236147a0c2bc9c8efbdb75fd8f8c411df44ba/packages/adf-schema/src/next-schema/nodes/mediaGroup.ts#lines-12
+        var ignoredNodeTypes = mediaGroup ? [text.name, hardBreak.name, mediaGroup.name] : [text.name, hardBreak.name];
         editorView.state.doc.descendants(function (node, pos) {
           // Skip text nodes, hard breaks and nodes that already have local IDs
           if (!ignoredNodeTypes.includes(node.type.name) && !node.attrs.localId) {
             localIdWasAdded = true;
-            addLocalIdToNode(node, pos, tr);
+            addLocalIdToNode(pos, tr);
           }
           return true; // Continue traversing
         });
@@ -64,11 +64,15 @@ export var createPlugin = function createPlugin() {
       var tr = newState.tr;
       var _newState$schema$node = newState.schema.nodes,
         text = _newState$schema$node.text,
-        hardBreak = _newState$schema$node.hardBreak;
-      var ignoredNodeTypes = [text === null || text === void 0 ? void 0 : text.name, hardBreak === null || hardBreak === void 0 ? void 0 : hardBreak.name];
+        hardBreak = _newState$schema$node.hardBreak,
+        mediaGroup = _newState$schema$node.mediaGroup;
+      // Media group is ignored for now
+      // https://bitbucket.org/atlassian/adf-schema/src/fb2236147a0c2bc9c8efbdb75fd8f8c411df44ba/packages/adf-schema/src/next-schema/nodes/mediaGroup.ts#lines-12
+      var ignoredNodeTypes = [text === null || text === void 0 ? void 0 : text.name, hardBreak === null || hardBreak === void 0 ? void 0 : hardBreak.name, mediaGroup === null || mediaGroup === void 0 ? void 0 : mediaGroup.name];
       var addedNodes = new Set();
       var addedNodePos = new Map();
       var localIds = new Set();
+
       // Process only the nodes added in the transactions
       transactions.forEach(function (transaction) {
         if (!transaction.docChanged) {
@@ -94,7 +98,7 @@ export var createPlugin = function createPlugin() {
                 addedNodePos.set(node, pos);
               } else {
                 if (!(node !== null && node !== void 0 && node.attrs.localId)) {
-                  addLocalIdToNode(node, pos, tr);
+                  addLocalIdToNode(pos, tr);
                 }
               }
               return true;
@@ -118,7 +122,7 @@ export var createPlugin = function createPlugin() {
             if (!node.attrs.localId || localIds.has(node.attrs.localId)) {
               var pos = addedNodePos.get(node);
               if (pos !== undefined) {
-                addLocalIdToNode(node, pos, tr);
+                addLocalIdToNode(pos, tr);
                 modified = true;
               }
             }
@@ -137,15 +141,10 @@ export var createPlugin = function createPlugin() {
  * Adds a local ID to a ProseMirror node
  *
  * This utility function updates a node's attributes to include a unique local ID.
- * It preserves all existing attributes and marks while adding the new localId.
  *
- * @param node - The ProseMirror node to add a local ID to
  * @param pos - The position of the node in the document
  * @param tr - The transaction to apply the change to
- * @returns The updated transaction with the node markup change
  */
-export var addLocalIdToNode = function addLocalIdToNode(node, pos, tr) {
-  tr.setNodeMarkup(pos, node.type, _objectSpread(_objectSpread({}, node.attrs), {}, {
-    localId: uuid.generate()
-  }), node.marks);
+export var addLocalIdToNode = function addLocalIdToNode(pos, tr) {
+  tr.setNodeAttribute(pos, 'localId', uuid.generate());
 };

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

@@ -1,5 +1,4 @@
 import { SafePlugin } from '@atlaskit/editor-common/safe-plugin';
-import type { Node as PMNode } from '@atlaskit/editor-prosemirror/model';
 import { PluginKey, type Transaction } from '@atlaskit/editor-prosemirror/state';
 export declare const localIdPluginKey: PluginKey<any>;
 export declare const createPlugin: () => SafePlugin<any>;
@@ -7,11 +6,8 @@ export declare const createPlugin: () => SafePlugin<any>;
  * Adds a local ID to a ProseMirror node
  *
  * This utility function updates a node's attributes to include a unique local ID.
- * It preserves all existing attributes and marks while adding the new localId.
  *
- * @param node - The ProseMirror node to add a local ID to
  * @param pos - The position of the node in the document
  * @param tr - The transaction to apply the change to
- * @returns The updated transaction with the node markup change
  */
-export declare const addLocalIdToNode: (node: PMNode, pos: number, tr: Transaction) => void;
+export declare const addLocalIdToNode: (pos: number, tr: Transaction) => void;

package/dist/types-ts4.5/pm-plugins/main.d.ts

@@ -1,5 +1,4 @@
 import { SafePlugin } from '@atlaskit/editor-common/safe-plugin';
-import type { Node as PMNode } from '@atlaskit/editor-prosemirror/model';
 import { PluginKey, type Transaction } from '@atlaskit/editor-prosemirror/state';
 export declare const localIdPluginKey: PluginKey<any>;
 export declare const createPlugin: () => SafePlugin<any>;
@@ -7,11 +6,8 @@ export declare const createPlugin: () => SafePlugin<any>;
  * Adds a local ID to a ProseMirror node
  *
  * This utility function updates a node's attributes to include a unique local ID.
- * It preserves all existing attributes and marks while adding the new localId.
  *
- * @param node - The ProseMirror node to add a local ID to
  * @param pos - The position of the node in the document
  * @param tr - The transaction to apply the change to
- * @returns The updated transaction with the node markup change
  */
-export declare const addLocalIdToNode: (node: PMNode, pos: number, tr: Transaction) => void;
+export declare const addLocalIdToNode: (pos: number, tr: Transaction) => void;

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@atlaskit/editor-plugin-local-id",
-	"version": "1.0.2",
+	"version": "1.0.4",
 	"description": "LocalId plugin for @atlaskit/editor-core",
 	"author": "Atlassian Pty Ltd",
 	"license": "Apache-2.0",
@@ -27,18 +27,15 @@
 	},
 	"sideEffects": false,
 	"atlaskit:src": "src/index.ts",
-	"af:exports": {
-		".": "./src/index.ts"
-	},
 	"dependencies": {
-		"@atlaskit/adf-schema": "^50.2.1",
+		"@atlaskit/adf-schema": "^50.2.2",
 		"@atlaskit/editor-prosemirror": "7.0.0",
 		"@atlaskit/platform-feature-flags": "^1.1.0",
 		"@babel/runtime": "^7.0.0",
 		"raf-schd": "^4.0.3"
 	},
 	"peerDependencies": {
-		"@atlaskit/editor-common": "^107.26.0",
+		"@atlaskit/editor-common": "^107.32.0",
 		"react": "^18.2.0"
 	},
 	"devDependencies": {