Version: v1.12.0 · File: batch/batch-runner.sh
The new read-only --status / --watch progress feature has two portability issues:
- Prerequisite gate: it runs only after
check_prerequisites, which exits if batch-input.tsv, batch-prompt.md, or the claude CLI is missing. A read-only progress check only needs batch-state.tsv, so inspecting a prior/remote run from an environment without Claude (or after input cleanup) never reaches print_status_table.
- bc dependency:
print_status_table computes score sum/average with bc, but check_prerequisites never verifies bc, and the rest of the script already migrated to awk. On minimal shells without bc, the average silently degrades to N/A.
Suggested fix: let --status/--watch short-circuit prerequisites (require only batch-state.tsv), and reuse the awk-based arithmetic from print_summary.
Flagged independently by Copilot, qodo, and Codex on a downstream sync PR.
Version: v1.12.0 · File:
batch/batch-runner.shThe new read-only
--status/--watchprogress feature has two portability issues:check_prerequisites, which exits ifbatch-input.tsv,batch-prompt.md, or theclaudeCLI is missing. A read-only progress check only needsbatch-state.tsv, so inspecting a prior/remote run from an environment without Claude (or after input cleanup) never reachesprint_status_table.print_status_tablecomputes score sum/average withbc, butcheck_prerequisitesnever verifiesbc, and the rest of the script already migrated toawk. On minimal shells withoutbc, the average silently degrades toN/A.Suggested fix: let
--status/--watchshort-circuit prerequisites (require onlybatch-state.tsv), and reuse theawk-based arithmetic fromprint_summary.Flagged independently by Copilot, qodo, and Codex on a downstream sync PR.