Repository navigation
Conversation
|
Thanks for updating your PR! It now meets our contributing guidelines. 👍 |
|
I dug into the failing The exposed assertion on this PR is:
The same Windows unit failure is also showing up on This PR only changes Based on that, this looks unrelated to the wrapper change in this PR and more like a Windows-only CI/environment issue in the existing test. |
|
Automated PR Cleanup Thank you for contributing to opencode. Due to the high volume of PRs from users and AI agents, we periodically close older PRs using automated criteria so maintainers can focus review time on the most active and community-supported contributions. This PR was closed because it matched the following cleanup criteria:
PRs created within the last month are not affected by this cleanup. If you believe this PR was closed incorrectly, or if you are still actively working on it, please leave a comment explaining why it should be reopened. A maintainer can review and reopen it if appropriate. Thanks again for taking the time to contribute. |
Issue for this PR
Closes #22769
Type of change
What does this PR do?
This PR fixes two issues with the
opencodebinary wrapper:"type": "module"is set inpackage.json.findBinaryto also check thedistfolder. This allows the wrapper to find locally built binaries in monorepo development environments, which was previously failing because the binaries weren't innode_modules.How did you verify your code works?
node packages/opencode/bin/opencode --versionon Windows with Node.js v25.9.0.distfolder after a local build (bun run script/build.ts --single).Screenshots / recordings
N/A (CLI change)
Checklist