Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR upgrades the codebase to use Node.js 24 as the minimum version and modernizes the testing framework from Jest to Node.js built-in test runner. The changes include replacing deprecated packages with native Node.js APIs and adding explicit .ts extensions to imports.
Key changes:
- Upgrade Node.js version requirement from 20.11+ to 24+ and switch test framework from Jest to Node.js native test runner
- Replace
uuidpackage with nativecrypto.randomUUID()and modernize import patterns by adding explicit.tsextensions - Update dependencies and build configurations to support Node.js 24 features
Reviewed Changes
Copilot reviewed 63 out of 65 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| package.json | Updates Node.js engine requirement, replaces Jest with native test runner, updates dependencies |
| src/**/*.ts | Replaces uuid with crypto.randomUUID(), adds .ts extensions to imports, updates util imports |
| src/**/*.spec.ts | Migrates from Jest to Node.js test runner, updates test syntax and mocking |
| .github/workflows/*.yml | Updates CI to use Node.js 24.x and latest npm version |
| eslint.config.mjs | Adds new ESLint configuration file |
Files not reviewed (1)
- src/check-imports/example-package-lock.json: Language not supported
src/github-api/index.ts
Outdated
| } else { | ||
| // |
There was a problem hiding this comment.
The empty else block with only a comment serves no purpose. Consider removing this else clause entirely since the function continues execution after the if block anyway.
| } else { | |
| // |
|
the coverage action error can be ignored because we added an empty 'prepare' npm script for consistency purpose, which is not available in main branch |
|
coverage action error can be ignored because we are adding an empty 'prepare' npm script for consistency purpose which was not available in main |
Fixes #92