fix(repos): repair moved GitHub slugs and preserve partial repository-stat cache - #1107
Open
castrojo wants to merge 1 commit into
Open
fix(repos): repair moved GitHub slugs and preserve partial repository-stat cache#1107castrojo wants to merge 1 commit into
castrojo wants to merge 1 commit into
Conversation
…-stat cache - Drop deja-dup/deja-dup and Flavius42/mission-center from fetch-github-repos.js: both projects fully moved development to GNOME GitLab and no longer have a canonical GitHub repository, so the fetch was always 404ing. Also drop the corresponding githubRepo props in projects.mdx so ProjectCard doesn't fall back to a wasted visitor-side GitHub API request for these cards. - Repoint tesk-g/refine to TheEvilSkeleton/Refine (the account was renamed) and update its icon/avatar accordingly. - fetch-github-repos.js now merges freshly fetched results on top of the existing cache instead of overwriting it wholesale, so a transient failure for one repo no longer erases previously cached stats for that repo. Entries for repos no longer tracked are dropped from the merged cache. Signed-off-by: castrojo <castrojo@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.
Fixes #1093
Problem
fetch-github-repos.jstracked three GitHub slugs that now 404:deja-dup/deja-dupFlavius42/mission-centertesk-g/refineSince the script overwrites its cache file with only the successful results of each run, every fetch failure (dead slug or transient) permanently erased that project's stats from
static/data/github-repos.json, andProjectCardthen fell back to visitor-side GitHub API requests for the missing entries.Investigation
deja-dup/deja-dupandFlavius42/mission-centerhave fully moved development to GNOME GitLab. Neither has a canonical GitHub repository anymore (thedeja-duporg only owns an unrelatedsnappackaging repo, and themission-center-devsorg has zero public GitHub repos), so there's nothing to fetch — these are dropped fromGITHUB_REPOSand theirgithubRepoprops removed fromdocs/donations/projects.mdx.tesk-grenamed toTheEvilSkeleton, andtesk-g/refine→TheEvilSkeleton/Refineis the current canonical GitHub slug, so that one is repointed (icon updated to match).Fix
fetch-github-repos.jsnow reads the existing cache before fetching and merges this run's successful results on top of it, instead of overwriting the file wholesale. A transient failure for a still-tracked repo now keeps its last-known-good stats rather than disappearing; repos no longer inGITHUB_REPOSare pruned from the merged cache.Testing
node --test scripts/fetch-github-repos.test.js(existing tests pass)— hive: backend=copilot model=claude-sonnet-5
🐝 Hive Agent:
contributor| SHA:7c2b2ca2