feat: Immediately delete completed tasks#724
Conversation
Previously activations that completed successfully would have their status updated, and then the upkeep thread would remove all the completed activations in batches. Now that the status update is happening in batches, the completed activations can also be immediately deleted instead of waiting for the upkeep thread.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit cf221e3. Configure here.
untitaker
left a comment
There was a problem hiding this comment.
for my understanding, the upkeep thread also used to batch deletes, right?
is it planned to remove some stuff from upkeep in a later change?
No, because the upkeep thread itself also moves things to "Completed". After it handles some errors states it will mark the task as completed and then delete it. The expectation though is that there are a lot less things being deleted by the upkeep thread. |

Previously activations that completed successfully would have their status updated, and then the upkeep thread would remove all the completed activations in batches. Now that the status update is happening in batches, the completed activations can also be immediately deleted instead of waiting for the upkeep thread.