test: hoist domain require to module scope

Address review feedback: move repeated require("domain") calls
to a single module-scope declaration.

Co-Authored-By: Claude <noreply@anthropic.com>

src/js/node/domain.ts

@@ -37,7 +37,7 @@ domain.createDomain = domain.create = function () {
     return function () {
       var args = Array.prototype.slice.$call(arguments);
       try {
-        fn.$apply(null, args);
+        return fn.$apply(null, args);
       } catch (err) {
         emitError(err);
       }
@@ -50,7 +50,7 @@ domain.createDomain = domain.create = function () {
       } else {
         var args = Array.prototype.slice.$call(arguments, 1);
         try {
-          fn.$apply(null, args);
+          return fn.$apply(null, args);
         } catch (err) {
           emitError(err);
         }
@@ -59,11 +59,10 @@ domain.createDomain = domain.create = function () {
   };
   d.run = function (fn) {
     try {
-      fn();
+      return fn();
     } catch (err) {
       emitError(err);
     }
-    return this;
   };
   d.dispose = function () {
     this.removeAllListeners();

test/regression/issue/24287.test.ts

@@ -0,0 +1,50 @@
+import { expect, test } from "bun:test";
+
+const domain = require("domain");
+
+test("domain.bind() returns the original function's return value", () => {
+  const d = domain.create();
+
+  const bound = d.bind(() => 42);
+  expect(bound()).toBe(42);
+});
+
+test("domain.bind() returns the original async function's promise", async () => {
+  const d = domain.create();
+
+  const bound = d.bind(async () => "hello");
+  const result = bound();
+  expect(result).toBeInstanceOf(Promise);
+  expect(await result).toBe("hello");
+});
+
+test("domain.intercept() returns the original function's return value", () => {
+  const d = domain.create();
+
+  const intercepted = d.intercept((...args: any[]) => args.join(","));
+  expect(intercepted(null, "a", "b")).toBe("a,b");
+});
+
+test("domain.intercept() returns the original async function's promise", async () => {
+  const d = domain.create();
+
+  const intercepted = d.intercept(async () => "world");
+  const result = intercepted(null);
+  expect(result).toBeInstanceOf(Promise);
+  expect(await result).toBe("world");
+});
+
+test("domain.run() returns the function's return value", () => {
+  const d = domain.create();
+
+  const result = d.run(() => 99);
+  expect(result).toBe(99);
+});
+
+test("domain.run() returns the async function's promise", async () => {
+  const d = domain.create();
+
+  const result = d.run(async () => "done");
+  expect(result).toBeInstanceOf(Promise);
+  expect(await result).toBe("done");
+});