-
Notifications
You must be signed in to change notification settings - Fork 6
chore(deps): update dependency @types/react to v18.3.28 - autoclosed #828
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -64,7 +64,7 @@ | |||||||||
| "devDependencies": { | ||||||||||
| "typescript": "^5", | ||||||||||
| "@types/node": "^20", | ||||||||||
| "@types/react": "^19", | ||||||||||
| "@types/react": "^18.3.27", | ||||||||||
| "@types/react-dom": "^19", | ||||||||||
|
Comment on lines
+67
to
68
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chain🏁 Script executed: # First, locate and read the calendar package.json file
find . -path "*/platforms/calendar/client/package.json" -type fRepository: MetaState-Prototype-Project/prototype Length of output: 121 🏁 Script executed: # Read the calendar client package.json around lines 50-70 to see React versions and types
cat -n platforms/calendar/client/package.json | sed -n '50,75p'Repository: MetaState-Prototype-Project/prototype Length of output: 926 🏁 Script executed: # Find and read the root package.json to check for overrides
find . -maxdepth 1 -name "package.json" -type f -exec cat -n {} \;Repository: MetaState-Prototype-Project/prototype Length of output: 2582 🏁 Script executed: # Check `@types/react-dom`@19 peer dependencies by searching for any lock files or package metadata
fd -t f "(pnpm-lock.yaml|package-lock.json|yarn.lock)" --max-results 1Repository: MetaState-Prototype-Project/prototype Length of output: 95 🏁 Script executed: # Check pnpm-lock.yaml for `@types/react-dom`@19 and its dependencies
grep -A 10 "@types/react-dom@19" pnpm-lock.yaml | head -20Repository: MetaState-Prototype-Project/prototype Length of output: 63 🏁 Script executed: # Search for the `@types/react-dom` package entry to see peer deps
grep -B 2 -A 15 "name: '@types/react-dom'" pnpm-lock.yaml | grep -A 15 "version: '19'"Repository: MetaState-Prototype-Project/prototype Length of output: 63 🌐 Web query:
💡 Result:
Although package metadata sources show
As of Nov 12, 2025, the latest published Sources: Citations:
This package runs React 19 ( To fix this, either:
Option A — align
|
||||||||||
| "@types/react": "^18.3.27", | |
| "@types/react-dom": "^19", | |
| "@types/react": "^18.3.27", | |
| "@types/react-dom": "^18", |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@platforms/calendar/client/package.json` around lines 67 - 68, The package
declares mismatched type packages for React ( `@types/react`: ^18.x vs
`@types/react-dom`: ^19 ) which will produce type errors; fix by either aligning
both to the same major version: change this workspace's dependency
`@types/react-dom` to an 18.x release to match the root-enforced `@types/react`
18.x, or remove/exclude this workspace from the monorepo root override so the
calendar package can use `@types/react`@^19 and `@types/react-dom`@^19 to match
react/react-dom ^19.0.0; update the package.json dependencies in the calendar
client (and the root overrides entry) accordingly and run a fresh install to
verify types.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Monorepo-wide
@types/reactoverride forces React 18 types onto the calendar package (React 19 runtime).The
overridesblock pins@types/reactto18.3.28for every workspace in the monorepo. Setting pnpm overrides like this will effectively fix the installed version by always resolving@types/reactto the specified version. Because the calendar package runs React 19 (react: ^19.0.0), it will be missing type definitions for React 19-specific APIs, and the resulting split (@types/react@18+@types/react-dom@19) risks JSX-contract incompatibilities as flagged above.If the monorepo intentionally standardises on React 18 types, both
@types/reactand@types/react-domshould be locked to^18in the override (and@types/react-dom: "^19"incalendar/client/package.jsonupdated accordingly). If the calendar package should eventually consume React 19 types, the override scope should be narrowed to only the workspaces that need v18.🤖 Prompt for AI Agents