Repository navigation
B-12: real-model benchmark (cargo xtask bench-model) - #66
Merged
austinintelligence merged 3 commits intoOct 15, 2026
Merged
Conversation
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01YAbe34cvkmUtsvXKH65Cq1
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Resets force-stop the app and restore Wi-Fi; a run whose goal already holds after reset is invalid and skipped. Unparsable --max-cost is an error with a finite default, killed runs are charged, token sums use explicit fields per CLI shape, and a failed reset or verifier no longer counts as an agent failure. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01YAbe34cvkmUtsvXKH65Cq1
testing.md keeps B-12's bench-model section and B-09's extended emulator-e2e paragraph; the xtask usage line lists both workflow-policy (A-02) and bench-model. Gates and helper javac pass. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NZLaa2TojSgBwyk35cJjco
austinintelligence
merged commit Oct 15, 2026
6766075
into
claude/friendly-dirac-embe85
1 check passed
austinintelligence
added a commit
that referenced
this pull request
Oct 15, 2026
A-01 budgets and fail-closed gates, with these merged into it in dependency order: B-10 #65, B-01 #62, A-20 #76, A-21 #61, A-09 #56, A-07 #68, A-04 #63, A-10 #59, A-11 #72, A-12 #57, A-13 #73, A-17 #58, A-18 #67, A-19 #70, B-11 #69, A-15 #64, B-02 #60, B-03 #71, B-05 #74, B-04 #75, B-06 #77, B-07 #78, B-13 #82, A-14 #86, A-16 #84, B-08 #87, B-09 #88, A-02 #79, A-03 #80, A-05 #83, A-06 #81, A-08 #85, B-12 #66. All checks green on Linux, macOS and Windows (Rust, helper build and format, installer, package smoke, CodeQL, dependency audit). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01NZLaa2TojSgBwyk35cJjco
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.
What changed
Adds
cargo xtask bench-model: runs a real agent CLI (command template with{prompt},{mcp},{model}) through 15 device goals intools/bench/tasks.json, each with reset commands and verifiers that decide success fromaureads of device state. Twoaubuilds (--a,--b) are run interleaved,--runsper task (default 5). MCP calls are counted by a relay shim (cargo xtask bench-shim) the agent launches as its MCP server. The report gives, per build and per task, success rate, calls, input tokens (when the agent CLI reports them), and wall time as mean with 95% CI, plus A−B with a Welch interval, as JSON and a table. Spend guard:--max-runs,--max-cost. Documented indocs/contributing/testing.md, including how to take the 1.2.0 baseline. CHANGELOG line added.Why
Every later 1.3.0 gate needs a measurement with a real model, not the scripted
benchmark-livepolicy.Safety and compatibility
adb, or page JavaScript; command forms changed incomputer/src/spec.rsonly. (Contributor tooling only; the agent still gets the unchangedau serve --mcp.)target/bench-model/.)Verification
cargo fmt --all -- --checkcargo clippy --workspace --all-targets --all-features -- -D warningscargo test --workspace --all-targetscargo xtask verify—cargo xtask check,docs, andsizepass (automation 2,128 of 2,350);verifystops here atnpm run lintbecauseoxlintis not installed in this container.npm testandnpm run lint— not run (no oxlint).New unit tests (
tools/src/bench_model.rs), each failing before the module existed:the_shipped_task_file_parses_with_a_verifier_per_task,a_template_fills_placeholders_and_keeps_the_prompt_one_argument,calls_are_counted_from_the_shim_log,usage_sums_input_token_fields_and_reads_cost,runs_interleave_builds_and_flip_order_each_round,confidence_intervals_use_the_t_distribution,the_report_compares_builds_per_task_and_overall.Also smoke-tested without a device: the shim relays a real
au serve --mcpsession and logs both calls, and a stub agent run reports tokens and trips the cost guard.Release impact
Not done here
Use Wi-Fi=off,Pair new device) are written from the documented screen format and have not been checked against a live emulator; expect to adjust some on the first real run.🤖 Generated with Claude Code
https://claude.ai/code/session_01YAbe34cvkmUtsvXKH65Cq1
Generated by Claude Code