Skip to content

fix: support windows and linux from package.json run - #3466

Closed
zendril wants to merge 8 commits into
anomalyco:opentuifrom
zendril:fix-hardcoded-run
Closed

zendril wants to merge 8 commits into
anomalyco:opentuifrom
zendril:fix-hardcoded-run

Conversation

@zendril

@zendril zendril commented Oct 26, 2025

Copy link
Copy Markdown

Running bun x opencode@opentui was giving error: interpreter executable "/bin/sh" not found in %PATH% when running bun x opencode-ai@opentui

The package.json bin was hardcoded to the linux ./bin/opencode
This puts a shim in place to figure out whether to execute ./bin/opencode or ./bin/opencode.cmd

@zendril

zendril commented Oct 26, 2025

Copy link
Copy Markdown
Author

I dunno why the script/format.ts is now failing.. seems to be confused about being on a branch.

Comment thread packages/opencode/bin/run.js Resolved
Comment thread package.json Resolved
@rekram1-node

Copy link
Copy Markdown
Collaborator

I just merged opentui branch into this to avoid confusion

@zendril

zendril commented Oct 27, 2025

Copy link
Copy Markdown
Author

Hold on before approving/merging.

It starts the TUI fine, but I think it isn't passing any args, so only the help shows no matter what args are passed.

@zendril

zendril commented Oct 27, 2025

Copy link
Copy Markdown
Author

updated with a change to opencode.cmd. args were not be parsed correctly in there.

Now it runs the TUI if no args and then runs the other args properly if passed.

Comment thread packages/opencode/package.json Resolved
@zendril

zendril commented Oct 31, 2025

Copy link
Copy Markdown
Author

anything else you need me to do/verify or are you waiting on other reviews?

@github-actions

Copy link
Copy Markdown
Contributor

Closing this pull request because it has had no updates for more than 60 days. If you plan to continue working on it, feel free to reopen or open a new PR.

@github-actions github-actions Bot closed this Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants