Repository navigation
feat(opencode): add ACP stdio server for Zed + CLI switch - #2422
josephschmitt wants to merge 7 commits into
Conversation
|
@josephschmitt Lazy question. Do you know if ADP supports running over http(s)/websockets? |
The spec says it only communicates via stdio:
I guess in theory you could add a wrapper that used something like |
|
Ah too bad. Thanks for the details! |
|
@josephschmitt It has been approved! |
|
@jpchauvel i don't think it has been approved by anybody at the opencode team (anybody can approve PRs on this repo). @kmk142789 seems like a crypto bot btw |
|
Would love to use this on Neovim with CodeCompanion |
f16de3d to
cc0d460
Compare
|
Welp it's been a month and no action. I’m going to try to rebase and see if this implementation is even still relevant. If it breaks too much I might abandon it. Hopefully the core team gets to it eventually |
|
@thdxr @adamdotdevin this branch is rebased and tests are passing. Typecheck is failing but that seems unrelated to my changes. Let me know what you want me to do with this PR. If you don't want to accept it that's fine, I'll abandon it and pursue an external wrapper that adds ACP support for OpenCode. |
|
@thdxr @adamdotdevin I created an alternate implementation that uses Zed’s official TypeScript sdk for ACP, this might be a better PR to merge. LMK what you think: #2947 |
Better, more modern implementation available at: #2947
Outdated implementation summary
Summary
What Changed
ACP Methods
Tooling Surface
Usage
Testing
Notes