mineflayer 3.17.0 → 4.1.0

package/lib/features.json

@@ -7,7 +7,7 @@
   {
     "name": "spawner",
     "description": "spawner is called spawner",
-    "versions": ["1.13", "1.18.1"]
+    "versions": ["1.13", "1.18.2"]
   },
   {
     "name": "blockMetadata",
@@ -17,12 +17,12 @@
   {
     "name": "blockStateId",
     "description": "block metadata is encoded as state id",
-    "versions": ["1.13", "1.18.1"]
+    "versions": ["1.13", "1.18.2"]
   },
   {
     "name": "creativeSleepNearMobs",
     "description": "can sleep near mobs in creative",
-    "versions": ["1.13", "1.18.1"]
+    "versions": ["1.13", "1.18.2"]
   },
   {
     "name": "fixedPointPosition",
@@ -32,7 +32,7 @@
   {
     "name": "doublePosition",
     "description": "Entity positions are represented with double",
-    "versions": ["1.9", "1.18.1"]
+    "versions": ["1.9", "1.18.2"]
   },
   {
     "name": "fixedPointDelta",
@@ -42,7 +42,7 @@
   {
     "name": "fixedPointDelta128",
     "description": "Delta of position are represented with fixed point numbers times 128",
-    "versions": ["1.9", "1.18.1"]
+    "versions": ["1.9", "1.18.2"]
   },
   {
     "name": "customChannelMCPrefixed",
@@ -52,12 +52,12 @@
   {
     "name": "customChannelIdentifier",
     "description": "custom channel is an identifier starting in minecraft namespace",
-    "versions": ["1.13", "1.18.1"]
+    "versions": ["1.13", "1.18.2"]
   },
   {
     "name": "dimensionDataIsAvailable",
     "description": "dimensionData is available, providing an additional information about the current dimension",
-    "versions": ["1.17", "1.18.1"]
+    "versions": ["1.17", "1.18.2"]
   },
   {
     "name": "useItemWithBlockPlace",
@@ -67,7 +67,7 @@
   {
     "name": "useItemWithOwnPacket",
     "description": "use item is done with its own packet",
-    "versions": ["1.9", "1.18.1"]
+    "versions": ["1.9", "1.18.2"]
   },
   {
     "name": "blockPlaceHasHeldItem",
@@ -87,7 +87,7 @@
   {
     "name": "blockPlaceHasInsideBlock",
     "description": "block_place packet has inside block",
-    "versions": ["1.14", "1.18.1"]
+    "versions": ["1.14", "1.18.2"]
   },
   {
     "name": "teleportUsesPositionPacket",
@@ -102,7 +102,7 @@
   {
     "name": "teleportUsesOwnPacket",
     "description": "teleport is done using its own packet",
-    "versions": ["1.9", "1.18.1"]
+    "versions": ["1.9", "1.18.2"]
   },
   {
     "name": "oneBlockForSeveralVariations",
@@ -112,7 +112,7 @@
   {
     "name": "blockSchemeIsFlat",
     "description": "all variations of a packet have their own id",
-    "versions": ["1.13", "1.18.1"]
+    "versions": ["1.13", "1.18.2"]
   },
   {
     "name": "tabCompleteHasNoToolTip",
@@ -122,7 +122,7 @@
   {
     "name": "tabCompleteHasAToolTip",
     "description": "tab complete has a tool tip",
-    "versions": ["1.13", "1.18.1"]
+    "versions": ["1.13", "1.18.2"]
   },
   {
     "name": "effectAreMinecraftPrefixed",
@@ -132,7 +132,7 @@
   {
     "name": "effectAreNotPrefixed",
     "description": "effect are not prefixed",
-    "versions": ["1.13", "1.18.1"]
+    "versions": ["1.13", "1.18.2"]
   },
   {
     "name": "itemsAreAlsoBlocks",
@@ -142,12 +142,12 @@
   {
     "name": "itemsAreNotBlocks",
     "description": "items are not block",
-    "versions": ["1.13", "1.18.1"]
+    "versions": ["1.13", "1.18.2"]
   },
   {
     "name": "fishingBobberCorrectlyNamed",
     "description": "the fishing hook entity is named fishing_bobber",
-    "versions": ["1.14", "1.18.1"]
+    "versions": ["1.14", "1.18.2"]
   },
   {
     "name": "editBookIsPluginChannel",
@@ -157,7 +157,7 @@
   {
     "name": "hasEditBookPacket",
     "description": "book editing is handled through a packet",
-    "versions": ["1.13", "1.18.1"]
+    "versions": ["1.13", "1.18.2"]
   },
   {
     "name": "clientUpdateBookIdWhenSign",
@@ -197,12 +197,12 @@
   {
     "name": "dimensionIsAWorld",
     "description": "description is a world name (string)",
-    "versions": ["1.16.2", "1.18.1"]
+    "versions": ["1.16.2", "1.18.2"]
   },
   {
     "name": "dimensionDataIsAvailable",
     "description": "dimensionData is available, describing additional dimension information",
-    "versions": ["1.17", "1.18.1"]
+    "versions": ["1.17", "1.18.2"]
   },
   {
     "name": "doesntHaveChestType",
@@ -227,22 +227,22 @@
   {
     "name": "hasAttackCooldown",
     "description": "if there is a cooldown after attacks to deal full damage",
-    "versions": ["1.9", "1.18.1"]
+    "versions": ["1.9", "1.18.2"]
   },
   {
     "name": "usesLoginPacket",
     "description": "uses the login packet as defined in mcData",
-    "versions": ["1.16", "1.18.1"]
+    "versions": ["1.16", "1.18.2"]
   },
   {
     "name": "usesMultiblockSingleLong",
     "description": "in the multi_block_change packet is stored as a single number",
-    "versions": ["1.16.2", "1.18.1"]
+    "versions": ["1.16.2", "1.18.2"]
   },
   {
     "name": "usesMultiblock3DChunkCoords",
     "description": "in the multi_block_change packet, all 3 axis coords are defined",
-    "versions": ["1.16.2", "1.18.1"]
+    "versions": ["1.16.2", "1.18.2"]
   },
   {
     "name": "setBlockUsesMetadataNumber",
@@ -257,7 +257,7 @@
   {
     "name": "selectingTradeMovesItems",
     "description": "selecting a trade automatically puts the required items into trading slots",
-    "versions": ["1.14", "1.18.1"]
+    "versions": ["1.14", "1.18.2"]
   },
   {
     "name": "resourcePackUsesHash",
@@ -272,7 +272,7 @@
   {
     "name": "teamUsesChatComponents",
     "description": "teams use chatcomponents for formatting",
-    "versions": ["1.13", "1.18.1"]
+    "versions": ["1.13", "1.18.2"]
   },
   {
     "name": "teamUsesScoreboard",
@@ -287,7 +287,7 @@
   {
     "name": "enderCrystalNameNoCapsWithUnderscore",
     "description": "this is when the end_crystal's entity name is end_crystal",
-    "versions": ["1.14", "1.18.1"]
+    "versions": ["1.14", "1.18.2"]
   },
   {
     "name": "entityNameUpperCaseNoUnderscore",
@@ -307,7 +307,7 @@
   {
     "name": "stateIdUsed",
     "description": "starting in 1.17.1, actionId has been replaced with stateId",
-    "versions": ["1.17.1", "1.18.1"]
+    "versions": ["1.17.1", "1.18.2"]
   },
   {
     "name": "actionIdUsed",
@@ -317,7 +317,7 @@
   {
     "name": "setSlotAsTransaction",
     "description": "use setslot as transaction instead of just hoping it'll work",
-    "versions": ["1.17", "1.18.1"]
+    "versions": ["1.17", "1.18.2"]
   },
   {
     "name": "armAnimationBeforeUse",
@@ -327,7 +327,7 @@
   {
     "name": "tallWorld",
     "description": "world starts at -64 and ends at 384",
-    "versions": ["1.18", "1.18.1"]
+    "versions": ["1.18", "1.18.2"]
   },
   {
     "name": "sendStringifiedSignText",

package/lib/plugins/anvil.js

@@ -1,5 +1,5 @@
 const assert = require('assert')
-const { callbackify, sleep } = require('../promise_utils')
+const { sleep } = require('../promise_utils')
 const { once } = require('events')
 
 module.exports = inject
@@ -105,11 +105,11 @@ function inject (bot) {
       await bot.transfer(options)
     }
 
-    anvil.combine = callbackify(combine)
-    anvil.rename = callbackify(rename)
+    anvil.combine = combine
+    anvil.rename = rename
 
     return anvil
   }
 
-  bot.openAnvil = callbackify(openAnvil)
+  bot.openAnvil = openAnvil
 }

package/lib/plugins/bed.js

@@ -1,5 +1,4 @@
 const { Vec3 } = require('vec3')
-const { callbackify } = require('../promise_utils')
 
 module.exports = inject
 
@@ -174,7 +173,7 @@ function inject (bot) {
   })
 
   bot.parseBedMetadata = parseBedMetadata
-  bot.wake = callbackify(wake)
-  bot.sleep = callbackify(sleep)
+  bot.wake = wake
+  bot.sleep = sleep
   bot.isABed = isABed
 }

package/lib/plugins/blocks.js

@@ -1,7 +1,7 @@
 const { Vec3 } = require('vec3')
 const assert = require('assert')
 const Painting = require('../painting')
-const { onceWithCleanup, callbackify } = require('../promise_utils')
+const { onceWithCleanup } = require('../promise_utils')
 
 const { OctahedronIterator } = require('prismarine-world').iterators
 
@@ -548,7 +548,7 @@ function inject (bot, { version, storageBuilder }) {
   bot.canSeeBlock = canSeeBlock
   bot.blockAt = blockAt
   bot._updateBlockState = updateBlockState
-  bot.waitForChunksToLoad = callbackify(waitForChunksToLoad)
+  bot.waitForChunksToLoad = waitForChunksToLoad
 }
 
 function onesInShort (n) {

package/lib/plugins/book.js

@@ -1,6 +1,5 @@
 const assert = require('assert')
 const { once } = require('events')
-const { callbackify } = require('../promise_utils')
 
 module.exports = inject
 
@@ -83,11 +82,11 @@ function inject (bot, { version }) {
     return book
   }
 
-  bot.writeBook = callbackify(async (slot, pages) => {
+  bot.writeBook = async (slot, pages) => {
     await write(slot, pages, null, null, false)
-  })
+  }
 
-  bot.signBook = callbackify(async (slot, pages, author, title) => {
+  bot.signBook = async (slot, pages, author, title) => {
     await write(slot, pages, author, title, true)
-  })
+  }
 }

package/lib/plugins/chat.js

@@ -1,5 +1,4 @@
 const { once } = require('events')
-const { printCallbackDepreciation } = require('../promise_utils')
 
 module.exports = inject
 
@@ -170,7 +169,7 @@ function inject (bot, options) {
     chatWithHeader('', message)
   }
 
-  bot.tabComplete = callbackify(tabComplete)
+  bot.tabComplete = tabComplete
 
   function addDefaultPatterns () {
     if (!defaultChatPatterns) return
@@ -194,21 +193,3 @@ function inject (bot, options) {
   }
   bot.awaitMessage = awaitMessage
 }
-
-function callbackify (f) { // specifically for this function because cb isn't the last parameter
-  return function (...args) {
-    const cb = args[1]
-    args.splice(1, 1)
-    return f(...args).then(r => {
-      if (cb) {
-        printCallbackDepreciation()
-        cb(undefined, r)
-      } return r
-    }, err => {
-      if (cb) {
-        printCallbackDepreciation()
-        cb(err)
-      } else throw err
-    })
-  }
-}

package/lib/plugins/chest.js

@@ -1,4 +1,3 @@
-const { callbackify } = require('../promise_utils')
 
 module.exports = inject
 
@@ -26,7 +25,7 @@ function inject (bot) {
     return chest
   }
 
-  bot.openContainer = callbackify(openContainer)
-  bot.openChest = callbackify(openContainer)
-  bot.openDispenser = callbackify(openContainer)
+  bot.openContainer = openContainer
+  bot.openChest = openContainer
+  bot.openDispenser = openContainer
 }

package/lib/plugins/craft.js

@@ -1,6 +1,5 @@
 const assert = require('assert')
 const { once } = require('events')
-const { callbackify } = require('../promise_utils')
 
 module.exports = inject
 
@@ -226,7 +225,7 @@ function inject (bot, { version }) {
     return true
   }
 
-  bot.craft = callbackify(craft)
+  bot.craft = craft
   bot.recipesFor = recipesFor
   bot.recipesAll = recipesAll
 }

package/lib/plugins/creative.js

@@ -1,6 +1,6 @@
 const assert = require('assert')
 const { Vec3 } = require('vec3')
-const { callbackify, sleep } = require('../promise_utils')
+const { sleep } = require('../promise_utils')
 const { once } = require('events')
 
 module.exports = inject
@@ -10,8 +10,8 @@ function inject (bot, { version }) {
 
   // these features only work when you are in creative mode.
   bot.creative = {
-    setInventorySlot: callbackify(setInventorySlot),
-    flyTo: callbackify(flyTo),
+    setInventorySlot: setInventorySlot,
+    flyTo: flyTo,
     startFlying,
     stopFlying
   }
@@ -23,7 +23,7 @@ function inject (bot, { version }) {
     }
   })
 
-  // WARN: This method should not be called twice on the same slot before first callback succeeds
+  // WARN: This method should not be called twice on the same slot before first promise succeeds
   async function setInventorySlot (slot, item) {
     assert(slot >= 0 && slot <= 44)
 

package/lib/plugins/digging.js

@@ -1,5 +1,5 @@
 const { performance } = require('perf_hooks')
-const { createDoneTask, createTask, printCallbackDepreciation } = require('../promise_utils')
+const { createDoneTask, createTask } = require('../promise_utils')
 const BlockFaces = require('prismarine-world').iterators.BlockFace
 const { Vec3 } = require('vec3')
 
@@ -198,35 +198,12 @@ function inject (bot) {
     return block.digTime(type, creative, bot.entity.isInWater, !bot.entity.onGround, enchantments, bot.entity.effects)
   }
 
-  bot.dig = callbackify(dig)
+  bot.dig = dig
   bot.stopDigging = noop
   bot.canDigBlock = canDigBlock
   bot.digTime = digTime
 }
 
-function callbackify (f) { // specifically for this function because cb could be the non-last parameter
-  return function (...args) {
-    const cbIndex = typeof args[1] === 'function' ? 1 : (typeof args[2] === 'function' ? 2 : 3)
-    const cb = args[cbIndex]
-
-    if (cbIndex === 1) args[1] = true
-    else if (typeof args[1] === 'string') args[1] = args[1] === 'ignore' ? args[1] : false
-    else args[1] = !!args[1] // coerce to boolean
-
-    return f(...args).then(r => {
-      if (cb) {
-        printCallbackDepreciation()
-        cb(undefined, r)
-      } return r
-    }, err => {
-      if (cb) {
-        printCallbackDepreciation()
-        cb(err)
-      } else throw err
-    })
-  }
-}
-
 function noop (err) {
   if (err) throw err
 }

package/lib/plugins/enchantment_table.js

@@ -1,6 +1,5 @@
 const assert = require('assert')
 const { once } = require('events')
-const { callbackify } = require('../promise_utils')
 
 module.exports = inject
 
@@ -15,10 +14,10 @@ function inject (bot) {
 
     resetEnchantmentOptions()
 
-    enchantmentTable.enchant = callbackify(enchant)
-    enchantmentTable.takeTargetItem = callbackify(takeTargetItem)
-    enchantmentTable.putTargetItem = callbackify(putTargetItem)
-    enchantmentTable.putLapis = callbackify(putLapis)
+    enchantmentTable.enchant = enchant
+    enchantmentTable.takeTargetItem = takeTargetItem
+    enchantmentTable.putTargetItem = putTargetItem
+    enchantmentTable.putLapis = putLapis
     enchantmentTable.targetItem = function () { return this.slots[0] }
 
     bot._client.on('craft_progress_bar', onUpdateWindowProperty)
@@ -100,5 +99,5 @@ function inject (bot) {
     }
   }
 
-  bot.openEnchantmentTable = callbackify(openEnchantmentTable)
+  bot.openEnchantmentTable = openEnchantmentTable
 }

package/lib/plugins/entities.js

@@ -588,6 +588,7 @@ function inject (bot, { version }) {
   }
 
   function useEntity (target, leftClick, x, y, z) {
+    const sneaking = bot.getControlState('sneak')
     if (x && y && z) {
       bot._client.write('use_entity', {
         target: target.id,
@@ -595,13 +596,13 @@ function inject (bot, { version }) {
         x,
         y,
         z,
-        sneaking: false
+        sneaking
       })
     } else {
       bot._client.write('use_entity', {
         target: target.id,
         mouse: leftClick,
-        sneaking: false
+        sneaking
       })
     }
   }

package/lib/plugins/fishing.js

@@ -1,5 +1,5 @@
 const { Vec3 } = require('vec3')
-const { callbackify, createDoneTask, createTask } = require('../promise_utils')
+const { createDoneTask, createTask } = require('../promise_utils')
 
 module.exports = inject
 
@@ -54,5 +54,5 @@ function inject (bot) {
     await fishingTask.promise
   }
 
-  bot.fish = callbackify(fish)
+  bot.fish = fish
 }

package/lib/plugins/furnace.js

@@ -1,5 +1,4 @@
 const assert = require('assert')
-const { callbackify } = require('../promise_utils')
 
 module.exports = inject
 
@@ -25,11 +24,11 @@ function inject (bot) {
     furnace.totalProgress = null
     furnace.progress = null
     furnace.progressSeconds = null
-    furnace.takeInput = callbackify(takeInput)
-    furnace.takeFuel = callbackify(takeFuel)
-    furnace.takeOutput = callbackify(takeOutput)
-    furnace.putInput = callbackify(putInput)
-    furnace.putFuel = callbackify(putFuel)
+    furnace.takeInput = takeInput
+    furnace.takeFuel = takeFuel
+    furnace.takeOutput = takeOutput
+    furnace.putInput = putInput
+    furnace.putFuel = putFuel
     furnace.inputItem = function () { return this.slots[0] }
     furnace.fuelItem = function () { return this.slots[1] }
     furnace.outputItem = function () { return this.slots[2] }
@@ -118,5 +117,5 @@ function inject (bot) {
     return ticks * 0.05
   }
 
-  bot.openFurnace = callbackify(openFurnace)
+  bot.openFurnace = openFurnace
 }

package/lib/plugins/inventory.js

@@ -1,7 +1,7 @@
 const assert = require('assert')
 const { Vec3 } = require('vec3')
 const { once } = require('events')
-const { callbackify, sleep, createDoneTask, createTask, withTimeout } = require('../promise_utils')
+const { sleep, createDoneTask, createTask, withTimeout } = require('../promise_utils')
 
 module.exports = inject
 
@@ -306,36 +306,38 @@ function inject (bot, { version, hideErrors }) {
       window.emit('close')
       closeWindow(window)
     }
-    window.withdraw = callbackify(async (itemType, metadata, count) => {
+
+    window.withdraw = async (itemType, metadata, count, nbt) => {
       if (bot.inventory.emptySlotCount() === 0) {
         throw new Error('Unable to withdraw, Bot inventory is full.')
       }
-
       const options = {
         window,
         itemType,
         metadata,
         count,
+        nbt,
         sourceStart: 0,
         sourceEnd: window.inventoryStart,
         destStart: window.inventoryStart,
         destEnd: window.inventoryEnd
       }
       await transfer(options)
-    })
-    window.deposit = callbackify(async (itemType, metadata, count) => {
+    }
+    window.deposit = async (itemType, metadata, count, nbt) => {
       const options = {
         window: window,
         itemType,
         metadata,
         count,
+        nbt,
         sourceStart: window.inventoryStart,
         sourceEnd: window.inventoryEnd,
         destStart: 0,
         destEnd: window.inventoryStart
       }
       await transfer(options)
-    })
+    }
   }
 
   async function openBlock (block) {
@@ -643,21 +645,21 @@ function inject (bot, { version, hideErrors }) {
     bot.emit(`setWindowItems:${window.id}`)
   })
 
-  bot.activateBlock = callbackify(activateBlock)
-  bot.activateEntity = callbackify(activateEntity)
-  bot.activateEntityAt = callbackify(activateEntityAt)
-  bot.consume = callbackify(consume)
+  bot.activateBlock = activateBlock
+  bot.activateEntity = activateEntity
+  bot.activateEntityAt = activateEntityAt
+  bot.consume = consume
   bot.activateItem = activateItem
   bot.deactivateItem = deactivateItem
 
   // not really in the public API
-  bot.clickWindow = callbackify(clickWindow)
+  bot.clickWindow = clickWindow
   bot.putSelectedItemRange = putSelectedItemRange
   bot.putAway = putAway
   bot.closeWindow = closeWindow
-  bot.transfer = callbackify(transfer)
-  bot.openBlock = callbackify(openBlock)
-  bot.openEntity = callbackify(openEntity)
-  bot.moveSlotItem = callbackify(moveSlotItem)
+  bot.transfer = transfer
+  bot.openBlock = openBlock
+  bot.openEntity = openEntity
+  bot.moveSlotItem = moveSlotItem
   bot.updateHeldItem = updateHeldItem
 }

package/lib/plugins/physics.js

@@ -3,7 +3,7 @@ const assert = require('assert')
 const math = require('../math')
 const conv = require('../conversions')
 const { performance } = require('perf_hooks')
-const { callbackify, createDoneTask, createTask } = require('../promise_utils')
+const { createDoneTask, createTask } = require('../promise_utils')
 
 const { Physics, PlayerState } = require('prismarine-physics')
 
@@ -223,7 +223,7 @@ function inject (bot, { physicsEnabled }) {
     }
   })
 
-  bot.look = callbackify(async (yaw, pitch, force) => {
+  bot.look = async (yaw, pitch, force) => {
     if (!lookingTask.done) {
       lookingTask.finish() // finish the previous one
     }
@@ -245,15 +245,15 @@ function inject (bot, { physicsEnabled }) {
     }
 
     await lookingTask.promise
-  })
+  }
 
-  bot.lookAt = callbackify(async (point, force) => {
+  bot.lookAt = async (point, force) => {
     const delta = point.minus(bot.entity.position.offset(0, bot.entity.height, 0))
     const yaw = Math.atan2(-delta.x, -delta.z)
     const groundDistance = Math.sqrt(delta.x * delta.x + delta.z * delta.z)
     const pitch = Math.atan2(delta.y, groundDistance)
     await bot.look(yaw, pitch, force)
-  })
+  }
 
   // player position and look (clientbound)
   bot._client.on('position', (packet) => {