Skip to content

Conversation

@RusseII
Copy link
Owner

@RusseII RusseII commented Jul 29, 2025

Summary

  • Fix spacing issues in GC Status lookup for public chat
  • Clear channel subscription status on logout to prevent stale UI state
  • Convert all logging to debug level to reduce production noise
  • Reduce reconnection message spam (only show every 5 attempts)
  • Clarify connection limit messaging to explicitly mention 200 player cap

Changes

Bug Fixes

  • Added replaceAll("\\s+", " ").trim() to remove extra spaces from icon placeholders in player name extraction
  • Added channelSubscriptionStatus.clear() in closeConnection() to properly reset state on logout
  • Removed channel subscription debug display from UI panel

Logging Improvements

  • Replaced all log.error, log.warn, and log.info with log.debug
  • Reduces log noise in production while maintaining debug capability

User Experience

  • Connection retry messages now only show every 5 attempts (reduces spam from ~1/min to ~1/5min)
  • Messages now clearly state "Global Chat at connection limit (200 players max)" instead of vague "connection issue"
  • Error messages now say "Global Chat full - 200 player limit reached!" to make the capacity issue explicit
  • Both messages encourage Patreon support to increase limits

Test Plan

  • Test GC Status lookup works correctly in public chat without spacing issues
  • Verify UI panel clears channel info after logout
  • Confirm reconnection messages appear at appropriate intervals
  • Verify connection limit messaging is clear to users

🤖 Generated with Claude Code

RusseII and others added 3 commits July 29, 2025 00:35
- Only show connection issue messages every 5 attempts
- Applies to both retry and error messages
- Reduces chat spam from ~1 message/minute to ~1 message/5 minutes

🤖 Generated with Claude

Co-Authored-By: Claude <noreply@anthropic.com>
- Connection retry message now mentions supporting on Patreon
- Connection failure message suggests limit may be reached
- Helps users understand they can help by supporting the project

🤖 Generated with Claude

Co-Authored-By: Claude <noreply@anthropic.com>
- Make it explicit that Global Chat is at 200 player limit
- Change 'connection issue' to 'at connection limit'
- Change 'connection failed' to 'Global Chat full'
- Users now clearly understand the issue is capacity, not technical

🤖 Generated with Claude

Co-Authored-By: Claude <noreply@anthropic.com>
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.

2 participants