[query-client] Omit params property when there are no params

[danielpza]

Currently we need to pass an empty object as params even if no param is needed:

  client.test.useQuery(["something"], {
    query: { someQuery },
    params: {}, // cannot omit param object or ts complains
  });

The param property should be (imo) optional when it's not needed

[oliverbutler]

Hey this should be functioning already so I'm going to make as a bug and try write a failing test for this.

Thanks!

[oliverbutler]

Hey @danielpza I can't create a minimal reproduction of this my side

Can you check that you're using strict mode in TS in this app?

Also can you send the contract you used for this EP please

[oliverbutler]

[danielpza]

Hey @oliverbutler, I was able to reproduce it here https://github.com/danielpza/ts-rest-issue-303

Can you check that you're using strict mode in TS in this app?

Yes I'm using strict mode

[oliverbutler]

Thank you! I'll take a look :)

[oliverbutler]

@danielpza I've found the reason your example doesn't work and I'm not seeing the issue my side, Zod.

Right now we rely on Zod for some underlying generics in @ts-rest/core

Installing Zod, even just effectively as a dev dependency should make this work for you.

We had an issue #44 but as of right now this isn't something we're working on removing, although not relying on Zod is definitely a goal of mine for ts-rest.

[oliverbutler]

Closing for right now, but I'll see if we've adequately documented this limitation (I'm doubtful we did!)