InlineCoordsScan.scan can cause RangeError: Maximum call stack size exceeded

craftzdog commented

2026-04-28 09:23:16 +02:00

Since @codemirror/view@6.41.0, it randomly causes an error when moving the cursor vertically with j/k keys with the vim extension.
A workaround is to downgrade it to v6.40.0.
It looks like it's introduced by commit 49d72c4.
I think it's not a bug of @replit/codemirror-vim, so I report it here.

Stack trace

index.js:1358 RangeError: Maximum call stack size exceeded
    at InlineCoordsScan.scan (index.js:3834:1)
    at InlineCoordsScan.scan (index.js:3914:1)
    at InlineCoordsScan.scan (index.js:3918:1)
    at InlineCoordsScan.scan (index.js:3914:1)
    at InlineCoordsScan.scan (index.js:3918:1)
    at InlineCoordsScan.scan (index.js:3914:1)
    at InlineCoordsScan.scan (index.js:3918:1)
    at InlineCoordsScan.scan (index.js:3914:1)
    at InlineCoordsScan.scan (index.js:3918:1)
    at InlineCoordsScan.scan (index.js:3914:1)
    ...

How to reproduce

Use the vim extension
Load a document with mixed-height inline content (e.g. a 200% heading next to normal text)
Call view.posAtCoords({x, y}, false, 1) with a y that overlaps both neighbors

Since @codemirror/view@6.41.0, it randomly causes an error when moving the cursor vertically with `j/k` keys with [the vim extension](https://github.com/replit/codemirror-vim). A workaround is to downgrade it to v6.40.0. It looks like it's introduced by commit [49d72c4](https://code.haverbeke.berlin/codemirror/view/commit/49d72c4). I think it's not a bug of `@replit/codemirror-vim`, so I report it here. ## Stack trace ``` index.js:1358 RangeError: Maximum call stack size exceeded at InlineCoordsScan.scan (index.js:3834:1) at InlineCoordsScan.scan (index.js:3914:1) at InlineCoordsScan.scan (index.js:3918:1) at InlineCoordsScan.scan (index.js:3914:1) at InlineCoordsScan.scan (index.js:3918:1) at InlineCoordsScan.scan (index.js:3914:1) at InlineCoordsScan.scan (index.js:3918:1) at InlineCoordsScan.scan (index.js:3914:1) at InlineCoordsScan.scan (index.js:3918:1) at InlineCoordsScan.scan (index.js:3914:1) ... ``` ## How to reproduce - Use the vim extension - Load a document with mixed-height inline content (e.g. a 200% heading next to normal text) - Call `view.posAtCoords({x, y}, false, 1)` with a `y` that overlaps both neighbors

marijn commented

2026-04-28 10:14:08 +02:00

Owner

I didn't manage to reproduce this, even when intentionally making elements overlap with a low line-height. Also, your reproduction instructions don't seem to use the vim extension, yet say to load it. It would be helpful if you could set up a small self-contained demo for this. And also let me know which browser(s) you can reproduce it in.

craftzdog commented

2026-04-28 13:41:01 +02:00

Author

Now I can't even reproduce it on my Electron app 🤯
I use electron@39.8.7.
Sorry, I will try to investigate further when this happens again.

Now I can't even reproduce it on my Electron app 🤯 I use `electron@39.8.7`. Sorry, I will try to investigate further when this happens again.

marijn referenced this issue from a commit

2026-05-01 12:08:54 +02:00

Limit recursion depth in posAtCoords

marijn closed this issue

2026-05-01 12:08:54 +02:00

marijn commented

2026-05-01 12:09:40 +02:00

Owner

I went ahead and added a safeguard for this, though I didn't manage to actually reproduce the issue.

😆 1

craftzdog commented

2026-05-01 13:16:18 +02:00

Author

Wow thank you!
btw, a link in the email notification about the commit seems to be broken: https://code.haverbeke.berlin/codemirror/dev/commit/92bf92375cc1e174c2e5157ad5cd6f7bdb0296ca

@marijn closed #1699 in commit 92bf92375c.

It points to the codemirror/dev repo

Wow thank you! btw, a link in the email notification about the commit seems to be broken: <https://code.haverbeke.berlin/codemirror/dev/commit/92bf92375cc1e174c2e5157ad5cd6f7bdb0296ca> > **@marijn** closed [#1699](https://code.haverbeke.berlin/codemirror/dev/issues/1699) in commit [**92bf92375c**](https://code.haverbeke.berlin/codemirror/dev/commit/92bf92375cc1e174c2e5157ad5cd6f7bdb0296ca). It points to the `codemirror/dev` repo

marijn commented

2026-05-01 13:26:47 +02:00

Owner

That appears to be a Forgejo bug. I've filed https://codeberg.org/forgejo/forgejo/issues/12367

👍 1

Rows
Columns

InlineCoordsScan.scan can cause RangeError: Maximum call stack size exceeded #1699

Stack trace

How to reproduce