Skip to content

test: add agent evals - #229

Open
angeloashmore wants to merge 40 commits into
mainfrom
aa/evals
Open

test: add agent evals#229
angeloashmore wants to merge 40 commits into
mainfrom
aa/evals

Conversation

@angeloashmore

@angeloashmore angeloashmore commented Jul 21, 2026

Copy link
Copy Markdown
Member

Resolves:

Description

Adds an eval suite that measures how well AI agents (like Claude Code) use the CLI.

Each eval gives an agent a task in a temp project. The agent runs the real CLI against a real repo, and the test checks the outcome. A few notes on the design:

  • Evals are plain Vitest tests. They reuse the existing e2e fixtures and grade results with normal asserts, plus an LLM judge for criteria that need judgement (for example, "are these field types sensible?").
  • Each eval runs several trials. We track pass rates per run instead of a single pass/fail, since agent behavior varies.
  • Evals are measurements, not gates. Evals the agent fails today stay enabled and score 0% until the CLI improves; an eval is only skipped when the CLI cannot pass it (missing feature).
  • Each eval gets its own throwaway repository, so evals run concurrently without interfering and agents are free to push. Global teardown sweeps repositories created by agents that the harness cannot name in advance (e.g. prismic init).
  • Results are a single evals/results.json the reporter overwrites each run: trials grouped by eval, sorted by name, with each trial's pass/fail, cost, duration, and the prismic commands it ran. node --run evals:report (a jq one-liner) prints the latest run.
  • Run history is deliberately delegated to git: every committed run is a version of results.json, so git acts as the time-aware database. git log -- evals/results.json lists past runs, git show <rev>:evals/results.json reads any of them, and an analysis agent can walk those versions to compare pass rates, cost, and command usage across CLI commits — no run ids, timestamps, or history tooling in the harness itself.
  • Iterating is safe by default: filtered runs (node --run evals -- sync-models, -t "adds a field", it.only) write only the gitignored evals/results.local.json, never results.json — the reporter detects them, so a committed snapshot is always a full run. Agents can diff results.local.json against results.json to compare a working-tree run with the last recorded one. The one convention: permanently disabled evals use it.todo, since vitest reports it.skip identically to a filter.
  • Evals are opt-in (node --run evals) and refuse to run outside a container or CI, because the agent runs with permissions bypassed.
  • A full run is 33 evals × 3 trials: roughly $31 in API-equivalent cost and 10 minutes.

Checklist

  • A comprehensive Linear ticket, providing sufficient context and details to facilitate the review of the PR, is linked to the PR.
  • If my changes require tests, I added them.
  • If my changes affect backward compatibility, it has been discussed.
  • If my changes require an update to the CONTRIBUTING.md guide, I updated it.

Preview

node --run evals runs the suite as normal Vitest tests and writes evals/results.json. The file is stored on a single line so each recorded run diffs as one changed line; formatted excerpt:

{
	"model": "claude-sonnet-5",
	"evals": {
		"adds a field": [
			{
				"pass": true,
				"costUsd": 0.35,
				"durationS": 20,
				"calls": [
					"field add --help",
					"field add rich-text excerpt --to-type article --label Excerpt"
				]
			}
		]
	}
}

node --run evals:report prints the latest run, one line per eval (excerpt):

3/3  20s   adds a field
2/3  29s   commits and pushes local model changes
0/3  35s   models a title as single-heading rich text and a social media handle as key text
3/3  216s  models and wires a landing page end to end

How to QA 1

Inside a disposable container with .env.test.local available:

PRISMIC_ALLOW_EVALS=true node --run evals
node --run evals:report
git diff evals/results.json

🤖 Generated with Claude Code


Note

Medium Risk
Evals run a real agent with permission bypass against live Prismic accounts and billable Anthropic usage; risk is confined to isolated/CI runs, with a small CLI change to telemetry env overrides.

Overview
Adds an opt-in Vitest eval project that drives the real prismic CLI through the Claude Agent SDK (with permissions bypassed) against disposable repos and temp Next.js projects, so agent behavior can be measured over multiple trials instead of gated as unit tests.

The harness in evals/it.ts wires the built CLI into each project, appends the published Prismic skill, records every prismic invocation, and exposes matchers like toHaveRun and LLM-based toSatisfyJudge. Scenarios live in new *.eval.ts files (modeling, routes, repo setup, sync/push, init, safety checks, slice components, etc.). A custom reporter writes evals/results.json on full runs and evals/results.local.json on filtered runs, with an evals:report script to summarize pass rates, cost, and duration.

Tooling changes: Vitest is split into tests vs evals projects; node --run evals is guarded by PRISMIC_ALLOW_EVALS=true. E2E fixtures gain a generic exec helper; .env.test.example documents ANTHROPIC_API_KEY. Telemetry/Sentry in src/index.ts now honor explicit PRISMIC_*_ENABLED env overrides (used to quiet eval runs). A baseline evals/results.json snapshot is committed for git-based history.

Reviewed by Cursor Bugbot for commit 70b0523. Bugbot is set up for automated code reviews on this repo. Configure here.

Footnotes

  1. Please use these labels when submitting a review:
    ❓ #ask: Ask a question.
    💡 #idea: Suggest an idea.
    ⚠️ #issue: Strongly suggest a change.
    🎉 #nice: Share a compliment.

angeloashmore and others added 10 commits July 20, 2026 21:06
Plain vitest evals that drive a real Claude Code agent (via the Agent SDK)
against the built CLI in a generic, hermetic config. Adds the `agent` fixture,
an `toHaveRun` matcher, an LLM-judge matcher with a never-regress ratchet, and a
per-run reporter that records efficiency metrics and the agent's prismic calls.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Merge the agent fixture, matchers, judge, config isolation, and result
recording into eval/it.ts; delete claude.ts, report.ts, and setup.ts.

Move the judge ratchet from a read-back of results.jsonl into an inline
threshold literal that vitest -u rewrites like a snapshot, so results.jsonl
is write-only. Trim recorded fields to effectiveness and effort (turns,
tokens, duration); drop cost, which swings with provider pricing.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
- Judge uses the SDK's native structured output instead of hand-parsing JSON;
  score is range-checked to [0, 1]
- Create a throwaway Claude config dir per agent/judge call and clean it up;
  set persistSession: false so runs stay ephemeral
- Detect -u from vitest's update state; the worker's process.argv is empty
- Fix raiseLiteral rewriting the wrong literal on the callsite line, and
  support inline bars
- Fix the evals npm script so vitest also receives RUN_EVALS

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Each eval now runs EVAL_TRIALS times (default 3) via it.for; a custom
reporter prints per-eval pass rates with infra failures excluded and
appends per-trial rows (pass, cost, tokens, turns, commands) to
results.jsonl. The judge is binary like agent-eval's: prose criteria,
pass/fail verdicts, no scores, no ratcheting bars. The full eval catalog
is implemented as it.skip drafts; only the rich-text smoke eval runs.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
`node --run evals:report` prints one block per eval with one line per
run (pass rate, cost, mean tokens/turns/duration) and a latest-run
footer with best-of and per-trial rates. Rows now carry a per-run id so
the digest groups runs exactly.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Rows record the git SHA that produced them, shown in the digest so rate
changes point at a commit range. The file keeps the newest 100 runs; the
digest shows the last 10 per eval. results.jsonl merges with the union
driver so branch runs never conflict.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The judge now calls the Messages API directly with structured outputs
instead of spawning a Claude Code session. The agent fixture writes its
own task.meta via the built-in task context, trial rows no longer carry
a parsed trial number, and eval env vars are parsed with a zod schema.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Enable 26 validated evals with per-trial runs and skip 3 that fail
consistently, with the reason noted on each. Trim duplicate evals and
merge the two slice component evals into one.

Each eval now gets its own repository via an isolatedRepo fixture,
letting evals run 8-way concurrent without cross-test interference.
Global teardown sweeps orphaned repositories created by agents
(e.g. prismic init) that the harness cannot name in advance.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Add four evals from the Next.js happy-path docs: init with an existing
repository, page types vs custom types, commit-then-push flow, and
production previews after a deploy. The last two fail today and stay
enabled: evals are measurements, and their pass rates track CLI gaps.
Un-skip the other agent-capability evals for the same reason; only the
field rename eval stays skipped since the CLI cannot rename fields.

toHaveRun now evaluates each shell segment separately and ignores
--help invocations, so reading help no longer counts as running a
command in either direction.

Scope the results digest to the last EVAL_HISTORY runs (default 10)
with a per-run footer, and document the results.jsonl schema.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@angeloashmore angeloashmore changed the title feat: add agent evals test: add agent evals Jul 21, 2026
angeloashmore and others added 16 commits July 22, 2026 02:19
Adopts main's isolateRepo fixture API from #231: drops the serial test
project split, moves the exec fixture into eval/it.ts, and switches the
eval repo isolation to isolateRepo: true.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Evals only shell out to git; the generic runner's options plumbing was
left over from the pre-#231 CLI fixture. throwOnError surfaces setup
failures that previously passed silently.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
… builds

Evals now require PRISMIC_ALLOW_EVALS=true and an explicit --project evals;
bare vitest runs only unit tests. Tests run against the prebuilt dist (no
MODE=test build), sync's poll interval is a validated PRISMIC_SYNC_POLL_MS
env var, agent-created repos are cleaned up per test instead of an
account-wide sweep, and dead MODE/DEV env plumbing is removed.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…elper

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Take main's test-build infra from #232 (CLI built in test global setup,
always-minified tsdown config) and keep the evals project config on top.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Export AgentRecord with field docs, build Row from it in the reporter,
and reuse that type in the report instead of a duplicated schema comment.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…n read

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Vitest silently ignores `sequence` inside a `projects` entry, so every
test in a file ran serially. Root-level `sequence.concurrent` propagates
to projects and restores within-file concurrency.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@lihbr
lihbr marked this pull request as ready for review July 24, 2026 01:28
Comment thread evals/it.ts
Comment thread evals/it.ts Outdated
Comment thread evals/reporter.ts Outdated

@lihbr lihbr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Look good to me, I like what you did with Vite and using a custom reporter, the failing evals are also really insightful :)

Comment thread evals/it.ts Outdated
Comment thread eval/evals.eval.ts Outdated
…ILL.md

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Comment thread evals/it.ts Outdated
angeloashmore and others added 2 commits July 27, 2026 23:41
…ts.json

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Comment thread evals/it.ts
Comment thread evals/it.ts
Comment thread evals/it.ts
@angeloashmore
angeloashmore marked this pull request as draft July 27, 2026 23:51
angeloashmore and others added 7 commits July 28, 2026 00:18
Partial runs (file filters, -t, it.only) are detected by the reporter and
never overwrite results.json, so it always holds one full run. Permanent
disables must be it.todo, which vitest keeps distinguishable from filter
skips. Also fixes the remaining bin/ mkdir race in the project fixture.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Every run rewrites most values, so a pretty-printed file only inflates PR
diffs; one line keeps run-over-run changes to one changed line. Read it
with jq or node --run evals:report.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Aligns the eval fixtures with test/it.ts and drops the vitest 4.1
builder-pattern API, shrinking the lockfile diff to the agent SDK.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…nfig

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Tests that assert non-zero exit codes need the result object back, not a
thrown error. Restores main's behavior; the eval git setup surfaces
failures through its own assertions.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@angeloashmore
angeloashmore marked this pull request as ready for review July 28, 2026 01:08

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c5fdf22. Configure here.

Comment thread evals/it.ts
Comment thread evals/reporter.ts
angeloashmore and others added 3 commits July 28, 2026 01:29
… agent

A fixture failure before the prompt leaves a failed test with no recorded
trial. Recording the rest as a full run would misstate pass rates, so such
runs now count as partial.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Every CLI invocation fired a fire-and-forget /profile request even with
telemetry disabled, since the call sat outside the telemetry gates. In CI
this put every spawned CLI process on user-service concurrently, which
starts returning 500s at roughly eight concurrent same-user requests.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.

2 participants