Skip to content

chore: update server dependencies - #63

Merged
jackshim415 merged 7 commits into
mainfrom
chore/update-server-dependencies
Sep 9, 2026
Merged

jackshim415 merged 7 commits into
mainfrom
chore/update-server-dependencies

Conversation

@jackshim415

Copy link
Copy Markdown
Contributor

Summary

  • update direct production and development dependencies to their latest compatible releases
  • refresh the npm lockfile and add direct lint/type dependencies required by the checked-in configuration
  • retain Prisma 6, Express 4, and TypeScript 5.9 because their newer major versions require application migrations

Validation

  • npm exec prisma generate
  • npm run build
  • npm run lint (passes with existing warnings)

dependabot Bot and others added 2 commits September 6, 2026 23:36
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>
@railway-app

railway-app Bot commented Sep 8, 2026

Copy link
Copy Markdown

🚅 Environment lovat-server-pr-63 in lovat has no services deployed.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 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.json for 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.

Comment thread package.json Outdated
jackshim415 and others added 3 commits September 8, 2026 18:24
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 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-dev does not appear to be used by any npm script (the dev script uses tsx watch), and it currently pulls in deprecated tooling (e.g., rimraf@2 -> glob@7). If it’s not needed, removing ts-node-dev will reduce dependency footprint and avoid carrying deprecated transitive packages.

package.json:23

  • @types/react is a type-only package and there are no React/JSX references in src/ or in the checked-in ESLint/TS config, so including it under production dependencies unnecessarily 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 to devDependencies, 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

@jackshim415
jackshim415 merged commit cd5b7e8 into main Sep 9, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants