Skip to content

feat: implement freeze list command and enhance freeze functionality#30

Open
Timonso-1 wants to merge 8 commits into
SLNE-Development:version/26.1.1from
Timonso-1:version/26.1.1
Open

feat: implement freeze list command and enhance freeze functionality#30
Timonso-1 wants to merge 8 commits into
SLNE-Development:version/26.1.1from
Timonso-1:version/26.1.1

Conversation

@Timonso-1

Copy link
Copy Markdown
Contributor

No description provided.

@MikeyLLP MikeyLLP left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Der hat immer eine UUID

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Implements a new /freezelist command to display currently frozen players, and enhances the freeze system to store richer metadata (who froze whom, when, and until when) to support listing and UI hover details.

Changes:

  • Replace the freeze cache value from Long duration to FreezeData (with timestamps + actor metadata) and add query helpers.
  • Add and register the /freezelist command with paginated output and an “Entfrieren” action.
  • Update /freeze to record the freezing actor, plus README + version bump.

Reviewed changes

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

Show a summary per file
File Description
src/main/kotlin/dev/slne/surf/moderation/tools/service/FreezeService.kt Stores FreezeData in cache; adds listing/query APIs for freeze metadata.
src/main/kotlin/dev/slne/surf/moderation/tools/PaperCommandManager.kt Registers the new freezeListCommand().
src/main/kotlin/dev/slne/surf/moderation/tools/commands/FreezeListCommand.kt New paginated /freezelist command showing freeze metadata and unfreeze action.
src/main/kotlin/dev/slne/surf/moderation/tools/commands/FreezeCommand.kt Passes frozenBy + frozenByName into FreezeService.freeze().
README.md Documents /freezelist (needs Markdown fix).
gradle.properties Bumps snapshot version to 2.1.3-SNAPSHOT.
Comments suppressed due to low confidence (1)

src/main/kotlin/dev/slne/surf/moderation/tools/service/FreezeService.kt:43

  • getFrozenPlayers() currently returns the live keys view of the cache’s backing map. Callers can mutate this collection (e.g., remove) and inadvertently modify the cache. Returning a snapshot avoids accidental external mutation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md Outdated
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.

3 participants