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,state{"body":"### What do you want to change?\n\nChange all shebangs of `*.sh` files to `#!/usr/bin/env bash`\n\n### Why?\n\nUsing `bash` found in `PATH` is more flexible and portable.\n\nFor example on NixOS, there is no `/bin/bash` and therefore the script is not executable on its own.\n\n### How? (optional)\n\n_No response_","comments":[],"labels":[],"state":"OPEN","title":"Adapt bash shebangs to use env"}This 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 -50./.husky/_/husky.sh
./scripts/build-binaries.sh
./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/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
./packages/mom/dev.sh
./packages/mom/docker.sh
./pi-test.sh
./test.shLet 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"; done=== ./.husky/_/husky.sh ===
echo "husky - DEPRECATED
=== ./scripts/build-binaries.sh ===
#!/usr/bin/env bash
=== ./packages/coding-agent/binaries/darwin-arm64/examples/extensions/doom-overlay/doom/build.sh ===
#!/bin/bash
=== ./packages/coding-agent/examples/extensions/doom-overlay/doom/build.sh ===
#!/bin/bash
=== ./packages/coding-agent/scripts/migrate-sessions.sh ===
#!/bin/bash
=== ./packages/pods/dist/scripts/pod_setup.sh ===
#!/usr/bin/env bash
=== ./packages/pods/dist/scripts/model_run.sh ===
#!/usr/bin/env bash
=== ./packages/pods/scripts/pod_setup.sh ===
#!/usr/bin/env bash
=== ./packages/pods/scripts/model_run.sh ===
#!/usr/bin/env bash
=== ./packages/mom/dev.sh ===
#!/bin/bash
=== ./packages/mom/docker.sh ===
#!/bin/bash
=== ./pi-test.sh ===
#!/usr/bin/env bash
=== ./test.sh ===
#!/bin/bashNow 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"packages/coding-agent/binaries/darwin-arm64/examples/extensions/doom-overlay/doom/build.sh
IGNOREDThe 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"packages/pods/dist/scripts/pod_setup.sh
IGNOREDGood, 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"Running formatting, linting, and type checking...
> pi-monorepo@0.0.3 check
> biome check --write . && tsgo --noEmit && npm run check -w @mariozechner/pi-web-ui
Checked 416 files in 262ms. No fixes applied.
> @mariozechner/pi-web-ui@0.49.3 check
> biome check --write . && tsc --noEmit && cd example && biome check --write . && tsc --noEmit
Checked 73 files in 41ms. No fixes applied.
Checked 3 files in 8ms. No fixes applied.
✅ All pre-commit checks passed!
[main 7af1919d] fix: use portable shebang in shell scripts
5 files changed, 5 insertions(+), 5 deletions(-)Commit successful. Now push and comment on the issue.
>git pushTo https://github.com/badlogic/pi-mono.git
9b84857b..7af1919d main -> mainNow add the comment to the issue.
>gh issue comment 910 --repo badlogic/pi-mono --body "Thanks, fixed in main"https://github.com/badlogic/pi-mono/issues/910#issuecomment-3786611942Done! 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.