Repository navigation

[Test, do not merge] Claude review early exit and cancel - #26310

Closed

jkmassel wants to merge 45 commits into

trunkfrom

jkmassel/claude-review-exit-test

Closed

[Test, do not merge] Claude review early exit and cancel#26310
jkmassel wants to merge 45 commits into
trunkfrom
jkmassel/claude-review-exit-test

Commits on Oct 29, 2026
- Add an experimental Claude PR review CI check
  jkmassel
  committed
  ee733de
- Give the Claude PR review the Xcode MCP tools
  jkmassel
  committed
  c49eefc
- Grant headless Xcode the checkout folder for the Claude PR review
  jkmassel
  committed
  11e9001
- Start the Claude PR review without waiting for the Jetpack build
  jkmassel
  committed
  a65da95
- Address the Claude PR review's own findings
  jkmassel
  committed
  2d40609
- Post the full Claude PR review result as a PR comment
  jkmassel
  committed
  ac2066f
- Show Claude PR review screenshots on GitHub via Spacefast
  jkmassel
  committed
  6e1bfe4
- Run the Claude PR review as a separate macOS user
  jkmassel
  committed
  30f402a
- Unmount the VM's shared credentials before the Claude review
  jkmassel
  committed
  5725111
- Don't copy the reviewer's evidence back as root
  jkmassel
  committed
  483ac9e
- Write the reviewer's Xcode MCP setting to its own home
  jkmassel
  committed
  ec3cc27
- Have the reviewer enable Xcode MCP from its own session
  jkmassel
  committed
  8b2625d
- Give the reviewer an unlocked keychain before enabling Xcode MCP
  jkmassel
  committed
  5d9bc78
- Only offer Xcode MCP to the review when the bridge really starts
  jkmassel
  committed
  b99ce62
- Let the Claude PR review tap around and create WordPress test sites
  jkmassel
  committed
  d864a80
- Address the Claude PR review's test site and AXe findings
  jkmassel
  committed
  e06140b
- Validate end to end that the Claude PR review can test real PRs
  jkmassel
  committed
  461880a
- Pass the setup validation flag as an argument
  jkmassel
  committed
  7989ae7
- Publish review screenshots to jkmassel-screenshots; check the secrets env file
  jkmassel
  committed
  47aabe0
- Make the secrets env checker unable to print parts of values
  jkmassel
  committed
  05bcfbb
- Stop running the Claude review setup validation on every build
  jkmassel
  committed
  103f327
- Remove the temporary secrets env file check
  jkmassel
  committed
  36a3b77
- Make the Claude review wait for its builds
  jkmassel
  committed
  7315c6f
- Keep only the current Claude review verdict on a PR
  jkmassel
  committed
  c3fd877
- Re-run the Claude review when wpmobilebot is asked to review
  jkmassel
  committed
  af06425
- Record everything the Claude PR review does on the VM
  jkmassel
  committed
  86c9dcb
- Keep the transcript's output away from the test site servers
  jkmassel
  committed
  24d8a45
- Show the Claude review's tool calls in the build log as they happen
  jkmassel
  committed
  2397b2b
Commits on Oct 30, 2026
- Write the Claude transcript to an append-only file
  jkmassel
  committed
  93ce08d
- Don't wait forever for the audit logs to stop
  jkmassel
  committed
  6692ed9
- Keep the Anthropic key out of the audit logs
  jkmassel
  committed
  dafd975
- Upload the audit logs during the Claude review, and cut their noise
  jkmassel
  committed
  3663687
- Compress large audit summaries before upload
  jkmassel
  committed
  1a14944
- Keep the Anthropic key out of Claude's launch arguments and the audit log
  jkmassel
  committed
  56389e1
- Stop the reviewer's processes before any step that uses a secret
  jkmassel
  committed
  afaa27c
- Upload the audit results as one archive
  jkmassel
  committed
  5c972db
- Clear the Claude review request however the run ends
  jkmassel
  committed
  89637ab
- Give the Claude reviewer the Swift packages up front
  jkmassel
  committed
  bea8ec8
- TEMP: run the setup validation to measure a real build
  jkmassel
  committed
  b4dec67
- Revert "TEMP: run the setup validation to measure a real build"
  jkmassel
  committed
  09c2c2a
- Merge remote-tracking branch 'origin/trunk' into jkmassel/claude-blockers-pr-prototype
  jkmassel
  committed
  a6cbd62
- Restore the Swift package cache for the Claude reviewer
  jkmassel
  committed
  3bc1f20
- TEST: fail right after setup, with only the Claude review step
  jkmassel
  committed
  ff864c7
- TEST: wait instead of running Claude, to be cancelled
  jkmassel
  committed
  5854fce
- Clear the Claude review request when the job starts
  jkmassel
  committed
  fd6b660

Commits

Commits on Oct 29, 2026

Commits on Oct 30, 2026