Skip to content

Fix UI workflow delete 404 by wiring cleanup route#174

Merged
AbirAbbas merged 1 commit intomainfrom
fix/ui-workflow-delete-cleanup-route-404
Feb 9, 2026
Merged

Fix UI workflow delete 404 by wiring cleanup route#174
AbirAbbas merged 1 commit intomainfrom
fix/ui-workflow-delete-cleanup-route-404

Conversation

@santoshkumarradha
Copy link
Member

Summary

  • register missing UI cleanup route: DELETE /api/ui/v1/workflows/:workflowId/cleanup
  • add route regression test to ensure the cleanup endpoint is wired and reachable
  • make stub cleanup response non-nil in route tests so handler assertions are meaningful

Root cause

The frontend correctly called /api/ui/v1/workflows/:id/cleanup?confirm=true, but setupRoutes() never registered that route, so Gin returned a framework-level 404 before CleanupWorkflowHandler executed.

Validation

  • go test ./internal/server -run "TestSetupRoutesRegistersWorkflowCleanupUIRoute" (from control-plane/)

Issue

Closes #173

@AbirAbbas AbirAbbas merged commit ee47f56 into main Feb 9, 2026
21 checks passed
@AbirAbbas AbirAbbas deleted the fix/ui-workflow-delete-cleanup-route-404 branch February 9, 2026 17:49
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.

Workflow Delete in UI returns 404: cleanup route not registered

2 participants

Comments