fix(cli): restore executable bit on opencode build scripts - #11799
Merged
Conversation
PR #11591 rewrote build.ts during the sandbox refactor, which recreated the file with default 644 permissions and dropped the executable bit (100755 -> 100644). The same change stripped the bit from build-node.ts and publish.ts. The pre-release publish workflow invokes ./packages/opencode/script/build.ts directly, relying on its #!/usr/bin/env bun shebang. On Linux CI the non-executable mode fails with Permission denied before the build starts. Restore mode 100755 on all three scripts. Content is unchanged.
marius-kilocode
enabled auto-merge
June 29, 2026 12:56
imanolmzd-svg
approved these changes
Jun 29, 2026
t7tran
pushed a commit
to t7tran/kilocode
that referenced
this pull request
Aug 14, 2026
fix(cli): restore executable bit on opencode build scripts
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The pre-release publish workflow invokes the CLI build script directly:
This relies on the script's
#!/usr/bin/env bunshebang plus the executable mode bit. #11591 rewrotebuild.tsduring the sandbox hardening refactor, and a full-file rewrite recreates the file with default644permissions, so the mode flipped from100755to100644. The same change stripped the executable bit frombuild-node.tsandpublish.ts.On Linux CI a non-executable file invoked as
./script.tsfails withPermission deniedbefore Bun ever runs, which breaks the pre-release build (failing run).Fix
Restore mode
100755on all three scripts. Onlybuild.tsis invoked via direct./execution today, butbuild-node.tsandpublish.tswere also unintentionally stripped and carry the same Bun shebang, so all three are returned to their pre-#11591 state. The change is mode-only; file contents are untouched.