Skip to content

chore(deps): express 4→5 major#60

Merged
terraboops merged 3 commits into
mainfrom
deps/express-5
Jul 6, 2026
Merged

chore(deps): express 4→5 major#60
terraboops merged 3 commits into
mainfrom
deps/express-5

Conversation

@terraboops

Copy link
Copy Markdown
Owner

Bumps `@autonav/pack-server` to Express 5.

  • `express` `^4.18.2`→`^5.2.1`, `@types/express` `^4.17.21`→`^5.0.6`
  • Existing routes already Express-5-safe (named params only; no `*` wildcards, optional params, `app.del`, or regex routes).
  • Express 5 breakage fixed: @types/express 5 types `req.params` values as `string | string[]` (path-to-regexp v8). Added param generics to the four route handlers (`Request<{ packName: string }>`, `Request<{ packName: string; version: string }>`) since these routes are always single-value.

Test

  • `npm install` (root) — express@5.2.1 resolved for pack-server
  • `npm run typecheck` (builds core + comm-layer, typechecks all workspaces) — pass
  • `npm test` — pass (autonav 112 passed/9 skipped; comm-layer 45 passed/13 skipped; pack-server has no tests)
  • Runtime smoke (built server, Express 5): `GET /health`→200; invalid pack name→400 INVALID_PACK_NAME; unknown route→404 via pathless `app.use`

@terraboops
terraboops merged commit 562e3cc into main Jul 6, 2026
@terraboops
terraboops deleted the deps/express-5 branch July 6, 2026 15:08
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.

1 participant