Skip to content

chore: rename dataset to eval_set, derive from YAML name field#678

Merged
christso merged 2 commits into
mainfrom
chore/rename-dataset-to-eval-set
Mar 19, 2026
Merged

chore: rename dataset to eval_set, derive from YAML name field#678
christso merged 2 commits into
mainfrom
chore/rename-dataset-to-eval-set

Conversation

@christso
Copy link
Copy Markdown
Collaborator

@christso christso commented Mar 19, 2026

Summary

  • Renames the dataset property to eval_set across types (EvalTest, EvaluationResult), schema, parsers, orchestrator, CLI commands, OTel exporter, and JSON schema
  • Changes the derivation logic so eval_set is sourced from the YAML name field (not the old dataset field), with filename as fallback
  • Updates JSONL sidecar metadata to read name instead of dataset
  • Replaces dataset: with name: in 3 example YAML files that previously used dataset:
  • Adds name: field to 10 representative example YAML files that only had description:
  • Renames "dataset" keys to "eval_set" in all baseline/results JSONL files (41 files)
  • Updates CLI --group-by option from dataset to eval-set
  • Updates README documentation examples

Test plan

  • Core package type-checks clean (tsc --noEmit)
  • All pre-push hooks pass (Build, Typecheck, Lint, Test)
  • Core tests pass: orchestrator, yaml-parser-metadata, jsonl-parser, baseline, evaluators, execution-metrics, execution-status, matrix-targets, streaming-observer (262+ tests)
  • CLI tests pass: output-writers, export, export-e2e-providers, trace (94 tests)
  • Verify no downstream consumers depend on the dataset field name in JSONL output

🤖 Generated with Claude Code

christso and others added 2 commits March 19, 2026 03:01
- Rename `dataset` property to `eval_set` in EvalTest, EvaluationResult,
  and eval-file schema types
- Change parser derivation logic to read `name` field from YAML (not
  `dataset`) with filename fallback for eval_set
- Update JSONL parser sidecar metadata to read `name` instead of `dataset`
- Update orchestrator, CLI commands (junit-writer, trace show/stats),
  and OTel exporter to use the new field name
- Replace `dataset:` with `name:` in 3 example YAML files that had it
- Add `name:` field to 10 representative example YAML files
- Rename `"dataset"` keys to `"eval_set"` in all baseline/results JSONL
- Update JSON schema and documentation

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying agentv with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8737766
Status: ✅  Deploy successful!
Preview URL: https://d7d1dc6b.agentv.pages.dev
Branch Preview URL: https://chore-rename-dataset-to-eval.agentv.pages.dev

View logs

@christso christso marked this pull request as ready for review March 19, 2026 03:33
@christso christso merged commit fb84821 into main Mar 19, 2026
1 check passed
@christso christso deleted the chore/rename-dataset-to-eval-set branch March 19, 2026 03:56
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.

1 participant