agentv-eval-review

name	agentv-eval-review
description	Use when reviewing eval YAML files for quality issues, linting eval files before committing, checking eval schema compliance, or when asked to "review these evals", "check eval quality", "lint eval files", or "validate eval structure". Do NOT use for writing evals (use agentv-eval-writer) or running evals (use agentv-bench).

Eval Review

Overview

Lint and review AgentV eval YAML files for structural issues, schema compliance, and quality problems. Apply this checklist deterministically first, then layer LLM judgment for semantic issues a checklist cannot catch.

Process

Step 1: Structural checklist

Walk every target eval file and report violations grouped by severity (error > warning > info). For each finding, include the file path and a concrete fix.

• File extension is .eval.yaml (error if not).
• description field is present at the top level (error if missing).
• Each entry under tests has id, input, and at least one of criteria / expected_output / assert (error if missing).
• File-typed inputs (type: file) use a leading / in their path (error if relative).
• Tests have an assert block — flag tests that rely solely on expected_output (warning).
• Flag criteria that duplicates assertion strings when assertions already express the grading contract (warning — remove the duplicate criteria).
• Prefer plain assertion strings over multiple named type: llm-rubric blocks when the default LLM rubric grader can evaluate the checks (info unless custom prompts or grader targets are present).
• Detect expected_output prose patterns like "The agent should..." or "The output is..." (warning — expected_output should be a golden/reference answer; scoring rules belong in assertions or, for implicit-grader cases, criteria).
• For historical or repo-state evals, verify the relevant repo is pinned under workspace.repos[].commit; a SHA mentioned only in prompt prose or metadata is not an operational checkout (warning).
• Identical file inputs repeated across multiple tests in the same eval should be hoisted to a top-level input (info).
• Eval files in the same directory should share a common id prefix (info — flag drift).

Step 2: Semantic review (LLM judgment)

The structural checklist catches mechanical issues but cannot assess:

• Factual accuracy — Do tool/command names in expected_output match what the skill documents?
• Coverage gaps — Are important edge cases missing?
• Assertion discriminability — Would assertions pass for both good and bad output?
• Cross-file consistency — Do output filenames match across evals and skills?

Read the relevant SKILL.md files and cross-check against the eval content for these issues.

Accessing reference files

To load a specific reference without pulling the entire skill into context:

agentv skills get agentv-eval-review --ref <filename>

Or resolve the skill directory and read files directly:

cat $(agentv skills path agentv-eval-review)/references/<filename>.md