Skip to content
This repository was archived by the owner on Oct 16, 2026. It is now read-only.

docs(export): define Opik post-run boundary - #1406

Merged
christso merged 1 commit into
mainfrom
feat/sdk-dx-opik-export
Jun 18, 2026
Merged

christso merged 1 commit into
mainfrom
feat/sdk-dx-opik-export

Conversation

@christso

@christso christso commented Jun 17, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • add an ADR documenting the AgentV -> Opik post-run export boundary
  • keep Opik as a post-run adapter over completed AgentV result bundles
  • document the privacy, failure-mode, and identity constraints for the future adapter

Decision

  • Merge-ready as an ADR-only decision artifact.
  • Direct Opik upload/export implementation remains out of scope for this PR.
  • Implementation should continue through av-vwa.16.2 stable external identity / duplicate policy, then av-vwa.16.4 vendor-neutral projection bundle, then av-vwa.15.2 Opik adapter.

Findings

  • AgentV already has the right post-run inputs: EvaluationResult / index.jsonl, per-test artifacts from packages/core/src/evaluation/run-artifacts.ts, and canonical agentv.trace.v1 sidecars.
  • run-artifacts.ts currently writes execution-trace sidecars with full capture, even though buildTraceEnvelopeFromEvaluationResult() defaults to metadata-only capture.
  • Opik's documented manual trace/span APIs require UUID-form ids, while AgentV's canonical trace/span ids are stable hashed ids. The adapter should translate ids and preserve AgentV ids in metadata instead of reusing them directly.
  • av-vwa.15.2 already depends on av-vwa.16.4, and the id/retry semantics overlap av-vwa.16.2, so a direct exporter implementation here would duplicate or preempt the shared export work.

Validation

  • bun run lint - passed (Checked 727 files in 1946ms. No fixes applied.)
  • No runtime code changed; example/eval validation was not run locally because this PR only adds an ADR.
  • GitHub checks are green on head 8907dd12: Build, Check Links, Cloudflare Pages, Lint, Test, Typecheck, Validate Evals, and Validate Marketplace.

Post-Deploy Monitoring & Validation

No additional operational monitoring required. This PR adds a docs/ADR decision only and does not change runtime code, CLI behavior, artifacts, or deployed surfaces.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 17, 2026 •

Copy link
Copy Markdown

Deploying agentv with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8907dd1
Status: ✅  Deploy successful!
Preview URL: https://2530894c.agentv.pages.dev
Branch Preview URL: https://feat-sdk-dx-opik-export.agentv.pages.dev

View logs

@christso
christso force-pushed the feat/sdk-dx-opik-export branch from 816d173 to 8907dd1 Compare June 18, 2026 04:28
@christso
christso marked this pull request as ready for review June 18, 2026 04:28
@christso
christso merged commit 1e701f7 into main Jun 18, 2026
8 checks passed
@christso
christso deleted the feat/sdk-dx-opik-export branch June 18, 2026 04:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant