Skip to content

feat(astro-kbve): MC players page with panic-safe RCON#7622

Merged
h0lybyte merged 1 commit intodevfrom
trunk/mc-players-ui-1772750047
Mar 5, 2026
Merged

feat(astro-kbve): MC players page with panic-safe RCON#7622
h0lybyte merged 1 commit intodevfrom
trunk/mc-players-ui-1772750047

Conversation

@h0lybyte
Copy link
Member

@h0lybyte h0lybyte commented Mar 5, 2026

Summary

  • Fixes panic safety in MC RCON background task: uses tokio::task::spawn for panic isolation, unwrap_or_else(|e| e.into_inner()) for poisoned mutex recovery
  • Adds McPlayerList React component (src/components/mc/McPlayerList.tsx) with auto-refresh, Starlight theme, and mc-heads.net avatar rendering
  • Adds mc/players.mdx page at kbve.com/mc/players with live player list
  • Adds Minecraft sidebar section to Starlight config

Test plan

  • cargo check -p axum-kbve passes clean
  • All 4 MC unit tests pass
  • ESLint + Prettier pass on component (lint-staged)
  • Verify kbve.com/mc/players renders the player list
  • Verify component handles 503 (MC not configured) gracefully with retry button
  • Verify background task survives RCON connection failures without crashing

- Fix panic safety: background RCON task now recovers from panics
  via tokio::task::spawn, poisoned mutexes handled with into_inner()
- Add McPlayerList React component with auto-refresh, Starlight
  theme integration, and mc-heads.net avatar rendering
- Add mc/players.mdx page at kbve.com/mc/players
- Add Minecraft sidebar section to Starlight config
@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@h0lybyte h0lybyte merged commit e3d12c8 into dev Mar 5, 2026
5 checks passed
@h0lybyte h0lybyte deleted the trunk/mc-players-ui-1772750047 branch March 5, 2026 23:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant