fix: path parameters always win over same-named query parameters in generated TypeScript interfaces

When a PathItem defines both a path parameter and a query parameter with the
same name (e.g. Kubernetes /api/v1/nodes/{name}/proxy/{path}), the previous
implementation used the parameter name alone as the deduplication key in
generatePropertySignatures. This caused the optional query parameter to
overwrite the required path parameter, producing `path?: string` instead of
the correct `path: string`.

Fix: sort parameters before reducing so that path parameters are processed last.
Since they win in the final reduce assignment, their required status is always
preserved in the generated TypeScript interface.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: Himenon

src/internal/OpenApiTools/components/Parameter.ts

@@ -109,7 +109,16 @@ export const generatePropertySignatures = (
   context: ToTypeNode.Context,
   converterContext: ConverterContext.Types,
 ): string[] => {
-  const typeElementMap = parameters.reduce<Record<string, string>>((all, parameter) => {
+  // Path parameters must be processed last so they win over same-named query/header
+  // parameters when building the TypeScript interface (path params are always required).
+  const sorted = [...parameters].sort((a, b): number => {
+    const aIsPath = !Guard.isReference(a) && a.in === "path";
+    const bIsPath = !Guard.isReference(b) && b.in === "path";
+    if (aIsPath && !bIsPath) return 1;
+    if (!aIsPath && bIsPath) return -1;
+    return 0;
+  });
+  const typeElementMap = sorted.reduce<Record<string, string>>((all, parameter) => {
     const { name, typeElement } = generatePropertySignatureObject(
       entryPoint,
       currentPoint,

test/__tests__/class/__snapshots__/cloudflare-test.ts.snap

@@ -48,8 +48,8 @@ export interface Response$getReferenceItems$Status$200 {
     };
 }
 export interface Parameter$searchBook {
-    "book.name": string;
     "from.publishedAt"?: number;
+    "book.name": string;
 }
 export interface Response$searchBook$Status$200 {
     "application/json": {

test/__tests__/class/__snapshots__/kubernetes-test.ts.snap

@@ -334,8 +334,8 @@ export interface Response$getReferenceItems$Status$200 {
     };
 }
 export interface Parameter$searchBook {
-    "book.name": string;
     "from.publishedAt"?: number;
+    "book.name": string;
 }
 export interface Response$searchBook$Status$200 {
     "application/json": {
@@ -1004,8 +1004,8 @@ export interface Response$getReferenceItems$Status$200 {
     };
 }
 export interface Parameter$searchBook {
-    "book.name": string;
     "from.publishedAt"?: number;
+    "book.name": string;
 }
 export interface Response$searchBook$Status$200 {
     "application/json": {
@@ -1757,20 +1757,20 @@ export interface Response$getItemById$Status$200 {
     };
 }
 export interface Parameter$getNodeProxyWithPath {
+    path: string;
     name: string;
-    path?: string;
 }
 export interface Response$getNodeProxyWithPath$Status$200 {
     "application/json": string;
 }
 export interface Parameter$headNodeProxyWithPath {
+    path: string;
     name: string;
-    path?: string;
 }
 export interface Parameter$getUserPost {
+    include?: string;
     userId: string;
     postId: number;
-    include?: string;
 }
 export interface Response$getUserPost$Status$200 {
     "application/json": {

test/__tests__/class/__snapshots__/spit-code-test.ts.snap

@@ -48,8 +48,8 @@ export interface Response$getReferenceItems$Status$200 {
     };
 }
 export interface Parameter$searchBook {
-    "book.name": string;
     "from.publishedAt"?: number;
+    "book.name": string;
 }
 export interface Response$searchBook$Status$200 {
     "application/json": {

test/__tests__/class/__snapshots__/typedef-with-template-test.ts.snap

@@ -334,8 +334,8 @@ export interface Response$getReferenceItems$Status$200 {
     };
 }
 export interface Parameter$searchBook {
-    "book.name": string;
     "from.publishedAt"?: number;
+    "book.name": string;
 }
 export interface Response$searchBook$Status$200 {
     "application/json": {
@@ -1010,8 +1010,8 @@ export interface Response$getReferenceItems$Status$200 {
     };
 }
 export interface Parameter$searchBook {
-    "book.name": string;
     "from.publishedAt"?: number;
+    "book.name": string;
 }
 export interface Response$searchBook$Status$200 {
     "application/json": {
@@ -1774,20 +1774,20 @@ export interface Response$getItemById$Status$200 {
     };
 }
 export interface Parameter$getNodeProxyWithPath {
+    path: string;
     name: string;
-    path?: string;
 }
 export interface Response$getNodeProxyWithPath$Status$200 {
     "application/json": string;
 }
 export interface Parameter$headNodeProxyWithPath {
+    path: string;
     name: string;
-    path?: string;
 }
 export interface Parameter$getUserPost {
+    include?: string;
     userId: string;
     postId: number;
-    include?: string;
 }
 export interface Response$getUserPost$Status$200 {
     "application/json": {