Skip to content

Make #lounge the Home room and fail open CLI audio#324

Merged
mpiorowski merged 6 commits into
mainfrom
mateu/bugs
Jun 7, 2026
Merged

Make #lounge the Home room and fail open CLI audio#324
mpiorowski merged 6 commits into
mainfrom
mateu/bugs

Conversation

@mpiorowski
Copy link
Copy Markdown
Owner

Summary

  • Renames the canonical Home chat room from #general to #lounge across database constraints, chat services, moderation guards, dashboard rendering, help copy, and project context.
  • Lets the late CLI continue into SSH when local audio startup or output fails, reporting Icecast availability so a paired browser can take over playback.

Changes

  • Adds migration 074_rename_general_room_to_lounge.sql to convert the permanent auto-join room to kind/slug = lounge.
  • Routes News announcements, bot posts, Home dashboard chrome, room ordering, search/mod copy, and reserved-room protections through #lounge.
  • Deletes generated News announcement chat cards when an article is deleted and sends a silent removal event to active chat views.
  • Uses Unicode display width for markdown/chat wrapping and padding to avoid wide-glyph terminal overflow.
  • Adds browser Icecast fallback when a paired CLI reports icecast_output_available=false.
  • Adds small TUI QoL fixes for Artboard help close, Shop h/l subtab navigation, Minesweeper chord preview/accounting, and Nonogram active clue styling.

Behavior notes

  • Migration aborts if a #lounge room already exists before renaming #general.
  • Older paired clients remain compatible because missing icecast_output_available defaults to true.
  • CLI audio failures no longer block SSH access; affected users need browser pairing for audio playback.

Feature flags

  • None

Screenshots / Video

  • Not included in this draft; attach before review.

Testing

  • Automated: Added/updated unit coverage for lounge room behavior, Unicode-width wrapping, Artboard help close, and browser Icecast fallback; added integration coverage for deleting News announcement chat messages. Not run per repo policy.
  • Manual: Not run; verification should cover migration, Home #lounge rendering, article deletion cleanup, CLI no-audio startup, browser audio takeover, and affected TUI controls.

@mpiorowski mpiorowski merged commit dbedc89 into main Jun 7, 2026
3 checks passed
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