feat(results): simplify project + results config to one flat symmetric schema#1565
Merged
Conversation
…c schema
Replace the dual flat/nested results config with a single flat shape shared by
the project source repo and the results block: repo (slug or URL), path (local
checkout), branch, and auto_push. Drop the removed wire fields entirely (hard
cut, no back-compat): results.remote, repo_url/repo_path, nested repo objects,
sync.{auto_push,push_conflict_policy,require_push}, and branch_prefix.
The internal NormalizedResultsConfig still derives repo_url/repo_path and the
origin remote alias; only the YAML wire surface is simplified.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Deploying agentv with
|
| Latest commit: |
f09120c
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://941d155b.agentv.pages.dev |
| Branch Preview URL: | https://av-results-schema-simplify.agentv.pages.dev |
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
Replaces the dual flat/nested project + results config with one flat symmetric schema shared by the project source repo and the results block. Hard cut — no back-compat (per maintainer decision).
Removed wire fields (now fail validation)
results.remote,repo_url,repo_path, nestedrepo:/results.repo:objects,ref/source,results.sync.*(use flatauto_push),branch_prefix,push_conflict_policy,require_push.The internal
NormalizedResultsConfigstill derivesrepo_url/repo_pathand aremote='origin'alias — only the YAML wire surface is simplified.--results-require-pushremains a per-run CLI override.Rationale
resultsis nearly always a repo, so a flat shape is sufficient — nesting added no value.remotefield (name-vs-URL ambiguity). Like beads, the git remote name is repo-owned state; AgentV assumesoriginand never runsgit remote add/set-url.Changes
packages/core—ProjectResultsConfig,ResultsConfigwire type,parseResultsConfig,fromYaml/toYaml,normalizeResultsConfig, validation.apps/cliresults/run-eval mapping;apps/dashboardtypes/status.dashboard.mdx,results.mdx,running-evals.mdx) incl. a migration section.Verification
bun run build✅bun run lint✅~/.agentv/config.yaml; new build loads all 9 projects correctly.Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com