Fixed bash shebangs for portability
Public
Fixed bash shebangs for portability
Agent
Pi
Model
Claude Opus 4.5 (latest)
Started
Jan 22, 8:42 PM
Messages41
Downloads0
Analyzed GitHub issue and related files.
UnderstandUpdated bash shebangs in multiple scripts.
EditCommitted and pushed changes to main branch.
BuildCommented on issue confirming the fix.
Verify
Analyze GitHub issue(s): https://github.com/badlogic/pi-mono/issues/910
For each issue:
Read the issue in full, including all comments and linked issues/PRs.
For bugs:
For feature requests:
Do NOT implement unless explicitly asked. Analyze and propose only.
The user wants me to analyze GitHub issue #910. Let me fetch the issue details first.
>gh issue view 910 --repo badlogic/pi-mono --json title,body,comments,labels,stateThis is a feature request to change shebangs in shell scripts from #!/bin/bash to #!/usr/bin/env bash for better portability (e.g., NixOS doesn't have /bin/bash).
Let me find all .sh files in the repo and check their current shebangs.
>find . -name "*.sh" -type f | head -50Let me check the shebangs of all these shell scripts to see which ones need updating.
>for f in $(find . -name "*.sh" -type f); do echo "=== $f ==="; head -1 "$f"; doneNow I have a clear picture. Let me summarize which files need to be updated:
Files already using #!/usr/bin/env bash (no change needed):
./scripts/build-binaries.sh./packages/pods/dist/scripts/pod_setup.sh./packages/pods/dist/scripts/model_run.sh./packages/pods/scripts/pod_setup.sh./packages/pods/scripts/model_run.sh./pi-test.shFiles using #!/bin/bash (need update):
./packages/coding-agent/binaries/darwin-arm64/examples/extensions/doom-overlay/doom/build.sh./packages/coding-agent/examples/extensions/doom-overlay/doom/build.sh./packages/coding-agent/scripts/migrate-sessions.sh./packages/mom/dev.sh./packages/mom/docker.sh./test.shOther (no shebang):
./.husky/_/husky.sh - This doesn't have a shebang, it starts with echo "husky - DEPRECATED. This is likely sourced, not executed directly.Note: The binaries/darwin-arm64/ path looks like it might be a build artifact. Let me check if it's in gitignore.
>git check-ignore packages/coding-agent/binaries/darwin-arm64/examples/extensions/doom-overlay/doom/build.sh && echo "IGNORED" || echo "NOT IGNORED"The binaries/darwin-arm64 file is gitignored, so it shouldn't be in the repo and doesn't need to be changed. Let me also check the dist folder:
>git check-ignore packages/pods/dist/scripts/pod_setup.sh && echo "IGNORED" || echo "NOT IGNORED"Good, the dist/ folder is also gitignored. So I only need to track the source files that are actually committed.
Type: Feature request (portability improvement)
Summary: Change shebangs from #!/bin/bash to #!/usr/bin/env bash for better portability across systems like NixOS where /bin/bash doesn't exist.
After scanning all .sh files in the repo, these 6 files need their shebang updated:
| File | Current |
|---|---|
packages/coding-agent/examples/extensions/doom-overlay/doom/build.sh | #!/bin/bash |
packages/coding-agent/scripts/migrate-sessions.sh | #!/bin/bash |
packages/mom/dev.sh | #!/bin/bash |
packages/mom/docker.sh | #!/bin/bash |
test.sh | #!/bin/bash |
scripts/build-binaries.sh - already uses #!/usr/bin/env bashpackages/pods/scripts/pod_setup.sh - already correctpackages/pods/scripts/model_run.sh - already correctpi-test.sh - already correct.husky/_/husky.sh - no shebang (deprecated/sourced file)packages/coding-agent/binaries/... - gitignored build artifactpackages/pods/dist/... - gitignored build outputSimple one-line change in each file: replace #!/bin/bash with #!/usr/bin/env bash.
This is a low-risk change that improves compatibility with non-standard Linux distributions and BSDs where bash may not be at /bin/bash.
do it, commit and push and write "Thanks, fixed in main" as a comment on the issue
The user wants me to implement the changes, commit, push, and comment on the issue. Let me do the edits first.
All edits done. Now let me commit with the specific files, push, and comment on the issue.
>git add packages/coding-agent/examples/extensions/doom-overlay/doom/build.sh packages/coding-agent/scripts/migrate-sessions.sh packages/mom/dev.sh packages/mom/docker.sh test.sh && git commit -m "fix: use portable shebang in shell scripts
Change #!/bin/bash to #!/usr/bin/env bash for better portability
on systems like NixOS where /bin/bash doesn't exist.
Fixes #910"Commit successful. Now push and comment on the issue.
>git pushNow add the comment to the issue.
>gh issue comment 910 --repo badlogic/pi-mono --body "Thanks, fixed in main"Done! The issue should be automatically closed because I used "Fixes #910" in the commit message.
Done. Updated 5 files, committed with Fixes #910 (auto-closes the issue), pushed to main, and commented on the issue.