-
Notifications
You must be signed in to change notification settings - Fork 839
add opencode agent and cleanup dropdown #7374
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
31a8be4 to
266bda0
Compare
|
I'm thinking we can add a beta tag, because this agent edits files without requesting permission. |
mscolnick
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice cleanup!
I'm open to adding this. They all have their own quirks as well, but maybe opencode more than others |
📝 Summary
Fixes #7340
I cleaned up the dropdown for selecting a new session, but not the main panel. I think we can do that when we add another model. Seems okay now
Used this command to run ACP:
npx stdio-to-ws "npx opencode-ai acp" --port 3023However, there seems to be an issue where the response is received many times
(edit: fixed by modifying the handleResumeSession code)
🔍 Description of Changes
📋 Checklist