Repository navigation
Update downloads tracking snapshot - #27
Merged
Merged
Conversation
Bearsampp-bot
approved these changes
Oct 8, 2026
PR Summary by QodoRefresh release download snapshot and charts for October 8
AI Description
Diagram
High-Level Assessment
Files changed (6)
|
Code Review by Qodo
1. Dashboard shows stale snapshot coverage
|
![qodo-code-review[bot]](https://avatars.githubusercontent.com/in/484649?s=60&v=4){kind=small}
Comment on lines
+14
to
+20
| "day": false, | ||
| "week": true, | ||
| "month": true | ||
| }, | ||
| "snapshots": { | ||
| "windowDays": 45, | ||
| "count": 1, | ||
| "windowDays": 90, | ||
| "count": 2, |
There was a problem hiding this comment.
1. Dashboard shows stale snapshot coverage 🐞 Bug ≡ Correctness
stats/dashboard.md still hard-codes the previous 45-day window, one collected snapshot, and fully partial status. The workflow preserves this file while replacing the generated stats directory, so every published dashboard continues to contradict the new JSON metadata.
Agent Prompt
## Issue description
The snapshot update changes the coverage window, snapshot count, and partial flags, but the preserved Markdown dashboard still displays the previous values.
## Fix Focus Areas
- stats/downloads.json[13-25]
- stats/dashboard.md[40-46]
- .github/workflows/stats-daily-with-chart.yml[100-126]
## Recommended Fix
Update the dashboard metadata from the generated JSON during the stats workflow, or generate the Markdown dashboard from the current snapshot so its window, count, dates, and partial status cannot remain stale after an automated snapshot update.
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
Comment on lines
8
to
15
| "total": 915, | ||
| "day": 0, | ||
| "week": 0, | ||
| "month": 0 | ||
| }, | ||
| "partial": { | ||
| "day": true, | ||
| "day": false, | ||
| "week": true, |
There was a problem hiding this comment.
2. Offline dashboard shows old totals 🐞 Bug ≡ Correctness
The embedded snapshot-data fallback in stats/dashboard.html still contains the previous 5,924 total, one snapshot, and 45-day window instead of the new snapshot values. When fetching downloads.json fails for a file:// or offline preview, rendering remains on this stale fallback and shows users obsolete statistics.
Agent Prompt
## Issue description
The dashboard's embedded fallback data is not updated when the automated snapshot changes, causing offline or file-based views to display obsolete totals and coverage.
## Fix Focus Areas
- stats/downloads.json[6-25]
- stats/dashboard.html[158-182]
- stats/dashboard.html[255-269]
- .github/workflows/stats-daily-with-chart.yml[100-126]
## Recommended Fix
Synchronize the `snapshot-data` JSON in `stats/dashboard.html` with the generated `downloads.json` during the stats workflow, or change the build process to regenerate the fallback block whenever the snapshot is replaced.
ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools
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
2 participants
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.
Automated update of the release download snapshot and charts.
Total: 915
Day: 0
Week: 0
Month: 0