Skip to content

build: OTel version-currency gate#828

Merged
jithinraj merged 1 commit into
mainfrom
build/otel-doc-currency-sweep
Jun 6, 2026
Merged

build: OTel version-currency gate#828
jithinraj merged 1 commit into
mainfrom
build/otel-doc-currency-sweep

Conversation

@jithinraj

@jithinraj jithinraj commented Jun 6, 2026

Copy link
Copy Markdown
Member

Summary

Add a version-currency gate for source surfaces and correct a stale telemetry package-version constant in the optional OpenTelemetry adapter. No protocol surface change.

Scope

Tooling, its test, release-version source surfaces, and the optional @peac/telemetry-otel adapter. No PEAC record shape, schema, registry, wire format, conformance vector, or signing change. No OpenTelemetry dependency or lockfile bump. The only runtime-observable change is that the telemetry adapter now reports the correct package version in the optional peac.version span/metric attribute (see below). Public-surface sentinels are unchanged.

OpenTelemetry comparison (inspected, no bump needed)

Surface PEAC value Current upstream Action
@opentelemetry/api ^1.9.0 1.9.1 none (range covers latest)
@opentelemetry/sdk-metrics ^2.0.0 2.7.1 none (range covers latest)
@opentelemetry/sdk-trace-base ^2.0.0 2.7.1 none (range covers latest)
@opentelemetry/exporter-trace-otlp-http (example only) ^0.212.0 0.218.0 none (example-only; no bump-for-latest)

OpenTelemetry Specification is currently 1.57.0 and semantic-conventions 1.41.1; PEAC hardcodes no schema URL or semconv pin and references no OTEL_* environment variables (prior matches were PEAC internal identifiers). The MCP Registry wording already reads "Preview" with no overclaim. PEAC composes with OpenTelemetry as an optional, vendor-neutral adapter and is not an observability backend.

Notes

No OpenTelemetry dependency or lockfile bump. Historical 0.9.22 references (CHANGELOG, the architecture ADR, the privacy hash seed, and privacy filter-input test fixtures) are intentionally left unchanged.

Consolidate the @peac/telemetry-otel package version to a single source
(packages/telemetry-otel/src/version.ts) and add a source-version-currency
gate so it cannot drift.

- provider.ts derived its tracer/meter version (emitted as the optional
  peac.version span/metric attribute) from a hardcoded 0.9.22 even though
  the package is 0.15.0. It now derives from the single version source, so
  the emitted value is correct and cannot drift. index.ts re-exports the
  same source.
- privacy.ts DEFAULT_SALT is unchanged (a stable hashing seed; changing it
  would alter every hashed identifier) with a clarifying comment.
- scripts/verify-doc-version-currency.mjs checks enumerated source sites
  (the telemetry version source and the smithery.yaml mcp-server pin)
  against docs/releases/current.json, plus an anti-drift guard that fails
  if the provider reintroduces a hardcoded version literal. It complements
  tests/tooling/docs-version-banner-truth.test.ts and does not duplicate
  the markdown-banner scan.
- tests/tooling/doc-version-currency.test.ts covers the pure checks and the
  CLI via a synthetic --root tree; import-smoke now asserts the version
  constant equals the package version.

No OpenTelemetry dependency or lockfile bump; no PEAC record/schema/wire/
signing/conformance change. Historical 0.9.22 references (CHANGELOG, the
architecture ADR, the privacy hash seed, and privacy filter-input test
fixtures) are left unchanged.
@jithinraj jithinraj changed the title build: add OTel version-currency gate build: OTel version-currency gate Jun 6, 2026
@jithinraj
jithinraj merged commit 9f72bd5 into main Jun 6, 2026
28 checks passed
@jithinraj
jithinraj deleted the build/otel-doc-currency-sweep branch June 6, 2026 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant