Conversation
split from other deploy PR
There was a problem hiding this comment.
Pull request overview
This PR fixes production build issues with WalletConnect/AppKit by moving the initialization from the root app.vue component to a lazy-loaded pattern within the useAppKit composable. The changes implement SSR-safe initialization and create plain chain objects to avoid Viem Proxy serialization issues during build.
Changes:
- Removed AppKit initialization from app.vue root component
- Implemented lazy initialization in useAppKit composable with SSR guards
- Added module-level state tracking to prevent duplicate initialization
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| packages/auth-server/app.vue | Removed createAppKit initialization and related imports from root component |
| packages/auth-server/composables/useAppKit.ts | Added lazy initialization logic with SSR guards, plain chain object creation, and module-level state tracking |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
Visit the preview URL for this PR (updated for commit 3c46d13): https://zksync-auth-server-staging--pr260-fix-prod-8mql8pr4.web.app (expires Thu, 22 Jan 2026 15:47:46 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 509a9c9ea42583076f531c53cf2979c544d5d0b7 |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
matched other PR
Description
Move dynamic imports
Additional context
fixes prod, using preview build:
