auto-claude: subtask-1-1 - Update cancelReview callback to handle both success and failure cases (#1551)

- When IPC returns success=true, set message 'Review cancelled by user'
- When IPC returns success=false (process not found), set message 'Review stopped - process not found'
- Always update store state to exit 'reviewing' state, preventing UI from getting stuck
This commit is contained in:
Andy
2026-01-27 19:03:43 +01:00
committed by GitHub
parent 9b07ed4646
commit d8f00fe5ae
@@ -423,12 +423,14 @@ export function useGitHubPRs(
try {
const success = await window.electronAPI.github.cancelPRReview(projectId, prNumber);
if (success) {
// Update store to mark review as cancelled
usePRReviewStore
.getState()
.setPRReviewError(projectId, prNumber, "Review cancelled by user");
}
// Always update store state to exit the "reviewing" state
// Use different messages based on whether the process was found and killed
const message = success
? "Review cancelled by user"
: "Review stopped - process not found";
usePRReviewStore
.getState()
.setPRReviewError(projectId, prNumber, message);
return success;
} catch (err) {
setError(err instanceof Error ? err.message : "Failed to cancel review");