Open dmmulroy opened 4 months ago
Latest commit: c6ad317828515c665f6f0f6efb8e3aa90e7ba3a7
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
Currently the
parseError
function inparseErrors.ts
is incorrectly calculating theendIndex
of each match.The code currently is written as:
This causing
endIndex
to always equalstartIndex
. The use of??
was likely used to satisfy the TypeScript compiler asindexOf
can returnundefined
. However, since we know thatmatchElem
will aways be inmessage
(it comes from regexp match) it'll never actually beundefined
.This has the consequence of
startIndex
always being truthy and never evaluating?? 0 + matchElem.length
This PR changes the code to:
In addition, I added a test for this and updated the snapshot tests. Happy to make any additional changes!