Skip to content

feat(new-player-system): rework of the user system. - #260

Merged
theShadowsDust merged 41 commits into
mainfrom
theshadowsdust/feature/rewrite-usersystem
Nov 10, 2026
Merged

theShadowsDust merged 41 commits into
mainfrom
theshadowsdust/feature/rewrite-usersystem

Conversation

@theShadowsDust

Copy link
Copy Markdown
Collaborator

Overview

This pull requests target to replace the old user and user-properties system

Description

This pull requests replaces the old user system with an better one to improve the maintainability

Submitter Checklist

  • Make sure you are opening from a topic branch (/feature/fix/docs/ branch (right side)) and not your main branch.
  • Ensure that the pull request title represents the desired changelog entry.
  • New public fields and methods are annotated with @since TODO.
  • I read and followed the contribution guidelines.

theShadowsDust and others added 30 commits October 23, 2026 20:09
StardustPlayer becomes the Hibernate entity for table stardust_player:
UUIDv7 stardust_id, minecraft_uuid as natural id, settings stored as a
JSON map keyed by SettingKey#key() containing only non-default values,
with lazy schema migration on read. PlayerSetting becomes a record
(value, schema, updatedAt). Adds a Gson-based Hibernate FormatMapper
since Paper ships Gson but neither Jackson nor JSON-B.

Claude-Session: https://claude.ai/code/session_018ffDGNyYNFXCgW9ZzvP4qW
@theShadowsDust
theShadowsDust requested a review from a team as a code owner November 7, 2026 17:04
@github-actions

github-actions Bot commented Nov 7, 2026 •

Copy link
Copy Markdown
Contributor

Test results

 30 files  30 suites   12s ⏱️
 40 tests 13 ✅ 27 💤 0 ❌
120 runs  39 ✅ 81 💤 0 ❌

Results for commit c491b3c.

♻️ This comment has been updated with latest results.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@TheMeinerLP TheMeinerLP 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.

LGTM

@theShadowsDust
theShadowsDust merged commit 95ad625 into main Nov 10, 2026
11 of 12 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