-
Notifications
You must be signed in to change notification settings - Fork 0
TinyPEN npm Package Updates #25
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
Conversation
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.
Pull request overview
This pull request updates npm package dependencies and Node.js runtime requirements for the TinyPEN project, with key changes to support Express 5.x and modernize the Jest CLI usage pattern.
- Updates Express from 5.1.0 to 5.2.1 and http-errors from 2.0.0 to 2.0.1
- Raises minimum Node.js requirement to 24.12.0 and npm to 11.7.0
- Refactors test suite for Express 5.x compatibility by migrating from
app._router.stackwithregexptoapp.router.stackwithmatchers - Updates Jest CLI import to use
@jest/coreinstead of deprecated jest-cli pattern
Reviewed changes
Copilot reviewed 4 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| package.json | Adds engine requirements (Node >=24.12.0, npm >=11.7.0) and updates express, http-errors versions |
| package-lock.json | Updates dependency tree with new package versions and transitive dependencies |
| bin/testApp.js | Modernizes Jest CLI usage by importing runCLI from @jest/core instead of jest-cli |
| tests/mount.test.js | Refactors route detection for Express 5.x compatibility with new routeExists helper function using matchers; updates TinyNode references to TinyPen |
| .github/workflows/cd_prod.yaml | Updates CI Node.js version from 22 to 24 |
| .github/workflows/cd_dev.yaml | Updates CI Node.js version from 22 to 24 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
cubap
left a comment
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.
local tests passing
Summary
Updates npm packages, Node.js version requirements, and fixes Express 5.x compatibility in test suite.
Package Updates
Minor/Patch Updates
New Dependencies
@jest/corefor proper Jest CLI programmatic usageRuntime Requirements
>=24.12.0>=11.7.0CI/CD Changes
22to24Code Changes
Test Suite (Express 5.x Compatibility)
__tests__/mount.test.jsto use Express 5.x route detectionapp._router.stackwithregexptoapp.router.stackwithmatchersrouteExists()helper function for cleaner route verificationJest CLI Fix
bin/testApp.jsto importrunCLIfrom@jest/coreinstead of deprecated jest-cli patternRelated PRs
This PR is part of a coordinated package update across the TPEN 3.0 stack: