fix: preserve explicit null terminal abort reasons

CHANGELOG.md

@@ -6,6 +6,8 @@ All notable changes to `@openclaw/libterminal` will be documented in this file.
 
 ### Fixed
 
+- Preserve explicitly null cancellation reasons in browser terminal startup and Node PTY creation.
+
 - Recheck whether the application still needs a terminal hub connection when a delayed reconnect fires.
 
 - Split large Worker output acknowledgements into decodable chunks so flow-control accounting stays valid above one MiB.

src/browser.ts

@@ -304,7 +304,7 @@ async function loadRuntimeFromModule(
 
 function throwIfAborted(signal?: AbortSignal): void {
   if (signal?.aborted) {
-    throw signal.reason ?? new DOMException("The operation was aborted", "AbortError");
+    throw signal.reason;
   }
 }
 

src/node.ts

@@ -386,7 +386,7 @@ async function findBrowserExternalAsset(distPath: string): Promise<string> {
 
 function throwIfAborted(signal?: AbortSignal): void {
   if (signal?.aborted) {
-    throw signal.reason ?? new Error("The operation was aborted");
+    throw signal.reason;
   }
 }
 

test/browser.test.ts

@@ -67,6 +67,27 @@ describe("createGhosttyTerminal", () => {
     await expect(created).rejects.toBe(reason);
   });
 
+  it("preserves an explicit null abort reason during startup", async () => {
+    const abort = new AbortController();
+    const dispose = vi.fn();
+    class Terminal extends TestGhosttyTerminal {
+      dispose = dispose;
+    }
+    await expect(
+      createGhosttyTerminal({
+        parent: {} as HTMLElement,
+        runtime: { ghostty: {}, Terminal, FitAddon: TestGhosttyFitAddon } as never,
+        signal: abort.signal,
+        onStatus: ({ state }) => {
+          if (state === "ready") {
+            abort.abort(null);
+          }
+        },
+      }),
+    ).rejects.toBeNull();
+    expect(dispose).toHaveBeenCalledOnce();
+  });
+
   it("removes the caller abort listener when the terminal is disposed", async () => {
     const listeners = new Set<() => void>();
     const signal = {

test/node.test.ts

@@ -14,6 +14,16 @@ import {
 import { collectTerminalOutput, terminalText } from "../src/testing.js";
 
 describe("spawnLocalPty", () => {
+  it("preserves an explicit null abort reason before spawning", async () => {
+    const abort = new AbortController();
+    abort.abort(null);
+    const driver = new FakePtyDriver();
+    await expect(
+      spawnLocalPty({ command: "sh", cwd: "/workspace", driver, signal: abort.signal }),
+    ).rejects.toBeNull();
+    expect(driver.spawnCalls).toBe(0);
+  });
+
   it("exchanges UTF-8 input and observes exit through the native PTY", async ({
     onTestFinished,
   }) => {