feat: waku migration #26
Merged
+2,899
−3,666
Conversation
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
- Use process.env instead of import.meta.env for TEMPO_ENV - Remove import.meta.env fallback for INDEXER_API_KEY - Add process.env fallback for PRESTO_RPC_AUTH in local dev - Fix indentation in fetchAssets Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Wrap Layout with client providers instead of wrapping children inside Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add src/lib/server/env.ts with shared env utilities - Extract getTempoEnv, getRpcAuth, getIndexerApiKey helpers - Add getRpcUrl, shouldUseAuth, getChainId pure functions - Update transactions, assets, access-keys, onramp server files - Remove ~30 lines of duplicated cloudflare/process.env logic Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
struong
reviewed
Jan 19, 2026
struong
reviewed
Jan 19, 2026
* checks * checks * cloudflare * refactor: use config module for onramp instead of direct env access * feat: add useTestOtpCode config for onramp OTP bypass in dev/testnet * refactor: extract env helpers for fulfill.ts * chore: pnpm check * chore: delete redundant rpc.ts --------- Co-authored-by: Steven Truong <struong@users.noreply.github.com> Co-authored-by: Pierre Bertet <hi@bpier.re>
📦 Bundle Size Report📉 Total: 2.8 MB (-533.1 KB (-15.5%))
Top 10 chunks
|
## Summary - Consolidate 4 deploy workflows into 2 using matrix strategy - Create reusable deploy action shared by preview and prod - Fix PR comment race condition (was showing only one env) - Remove duplicate verify run on PRs - Remove build from verify (deploys build anyway) - saves 1 build cycle - Improve deployment table format ## Before/After **Before:** - 4 workflows + 1 action (436 lines) - Verify runs twice on PRs - 3 builds per PR (verify + 2 deploys) - Race condition on PR comment **After:** - 2 workflows + 1 action (177 lines) - Verify runs once - 2 builds per PR (deploy only) - Single PR comment with both envs
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.
https://waku.gg/
https://waku.gg/blog/cloudflare-workers-out-of-the-box