Multiple review comments identified bugs in .just/copilot.just:
sed -i not portable (13/15 PRs) - BSD vs GNU incompatibility at line 208. Fix: use temp-file approach for cross-platform compatibility
copilot_rollback lossy path reconstruction (12/15 PRs) - Using _ for / replacement is ambiguous. Fix: use URL-encoding or base64
copilot_pick ambiguous line-number lookup (11/15 PRs) - Matches by line number alone, not path + line. Fix: match by both
- Predictable temp file paths (4/15 PRs) - Use
mktemp instead of hardcoded /tmp/copilot_count_*
gum spin serialization fragile (8/15 PRs) - Using declare -f can break with complex function state
Identified in code reviews across 15 PRs using gh-process v5.8.
Multiple review comments identified bugs in
.just/copilot.just:sed -inot portable (13/15 PRs) - BSD vs GNU incompatibility at line 208. Fix: use temp-file approach for cross-platform compatibilitycopilot_rollbacklossy path reconstruction (12/15 PRs) - Using_for/replacement is ambiguous. Fix: use URL-encoding or base64copilot_pickambiguous line-number lookup (11/15 PRs) - Matches by line number alone, not path + line. Fix: match by bothmktempinstead of hardcoded/tmp/copilot_count_*gum spinserialization fragile (8/15 PRs) - Usingdeclare -fcan break with complex function stateIdentified in code reviews across 15 PRs using gh-process v5.8.