prosemirror-transform 1.12.0 → 1.12.2

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+## 1.12.2 (2026-09-25)
+
+### Bug fixes
+
+Fix a bug that caused `StepMap.mapResult` to set deletion flags for insertions adjacent to the mapped position.
+
+## 1.12.1 (2026-09-02)
+
+### Bug fixes
+
+Fix an issue where replace step fitting could create an invalid slice, and then crash or loop forever.
+
 ## 1.12.0 (2026-03-30)
 
 ### Bug fixes

package/CONTRIBUTING.md

@@ -12,7 +12,7 @@ Community discussion, questions, and informal bug reporting is done on the
 ## Submitting bug reports
 
 Report bugs on the
-[GitHub issue tracker](http://github.com/prosemirror/prosemirror/issues).
+[issue tracker](https://code.haverbeke.berlin/prosemirror/prosemirror/issues).
 Before reporting a bug, please read these pointers.
 
 - The issue tracker is for *bugs*, not requests for help. Questions
@@ -34,25 +34,28 @@ Before reporting a bug, please read these pointers.
 
 ## Contributing code
 
+Code generated by a language model is not welcome in this project.
+Please don't waste my time with it.
+
 If you want to make a change that involves a significant overhaul of
 the code or introduces a user-visible new feature, create an
-[RFC](https://github.com/ProseMirror/rfcs/) first with your proposal.
+[issue](https://code.haverbeke.berlin/prosemirror/prosemirror/issues/)
+first with your proposal.
 
-- Make sure you have a [GitHub Account](https://github.com/signup/free)
+- Make sure you have a [Codeberg](https://codeberg.org/user/sign_up)
+  or [GitHub](https://github.com/signup/free) account.
+  
+- Use that to create a [code.haverbeke.berlin
+  account](https://code.haverbeke.berlin/user/login).
 
-- Fork the relevant repository
-  ([how to fork a repo](https://help.github.com/articles/fork-a-repo))
+- Fork the relevant repository.
 
 - Create a local checkout of the code. You can use the
-  [main repository](https://github.com/prosemirror/prosemirror) to
+  [main repository](https://code.haverbeke.berlin/prosemirror/prosemirror) to
   easily check out all core modules.
 
 - Make your changes, and commit them
 
-- Follow the code style of the rest of the project (see below). Run
-  `npm run lint` (in the main repository checkout) to make sure that
-  the linter is happy.
-
 - If your changes are easy to test or likely to regress, add tests in
   the relevant `test/` directory. Either put them in an existing
   `test-*.js` file, if they fit there, or add a new file.
@@ -60,13 +63,13 @@ the code or introduces a user-visible new feature, create an
 - Make sure all tests pass. Run `npm run test` to verify tests pass
   (you will need Node.js v6+).
 
-- Submit a pull request ([how to create a pull request](https://help.github.com/articles/fork-a-repo)).
-  Don't put more than one feature/fix in a single pull request.
+- Submit a pull request. Don't put more than one feature/fix in a
+  single pull request.
 
 By contributing code to ProseMirror you
 
  - Agree to license the contributed code under the project's [MIT
-   license](https://github.com/ProseMirror/prosemirror/blob/master/LICENSE).
+   license](https://code.haverbeke.berlin/prosemirror/prosemirror/blob/main/LICENSE).
 
  - Confirm that you have the right to contribute and license the code
    in question. (Either you hold all rights on the code, or the rights
@@ -86,19 +89,16 @@ By contributing code to ProseMirror you
 - Follow the surrounding code when it comes to spacing, brace
   placement, etc.
 
-- Brace-less single-statement bodies are encouraged (whenever they
-  don't impact readability).
+- Brace-less single-statement bodies are encouraged whenever they
+  don't impact readability.
 
-- [getdocs](https://github.com/marijnh/getdocs)-style doc comments
+- [getdocs-ts](https://code.haverbeke.berlin/marijn/getdocs-ts)-style doc comments
   above items that are part of the public API.
 
 - When documenting non-public items, you can put the type after a
   single colon, so that getdocs doesn't pick it up and add it to the
   API reference.
 
-- The linter (`npm run lint`) complains about unused variables and
-  functions. Prefix their names with an underscore to muffle it.
-
 - ProseMirror does *not* follow JSHint or JSLint prescribed style.
   Patches that try to 'fix' code to pass one of these linters will not
   be accepted.

package/README.md

@@ -1,6 +1,6 @@
 # prosemirror-transform
 
-[ [**WEBSITE**](https://prosemirror.net) | [**ISSUES**](https://github.com/prosemirror/prosemirror/issues) | [**FORUM**](https://discuss.prosemirror.net) | [**CHANGELOG**](https://github.com/ProseMirror/prosemirror-transform/blob/master/CHANGELOG.md) ]
+[ [**WEBSITE**](https://prosemirror.net) | [**ISSUES**](https://code.haverbeke.berlin/prosemirror/prosemirror/issues) | [**FORUM**](https://discuss.prosemirror.net) | [**CHANGELOG**](https://code.haverbeke.berlin/prosemirror/prosemirror-transform/src/branch/main/CHANGELOG.md) ]
 
 This is a [core module](https://prosemirror.net/docs/ref/#transform) of [ProseMirror](https://prosemirror.net).
 ProseMirror is a well-behaved rich semantic content editor based on
@@ -17,10 +17,10 @@ number of [examples](https://prosemirror.net/examples/) and the
 [documentation](https://prosemirror.net/docs/).
 
 This code is released under an
-[MIT license](https://github.com/prosemirror/prosemirror/tree/master/LICENSE).
+[MIT license](https://code.haverbeke.berlin/prosemirror/prosemirror/src/branch/main/LICENSE).
 There's a [forum](http://discuss.prosemirror.net) for general
 discussion and support requests, and the
-[Github bug tracker](https://github.com/prosemirror/prosemirror/issues)
+[bug tracker](https://code.haverbeke.berlin/prosemirror/prosemirror/issues)
 is the place to report issues.
 
 We aim to be an inclusive, welcoming community. To make that explicit,

package/dist/index.cjs

@@ -107,8 +107,11 @@ var StepMap = function () {
           var result = start + diff + (side < 0 ? 0 : newSize);
           if (simple) return result;
           var recover = pos == (assoc < 0 ? start : end) ? null : makeRecover(i / 3, pos - start);
-          var del = pos == start ? DEL_AFTER : pos == end ? DEL_BEFORE : DEL_ACROSS;
-          if (assoc < 0 ? pos != start : pos != end) del |= DEL_SIDE;
+          var del = 0;
+          if (oldSize) {
+            del |= pos == start ? DEL_AFTER : pos == end ? DEL_BEFORE : DEL_ACROSS;
+            if (assoc < 0 ? pos != start : pos != end) del |= DEL_SIDE;
+          }
           return new MapResult(result, del, recover);
         }
         diff += newSize - oldSize;
@@ -1254,9 +1257,9 @@ var Fitter = function () {
         content = _this$unplaced.content,
         openStart = _this$unplaced.openStart,
         openEnd = _this$unplaced.openEnd;
-      var inner = contentAt(content, openStart);
-      if (!inner.childCount || inner.firstChild.isLeaf) return false;
-      this.unplaced = new prosemirrorModel.Slice(content, openStart + 1, Math.max(openEnd, inner.size + openStart >= content.size - openEnd ? openStart + 1 : 0));
+      if (maxOpen(content, -1) <= openStart) return false;
+      if (this.unplaced.size > 1 && maxOpen(content, 1) > openEnd) openEnd++;
+      this.unplaced = new prosemirrorModel.Slice(content, openStart + 1, openEnd);
       return true;
     }
   }, {
@@ -1432,6 +1435,13 @@ function invalidMarks(type, fragment, start) {
 function definesContent(type) {
   return type.spec.defining || type.spec.definingForContent;
 }
+function maxOpen(frag, side) {
+  for (var count = 0;; count++) {
+    var ch = side < 0 ? frag.firstChild : frag.lastChild;
+    if (!ch || ch.isAtom) return count;
+    frag = ch.content;
+  }
+}
 function _replaceRange(tr, from, to, slice) {
   if (!slice.size) return tr.deleteRange(from, to);
   var $from = tr.doc.resolve(from),

package/dist/index.d.cts

@@ -1,4 +1,4 @@
-import { Node, Schema, Slice, Fragment, NodeRange, NodeType, Attrs, Mark, MarkType, ContentMatch } from 'prosemirror-model';
+import { Node, Slice, Schema, Fragment, NodeRange, NodeType, Attrs, Mark, MarkType, ContentMatch } from 'prosemirror-model';
 
 /**
 There are several things that positions can be mapped through.
@@ -853,4 +853,5 @@ would be a no-op (an empty slice over an empty range).
 */
 declare function replaceStep(doc: Node, from: number, to?: number, slice?: Slice): Step | null;
 
-export { AddMarkStep, AddNodeMarkStep, AttrStep, DocAttrStep, MapResult, type Mappable, Mapping, RemoveMarkStep, RemoveNodeMarkStep, ReplaceAroundStep, ReplaceStep, Step, StepMap, StepResult, Transform, canJoin, canSplit, dropPoint, findWrapping, insertPoint, joinPoint, liftTarget, replaceStep };
+export { AddMarkStep, AddNodeMarkStep, AttrStep, DocAttrStep, MapResult, Mapping, RemoveMarkStep, RemoveNodeMarkStep, ReplaceAroundStep, ReplaceStep, Step, StepMap, StepResult, Transform, canJoin, canSplit, dropPoint, findWrapping, insertPoint, joinPoint, liftTarget, replaceStep };
+export type { Mappable };

package/dist/index.d.ts

@@ -1,4 +1,4 @@
-import { Node, Schema, Slice, Fragment, NodeRange, NodeType, Attrs, Mark, MarkType, ContentMatch } from 'prosemirror-model';
+import { Node, Slice, Schema, Fragment, NodeRange, NodeType, Attrs, Mark, MarkType, ContentMatch } from 'prosemirror-model';
 
 /**
 There are several things that positions can be mapped through.
@@ -853,4 +853,5 @@ would be a no-op (an empty slice over an empty range).
 */
 declare function replaceStep(doc: Node, from: number, to?: number, slice?: Slice): Step | null;
 
-export { AddMarkStep, AddNodeMarkStep, AttrStep, DocAttrStep, MapResult, type Mappable, Mapping, RemoveMarkStep, RemoveNodeMarkStep, ReplaceAroundStep, ReplaceStep, Step, StepMap, StepResult, Transform, canJoin, canSplit, dropPoint, findWrapping, insertPoint, joinPoint, liftTarget, replaceStep };
+export { AddMarkStep, AddNodeMarkStep, AttrStep, DocAttrStep, MapResult, Mapping, RemoveMarkStep, RemoveNodeMarkStep, ReplaceAroundStep, ReplaceStep, Step, StepMap, StepResult, Transform, canJoin, canSplit, dropPoint, findWrapping, insertPoint, joinPoint, liftTarget, replaceStep };
+export type { Mappable };

package/dist/index.js

@@ -115,9 +115,12 @@ class StepMap {
                 if (simple)
                     return result;
                 let recover = pos == (assoc < 0 ? start : end) ? null : makeRecover(i / 3, pos - start);
-                let del = pos == start ? DEL_AFTER : pos == end ? DEL_BEFORE : DEL_ACROSS;
-                if (assoc < 0 ? pos != start : pos != end)
-                    del |= DEL_SIDE;
+                let del = 0;
+                if (oldSize) {
+                    del |= pos == start ? DEL_AFTER : pos == end ? DEL_BEFORE : DEL_ACROSS;
+                    if (assoc < 0 ? pos != start : pos != end)
+                        del |= DEL_SIDE;
+                }
                 return new MapResult(result, del, recover);
             }
             diff += newSize - oldSize;
@@ -1474,10 +1477,11 @@ class Fitter {
     }
     openMore() {
         let { content, openStart, openEnd } = this.unplaced;
-        let inner = contentAt(content, openStart);
-        if (!inner.childCount || inner.firstChild.isLeaf)
+        if (maxOpen(content, -1) <= openStart)
             return false;
-        this.unplaced = new Slice(content, openStart + 1, Math.max(openEnd, inner.size + openStart >= content.size - openEnd ? openStart + 1 : 0));
+        if (this.unplaced.size > 1 && maxOpen(content, 1) > openEnd)
+            openEnd++;
+        this.unplaced = new Slice(content, openStart + 1, openEnd);
         return true;
     }
     dropNode() {
@@ -1647,6 +1651,14 @@ function invalidMarks(type, fragment, start) {
 function definesContent(type) {
     return type.spec.defining || type.spec.definingForContent;
 }
+function maxOpen(frag, side) {
+    for (let count = 0;; count++) {
+        let ch = side < 0 ? frag.firstChild : frag.lastChild;
+        if (!ch || ch.isAtom)
+            return count;
+        frag = ch.content;
+    }
+}
 function replaceRange(tr, from, to, slice) {
     if (!slice.size)
         return tr.deleteRange(from, to);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "prosemirror-transform",
-  "version": "1.12.0",
+  "version": "1.12.2",
   "description": "ProseMirror document transformations",
   "type": "module",
   "main": "dist/index.cjs",
@@ -21,7 +21,7 @@
   ],
   "repository": {
     "type": "git",
-    "url": "git://github.com/prosemirror/prosemirror-transform.git"
+    "url": "git+https://code.haverbeke.berlin/prosemirror/prosemirror-transform.git"
   },
   "dependencies": {
     "prosemirror-model": "^1.21.0"

package/src/attr_step.ts

@@ -95,4 +95,4 @@ export class DocAttrStep extends Step {
   }
 }
 
-Step.jsonID("docAttr", DocAttrStep)
+Step.jsonID("docAttr", DocAttrStep)

package/src/map.ts

@@ -106,8 +106,11 @@ export class StepMap implements Mappable {
         let result = start + diff + (side < 0 ? 0 : newSize)
         if (simple) return result
         let recover = pos == (assoc < 0 ? start : end) ? null : makeRecover(i / 3, pos - start)
-        let del = pos == start ? DEL_AFTER : pos == end ? DEL_BEFORE : DEL_ACROSS
-        if (assoc < 0 ? pos != start : pos != end) del |= DEL_SIDE
+        let del = 0
+        if (oldSize) {
+          del |= pos == start ? DEL_AFTER : pos == end ? DEL_BEFORE : DEL_ACROSS
+          if (assoc < 0 ? pos != start : pos != end) del |= DEL_SIDE
+        }
         return new MapResult(result, del, recover)
       }
       diff += newSize - oldSize

package/src/replace.ts

@@ -155,10 +155,9 @@ class Fitter {
 
   openMore() {
     let {content, openStart, openEnd} = this.unplaced
-    let inner = contentAt(content, openStart)
-    if (!inner.childCount || inner.firstChild!.isLeaf) return false
-    this.unplaced = new Slice(content, openStart + 1,
-                              Math.max(openEnd, inner.size + openStart >= content.size - openEnd ? openStart + 1 : 0))
+    if (maxOpen(content, -1) <= openStart) return false
+    if (this.unplaced.size > 1 && maxOpen(content, 1) > openEnd) openEnd++
+    this.unplaced = new Slice(content, openStart + 1, openEnd)
     return true
   }
 
@@ -331,6 +330,14 @@ function definesContent(type: NodeType) {
   return type.spec.defining || type.spec.definingForContent
 }
 
+function maxOpen(frag: Fragment, side: -1 | 1) {
+  for (let count = 0;; count++) {
+    let ch = side < 0 ? frag.firstChild : frag.lastChild
+    if (!ch || ch.isAtom) return count
+    frag = ch.content
+  }
+}
+
 export function replaceRange(tr: Transform, from: number, to: number, slice: Slice) {
   if (!slice.size) return tr.deleteRange(from, to)