Skip to content

Consolidate player data, location state, command priorities, and server patches - #18

Open
NineOfGaming wants to merge 8 commits into
DFOnline:mainfrom
NineOfGaming:fork
Open

Consolidate player data, location state, command priorities, and server patches#18
NineOfGaming wants to merge 8 commits into
DFOnline:mainfrom
NineOfGaming:fork

Conversation

@NineOfGaming

@NineOfGaming NineOfGaming commented Aug 1, 2026

Copy link
Copy Markdown

Summary

This PR consolidates and supersedes the following existing pull requests:

The changes were integrated together, with conflicts and overlapping behavior resolved where necessary.

Included changes

  • Add /whois profile parsing and structured player profile/rank data
  • Add public player profile and location lookup APIs
  • Improve /locate, mode tracking, and confirmed location state
  • Add server patch tracking and actiondump patch metadata
  • Add an internal priority queue for important commands
  • Improve the state debug display and test commands
  • Update the public API and README documentation

Differences from the original PRs

  • Plot-command support from CommandSenderFeature plot commands and internal queue #16 is intentionally excluded because encouraging its use could conflict with server rules
  • Server patch data from Server patch detection and actiondump metadata #17 has been simplified and changed to include a node id suffix for special nodes
  • Add actiondump test command
  • Additional fixes and consistency improvements were made while integrating the branches
  • Add User.getPlotPosition() for retrieving the player's plot-relative position when the development origin is known
  • Format position values consistently as x, y, z with one decimal place in the debug display
  • Add online-player autocomplete to the locate and whois test commands
  • Add User.isOnDiamondFire(); self explanatory

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