Skip to content

Conversation

@rtm516
Copy link
Member

@rtm516 rtm516 commented Jan 7, 2026

Bedrock now seems to require this new type field in the NBT which seems to always be equal to the block you are brushing, without this field the brushed item doesn't show.

@rtm516 rtm516 added the PR: Bugfix When a PR contains a bugfix label Jan 7, 2026
@rtm516 rtm516 marked this pull request as ready for review January 7, 2026 01:26
Copilot AI review requested due to automatic review settings January 7, 2026 01:26
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes an issue where items were not displaying correctly on brushable blocks (suspicious sand/gravel) in Bedrock Edition. The fix addresses NBT format changes in Java Edition and adds a required block type field for Bedrock.

  • Updated NBT tag access to use the correct field name and data type ("count" as int instead of "Count" as byte)
  • Added a "type" field to the Bedrock NBT that specifies the brushable block type
  • Updated copyright year to 2026

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rtm516 rtm516 requested a review from onebeastchris January 8, 2026 18:24
if (identifier == null) {
identifier = blockState.block().javaIdentifier().value();
}
bedrockNbt.putString("type", identifier);
Copy link
Member

Choose a reason for hiding this comment

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

How does the client handle weird types? e.g. sending "minecraft:glass" for some reason?

Malformed blockentity nbt is a thing, would be lovely if we'd have a safeguard

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Bugfix When a PR contains a bugfix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants