Skip to content

fix: update execArgv configuration in build script to futher disable toml - #5916

Closed
OpeOginni wants to merge 1 commit into
anomalyco:devfrom
OpeOginni:fix/bun-toml-bug
Closed

OpeOginni wants to merge 1 commit into
anomalyco:devfrom
OpeOginni:fix/bun-toml-bug

Conversation

@OpeOginni

@OpeOginni OpeOginni commented Dec 21, 2025 •

Copy link
Copy Markdown
Contributor

Not a real complete fix, as it still doesnt answer why the toml is loaded even when compiled to disable it.

But adding a null config path to the execArgs works

https://bun.com/docs/runtime#param-config

Potential fix to #5349

@rekram1-node

Copy link
Copy Markdown
Contributor

are we sure this isnt a bun bug?

@OpeOginni

OpeOginni commented Dec 22, 2025 •

Copy link
Copy Markdown
Contributor Author

Did some extra tests, It is a bun bug.

https://github.com/sst/opencode/blob/dev/packages/opencode/script/build.ts#L125-L149

adding execArgv ignores the fact that we have the autoloadBunfig flag as false, it shows this in the codebase.

https://github.com/oven-sh/bun/blob/main/src/cli.zig#L687-L698

Im not sure if its deliberate from them, but if it is then our solutions are to remove the execArgv or just force to use a null config

@OpeOginni

Copy link
Copy Markdown
Contributor Author

Made a PR that should ideally fix it, lets see what the team thinks of it

oven-sh/bun#25641

@OpeOginni OpeOginni closed this Jan 25, 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