docs: expand address, provider, wallet, and client guides#54
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR upgrades core dependencies and reorganizes documentation with major version bumps to Next.js 16, React 19.2, and fumadocs packages. The changes include dependency updates, configuration adjustments for lazy loading and TypeScript, and comprehensive documentation restructuring for wallets and providers.
Key Changes:
- Dependency upgrades: Next.js 15.5.0 → 16.0.1, React 19.1.1 → 19.2.0, fumadocs packages to v16
- TypeScript configuration changes for React JSX compilation
- Documentation reorganization with new wallet and provider guides
- Build optimization with lazy loading for Shiki themes/languages
Reviewed Changes
Copilot reviewed 34 out of 37 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| pnpm-lock.yaml | Major dependency version bumps including Next.js, React, fumadocs, and related packages |
| docs/tsconfig.json | Changed JSX from "preserve" to "react-jsx" and added dev types path |
| docs/source.config.ts | Added lazy loading, file system cache, and optimized language configuration |
| docs/next-env.d.ts | Changed from triple-slash reference to import for routes.d.ts |
| docs/package.json | Updated package versions matching lock file changes |
| docs/content/docs/wallets/* | Restructured wallet documentation with new guides |
| docs/content/docs/providers/* | Added new provider use cases and submission documentation |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "resolveJsonModule": true, | ||
| "isolatedModules": true, | ||
| "jsx": "preserve", | ||
| "jsx": "react-jsx", |
There was a problem hiding this comment.
The JSX compiler setting changed from 'preserve' to 'react-jsx'. With Next.js 16 and React 19.2, verify this is the intended configuration. Next.js typically uses 'preserve' to handle JSX transformation itself during the build process. This change may affect how JSX is compiled and could impact build performance or runtime behavior.
| }, | ||
| langs: ['typescript', 'javascript', 'tsx', 'jsx', 'json', 'bash', 'shell', 'sh', 'mermaid'], | ||
| // Explicit language list for lazy loading - add all JS/TS variants | ||
| langs: ['typescript', 'ts', 'tsx', 'javascript', 'js', 'jsx', 'bash', 'sh'], |
There was a problem hiding this comment.
The language list includes redundant aliases ('typescript'/'ts', 'javascript'/'js'). Shiki typically treats these as the same language. Including both may cause unnecessary registration overhead. Consider removing the long-form aliases and keeping only the short forms: ['ts', 'tsx', 'js', 'jsx', 'bash', 'sh'].
d80eb2d to
fc580e1
Compare
Overview
This PR significantly expands the documentation for the Evolution SDK, providing comprehensive guides for developers.
Changes
Stats