Error("Length exceeded!") should not be a module variable

[dev-guy]

Checklist

[x] Are you reporting a bug? Use github issues for bug reports and feature requests. For general questions, please use https://discuss.yjs.dev/ [x] Try to report your issue in the correct repository. Yjs consists of many modules. When in doubt, report it to https://github.com/yjs/yjs/issues/

Is your feature request related to a problem? Please describe. The error "Length exceeded!" is thrown in a couple places[1]. Because the Error object is a module variable, its stack doesn't reflect where the exception was actually thrown. Therefore, it's impossible to know for sure what the error is (which method was called).

[1] src/types/AbstractType.js line 686 const lengthExceeded = error.create('Length exceeded!')

Describe the solution you'd like

const lengthExceeded = "Length exceeded!"
...
throw new Error(lengthExceeded);

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

Version 13.6.7

The error occurs when using splice() to insert an element into an array. I'm using SvelteSyncedStore so I don't exactly know how YJs is being used. I'm sure YArray is involved.

I guess it would be nice if the error message contained the provided index that is out of bounds. I'm trying to get that logged somehow.

_{Huly®: YJS-381}

[dmonad]

Right, thanks. I fixed the issue. This will be added to the next release.