fix(ci): surface real xcodebuild errors on App target failure #170

2026-05-13T14:29:14Z

KofTwentyTwo commented

2026-05-13 14:29:14 +00:00

(Migrated from github.com)

Summary

CI run 25803540736 (and earlier runs) reported ** BUILD FAILED ** for the App target with no further diagnostics — -quiet 2>&1 | tail -5 swallowed every error line. We had to reproduce locally to discover the real failure (Found unhandled resource at packages/Harness/Corpora, fixed in #169).

This PR keeps the success path quiet (tail -5 of stdout) and adds proper diagnostics on the failure path:

Capture full xcodebuild output to a tempfile.
On success: behaves as before, prints the last 5 lines.
On failure: greps for error: / fatal error: lines, then prints the last 80 lines of output for context.

Test plan

bash scripts/check-app-builds.sh on a passing build — output unchanged from before.
On the next App-target failure, CI logs surface the real error: lines instead of just ** BUILD FAILED **.

🤖 Generated with Claude Code

## Summary CI run [25803540736](https://github.com/KofTwentyTwo/Jarvis/actions/runs/25803540736) (and earlier runs) reported `** BUILD FAILED **` for the App target with no further diagnostics — `-quiet 2>&1 | tail -5` swallowed every error line. We had to reproduce locally to discover the real failure (`Found unhandled resource at packages/Harness/Corpora`, fixed in #169). This PR keeps the success path quiet (`tail -5` of stdout) and adds proper diagnostics on the failure path: - Capture full xcodebuild output to a tempfile. - On success: behaves as before, prints the last 5 lines. - On failure: greps for `error:` / `fatal error:` lines, then prints the last 80 lines of output for context. ## Test plan - [x] `bash scripts/check-app-builds.sh` on a passing build — output unchanged from before. - [ ] On the next App-target failure, CI logs surface the real `error:` lines instead of just `** BUILD FAILED **`. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: KofTwentyTwo/Jarvis#170