hanzo/chat
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ci: ignore all three transitive-pinned otel packages in depcheck

Browse Source 
The unpinned depcheck (workflow installs latest each run) shifts which
@opentelemetry/* packages it reports: it flagged only @opentelemetry/core
before, now also exporter-trace-otlp-http and sdk-trace-base. All three are
transitive deps of @opentelemetry/sdk-node (imported/driven via NodeSDK in
packages/api/src/telemetry/sdk.ts), pinned top-level for otel version
alignment, none imported by name. Ignore the complete set so ROOT_UNUSED is
empty regardless of depcheck's version-dependent otel detection. Only
@opentelemetry/api is imported directly and stays checked.

Verified locally: with the three ignored, depcheck's root output contains only
@opentelemetry/api, which the workflow's used-in-code list subtracts -> empty.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This commit is contained in:
zeekay
2026-07-04 04:47:57 -07:00
co-authored by Claude Opus 4.8
parent 52bcd7de06
commit 650ad06b15
1 changed files with 10 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.depcheckrc.yml
+10 -6
View File
@@ -2,11 +2,15 @@
#
# ignores: dependencies that are present on purpose but never imported directly,
# so depcheck's static scan reports them as a false positive.
# - @opentelemetry/core: intentional version-pin. It is a transitive
# requirement of the OpenTelemetry stack we do use (@opentelemetry/sdk-node,
# sdk-trace-base, exporter-trace-otlp-http); pinning it at the top level
# keeps a single, aligned core version across the tree. Never imported by
# name, so depcheck flags it — this is expected, not dead. Do NOT remove the
# dependency (that would unpin the otel core version).
#
# The three @opentelemetry/* entries below are all transitive dependencies of
# @opentelemetry/sdk-node, which we DO import and drive (NodeSDK in
# packages/api/src/telemetry/sdk.ts). They are pinned at the top level to keep
# every otel package on one aligned version (mismatched otel versions break at
# runtime). None is imported by name, so depcheck flags them — expected, not
# dead. Do NOT remove them (that would unpin the otel version set). Only
# @opentelemetry/api is imported directly and stays checked.
ignores:
- "@opentelemetry/core"
- "@opentelemetry/exporter-trace-otlp-http"
- "@opentelemetry/sdk-trace-base"