Skip to content

[codex] fix SPA useSsrCookies default#607

Draft
cneuralnetwork wants to merge 1 commit intonuxt-modules:mainfrom
cneuralnetwork:codex/spa-use-ssr-cookies-default
Draft

[codex] fix SPA useSsrCookies default#607
cneuralnetwork wants to merge 1 commit intonuxt-modules:mainfrom
cneuralnetwork:codex/spa-use-ssr-cookies-default

Conversation

@cneuralnetwork
Copy link
Copy Markdown

Summary

Fixes #605 by resolving useSsrCookies from Nuxt SSR mode when the option is not explicitly configured.

  • Defaults useSsrCookies to false for ssr: false apps so the client plugin uses the non-SSR Supabase client path and restores the session from storage before auth middleware runs.
  • Preserves explicit useSsrCookies: true or false settings.
  • Documents SPA / CSR-only behavior and the changed default.
  • Adds focused unit coverage for the resolver.

Validation

  • vitest run test/resolveUseSsrCookies.test.ts
  • targeted ESLint on changed code files
  • vue-tsc --noEmit -p tsconfig.json
  • npm run build
  • git diff --check

Note: full npm run lint was blocked locally because the npm-based setup did not generate demo/.nuxt/eslint.config.mjs; the targeted changed-file lint passed.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 2, 2026

@cneuralnetwork is attempting to deploy a commit to the NuxtLabs Team on Vercel.

A member of the Team first needs to authorize it.

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 2, 2026

npm i https://pkg.pr.new/@nuxtjs/supabase@607

commit: 490a371

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Default useSsrCookies: true + ssr: false silently breaks SPA refresh; undocumented

1 participant