docs: update instrumentation integrations for the provider layout - #2068
Closed
chadhietala wants to merge 1 commit into
Closed
chadhietala wants to merge 1 commit into
chadhietala wants to merge 1 commit into
Conversation
Rewrite the seven OTLP/OTel backends (PostHog, Sentry, Datadog, Honeycomb, Arize, Raindrop, Jaeger) from the legacy single-file defineInstrumentation + registerOTel setup form to otelIntegration destinations exported from agent/instrumentation/<slug>.ts. Braintrust is simplified to use the SDK's self-initializing form (pending braintrust-sdk PR #2363). Registry install targets, the instrumentation registry validator, the docs integration presentations, and the install-integrations layout sentence are updated to the provider-directory layout. Signed-off-by: Chad Hietala <chad.hietala@vercel.com>
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
chadhietala
changed the title
chadhietala
changed the title
![vercel[bot]](https://avatars.githubusercontent.com/in/8329?s=60&v=4){kind=small}
| }); | ||
| }, | ||
| }) as Parameters<typeof defineInstrumentation>[0], | ||
| recordInputs: true, |
Contributor
There was a problem hiding this comment.
braintrustEveInstrumentation({ recordInputs, recordOutputs }) uses options that don't exist on the pinned braintrust@3.24.0 function signature, breaking the docs type-check build.
This branch had an error being deployed
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
1 participant
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
Updates the instrumentation integration docs and registry scaffolding on eve.dev/integrations for the provider-directory layout (
agent/instrumentation/<slug>.ts), replacing the legacy single-filedefineInstrumentation({ setup: () => registerOTel(...) })form.The seven OTLP/OTel backends (PostHog, Sentry, Datadog, Honeycomb, Arize, Raindrop, Jaeger) now export an
otelIntegration({ traceExporter })destination fromeve/instrumentation/otel. The framework folds all declared destinations into one pipeline at startup, so per-backendregisterOTel/setupis no longer needed. PostHog'sstep.starteddistinct_id attribution and Arize'sopeninference.project.nameresource attribute are dropped — the former is not expressible as a destination, and the latter belongs onotel(), not per-destination.Braintrust is not OTel and stays on its own SDK helper (
braintrustEveInstrumentation), simplified to the self-initializing form withoutinitLogger/defineState/setup(pending braintrust-sdk PR #2363).Registry install targets, the instrumentation registry validator, the integration-page docs presentations, and the
install-integrations.mdxlayout sentence are updated to the directory layout.Note: The registry
tsccheck will fail until the Braintrust SDK PR ships and thebraintrustdependency is bumped. This draft is not intended to merge before then.Validation
pnpm --filter eve-docs run registry:validate:instrumentation— passespnpm --filter eve-docs run test:unit— 154 passedpnpm docs:check— 82 files, 262 snippets, 82 mdx oknode scripts/validate-public-registry-requirements.mjs— passespnpm lint,pnpm fmt— cleantsc --project registry/tsconfig.json— fails onbraintrust.tsonly (expected, pending SDK fix(eve): avoid npm override conflict in web chat setup #2363)Checklist
CONTRIBUTING.mdevepackagegit commit --signoff)