Pass rootScope to error serializer

Author: sebastianwessel

src/sandbox/handleToNative/handleToNative.ts

@@ -168,7 +168,7 @@ export const handleToNative = (ctx: QuickJSContext | QuickJSAsyncContext, handle
 		if (typeof errorType === 'string') {
 			const s = getSerializer(errorType)
 			if (s) {
-				const ret = s(ctx, handle)
+				const ret = s(ctx, handle, rootScope)
 				if (ret) {
 					return ret
 				}
@@ -204,9 +204,9 @@ export const handleToNative = (ctx: QuickJSContext | QuickJSAsyncContext, handle
 			handle,
 		).consume(r => ctx.dump(r))
 
-                const serializer = getSerializer(constructorName)
-                if (serializer) {
-                        const ret = serializer(ctx, handle, rootScope)
+		const serializer = getSerializer(constructorName)
+		if (serializer) {
+			const ret = serializer(ctx, handle, rootScope)
 			if (ret) {
 				return ret
 			}

src/test-regression/82-missing-root-scope.test.ts

@@ -0,0 +1,36 @@
+import { describe, expect, it } from 'bun:test'
+import { loadQuickJs } from '../loadQuickJs.js'
+import type { ErrorResponse } from '../types/ErrorResponse.js'
+import type { SandboxOptions } from '../types/SandboxOptions.js'
+
+describe('bugfix - #82', () => {
+	it('handles async env errors without crashing', async () => {
+		const runtime = await loadQuickJs()
+
+		const thrower = async (_input: string) => {
+			throw new Error('test')
+		}
+
+		const options: SandboxOptions = {
+			allowFetch: false,
+			env: {
+				thrower,
+			},
+		}
+
+		const code = `
+      const fn = async ()=>{
+        await env.thrower('some-id')
+        return true
+      }
+
+      export default await fn()
+    `
+
+		const result = await runtime.runSandboxed(async ({ evalCode }) => evalCode(code), options)
+
+		expect(result.ok).toBeFalse()
+		expect((result as ErrorResponse).error.name).toBe('Error')
+		expect((result as ErrorResponse).error.message).toBe('test')
+	})
+})