Fix villager suppot

Versions

• mineflayer: 2.12.0
• server: LAN 1.8.8
• node: v13.3.0

Detailed description of a problem

At first when I try to use bot.openVillager(villagerEntity) I get this error:

(node:10190) UnhandledPromiseRejectionWarning: AssertionError [ERR_ASSERTION]: Expected values to be strictly equal:
120 !== undefined
at Bot.openVillager (/dir/node_modules/mineflayer/lib/plugins/villager.js:56:12)

Which is easy to solve (villager in 1.8 has a capital letter), but lets just comment that assert for simplicity. The problem comes when I try to run the code without that assert and I get this error:

/dir/node_modules/minecraft-protocol/src/transforms/framing.js:67
          } else { throw e }
                   ^

Error: Read error for trades.3.firstInput.unknown.true.nbtData.type : 18 is not in the mappings value
    at ProtoDef.readMapper (/dir/node_modules/protodef/src/datatypes/utils.js:30:34)
    at ProtoDef.read (/dir/node_modules/protodef/src/protodef.js:36:25)
    at ProtoDef.read (/dir/node_modules/protodef/src/protodef.js:110:29)
    at /dir/node_modules/protodef/src/datatypes/structures.js:45:32
    at tryCatch (/dir/node_modules/protodef/src/utils.js:50:16)
    at tryDoc (/dir/node_modules/protodef/src/utils.js:54:10)
    at /dir/node_modules/protodef/src/datatypes/structures.js:44:5
    at Array.forEach (<anonymous>)
    at ProtoDef.readContainer (/dir/node_modules/protodef/src/datatypes/structures.js:43:12)
    at ProtoDef.read (/dir/node_modules/protodef/src/protodef.js:36:25) {
  field: 'trades.3.firstInput.unknown.true.nbtData.type'
}

I guess some changes broke backwards compatibility with 1.8 villagers.

Current code

let villager = bot.openVillager(villagerEntity)

Expected behavior

Shouldn’t throw error

[rom1504]

ok got it,the openVillager callback was called wrong, fixed and released as 2.14.1

[rom1504]

and why is village instance printed, is that related ?