lib: fix ERR_INVALID_ARG_TYPE with --enable-source-maps

Author: kimjune01

lib/internal/errors/error_source.js

@@ -32,14 +32,19 @@ function getErrorSourceLocation(error) {
     startColumn,
   } = pos;
 
+  if (!sourceLine) {
+    return;
+  }
+
   // Source map is not enabled. Return the source line directly.
   if (!getSourceMapsSupport().enabled) {
     return { sourceLine, startColumn };
   }
 
   const sm = findSourceMap(scriptResourceName);
   if (sm === undefined) {
-    return;
+    // No source map for this file — use the generated source line.
+    return { sourceLine, startColumn };
   }
   const {
     originalLine,
@@ -49,7 +54,9 @@ function getErrorSourceLocation(error) {
   const originalSourceLine = getSourceLine(sm, originalSource, originalLine, originalColumn);
 
   if (!originalSourceLine) {
-    return;
+    // Source map exists but original source is unavailable — use the
+    // generated source line rather than returning undefined.
+    return { sourceLine, startColumn };
   }
 
   return {

test/parallel/test-assert-ok-source-maps.js

@@ -0,0 +1,67 @@
+'use strict';
+
+// Regression test for https://github.com/nodejs/node/issues/63169
+// When --enable-source-maps is enabled but the script has no source map,
+// assert(false) should throw AssertionError, not TypeError ERR_INVALID_ARG_TYPE.
+
+require('../common');
+const assert = require('node:assert');
+const { spawnSync } = require('node:child_process');
+const { test } = require('node:test');
+const fs = require('node:fs');
+const path = require('node:path');
+const tmpdir = require('../common/tmpdir');
+
+tmpdir.refresh();
+
+function runWithSourceMaps(filename, code) {
+  const filePath = path.join(tmpdir.path, filename);
+  fs.writeFileSync(filePath, code);
+  const result = spawnSync(process.execPath, [
+    '--enable-source-maps',
+    filePath,
+  ]);
+  return result;
+}
+
+test('assert(false) in .mjs with --enable-source-maps throws AssertionError', () => {
+  const result = runWithSourceMaps('repro.mjs', `
+import { strict as assert } from 'node:assert';
+try {
+  assert(false);
+} catch (e) {
+  process.stdout.write(e.constructor.name + '\\n');
+  process.stdout.write(e.code + '\\n');
+  process.stdout.write(e.message + '\\n');
+  process.exit(0);
+}
+process.exit(1);
+`);
+
+  assert.strictEqual(result.status, 0, `process exited with ${result.status}: ${result.stderr}`);
+  const lines = result.stdout.toString().trim().split('\n');
+  assert.strictEqual(lines[0], 'AssertionError');
+  assert.strictEqual(lines[1], 'ERR_ASSERTION');
+  assert.match(lines[2], /assert\(false\)/);
+});
+
+test('assert.ok(false) in .cjs with --enable-source-maps throws AssertionError', () => {
+  const result = runWithSourceMaps('repro.cjs', `
+const assert = require('node:assert');
+try {
+  assert.ok(false);
+} catch (e) {
+  process.stdout.write(e.constructor.name + '\\n');
+  process.stdout.write(e.code + '\\n');
+  process.stdout.write(e.message + '\\n');
+  process.exit(0);
+}
+process.exit(1);
+`);
+
+  assert.strictEqual(result.status, 0, `process exited with ${result.status}: ${result.stderr}`);
+  const lines = result.stdout.toString().trim().split('\n');
+  assert.strictEqual(lines[0], 'AssertionError');
+  assert.strictEqual(lines[1], 'ERR_ASSERTION');
+  assert.match(lines[2], /assert\.ok\(false\)/);
+});