Better typesafety for useInfiniteQuery

[patrickariel]

Changes

Makes pageParamName typesafe and required. This PR also removes the fallbacks for pageParamName and pageParam, because they shouldn't have default values in the first place (and they're also no longer needed).

The should use custom cursor params test is also removed, because I believe that it shouldn't be allowed now that the param is typesafe. Users can always cast the type if they think the schema isn't right.

How to Review

Most of the changes here are type-level. I'm still not very familiar with the type utilities, so if there are any mistakes let me know.

Checklist

• Unit tests updated
• docs/ updated (if necessary)
• pnpm run update:examples run (only applicable for openapi-typescript)

[netlify]

👷 Deploy request for openapi-ts pending review.

Visit the deploys page to approve it

Name	Link
🔨 Latest commit	565f502

[changeset-bot]

⚠️ No Changeset found

Latest commit: 565f502

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[minchodang]

Yes, I agree with this PR.
Initially, when I looked at the design of useInfiniteQuery, the pageParamName contains cursor by default.
So, when I tested it, queryParams automatically contains cursor on default request.

[lukasedw]

Maybe here we could use a different param name other than the default one?

[patrickariel]

Hmm, not sure what you mean by "the default one". The schema only provides cursor and limit as parameters, so any other param name here will trigger a type error (due to the changes in this PR). Or are you suggesting a separate schema for the second test?