Skip to content

Don't get out of sync when background task creates new init/result#386

Merged
benbrandt merged 2 commits intoagentclientprotocol:mainfrom
SteffenDE:sd-issue-336
Mar 9, 2026
Merged

Don't get out of sync when background task creates new init/result#386
benbrandt merged 2 commits intoagentclientprotocol:mainfrom
SteffenDE:sd-issue-336

Conversation

@SteffenDE
Copy link
Copy Markdown
Contributor

Relates to #336.

This does NOT ensure that the results are immediately sent to the client.

See #336 (comment).

I guess some tests for this would be useful.

@benbrandt benbrandt merged commit 3524606 into agentclientprotocol:main Mar 9, 2026
2 checks passed
SteffenDE added a commit to SteffenDE/claude-code-acp that referenced this pull request Mar 9, 2026
SteffenDE added a commit to SteffenDE/claude-code-acp that referenced this pull request Mar 9, 2026
pull bot pushed a commit to ryoppippi/claude-code-acp that referenced this pull request Mar 9, 2026
benbrandt pushed a commit that referenced this pull request Mar 13, 2026
…ry 2) (#400)

Followup for
#386.

When sending `/compact`, there's no replay for the user message, so we
did not resolve the prompt. Same for `/context`.

I tried what happens when sending a `/context` or `/compact` while
another prompt is running and Claude does not properly handle it.
Instead, it responds with something like

> I also noticed you sent /compact while I was working. That's a
built-in CLI command (not a skill), which compacts the conversation
context. It should be invoked directly in the CLI rather than as a
message to me.

The important part is
f52f214.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants