fix(claude): support Claude Desktop transcript format#739
fix(claude): support Claude Desktop transcript format#739WenhuaXia wants to merge 1 commit intositeboon:mainfrom
Conversation
… messages lack message.role Claude Code Desktop writes JSONL entries with a top-level `type: 'assistant'` field instead of `message.role: 'assistant'`. The session parser only checked `message.role`, causing all assistant messages to be silently skipped and users seeing no replies in the chat UI. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe PR extends transcript format detection in the Claude sessions provider to support both CLI-style ( ChangesTranscript Format Support Expansion
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 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: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.Comment |
Summary
type: 'assistant'field instead ofmessage.role: 'assistant'message.role, causing all assistant messages to be silently skippedFix
Check both
raw.message?.role === 'assistant'andraw.type === 'assistant'to support both CLI and Desktop transcript formats.Test plan
message.role) still work correctly🤖 Generated with Claude Code
Summary by CodeRabbit