Skip to content

[Fix] Task reliability metrics include environment snapshot maintenance - #1364

Draft
roomote-roomote[bot] wants to merge 1 commit into
developfrom
fix/settlement-setup-friction-3rj1nb3wvot5y
Draft

[Fix] Task reliability metrics include environment snapshot maintenance#1364
roomote-roomote[bot] wants to merge 1 commit into
developfrom
fix/settlement-setup-friction-3rj1nb3wvot5y

Conversation

@roomote-roomote

Copy link
Copy Markdown
Contributor

​Created by Roomote. Follow up by mentioning @roomote-roomote, in the web UI, or in Slack.

What changed

  • Exclude background environment snapshot maintenance from task_settled, matching the existing task_created analytics contract.
  • Let the centralized terminal run path own failed snapshot state instead of issuing a second worker-side status update.
  • Cover snapshot settlement exclusion across completed, failed, and canceled outcomes, plus worker failure delegation.

Why this change was made

Product task reliability could appear worse when background snapshot maintenance failed because snapshot runs were counted only on the settlement side. Failed snapshots also attempted two status transitions, producing misleading conflict errors after the centralized terminal path had already finalized them.

Impact

Task settlement analytics now describe product task outcomes consistently, while failed environment snapshots still reach a terminal failed state through the shared finalization path without duplicate status-write noise.

@roomote-community

roomote-community Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

No code issues found. See task

Reviewed a6589de

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.

1 participant