Query params shouldn't accept Record<string, unknown>

[jpenna]

Description

The lib is generating a union with Record<string, unknown> for query parameters, which doesn't throw type errors when the query schema changes.

• If one doesn't set any query parameters in the schema, it should have type never.
• If one sets query parameters, it should only accept the configured schema.

I realized it also happens for body if it isn't defined (check the sample code). I think it should be never when it isn't defined.

Maybe add a configuration like --use-never to default any unset value to never.

Reproducible example or configuration

Check App.tsx:73

https://stackblitz.com/edit/hey-api-client-fetch-example-ssbzuv?file=src%2FApp.tsx&view=editor

OpenAPI specification (optional)

Use the example above.

System information (optional)

@hey-api/openapi-ts: v0.53.6

[stackblitz]

Start a new pull request in StackBlitz Codeflow.

• hey-api-client-fetch-example-ssbzuv (Open reproduction)

[mrlubos]

Hi @jpenna, can you try with the experimental parser? I think it might be fixed there https://heyapi.dev/openapi-ts/configuration.html#parser

[jpenna]

@mrlubos Will try it Monday!

[jpenna]

Hello @mrlubos , I just upgraded to the latest version 0.53.12 and added the option for the parser (it is actually experimental_parser with underscore, according to the types).

But it is still including the Record<string, unknown>

 path: Record<string, unknown> & {
    threadId: string;

[mrlubos]

Thanks for confirming, will fix!