HoverEvent.ShowItem not working in 1.20.5+

[ShaneBeee]

Expected behavior

I expect this method to work like on 1.20.4 where it would show all item modifications

Observed/Actual behavior

When using the hover event to show item, the vanilla/base item is shown, and no modifications are show (ex: enchantments, display name).

Steps/models to reproduce

Test Code:

ItemStack itemStack = new ItemStack(Material.DIAMOND);
itemStack.editMeta(itemMeta -> {
    itemMeta.displayName(Component.text("Mr Diamond"));
    itemMeta.addEnchant(Enchantment.SHARPNESS, 3, true);
});
TextComponent letsSeeThatItem = Component.text("lets see that item");
BinaryTagHolder binaryTagHolder = BinaryTagHolder.binaryTagHolder(itemStack.getItemMeta().getAsString());
HoverEvent.ShowItem showItem = HoverEvent.ShowItem.showItem(itemStack.getType().key(), 1, binaryTagHolder);
TextComponent textComponent = letsSeeThatItem.hoverEvent(HoverEvent.showItem(showItem));
sender.sendMessage(textComponent);

Plugin and Datapack List

[09:30:53 INFO]: Server Plugins (8):
[09:30:53 INFO]: Bukkit Plugins:
[09:30:53 INFO]:  - BeeConomy, RandomTeleport, SkBee, SkBriggy, Skript, skript-reflect, spark, Vault

Paper version

Paper version git-Paper-49 (MC: 1.20.6)

Other

No response

[NonSwag]

BinaryTagHolder doesnt work anymore, you should use ItemStack#asHoverEvent

[ShaneBeee]

oooo awesome, ok thank you!

[NonSwag]

No problem, had the exact same issue the day after release of paper 1.20.5.

[jpenilla]

Vanilla expects item components instead of a single nbt compound now. asHoverEvent will handle populating the component map for you.

[ShaneBeee]

I tried it, and it works perfectly :) thanks!
I guess you guys can close this issue.