Skip to content

refactor: use viem token constants - #103

Draft
decofe wants to merge 2 commits into
mainfrom
centaur/use-viem-token-constants-1792422700
Draft

decofe wants to merge 2 commits into
mainfrom
centaur/use-viem-token-constants-1792422700

Conversation

@decofe

@decofe decofe commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

Replace duplicated currency contract addresses in application and library code with chain-specific definitions from viem/tokens, preserving the configured assets. Test fixtures retain their literals. Upgrade TanStack Start to 1.168.60 and align its router dependency to resolve critical dependency audit findings. Retain the Worker-compatible server-entry patch and pin compatible seroval/seroval-plugins versions for browser hydration.

Validation: installation, pnpm audit, pnpm check, and pnpm check:types, and pnpm build pass. Local browser tests pass 18/22; the isolated signing failure shows Moderato RPC calls rejected by the sandbox browser certificate trust configuration. Local Worker tests cannot mint the fee payer because the fallback local RPC returns HTTP 400. All GitHub checks pass, including Worker tests using the pinned Tempo Docker image and browser E2E tests.

Prompted by: @brendanjryan

Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
@socket-security

socket-security Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​tanstack/​react-router@​1.169.2 ⏵ 1.170.4193 +110087 +398 +1100
Updated@​tanstack/​react-start@​1.167.65 ⏵ 1.168.6099100 +7588 +597 -1100

View full report

Co-authored-by: Derek Cofausper <256792747+decofe@users.noreply.github.com>
@decofe
decofe force-pushed the centaur/use-viem-token-constants-1792422700 branch from ed21537 to 51f84a7 Compare October 19, 2026 18:29

This branch has not been deployed

No deployments
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.

2 participants