Upgrade to Svelte 5 / SvelteKit 3 and fix broken features - #175
Merged
Merged
Conversation
- Svelte 5 runes, SvelteKit 3, Vite 8, TypeScript 6, adapter-vercel 7 - Skeleton v5 + Tailwind 4 (legacy theme), custom light switch - Replace deprecated Web3Modal with Reown AppKit (ethers5 adapter) - Fetch token balances on-chain (Peanut token list + Multicall3) instead of Ankr - Peanut SDK 0.6.2, wait for each tx so ERC20 approvals land before deposits - Sentry 11 (server init in instrumentation.server.ts), ESLint 9 flat config - Declare env vars in src/env.ts, use #lib subpath imports - Fix simple mode "Send emails" never showing and hardcoded recipient - Fix multi-reward selection, null commit authors, badge URL escaping - Validate /api/mail input, improve email subject/body - Drop dead networks (goerli, mumbai, zkEVM), add sepolia - pnpm only, CI on Node 22 with build step Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Keep the branch's package.json and pnpm-lock.yaml: the dependabot bumps on main are superseded (packages upgraded further or removed). package-lock.json stays deleted. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This branch was successfully deployed
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.
Summary
The project no longer built on current Node, and several features were broken. This PR upgrades the stack to Svelte 5 and fixes the broken features. The feature set stays the same.
Stack changes
legacytheme)SvelteKit 3 migration
svelte.config.js→ options passed tosveltekit()invite.config.ts$lib→#libsubpath imports (explicit extensions)src/env.ts($app/env/private)instrumentation.server.ts@vercel/analytics/sveltekitstill imports$app/stores, so it's replaced by the genericinject()Bug fixes
/api/mailvalidates input (valid email, repo name,peanut.tolink only)Tooling
packageManager), with a build step added.env.exampleaddedTest plan
pnpm run check→ 0 errorspnpm run lintpnpm run build(adapter-vercel)/api/githubhappy path + 400/404;/api/mailvalidation rejects bad inputprepareDepositTxsfor native (1 tx) and USDC (approve + deposit)Deploy notes
ANKR_PROVIDER/COVALENT_API_KEYvars on VercelGITHUB_TOKEN(optional),SMTP_HOST,SMTP_PORT,SMTP_USER,SMTP_PASSWORD🤖 Generated with Claude Code