Error is thrown when there's a warning in a file that is transformed into a bigger file

[abdulsattar]

Rollup Version

3.18.0

Operating System (or Browser)

macOS Ventura

Node Version (if applicable)

No response

Link To Reproduction

https://stackblitz.com/edit/rollup-repro-wmdqgq?file=rollup.config.js,dist%2Fmain.js,src%2Fmain.js,package.json

Expected Behaviour

Working in 3.17.3: https://stackblitz.com/edit/rollup-repro-eeubaz?file=rollup.config.js,dist%2Fmain.js,src%2Fmain.js,package.json

Just warns about the missing export, but successfully completes compilation.

Actual Behaviour

Broken repro: https://stackblitz.com/edit/rollup-repro-wmdqgq?file=rollup.config.js,dist%2Fmain.js,src%2Fmain.js,package.json

Throws this error:

TypeError: Cannot read properties of undefined (reading 'slice')
    at getCodeFrame (file:///home/projects/rollup-repro-wmdqgq/node_modules/rollup/dist/shared/rollup.js:150:65)
    at augmentCodeLocation (file:///home/projects/rollup-repro-wmdqgq/node_modules/rollup/dist/shared/rollup.js:288:28)
    at Module.addLocationToLogProps (file:///home/projects/rollup-repro-wmdqgq/node_modules/rollup/dist/shared/rollup.js:14193:13)
    at Module.warn (file:///home/projects/rollup-repro-wmdqgq/node_modules/rollup/dist/shared/rollup.js:14075:14)
    at resolveNamespaceVariables (file:///home/projects/rollup-repro-wmdqgq/node_modules/rollup/dist/shared/rollup.js:9876:24)

My understanding, when I debugged this, is it's trying to get the line number in the original source code, not in the transformed source code (sourcemaps don't exist) and it's throwing this error.

[lukastaegert]

Fix at #4922. The problem is that your plugin changes lines in a file without generating a sourcemap. This is a problem in so far as warnings and errors will report wrong locations. You should ideally use something like magic-string to generate a source map for your changes.
The fix will not fix the warning location but prevent the crash.

[rollup-bot]

This issue has been resolved via #4922 as part of rollup@3.20.2. You can test it via npm install rollup.

[abdulsattar]

Thank you for the quick fix @lukastaegert!