Skip to content

Update CI to Node 24 - #29

Merged
rom1504 merged 2 commits into
masterfrom
update-ci-node-24
Mar 30, 2026
Merged

rom1504 merged 2 commits into
masterfrom
update-ci-node-24

Conversation

@rom1504

@rom1504 rom1504 commented Mar 29, 2026

Copy link
Copy Markdown
Member

Summary

  • Update CI node version to 24

🤖 Generated with Claude Code

rom1504 and others added 2 commits March 29, 2026 23:48
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The registryEntryHolder write handler in minecraft-protocol has a bug
where it does `offset += 1` instead of writing a 0x00 varint byte for
inline sound data. Combined with protodef using Buffer.allocUnsafe,
this leaves uninitialized memory in the serialized buffer, causing
non-deterministic round-trip failures for sound_effect packets in
versions 1.19.3+. Node 24 exposes this more frequently.

Workaround: replace Buffer.allocUnsafe with Buffer.alloc during tests
to ensure zeroed buffers, neutralizing the upstream bug.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@rom1504
rom1504 merged commit cd70884 into master Mar 30, 2026
3 checks passed
@rom1504

rom1504 commented Mar 30, 2026

Copy link
Copy Markdown
Member Author

/makerelease

1 similar comment
@rom1504

rom1504 commented Mar 30, 2026

Copy link
Copy Markdown
Member Author

/makerelease

@rom1504bot rom1504bot mentioned this pull request Mar 30, 2026
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