feat(benchmarks): Support local Claude UI benchmark suites

[cameroncooke]

Add a local Claude Code benchmark harness for measuring UI task runs against XcodeBuildMCP.

This keeps the committed suite focused on first-party XcodeBuildMCP coverage while allowing private/local benchmark suites to live under a generic ignored benchmarks/claude-ui/local/ tree. The harness records observed benchmark data rather than treating metric deltas as regression failures, writes per-run artifacts, and reports task completion separately from measured tool usage and timing.

The committed baselines were refreshed from three canonical full-suite runs. Local/private suite results are intentionally not committed.

This PR is stacked on #427.

[cameroncooke]

• feat(benchmarks): Support local Claude UI benchmark suites #429 👈 (View in Graphite)
• feat(benchmarks): Add Claude UI benchmark harness #427
• feat(ui-automation): Add rs/1 runtime automation parity #416
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix prepared a fix for the issue found in the latest run.

• ✅ Fixed: Old sequence config key silently ignored instead of rejected
  • Added 'sequence' key to rejectRemovedConfigKeys with migration message 'removed; sequence checking is now observational only'

Or push these changes by commenting:

@cursor push 8dbf5edd5a

Preview (8dbf5edd5a)

diff --git a/src/benchmarks/claude-ui/config.ts b/src/benchmarks/claude-ui/config.ts
--- a/src/benchmarks/claude-ui/config.ts
+++ b/src/benchmarks/claude-ui/config.ts
@@ -247,6 +247,7 @@
     allowedVariance: 'removed; baselines are observed data only',
     expectedFailures: 'removed; benchmark stumbles are observed data',
     expectedToolSequence: 'renamed to baselineToolSequence',
+    sequence: 'removed; sequence checking is now observational only',
   };
   for (const [key, message] of Object.entries(removedKeys)) {
     if (raw[key] !== undefined) throw new Error(`${source}.${key}: ${message}`);

_{You can send follow-ups to the cloud agent here.}

^{Reviewed by Cursor Bugbot for commit ee9adf9. Configure here.}

[cameroncooke]

For the Bugbot summary about the removed sequence key: this is already addressed in b8e9353. The config loader now rejects sequence with a migration message instead of silently ignoring it.

[pkg-pr-new]

Open in StackBlitz

npm i https://pkg.pr.new/xcodebuildmcp@429

commit: 1a06b6d

[cameroncooke]

Addressed the additional benchmark hardening audit in d1625f5: ignored failure patterns no longer suppress unrelated real failures in the same result, claude.maxClaudeSeconds now rejects non-finite/non-positive values, and aggregate artifact roots now use path-aware containment instead of string-prefix comparison.