Skip to content

Update downloads tracking snapshot #34

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
N6REJ merged 1 commit into from
Oct 7, 2026
Merged

Update downloads tracking snapshot #34

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions stats/charts/daily--black.svg
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
4 changes: 2 additions & 2 deletions stats/charts/monthly--black.svg
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
4 changes: 2 additions & 2 deletions stats/charts/total-trend--black.svg
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
4 changes: 2 additions & 2 deletions stats/charts/weekly--black.svg
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
4 changes: 2 additions & 2 deletions stats/downloads-trend.svg
View file
Open in desktop
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
8 changes: 4 additions & 4 deletions stats/downloads.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"owner": "Bearsampp",
"repo": "module-postgresql",
"visibility": "public",
"generatedAt": "2026-10-06T06:07:31.366Z",
"generatedAt": "2026-10-07T09:36:48.818Z",
"stats": {
"total": 3997,
"day": 0,
Expand All @@ -18,10 +18,10 @@
"snapshots": {
"windowDays": 45,
"count": 1,
"firstDate": "2026-10-06",
"lastDate": "2026-10-06",
"firstDate": "2026-10-07",
"lastDate": "2026-10-07",
"series": {
"2026-10-06": 3997
"2026-10-07": 3997
Comment on lines +21 to +24

@qodo-code-review qodo-code-review Bot Oct 7, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Remediation recommended

1. The download trend loses its previous day 🐞 Bug ≡ Correctness

The updated snapshot replaces the series with only 2026-10-07 while retaining a 45-day window and
count of 1. Each daily update therefore discards the prior date instead of preserving it for the
rolling history and trend charts.
Agent Prompt 
## Issue description
The generated snapshot removes `2026-10-06` and retains only `2026-10-07`, despite declaring a 45-day history window. Daily updates should preserve existing snapshot dates until they fall outside the configured window.

## Fix Focus Areas
- stats/downloads.json[19-24]
- .github/workflows/stats-daily-with-chart.yml[34-42]

## Recommended Fix
Update the snapshot generation or staging-branch flow so each daily run merges the new date into the existing series, removes only entries older than 45 days, and recalculates `count`, `firstDate`, and `lastDate` from the retained series.

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Dismiss ↗ | View ↗

}
},
"profile": {
Expand Down
Loading