Merged
Conversation
+ Added /admin start <bundle name> + New Game.startTime property + Added onLoad() to Minigame Health Shop: ~ increase oak plans from 24 to 64 ~ make supply chests rarer Gardening: make higher power shoot even farther Add Mineguessr minigame Speed Builders: add structure name text Level system: 15 XP per 30 seconds played
+ Add Dokka for docs generation + Minigame: add handleCreatureSpawn to handle players using a spawn egg + Minigame: change parameter of startCountdown from milliseconds to ticks Plugin: + Damage Dealer: bug fixes + Speed Builders: add entity support + Statistics: track multiple statistics with placeholders + Sidebar: add statistics, remove beta text + Boosters: add basic booster system + Mineguessr: make chunk loading async
/admin skip - instantly ends the current minigame Plugin: Health Shop: - Fixed a bug where you could get an insane amount of points from survival time. - Made supply chests appear less often. - Increased price of double jump to 10 health. Gardening: - Removed from Family Night. - Updated some weights of objects. Mineguessr: - Biomes now use their translated name (multi-language support). - Balanced scores. Speed Builders: - Corner stairs are now properly handled, no more impossible structures. - Incorrect blocks are shown after judging. - Lots of new structures. - Balanced scores.
- revamped scoring, score is now reset for each minigame and stars are used instead - STATIC introduction, available in Minigame constructor - yaw and pitch control for start positions Plugin: - new minigame Gravjump in alpha (currently 2 maps are built, lots of features missing) - massive health shop updates: - shop split into paginated categories (combat, utility, potion, miscellaneous) - lots of new items - kits (8 permission-locked + 1 last used) - lots of bug fixes - use bundle name instead of QueueType enum - new xp calculation
Plugin: Health Shop: - Added posion, blindness and levitation potions - Tweaked world border
There was a problem hiding this comment.
Pull request overview
This pull request implements a major architectural refactoring that separates the Party Games system into distinct modules: an API module (pgame-api), the main plugin implementation (pgame-plugin), and an example test minigame module (test-minigame). The changes enable third-party developers to create custom minigames as separate plugins that depend on the core API.
- Restructured the project into a multi-module Gradle build with separate API and plugin modules
- Added comprehensive example minigames (SimpleMinigame, PlaceBlockMinigame, JavaMinigame) demonstrating API usage
- Moved all existing game implementations and resources from
src/topgame-plugin/src/ - Updated dependencies in paper-plugin.yml to reflect the new modular architecture
Reviewed changes
Copilot reviewed 91 out of 146 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| settings.gradle.kts | Added three new modules: pgame-api, pgame-plugin, and test-minigame |
| test-minigame/src/main/resources/paper-plugin.yml | Configuration for the example minigame plugin with dependency on PartyGamesCore |
| test-minigame/src/main/kotlin/info/mester/network/testminigame/*.kt | Three example minigame implementations (Simple, PlaceBlock, and Java) demonstrating API features |
| test-minigame/build.gradle.kts | Build configuration for the test minigame module with API dependency |
| pgame-plugin/src/main/resources/paper-plugin.yml | Updated plugin dependencies, removed Loader, added PartyGamesCore and optional party/friend integrations |
| pgame-plugin/src/main/resources/*.yml | New/updated configuration files for minigames, speed builders, sniffer hunt, and gravjump |
| pgame-plugin/src/main/resources/speedbuilders/*.nbt | NBT structure files for speed builders minigame |
| pgame-plugin/src/main/resources/gravjump/*.nbt | NBT structure files for gravjump minigame |
| pgame-plugin/src/main/kotlin/info/mester/network/partygames/game/snifferhunt/TreasureMap.kt | Added unused import for DAMPING_RADIUS constant |
| src/* (deleted files) | Removed old source files, configuration, and resources from root src directory |
Comments suppressed due to low confidence (1)
pgame-plugin/src/main/kotlin/info/mester/network/partygames/game/snifferhunt/TreasureMap.kt:3
- This import is unused and should be removed. The constant
DAMPING_RADIUSis not being used in the visible code, and if it's used elsewhere in the file, it should be accessed viaTreasureMap.DAMPING_RADIUSwithout an explicit import since it's in the same file.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
yeah honestly the original plan was to separate the api and plugin from one another but then i just kept adding more and more changes and i have no idea what the actual changes are now lmao