Commit 6e7c322
feat(config)!: hard-deprecate the graders: config block — a grader is just a target
Peer-framework research this session (Margin-Lab/evals, Harbor) found neither
maintains a separate grader/judge schema type distinct from their target/agent
config — Margin-Lab's own docs say it outright: "there is no separate grader
abstraction." AgentV's own design principles already prefer config-level
grader selection through `defaults.grader` over target-level `grader_target`,
and the live `.agentv/targets.yaml` execution path already resolves
`grader_target`/`defaults.grader` against the same target-id namespace. The
only place a separate "grader" entity still existed was the newer, dead
`graders:` top-level config-graph schema (`NormalizedGraderConfig`) — zero
implementation depth, nothing ever turned it into a Provider. A grader can
also need grading itself (oracle/calibration checks), which only works for
free if it's addressable the same way as any other target.
- config-graph.ts: remove `NormalizedGraderConfig`/`parseGraders`/the
`graders` field from `ComposableConfigGraph`; authoring a `graders:` block
now throws a clear error pointing at folding entries into `targets`.
`defaults.grader` now validates against the same `graph.targets` pool as
`defaults.target`, instead of the removed `graph.graders`.
- eval-file.schema.ts: same hard-rejection for the Zod schema backing the
published eval.schema.json (IDE-autocomplete surface), matching the existing
pattern for other removed fields (`eval_cases`, `providerPromptMap`).
- config-loader.ts: drop the `graders` pass-through.
- Updated the one place in docs, README.md, and the readme-quickstart example
that authored a `graders:` block, folding each grader into `targets:`.
readme-quickstart's targets.yaml/graders.yaml also still had the legacy
`${{ VAR }}` env syntax (pre-existing, unrelated breakage) — fixed to
`{{ env.VAR }}` in the same pass since the file needed touching anyway.
- Added a CONCEPTS.md entry disambiguating grader-selection (a target) from
grader-method (an assertion type) and grader-prompt (criteria/rubric).
Verified: full workspace typecheck and lint clean; 935 tests across the
touched loader/validation/orchestrator suites pass; all 109 example eval
files validate; live CLI check confirms the error fires end-to-end and the
folded-into-targets shape validates cleanly.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>1 parent 589c01a commit 6e7c322
20 files changed
Lines changed: 156 additions & 128 deletions
File tree
- apps/web/src/content/docs/docs/next
- evaluation
- targets
- examples/features/readme-quickstart
- .agentv
- packages/core
- src/evaluation
- loaders
- validation
- test/evaluation
- loaders
- validation
- skills-data/agentv-eval-writer/references
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| 13 | + | |
| 14 | + | |
13 | 15 | | |
14 | 16 | | |
15 | 17 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
47 | | - | |
48 | 47 | | |
49 | 48 | | |
| 49 | + | |
50 | 50 | | |
51 | 51 | | |
52 | 52 | | |
| |||
Lines changed: 10 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
474 | 474 | | |
475 | 475 | | |
476 | 476 | | |
477 | | - | |
| 477 | + | |
| 478 | + | |
| 479 | + | |
478 | 480 | | |
479 | 481 | | |
480 | 482 | | |
| |||
486 | 488 | | |
487 | 489 | | |
488 | 490 | | |
489 | | - | |
490 | 491 | | |
491 | 492 | | |
| 493 | + | |
492 | 494 | | |
493 | 495 | | |
494 | 496 | | |
| |||
511 | 513 | | |
512 | 514 | | |
513 | 515 | | |
514 | | - | |
515 | 516 | | |
516 | 517 | | |
517 | 518 | | |
| |||
523 | 524 | | |
524 | 525 | | |
525 | 526 | | |
| 527 | + | |
| 528 | + | |
| 529 | + | |
| 530 | + | |
| 531 | + | |
| 532 | + | |
526 | 533 | | |
527 | 534 | | |
528 | 535 | | |
| |||
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
24 | | - | |
25 | 24 | | |
26 | 25 | | |
| 26 | + | |
27 | 27 | | |
28 | 28 | | |
29 | 29 | | |
| |||
133 | 133 | | |
134 | 134 | | |
135 | 135 | | |
136 | | - | |
137 | 136 | | |
138 | 137 | | |
| 138 | + | |
139 | 139 | | |
140 | 140 | | |
141 | 141 | | |
| |||
Lines changed: 4 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
18 | | - | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
19 | 21 | | |
20 | 22 | | |
21 | 23 | | |
| |||
27 | 29 | | |
28 | 30 | | |
29 | 31 | | |
30 | | - | |
31 | 32 | | |
32 | 33 | | |
| 34 | + | |
33 | 35 | | |
34 | 36 | | |
35 | 37 | | |
| |||
Lines changed: 15 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
13 | | - | |
14 | | - | |
| 13 | + | |
| 14 | + | |
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
| |||
43 | 43 | | |
44 | 44 | | |
45 | 45 | | |
46 | | - | |
47 | 46 | | |
48 | 47 | | |
| 48 | + | |
49 | 49 | | |
50 | 50 | | |
51 | 51 | | |
| |||
61 | 61 | | |
62 | 62 | | |
63 | 63 | | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
64 | 69 | | |
65 | 70 | | |
66 | 71 | | |
| |||
128 | 133 | | |
129 | 134 | | |
130 | 135 | | |
131 | | - | |
132 | 136 | | |
133 | 137 | | |
| 138 | + | |
134 | 139 | | |
135 | 140 | | |
136 | 141 | | |
| |||
151 | 156 | | |
152 | 157 | | |
153 | 158 | | |
154 | | - | |
155 | 159 | | |
156 | 160 | | |
157 | 161 | | |
| |||
167 | 171 | | |
168 | 172 | | |
169 | 173 | | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
170 | 180 | | |
171 | 181 | | |
172 | 182 | | |
| |||
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
184 | 184 | | |
185 | 185 | | |
186 | 186 | | |
187 | | - | |
188 | 187 | | |
189 | 188 | | |
| 189 | + | |
190 | 190 | | |
191 | 191 | | |
192 | 192 | | |
| |||
Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
66 | 66 | | |
67 | 67 | | |
68 | 68 | | |
69 | | - | |
| 69 | + | |
70 | 70 | | |
71 | 71 | | |
| 72 | + | |
72 | 73 | | |
73 | 74 | | |
74 | 75 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | | - | |
3 | 2 | | |
4 | 3 | | |
This file was deleted.
0 commit comments