Skip to content

fix(deps): clear the dependency audit (MCP SDK, proxy-addr, sprintf-js) - #22

Merged
willynikes2 merged 1 commit into
masterfrom
fix/dependency-audit
Oct 13, 2026
Merged

willynikes2 merged 1 commit into
masterfrom
fix/dependency-audit

Conversation

@willynikes2

Copy link
Copy Markdown
Owner

Problem

The "Dependency audit" CI job is red on master. Three advisories were published after 2026-10-05; no code change caused it.

Change

Package Before After Advisory
@modelcontextprotocol/sdk 1.27.1 1.32.1 GHSA-6qxp-vccf-f47h (high)
proxy-addr 2.0.7 2.0.8 GHSA-jqcg-44mw-7w3h (critical)
sprintf-js 1.0.3 removed GHSA-hp3w-g68c-fv3c (moderate)

sprintf-js has no fixed version. It is pulled in only by argparse 1.x, which js-yaml 3 uses in its command-line script and never in the library that gray-matter calls. An overrides entry pins argparse to 2.x, which does not depend on sprintf-js. The alternative npm audit fix --force would downgrade gray-matter to 2.0.1.

Only package.json (the override) and package-lock.json change.

Tests

  • npm audit --omit=dev --audit-level=moderate: 0 vulnerabilities. npm audit signatures: all verified.
  • Full suite on a fresh npm ci, isolated HOME: 146 pass, 0 fail (front matter parsing included).
  • HTTP MCP tools/list under SDK 1.32.1: the same 12 tools, identical to before, all still carrying securitySchemes.
  • ChatGPT-style OAuth run against a local server on the new install: register, sign in, consent, token, initialize, tools/list, kb_search, refresh all pass.
  • kb.js mcp (stdio) boots, lists 22 tools, runs kb_search.

Deploy

Merge, then in the live checkout: git pull --ff-only, npm ci, sudo systemctl restart kb-server.

🤖 Generated with Claude Code

Three advisories published after 2026-10-05 turned the audit job red:

- @modelcontextprotocol/sdk 1.27.1 -> 1.32.1 (GHSA-6qxp-vccf-f47h, high)
- proxy-addr 2.0.7 -> 2.0.8 (GHSA-jqcg-44mw-7w3h, critical)
- sprintf-js (GHSA-hp3w-g68c-fv3c, moderate): every version is affected.
  It arrives only through argparse 1.x, which js-yaml 3 loads in its
  command-line script and never in the library gray-matter uses. An
  override pins argparse to 2.x, which has no sprintf-js dependency.

npm audit --omit=dev reports 0 vulnerabilities; registry signatures verify.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@willynikes2
willynikes2 merged commit f7dfeb6 into master Oct 13, 2026
9 checks passed
@willynikes2
willynikes2 deleted the fix/dependency-audit branch October 13, 2026 12:23
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