Skip to content

chore: migrate backend to Express 5 - #26

Merged
Inmerson merged 3 commits into
mainfrom
security/express-5-upgrade
Sep 12, 2026
Merged

Inmerson merged 3 commits into
mainfrom
security/express-5-upgrade

Conversation

@Inmerson

@Inmerson Inmerson commented Sep 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Migrates the backend from Express 4 to Express 5.2.1 and @types/express 5.0.6.
  • Resolves the remaining qs advisory by locking the Express 5 dependency graph to qs 6.16.0.
  • Preserves existing validation semantics for bodyless POSTs under Express 5.
  • Handles EADDRINUSE startup failures explicitly instead of printing false startup-success messages.
  • Strengthens test-api.js into an enforceable smoke/regression gate with 19 endpoint/status checks.

Verification

Fresh verification on this exact branch:

  • Node 22 TypeScript build: passed.
  • Live API regression suite: 19 passed, 0 failed.
  • Bodyless /api/matrix/add and /api/exam/result: HTTP 400 as intended.
  • Port-collision regression: second server exits code 1, failure log present, no false success log.
  • npm audit --json: 0 vulnerabilities.
  • npm ci --dry-run --ignore-scripts: passed.
  • git diff --check: passed.
  • Independent re-review: APPROVED, no remaining Critical or Important findings.
  • GitHub CI: passed.

Dependency context

Dependency refresh PR #23 is merged into main. This PR is now based directly on main and contains only the Express 5/backend migration on top of that merged refresh.

Copilot AI lite review requested due to automatic review settings September 12, 2026 23:29
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Inmerson
Inmerson changed the base branch from security/dependency-refresh-2026-09-12 to main September 12, 2026 23:31
@Inmerson
Inmerson merged commit 1ee0fce into main Sep 12, 2026
1 check passed
@Inmerson
Inmerson deleted the security/express-5-upgrade branch September 12, 2026 23:54
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