fix(deps): clear the high-severity advisories blocking the release audit - #2055
Merged
Conversation
Astro-Han
marked this pull request as ready for review
August 3, 2026 20:08
This was referenced Aug 3, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The
Release macOS arm64run for 0.1.4 failed at its first real gate,npm audit --omit=dev --audit-level=high, before it reached packaging or reserved a release identity (run 30848094969). Four production advisories were outstanding, two of them high:undicifast-uri@hono/node-server@modelcontextprotocol/sdkLockfile only. Every fixed version already satisfies the existing declared ranges —
undici^8.7.0in runtime and headless,@modelcontextprotocol/sdk^1.24.3in mcp, andfast-urithroughajv's^3.0.1— so nopackage.jsonchanges are needed. SDK 1.30.0 widens its own range to^1.19.9 || ^2.0.5, which is what lets@hono/node-servermove off the vulnerable line.@hono/node-server2.x requires Node >=20; the repository already requires >=22.19.0.Verification
npm audit --omit=dev --audit-level=high: 0 vulnerabilities (was 4).npm cithennpm run build: clean.@maka/mcp11/11 pass,@maka/headless1337/1337 pass.@maka/runtime3076/3089 pass. The 4 failures arebuiltin-toolsworkspace-containment cases that fail identically on unmodifiedmainon macOS, where the temp dir resolves through the/var→/private/varsymlink; they are environment-specific and unrelated to this change. CI runs on Linux and does not hit them.npm run format:checkandnpm run lint: clean.Review focus
undicimoves three minors. Its use is confined to bot bridges (WebSocket) and the network layer (Agent,ProxyAgent,EnvHttpProxyAgent,fetch). Worth noting:packages/headless/src/provider-auth-proxy.tsdocuments the 8.7-vs-8.8 h2 multiplexing gate and deliberately forces h1 pools so it does not depend on which undici build serves the fetch — that comment stays accurate at 8.10.Rollout
Once this is on
mainand CI is green, re-dispatchRelease macOS arm64. The failed run stopped beforeResolve and reserve release identity, so thev0.1.4tag and release name are still unused and the version does not need to be bumped again.