Conversation
React 19 deprecated FormEvent in favor of specific event types. Updated form submit handlers to use SubmitEvent.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@cursoragent address CI issue |
|
Fixed the CI failure in What changed: Replaced Verification: All 10 tests pass locally (4 build-output tests are skipped as expected when no dist directory exists). The fix has been committed and pushed to |
import.meta.resolve is not supported in Vitest's module runner, causing CI to fail. Use a direct node_modules path to locate the astro-mermaid integration entry point instead.
There was a problem hiding this comment.
Pull request overview
This pull request updates project dependencies to improve build compatibility, remove warnings, and address security issues. However, it introduces several critical bugs related to React event types and Firebase package imports that will break the application.
Changes:
- Updated all npm dependencies to their latest versions
- Changed Firebase imports from modular
firebase/authandfirebase/appto scoped@firebase/authand@firebase/apppackages - Incorrectly changed React form event types from
FormEventto non-existentSubmitEvent - Updated Astro font configuration to use new
fontProviders.local()API - Downgraded Vite from v7 to v6
- Simplified package overrides from 19 security-related entries to 4
- Updated pnpm version and changed script runners from
nodetotsx
Reviewed changes
Copilot reviewed 9 out of 10 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| src/stores/auth.ts | Changed Firebase imports from firebase/auth to @firebase/auth |
| src/lib/firebase/auth.ts | Changed Firebase imports from firebase/auth to @firebase/auth |
| src/lib/firebase/app.ts | Changed Firebase imports from firebase/app to @firebase/app |
| src/components/react/Faucet/FaucetForm.tsx | Incorrectly changed React event type from FormEvent to non-existent SubmitEvent |
| src/components/chat-widget/chat-input.tsx | Incorrectly changed React event type from FormEvent to non-existent SubmitEvent |
| package.json | Updated all dependencies, downgraded Vite, reduced security overrides, changed script runners to tsx |
| astro.config.mjs | Updated font configuration to use new fontProviders.local() API with options structure |
| .tool-versions | Updated pnpm version to match package.json |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The deps update in #382 bumped @astrojs/vercel from 9.0.2 to 9.0.4, which changed dist/index.js enough that the cspMode patch no longer applied. The .d.ts hunk still matched so TypeScript accepted the config, but the runtime JS was unpatched — every static page got its own CSP route entry in config.json, exceeding Vercel's 5MB body limit. Regenerated the patch against 9.0.4 so the global cspMode collapses all per-route CSP headers into a single catch-all route. Made-with: Cursor
The deps update in #382 bumped @astrojs/vercel from 9.0.2 to 9.0.4, which changed dist/index.js enough that the cspMode patch no longer applied. The .d.ts hunk still matched so TypeScript accepted the config, but the runtime JS was unpatched — every static page got its own CSP route entry in config.json, exceeding Vercel's 5MB body limit. Regenerated the patch against 9.0.4 so the global cspMode collapses all per-route CSP headers into a single catch-all route. Made-with: Cursor
Updates all dependencies accordingly to make the build work correctly, remove warnings, and reduce security issues