Repository navigation
Update dependencies, including SvelteKit 3 - #33
Merged
Merged
Conversation
Kit 2 to 3 and adapter-vercel 6 to 7, plus patch bumps to pg, the Supabase CLI, vite, and vitest.
TypeScript stays at 6: kit 3 requires ^6 and svelte-check still caps it there.
What Kit 3 required:
- svelte.config.js is gone; its options are passed to sveltekit() in vite.config.ts.
- tsconfig.json extends the generated $app/tsconfig and lists its own include.
- $lib is now #lib, declared under imports in package.json. Kit's alias option is deprecated in
favor of the same mechanism, so $database, $routes, and $types became #database, #routes, and
#types. Subpath imports need real file extensions, so each import now names its file.
- $app/environment is $app/env, the last $app/stores uses moved to $app/state, and
$env/static/public is $app/env/public, with the two Supabase variables declared in src/env.ts.
- error(404, { message }) is error(404, message).
- goto options: replaceState to replace, keepFocus: true to reset: false, invalidateAll to
refreshAll. invalidateAll() itself is deprecated in favor of refreshAll().
- data-sveltekit-reload takes 'false' rather than 'off'; Handle comes from @sveltejs/kit/hooks.
- engines.node is >=22.17, Kit 3's floor.
Kit 3 sends rendering errors to the error page where Kit 2 logged them and carried on. That turned
a latent bug into a visible one: a person's page showed "Internal Error" when loaded directly,
because ProfileLink read an organization context that does not exist there, and its table had rows
with no tbody. Both are fixed.
Kit 3 depends on cookie 2, which clears the npm audit warning. The regenerated database types lose
a trailing blank line with Supabase CLI 2.119. Two CI comments that described Kit 2 are updated.
Verified locally: svelte-check clean, 288 unit tests, 208 e2e tests, 74 Playwright tests, build,
vercel build, and format all pass, plus a browser click-through of the dev and production builds.
Co-Authored-By: Claude Fable 5.1 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates
@sveltejs/kit2 → 3 and@sveltejs/adapter-vercel6 → 7, plus patch bumps topg, the Supabase CLI,vite, andvitest. TypeScript stays at 6 because Kit 3 requires^6andsvelte-checkstill caps it there.What Kit 3 required
svelte.config.jsis removed; its options are passed tosveltekit()invite.config.ts.tsconfig.jsonextends the generated$app/tsconfigand lists its owninclude.$libis now#lib, declared underimportsinpackage.json. Kit'saliasoption is deprecated in favor of the same mechanism, so$database,$routes, and$typesbecame#database,#routes, and#types. Subpath imports need real file extensions, so each import now names its file. This is most of the diff.$app/environment→$app/env,$app/stores→$app/state, and$env/static/public→$app/env/public, with the two Supabase variables declared in the newsrc/env.ts.error(404, { message })→error(404, message).gotooptions renamed (replace,reset: false,refreshAll), andinvalidateAll()→refreshAll().engines.nodeis now>=22.17.Bug fix
Kit 3 sends rendering errors to the error page where Kit 2 logged them and carried on. That exposed a latent bug: a person's page showed "Internal Error" when loaded directly, because
ProfileLinkread an organization context that does not exist there, and its table had rows with no<tbody>. Both are fixed.Behavior changes from Kit 3
updated.currentnow detects a new deployment on its own, which drives the existing reload-on-navigate behavior.Verification
Locally:
svelte-checkclean, 288 unit tests, 208 e2e tests, 74 Playwright tests,npm run build,vercel build, and prettier all pass, andnpm auditreports 0 vulnerabilities. A browser click-through of both the dev server and the production build covered login, filters, creating and deleting a role and a change, realtime refresh across two tabs, 404 pages, and a direct load of every reachable page signed in and signed out.Not covered locally: a real Vercel deployment with adapter-vercel 7. Merging to
maindeploys to production, so check the preview deployment first.🤖 Generated with Claude Code