Skip to content

fix(eve): migrate provider to tracePolicy - #2417

Merged
Luca Forstner (lforst) merged 4 commits into
braintrustdata:mainfrom
chadhietala:chad/trace-policy
Sep 2, 2026
Merged

fix(eve): migrate provider to tracePolicy#2417
Luca Forstner (lforst) merged 4 commits into
braintrustdata:mainfrom
chadhietala:chad/trace-policy

Conversation

@chadhietala

@chadhietala Chad Hietala (chadhietala) commented Aug 31, 2026

Copy link
Copy Markdown
Contributor

Summary

Replace the deprecated Eve provider capture: "content" option with the equivalent tracePolicy decision introduced in vercel/eve#2721.

The policy continues to emit every trace and records both inputs and outputs, preserving the existing Braintrust provider behavior.

Changes

  • Replace capture: "content" in js/src/instrumentation/plugins/eve-provider.ts.
  • Update the narrow vendored EveProviderDefinition type with Eve's trace-policy shape.
  • Update the focused provider test.

Verification

  • pnpm exec vitest run src/instrumentation/plugins/eve-provider.test.ts
  • Focused Prettier check
  • Focused ESLint check (0 errors; test file is excluded by the repository ignore rules)

@chadhietala Chad Hietala (chadhietala) changed the title feat(eve): migrate to tracePolicy API and bump e2e eve versions fix(eve): migrate provider to tracePolicy Aug 31, 2026
@chadhietala
Chad Hietala (chadhietala) marked this pull request as ready for review August 31, 2026 20:37
@lforst
Luca Forstner (lforst) merged commit 8ddfbed into braintrustdata:main Sep 2, 2026
47 checks passed
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.

2 participants