fix(results): isolate target bundles under run timestamps#1558
Merged
Conversation
Deploying agentv with
|
| Latest commit: |
f656fd4
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://e72a68d4.agentv.pages.dev |
| Branch Preview URL: | https://result-row-id-sidecars.agentv.pages.dev |
45ad352 to
458d292
Compare
458d292 to
e28b3c2
Compare
Merge PR #1560 for Bead av-k0e after independent read-only code review reported no actionable issues and verification passed.
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
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
Multi-target eval runs now fan out into isolated target/variant result bundles under the invocation timestamp, so two targets with the same
test_idno longer point at and overwrite the same sidecar files. Within each bundle, row sidecar directories use deterministic compact row IDs based on eval source, suite, test id, target, and variant, whileindex.jsonlremains the source of truth for identity and artifact paths.Dashboard and result readers now discover nested bundle manifests under timestamp folders while preserving legacy root-level
index.jsonlbundles. They may walk storage folders to find manifests, but target and variant semantics continue to come from loaded row/run metadata, not path parsing.Fixes #1557.
Validation
origin/main; branch freshness checkgit log HEAD..origin/mainis empty.bun installafter rebase to materialize declared dependencies.bun run buildbun run typecheckbun run test: core 2063 passed, CLI 723 passed, SDK 88 passed, Dashboard 142 passed.mock-alpha/index.jsonlandmock-beta/index.jsonlbundles with separate row IDs and answer files.test_id/duplicate-suite dogfood: produced two distinct row sidecar directories inside one target bundle and preserved both summary rows.agentv/results/v1branch: pushed nested bundles, cleared local materialized results/cache, synced through Dashboard project APIs, and loaded remote list/detail/compare/drilldown successfully.codextarget withapi_format: responsesplusopenaigrader target withapi_format: chat, all credentials/model routed throughLOCAL_OPENAI_PROXY_*env refs. Result: PASS, 1/1, score 1.0. Run bundle:/tmp/agentv-av9vi-live-utlpOx/.agentv/results/av9vi-live-dogfood/2026-06-29T02-51-25-032Z/codex-local-proxy/index.jsonl; row sidecar:live-proxy-case--3d1e8b8bde59/run-1/; grader type:llm-grader; answer:agentv live dogfood ok.Private evidence: https://github.com/EntityProcess/agentv-private/tree/evidence/av-9vi-result-row-sidecars (commit 0b585d2) preserves the live local-proxy Codex target + OpenAI grader run bundle on an orphan branch.