Replace final character in string, as opposted to first instance of *

[gingerbenw]

Goal

Update node export resolver code to fix code scanning alert 125

[Copilot]

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

jest/node-exports-resolver.js:147

• Using /$/ matches the end-of-string position (a zero-width match) rather than the final character. If the goal is to replace the last character of the string, consider using /.$/ instead.

+                  /\$/, submoduleName.slice(exportKey.length - 1)

[github-actions]

@bugsnag/browser bundle size diff

	Minified	Minfied + Gzipped
Before	56.43 kB	17.15 kB
After	56.43 kB	17.15 kB
±	No change	No change

code coverage diff

<temporarily disabled>

Generated by 🚫 dangerJS against b4658e7

[Copilot]

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

jest/node-exports-resolver.js:146

• The replacement regex /.$/ replaces the last character regardless of its value, which may lead to unintended behavior if exportValue does not actually end with ''. Consider adding a check to ensure exportValue ends with '' before performing the replacement.

exportValue.replace(/.$/, submoduleName.slice(exportKey.length - 1))