@codewithagents/openapi-react-query 3.3.1 → 3.5.0

package/src/__tests__/hooks.test.ts

@@ -17,7 +17,7 @@ describe('generateHooks — task-hooks.json fixture', () => {
 
   it('contains auto-generated header comment', () => {
     expect(content).toContain(
-      '// This file is auto-generated by @codewithagents/openapi-react-query — do not edit',
+      '// This file is auto-generated by @codewithagents/openapi-react-query — do not edit'
     )
   })
 
@@ -40,8 +40,8 @@ describe('generateHooks — task-hooks.json fixture', () => {
   it('generates taskKeys factory with all(), list(), and detail() entries', () => {
     expect(content).toContain('export const taskKeys')
     expect(content).toContain("all: () => ['tasks'] as const")
-    expect(content).toContain("list:")
-    expect(content).toContain("detail:")
+    expect(content).toContain('list:')
+    expect(content).toContain('detail:')
   })
 
   it('useListTasks uses taskKeys.list(params) as queryKey', () => {
@@ -269,13 +269,19 @@ describe('generateHooks — Bug #53: key factory includes query params when oper
   })
 
   it('key factory detail entry includes required params when path + required query params', () => {
-    const { content } = generateHooks(specWithPathAndRequiredQueryParams, { staleTime: 0, gcTime: 0 })
+    const { content } = generateHooks(specWithPathAndRequiredQueryParams, {
+      staleTime: 0,
+      gcTime: 0,
+    })
     // Key factory should accept required params (no ?)
     expect(content).toContain('(id: string, params: Parameters<typeof getReport>[1])')
   })
 
   it('hook marks params as required when path param + required query param', () => {
-    const { content } = generateHooks(specWithPathAndRequiredQueryParams, { staleTime: 0, gcTime: 0 })
+    const { content } = generateHooks(specWithPathAndRequiredQueryParams, {
+      staleTime: 0,
+      gcTime: 0,
+    })
     expect(content).toContain('params: Parameters<typeof getReport>[1]')
     expect(content).not.toContain('params?: Parameters<typeof getReport>[1]')
   })
@@ -284,25 +290,43 @@ describe('generateHooks — Bug #53: key factory includes query params when oper
 // ── Bug #3: Mutation hook argument shapes ───────────────────────────────────────
 
 describe('generateHooks — Bug #54: mutation hook handles all argument shapes', () => {
-  const makeSpec = (pathSegment: string, method: string, hasBody: boolean, queryParams: Array<{name: string; required?: boolean}>): OpenAPIV3_1.Document => ({
+  const makeSpec = (
+    pathSegment: string,
+    method: string,
+    hasBody: boolean,
+    queryParams: Array<{ name: string; required?: boolean }>
+  ): OpenAPIV3_1.Document => ({
     openapi: '3.1.0',
     info: { title: 'Test', version: '1.0.0' },
     paths: {
       [pathSegment]: {
         [method]: {
           operationId: `${method}Op`,
-          parameters: queryParams.map((p) => ({ name: p.name, in: 'query' as const, required: p.required, schema: { type: 'string' as const } })),
-          ...(hasBody ? {
-            requestBody: {
-              required: true,
-              content: { 'application/json': { schema: { $ref: '#/components/schemas/Body' } } },
-            },
-          } : {}),
+          parameters: queryParams.map((p) => ({
+            name: p.name,
+            in: 'query' as const,
+            required: p.required,
+            schema: { type: 'string' as const },
+          })),
+          ...(hasBody
+            ? {
+                requestBody: {
+                  required: true,
+                  content: {
+                    'application/json': { schema: { $ref: '#/components/schemas/Body' } },
+                  },
+                },
+              }
+            : {}),
           responses: { '200': { description: 'ok' } },
         },
       },
     },
-    components: { schemas: { Body: { type: 'object' as const, properties: { name: { type: 'string' as const } } } } },
+    components: {
+      schemas: {
+        Body: { type: 'object' as const, properties: { name: { type: 'string' as const } } },
+      },
+    },
   })
 
   it('mutation with query params only (no body, no path): variables type is Parameters[0], mutationFn is (vars) => fn(vars)', () => {
@@ -344,7 +368,9 @@ describe('generateHooks — Bug #54: mutation hook handles all argument shapes',
           },
         },
       },
-      components: { schemas: { Body: { type: 'object', properties: { name: { type: 'string' } } } } },
+      components: {
+        schemas: { Body: { type: 'object', properties: { name: { type: 'string' } } } },
+      },
     }
     const { content } = generateHooks(spec, { staleTime: 0, gcTime: 0 })
     // Variables includes id, body, and params
@@ -391,13 +417,17 @@ describe('generateHooks — Bug #54: mutation hook handles all argument shapes',
             operationId: 'createTask',
             requestBody: {
               required: true,
-              content: { 'application/json': { schema: { $ref: '#/components/schemas/CreateTask' } } },
+              content: {
+                'application/json': { schema: { $ref: '#/components/schemas/CreateTask' } },
+              },
             },
             responses: { '201': { description: 'created' } },
           },
         },
       },
-      components: { schemas: { CreateTask: { type: 'object', properties: { title: { type: 'string' } } } } },
+      components: {
+        schemas: { CreateTask: { type: 'object', properties: { title: { type: 'string' } } } },
+      },
     }
     const { content } = generateHooks(spec, { staleTime: 0, gcTime: 0 })
     expect(content).toContain('Parameters<typeof createTask>[0]')
@@ -414,7 +444,14 @@ describe('generateHooks — tag name to camelCase key factory', () => {
       [`/${pathSegment}`]: {
         get: {
           operationId: `list${pathSegment}`,
-          responses: { '200': { description: 'ok', content: { 'application/json': { schema: { type: 'array', items: { type: 'object' } } } } } },
+          responses: {
+            '200': {
+              description: 'ok',
+              content: {
+                'application/json': { schema: { type: 'array', items: { type: 'object' } } },
+              },
+            },
+          },
         },
       },
     },
@@ -514,6 +551,130 @@ describe('generateHooks — Feature #60: suspense query variants', () => {
     const { content } = generateHooks(suspenseSpec, { staleTime: 0, gcTime: 0, suspense: false })
     expect(content).not.toContain('useSuspenseQuery')
   })
+
+  it('suspense hook with 1 path param + optional query param generates params? signature', () => {
+    // Covers the `pathParams.length === 1 && hasQueryParams` branch (hooks.ts line 309-310)
+    // and the `paramsRequired === false` (false) branch at line 296 → paramsToken = 'params?'
+    const spec: OpenAPIV3_1.Document = {
+      openapi: '3.1.0',
+      info: { title: 'Test', version: '1.0.0' },
+      paths: {
+        '/repos/{repoId}/commits': {
+          get: {
+            operationId: 'listRepoCommits',
+            parameters: [
+              { name: 'repoId', in: 'path', required: true, schema: { type: 'string' } },
+              { name: 'branch', in: 'query', required: false, schema: { type: 'string' } },
+            ],
+            responses: { '200': { description: 'ok' } },
+          },
+        },
+      },
+    }
+    const { content } = generateHooks(spec, { staleTime: 0, gcTime: 0, suspense: true })
+    expect(content).toContain('export function useSuspenseListRepoCommits')
+    expect(content).toContain('repoId: string')
+    expect(content).toContain('params?:')
+  })
+
+  it('suspense hook with 1 path param + required query param generates params (non-optional) signature', () => {
+    // Covers the `paramsRequired === true` (true) branch at line 296 → paramsToken = 'params'
+    const spec: OpenAPIV3_1.Document = {
+      openapi: '3.1.0',
+      info: { title: 'Test', version: '1.0.0' },
+      paths: {
+        '/search/{category}': {
+          get: {
+            operationId: 'searchCategory',
+            parameters: [
+              { name: 'category', in: 'path', required: true, schema: { type: 'string' } },
+              { name: 'q', in: 'query', required: true, schema: { type: 'string' } },
+            ],
+            responses: { '200': { description: 'ok' } },
+          },
+        },
+      },
+    }
+    const { content } = generateHooks(spec, { staleTime: 0, gcTime: 0, suspense: true })
+    expect(content).toContain('export function useSuspenseSearchCategory')
+    expect(content).toContain('category: string')
+    // Required query param → non-optional params token (no ?)
+    expect(content).toContain('params: Parameters')
+  })
+
+  it('suspense hook with 2 path params builds multi-param queryKey call', () => {
+    // Covers the multi-path-param (pathParams.length > 1) queryKey branch in suspense generation
+    const spec: OpenAPIV3_1.Document = {
+      openapi: '3.1.0',
+      info: { title: 'Test', version: '1.0.0' },
+      paths: {
+        '/orgs/{orgId}/repos/{repoId}': {
+          get: {
+            operationId: 'getOrgRepo',
+            parameters: [
+              { name: 'orgId', in: 'path', required: true, schema: { type: 'string' } },
+              { name: 'repoId', in: 'path', required: true, schema: { type: 'string' } },
+            ],
+            responses: { '200': { description: 'ok' } },
+          },
+        },
+      },
+    }
+    const { content } = generateHooks(spec, { staleTime: 0, gcTime: 0, suspense: true })
+    expect(content).toContain('export function useSuspenseGetOrgRepo')
+    // Both path params appear in the function signature
+    expect(content).toContain('orgId: string')
+    expect(content).toContain('repoId: string')
+  })
+
+  it('suspense hook with 2 path params + query params builds correct queryKey', () => {
+    // Covers the else branch: pathParams.length > 1 && hasQueryParams
+    const spec: OpenAPIV3_1.Document = {
+      openapi: '3.1.0',
+      info: { title: 'Test', version: '1.0.0' },
+      paths: {
+        '/orgs/{orgId}/repos/{repoId}/commits': {
+          get: {
+            operationId: 'listOrgRepoCommits',
+            parameters: [
+              { name: 'orgId', in: 'path', required: true, schema: { type: 'string' } },
+              { name: 'repoId', in: 'path', required: true, schema: { type: 'string' } },
+              { name: 'branch', in: 'query', required: false, schema: { type: 'string' } },
+            ],
+            responses: { '200': { description: 'ok' } },
+          },
+        },
+      },
+    }
+    const { content } = generateHooks(spec, { staleTime: 0, gcTime: 0, suspense: true })
+    expect(content).toContain('export function useSuspenseListOrgRepoCommits')
+    expect(content).toContain('orgId: string')
+    expect(content).toContain('repoId: string')
+    expect(content).toContain('params?:')
+  })
+
+  it('deprecated GET with suspense: true emits @deprecated JSDoc on the suspense hook', () => {
+    // Covers the `if (op.deprecated)` branch in suspense hook generation
+    const spec: OpenAPIV3_1.Document = {
+      openapi: '3.1.0',
+      info: { title: 'Test', version: '1.0.0' },
+      paths: {
+        '/legacy': {
+          get: {
+            operationId: 'listLegacy',
+            deprecated: true,
+            responses: { '200': { description: 'ok' } },
+          },
+        },
+      },
+    }
+    const { content } = generateHooks(spec, { staleTime: 0, gcTime: 0, suspense: true })
+    const suspenseIdx = content.indexOf('export function useSuspenseListLegacy')
+    expect(suspenseIdx).toBeGreaterThan(-1)
+    // @deprecated comment appears immediately before the function declaration
+    const before = content.slice(0, suspenseIdx)
+    expect(before.slice(before.lastIndexOf('/**'))).toContain('@deprecated')
+  })
 })
 
 // ── Feature #59: Per-resource staleTime and gcTime overrides ───────────────────
@@ -622,7 +783,11 @@ describe('generateHooks — Bug #75: auto-invalidation skips detail when multipl
           parameters: [{ name: 'id', in: 'path', required: true, schema: { type: 'string' } }],
           requestBody: {
             required: true,
-            content: { 'application/json': { schema: { type: 'object', properties: { name: { type: 'string' } } } } },
+            content: {
+              'application/json': {
+                schema: { type: 'object', properties: { name: { type: 'string' } } },
+              },
+            },
           },
           responses: { '200': { description: 'ok' } },
         },
@@ -638,13 +803,21 @@ describe('generateHooks — Bug #75: auto-invalidation skips detail when multipl
   }
 
   it('key factory does NOT use "detail:" when there are multiple detail ops', () => {
-    const { content } = generateHooks(multiDetailSpec, { staleTime: 0, gcTime: 0, autoInvalidate: true })
+    const { content } = generateHooks(multiDetailSpec, {
+      staleTime: 0,
+      gcTime: 0,
+      autoInvalidate: true,
+    })
     // With 2 detail ops (getItem + getItemSummary), the key factory uses operation-derived names
     expect(content).not.toContain('detail:')
   })
 
   it('mutation hook does NOT reference itemKeys.detail( when there are multiple detail ops', () => {
-    const { content } = generateHooks(multiDetailSpec, { staleTime: 0, gcTime: 0, autoInvalidate: true })
+    const { content } = generateHooks(multiDetailSpec, {
+      staleTime: 0,
+      gcTime: 0,
+      autoInvalidate: true,
+    })
     const updateStart = content.indexOf('export function useUpdateItem')
     const updateEnd = content.indexOf('\n}', updateStart) + 2
     const updateContent = content.slice(updateStart, updateEnd)
@@ -652,7 +825,11 @@ describe('generateHooks — Bug #75: auto-invalidation skips detail when multipl
   })
 
   it('mutation hook DOES contain itemKeys.all() for broad invalidation', () => {
-    const { content } = generateHooks(multiDetailSpec, { staleTime: 0, gcTime: 0, autoInvalidate: true })
+    const { content } = generateHooks(multiDetailSpec, {
+      staleTime: 0,
+      gcTime: 0,
+      autoInvalidate: true,
+    })
     const updateStart = content.indexOf('export function useUpdateItem')
     const updateEnd = content.indexOf('\n}', updateStart) + 2
     const updateContent = content.slice(updateStart, updateEnd)
@@ -768,7 +945,11 @@ describe('feature interactions', () => {
 
 describe('auto-invalidate mutations (#58)', () => {
   it('autoInvalidate: false — no useQueryClient or invalidateQueries in output', () => {
-    const { content } = generateHooks(spec, { staleTime: 0, gcTime: 300_000, autoInvalidate: false })
+    const { content } = generateHooks(spec, {
+      staleTime: 0,
+      gcTime: 300_000,
+      autoInvalidate: false,
+    })
     expect(content).not.toContain('useQueryClient')
     expect(content).not.toContain('invalidateQueries')
   })
@@ -851,7 +1032,14 @@ describe('mutation hook variables type — all 12 shapes', () => {
           operationId: 'postC',
           requestBody: {
             required: true,
-            content: { 'application/json': { schema: { type: 'object' as const, properties: { name: { type: 'string' as const } } } } },
+            content: {
+              'application/json': {
+                schema: {
+                  type: 'object' as const,
+                  properties: { name: { type: 'string' as const } },
+                },
+              },
+            },
           },
           responses: { '201': { description: 'created' } },
         },
@@ -863,7 +1051,14 @@ describe('mutation hook variables type — all 12 shapes', () => {
           parameters: [{ name: 'dryRun', in: 'query', schema: { type: 'string' } }],
           requestBody: {
             required: true,
-            content: { 'application/json': { schema: { type: 'object' as const, properties: { name: { type: 'string' as const } } } } },
+            content: {
+              'application/json': {
+                schema: {
+                  type: 'object' as const,
+                  properties: { name: { type: 'string' as const } },
+                },
+              },
+            },
           },
           responses: { '200': { description: 'ok' } },
         },
@@ -894,7 +1089,14 @@ describe('mutation hook variables type — all 12 shapes', () => {
           parameters: [{ name: 'id', in: 'path', required: true, schema: { type: 'string' } }],
           requestBody: {
             required: true,
-            content: { 'application/json': { schema: { type: 'object' as const, properties: { name: { type: 'string' as const } } } } },
+            content: {
+              'application/json': {
+                schema: {
+                  type: 'object' as const,
+                  properties: { name: { type: 'string' as const } },
+                },
+              },
+            },
           },
           responses: { '200': { description: 'ok' } },
         },
@@ -909,7 +1111,14 @@ describe('mutation hook variables type — all 12 shapes', () => {
           ],
           requestBody: {
             required: true,
-            content: { 'application/json': { schema: { type: 'object' as const, properties: { name: { type: 'string' as const } } } } },
+            content: {
+              'application/json': {
+                schema: {
+                  type: 'object' as const,
+                  properties: { name: { type: 'string' as const } },
+                },
+              },
+            },
           },
           responses: { '200': { description: 'ok' } },
         },
@@ -947,7 +1156,14 @@ describe('mutation hook variables type — all 12 shapes', () => {
           ],
           requestBody: {
             required: true,
-            content: { 'application/json': { schema: { type: 'object' as const, properties: { name: { type: 'string' as const } } } } },
+            content: {
+              'application/json': {
+                schema: {
+                  type: 'object' as const,
+                  properties: { name: { type: 'string' as const } },
+                },
+              },
+            },
           },
           responses: { '200': { description: 'ok' } },
         },
@@ -963,7 +1179,14 @@ describe('mutation hook variables type — all 12 shapes', () => {
           ],
           requestBody: {
             required: true,
-            content: { 'application/json': { schema: { type: 'object' as const, properties: { name: { type: 'string' as const } } } } },
+            content: {
+              'application/json': {
+                schema: {
+                  type: 'object' as const,
+                  properties: { name: { type: 'string' as const } },
+                },
+              },
+            },
           },
           responses: { '200': { description: 'ok' } },
         },
@@ -975,28 +1198,38 @@ describe('mutation hook variables type — all 12 shapes', () => {
 
   it('case 1 (0 params, no body, no query): variablesType is void, mutationFn is () => fn()', () => {
     // UseMutationOptions<..., ApiError, void> — void is the variables type
-    expect(shapesContent).toContain('UseMutationOptions<Awaited<ReturnType<typeof deleteA>>, ApiError, void>')
+    expect(shapesContent).toContain(
+      'UseMutationOptions<Awaited<ReturnType<typeof deleteA>>, ApiError, void>'
+    )
     expect(shapesContent).toContain('mutationFn: () => deleteA()')
   })
 
   it('case 2 (0 params, no body, has query): variablesType is Parameters<typeof fn>[0], mutationFn is (vars) => fn(vars)', () => {
-    expect(shapesContent).toContain('UseMutationOptions<Awaited<ReturnType<typeof postB>>, ApiError, Parameters<typeof postB>[0]>')
+    expect(shapesContent).toContain(
+      'UseMutationOptions<Awaited<ReturnType<typeof postB>>, ApiError, Parameters<typeof postB>[0]>'
+    )
     expect(shapesContent).toContain('mutationFn: (vars) => postB(vars)')
   })
 
   it('case 3 (0 params, has body, no query): variablesType is Parameters<typeof fn>[0], mutationFn is (vars) => fn(vars)', () => {
-    expect(shapesContent).toContain('UseMutationOptions<Awaited<ReturnType<typeof postC>>, ApiError, Parameters<typeof postC>[0]>')
+    expect(shapesContent).toContain(
+      'UseMutationOptions<Awaited<ReturnType<typeof postC>>, ApiError, Parameters<typeof postC>[0]>'
+    )
     expect(shapesContent).toContain('mutationFn: (vars) => postC(vars)')
   })
 
   it('case 4 (0 params, has body, has query): variablesType is { body: ...[0]; params: ...[1] }, mutationFn destructures both', () => {
-    expect(shapesContent).toContain('body: Parameters<typeof postD>[0]; params: Parameters<typeof postD>[1]')
+    expect(shapesContent).toContain(
+      'body: Parameters<typeof postD>[0]; params: Parameters<typeof postD>[1]'
+    )
     expect(shapesContent).toContain('mutationFn: ({ body, params }) => postD(body, params)')
   })
 
   it('case 5 (1 path param, no body, no query): variablesType is string, mutationFn is (id) => fn(id)', () => {
     // string is the entire variables type — only case that uses literal 'string'
-    expect(shapesContent).toContain('UseMutationOptions<Awaited<ReturnType<typeof deleteE>>, ApiError, string>')
+    expect(shapesContent).toContain(
+      'UseMutationOptions<Awaited<ReturnType<typeof deleteE>>, ApiError, string>'
+    )
     expect(shapesContent).toContain('mutationFn: (id) => deleteE(id)')
   })
 
@@ -1011,28 +1244,44 @@ describe('mutation hook variables type — all 12 shapes', () => {
   })
 
   it('case 8 (1 path param, has body, has query): variablesType is { id: string; body: ...[1]; params: ...[2] }, mutationFn destructures all', () => {
-    expect(shapesContent).toContain('id: string; body: Parameters<typeof putH>[1]; params: Parameters<typeof putH>[2]')
+    expect(shapesContent).toContain(
+      'id: string; body: Parameters<typeof putH>[1]; params: Parameters<typeof putH>[2]'
+    )
     expect(shapesContent).toContain('mutationFn: ({ id, body, params }) => putH(id, body, params)')
   })
 
   it('case 9 (2+ params, no body, no query): variablesType is { projectId: string; taskId: string }, mutationFn destructures both path params', () => {
     expect(shapesContent).toContain('projectId: string; taskId: string }')
-    expect(shapesContent).toContain('mutationFn: ({ projectId, taskId }) => deleteI(projectId, taskId)')
+    expect(shapesContent).toContain(
+      'mutationFn: ({ projectId, taskId }) => deleteI(projectId, taskId)'
+    )
   })
 
   it('case 10 (2+ params, no body, has query): variablesType includes both path params and params, mutationFn passes all', () => {
-    expect(shapesContent).toContain('projectId: string; taskId: string; params: Parameters<typeof deleteJ>[2]')
-    expect(shapesContent).toContain('mutationFn: ({ projectId, taskId, params }) => deleteJ(projectId, taskId, params)')
+    expect(shapesContent).toContain(
+      'projectId: string; taskId: string; params: Parameters<typeof deleteJ>[2]'
+    )
+    expect(shapesContent).toContain(
+      'mutationFn: ({ projectId, taskId, params }) => deleteJ(projectId, taskId, params)'
+    )
   })
 
   it('case 11 (2+ params, has body, no query): variablesType includes both path params and body at index [2]', () => {
-    expect(shapesContent).toContain('projectId: string; taskId: string; body: Parameters<typeof putK>[2]')
-    expect(shapesContent).toContain('mutationFn: ({ projectId, taskId, body }) => putK(projectId, taskId, body)')
+    expect(shapesContent).toContain(
+      'projectId: string; taskId: string; body: Parameters<typeof putK>[2]'
+    )
+    expect(shapesContent).toContain(
+      'mutationFn: ({ projectId, taskId, body }) => putK(projectId, taskId, body)'
+    )
   })
 
   it('case 12 (2+ params, has body, has query): variablesType includes both path params, body at [2], params at [3]', () => {
-    expect(shapesContent).toContain('projectId: string; taskId: string; body: Parameters<typeof putL>[2]; params: Parameters<typeof putL>[3]')
-    expect(shapesContent).toContain('mutationFn: ({ projectId, taskId, body, params }) => putL(projectId, taskId, body, params)')
+    expect(shapesContent).toContain(
+      'projectId: string; taskId: string; body: Parameters<typeof putL>[2]; params: Parameters<typeof putL>[3]'
+    )
+    expect(shapesContent).toContain(
+      'mutationFn: ({ projectId, taskId, body, params }) => putL(projectId, taskId, body, params)'
+    )
   })
 })
 
@@ -1063,7 +1312,14 @@ describe('auto-invalidate: method-based detail invalidation', () => {
           parameters: [{ name: 'id', in: 'path', required: true, schema: { type: 'string' } }],
           requestBody: {
             required: true,
-            content: { 'application/json': { schema: { type: 'object' as const, properties: { name: { type: 'string' as const } } } } },
+            content: {
+              'application/json': {
+                schema: {
+                  type: 'object' as const,
+                  properties: { name: { type: 'string' as const } },
+                },
+              },
+            },
           },
           responses: { '200': { description: 'ok' } },
         },
@@ -1073,7 +1329,14 @@ describe('auto-invalidate: method-based detail invalidation', () => {
           parameters: [{ name: 'id', in: 'path', required: true, schema: { type: 'string' } }],
           requestBody: {
             required: true,
-            content: { 'application/json': { schema: { type: 'object' as const, properties: { name: { type: 'string' as const } } } } },
+            content: {
+              'application/json': {
+                schema: {
+                  type: 'object' as const,
+                  properties: { name: { type: 'string' as const } },
+                },
+              },
+            },
           },
           responses: { '200': { description: 'ok' } },
         },
@@ -1087,7 +1350,11 @@ describe('auto-invalidate: method-based detail invalidation', () => {
     },
   }
 
-  const { content: thingsContent } = generateHooks(thingsSpec, { staleTime: 0, gcTime: 0, autoInvalidate: true })
+  const { content: thingsContent } = generateHooks(thingsSpec, {
+    staleTime: 0,
+    gcTime: 0,
+    autoInvalidate: true,
+  })
 
   it('thingKeys key factory has a detail entry (exactly one detail GET op)', () => {
     expect(thingsContent).toContain('export const thingKeys')
@@ -1161,7 +1428,10 @@ describe('query hook key builder — 2+ path params with query params', () => {
     },
   }
 
-  const { content: multiParamContent } = generateHooks(multiParamQuerySpec, { staleTime: 0, gcTime: 0 })
+  const { content: multiParamContent } = generateHooks(multiParamQuerySpec, {
+    staleTime: 0,
+    gcTime: 0,
+  })
 
   it('key factory detail entry accepts both path params as typed args', () => {
     expect(multiParamContent).toContain('(projectId: string, taskId: string,')
@@ -1173,7 +1443,9 @@ describe('query hook key builder — 2+ path params with query params', () => {
   })
 
   it('enabled guard ANDs both path params', () => {
-    expect(multiParamContent).toContain('enabled: projectId != null && taskId != null && (options?.enabled ?? true)')
+    expect(multiParamContent).toContain(
+      'enabled: projectId != null && taskId != null && (options?.enabled ?? true)'
+    )
   })
 
   it('queryKey call includes both path params with non-null assertions and query params', () => {
@@ -1193,7 +1465,10 @@ describe('generateHooks — edge cases', () => {
           post: {
             operationId: 'createPet',
             deprecated: true,
-            requestBody: { required: true, content: { 'application/json': { schema: { type: 'object' } } } },
+            requestBody: {
+              required: true,
+              content: { 'application/json': { schema: { type: 'object' } } },
+            },
             responses: { '201': { description: 'Created' } },
           },
         },
@@ -1243,14 +1518,28 @@ describe('generateHooks — edge cases', () => {
         '/items': {
           get: {
             operationId: 'listItems',
-            responses: { '200': { description: 'ok', content: { 'application/json': { schema: { type: 'array', items: { type: 'object' } } } } } },
+            responses: {
+              '200': {
+                description: 'ok',
+                content: {
+                  'application/json': { schema: { type: 'array', items: { type: 'object' } } },
+                },
+              },
+            },
           },
         },
         '/items/search': {
           get: {
             operationId: 'searchItems',
             parameters: [{ name: 'q', in: 'query', schema: { type: 'string' } }],
-            responses: { '200': { description: 'ok', content: { 'application/json': { schema: { type: 'array', items: { type: 'object' } } } } } },
+            responses: {
+              '200': {
+                description: 'ok',
+                content: {
+                  'application/json': { schema: { type: 'array', items: { type: 'object' } } },
+                },
+              },
+            },
           },
         },
       },
@@ -1289,7 +1578,10 @@ describe('Bug #105: cache key collision when multiple GET ops share same path pa
     },
   }
 
-  const { content: multiDetailResult } = generateHooks(specWithMultipleDetailOps, { staleTime: 0, gcTime: 0 })
+  const { content: multiDetailResult } = generateHooks(specWithMultipleDetailOps, {
+    staleTime: 0,
+    gcTime: 0,
+  })
 
   it('both operations have distinct named entries in the key factory', () => {
     expect(multiDetailResult).toContain('getItemById:')
@@ -1380,7 +1672,9 @@ describe('generateHooks — path segment sanitization in deriveOperationName', (
       paths: {
         '/api-keys/{api-key-id}': {
           get: {
-            parameters: [{ name: 'api-key-id', in: 'path', required: true, schema: { type: 'string' } }],
+            parameters: [
+              { name: 'api-key-id', in: 'path', required: true, schema: { type: 'string' } },
+            ],
             responses: { '200': { description: 'ok' } },
           },
         },
@@ -1423,7 +1717,14 @@ describe('generateHooks — toKeyFactoryName sanitizes special chars', () => {
         '/attachments/{attachment-publicUUID}': {
           get: {
             operationId: 'getAttachment',
-            parameters: [{ name: 'attachment-publicUUID', in: 'path', required: true, schema: { type: 'string' } }],
+            parameters: [
+              {
+                name: 'attachment-publicUUID',
+                in: 'path',
+                required: true,
+                schema: { type: 'string' },
+              },
+            ],
             responses: { '200': { description: 'ok' } },
           },
         },
@@ -1452,7 +1753,10 @@ describe('Bug #106: ...options spread must not overwrite auto-invalidation onSuc
         },
         post: {
           operationId: 'createItem',
-          requestBody: { required: true, content: { 'application/json': { schema: { $ref: '#/components/schemas/Item' } } } },
+          requestBody: {
+            required: true,
+            content: { 'application/json': { schema: { $ref: '#/components/schemas/Item' } } },
+          },
           responses: { '201': { description: 'ok' } },
         },
       },
@@ -1491,10 +1795,51 @@ describe('Bug #106: ...options spread must not overwrite auto-invalidation onSuc
   })
 
   it('mutation hook without auto-invalidation is unaffected (no onSuccess generated)', () => {
-    const { content: plain } = generateHooks(specWithAutoInvalidate, { staleTime: 0, gcTime: 0, autoInvalidate: false })
+    const { content: plain } = generateHooks(specWithAutoInvalidate, {
+      staleTime: 0,
+      gcTime: 0,
+      autoInvalidate: false,
+    })
     const hookStart = plain.indexOf('export function useCreateItem')
     const hookEnd = plain.indexOf('\n}', hookStart) + 2
     const hookContent = plain.slice(hookStart, hookEnd)
     expect(hookContent).not.toContain('onSuccess:')
   })
 })
+
+describe('coverage: spec without paths property — generates empty output', () => {
+  it('spec with paths=undefined produces a valid but empty hooks file', () => {
+    // Covers the `if (paths !== undefined)` false branch in generateHooks (line 542)
+    const spec = {
+      openapi: '3.1.0',
+      info: { title: 'Empty', version: '1.0.0' },
+      // no paths property
+    } as OpenAPIV3_1.Document
+    const { content } = generateHooks(spec, { staleTime: 0, gcTime: 0 })
+    expect(content).toBeDefined()
+    // No hooks generated — just the header boilerplate
+    expect(content).not.toContain('export function use')
+  })
+})
+
+describe('coverage: requestBody with no content — hasBody but no bodyTypeName', () => {
+  it('POST with requestBody missing content property produces mutation without typed body', () => {
+    // Covers `if (content === undefined) return { hasBody: true, bodyTypeName: undefined }` (line 112)
+    const spec: OpenAPIV3_1.Document = {
+      openapi: '3.1.0',
+      info: { title: 'Test', version: '1.0.0' },
+      paths: {
+        '/items': {
+          post: {
+            operationId: 'createItem',
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any
+            requestBody: { required: true } as any, // no content property
+            responses: { '201': { description: 'created' } },
+          },
+        },
+      },
+    }
+    const { content } = generateHooks(spec, { staleTime: 0, gcTime: 0 })
+    expect(content).toContain('export function useCreateItem')
+  })
+})