Merge updates to current upstream#3841
Draft
GenerelSchwerz wants to merge 133 commits intoPrismarineJS:masterfrom
Draft
Merge updates to current upstream#3841GenerelSchwerz wants to merge 133 commits intoPrismarineJS:masterfrom
GenerelSchwerz wants to merge 133 commits intoPrismarineJS:masterfrom
Conversation
ADD ABILITIES
Fixes PrismarineJS#3764 This commit adds proper handling of the configuration phase introduced in Minecraft 1.20.2 and extended in 1.21+, which is required for server transfers via Velocity proxy. Changes: - New velocity plugin to manage configuration phase state - Modified physics plugin to block movement packets during configuration - Automatic resource pack acceptance during configuration phase - New examples demonstrating Velocity proxy connection and transfers The bot will no longer be kicked with 'Disconnected, while in configuration phase' errors during Velocity server transfers.
The velocity plugin was activating during the initial 1.20.2+ login configuration phase, which is already handled by node-minecraft-protocol. This caused duplicate finish_configuration packets and disabled physics during login, making the bot unable to spawn on 1.20.2+ servers. Changes: - Only activate configuration phase handling after first spawn - Remove duplicate finish_configuration write (minecraft-protocol handles it) - Remove redundant registry_data/select_known_packs listeners - Fix missing newlines at EOF (lint errors) - Fix object-shorthand lint warnings Co-authored-by: Cursor <[email protected]>
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
All alerts resolved. Learn more about Socket for GitHub. This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
rom1504
reviewed
Feb 21, 2026
examples/elytra.js
Outdated
| @@ -1,5 +1,5 @@ | |||
| // This example will shoot the player that said "fire" in chat, when it is said in chat. | |||
| const mineflayer = require('mineflayer') | |||
| const mineflayer = require('../') | |||
Contributor
Author
|
Re-visiting this. I've merged a few various PRs into this fork, including: #3834 and @bindotmasterpro 's fork. |
Contributor
Author
|
All tests pass. Improvements include:
At this point, I cannot separate this PR into other sections since I've been maintaining my own fork. This PR is fully up-to-date with current upstream and passes all tests. |
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
6 participants
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.
No description provided.