Repository navigation
chore(deps): bump hono and @hono/node-server - #1134
horsefacts wants to merge 2 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
LGTM. The follow-up commit removes the hono@<4.13.16 and @ hono/node-server@<2.1.6 overrides, so transitive consumers are no longer pinned to exact versions.
What was reviewed:
- The lockfile matches the new
^4.13.16specifier and thewata>@ hono/node-server: 2.1.6override. The@ hono/node-server@ 2.1.6peer range is back to^4 || ^5.0.0-0, and 4.13.14 is gone. - Two hono copies remain: 4.13.18 for the root and the mppx 0.13.3 peer, and 4.13.16 for the transitive consumers. Both are at or above the 4.13.16 floor and all the ranges allow either, so a later dedupe is optional.
- The comment above the wata override is still accurate for the 2.1.5 to 2.1.6 patch bump.
- Not run:
pnpm build,pnpm check:typesandpnpm test. I only read the code.
Extended reasoning...
The change touches only dependency metadata: one direct specifier bump in package.json, one override version bump in pnpm-workspace.yaml, and the regenerated pnpm-lock.yaml. It changes no application code and no auth, crypto or data-handling code. My earlier inline concern was about the open-ended hono and node-server overrides pinning transitive consumers, and commit 892f681 addressed it by removing both overrides. No CODEOWNERS file covers these paths, the timeline shows no other reviewer objections, and the bug hunt ran to a dry streak with no findings.
Bumps hono to >= 4.13.16 and @hono/node-server to >= 2.1.6, updating overrides where needed. The lockfile was regenerated with the repository's pinned package manager in lockfile-only mode.