chore(deps): override protobufjs to 7.5.6+ (GHSA-q6x5-8v7m-xcrf et al)#52
Closed
jrphilo wants to merge 1 commit into
Closed
chore(deps): override protobufjs to 7.5.6+ (GHSA-q6x5-8v7m-xcrf et al)#52jrphilo wants to merge 1 commit into
jrphilo wants to merge 1 commit into
Conversation
Clears 7 Dependabot alerts on transitive protobufjs (via posthog-js → @opentelemetry/exporter-logs-otlp-http). All patched at 7.5.6+: - GHSA-685m-2w69-288q (CVE-2026-44289): DoS via unbounded recursion - GHSA-66ff-xgx4-vchm (CVE-2026-44293): Code injection in toObject - GHSA-2pr8-phx7-x9h3 (CVE-2026-44294): DoS from crafted field names - GHSA-fx83-v9x8-x52w (CVE-2026-44292): Prototype injection - GHSA-75px-5xx7-5xc7 (CVE-2026-44291): Codegen gadget after pollution - GHSA-jvwf-75h9-cwgg (CVE-2026-44290): Process-wide DoS via option paths - GHSA-q6x5-8v7m-xcrf (CVE-2026-44288): @protobufjs/utf8 overlong decode Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
docs-preview | 63c1ad8 | Commit Preview URL Branch Preview URL |
May 13 2026, 01:37 PM |
Collaborator
Author
|
Ralphie verified this — ready to merge. Verification
Changelog highlights
Investigation
RecommendationSafe to merge. The override is mechanical, the resolved version (7.5.8) clears all seven advisories' patched floors (7.5.6), and our exposure to the vulnerable surfaces is zero — |
7 tasks
Collaborator
Author
Closed
3 tasks
jrphilo
added a commit
that referenced
this pull request
May 13, 2026
Combines #52 and #53 into a single PR off main, avoiding the rebase cascade after #44/#42/#54/#55 churn. Both are mechanical pnpm.overrides additions: - protobufjs ^7.5.6 — clears alerts #7-14 (GHSA-q6x5-8v7m-xcrf, GHSA-jvwf-75h9-cwgg, GHSA-75px-5xx7-5xc7, GHSA-fx83-v9x8-x52w, GHSA-2pr8-phx7-x9h3, GHSA-66ff-xgx4-vchm, GHSA-685m-2w69-288q). Reached via posthog-js OTLP exporter; not directly imported by our app. - fast-xml-builder ^1.1.7 — clears alerts #3, #4 (GHSA-5wm8-gmm8-39j9, GHSA-45c6-75p6-83cc). Reached via @opennextjs/cloudflare → AWS SDK build/deploy tooling; not directly used. Replaces #52, #53. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
jrphilo
added a commit
that referenced
this pull request
May 13, 2026
…er (#57) Combines #52 and #53 into a single PR off main, avoiding the rebase cascade after #44/#42/#54/#55 churn. Both are mechanical pnpm.overrides additions: - protobufjs ^7.5.6 — clears alerts #7-14 (GHSA-q6x5-8v7m-xcrf, GHSA-jvwf-75h9-cwgg, GHSA-75px-5xx7-5xc7, GHSA-fx83-v9x8-x52w, GHSA-2pr8-phx7-x9h3, GHSA-66ff-xgx4-vchm, GHSA-685m-2w69-288q). Reached via posthog-js OTLP exporter; not directly imported by our app. - fast-xml-builder ^1.1.7 — clears alerts #3, #4 (GHSA-5wm8-gmm8-39j9, GHSA-45c6-75p6-83cc). Reached via @opennextjs/cloudflare → AWS SDK build/deploy tooling; not directly used. Replaces #52, #53. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
jrphilo
added a commit
that referenced
this pull request
May 13, 2026
…es (#59) Adds PROMPT_dependency_batch.md and a Phase 3 invocation to dependency.sh. After Phase 2 drains, if ≥2 PRs are ralphie:ready-to-merge AND share at least one touched file path (in practice always pnpm-lock.yaml), Phase 3 opens a single batch PR off fresh origin/main that combines their package.json edits, regenerates the lockfile once, verifies the combined diff (lint/build/check:links), and closes the constituents with ralphie:replaced-by-newer-pr. Motivates: today's wave required manually batching #52 and #53 after merge-cascade conflicts. Same pattern will repeat weekly per the Dependabot schedule; this automates it. New flags: - --batch-only: skip Phase 1+2, batch the current ready set. - --no-batch: skip Phase 3 entirely. dependency-rules.md gains a Phase 3 section: eligibility gates (≥2 ready, ≥1 file overlap), batchable edit shapes (pnpm.overrides, dependencies versions, same-package version conflicts take the higher), verification recipe, outcomes table, batch PR body shape, and constituent comment shapes. Co-authored-by: Claude Opus 4.7 (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
pnpm.overrides.protobufjsset to^7.5.6to clear seven Dependabot alerts on the transitiveprotobufjs(and a related alert on@protobufjs/utf8). Each is patched atprotobufjs@7.5.6(@protobufjs/utf8@1.1.1ships with that bump).toObjectcode (CVE-2026-44293)protobufjsis reached only throughposthog-js's OpenTelemetry logs exporter; our app never importsprotobufjsdirectly (no source-tree references insrc/orscripts/). We don't author.protofiles, don't decode attacker-controlled protobuf payloads, and don't expose generatedtoObjectcodepaths. The advisories all describe attacker-supplied protobuf data reaching generated decode/encode code, which isn't reachable through our usage. Bumping is still the right move — it's a deterministic transitive fix with no behavior change for our integration.Test plan
pnpm installsucceeds; lockfile updatedpnpm why protobufjsreports7.5.8(>= 7.5.6 patched floor)AI-assistance disclosure: this PR was drafted by an AI agent (Claude / Ralphie) as part of the docs repo dependency-triage loop. The change is mechanical (a
pnpm.overridesentry) and the exposure assessment is grounded in a repo grep and the advisory text. A maintainer should still skim the diff and the advisories before merging.