Merge remote-tracking branch 'origin/main' into support-custom-path-serializer

Author: luxass

.changeset/empty-rooms-attack.md

@@ -0,0 +1,5 @@
+---
+"openapi-fetch": patch
+---
+
+Fix Request parameter being ignored by client methods

.github/workflows/issue-stale.yml

@@ -8,7 +8,7 @@ jobs:
   close-issues:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/stale@v9
+      - uses: actions/stale@v10
         with:
           stale-issue-message: "This issue is stale because it has been open for 180 days with no activity. If there is no activity in the next 7 days, the issue will be closed."
           close-issue-message: "This issue was closed because it has been inactive for 7 days since being marked as stale. Please open a new issue if you believe you are encountering a related problem."

packages/openapi-fetch/package.json

@@ -65,7 +65,7 @@
     "axios": "^1.12.2",
     "execa": "catalog:",
     "express": "^5.1.0",
-    "feature-fetch": "^0.0.43",
+    "feature-fetch": "^0.0.51",
     "node-forge": "^1.3.1",
     "openapi-typescript": "workspace:^",
     "openapi-typescript-codegen": "^0.29.0",

packages/openapi-fetch/src/index.js

@@ -113,7 +113,7 @@ export default function createClient(clientOptions) {
 
     let id;
     let options;
-    let request = new CustomRequest(
+    let request = new Request(
       createFinalURL(schemaPath, { baseUrl: finalBaseUrl, params, querySerializer, pathSerializer }),
       requestInit,
     );
@@ -148,7 +148,7 @@ export default function createClient(clientOptions) {
             id,
           });
           if (result) {
-            if (result instanceof CustomRequest) {
+            if (result instanceof Request) {
               request = result;
             } else if (result instanceof Response) {
               response = result;

packages/openapi-fetch/test/common/request.test.ts

@@ -296,7 +296,7 @@ describe("request", () => {
   });
 
   test("uses provided Request class", async () => {
-    // santity check to make sure the profided fetch function is actually called
+    // sanity check to make sure the provided fetch function is actually called
     expect.assertions(1);
 
     class SpecialRequestImplementation extends Request {}
@@ -316,6 +316,26 @@ describe("request", () => {
     await client.GET("/resources");
   });
 
+  test("Can use custom Request class", async () => {
+    // sanity check to make sure the provided fetch function is actually called
+    expect.assertions(1);
+
+    class SpecialRequestImplementation extends Request {}
+
+    const customFetch = async (input: Request) => {
+      // make sure that the request is actually an instance of the custom request we provided
+      expect(input).instanceOf(SpecialRequestImplementation);
+      return Promise.resolve(Response.json({ hello: "world" }));
+    };
+
+    const client = createClient<paths>({
+      baseUrl: "https://fakeurl.example",
+      fetch: customFetch,
+    });
+
+    await client.GET("/resources", { Request: SpecialRequestImplementation });
+  });
+
   test("can attach custom properties to request", async () => {
     function createCustomFetch(data: any) {
       const response = {