Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1.21 #3480

Merged
merged 20 commits into from
Oct 26, 2024
Merged

1.21 #3480

merged 20 commits into from
Oct 26, 2024

Conversation

Madlykeanu
Copy link
Contributor

No description provided.

@Madlykeanu Madlykeanu changed the title add 1.21 to version.js 1.21 Oct 16, 2024
Copy link

socket-security bot commented Oct 18, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher

🚮 Removed packages: npm/[email protected]

View full report↗︎

@Madlykeanu
Copy link
Contributor Author

chat works but its recieving it differently for some reason in 1.21
image

@Madlykeanu
Copy link
Contributor Author

in 1.21 the type in the message is 1 for some reason, but in 1.20.6 it is 0, idk why this is happening, isnt type 1 mean its an action/emote message? whereas type 0 is player?

1.20.6
image

1.21
image

@rom1504
Copy link
Member

rom1504 commented Oct 22, 2024

Mojang usually breaks things for no reason.

Add a feature in mcdata to distinguish

@SuperGamerTron
Copy link

Looks like the chat types got incremented in the newest protocol version for whatever reason.
image
I was able to fix it with this: SuperGamerTron/prismarine-registry@6d2e1ca

@rom1504
Copy link
Member

rom1504 commented Oct 25, 2024 via email

@rom1504
Copy link
Member

rom1504 commented Oct 25, 2024

@rom1504
Copy link
Member

rom1504 commented Oct 25, 2024

Chat test fixed

@rom1504
Copy link
Member

rom1504 commented Oct 25, 2024

Failing at consume now with this

TypeError: SizeOf error for undefined : Cannot read properties of undefined (reading 'x')
at Object.vec2f (eval at compile (/home/runner/work/mineflayer/mineflayer/node_modules/protodef/src/compiler.js:262:12), :83:21)
at Object.packet_use_item (eval at compile (/home/runner/work/mineflayer/mineflayer/node_modules/protodef/src/compiler.js:262:12), :2308:26)
at eval (eval at compile (/home/runner/work/mineflayer/mineflayer/node_modules/protodef/src/compiler.js:262:12), :2376:56)
at packet (eval at compile (/home/runner/work/mineflayer/mineflayer/node_modules/protodef/src/compiler.js:262:12), :2379:9)
at CompiledProtodef.sizeOf (/home/runner/work/mineflayer/mineflayer/node_modules/protodef/src/compiler.js:89:14)
at e.message (/home/runner/work/mineflayer/mineflayer/node_modules/protodef/src/compiler.js:96:40)
at tryCatch (/home/runner/work/mineflayer/mineflayer/node_modules/protodef/src/utils.js:50:16)
at CompiledProtodef.createPacketBuffer (/home/runner/work/mineflayer/mineflayer/node_modules/protodef/src/compiler.js:96:20)
at Serializer.createPacketBuffer

Seems like wrong packet definition

@rom1504
Copy link
Member

rom1504 commented Oct 26, 2024

so the problem is pchat toString is producing "* flatbot hello" instead of " hello" which makes the test get stuck

@rom1504
Copy link
Member

rom1504 commented Oct 26, 2024

fixed that; we're now at fishing test

@rom1504
Copy link
Member

rom1504 commented Oct 26, 2024

I merged and released most deps

I left some comments in nmp and mcdata prs; would be nice to look into it so we can merge those too

@rom1504
Copy link
Member

rom1504 commented Oct 26, 2024

all external tests working

now only internal tests to fix

@rom1504
Copy link
Member

rom1504 commented Oct 26, 2024

now all passing but there seems to be a weird regression only in 1.19.2 https://github.com/PrismarineJS/mineflayer/actions/runs/11534777950/job/32109169450

@rom1504 rom1504 marked this pull request as ready for review October 26, 2024 21:35
@rom1504
Copy link
Member

rom1504 commented Oct 26, 2024

I do not reproduce any failure in 1.19.2 locally

@rom1504
Copy link
Member

rom1504 commented Oct 26, 2024

rerunning fixes it; something flaky

@rom1504
Copy link
Member

rom1504 commented Oct 26, 2024

tested jumper digger inventory manually, it works

@rom1504 rom1504 merged commit 4aa10fb into PrismarineJS:master Oct 26, 2024
24 checks passed
@rom1504
Copy link
Member

rom1504 commented Oct 26, 2024

/makerelease

@rom1504bot rom1504bot mentioned this pull request Oct 26, 2024
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.

4 participants