Skip to content

Fix plan stop not transitioning processing state to failed#233

Merged
neoneye merged 1 commit intomainfrom
fix/plan-stop-processing-state
Mar 10, 2026
Merged

Fix plan stop not transitioning processing state to failed#233
neoneye merged 1 commit intomainfrom
fix/plan-stop-processing-state

Conversation

@neoneye
Copy link
Member

@neoneye neoneye commented Mar 10, 2026

Summary

  • The /plan/stop endpoint in frontend_multi_user only transitioned pending plans to failed, but left processing plans stuck with stop_requested=True and no state change
  • This caused plans to get stuck in "processing" with no Stop/Resume/Retry buttons available in the UI
  • Fix: transition both pending and processing states to failed on stop, aligning with mcp_cloud/db_queries.py which already handles both states correctly

Test plan

  • Stop a plan that is in processing state — verify it transitions to failed and Retry/Resume buttons appear
  • Stop a plan that is in pending state — verify existing behavior unchanged
  • Verify completed plans still ignore stop requests (guard on line 3337)

🤖 Generated with Claude Code

The /plan/stop endpoint only transitioned pending plans to failed,
leaving processing plans stuck with stop_requested=True but no state
change. This made Stop/Resume/Retry buttons unavailable in the UI.
Align with mcp_cloud behavior which already handles both states.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@neoneye neoneye merged commit b264e5b into main Mar 10, 2026
3 checks passed
@neoneye neoneye deleted the fix/plan-stop-processing-state branch March 10, 2026 08:45
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