Skip to content

Update downloads tracking snapshot #14

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 6, 2026
Merged

Update downloads tracking snapshot #14

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 gh-dl/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 gh-dl/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.
18 changes: 9 additions & 9 deletions gh-dl/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 gh-dl/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.
18 changes: 9 additions & 9 deletions gh-dl/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 gh-dl/downloads.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
{
"schemaVersion": "1",
"owner": "Bearsampp",
"repo": "Bearsampp",
"repo": "module-phpmyadmin",
"visibility": "public",
"generatedAt": "2026-10-06T00:45:27.711Z",
"generatedAt": "2026-10-06T06:07:26.909Z",
"stats": {
"total": 5924,
"total": 942,
Comment on lines +4 to +8

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Action required

1. Bearsampp totals use another project 🐞 Bug ≡ Correctness

The generated snapshot changes the repository from Bearsampp to module-phpmyadmin while
retaining the Bearsampp owner. The workflow runs in the Bearsampp repository without a
target-repository override, so merging these files makes the published totals and charts represent
phpMyAdmin rather than Bearsampp.
Agent Prompt 
### Issue description
The generated snapshot identifies `Bearsampp/module-phpmyadmin` even though this workflow publishes download data for the current Bearsampp repository, causing the totals and charts to report another project's releases.

### Fix Focus Areas
- gh-dl/downloads.json[3-8]
- .github/workflows/gh-dl-daily-with-chart.yml[35-53]
- gh-dl/charts/total-trend--black.svg[2-24]

### Recommended Fix
Correct the download action's repository selection or generation inputs so the artifacts consistently target `Bearsampp/Bearsampp`, then regenerate and verify `owner`, `repo`, totals, chart labels, and chart values before merging.

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

Dismiss ↗ | View ↗

Comment on lines +6 to +8

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Remediation recommended

2. Offline dashboard shows yesterday's totals 🐞 Bug ☼ Reliability

The dashboard's embedded FALLBACK snapshot still identifies Bearsampp/Bearsampp and reports
5,924 downloads instead of the new snapshot's 942. When fetching downloads.json fails, including
file previews and offline use, render(FALLBACK) remains active and displays those obsolete values
beside the newly generated charts.
Agent Prompt 
### Issue description
The dashboard embeds the previous snapshot as its offline fallback, so it displays stale repository identity and download totals whenever the live JSON fetch is unavailable.

### Fix Focus Areas
- gh-dl/dashboard.html[157-172]
- gh-dl/dashboard.html[262-268]
- gh-dl/downloads.json[4-8]

### Recommended Fix
Update the embedded snapshot whenever `downloads.json` changes, preferably by generating the dashboard fallback from the same snapshot during the workflow; ensure the fallback's owner, repo, generatedAt, totals, and series match the published JSON.

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

Dismiss ↗ | View ↗

"day": 0,
"week": 0,
"month": 0
Expand All @@ -21,7 +21,7 @@
"firstDate": "2026-10-06",
"lastDate": "2026-10-06",
"series": {
"2026-10-06": 5924
"2026-10-06": 942
}
},
"profile": {
Expand Down
Loading