Skip to content

feat(kilo-chat): auto-open chat conversations#3311

Open
evanjacobson wants to merge 2 commits into
mainfrom
improvement/kiloclaw-chat-instant-availability
Open

feat(kilo-chat): auto-open chat conversations#3311
evanjacobson wants to merge 2 commits into
mainfrom
improvement/kiloclaw-chat-instant-availability

Conversation

@evanjacobson
Copy link
Copy Markdown
Contributor

Summary

Kilo Chat could leave users sitting on an empty root screen even when a conversation was available or could be created for them. This updates the chat landing flow to open the latest conversation or create an empty one when appropriate, while preserving an explicit suppression path for cases like leaving an active conversation.

The implementation follows the existing React Query and component-state patterns in Kilo Chat: conversation creation remains routed through the shared mutation/cache helpers, and the sidebar now surfaces history and pagination failures with retry affordances.

Additional details
  • Adds root landing decision helpers and tests for auto-open, auto-create, suppression, and stale mutation-result handling.
  • Extends conversation list UI state to cover history and pagination errors without blocking manual conversation creation.
  • Allows useConversations callers to opt into refetchOnMount: 'always' for fresh root landing decisions.

Verification

  • Not manually verified yet; opened as a draft PR for review.

Visual Changes

Not captured yet. This changes Kilo Chat root navigation behavior and sidebar error/retry states.

Reviewer Notes

Focus areas: root landing edge cases, suppression after leaving a conversation, and stale create-conversation results when switching sandboxes or routes.

@evanjacobson evanjacobson marked this pull request as ready for review May 18, 2026 19:09
Comment thread packages/kilo-chat-hooks/src/use-conversations-options.test.ts Outdated
@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented May 18, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

The previously flagged vi.mock violation in use-conversations-options.test.ts has been resolved by deleting that test file; no remaining issues across all reviewed files.

Files Reviewed (9 files)
  • apps/web/src/app/(app)/claw/kilo-chat/components/ConversationList.tsx
  • apps/web/src/app/(app)/claw/kilo-chat/components/KiloChatConversationPage.tsx
  • apps/web/src/app/(app)/claw/kilo-chat/components/KiloChatLayout.tsx
  • apps/web/src/app/(app)/claw/kilo-chat/components/chat-root-selection-state.ts
  • apps/web/src/app/(app)/claw/kilo-chat/components/chat-root-selection-state.test.ts
  • apps/web/src/app/(app)/claw/kilo-chat/components/conversation-list-state.test.ts
  • apps/web/src/app/(app)/claw/kilo-chat/hooks/useConversations.ts
  • packages/kilo-chat-hooks/src/use-conversations.ts
  • packages/kilo-chat-hooks/src/use-conversations-options.test.ts — deleted (resolved WARNING: vi.mock violation)

Reviewed by claude-4.6-sonnet-20260217 · 371,641 tokens

Review guidance: REVIEW.md from base branch main

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