Fully isolate job from Solid's owner

src/utils.ts

@@ -1,25 +1,22 @@
-import { createRoot, runWithOwner } from "solid-js";
+import { runWithOwner } from "solid-js";
 
 export function isolate<T>(fn: () => T): T {
-  let error;
-  let hasErrored = false;
-  const owner = null as any;
+  let error: unknown;
+  let hasError = false;
 
-  const result = runWithOwner(owner, () => {
-    return createRoot((dispose) => {
-      try {
-        return fn();
-      } catch (e) {
-        hasErrored = true;
-        error = e;
-        return;
-      } finally {
-        dispose();
-      }
-    });
+  const result = runWithOwner(null, () => {
+    try {
+      return fn();
+    } catch (e) {
+      hasError = true;
+      error = e;
+      throw e;
+    }
   })!;
 
-  if (hasErrored) throw error;
+  if (hasError) {
+    throw error;
+  }
 
   return result;
 }

tests/browser/task.test.tsx

@@ -1,10 +1,7 @@
 import { test, expect } from "@playwright/experimental-ct-solid";
 import { TodoList } from "../fixtures/todo-list";
 
-test.only("adds only one todo even when clicked twice", async ({
-  mount,
-  page,
-}) => {
+test("adds only one todo even when clicked twice", async ({ mount, page }) => {
   await mount(<TodoList />);
 
   const addTodo = page.getByRole("button", { name: "Add todo" });

tests/vitest/job.test.ts

@@ -1,7 +1,7 @@
 import { work, timeout } from "../../src/work";
 import { createJob, JobMode } from "../../src/job";
 import { describe, test, expect } from "vitest";
-import { createRoot } from "solid-js";
+import { createRoot, getOwner } from "solid-js";
 
 describe("job", () => {
   describe("#perform", () => {
@@ -119,4 +119,15 @@ describe("job", () => {
       expect(job.isPending).toBe(false);
     });
   });
+
+  test("runs without owner", async () => {
+    await createRoot(async () => {
+      const job = createJob(async () => {
+        expect(getOwner()).toBe(null);
+      });
+
+      expect(getOwner()).not.toBe(null);
+      await job.perform();
+    });
+  });
 });