Skip to content

Commit f685a85

Browse files
logaretmclaude
andauthored
fix(e2e): Pin @opentelemetry/api to 1.9.0 in ts3.8 test app (#19992)
## Summary - `@opentelemetry/[email protected]` was released on Mar 25 and introduced `export { Foo, type Bar }` syntax (inline type modifiers) in its `.d.ts` files, which requires TypeScript 4.5+ - The `generic-ts3.8` E2E test runs with `skipLibCheck: false` and TypeScript 3.8, so it tries to parse OTel's types and fails - This pins `@opentelemetry/api` to `1.9.0` in the ts3.8 test app via `pnpm.overrides` - We can't pin repo-wide in published packages because OTel uses a global singleton pattern — version mismatches with `@opentelemetry/sdk-trace-base` cause the tracer to become a no-op - Our published `.d.ts` files are unaffected — only OTel's own types use the incompatible syntax ## Test plan - [x] Verified locally: `yarn test:run generic-ts3.8` passes with the pin - [ ] CI `E2E generic-ts3.8 Test` should go green 🤖 Generated with [Claude Code](https://claude.com/claude-code) Closes #19998 (added automatically) Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
1 parent 4a43db2 commit f685a85

File tree

1 file changed

+5
-0
lines changed
  • dev-packages/e2e-tests/test-applications/generic-ts3.8

1 file changed

+5
-0
lines changed

dev-packages/e2e-tests/test-applications/generic-ts3.8/package.json

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,11 @@
2020
"@sentry-internal/replay": "latest || *",
2121
"@sentry/wasm": "latest || *"
2222
},
23+
"pnpm": {
24+
"overrides": {
25+
"@opentelemetry/api": "1.9.0"
26+
}
27+
},
2328
"volta": {
2429
"extends": "../../package.json"
2530
}

0 commit comments

Comments
 (0)