chore: refresh metagame feeds - #6226
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the MTGGoldfish metagame feeds for Modern, Pioneer, and Standard formats, including updating the last updated timestamp and modifying decklists. A review comment points out that there are duplicate card entries (such as multiple "Mountain" entries) in the Pioneer feed, which should be aggregated to prevent file bloat and potential UI rendering issues.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| "count": 4, | ||
| "name": "Burst Lightning" | ||
| "name": "Mountain" | ||
| }, | ||
| { | ||
| "count": 2, | ||
| "name": "Den of the Bugbear" | ||
| "count": 1, | ||
| "name": "Mountain" | ||
| }, | ||
| { | ||
| "count": 1, | ||
| "name": "Mountain" | ||
| }, | ||
| { | ||
| "count": 1, | ||
| "name": "Mountain" | ||
| }, |
There was a problem hiding this comment.
[MED] Duplicate card entries in metagame feeds. Evidence: client/public/feeds/mtggoldfish-pioneer.json:22-36.
Why it matters: Duplicate entries for the same card (e.g., "Mountain", "Swamp", "Emeritus of Ideation") bloat the feed files, reduce parsing efficiency, and can cause rendering bugs in the UI.
Suggested fix: Update the automated scraper script to aggregate card counts by name before writing the JSON feeds.
{
"count": 7,
"name": "Mountain"
},
Automated daily metagame feed refresh from MTGGoldfish.