Skip to content

Use cursor official ACP#394

Open
Lenbok wants to merge 2 commits intoxenodium:mainfrom
Lenbok:fix-386-cursor-acp-official
Open

Use cursor official ACP#394
Lenbok wants to merge 2 commits intoxenodium:mainfrom
Lenbok:fix-386-cursor-acp-official

Conversation

@Lenbok
Copy link
Contributor

@Lenbok Lenbok commented Mar 10, 2026

I've been using this a fair bit over the last day and it generally works well.

Cursor has some extension points, and at least for _cursor/update_todos this caused the agent session to hang. I've implemented an error response to unknown messages, and this allows the session to proceed. It seems like an appropriate response in general. We can decide whether to support their custom extension points later.

Fixes: #386

Checklist

  • [ x] I agree to communicate (PR description and comments) with the author myself (not AI-generated).
  • I've reviewed all code in PR myself and will vouch for its quality.
  • I've read and followed the Contributing guidelines.
  • I've filed a feature request/discussion for a new feature.
  • I've added tests where applicable.
  • I've updated documentation where necessary.
  • I've run M-x checkdoc and M-x byte-compile-file.

Lenbok added 2 commits March 10, 2026 13:46
cursor has some extension methods that we do not (yet) implement. When one
comes in (e.g. _cursor/update_todos), the session hangs unless we send a
response.
@Lenbok Lenbok mentioned this pull request Mar 10, 2026
5 tasks
@Lenbok
Copy link
Contributor Author

Lenbok commented Mar 10, 2026

An example traffic session showing the _cursor/update_todos, the error response, and the session continuing:

agent-shell-cursor-update-todos.txt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update cursor to use official ACP

1 participant