fix(types): add pageParam and direction as optional to "normal" queryFn

[TkDodo]

because that's what we get when using a defaultQueryFn

closes #6715

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
query	⬜️ Ignored (Inspect)	Visit Preview		Apr 2, 2024 11:45am

[nx-cloud]

☁️ Nx Cloud Report

CI is running/has finished running commands for commit d227630. As they complete they will appear below. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this CI Pipeline Execution

---

✅ Successfully ran 1 target

• nx affected --targets=test:format,test:sherif,test:knip,test:eslint,test:lib,test:types,test:build,build,test:attw --parallel=3

---

Sent with 💌 from NxCloud.

[TkDodo]

@DamianOsipiuk do you know why this small type change leads to this error?

src/createPersister.ts(81,17): error TS2742: The inferred type of 'experimental_createPersister' cannot be named without a reference to '../node_modules/@tanstack/query-core/build/modern/queryClient-lGFVn5Ps'. This is likely not portable. A type annotation is necessary.

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit d227630:

Sandbox	Source
@tanstack/query-example-angular-basic	Configuration
@tanstack/query-example-react-basic-typescript	Configuration
@tanstack/query-example-solid-basic-typescript	Configuration
@tanstack/query-example-svelte-basic	Configuration
@tanstack/query-example-vue-basic	Configuration

[DamianOsipiuk]

@DamianOsipiuk do you know why this small type change leads to this error?

src/createPersister.ts(81,17): error TS2742: The inferred type of 'experimental_createPersister' cannot be named without a reference to '../node_modules/@tanstack/query-core/build/modern/queryClient-lGFVn5Ps'. This is likely not portable. A type annotation is necessary.

This is potentially due to how tsup is bundling types.
I remember that @Andarist potentially fixed that in TS 5.4.
#6318 (comment)

Potential workaround is to import module from which problematic type is referenced.

More long term fix would be to fix it in tsup or migrate to rollup.

Or depend on users upgrading their TS version 😞

[Andarist]

I remember that @Andarist potentially fixed that in TS 5.4.

that fix has been reverted though (see microsoft/TypeScript#57849 )