fix(deps): bump otlp-transformer to 0.214.0 so npm can resolve the tree - #2224
Conversation
bedrock-agentcore (added for the runtime shell) declares an optional peer on @strands-agents/sdk, which in turn wants @opentelemetry/sdk-trace-base ^2.6.1. The pinned otlp-transformer 0.213.0 depends on sdk-trace-base 2.6.0, so npm fails with ERESOLVE for anyone installing the published tarball. bun does not enforce optional peer ranges, which is why CI never saw it. 0.214.0 is the only newer release that still publishes both generated request decoders the dev collector imports (0.215.0 drops ExportLogsServiceRequest, 0.218.0 drops the generated root entirely) and it pins sdk-trace-base 2.6.1.
|
Claude Security Review: no high-confidence findings. (run) |
There was a problem hiding this comment.
AgentCore Harness Review
Verdict: Looks good
This is a minimal patch that bumps the exact-pinned @opentelemetry/otlp-transformer from 0.213.0 to 0.214.0 in package.json and updates bun.lock accordingly.
Verified:
- The only in-source use is
src/cli/operations/dev/otel/collector.ts, which does a deeprequire('@opentelemetry/otlp-transformer/build/src/generated/root')for the protobuf-generatedExportTraceServiceRequest/ExportLogsServiceRequesttypes. That path and both symbols still exist in the 0.214.0 tarball, so the exact-pin (which is presumably why the direct dep is here separate from@opentelemetry/exporter-metrics-otlp-http) continues to work. - No API surface change is being consumed, and no test/source updates are required.
One non-blocking note (feel free to ignore): the local package.json in the workspace still shows ^0.213.0 with a caret — the PR base file has the exact pin. Just double-check that the exact-pin (no ^) is what you want going forward, since the whole reason for keeping this as a direct dep appears to be locking the deep import path across releases.
No changes required to merge.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## refactor #2224 +/- ##
=========================================
Coverage 97.05% 97.05%
=========================================
Files 555 555
Lines 38385 38385
=========================================
Hits 37253 37253
Misses 1132 1132 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Problem
npm installof the CLI (the published tarball, ornpm i -g ./agentcore-1.0.0.tgzper the README) fails with ERESOLVE:bedrock-agentcore(added in feat(runtime): add interactive shell #2198 forruntime shell) has an optional peer on@strands-agents/sdk@opentelemetry/sdk-trace-base@^2.6.1@opentelemetry/otlp-transformer@0.213.0depends onsdk-trace-base@2.6.0, which does not satisfy^2.6.1bun does not enforce optional peer ranges, so
bun install --frozen-lockfilein CI never hit this. Only npm consumers do, which means every npm install of the published package.Fix
Bump the transformer pin to 0.214.0. It pins
sdk-trace-base@2.6.1, and it is the newest release that still publishes both generated request decoderssrc/core/dev/otel/collector.tsimports:The lockfile diff is the transformer subtree only, 2.6.0 to 2.6.1.
Verification
bun run typecheckpassesbun test src/core/dev/otel: 38 passbun test: 2978 pass, 0 failbun run build, thenbun pm packandnpm install -g --prefix <tmp> agentcore-1.0.0.tgzsucceeds on npm 10.8.2 (the version that failed before), and the installedagentcore --versionrunsFollow-ups (not in this PR)
EBADENGINEwarnings:package.jsondeclares node>=20.12.0butcommander@15,ink@7, andcli-truncate@6require node 22. Pre-existing.