Skip to content

Fix legacy nausea effect intensity on 1.21.5+ clients#4964

Merged
kennytv merged 4 commits into
ViaVersion:masterfrom
itslilscorp:1.21.5-Nausea-Fix
Jun 9, 2026
Merged

Fix legacy nausea effect intensity on 1.21.5+ clients#4964
kennytv merged 4 commits into
ViaVersion:masterfrom
itslilscorp:1.21.5-Nausea-Fix

Conversation

@itslilscorp

Copy link
Copy Markdown
Contributor

What changed

Sets the mob effect blend flag for nausea when translating UPDATE_MOB_EFFECT from 1.21.4 to 1.21.5.

Why

Nausea has explicit blend timing in 1.21.5. Without the blend flag, modern clients skip the transition and short/low-intensity nausea effects can appear at full strength immediately.

This preserves all existing mob effect flags and only adds the blend flag for nausea.

Testing

  • Verified on a 1.12.2 server with a modern client that short nausea effects no longer render at full strength immediately.

AI was used to help investigate the issue and prepare the patch. The behavior was reproduced and the resulting change was manually tested.

Set the 1.21.5 mob effect blend flag for translated nausea effects.

Nausea gained explicit blend timing in 1.21.5, and packets without the blend flag cause modern clients to skip the transition and render the effect at full strength immediately. Preserve the existing mob effect flags and only add the blend flag for nausea.

@kennytv kennytv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This doesn't work, the id is wrong. You need to actually verify AI output

Target the nausea effect (id 8) instead of id 9. :)
@itslilscorp

Copy link
Copy Markdown
Contributor Author

Ah, that's my bad. Before narrowing the fix down to just changing nausea, the AI previously incorrectly applied this to all effects. So, it looks like when I was verifying this build, I left that old version in my testing environment by accident, so as far as I knew, it "worked". I've updated the id to properly target nausea AND tested with the actual proper jar this time. :)

@itslilscorp itslilscorp requested a review from kennytv June 7, 2026 22:25

@kennytv kennytv left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks

@kennytv kennytv merged commit d05985f into ViaVersion:master Jun 9, 2026
1 check 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