[Player Counter] Use uuid for avatar image

[LetterN]

UUID is stable, usernames are not.

Summary by CodeRabbit

• Bug Fixes
  • Fixed player avatar image retrieval to display correctly for each player.

[coderabbitai]

📝 Walkthrough

Walkthrough

The avatar image URL in the Players table is now generated using each player's unique ID instead of their name when constructing the cravatar.eu image path.

Changes

Avatar Image URL Update

Layer / File(s)	Summary
Display Configuration player-counter/src/Filament/Server/Pages/PlayersPage.php	Avatar image URL now uses $record['id'] instead of $record['name'] for the cravatar.eu image path.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A rabbit hops through avatar land,
Where names once reigned with steady hand,
But IDs shine with clearer light,
Each player's face now framed just right! 🐰✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Title check	✅ Passed	The title accurately reflects the main change: updating avatar image URL generation to use UUID instead of username/name field for the cravatar.eu image path.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Boy132]

This won't work. For one you didn't edit the "is_whitelisted" check so it will still compare the username with the uuid. Also, the uuid isn't always known - depending on ping or query.

[LetterN]

(webedit pr issue) fixed it, should be correctly doing it now

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@player-counter/src/Filament/Server/Pages/PlayersPage.php`:
- Around line 141-143: The ImageColumn state currently uses $record['id'] which
may be the player name not a UUID; update the state closure in PlayersPage (the
ImageColumn::make('avatar')->state(...)) to prefer a real UUID field (e.g.,
$record['uuid'] or whatever MinecraftJavaQueryTypeSchema provides) and only fall
back to $record['id'] if no UUID exists or it does not match a UUID pattern;
ensure you reference the upstream GameQuery::runQuery /
MinecraftJavaQueryTypeSchema mapping so the state uses the canonical UUID when
present and otherwise uses a deterministic fallback (player name) so avatars
remain stable.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9e54ca83-fa47-4c1f-a11c-9911e53d385c

📥 Commits

Reviewing files that changed from the base of the PR and between 13bd696 and c28c5f1.

📒 Files selected for processing (1)

• player-counter/src/Filament/Server/Pages/PlayersPage.php

📜 Review details

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)

• GitHub Check: PHPStan (8.3)
• GitHub Check: PHPStan (8.4)
• GitHub Check: PHPStan (8.2)
• GitHub Check: PHPStan (8.5)

[LetterN]

Also, the uuid isn't always known - depending on ping or query.

on query protocol name is the uuid (so should be fine), but ping gives reliable-enough uuid (unless client disables)