Skip to content

🐛 Fix dashboard data discrepancy#153

Merged
rajbos merged 9 commits intorajbos:mainfrom
liubchigo:fix/data-discrepancy
Jan 30, 2026
Merged

🐛 Fix dashboard data discrepancy#153
rajbos merged 9 commits intorajbos:mainfrom
liubchigo:fix/data-discrepancy

Conversation

@liubchigo
Copy link
Contributor

@liubchigo liubchigo commented Jan 30, 2026

Fixes data discrepancy between main 🤖 Copilot Token Usage dashboard and Diagnostic Report showing different session counts:

  • Consistent session counts based on actual user activity
  • Diagnostic view shows 500 most recent sessions (not first 500 in directory order)
  • More accurate token tracking using actual chat interactions (last interaction date) vs file metadata(last modified)

closes #154

Copilot AI review requested due to automatic review settings January 30, 2026 12:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes data discrepancies between the main dashboard and Diagnostic Report by ensuring consistent session counting and date filtering logic across both views.

Changes:

  • Modified session filtering to skip empty sessions (zero interactions) in token tracking
  • Updated date-based filtering to use actual chat interaction timestamps instead of file modification times
  • Changed diagnostic view to sort sessions by modification time and include empty sessions for diagnostic purposes

liubchigo and others added 2 commits January 30, 2026 13:58
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@liubchigo liubchigo marked this pull request as draft January 30, 2026 13:00
@liubchigo liubchigo requested a review from Copilot January 30, 2026 13:08
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

liubchigo and others added 2 commits January 30, 2026 14:12
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@liubchigo liubchigo requested a review from Copilot January 30, 2026 13:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

@liubchigo liubchigo force-pushed the fix/data-discrepancy branch from 4058ca7 to 898148e Compare January 30, 2026 13:23
@liubchigo liubchigo force-pushed the fix/data-discrepancy branch from 898148e to a135640 Compare January 30, 2026 13:29
@liubchigo liubchigo marked this pull request as ready for review January 30, 2026 13:32
@rajbos rajbos enabled auto-merge January 30, 2026 14:25
@rajbos rajbos merged commit 295c72a into rajbos:main Jan 30, 2026
13 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.

[BUG] Fix data inconsistency between Copilot Token Usage and Diagnostic Report view

2 participants

Comments