Remote Functions: Refreshed queries that throw a `redirect` fail

[AdrianGonz97]

Describe the bug

Refreshed queries that throw a redirect (while being refreshed inside of a form remote function) cause a 500: Failed to execute remote function error on the client, and an uncaught redirect error ([500] POST / Error: {"status":307,"location":"/reset"}) on the server.

Reproduction

1. Visit the StackBlitz repro or run git clone https://github.com/AdrianGonz97/sveltekit-refreshed-query-redirect-error
2. pnpm install && pnpm dev
3. navigate to / and increment the count to be greater than 5

Logs

[500] POST /
Error: {"status":307,"location":"/reset"}
    at coalesce_to_error (/home/koala/Reproductions/refreshed-query-redirect-error/node_modules/.pnpm/@sveltejs+kit@2.49.0_@sveltejs+vite-plugin-svelte@6.2.1_svelte@5.43.14_vite@7.2.4__svelte@5.43.14_vite@7.2.4/node_modules/@sveltejs/kit/src/utils/error.js:11:5)
    at handle_fatal_error (/home/koala/Reproductions/refreshed-query-redirect-error/node_modules/.pnpm/@sveltejs+kit@2.49.0_@sveltejs+vite-plugin-svelte@6.2.1_svelte@5.43.14_vite@7.2.4__svelte@5.43.14_vite@7.2.4/node_modules/@sveltejs/kit/src/runtime/server/utils.js:86:64)
    at resolve (/home/koala/Reproductions/refreshed-query-redirect-error/node_modules/.pnpm/@sveltejs+kit@2.49.0_@sveltejs+vite-plugin-svelte@6.2.1_svelte@5.43.14_vite@7.2.4__svelte@5.43.14_vite@7.2.4/node_modules/@sveltejs/kit/src/runtime/server/respond.js:705:17)
    at process.processTicksAndRejections (node:internal/process/task_queues:105:5)
    at async fn (/home/koala/Reproductions/refreshed-query-redirect-error/node_modules/.pnpm/@sveltejs+kit@2.49.0_@sveltejs+vite-plugin-svelte@6.2.1_svelte@5.43.14_vite@7.2.4__svelte@5.43.14_vite@7.2.4/node_modules/@sveltejs/kit/src/runtime/server/respond.js:422:12)
    at async internal_respond (/home/koala/Reproductions/refreshed-query-redirect-error/node_modules/.pnpm/@sveltejs+kit@2.49.0_@sveltejs+vite-plugin-svelte@6.2.1_svelte@5.43.14_vite@7.2.4__svelte@5.43.14_vite@7.2.4/node_modules/@sveltejs/kit/src/runtime/server/respond.js:404:20)
    at async file:///home/koala/Reproductions/refreshed-query-redirect-error/node_modules/.pnpm/@sveltejs+kit@2.49.0_@sveltejs+vite-plugin-svelte@6.2.1_svelte@5.43.14_vite@7.2.4__svelte@5.43.14_vite@7.2.4/node_modules/@sveltejs/kit/src/exports/vite/dev/index.js:562:22

System Info

System:
    OS: Linux 6.6 Ubuntu 22.04.5 LTS 22.04.5 LTS (Jammy Jellyfish)
    CPU: (24) x64 AMD Ryzen 9 5900X 12-Core Processor
    Memory: 22.60 GB / 31.30 GB
    Container: Yes
    Shell: 5.8.1 - /usr/bin/zsh
  Binaries:
    Node: 22.15.0 - /home/koala/.volta/tools/image/node/22.15.0/bin/node
    Yarn: 1.22.22 - /home/koala/.volta/tools/image/yarn/1.22.22/bin/yarn
    npm: 10.9.2 - /home/koala/.volta/tools/image/node/22.15.0/bin/npm
    pnpm: 10.9.0 - /home/koala/.volta/bin/pnpm
    bun: 1.1.38 - /home/koala/.bun/bin/bun
    Deno: 2.1.1 - /home/koala/.deno/bin/deno
  Browsers:
    Chrome: 127.0.6533.119
  npmPackages:
    @sveltejs/adapter-auto: ^7.0.0 => 7.0.0 
    @sveltejs/kit: ^2.49.0 => 2.49.0 
    @sveltejs/vite-plugin-svelte: ^6.2.1 => 6.2.1 
    svelte: ^5.43.14 => 5.43.14 
    vite: ^7.2.4 => 7.2.4

Severity

annoyance

Additional Information

No response

[dummdidumm]

I do wonder what should even happen in this case.

• Do we redirect automatically? Then what if you have two refreshes and the first one throws, should the second one still be called?
• Do we ignore redirects in this case?

[teemingc]

I don’t really like the idea of redirects in queries. I feel like the component itself should check the result of the query and redirect there. This makes the query more reusable in different locations (unless your expected behaviour is to always redirect no matter where the query is being called from).

I think the only place throwing a redirect makes sense in is the form remote function. It’s probably the one thing you wouldn’t call in other functions.

[dummdidumm]

You cannot redirect inside a component though (at least right now you can't). And it feels a bit weirdly spread out between remote function and component to do it that way.

[teemingc]

Ah true. I know of some people who tried redirect in a component when it's suppose to be goto

[teemingc]

You cannot redirect inside a component though (at least right now you can't).

I wonder if this inadvertently works now because of https://github.com/sveltejs/kit/pull/15222/changes#diff-f8956b07fb67847bf592e4b63a47d10053d7bdd1b484a4897eb385aab567dd2dR360

[Nic-Polumeyv]

This is being worked through in #16192. The current direction is for a redirecting query to stay in that state until it's refreshed, and the implementation is taking shape in #16330.