Clear three advisories, take sdk 2.7.3, release 0.4.9 - #86
Merged
Conversation
Clears three advisories and takes the sdk we just published.
brace-expansion (two High, one per locked copy) and postcss all had fixed
versions published already. None had been taken because bun does not
re-resolve a locked version that still satisfies its range, so a routine
refresh could never reach them. A clean re-lock takes all three.
The brace-expansion allowlist entry is deleted rather than waived: upstream
backported the fix to the 1.x line, so minimatch@3's own ^1.1.7 range reaches
it with no override, and osv-scanner reports the entry as an unused ignore.
Nothing coupled moved. incur is already at its latest release, so the exact pin
and patches/incur@0.4.26.patch stay together and the patch cannot silently
detach; viem 2.55.10, mppx 0.8.15, @solana/mpp 0.7.0 and @solana/kit 7 are all
unchanged. @agent-score/sdk picks up 2.7.3, which the bunfig exempts from the
release-age soak so a first-party publish reaches its consumer immediately.
Verified against the COMPILED BINARY rather than the node dist, since the
binary is what the patch exists to fix, and rebuilt after the version bump so
the evidence matches what ships:
- --version reports 0.4.9, not 0.0.0-dev, so build-binary.mjs injection works
- whoami --format yaml renders, so incur's createRequire yaml load resolves
- MCP initialize + tools/list handshake returns serverInfo 0.4.9 and the four
progressive-discovery tools, so the stdio transport resolves through the
patched literal specifiers
Plus lint, typecheck, knip, 570 tests, and osv-scanner v2.4.0 clean.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
Three advisories were reachable in the lockfile, each with a fixed version already published:
brace-expansion(two High, one per locked copy) andpostcss. None had been taken because bun does not re-resolve a locked version that still satisfies its range, so a routine refresh could never reach them. A clean re-lock takes all three.The
brace-expansionallowlist entry is deleted rather than waived. It said the 1.x line had no fixed release, which was true when written; upstream has since backported to 1.1.17 and then 1.1.18, sominimatch@3's own^1.1.7range reaches the fix with no override and none of the API breakage that made forcing 5.x impossible. osv-scanner now reports it as an unused ignore.Nothing coupled moved.
incuris already at its latest release (0.4.26), so the exact pin andpatches/incur@0.4.26.patchstay together and the patch cannot silently detach.viem2.55.10,mppx0.8.15,@solana/mpp0.7.0 and@solana/kit7 are all unchanged, so there is no coupled-set bump here and no live all-rails settle obligation.@agent-score/sdkpicks up 2.7.3, which the bunfig exempts from the release-age soak so a first-party publish reaches its consumer immediately.Type of change
Public API
None. No command, flag, output envelope or MCP tool surface changed. The MCP handshake still advertises the same four progressive-discovery tools (
search_tools,get_tool_details,call_read_tool,call_write_tool).Test plan
Run from a clean install (
rm -rf node_modules bun.lock && bun install):bun run lint,bun run typecheck,bun run knipall exit 0bun run test: 570 passed, 2 skippedbun.lock: no issues foundBecause the compiled binary is what the
incurpatch exists to fix, and the node dist can pass while the binary cannot resolve the transport, the smokes were run against the binary, and rebuilt after the version bump so the evidence matches what ships:--versionreports0.4.9, not0.0.0-dev, so thebuild-binary.mjsversion injection is intactwhoami --format yamlrenders, so incur'screateRequire-based lazyyamlload resolves inside the binaryinitialize+tools/listhandshake returnsserverInfo0.4.9and all four tools, so the stdio transport resolves through the patch's literal specifiersThe patch was also confirmed applied to the installed tree by reading the rewritten
dist/Mcp.js, rather than trusting that thepatchedDependenciesentry was still wired.Checklist