Skip to content

Conversation

@mike1858
Copy link
Member

@mike1858 mike1858 commented Jan 18, 2026

Closes #104.

Summary by CodeRabbit

  • Refactor
    • Internal improvements to progress entry handling in code analyzer

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 18, 2026

📝 Walkthrough

Walkthrough

Added support for Claude Code's new progress message type in JSONL file parsing. Introduced an internal ClaudeCodeProgressEntry struct and a Progress variant to the ClaudeCodeEntry enum, treating it as a skippable entry alongside existing non-message types.

Changes

Cohort / File(s) Summary
Claude Code Parser Enhancement
src/analyzers/claude_code.rs
Added ClaudeCodeProgressEntry struct and Progress variant to ClaudeCodeEntry enum. Updated parse_jsonl_file to skip Progress entries alongside FileHistorySnapshot and QueueOperation. No public API changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A progress variant hops along,
New message types, where they belong,
Skip and parse with ease once more,
Claude's format at our door! 🎉

🚥 Pre-merge checks | ✅ 3 | ❌ 2
❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ❓ Inconclusive The title 'Fix #104' is vague and does not clearly describe what is being fixed. While it references the issue, it provides no meaningful information about the changeset content. Replace with a more descriptive title like 'Add support for Claude Code progress variant' that clearly conveys the main change.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed The pull request successfully addresses the objective from issue #104 by adding support for the new 'progress' variant in Claude Code JSONL parsing, eliminating Serde parsing errors.
Out of Scope Changes check ✅ Passed All changes are directly related to handling the new 'progress' variant in Claude Code JSONL files. No out-of-scope modifications are present.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mike1858 mike1858 merged commit a8c2ba4 into main Jan 18, 2026
6 checks passed
@mike1858 mike1858 deleted the fix/104 branch January 18, 2026 01:48
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.

Unknown variant progress errors when parsing Claude Code JSONL files

2 participants