Skip to content

refactor: collapse command facade#21981

Merged
kitlangton merged 2 commits intodevfrom
kit/facade-command-collapse
Apr 11, 2026
Merged

refactor: collapse command facade#21981
kitlangton merged 2 commits intodevfrom
kit/facade-command-collapse

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

Summary

  • remove the Command.list() runtime facade from packages/opencode/src/command/index.ts
  • switch the server command-list route to AppRuntime.runPromise(Command.Service.use(...))
  • preserve existing command listing behavior while keeping command access on the service boundary

Verification

  • bun run typecheck in packages/opencode

@kitlangton kitlangton force-pushed the kit/facade-command-collapse branch from f9bbe1c to a4e05f0 Compare April 11, 2026 03:49
@kitlangton kitlangton marked this pull request as ready for review April 11, 2026 03:50
@kitlangton kitlangton merged commit 5e3dc80 into dev Apr 11, 2026
7 of 9 checks passed
@kitlangton kitlangton deleted the kit/facade-command-collapse branch April 11, 2026 03:52
mrsimpson pushed a commit to mrsimpson/opencode that referenced this pull request Apr 14, 2026
xywsxp pushed a commit to xywsxp/opencode that referenced this pull request Apr 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant