chore(code): Upgrade Electron 42, Vite 8, React 19.2 and deps#2512
chore(code): Upgrade Electron 42, Vite 8, React 19.2 and deps#2512charlesvien wants to merge 13 commits into
Conversation
What T-Rex did
ArtifactsDependency upgrade validation log
Details of @vitest/coverage-v8 peer dependency mismatch with vitest 2.x
Validation script invocation details
Prompt To Fix All With AIFix the following 1 code review issue. Work through them one at a time, proposing concise fixes.
---
### Issue 1 of 1
packages/electron-trpc/package.json:35
**Update coverage provider**
`test:ci` runs Vitest with `--coverage`, but this package still installs `@vitest/coverage-v8@^0.34.0` while Vitest resolves to 2.x. The lockfile shows that coverage provider declares a peer range of `vitest >=0.32.0 <1`, so coverage runs for `@posthog/electron-trpc` can fail or run against an unsupported provider. Please bump `@vitest/coverage-v8` to the matching Vitest major.
Reviews (1): Last reviewed commit: "upgrade vite to 8 (rolldown)" | Re-trigger Greptile |
c7bf83f to
c505ae8
Compare
k11kirky
left a comment
There was a problem hiding this comment.
Unblocking your merge. looks safe, just want to trim comments and test further
eda4c7c to
89dfb6d
Compare
c505ae8 to
978596d
Compare
|
React Doctor found no issues in the changed files. 🎉 Reviewed by React Doctor for commit |
f71d76f to
e3e9e87
Compare
e3e9e87 to
895c6ae
Compare
326ff5f to
38c0c38
Compare
38c0c38 to
1c5a673
Compare
b815015 to
576a55b
Compare
1c5a673 to
08ccece
Compare
08ccece to
19d0201
Compare
3f23132 to
f3407cf
Compare
There was a problem hiding this comment.
Gates denied this PR: it touches dependency/toolchain files (package.json, pnpm-lock.yaml, pnpm-workspace.yaml, Vite configs, Electron version) and exceeds the size ceiling at 7715 lines across 13 files. Dependency and toolchain upgrades of this scope require human review.
f3407cf to
0abb964
Compare

Problem
We've fallen several majors behind (Electron 41, Vite 6), missing performance, security and bug fixes.
Changes
How did you test this?
Manually
Automatic notifications