fix(slack): clean up Chat SDK on disconnect

[RSO]

Summary

• Clean up the Chat SDK Slack workspace installation when a Slack integration is disconnected.
• Keep the platform integration row if Chat SDK cleanup fails, avoiding mismatched database and SDK state.
• Add unit coverage for successful cleanup, cleanup failure, and legacy rows without platform_installation_id.

Verification

N/A - no manual verification performed; backend cleanup behavior is covered by automated tests.

Visual Changes

N/A

Reviewer Notes

• The disconnect path now deletes Chat SDK state before deleting platform_integrations, so transient Redis/state failures leave the integration row available for retry.

[kilo-code-bot]

Code Review Summary

Status: 1 Issues Found | Recommendation: Address before merge

Overview

Severity	Count
CRITICAL	0
WARNING	1
SUGGESTION	0

Issue Details (click to expand)

WARNING

File	Line	Issue
apps/web/src/lib/integrations/slack-service.ts	243	Revoking the Slack token before Chat SDK cleanup can still leave an active integration row backed by invalid credentials when cleanup fails.

Other Observations (not in diff)

No additional observations.

Files Reviewed (4 files)

• apps/web/src/lib/bot-identity.ts - 0 issues
• apps/web/src/lib/integrations/slack-service.test.ts - 0 issues
• apps/web/src/lib/integrations/slack-service.ts - 1 issue
• apps/web/src/routers/slack-router.ts - 0 issues

Fix these issues in Kilo Cloud

---

_{Reviewed by gpt-5.4-20260305 · 351,064 tokens}