Update game cover image iconUrls #192
Open
Conversation
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
WalkthroughUpdated meta.iconUrl values in multiple game YAML files to reference .webp assets (replacing PNG filenames or None). No other fields or logic were changed. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. 📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 💡 Knowledge Base configuration:
You can enable these sources in your CodeRabbit configuration. ⛔ Files ignored due to path filters (157)
📒 Files selected for processing (7)
🚧 Files skipped from review as they are similar to previous changes (6)
✨ Finishing Touches🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
VilppeRiskidev
force-pushed
the
cover-image-renames
branch
from
7c4b7d1 to
5a7b626
Compare
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
⛔ Files ignored due to path filters (157)
games/data/generated/20-minutes-till-dawn.ymlis excluded by!**/generated/**games/data/generated/9-kings.ymlis excluded by!**/generated/**games/data/generated/across-the-obelisk.ymlis excluded by!**/generated/**games/data/generated/against-the-storm.ymlis excluded by!**/generated/**games/data/generated/against.ymlis excluded by!**/generated/**games/data/generated/ale-and-tale-tavern.ymlis excluded by!**/generated/**games/data/generated/aloft.ymlis excluded by!**/generated/**games/data/generated/among-us.ymlis excluded by!**/generated/**games/data/generated/ancient-dungeon-vr.ymlis excluded by!**/generated/**games/data/generated/aneurism-iv.ymlis excluded by!**/generated/**games/data/generated/another-crabs-treasure.ymlis excluded by!**/generated/**games/data/generated/arcus-chroma.ymlis excluded by!**/generated/**games/data/generated/aska.ymlis excluded by!**/generated/**games/data/generated/atlyss.ymlis excluded by!**/generated/**games/data/generated/atomicrops.ymlis excluded by!**/generated/**games/data/generated/atrio-the-dark-wild.ymlis excluded by!**/generated/**games/data/generated/back-to-the-dawn.ymlis excluded by!**/generated/**games/data/generated/backpack-hero.ymlis excluded by!**/generated/**games/data/generated/balatro.ymlis excluded by!**/generated/**games/data/generated/below-the-stone.ymlis excluded by!**/generated/**games/data/generated/betrayal-beach.ymlis excluded by!**/generated/**games/data/generated/bomb-rush-cyberfunk.ymlis excluded by!**/generated/**games/data/generated/bonelab.ymlis excluded by!**/generated/**games/data/generated/boneworks.ymlis excluded by!**/generated/**games/data/generated/bopl-battle.ymlis excluded by!**/generated/**games/data/generated/brotato.ymlis excluded by!**/generated/**games/data/generated/castle-story.ymlis excluded by!**/generated/**games/data/generated/cats-are-liquid.ymlis excluded by!**/generated/**games/data/generated/chrono-ark.ymlis excluded by!**/generated/**games/data/generated/cities-skylines-ii.ymlis excluded by!**/generated/**games/data/generated/content-warning.ymlis excluded by!**/generated/**games/data/generated/core-keeper.ymlis excluded by!**/generated/**games/data/generated/cult-of-the-lamb.ymlis excluded by!**/generated/**games/data/generated/dale-dawson-stationery-supplies.ymlis excluded by!**/generated/**games/data/generated/deep-rock-galactic-survivor.ymlis excluded by!**/generated/**games/data/generated/depo.ymlis excluded by!**/generated/**games/data/generated/disco-elysium.ymlis excluded by!**/generated/**games/data/generated/distance.ymlis excluded by!**/generated/**games/data/generated/dome-keeper.ymlis excluded by!**/generated/**games/data/generated/dredge.ymlis excluded by!**/generated/**games/data/generated/dusk.ymlis excluded by!**/generated/**games/data/generated/dyson-sphere-program.ymlis excluded by!**/generated/**games/data/generated/ena-dream-bbq.ymlis excluded by!**/generated/**games/data/generated/enter-the-gungeon.ymlis excluded by!**/generated/**games/data/generated/erenshor.ymlis excluded by!**/generated/**games/data/generated/five-nights-at-freddys-into-the-pit.ymlis excluded by!**/generated/**games/data/generated/for-the-king.ymlis excluded by!**/generated/**games/data/generated/gang-beasts.ymlis excluded by!**/generated/**games/data/generated/garfield-kart-furious-racing.ymlis excluded by!**/generated/**games/data/generated/gatekeeper.ymlis excluded by!**/generated/**games/data/generated/gladio-mori.ymlis excluded by!**/generated/**games/data/generated/gloomwood.ymlis excluded by!**/generated/**games/data/generated/goodbye-volcano-high.ymlis excluded by!**/generated/**games/data/generated/gorebox.ymlis excluded by!**/generated/**games/data/generated/green-hell-vr.ymlis excluded by!**/generated/**games/data/generated/gtfo.ymlis excluded by!**/generated/**games/data/generated/guilty-as-sock.ymlis excluded by!**/generated/**games/data/generated/h3vr.ymlis excluded by!**/generated/**games/data/generated/hades-ii.ymlis excluded by!**/generated/**games/data/generated/hard-bullet.ymlis excluded by!**/generated/**games/data/generated/hard-time-3.ymlis excluded by!**/generated/**games/data/generated/hotds.ymlis excluded by!**/generated/**games/data/generated/human-fall-flat.ymlis excluded by!**/generated/**games/data/generated/i-am-your-beast.ymlis excluded by!**/generated/**games/data/generated/inscryption.ymlis excluded by!**/generated/**games/data/generated/labyrinthine.ymlis excluded by!**/generated/**games/data/generated/last-train-outta-wormtown.ymlis excluded by!**/generated/**games/data/generated/lethal-company.ymlis excluded by!**/generated/**games/data/generated/lethal-league-blaze.ymlis excluded by!**/generated/**games/data/generated/lost-skies.ymlis excluded by!**/generated/**games/data/generated/lycans.ymlis excluded by!**/generated/**games/data/generated/magicite.ymlis excluded by!**/generated/**games/data/generated/magicraft.ymlis excluded by!**/generated/**games/data/generated/mechanica.ymlis excluded by!**/generated/**games/data/generated/meeple-station.ymlis excluded by!**/generated/**games/data/generated/miside.ymlis excluded by!**/generated/**games/data/generated/monster-train-2.ymlis excluded by!**/generated/**games/data/generated/muck.ymlis excluded by!**/generated/**games/data/generated/my-dream-setup.ymlis excluded by!**/generated/**games/data/generated/nasb.ymlis excluded by!**/generated/**games/data/generated/nearly-dead.ymlis excluded by!**/generated/**games/data/generated/nine-sols.ymlis excluded by!**/generated/**games/data/generated/northstar.ymlis excluded by!**/generated/**games/data/generated/odd-remedy.ymlis excluded by!**/generated/**games/data/generated/old-market-simulator.ymlis excluded by!**/generated/**games/data/generated/outward.ymlis excluded by!**/generated/**games/data/generated/painting-vr.ymlis excluded by!**/generated/**games/data/generated/palworld.ymlis excluded by!**/generated/**games/data/generated/panicore.ymlis excluded by!**/generated/**games/data/generated/paquerette-down-the-bunburrows.ymlis excluded by!**/generated/**games/data/generated/patch-quest.ymlis excluded by!**/generated/**games/data/generated/peak.ymlis excluded by!**/generated/**games/data/generated/peaks-of-yore.ymlis excluded by!**/generated/**games/data/generated/peglin.ymlis excluded by!**/generated/**games/data/generated/pigface.ymlis excluded by!**/generated/**games/data/generated/plasma.ymlis excluded by!**/generated/**games/data/generated/potion-craft.ymlis excluded by!**/generated/**games/data/generated/pulsar-lost-colony.ymlis excluded by!**/generated/**games/data/generated/ravenfield.ymlis excluded by!**/generated/**games/data/generated/receiver-2.ymlis excluded by!**/generated/**games/data/generated/repo.ymlis excluded by!**/generated/**games/data/generated/return-of-the-obra-dinn.ymlis excluded by!**/generated/**games/data/generated/risk-of-rain-returns.ymlis excluded by!**/generated/**games/data/generated/riskofrain2.ymlis excluded by!**/generated/**games/data/generated/rogue-genesia.ymlis excluded by!**/generated/**games/data/generated/rogue-tower.ymlis excluded by!**/generated/**games/data/generated/rounds.ymlis excluded by!**/generated/**games/data/generated/rumble.ymlis excluded by!**/generated/**games/data/generated/sailwind.ymlis excluded by!**/generated/**games/data/generated/schedule-i.ymlis excluded by!**/generated/**games/data/generated/screw-drivers.ymlis excluded by!**/generated/**games/data/generated/shadows-of-doubt.ymlis excluded by!**/generated/**games/data/generated/shadows-over-loathing.ymlis excluded by!**/generated/**games/data/generated/shapez-2.ymlis excluded by!**/generated/**games/data/generated/skul-the-hero-slayer.ymlis excluded by!**/generated/**games/data/generated/slipstream-rogue-space.ymlis excluded by!**/generated/**games/data/generated/songs-of-conquest.ymlis excluded by!**/generated/**games/data/generated/sons-of-the-forest.ymlis excluded by!**/generated/**games/data/generated/stacklands.ymlis excluded by!**/generated/**games/data/generated/starsand.ymlis excluded by!**/generated/**games/data/generated/straftat.ymlis excluded by!**/generated/**games/data/generated/subnautica-below-zero.ymlis excluded by!**/generated/**games/data/generated/subnautica.ymlis excluded by!**/generated/**games/data/generated/subterranauts.ymlis excluded by!**/generated/**games/data/generated/subterror.ymlis excluded by!**/generated/**games/data/generated/sulfur.ymlis excluded by!**/generated/**games/data/generated/sun-haven.ymlis excluded by!**/generated/**games/data/generated/sunkenland.ymlis excluded by!**/generated/**games/data/generated/supermarket-together.ymlis excluded by!**/generated/**games/data/generated/talespire.ymlis excluded by!**/generated/**games/data/generated/tank-team.ymlis excluded by!**/generated/**games/data/generated/tcg-card-shop-simulator.ymlis excluded by!**/generated/**games/data/generated/techtonica.ymlis excluded by!**/generated/**games/data/generated/the-ouroboros-king.ymlis excluded by!**/generated/**games/data/generated/the-planet-crafter.ymlis excluded by!**/generated/**games/data/generated/thronefall.ymlis excluded by!**/generated/**games/data/generated/timberborn.ymlis excluded by!**/generated/**games/data/generated/totally-accurate-battle-simulator.ymlis excluded by!**/generated/**games/data/generated/touhou-lost-branch-of-legend.ymlis excluded by!**/generated/**games/data/generated/trombone-champ.ymlis excluded by!**/generated/**games/data/generated/ultimate-chicken-horse.ymlis excluded by!**/generated/**games/data/generated/ultrakill.ymlis excluded by!**/generated/**games/data/generated/v-rising.ymlis excluded by!**/generated/**games/data/generated/valheim.ymlis excluded by!**/generated/**games/data/generated/vertigo-2.ymlis excluded by!**/generated/**games/data/generated/voices-of-the-void.ymlis excluded by!**/generated/**games/data/generated/void-crew.ymlis excluded by!**/generated/**games/data/generated/vtol-vr.ymlis excluded by!**/generated/**games/data/generated/we-love-katamari-reroll-royal-reverie.ymlis excluded by!**/generated/**games/data/generated/webfishing.ymlis excluded by!**/generated/**games/data/generated/west-of-loathing.ymlis excluded by!**/generated/**games/data/generated/white-knuckle.ymlis excluded by!**/generated/**games/data/generated/wildfrost.ymlis excluded by!**/generated/**games/data/generated/wizard-of-legend.ymlis excluded by!**/generated/**games/data/generated/wizard-with-a-gun.ymlis excluded by!**/generated/**games/data/generated/wrestling-empire.ymlis excluded by!**/generated/**games/data/generated/zort.ymlis excluded by!**/generated/**
📒 Files selected for processing (7)
games/data/dusk.yml(1 hunks)games/data/ena-dream-bbq.yml(1 hunks)games/data/human-fall-flat.yml(1 hunks)games/data/magicite.yml(1 hunks)games/data/my-dream-setup.yml(1 hunks)games/data/odd-remedy.yml(1 hunks)games/data/zort.yml(1 hunks)
🧰 Additional context used
🧬 Code Graph Analysis (1)
games/data/human-fall-flat.yml (1)
games/src/models.ts (1)
GameDefinition(97-104)
🔇 Additional comments (6)
games/data/magicite.yml (1)
5-5: All iconUrl entries standardized to .webp and assets verified
- No legacy iconUrl values (None, .png/.jpg/.jpeg) found across games/data/*.yml
- Every referenced .webp asset is present in the repository
File: games/data/magicite.yml (line 5)
iconUrl: "magicite.webp"LGTM.
games/data/my-dream-setup.yml (1)
5-5: LGTM: filename normalized to kebab-case webpGood move from “MyDreamSetup.png” to “my-dream-setup.webp”; matches slug conventions and webp standardization.
games/data/odd-remedy.yml (1)
5-5: LGTM: updated to webp with consistent slugodd-remedy.webp aligns with label and common naming across the dataset.
games/data/ena-dream-bbq.yml (1)
5-5: LGTM: replacing placeholder with concrete webpUsing ena-dream-bbq.webp instead of "None" removes the placeholder and standardizes the asset.
games/data/dusk.yml (1)
5-5: LGTM: migrated from PNG to webpdusk.webp uses lowercase and matches the label; consistent with other entries.
games/data/zort.yml (1)
5-5: WebP migration approved — confirm asset presence and catch any leftover .png referencesThe rename to zort.webp aligns with the PR goals. Run the verification script (shared in the other comment) to ensure the asset exists in-repo and that all iconUrl values are consistently .webp.
| meta: | ||
| displayName: "Human Fall Flat" | ||
| iconUrl: "None" | ||
| iconUrl: "human-fall-flat.webp" |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
WebP icon update looks good — please verify asset existence and extension consistency across all games
To avoid broken icons at runtime, verify that every meta.iconUrl now points to an existing .webp asset and that no .png references remain.
Run this script from the repo root:
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "Scanning games/data for iconUrl entries and verifying .webp + file existence..."
python - << 'PY'
import os, re, sys
from glob import glob
# Collect all files' basenames across the repo for existence checks
all_basenames = set()
for root, dirs, files in os.walk(".", topdown=True):
# Skip typical VCS/build dirs to speed up
dirs[:] = [d for d in dirs if d not in {".git", "node_modules", "dist", "build"}]
for f in files:
all_basenames.add(f)
icon_entries = []
for path in glob("games/data/**/*.yml", recursive=True):
with open(path, "r", encoding="utf-8") as fh:
for ln, line in enumerate(fh, start=1):
m = re.match(r'^\s*iconUrl:\s*"([^"]+)"\s*$', line)
if m:
icon = m.group(1).strip()
icon_entries.append((path, ln, icon))
missing = []
non_webp = []
for path, ln, icon in icon_entries:
if not icon.lower().endswith(".webp"):
non_webp.append((path, ln, icon))
# Check by basename existence anywhere in repo
if icon not in all_basenames:
missing.append((path, ln, icon))
if non_webp:
print("Non-webp iconUrl entries detected:", file=sys.stderr)
for p,l,i in non_webp:
print(f" {p}:{l} -> {i}", file=sys.stderr)
if missing:
print("Missing icon assets (basename not found in repo):", file=sys.stderr)
for p,l,i in missing:
print(f" {p}:{l} -> {i}", file=sys.stderr)
if non_webp or missing:
sys.exit(1)
print("All iconUrl entries are .webp and assets exist by basename. ✅")
PYLength of output: 22772
Broken or Missing Icon URLs Detected
Running the verification script uncovered the following issues that will break icons at runtime:
-
Non-WebP reference (must be .webp)
• games/data/generated/valheim.yml:70 → Valheim.jpg -
Missing .webp assets for iconUrl entries in games/data/*.yml:
• games/data/bad-north.yml:5,19 → bad-north.webp
• games/data/lens-island.yml:5,19 → lens-island.webp
• games/data/logic-world.yml:5,19 → logic-world.webp
• games/data/lost-skies-ic.yml:5,21 → lost-skies-island-creator.webp
• games/data/mage-arena.yml:5,19 → mage-arena.webp
• games/data/mycopunk.yml:5,19 → mycopunk.webp
• games/data/ostranauts.yml:5,19 → ostranauts.webp
• games/data/pac-man-world-re-pac.yml:5,19 → pac-man-world-re-pac.webp
• games/data/patapon-1-2-replay.yml:5,19 → patapon-1-2-replay.webp
• games/data/spaghetti-kart.yml:5 → spaghetti-kart.webp
• games/data/vellum.yml:5,19 → vellum.webp
• games/data/word-play.yml:5,19 → word-play.webp
• games/data/darkwater.yml:5,19 → darkwater.webp
• games/data/dusk.yml:5 → dusk.webp
• games/data/ena-dream-bbq.yml:5 → ena-dream-bbq.webp
• games/data/human-fall-flat.yml:5 → human-fall-flat.webp
• games/data/magicite.yml:5 → magicite.webp
• games/data/my-dream-setup.yml:5 → my-dream-setup.webp
• games/data/odd-remedy.yml:5 → odd-remedy.webp
• games/data/zort.yml:5 → zort.webp
• games/data/paper-animal-adventure.yml:5,19 → paper-animal-adventure.webp…and 100+ additional missing entries under games/data/generated/*.yml.
Please add the missing .webp files or correct the iconUrl fields to match existing assets. Every iconUrl must resolve to a real .webp file before merging.
🤖 Prompt for AI Agents
In games/data/human-fall-flat.yml around line 5 the iconUrl references
"human-fall-flat.webp" but the verification report indicates that the .webp
asset is missing (or other files reference non-.webp names); ensure the iconUrl
points to an existing .webp asset: either add the missing human-fall-flat.webp
file into the repository assets directory where icons are served from, or update
iconUrl to the correct existing .webp filename/path; confirm the asset path
matches other games/data entries and re-run the validation to verify resolution.
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
games/data/dusk.yml (1)
5-5: Case change and WebP migration: double-check filename and asset presenceiconUrl changed from an uppercase PNG name to lowercase dusk.webp. On case-sensitive filesystems/CDNs, this must match the actual stored filename exactly.
- Verify dusk.webp exists with lowercase name.
- Ensure consumers (renderers, clients) accept WebP.
- If caching is in place, confirm cache invalidation for the old PNG isn’t needed.
You can reuse the asset verification script from the other comment to confirm presence and standardization.
If not already documented, standardize icon filenames to lowercase kebab-case with .webp in CONTRIBUTING or the schema docs to avoid future inconsistencies.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
⛔ Files ignored due to path filters (157)
games/data/generated/20-minutes-till-dawn.ymlis excluded by!**/generated/**games/data/generated/9-kings.ymlis excluded by!**/generated/**games/data/generated/across-the-obelisk.ymlis excluded by!**/generated/**games/data/generated/against-the-storm.ymlis excluded by!**/generated/**games/data/generated/against.ymlis excluded by!**/generated/**games/data/generated/ale-and-tale-tavern.ymlis excluded by!**/generated/**games/data/generated/aloft.ymlis excluded by!**/generated/**games/data/generated/among-us.ymlis excluded by!**/generated/**games/data/generated/ancient-dungeon-vr.ymlis excluded by!**/generated/**games/data/generated/aneurism-iv.ymlis excluded by!**/generated/**games/data/generated/another-crabs-treasure.ymlis excluded by!**/generated/**games/data/generated/arcus-chroma.ymlis excluded by!**/generated/**games/data/generated/aska.ymlis excluded by!**/generated/**games/data/generated/atlyss.ymlis excluded by!**/generated/**games/data/generated/atomicrops.ymlis excluded by!**/generated/**games/data/generated/atrio-the-dark-wild.ymlis excluded by!**/generated/**games/data/generated/back-to-the-dawn.ymlis excluded by!**/generated/**games/data/generated/backpack-hero.ymlis excluded by!**/generated/**games/data/generated/balatro.ymlis excluded by!**/generated/**games/data/generated/below-the-stone.ymlis excluded by!**/generated/**games/data/generated/betrayal-beach.ymlis excluded by!**/generated/**games/data/generated/bomb-rush-cyberfunk.ymlis excluded by!**/generated/**games/data/generated/bonelab.ymlis excluded by!**/generated/**games/data/generated/boneworks.ymlis excluded by!**/generated/**games/data/generated/bopl-battle.ymlis excluded by!**/generated/**games/data/generated/brotato.ymlis excluded by!**/generated/**games/data/generated/castle-story.ymlis excluded by!**/generated/**games/data/generated/cats-are-liquid.ymlis excluded by!**/generated/**games/data/generated/chrono-ark.ymlis excluded by!**/generated/**games/data/generated/cities-skylines-ii.ymlis excluded by!**/generated/**games/data/generated/content-warning.ymlis excluded by!**/generated/**games/data/generated/core-keeper.ymlis excluded by!**/generated/**games/data/generated/cult-of-the-lamb.ymlis excluded by!**/generated/**games/data/generated/dale-dawson-stationery-supplies.ymlis excluded by!**/generated/**games/data/generated/deep-rock-galactic-survivor.ymlis excluded by!**/generated/**games/data/generated/depo.ymlis excluded by!**/generated/**games/data/generated/disco-elysium.ymlis excluded by!**/generated/**games/data/generated/distance.ymlis excluded by!**/generated/**games/data/generated/dome-keeper.ymlis excluded by!**/generated/**games/data/generated/dredge.ymlis excluded by!**/generated/**games/data/generated/dusk.ymlis excluded by!**/generated/**games/data/generated/dyson-sphere-program.ymlis excluded by!**/generated/**games/data/generated/ena-dream-bbq.ymlis excluded by!**/generated/**games/data/generated/enter-the-gungeon.ymlis excluded by!**/generated/**games/data/generated/erenshor.ymlis excluded by!**/generated/**games/data/generated/five-nights-at-freddys-into-the-pit.ymlis excluded by!**/generated/**games/data/generated/for-the-king.ymlis excluded by!**/generated/**games/data/generated/gang-beasts.ymlis excluded by!**/generated/**games/data/generated/garfield-kart-furious-racing.ymlis excluded by!**/generated/**games/data/generated/gatekeeper.ymlis excluded by!**/generated/**games/data/generated/gladio-mori.ymlis excluded by!**/generated/**games/data/generated/gloomwood.ymlis excluded by!**/generated/**games/data/generated/goodbye-volcano-high.ymlis excluded by!**/generated/**games/data/generated/gorebox.ymlis excluded by!**/generated/**games/data/generated/green-hell-vr.ymlis excluded by!**/generated/**games/data/generated/gtfo.ymlis excluded by!**/generated/**games/data/generated/guilty-as-sock.ymlis excluded by!**/generated/**games/data/generated/h3vr.ymlis excluded by!**/generated/**games/data/generated/hades-ii.ymlis excluded by!**/generated/**games/data/generated/hard-bullet.ymlis excluded by!**/generated/**games/data/generated/hard-time-3.ymlis excluded by!**/generated/**games/data/generated/hotds.ymlis excluded by!**/generated/**games/data/generated/human-fall-flat.ymlis excluded by!**/generated/**games/data/generated/i-am-your-beast.ymlis excluded by!**/generated/**games/data/generated/inscryption.ymlis excluded by!**/generated/**games/data/generated/labyrinthine.ymlis excluded by!**/generated/**games/data/generated/last-train-outta-wormtown.ymlis excluded by!**/generated/**games/data/generated/lethal-company.ymlis excluded by!**/generated/**games/data/generated/lethal-league-blaze.ymlis excluded by!**/generated/**games/data/generated/lost-skies.ymlis excluded by!**/generated/**games/data/generated/lycans.ymlis excluded by!**/generated/**games/data/generated/magicite.ymlis excluded by!**/generated/**games/data/generated/magicraft.ymlis excluded by!**/generated/**games/data/generated/mechanica.ymlis excluded by!**/generated/**games/data/generated/meeple-station.ymlis excluded by!**/generated/**games/data/generated/miside.ymlis excluded by!**/generated/**games/data/generated/monster-train-2.ymlis excluded by!**/generated/**games/data/generated/muck.ymlis excluded by!**/generated/**games/data/generated/my-dream-setup.ymlis excluded by!**/generated/**games/data/generated/nasb.ymlis excluded by!**/generated/**games/data/generated/nearly-dead.ymlis excluded by!**/generated/**games/data/generated/nine-sols.ymlis excluded by!**/generated/**games/data/generated/northstar.ymlis excluded by!**/generated/**games/data/generated/odd-remedy.ymlis excluded by!**/generated/**games/data/generated/old-market-simulator.ymlis excluded by!**/generated/**games/data/generated/outward.ymlis excluded by!**/generated/**games/data/generated/painting-vr.ymlis excluded by!**/generated/**games/data/generated/palworld.ymlis excluded by!**/generated/**games/data/generated/panicore.ymlis excluded by!**/generated/**games/data/generated/paquerette-down-the-bunburrows.ymlis excluded by!**/generated/**games/data/generated/patch-quest.ymlis excluded by!**/generated/**games/data/generated/peak.ymlis excluded by!**/generated/**games/data/generated/peaks-of-yore.ymlis excluded by!**/generated/**games/data/generated/peglin.ymlis excluded by!**/generated/**games/data/generated/pigface.ymlis excluded by!**/generated/**games/data/generated/plasma.ymlis excluded by!**/generated/**games/data/generated/potion-craft.ymlis excluded by!**/generated/**games/data/generated/pulsar-lost-colony.ymlis excluded by!**/generated/**games/data/generated/ravenfield.ymlis excluded by!**/generated/**games/data/generated/receiver-2.ymlis excluded by!**/generated/**games/data/generated/repo.ymlis excluded by!**/generated/**games/data/generated/return-of-the-obra-dinn.ymlis excluded by!**/generated/**games/data/generated/risk-of-rain-returns.ymlis excluded by!**/generated/**games/data/generated/riskofrain2.ymlis excluded by!**/generated/**games/data/generated/rogue-genesia.ymlis excluded by!**/generated/**games/data/generated/rogue-tower.ymlis excluded by!**/generated/**games/data/generated/rounds.ymlis excluded by!**/generated/**games/data/generated/rumble.ymlis excluded by!**/generated/**games/data/generated/sailwind.ymlis excluded by!**/generated/**games/data/generated/schedule-i.ymlis excluded by!**/generated/**games/data/generated/screw-drivers.ymlis excluded by!**/generated/**games/data/generated/shadows-of-doubt.ymlis excluded by!**/generated/**games/data/generated/shadows-over-loathing.ymlis excluded by!**/generated/**games/data/generated/shapez-2.ymlis excluded by!**/generated/**games/data/generated/skul-the-hero-slayer.ymlis excluded by!**/generated/**games/data/generated/slipstream-rogue-space.ymlis excluded by!**/generated/**games/data/generated/songs-of-conquest.ymlis excluded by!**/generated/**games/data/generated/sons-of-the-forest.ymlis excluded by!**/generated/**games/data/generated/stacklands.ymlis excluded by!**/generated/**games/data/generated/starsand.ymlis excluded by!**/generated/**games/data/generated/straftat.ymlis excluded by!**/generated/**games/data/generated/subnautica-below-zero.ymlis excluded by!**/generated/**games/data/generated/subnautica.ymlis excluded by!**/generated/**games/data/generated/subterranauts.ymlis excluded by!**/generated/**games/data/generated/subterror.ymlis excluded by!**/generated/**games/data/generated/sulfur.ymlis excluded by!**/generated/**games/data/generated/sun-haven.ymlis excluded by!**/generated/**games/data/generated/sunkenland.ymlis excluded by!**/generated/**games/data/generated/supermarket-together.ymlis excluded by!**/generated/**games/data/generated/talespire.ymlis excluded by!**/generated/**games/data/generated/tank-team.ymlis excluded by!**/generated/**games/data/generated/tcg-card-shop-simulator.ymlis excluded by!**/generated/**games/data/generated/techtonica.ymlis excluded by!**/generated/**games/data/generated/the-ouroboros-king.ymlis excluded by!**/generated/**games/data/generated/the-planet-crafter.ymlis excluded by!**/generated/**games/data/generated/thronefall.ymlis excluded by!**/generated/**games/data/generated/timberborn.ymlis excluded by!**/generated/**games/data/generated/totally-accurate-battle-simulator.ymlis excluded by!**/generated/**games/data/generated/touhou-lost-branch-of-legend.ymlis excluded by!**/generated/**games/data/generated/trombone-champ.ymlis excluded by!**/generated/**games/data/generated/ultimate-chicken-horse.ymlis excluded by!**/generated/**games/data/generated/ultrakill.ymlis excluded by!**/generated/**games/data/generated/v-rising.ymlis excluded by!**/generated/**games/data/generated/valheim.ymlis excluded by!**/generated/**games/data/generated/vertigo-2.ymlis excluded by!**/generated/**games/data/generated/voices-of-the-void.ymlis excluded by!**/generated/**games/data/generated/void-crew.ymlis excluded by!**/generated/**games/data/generated/vtol-vr.ymlis excluded by!**/generated/**games/data/generated/we-love-katamari-reroll-royal-reverie.ymlis excluded by!**/generated/**games/data/generated/webfishing.ymlis excluded by!**/generated/**games/data/generated/west-of-loathing.ymlis excluded by!**/generated/**games/data/generated/white-knuckle.ymlis excluded by!**/generated/**games/data/generated/wildfrost.ymlis excluded by!**/generated/**games/data/generated/wizard-of-legend.ymlis excluded by!**/generated/**games/data/generated/wizard-with-a-gun.ymlis excluded by!**/generated/**games/data/generated/wrestling-empire.ymlis excluded by!**/generated/**games/data/generated/zort.ymlis excluded by!**/generated/**
📒 Files selected for processing (7)
games/data/dusk.yml(1 hunks)games/data/ena-dream-bbq.yml(1 hunks)games/data/human-fall-flat.yml(1 hunks)games/data/magicite.yml(1 hunks)games/data/my-dream-setup.yml(1 hunks)games/data/odd-remedy.yml(1 hunks)games/data/zort.yml(1 hunks)
🔇 Additional comments (5)
games/data/zort.yml (1)
5-5: LGTM: migrated zort.png → zort.webp.Looks consistent with the PR’s WebP standardization. Please confirm zort.webp exists with exact casing and that any old zort.png is no longer referenced elsewhere.
You can reuse the verification script provided in games/data/magicite.yml to validate asset presence and detect lingering PNG references.
games/data/my-dream-setup.yml (1)
5-5: LGTM; watch for case-sensitive filename changes.Previous asset was MyDreamSetup.png; new is my-dream-setup.webp (lowercase, hyphenated). On case-sensitive filesystems/CDNs this must match exactly. Also ensure cache/CDN invalidation if the asset is served publicly.
Run the verification script linked in games/data/magicite.yml to confirm my-dream-setup.webp exists and no PNG references remain.
games/data/odd-remedy.yml (1)
5-5: LGTM: icon moved to WebP.No schema/structure changes otherwise. Just verify odd-remedy.webp is present and remove any unused OddRemedy.png if it still exists to avoid confusion.
Use the repo-wide icon audit script in games/data/magicite.yml to validate asset presence and detect lingering PNGs.
games/data/human-fall-flat.yml (1)
5-5: LGTM; added icon where it was previously None.Since this introduces an icon where none existed, double-check the asset exists and that any consumers don’t assume “absence” semantics when iconUrl was previously null/None.
Run the verification script provided in games/data/magicite.yml to ensure human-fall-flat.webp is present and resolvable.
games/data/ena-dream-bbq.yml (1)
5-5: WebP assets exist; please verify pipeline & UI logic
- PASS: All
.webpfiles referenced byiconUrl(includingena-dream-bbq.webp) were found in the repo.- NOTE: 2
iconUrlentries still use non‐.webpextensions—consider standardizing these to.webp.Checks to do:
- Confirm your asset serving pipeline (CDN/bucket) returns
Content-Type: image/webpfor these files.- Verify no UI logic depends on
iconUrlbeing unset ornullto hide icons (to avoid regressions).Optional refactor:
- Add a CI schema/check to enforce kebab-case
.webpnaming and verify asset existence.
| meta: | ||
| displayName: "Magicite" | ||
| iconUrl: "None" | ||
| iconUrl: "magicite.webp" |
There was a problem hiding this comment.
💡 Verification agent
🧩 Analysis chain
Switch to WebP icon looks good; please verify asset presence and WebP support in consumers.
Change is straightforward. Two quick checks recommended:
- Ensure magicite.webp exists in the repo (correct case) or at the expected CDN location.
- Confirm any downstream renderers (site, docs, thumbnails) accept WebP for this field.
Run this script to validate all iconUrl references across games and flag missing assets or lingering PNGs:
🏁 Script executed:
#!/bin/bash
set -euo pipefail
# 1. Verify magicite.webp exists in the repo
echo "Checking for magicite.webp asset..."
webp_paths=$(fd -i -a -t f -g "magicite.webp" || true)
if [[ -z "$webp_paths" ]]; then
echo "ERROR: magicite.webp not found in repository!"
missing=1
else
echo "OK: Found magicite.webp at:"
echo "$webp_paths"
missing=0
fi
# 2. Look for any remaining PNG references in iconUrl fields
echo
echo "Scanning games/data/*.yml for iconUrl PNG references..."
rg -n --type=yml 'iconUrl:\s*".*\.png"' games/data || true
exit $missingLength of output: 501
🏁 Script executed:
#!/bin/bash
set -euo pipefail
echo "1. Searching for magicite.webp (case-insensitive) in entire repo..."
rg -n --ignore-case -e 'magicite\.webp' || true
echo
echo "2. Listing all .webp files in repo..."
fd -e webp || true
echo
echo "3. Scanning for iconUrl PNG references in YAML files..."
rg -n -e 'iconUrl:\s*".*\.png"' -g '*.yml' -g '*.yaml' || trueLength of output: 689
Missing WebP asset: magicite.webp
References to magicite.webp were found in:
- games/data/magicite.yml (line 5)
- games/data/generated/magicite.yml (lines 5 and 12)
However, no magicite.webp (or any .webp) file exists in the repository. Please:
- Add
magicite.webpat the expected path (with correct filename casing), or - Ensure the file is hosted and served from your CDN as intended.
No remaining iconUrl: "*.png" references were detected. Once the asset is available, confirm that all downstream renderers (site, docs, thumbnails) support WebP.
🤖 Prompt for AI Agents
In games/data/magicite.yml around line 5 the iconUrl points to "magicite.webp"
but the referenced WebP asset is missing from the repository; add the file at
the exact path and filename casing expected (e.g., assets/icons/magicite.webp or
the repository location your build expects) or update the YAML to point to a
hosted CDN URL that serves the WebP, then verify downstream renderers (site,
docs, thumbnails) support WebP and update any build/static asset manifests to
include the new file.
anttimaki
approved these changes
Aug 21, 2025
VilppeRiskidev
force-pushed
the
cover-image-renames
branch
from
5a7b626 to
ac29e2d
Compare
anttimaki
approved these changes
Aug 27, 2025
|
This is still waiting for mod manager changes to get merged. |
VilppeRiskidev
force-pushed
the
cover-image-renames
branch
from
ac29e2d to
3b3ca17
Compare
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.
Summary by CodeRabbit