Do not fail if the location of a warning is outside the original sour…

…ce (#4922)

* Do not fail if the location of a warning is outside the original source

* Fix comparison

src/utils/getCodeFrame.ts

@@ -14,6 +14,8 @@ const ELLIPSIS = '...';
 
 export default function getCodeFrame(source: string, line: number, column: number): string {
 	let lines = source.split('\n');
+	// Needed if a plugin did not generate correct sourcemaps
+	if (line > lines.length) return '';
 	const maxLineLength = Math.max(
 		tabsToSpaces(lines[line - 1].slice(0, column)).length +
 			MIN_CHARACTERS_SHOWN_AFTER_LOCATION +

test/function/samples/warning-incorrect-sourcemap-location/_config.js

@@ -0,0 +1,34 @@
+const { join } = require('node:path');
+const ID_MAIN = join(__dirname, 'main.js');
+const ID_CONSTANTS = join(__dirname, 'constants.js');
+
+module.exports = {
+	description: 'does not fail if a warning has an incorrect location due to missing sourcemaps',
+	options: {
+		plugins: [
+			{
+				name: 'test',
+				transform(code) {
+					return '/* injected */;\n\n\n\n\n\n\n\n' + code;
+				}
+			}
+		]
+	},
+	warnings: [
+		{
+			binding: 'NON_EXISTENT',
+			code: 'MISSING_EXPORT',
+			exporter: ID_CONSTANTS,
+			frame: '',
+			id: ID_MAIN,
+			loc: {
+				column: 15,
+				file: ID_MAIN,
+				line: 12
+			},
+			message: '"NON_EXISTENT" is not exported by "constants.js", imported by "main.js".',
+			pos: 111,
+			url: 'https://rollupjs.org/troubleshooting/#error-name-is-not-exported-by-module'
+		}
+	]
+};

test/function/samples/warning-incorrect-sourcemap-location/constants.js

@@ -0,0 +1 @@
+export const q = 'Queue';

test/function/samples/warning-incorrect-sourcemap-location/main.js

@@ -0,0 +1,5 @@
+import * as CONSTANTS from './constants';
+
+export default class Sample {
+	x = CONSTANTS.NON_EXISTENT;
+}