fix: Deduplicate additional headers

Author: eliassjogreen

.gitignore

@@ -1,4 +1,5 @@
 npm/
 examples/
 typefetch.d.ts
-tests/*/schemas/*.ts
+tests/*/typefetch.ts
+tests/*/schemas/*.ts

deno.json

@@ -1,6 +1,6 @@
 {
   "name": "@denosaurs/typefetch",
-  "version": "0.0.29",
+  "version": "0.0.30",
   "exports": {
     ".": "./main.ts"
   },
@@ -13,7 +13,7 @@
     "openapi-types": "npm:[email protected]",
     "ts-morph": "npm:[email protected]"
   },
-  "exclude": ["tests/*/schemas/*.ts", "npm/*"],
+  "exclude": ["tests/*/schemas/*.ts", "tests/*/typefetch.ts", "npm/*"],
   "test": {
     "include": ["tests/test.ts"]
   },

mod.ts

@@ -48,7 +48,6 @@ export function escapeObjectKey(key: string): string {
  */
 function toSafeUnionString(
   type: string | undefined,
-  _: number,
   types: (string | undefined)[],
 ): string | undefined {
   if (type === "string" && types.length > 1) {
@@ -110,7 +109,7 @@ export function toSchemaType(
   if (schema.oneOf) {
     return schema.oneOf
       .map((schema) => toSchemaType(document, schema))
-      .map(toSafeUnionString)
+      .map((type, _, types) => toSafeUnionString(type, types))
       .filter(Boolean)
       .join("|");
   }
@@ -131,7 +130,7 @@ export function toSchemaType(
 
     return schema.anyOf
       .map((schema) => toSchemaType(document, schema))
-      .map(toSafeUnionString)
+      .map((type, _, types) => toSafeUnionString(type, types))
       .filter(Boolean)
       .join("|");
   }
@@ -431,17 +430,25 @@ export function toHeadersInitType(
   parameters: ParameterObjectMap,
   additionalHeaders: string[] = [],
 ): string | undefined {
-  const headersInitProperties = [...additionalHeaders];
+  const headersInitProperties = [];
 
   for (const parameter of parameters.values()) {
     if (parameter.in !== "header") continue;
+
+    // If the header has a predefined default in the additionalHeaders argument discard it
+    additionalHeaders = additionalHeaders.filter((header) =>
+      !header.startsWith(`"${parameter.name}"`)
+    );
+
     headersInitProperties.push(
       `"${parameter.name}"${parameter.required ? "" : "?"}: ${
         toSchemaType(document, parameter.schema) ?? "string"
       }`,
     );
   }
 
+  headersInitProperties.unshift(...additionalHeaders);
+
   if (headersInitProperties.length === 0) return undefined;
   return `TypedHeadersInit<{ ${headersInitProperties.join("; ")} }>`;
 }

tests/test.ts

@@ -1,7 +1,7 @@
 import { walk } from "jsr:@std/fs";
 import { assert } from "jsr:@std/assert";
 
-import { globToRegExp } from "@std/path";
+import { dirname, globToRegExp } from "@std/path";
 
 const pwd = import.meta.resolve("../");
 const main = import.meta.resolve("../main.ts");
@@ -22,6 +22,30 @@ Deno.test("Generate schemas", async ({ step }) => {
       assert(output.success);
     });
   }
+
+  for await (
+    const schema of walk("./tests/", {
+      match: [globToRegExp("**/schema.ts")],
+    })
+  ) {
+    const { default: path } = await import(`../${schema.path}`);
+    const dir = dirname(schema.path);
+    await step(`Generating schema for ${path}`, async () => {
+      const output = await new Deno.Command("deno", {
+        args: [
+          "run",
+          "-A",
+          main,
+          `-o=${dir}/typefetch.ts`,
+          `--import=${pwd}`,
+          path,
+        ],
+        stdout: "inherit",
+        stderr: "inherit",
+      }).output();
+      assert(output.success);
+    });
+  }
 });
 
 Deno.test("Check types", async ({ step }) => {

tests/tmdb3/schema.ts

@@ -0,0 +1 @@
+export default "https://developer.themoviedb.org/openapi/64542913e1f86100738e227f";

tests/tmdb4/schema.ts

@@ -0,0 +1 @@
+export default "https://developer.themoviedb.org/openapi/6453cc549c91cf004cd2a015";