Skip to content

agent: Fix race where compaction would be marked as cancelled - #59014

Merged
bennetbo merged 1 commit into
mainfrom
fix-race-compaction-cancelled
Jun 10, 2026
Merged

agent: Fix race where compaction would be marked as cancelled#59014
bennetbo merged 1 commit into
mainfrom
fix-race-compaction-cancelled

Conversation

@bennetbo

@bennetbo bennetbo commented Jun 10, 2026

Copy link
Copy Markdown
Member

Fixes an issue where compaction would get marked as cancelled if the previous turn took a while to cancel. E.g. you could reproduce this when sending a normal message, and then interrupting generation by sending /compact. If the task for the prior turn took a while to complete, it would mark the compaction triggered by /compact as cancelled, even though it was not.

Release Notes:

  • N/A

@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label Jun 10, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label Jun 10, 2026
@bennetbo
bennetbo enabled auto-merge June 10, 2026 09:43
@bennetbo
bennetbo added this pull request to the merge queue Jun 10, 2026
Merged via the queue into main with commit 5c90b06 Jun 10, 2026
44 checks passed
@bennetbo
bennetbo deleted the fix-race-compaction-cancelled branch June 10, 2026 09:48
This was referenced Jun 18, 2026
jonx pushed a commit to jonx/zed-aros that referenced this pull request Jul 17, 2026
…dustries#59014)

Fixes an issue where compaction would get marked as cancelled if the
previous turn took a while to cancel. E.g. you could reproduce this when
sending a normal message, and then interrupting generation by sending
`/compact`. If the task for the prior turn took a while to complete, it
would mark the compaction triggered by `/compact` as cancelled, even
though it was not.

Release Notes:

- N/A
jolutz pushed a commit to jolutz/zed that referenced this pull request Aug 8, 2026
…dustries#59014)

Fixes an issue where compaction would get marked as cancelled if the
previous turn took a while to cancel. E.g. you could reproduce this when
sending a normal message, and then interrupting generation by sending
`/compact`. If the task for the prior turn took a while to complete, it
would mark the compaction triggered by `/compact` as cancelled, even
though it was not.

Release Notes:

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

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants