Repository navigation
fix(deps): patch Hono and ip-address advisories and align qs - #7647
Conversation
|
Caution Review failedThe pull request is closed. Finishing Touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
✅ Deterministic PR hygiene checks passed. |
Local CI-equivalent (Linux) at 69f189eOverall: PASS (temporary rule; raw base-identical test failures retained) under the temporary local-verification rule. This is local Linux evidence, not hosted CI.
Failure classification (failed commands remain failed):
Base comparisons (same Bun/environment; once per distinct failing file, reused where recorded):
NOT RUN: test-3-of-4 batches 15–64 (original script fail-fast); test-4-of-4 batches 11–66 (original script fail-fast); Hosted CI and platforms/jobs outside the requested Linux scope All original check statuses are retained. No head checks were retried. Overall disposition uses the explicit base-identical exception; this does not claim the fail-fast shard remainders passed. Diagnostic attribution counts (separate from primary evidence where applicable): {'pass': 0, 'fail': 0, 'skip': 0}. Head re-fetched before posting: Recorded UTC: 2026-10-24T10:30:01.231257+00:00 |
Local CI-equivalent (Linux) coverage addendum at 69f189eFinal verdict: PASS (temporary rule; raw base-identical test failures retained). This supersedes the earlier verdict’s incomplete coverage: #7647 (comment) Host All new suite/base invocations use documented Batch file lists are generated by the exact head’s original planner. Only its script-directory anchors point back to the original location; its test command/deadlines are unchanged. Every previously executed batch file list was checked against the reconstructed manifest. Remaining batches ran serially; any new failure stayed failed and did not suppress later missing batches.
Remaining primary batches — each command is
Remainder totals: 21763 pass / 0 fail / 79 skip. Combined original + remainder command counts: 55633 pass / 1 fail / 167 skip. Failed checks remain failed; final disposition uses only the explicit BASE-IDENTICAL exception. Failure classification across original and remainder evidence:
Base comparisons (cached once per distinct file/exact failed batch):
Every original non-test gate result is retained in #7647 (comment). No source edits, pushes, approvals or merges were performed. Source status: clean ( NOT RUN: Hosted CI and platforms/jobs outside the requested Linux scope Exact manifests and raw logs (retained on the named Linux host):
UTC: 2026-10-24T11:32:01.161432+00:00 |
|
Integration evidence for exact head 69f189e (lane L6; dependency change, Security summary in the description):
Static merge simulation on current dev 047c5dc (this Mac): dev=047c5dc60695351657e1f6ea7b85e12015cb67e4 base=47316142fe06171a94de02d63f1727dbed4c0382 pr_files= 2 overlap=[]; merge clean; install exit=0; gui install exit=0; tsc exit=0; tsc doctor exit=0; structure:check exit=0; privacy:scan exit=0; layout+ratchet guards exit=0 52 pass 0 fail ; Merged by the triage manager as maintainer integration (MAINTAINERS.md, dev only) under the owner's static-judgment rule while GitHub Actions is degraded; dev push CI will be checked when Actions recovers. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Summary
hono: 4.13.13,ip-address: ^10.7.4, andqs: ^6.16.1; refresh only those three lockfile resolutions.Closes #7641
Closes #7640
Closes #7639
Security summary
^4.11.4, and node-server peers on^4, so the pin stays within both contracts (SDK registry metadata).^10.2.0range.^6.16.1requirement; move the floor and resolution to 6.16.1. This is a dependency-range correction; the two named GHSAs apply to Hono and ip-address. Primary metadata checked 2026-10-24.Verification
bun install --lockfile-only: exit 0; parsed lock comparison confirms exactlyhono,ip-address, andqschanged. No broad update.bun install --frozen-lockfilein a fresh archive of this commit with no node_modules: exit 0, 108 packages installed; package.json and bun.lock hashes unchanged.bun test ./tests/providers/qoder-mcp-server.test.ts ./tests/providers/codebuddy-mcp-server.test.ts ./tests/providers/cursor/cursor-mcp-manager.test.ts ./tests/providers/cursor/cursor-mcp-stdio.test.ts ./tests/server/server-live.test.ts: exit 0, 83 pass / 0 fail.bun run typecheck: exit 0.bun audit --json: exit 1; Hono/ip-address/qs have no findings, and the four pre-existing fast-uri findings (two moderate, two low) are byte-for-byte unchanged from the baseline. This is an audit limitation, not a passing plain audit.9d11593878320c2bcf9deef756d06fbbf4ed5402, after merging currentdevatceef4889e4666dc0492a375d272eafa5b785361e. PR diff remains exactly the two dependency files.bun run typecheck,bun run privacy:scan, andbun run structure:checkall passed again on this head.bun run test:changedon the exact final head, through the cxc receipt runner with the documented private trusted-fixture setting: exit 0, 29,787 pass / 46 skip / 0 fail, 1,319 files, successful current-head test receipt. The pre-merge changed run also passed with the same counts. Root and GUIbun run audit:high: exit 0 (root 144 packages, four below threshold; GUI 82 packages).bun run teston the final head (through the cxc receipt runner): exit 1, aggregate 52,117 pass / 167 skip / 20 fail across the main and serial batches. Nineteen failures are trusted-Bun fixture placement (bun-runtime,ocx-launcher-fixture-trust,ocx-launcher-runtime); the other is Desktop catalog ordering attests/claude-integration/claude-desktop-remote-hub.test.ts:249. No successful receipt was produced.devatceef4889, old dependency resolutions, frozen install, then the same four failing files: exit 1, 84 pass / 2 skip / 20 fail, with the same twenty failure names. These failures predate this patch. The documentedOPENCODEX_TEST_TRUSTED_FIXTURE_DIRsetting can select a private trusted scratch directory without relaxing the product trust rule; the four-file diagnostic with that setting exits 1 with 103 pass / 2 skip / 1 fail, leaving only the Desktop ordering failure. This is not substituted for the failed full run.gpt-6.1-sol/ high security review of the exact final head: PASS, zero actionable tracked-diff/integration findings. It independently checked the merge, advisory floors, npm integrities and unchanged dependency contracts. Maintainers retain the merge/security decision.workflow_dispatchwithlane=all: confirmedheadSha=9d11593878320c2bcf9deef756d06fbbf4ed5402; currently queued, no conclusion. PR workflow 40052389994 is also pending on this head. Older-head CI is not passing evidence for this head.test:changedcompleted successfully with the supported fixture setting. No merge performed.Checklist