fix debug.traceback failing on anonymous function (#1665) (#1666)

ChouUn · claude · web-flow · commit a3f2eea75ac3 · 2026-03-03T21:46:49.000+01:00
* fix `debug.traceback` failing on anonymous function (#1665) * test: add regression test for anonymous function traceback mapping (#1665) Nested IIFEs produce <file.lua:N> notation in debug.traceback(). The old pattern (%S+) captured the "<" prefix, causing sourcemap lookup to fail for anonymous function definition locations. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * test: address review feedback for anonymous function traceback test (#1665) Use distinct sourcemap values and explicit <main.ts:N> assertions per reviewer request. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> * style: extract setLuaHeader string to variable for prettier Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
diff --git a/src/lualib/SourceMapTraceBack.ts b/src/lualib/SourceMapTraceBack.ts
@@ -48,7 +48,12 @@ export function __TS__SourceMapTraceBack(this: void, fileName: string, sourceMap
                 return `${file}:${line}`;
             };
 
-            let [result] = string.gsub(trace, "(%S+)%.lua:(%d+)", (file, line) =>
+            // Rewrite stack trace frames from "{PATH}.lua:{LINE}" to "{PATH}.ts:{ORIGINAL_LINE}".
+            // https://github.com/TypeScriptToLua/TypeScriptToLua/issues/1665
+            // Avoid matching anonymous function stack entries like `in function <...>`
+            // by excluding `<` before the file path.
+            // TODO: This will still fail for paths containing spaces.
+            let [result] = string.gsub(trace, "([^%s<]+)%.lua:(%d+)", (file, line) =>
                 replacer(`${file}.lua`, `${file}.ts`, line)
             );
 
diff --git a/test/unit/error.spec.ts b/test/unit/error.spec.ts
@@ -345,6 +345,56 @@ test("still works without debug module", () => {
         });
 });
 
+// https://github.com/TypeScriptToLua/TypeScriptToLua/issues/1665
+test("sourceMapTraceback maps anonymous function locations in .lua files (#1665)", () => {
+    // Nested IIFEs produce <file.lua:N> anonymous function notation in traceback.
+    // Old pattern (%S+)%.lua:(%d+) captures "<main" from "<main.lua:4>",
+    // failing the sourcemap lookup. Fix: ([^%s<]+) excludes "<".
+
+    // mapping is copied from the emitted Lua, not invented.
+    const mapping = `{["5"] = 1,["6"] = 2,["7"] = 3,["8"] = 4,["9"] = 3,["10"] = 2,["11"] = 1}`;
+
+    // Test harness executes via luaL_dostring (chunk names are [string "..."]), so we mock a file-based traceback.
+    const fakeTraceback = [
+        "stack traceback:",
+        "\tmain.lua:8: in function <main.lua:7>",
+        "\tmain.lua:7: in function <main.lua:6>",
+        "\t[C]: in ?",
+    ].join("\n");
+
+    // Inject sourcemap for "main.lua" and mock debug.traceback to return file-based frames.
+    const header = `
+        __TS__sourcemap = { ["main.lua"] = ${mapping} };
+        local __real_tb = debug.traceback
+        debug.traceback = function() return ${JSON.stringify(fakeTraceback)} end
+    `;
+
+    const builder = util.testFunction`
+        return (() => {
+            return (() => {
+                return (debug.traceback as (this: void) => string)();
+            })();
+        })();
+    `
+        .setLuaHeader(header)
+        .setOptions({ sourceMapTraceback: true });
+
+    const lua = builder.getMainLuaCodeChunk();
+    // Sanity check: emitted code registers the same mapping literal we inject above.
+    expect(lua).toContain(`__TS__SourceMapTraceBack(debug.getinfo(1).short_src, ${mapping});`);
+
+    const result = builder.getLuaExecutionResult();
+    expect(result).toEqual(expect.any(String));
+    // Both `main.lua:N` and `<main.lua:N>` frames should be rewritten using the sourcemap.
+    expect(result).not.toContain("main.lua");
+    // Regular line references
+    expect(result).toContain("\tmain.ts:4:");
+    expect(result).toContain("\tmain.ts:3:");
+    // Anonymous function references must keep <> format
+    expect(result).toContain("<main.ts:3>");
+    expect(result).toContain("<main.ts:2>");
+});
+
 util.testEachVersion(
     "error stacktrace omits constructor and __TS_New",
     () => util.testFunction`
