mirror of
https://github.com/mikeyobrien/ralph-orchestrator.git
synced 2026-04-25 07:05:57 +03:00
[GH-ISSUE #46] Issues in streaming formatting when running connected to Opencode via ACP #15
Labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ralph-orchestrator#15
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
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?
Originally created by @ramarivera on GitHub (Jan 11, 2026).
Original GitHub issue: https://github.com/mikeyobrien/ralph-orchestrator/issues/46
Example
so I don't know if it's something that can be fixed, if it's something easy to fix, if it affects the functionality at all, because maybe the streaming/formatting cuts off some keyword that Ralph is looking for and that causes him to loop and loop and loop, I don't know. Another alternative would be to check maybe an interface with open code as a CLI, for example, or via their SDK. Again, just options. So, I mean, assuming that we would have three paths, which would be ACP, which is what you're doing now, JSON streaming via the CLI, and SDK via their server. Yeah, so let me know what you think and I would be happy to take a look and implement either of the three solutions or anything else that you can think of.
@ramarivera commented on GitHub (Jan 11, 2026):
(speech to text was used, hence the ramblings and formatting)
@mikeyobrien commented on GitHub (Jan 12, 2026):
Since the ACP adapter was vibe-coded, it needs a serious looking over for correctness and extensibility. It's got a lot of rough edges that need cleaning up. That said, I'd like ACP to be the default since it's easy to plugin any new provider that supports ACP.
I'm not opposed to an OpenCode SDK adapter, since it would be the most feature rich (and follow the same pattern as claude). CLI invocation with PTY would be the simplest and least brittle.
@mikeyobrien commented on GitHub (Jan 16, 2026):
ACP has been dropped. Until the explicit adapter is added, it might work with: