Repository navigation
chore(deps): resolve Dependabot security alerts via pnpm overrides - #145
Conversation
Raise override floors to close open Dependabot advisories: - protobufjs <7.5.5 -> 7.5.8 (transitive via @grpc/grpc-js -> @grpc/proto-loader) - @grpc/grpc-js <1.14.4 -> 1.14.4 (transitive via OpenTelemetry OTLP/gRPC exporters) - fast-uri <3.1.2 -> 3.1.2 (dev-only, via ajv -> @commitlint) - qs <6.15.2 -> 6.15.2 (dev-only, via @exodus/test) The duplicate protobufjs@8.x copy (via @opentelemetry/otlp-transformer) is resolved separately by upgrading OpenTelemetry to 0.219.0, not by an override. The esbuild advisory is deferred: esbuild@0.28.x is incompatible with the current tsup (esbuild ^0.25) and esbuild is a build-time tool not shipped in dist. Local verification: build 16/16, typecheck clean, otel/core/interceptors tests pass, lint clean. Supersedes #118. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR introduces pnpm dependency overrides to mitigate critical and high-severity CVEs in transitive dependencies. A security changeset documents the override strategy, including rationale for version floor changes, dev-only constraints, and deferred advisories. The pnpm-workspace.yaml configuration implements the overrides for affected packages and updates version constraints. ChangesSecurity dependency overrides
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
@connectum/auth
@connectum/cli
@connectum/core
@connectum/events
@connectum/events-amqp
@connectum/events-kafka
@connectum/events-nats
@connectum/events-redis
@connectum/healthcheck
@connectum/interceptors
@connectum/otel
@connectum/reflection
@connectum/testing
commit: |
What
Raise pnpm override floors to close open Dependabot security advisories. Supersedes #118 (which only raised protobufjs to 7.5.6 — below the required 7.5.8 — and had a stale lockfile).
protobufjs<7.5.5 → 7.5.8@grpc/grpc-js → @grpc/proto-loader)@grpc/grpc-js<1.14.4 → 1.14.4fast-uri<3.1.2 → 3.1.2ajv → @commitlint)qs<6.15.2 → 6.15.2@exodus/test)Not in this PR (by design)
protobufjs@8.x(via@opentelemetry/otlp-transformer) — resolved separately by upgrading OpenTelemetry to0.219.0(its transformer droppedprotobufjsin 0.218), not by an override.esbuildadvisory — deferred.esbuild@0.28.xis incompatible with the currenttsup(esbuild ^0.25, no tsup release supports 0.28.x), and esbuild is a build-time tool that does not ship in the compileddist.Verification
Applied floors confirmed in the tree:
protobufjs@7.5.8,@grpc/grpc-js@1.14.4,fast-uri@3.1.2,qs@6.15.2.Local:
build16/16 ✅,typecheck✅,otel/core/interceptorstests ✅,lint✅.Summary by CodeRabbit
Release Notes