Summary

• Fix bug: rtk prettier --check incorrectly reports "All files formatted correctly" when files need formatting #221: rtk prettier reported "All files formatted correctly" when prettier wasn't installed or produced no output. Now shows the actual error from stderr instead.
• Smoke tests: Added 49 new assertions to test-all.sh (69 → 118) covering rewrite, verify, proxy, discover, diff, wc, smart, json edge cases, docker, and hook check.
• Benchmark: Added missing commands to benchmark.sh (cargo, diff, smart, wc, curl, wget), fixed Python/Go commands to use dedicated filters, fixed relative path bug with temp fixture dirs.

Changes

src/prettier_cmd.rs

• Early exit when prettier produces no stdout and exits non-zero — shows stderr as-is
• Guard in filter_prettier_output() for empty/whitespace-only input
• 2 new tests: test_filter_empty_output, test_filter_whitespace_only_output

scripts/test-all.sh

• assert_fails() helper for expected-failure tests
• Sections 32-41: rewrite (basic, Bug: RTK_DISABLED=1 is ignored by rtk rewrite #345, Bug: rtk rewrite breaks 2>&1 shell redirection #346, fix: passthrough --json/--jq/--template flags in all gh subcommands #196), verify, proxy, discover, diff, wc, smart, json edge cases, docker (conditional), hook check

scripts/benchmark.sh

• Added cargo (build/test/clippy/check), diff, smart, wc, curl, wget sections
• Fixed Python: rtk ruff check . instead of rtk test ruff check .
• Fixed Go: rtk golangci-lint run instead of rtk test golangci-lint run, added go build/go vet
• BENCH_DIR now absolute (fixes debug file writes from temp dirs)
• Fallback to installed rtk if target/release/rtk not found

Test plan

• cargo test — all tests pass including 2 new prettier tests
• cargo clippy — no warnings
• bash scripts/test-all.sh — 118 assertions
• bash scripts/benchmark.sh — all sections run

Fixes #221

🤖 Generated with Claude Code