Bug fixes, performance, game balance, accessibility, server hardening, and test coverage improvements#10
Merged
Conversation
…er, tests Phase 1 - Bug fixes: remove unwrap() in room manager, fix broadcast error logging, fix sync/async mismatch in WS handler, cap minimap walls, add WS reconnect timeout, scene clear preserves capacity. Phase 2 - Performance: frustum culling, pre-allocated render buffers, monotonic scene IDs, network queue preservation on reconnect. Phase 3 - Game balance: LaserTag post-stun invulnerability (1s), smoke zone LOS blocking, Tron bot 2-step lookahead AI, grinding mechanic tests. Phase 4 - Accessibility: skip link, 44px touch targets, focus trap for modals, button debouncing, tablet breakpoint at 900px. Phase 5 - Server robustness: first-message rejection logging, rate-limit drop counter, relay max_clients_per_room (16), poller 24h pruning + sliding window stats, defensive JSON navigation in webhooks. Phase 6 - Test coverage: SSE integration tests, rate limiting integration test, event claim test, relay client limit test, Tron proptests. 594 tests passing (up from 484), 19 files changed, +1028/-90 lines. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
AndrewAltimit
changed the title
Split queue.len() into a separate binding to avoid simultaneous mutable (drain) and immutable (len) borrows of the same VecDeque. The native target accepted this but the WASM target's borrow checker rejected it. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.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.
Summary
594 tests passing (up from 484), 19 files changed, +1028/-90 lines across 8 workspace crates.
Test plan
cargo fmt --all -- --checkpassescargo clippy --workspace --all-targets -- -D warningspassescargo test --workspace-- 594 tests, 0 failuresGenerated with Claude Code