Skip to content

Fix 1.21.11 WaterAnimal incorrectly assumed removed#67

Merged
extremeheat merged 1 commit intoPrismarineJS:bumpfrom
pau101:fix/entity-type-classes
Feb 16, 2026
Merged

Fix 1.21.11 WaterAnimal incorrectly assumed removed#67
extremeheat merged 1 commit intoPrismarineJS:bumpfrom
pau101:fix/entity-type-classes

Conversation

@pau101
Copy link
Contributor

@pau101 pau101 commented Feb 15, 2026

The WaterAnimal check in EntitiesDataGenerator was wrongly removed in #65, causing Cod, Pufferfish, Salmon, Tadpole, and Tropical Fish to no longer have the type of "water_creature".

In 1.21.11, WaterAnimal was not removed, it was instead moved under a "fish" package.

This PR restores the check, albeit the entity "type" property has debatable utility. Nonetheless, this addresses the 1.21.11 regression.

@extremeheat
Copy link
Member

Thanks for the catch

@extremeheat extremeheat merged commit f952749 into PrismarineJS:bump Feb 16, 2026
31 checks passed
extremeheat added a commit that referenced this pull request Feb 19, 2026
* Add version 1.21.11

* Fix 1.21.11 compilation errors from Minecraft API changes (#65)

* Initial plan

* Fix 1.21.11 compilation: Replace ResourceLocation with Identifier and update WaterAnimal/getSkyColor

Co-authored-by: extremeheat <13713600+extremeheat@users.noreply.github.com>

* Add type-safe instanceof check before casting sky color argument

Co-authored-by: extremeheat <13713600+extremeheat@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: extremeheat <13713600+extremeheat@users.noreply.github.com>

* Fix incorrect biome color representation (#66)

* Fix water_creature entities no longer being classified correctly (#67)

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: extremeheat <13713600+extremeheat@users.noreply.github.com>
Co-authored-by: SuperGamerTron <45374546+SuperGamerTron@users.noreply.github.com>
Co-authored-by: Paul Fulham <paul.fulham0@gmail.com>
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