Skip to content

Express dependency upgrades#37

Merged
kentcdodds merged 3 commits into
mainfrom
cursor/express-dependency-upgrades-bf00
Jan 30, 2026
Merged

Express dependency upgrades#37
kentcdodds merged 3 commits into
mainfrom
cursor/express-dependency-upgrades-bf00

Conversation

@kentcdodds
Copy link
Copy Markdown
Member

@kentcdodds kentcdodds commented Jan 30, 2026

Upgrades express, express-rate-limit, and @types/express to their latest versions. This required updating Express wildcard routes from string patterns to regex patterns to align with Express 5's routing behavior. An e2e test assertion was also updated to reflect a minor UI text change.

Test Plan

Full gate (npm run validate) was executed to ensure no regressions. No manual UI testing was performed as changes are dependency/server/test-only, and automated e2e tests cover UI flows.

Checklist

  • Tests updated
  • Docs updated

Screenshots

N/A - Dependency and server-side changes.


Open in Cursor Open in Web


Note

Medium Risk
Express major-version upgrade plus routing-pattern changes can subtly affect request matching and middleware execution order; changes are localized and covered by automated tests.

Overview
Upgrades server dependencies to express@5, express-rate-limit@8, and @types/express@5, with the corresponding package-lock.json refresh.

Updates Express route matching in server/index.ts to use regex-based catch-alls and static-path handlers (replacing '*' and ['/img/*','/favicons/*']) to align with Express 5 routing behavior, and adjusts one Playwright assertion to match updated “no past messages yet” UI copy.

Written by Cursor Bugbot for commit 9601d68. This will update automatically on new commits. Configure here.

cursoragent and others added 3 commits January 30, 2026 15:21
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
Co-authored-by: me <me@kentcdodds.com>
@cursor
Copy link
Copy Markdown

cursor Bot commented Jan 30, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@kentcdodds kentcdodds marked this pull request as ready for review January 30, 2026 15:57
@kentcdodds kentcdodds merged commit cc855bc into main Jan 30, 2026
6 checks passed
@kentcdodds kentcdodds deleted the cursor/express-dependency-upgrades-bf00 branch January 31, 2026 17:01
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