Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe changes update esbuild WASM initialization to support in-browser compilation with a multi-CDN fallback strategy for robustness, while removing a specific minimatch version constraint from dependency resolutions. Changes
Sequence DiagramsequenceDiagram
participant Browser
participant CDN1 as unpkg CDN
participant CDN2 as jsdelivr CDN
participant WASM as WebAssembly
participant esbuild
Browser->>Browser: ensureInitialized()
Browser->>Browser: initializeEsbuildWithFallbacks()
Browser->>CDN1: Fetch esbuild.wasm (unpkg)
CDN1-->>Browser: Response
alt Success
Browser->>WASM: WebAssembly.compile()
WASM-->>Browser: Module
Browser->>esbuild: esbuild.initialize({ wasmModule })
esbuild-->>Browser: Ready
else Fetch/Compile Failed
Browser->>CDN2: Fetch esbuild.wasm (jsdelivr)
CDN2-->>Browser: Response
alt Success
Browser->>WASM: WebAssembly.compile()
WASM-->>Browser: Module
Browser->>esbuild: esbuild.initialize({ wasmModule })
esbuild-->>Browser: Ready
else All Failed
Browser-->>Browser: Throw consolidated error
end
end
Estimated code review effort🎯 2 (Simple) | ⏱️ ~15 minutes Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary by CodeRabbit
Improvements
Chores