Repository navigation
chore: update server dependencies - #63
Conversation
Bumps [resend](https://github.com/resend/resend-node) from 2.1.0 to 6.26.0. - [Release notes](https://github.com/resend/resend-node/releases) - [Changelog](https://github.com/resend/resend-node/blob/canary/CHANGELOG.md) - [Commits](resend/resend-node@v2.1.0...v6.26.0) --- updated-dependencies: - dependency-name: resend dependency-version: 6.26.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
|
🚅 Environment lovat-server-pr-63 in lovat has no services deployed. |
There was a problem hiding this comment.
🟡 Changes recommended
The update introduces a Node runtime/types mismatch (@types/node 26 vs Node 22) and brings in explicitly deprecated transitive packages (glob@7, rimraf@2) flagged for security risk.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
Updates the server’s production and development npm dependencies and refreshes the lockfile to reflect the new resolved dependency graph, while keeping Prisma/Express/TypeScript on the current major versions.
Changes:
- Bumped multiple runtime dependencies (e.g., Express, Redis, Resend, Jose, Zod, axios).
- Updated lint/type tooling dependencies (ESLint, TypeScript, typescript-eslint and related packages).
- Regenerated
package-lock.jsonfor the updated dependency set.
File summaries
| File | Description |
|---|---|
| package.json | Updates direct prod/dev dependencies (including ESLint/TS tooling and type packages). |
| package-lock.json | Regenerates the lockfile with updated resolved versions and new transitive dependencies. |
Review details
- Files reviewed: 1/2 changed files
- Comments generated: 1
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
🔵 Needs a closer look
The updated lockfile includes explicitly deprecated/vulnerable transitive packages (e.g., glob@7 via deprecated tooling) and the PR adds questionable production dependencies (e.g., @types/react) that should be cleaned up before approval.
Review details
Suppressed comments (2)
Previously missed (1) — in code that hasn't changed since the last review.
package.json:60
ts-node-devdoes not appear to be used by any npm script (thedevscript usestsx watch), and it currently pulls in deprecated tooling (e.g.,rimraf@2->glob@7). If it’s not needed, removingts-node-devwill reduce dependency footprint and avoid carrying deprecated transitive packages.
package.json:23
@types/reactis a type-only package and there are no React/JSX references insrc/or in the checked-in ESLint/TS config, so including it under productiondependenciesunnecessarily increases the runtime install surface (and may be confusing to deploy environments). Consider removing it entirely, or (if some tooling truly needs it) moving it todevDependencies, then regenerating the lockfile.
"@prisma/extension-accelerate": "^1.3.0",
"@slack/web-api": "^8.1.1",
"@types/react": "^19.2.18",
"axios": "^1.20.0",
- Files reviewed: 1/2 changed files
- Comments generated: 0 new
- Review effort level: Lite
Summary
Validation