Skip to content

fix(tui): keep filtered session models in sync - #230

Merged
mike1858 merged 1 commit into
Piebald-AI:mainfrom
jimyag:fix/tui-model-filter-session-models
Aug 1, 2026
Merged

fix(tui): keep filtered session models in sync#230
mike1858 merged 1 commit into
Piebald-AI:mainfrom
jimyag:fix/tui-model-filter-session-models

Conversation

@jimyag

@jimyag jimyag commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

Summary

Fix the TUI model filter so Session rows no longer display models that do not match the active f filter.

Key Changes

  • Filter each session's per-day model collection.
  • Rebuild the session-level model collection from the filtered daily data.
  • Preserve compatibility with legacy sessions that do not have daily details.
  • Add a regression test for sessions using multiple models.

Validation

  • cargo fmt --all -- --check
  • cargo build --quiet
  • git diff --check

Notes

This PR only changes src/tui.rs and src/tui/tests.rs.

@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@jimyag, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 42 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 75cc6e5d-ace1-46d9-aaae-0fd62afe9185

📥 Commits

Reviewing files that changed from the base of the PR and between f614099 and f5a0b06.

📒 Files selected for processing (2)
  • src/tui.rs
  • src/tui/tests.rs

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.

Keep per-session and per-day model counts aligned after applying the model filter so unmatched models are not shown in session details. Add a regression case for sessions that use multiple models.

Signed-off-by: jimyag <git@jimyag.com>
@jimyag
jimyag force-pushed the fix/tui-model-filter-session-models branch from 762849f to f5a0b06 Compare August 1, 2026 15:43
@jimyag jimyag changed the title feat(grok): add session usage tracking and TUI filter fix fix(tui): keep filtered session models in sync Aug 1, 2026

@mike1858 mike1858 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@mike1858
mike1858 merged commit d010478 into Piebald-AI:main Aug 1, 2026
6 checks passed
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.

2 participants