Repository navigation
fix(ai): pretty integration titles in sidebar - #1183
Conversation
Mirror PR #1148 title formatting from ToolCallRenderer into CompactToolCallRenderer so the right sidebar resolves integration tool names via builtin providers and appends owner/repo[/path], channel, repo context suffixes — reaching parity with the main AiChatView. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe component now parses integration-style Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Review rate limit: 0/1 reviews remaining, refill in 60 minutes.Comment |
Mirror PR #1148 title formatting from ToolCallRenderer into CompactToolCallRenderer so the right sidebar resolves integration tool names via builtin providers and appends owner/repo[/path], channel, repo context suffixes — reaching parity with the main AiChatView. Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Summary
fix(ai): clean titles for integration tool calls) into the right sidebar's compact renderergetBuiltinProviderand appendsowner/repo[/path],channel,repocontext suffixesSidebarChatTab→CompactMessageRenderer→CompactToolCallRenderer) now matches the mainAiChatViewstyling for GitHub-style integration toolsWhy
PR #1148 only patched
ToolCallRenderer.tsx. The parallelCompactToolCallRenderer.tsxused by the right sidebar never received the fix, so integration tools there still showed raw names likeInt Github Search Reposwith no repo context.Test plan
pnpm --filter web typecheckpassespnpm --filter web test:unitpassesSearch Repos: 2witstudios/PageSpaceread_page,list_pages, etc.) still render their existing tool-name-gated titles — no regression🤖 Generated with Claude Code
Summary by CodeRabbit