Skip to content

Use cursor official ACP#394

Open
Lenbok wants to merge 1 commit intoxenodium:mainfrom
Lenbok:fix-386-cursor-acp-official
Open

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

Conversation

@Lenbok
Copy link
Copy Markdown
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 Lenbok mentioned this pull request Mar 10, 2026
5 tasks
@Lenbok
Copy link
Copy Markdown
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

@Lenbok Lenbok force-pushed the fix-386-cursor-acp-official branch 2 times, most recently from 8b88a53 to f8188f1 Compare March 25, 2026 18:20
@Lenbok Lenbok force-pushed the fix-386-cursor-acp-official branch from f8188f1 to 011d568 Compare March 30, 2026 09:19
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