chore: remove unused OpenTelemetry exporter deps (closes #19)#1
Open
alfep wants to merge 4 commits into
Open
Conversation
The --bytecode flag in bun build --compile produces structurally corrupted Mach-O binary headers on Apple Silicon (M4), causing macOS Gatekeeper to SIGKILL the process. Removing the flag restores standard binary format compatibility. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Prevents drive-by comments on resolved threads by locking PRs on close with lock_reason='resolved'. Uses minimal permissions (issues: write, pull-requests: write) and SHA-pinned actions to mitigate pwn-request attacks. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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>
.commandcode/ holds CommandCode tool stubs (taste.md etc.) growtopia-app/ is a separate side-project root Both are local-only and not part of the build artifact. 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 freecodexyz#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.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.