fix(resolve): if timeout is set, check for abort controller existence

Author: jonluca

.claude/settings.local.json

@@ -1,6 +1,11 @@
 {
   "permissions": {
-    "allow": ["Bash(npx vitest:*)", "Bash(bash:*)", "Bash(node test-debug.js:*)"]
+    "allow": [
+      "Bash(npx vitest:*)",
+      "Bash(bash:*)",
+      "Bash(node test-debug.js:*)",
+      "WebFetch(domain:github.com)"
+    ]
   },
   "enableAllProjectMcpServers": false
 }

lib/resolvers/http.ts

@@ -121,7 +121,7 @@ async function download<S extends object = JSONSchema>(
 async function get<S extends object = JSONSchema>(u: RequestInfo | URL, httpOptions: HTTPResolverOptions<S>) {
   let controller: any;
   let timeoutId: any;
-  if (httpOptions.timeout) {
+  if (httpOptions.timeout && typeof AbortController !== "undefined") {
     controller = new AbortController();
     timeoutId = setTimeout(() => controller.abort(), httpOptions.timeout);
   }

test/specs/resolvers/resolvers.spec.ts

@@ -298,4 +298,64 @@ describe("options.resolve", () => {
       properties: { test: { type: "string" } },
     });
   });
+
+  it("should properly resolve a remote schema", async () => {
+    const mockHttpResolver = {
+      order: 200,
+      canRead: true,
+      safeUrlResolver: false,
+      read() {
+        return {
+          type: "object",
+          properties: {
+            firstName: {
+              type: "string",
+            },
+          },
+        };
+      },
+    };
+
+    const schema = await $RefParser.dereference(
+      {
+        type: "object",
+        required: ["firstName"],
+        properties: {
+          firstName: {
+            $ref: "#/$defs/externalModel/properties/firstName",
+          },
+        },
+        $defs: {
+          externalModel: {
+            $ref: "http://localhost:5000/myModel",
+          },
+        },
+      },
+      {
+        resolve: {
+          http: mockHttpResolver,
+        },
+      } as ParserOptions,
+    );
+
+    expect(schema).to.deep.equal({
+      type: "object",
+      required: ["firstName"],
+      properties: {
+        firstName: {
+          type: "string",
+        },
+      },
+      $defs: {
+        externalModel: {
+          type: "object",
+          properties: {
+            firstName: {
+              type: "string",
+            },
+          },
+        },
+      },
+    });
+  });
 });