Skip to content

fix(cloud-agent): make session deletion non-blocking - #5531

Merged
eshurakov merged 2 commits into
mainfrom
session/agent_ea46d2ae-96c8-4e75-b26f-053635481721
Aug 27, 2026
Merged

fix(cloud-agent): make session deletion non-blocking#5531
eshurakov merged 2 commits into
mainfrom
session/agent_ea46d2ae-96c8-4e75-b26f-053635481721

Conversation

@kilo-code-bot

@kilo-code-bot kilo-code-bot Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Close the delete confirmation and show spinner on the session row

Verification

Verified locally

UI guidance

Followed the Kilo Cloud overlay and interaction-quality reference, retaining existing confirmation/sidebar primitives. Dedicated sidebar and destructive-confirmation recipes remain a design-system coverage gap; no unrelated visual changes.

@kilo-code-bot

kilo-code-bot Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor Author

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • apps/web/src/components/cloud-agent-next/CloudSidebarLayout.tsx
  • apps/web/src/components/cloud-agent-next/ChatSidebar.tsx
Previous Review Summaries (2 snapshots, latest commit ae72cb4)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit ae72cb4)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (4 files)
  • apps/web/src/components/cloud-agent-next/CloudSidebarLayout.tsx
  • apps/web/src/components/cloud-agent-next/ChatSidebar.tsx
  • apps/web/src/components/cloud-agent-next/session-deletion.ts
  • apps/web/src/components/cloud-agent-next/session-deletion.test.ts

Previous review (commit d8369c6)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (4 files)
  • apps/web/src/components/cloud-agent-next/CloudSidebarLayout.tsx
  • apps/web/src/components/cloud-agent-next/ChatSidebar.tsx
  • apps/web/src/components/cloud-agent-next/hooks/useSidebarSessions.ts
  • apps/web/src/components/cloud-agent-next/CloudSidebarLayout.test.ts

Reviewed by grok-4.6 · Input: 249.9K · Output: 8.3K · Cached: 199.6K

Review guidance: REVIEW.md from base branch main

@eshurakov eshurakov self-assigned this Aug 27, 2026
@eshurakov
eshurakov force-pushed the session/agent_ea46d2ae-96c8-4e75-b26f-053635481721 branch from ae72cb4 to c690a6f Compare August 27, 2026 09:57
@eshurakov
eshurakov merged commit de7a8e7 into main Aug 27, 2026
13 checks passed
@eshurakov
eshurakov deleted the session/agent_ea46d2ae-96c8-4e75-b26f-053635481721 branch August 27, 2026 11: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.

2 participants