Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@marius-kilocode
Copy link
Collaborator

@marius-kilocode marius-kilocode commented Jan 22, 2026

Summary

When using reasoning models in the Agent Manager, the reasoning/thinking content was displayed as a regular message with "Kilo said" title. This PR renders it as a collapsible "Thinking" block with a lightbulb icon, matching the main extension sidebar's behavior.

Changes

New Files

  • webview-ui/src/kilocode/agent-manager/components/ReasoningBlock.tsx - Collapsible reasoning block component
  • webview-ui/src/kilocode/agent-manager/components/__tests__/ReasoningBlock.spec.tsx - Test suite (10 tests)

Modified Files

  • webview-ui/src/kilocode/agent-manager/components/MessageList.tsx - Added case "reasoning": handler
  • webview-ui/src/kilocode/agent-manager/components/AgentManagerApp.css - Added .am-reasoning-* CSS styles
  • 15 translation files - Added thinking and thinkingSeconds keys

Features

  • Lightbulb icon and "Thinking" title
  • Collapsed by default
  • Click-to-toggle functionality
  • Elapsed time display during streaming
  • Chevron indicator that rotates based on state
  • Hover to reveal chevron

Testing

  • All 10 unit tests pass
  • Type check passes
  • Lint passes
  • Manual testing confirmed working
image image

@changeset-bot
Copy link

changeset-bot bot commented Jan 22, 2026

🦋 Changeset detected

Latest commit: cdbe6eb

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
kilo-code Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@marius-kilocode marius-kilocode requested a review from a team January 22, 2026 11:37
@marius-kilocode marius-kilocode enabled auto-merge (squash) January 22, 2026 11:37
@kiloconnect
Copy link
Contributor

kiloconnect bot commented Jan 22, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Overview

This PR adds a collapsible reasoning block component to the Agent Manager UI. The implementation is clean and well-structured:

  • New Component: ReasoningBlock.tsx - Displays AI reasoning in a collapsible format with elapsed time indicator during streaming
  • Styling: CSS follows existing patterns with proper VSCode theme variable usage
  • i18n: All user-facing strings are properly internationalized across 17+ locales
  • Tests: Comprehensive test coverage for the new component

Notes

  • The ts prop in ReasoningBlock is received but unused (aliased to _ts). The elapsed timer uses Date.now() at mount time instead. This appears intentional - showing streaming duration rather than total message age - but could be documented with a comment for clarity.
Files Reviewed (21 files)
  • .changeset/reasoning-block-agent-manager.md
  • webview-ui/src/i18n/locales/ar/agentManager.json
  • webview-ui/src/i18n/locales/ca/agentManager.json
  • webview-ui/src/i18n/locales/cs/agentManager.json
  • webview-ui/src/i18n/locales/de/agentManager.json
  • webview-ui/src/i18n/locales/en/agentManager.json
  • webview-ui/src/i18n/locales/es/agentManager.json
  • webview-ui/src/i18n/locales/pl/agentManager.json
  • webview-ui/src/i18n/locales/pt-BR/agentManager.json
  • webview-ui/src/i18n/locales/ru/agentManager.json
  • webview-ui/src/i18n/locales/th/agentManager.json
  • webview-ui/src/i18n/locales/tr/agentManager.json
  • webview-ui/src/i18n/locales/uk/agentManager.json
  • webview-ui/src/i18n/locales/vi/agentManager.json
  • webview-ui/src/i18n/locales/zh-CN/agentManager.json
  • webview-ui/src/i18n/locales/zh-TW/agentManager.json
  • webview-ui/src/kilocode/agent-manager/components/AgentManagerApp.css
  • webview-ui/src/kilocode/agent-manager/components/MessageList.tsx
  • webview-ui/src/kilocode/agent-manager/components/ReasoningBlock.tsx
  • webview-ui/src/kilocode/agent-manager/components/__tests__/ReasoningBlock.spec.tsx

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.

3 participants