Skip to content

feat(streamer): owner room deletion and is_host in user_list#175

Merged
Uno-Takashi merged 1 commit into
mainfrom
feature/owner-room-controls
Jun 23, 2026
Merged

feat(streamer): owner room deletion and is_host in user_list#175
Uno-Takashi merged 1 commit into
mainfrom
feature/owner-room-controls

Conversation

@Uno-Takashi

Copy link
Copy Markdown
Member

What

  • database_user_list / User format now include is_host so clients can identify the room owner.
  • New host-only delete_room action: broadcasts a room_deleted server message to every member (sender included) and logically deletes the room + its users. Non-host requests are ignored.

Tests

  • test_anime_party_consumer_delete_room_ok — all members receive room_deleted, room/users logically deleted.
  • test_anime_party_consumer_delete_room_non_host_ignored — guest request is ignored, room survives.
  • Extended the user_list test to assert is_host per participant.
  • Full suite: 13 passed.

🤖 Generated with Claude Code

Expose `is_host` on the user_list payload so clients can mark the room
owner, and add a host-only `delete_room` action that broadcasts
`room_deleted` to every member and logically deletes the room and its
users. Non-host requests are ignored.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@Uno-Takashi Uno-Takashi merged commit da8dcfc into main Jun 23, 2026
17 of 18 checks passed
@Uno-Takashi Uno-Takashi deleted the feature/owner-room-controls branch June 23, 2026 00:34
Uno-Takashi added a commit to d-party/d-party that referenced this pull request Jun 23, 2026
Advance backend and chrome-extension to the merged owner-controls work:
- backend: is_host in user_list + host-only delete_room (d-party/backend#175)
- chrome-extension: owner crown badge + hold-to-delete room (d-party/chrome-extension#21)

Co-authored-by: Claude Opus 4.8 <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.

1 participant