fix: keyedParameters use unique key

Author: liangskyli

packages/openapi-typescript/src/transform/path-item-object.ts

@@ -53,7 +53,8 @@ export default function transformPathItemObject(pathItem: PathItemObject, option
     if (!("$ref" in operationObject)) {
       // important: OperationObject parameters come last, and will override any conflicts with PathItem parameters
       for (const parameter of [...(pathItem.parameters ?? []), ...(operationObject.parameters ?? [])]) {
-        const name = "$ref" in parameter ? options.ctx.resolve<ParameterObject>(parameter.$ref)?.name : parameter.name;
+        // fix: #1798, use unique key
+        const name = "$ref" in parameter ? options.ctx.resolve<ParameterObject>(parameter.$ref)?.name : `${parameter.in}-${parameter.name}`;
         if (name) {
           keyedParameters[name] = parameter;
         }

packages/openapi-typescript/test/transform/path-item-object.test.ts

@@ -359,6 +359,66 @@ describe("transformPathItemObject", () => {
         },
       },
     ],
+    [
+      "parameters > header and cookie have same name param1",
+      {
+        given: {
+          get: {
+            parameters: [
+              {
+                "name": "param1",
+                "in": "header",
+                "required": true,
+                "schema": {
+                  "type": "string"
+                },
+                "description": "param1"
+              },
+              {
+                "name": "param1",
+                "in": "cookie",
+                "required": true,
+                "schema": {
+                  "type": "string"
+                },
+                "description": "param1"
+              }
+            ]
+          },
+        },
+        want: `{
+    parameters: {
+        query?: never;
+        header?: never;
+        path?: never;
+        cookie?: never;
+    };
+    get: {
+        parameters: {
+            query?: never;
+            header: {
+                /** @description param1 */
+                param1: string;
+            };
+            path?: never;
+            cookie: {
+                /** @description param1 */
+                param1: string;
+            };
+        };
+        requestBody?: never;
+        responses: never;
+    };
+    put?: never;
+    post?: never;
+    delete?: never;
+    options?: never;
+    head?: never;
+    patch?: never;
+    trace?: never;
+}`,
+      },
+    ],
   ];
 
   for (const [testName, { given, want, options = DEFAULT_OPTIONS, ci }] of tests) {