Set sequence field correctly in activateItem (#3445)

* updating activateItem

* updating activateItem

* fix lint

* moved sequence upward

* moved sequence upward

* removed comment, these tests passing feel like roulette.

* reorganized

---------

Co-authored-by: extremeheat <[email protected]>

Author: GenerelSchwerz

lib/plugins/inventory.js

@@ -26,6 +26,7 @@ function inject (bot, { hideErrors }) {
   const windows = require('prismarine-windows')(bot.version)
 
   let eatingTask = createDoneTask()
+  let sequence = 0
 
   let nextActionNumber = 0 // < 1.17
   let stateId = -1
@@ -43,6 +44,7 @@ function inject (bot, { hideErrors }) {
   bot.inventory = windows.createWindow(0, 'minecraft:inventory', 'Inventory')
   bot.currentWindow = null
   bot.usingHeldItem = false
+
   Object.defineProperty(bot, 'heldItem', {
     get: function () {
       return bot.inventory.slots[bot.QUICK_BAR_START + bot.quickBarSlot]
@@ -111,6 +113,8 @@ function inject (bot, { hideErrors }) {
 
   function activateItem (offHand = false) {
     bot.usingHeldItem = true
+    sequence++
+
     if (bot.supportFeature('useItemWithBlockPlace')) {
       bot._client.write('block_place', {
         location: new Vec3(-1, 255, -1),
@@ -123,17 +127,26 @@ function inject (bot, { hideErrors }) {
     } else if (bot.supportFeature('useItemWithOwnPacket')) {
       bot._client.write('use_item', {
         hand: offHand ? 1 : 0,
+        sequence,
         rotation: { x: 0, y: 0 }
       })
     }
   }
 
   function deactivateItem () {
-    bot._client.write('block_dig', {
+    const body = {
       status: 5,
       location: new Vec3(0, 0, 0),
       face: 5
-    })
+    }
+
+    if (bot.supportFeature('useItemWithOwnPacket')) {
+      body.face = 0
+      body.sequence = 0
+    }
+
+    bot._client.write('block_dig', body)
+
     bot.usingHeldItem = false
   }