Skip to content

add !mp addref <username> command#87

Merged
ILW8 merged 5 commits intomasterfrom
feature/addref
Sep 24, 2025
Merged

add !mp addref <username> command#87
ILW8 merged 5 commits intomasterfrom
feature/addref

Conversation

@ILW8
Copy link
Copy Markdown
Owner

@ILW8 ILW8 commented Sep 24, 2025

Referees can execute commands on the behalf of the host running this client. The referee list is local and ephemeral, it last only as long as the user running this client stays in the lobby and the list of referees is not preserved when passing host to another user.

@ILW8 ILW8 requested a review from Copilot September 24, 2025 01:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a !mp addref <username> command that allows hosts to designate referees in multiplayer lobbies. The referee system enables designated users to execute commands on behalf of the host.

Key changes:

  • Introduced MultiplayerRefereeTracker to manage referee state and automatically clear referees when leaving rooms
  • Added command parsing logic to detect !mp addref commands and resolve usernames to API users
  • Modified message handling to allow referees to execute chat commands on behalf of the host

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

File Description
MultiplayerMatchSubScreen.cs Implements MultiplayerRefereeTracker component to manage referee list and handle room state changes
OsuGame.cs Registers the new MultiplayerRefereeTracker component in the dependency injection container
OnlineMetadataClient.cs Removes implementation from UpdateActivity method (appears unrelated to referee functionality)
StandAloneChatDisplay.cs Adds referee command parsing, username resolution, and message handling logic for referee commands

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@ILW8 ILW8 merged commit be5b505 into master Sep 24, 2025
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.

2 participants