add more context to to errors

Author: extremeheat

lib/plugins/anvil.js

@@ -12,7 +12,7 @@ function inject (bot) {
   async function openAnvil (anvilBlock) {
     const anvil = await bot.openBlock(anvilBlock)
     if (!matchWindowType(anvil)) {
-      throw new Error('This is not a anvil-like window')
+      throw new Error('Not a anvil-like window: ' + JSON.stringify(anvil))
     }
 
     function err (name) {

lib/plugins/chat.js

@@ -144,7 +144,7 @@ function inject (bot, options) {
   function chatWithHeader (header, message) {
     if (typeof message === 'number') message = message.toString()
     if (typeof message !== 'string') {
-      throw new Error('Incorrect type! Should be a string or number.')
+      throw new Error('Chat message type must be a string or number: ' + typeof message)
     }
 
     if (!header && message.startsWith('/')) {

lib/plugins/chest.js

@@ -23,7 +23,7 @@ function inject (bot) {
       throw new Error('containerToOpen is neither a block nor an entity')
     }
 
-    if (!matchWindowType(chest)) { throw new Error('Non-container window used as a container') }
+    if (!matchWindowType(chest)) { throw new Error('Non-container window used as a container: ' + JSON.stringify(chest)) }
     return chest
   }
 

lib/plugins/craft.js

@@ -12,7 +12,7 @@ function inject (bot) {
     assert.ok(recipe)
     count = parseInt(count ?? 1, 10)
     if (recipe.requiresTable && !craftingTable) {
-      throw new Error('recipe requires craftingTable')
+      throw new Error('Recipe requires craftingTable, but one was not supplied: ' + JSON.stringify(recipe))
     }
 
     try {
@@ -41,7 +41,7 @@ function inject (bot) {
         windowCraftingTable = window
       }
       if (!windowCraftingTable.type.startsWith('minecraft:crafting')) {
-        throw new Error('crafting: non craftingTable used as craftingTable')
+        throw new Error('crafting: non craftingTable used as craftingTable: ' + windowCraftingTable.type)
       }
       await startClicking(windowCraftingTable, 3, 3)
     } else {

lib/plugins/enchantment_table.js

@@ -9,7 +9,7 @@ function inject (bot) {
     let ready = false
     const enchantmentTable = await bot.openBlock(enchantmentTableBlock)
     if (!enchantmentTable.type.startsWith('minecraft:enchant')) {
-      throw new Error('This is not an enchantment table')
+      throw new Error('Expected minecraft:enchant when opening table but got ' + enchantmentTable.type)
     }
 
     resetEnchantmentOptions()

lib/plugins/villager.js

@@ -78,7 +78,7 @@ function inject (bot, { version }) {
     bot._client.on(tradeListPacket, gotTrades)
     const villager = await villagerPromise
     if (villager.type !== 'minecraft:villager' && villager.type !== 'minecraft:merchant') {
-      throw new Error('This is not a villager')
+      throw new Error('Expected minecraft:villager or minecraft:mechant type, but got ' + villager.type)
     }
 
     villager.trades = null