chore: remove unused OpenTelemetry exporter deps (closes #19)#45
Open
alfep wants to merge 1 commit into
Open
Conversation
Closes freecodexyz#19 Verified unused by grepping all imports/requires/dynamic-imports in src/ and scripts/: - No static `import` of any @opentelemetry/exporter-* package - No static `import` of @opentelemetry/resources - No static `import` of @opentelemetry/semantic-conventions - No `import('@opentelemetry/...)` dynamic call anywhere - API, api-logs, core, sdk-* kept (used for SDK plumbing) Removed 12 packages from bun.lock via `bun install`. Net removal of ~13 transitive dependencies. Note: pre-existing build issue with `lodash-es/<subpath>.js` resolves is unrelated to this change and needs separate fix. Co-Authored-By: Claude Opus 4.6 <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
Closes #19 — remove 12 unused OpenTelemetry exporter packages from
package.jsonandbun.lock.Verified unused
Ran
grepfor all forms of import (static, dynamic, type-only) acrosssrc/andscripts/. Zero matches for:@opentelemetry/exporter-logs-otlp-grpc/-http/-proto@opentelemetry/exporter-metrics-otlp-grpc/-http/-proto@opentelemetry/exporter-prometheus@opentelemetry/exporter-trace-otlp-grpc/-http/-proto@opentelemetry/resources@opentelemetry/semantic-conventionsKept (still in use)
@opentelemetry/api— public API surface@opentelemetry/api-logs— log API@opentelemetry/core— SDK plumbing@opentelemetry/sdk-logs/-metrics/-trace-base— telemetry SDKChanges
package.jsonbun.lockRe-ran
bun installto sync the lockfile; net removal of 12 packages.Verification
Branch rebased on
upstream/main(db7d216). Cherry-picked cleanly.Caveat
There is a pre-existing build failure (unrelated to this change):
bun run build:deverrors onCould not resolve: "lodash-es/<subpath>.js". Will fix in a separate PR.