feat(deps): update all node dependencies incl. major versions - #1036
Open
BlacCello wants to merge 1 commit into
Open
feat(deps): update all node dependencies incl. major versions#1036BlacCello wants to merge 1 commit into
BlacCello wants to merge 1 commit into
Conversation
Bump every dependency in tadis-analyzer, tadis-ui and custom-example-analyzer to latest (including majors: TypeScript 6, ESLint 10, vitest 4, vite 8, @types/node 26, js-yaml 5, @types/supertest 7) and fix the resulting breakage. TypeScript 6 - Set explicit rootDir (TS6 requires it) and drop deprecated baseUrl / downlevelIteration; convert the few baseUrl-style 'src/...' imports to relative paths. - TS6 enables the strict family by default; keep it off (strictNullChecks / strict: false) to preserve the projects' established lenient typing. - Replace the `Function` supertype in core-typed with explicit constructor interfaces (name + new()). ESLint 10 (flat config) - ESLint 10 drops .eslintrc support; migrate both packages to eslint.config.mjs using the unified typescript-eslint package. - Remove unused imports/vars and underscore-ignore intentionally-unused args. vitest 4 / vite 8 - Convert chai-style assertions (expect(x).to.be.null) to native vitest matchers across the analyzer test suite. - Import defineConfig from vitest/config; add a vite/client types reference for CSS side-effect imports; keep esbuild as the CSS minifier (vite 8 defaults to lightningcss, which rejects a legacy IE star-hack in the CSS). Security - Pin patched transitive deps via overrides (multer ^2.2.0, diff ^4.0.4): all three packages now report 0 npm-audit vulnerabilities. Verified: build, test and lint pass for every package, and the tadis-analyzer and tadis-ui Docker images build (tests run green inside).
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (34)
💤 Files with no reviewable changes (8)
📝 WalkthroughWalkthroughUpgrades all three packages ( ChangesToolchain and Source Upgrade
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 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 |
BlacCello
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Updates every npm dependency across
tadis-analyzer,tadis-uiandcustom-example-analyzerto the latest version — including majors — and fixes all resulting breakage. All three packages build, test, lint clean and report 0 npm-audit vulnerabilities.Notable major bumps: TypeScript 5→6, ESLint 9→10, vitest 3→4, vite 7→8, @types/node →26, js-yaml 4→5, @types/supertest 6→7.
TypeScript 6
rootDirand deprecatesbaseUrl/downlevelIteration— setrootDir, removed the deprecated options, and converted the fewbaseUrl-stylesrc/...imports to relative paths.strict/strictNullChecks: false) to preserve each project's established lenient typing (strictwas already commented out).Functionsupertype incore-typed.tswith explicit constructor interfaces.ESLint 10 (flat config)
.eslintrcsupport — migrated both packages toeslint.config.mjsvia the unifiedtypescript-eslintpackage.vitest 4 / vite 8
expect(x).to.be.null) to native vitest matchers (expect(x).toBeNull()) across the analyzer test suite — no lint suppression needed.defineConfignow imported fromvitest/config; added avite/clienttypes reference for CSS side-effect imports; kept esbuild as the CSS minifier (vite 8 defaults to lightningcss, which rejects a legacy IE*-hack in the bundled CSS).Security
overrides:multer ^2.2.0(NestJS) anddiff ^4.0.4(ts-node).npm audit→ 0 vulnerabilities everywhere.Verification
npm run build/test/lintpass for every package.Summary by CodeRabbit
Chores
Tests