Merged
Conversation
Owner
rajbos
commented
Mar 27, 2026
rajbos
commented
- Update dashboard to reflect configurable lookback window and improve date range display
- Add backfill historical data functionality and update dashboard for sync coverage
- Enhance backfill functionality with progress reporting in dashboard
- Add Visual Studio session detection to backend services and update dependencies
- Enhance dashboard data handling by incorporating machineId for user identity resolution and updating backfill completion messaging
- Refactor trigger type determination to use environment variables for improved readability
…date range display
…dentity resolution and updating backfill completion messaging
…improved readability
…hecks to use async methods for better performance
…roximation The previous code computed localInteractions as: p.sessions * p.avgInteractionsPerSession But avgInteractionsPerSession is stored as Math.round(interactions / sessions), so the reverse multiplication could diverge significantly from the true total, causing the sync-coverage warning to trigger or be suppressed incorrectly. Fix: use the same daily-stats approach already used for the lookback < 30 branch — sum d.interactions across lastDailyStats entries within the lookback window. This gives the exact interaction count with no rounding loss. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…package.json, sessionDiscovery.ts - cli-publish.yml: keep env-var injection approach (EVENT_NAME/GIT_REF) from PR - package.json: keep 'AI fluency insights' description from PR; icon field from main - sessionDiscovery.ts: use main's inline pathExists style (simpler) CliBridge.cs and CliBridgeTests.cs auto-merged cleanly; GetAllDataAsync and its test are now present from main, fixing the GetAllDataAsync_ReturnsTask CI failure. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Merge introduced two copies: one from the PR branch (line 41) and one from main's async-IO refactor commit. Remove the second duplicate. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.