anomalyco/opencode
1
0
Fork 0
mirror of https://github.com/anomalyco/opencode.git synced 2026-07-22 02:05:26 -04:00
Code Issues 5.9k Packages Projects Releases 19 Wiki Activity

[PR #4868] Remove top-level awaits to enable bytecode compilation #11148

New Issue
Closed
opened 2026-02-16 18:15:56 -05:00 by yindo · 0 comments
yindo commented 2026-02-16 18:15:56 -05:00
Owner
Copy Link
Copy Source

Original Pull Request: https://github.com/anomalyco/opencode/pull/4868

State: closed
Merged: No

Currently this PR is not enough to enable bytecode compiliation

  • opentui also needs to remove top level awaits
  • bun --bytecode works by compiling to commonjs. some dependencies have bugs that do not allow it. Throwing the error Expected CommonJS module to have a function wrapper. This us often caused by the use of import.meta.url or other ESM only code. This needs more investigation

Preparation for enabling bytecode compilation (faster startup times).

Related to #4843 and https://github.com/sst/opentui/pull/356

  • global/index.ts: Wrapped async initialization code in an IIFE instead of top-level await
  • index.ts: Changed await cli.parse() to Promise.resolve(cli.parse()).catch().finally() pattern
  • worker.ts: Removed await from Log.init() call (fire-and-forget)

Once opentui PR lands and Bun types are updated, bytecode can be enabled in build.ts with bytecode: true.

**Original Pull Request:** https://github.com/anomalyco/opencode/pull/4868 **State:** closed **Merged:** No --- Currently this PR is not enough to enable bytecode compiliation - opentui also needs to remove top level awaits - bun --bytecode works by compiling to commonjs. some dependencies have bugs that do not allow it. Throwing the error `Expected CommonJS module to have a function wrapper`. This us often caused by the use of `import.meta.url` or other ESM only code. This needs more investigation Preparation for enabling bytecode compilation (faster startup times). Related to #4843 and https://github.com/sst/opentui/pull/356 - **global/index.ts**: Wrapped async initialization code in an IIFE instead of top-level await - **index.ts**: Changed `await cli.parse()` to `Promise.resolve(cli.parse()).catch().finally()` pattern - **worker.ts**: Removed `await` from `Log.init()` call (fire-and-forget) Once opentui PR lands and Bun types are updated, bytecode can be enabled in `build.ts` with `bytecode: true`.
yindo added the pull-request label 2026-02-16 18:15:56 -05:00
yindo closed this issue 2026-02-16 18:15:56 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
acp
bad
bug
discussion
docs
documentation
enhancement
extension
good first issue
growth
help-wanted
mcp
model-problem
needs:compliance
nix
opentui
perf
pull-request
Mirrored from GitHub Pull Request
 tui
web
windows
zen
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: anomalyco/opencode#11148
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?