get enchants() updated.

[IKorzI]

Added enchantment handling for versions 1.20.5 to the latest.
The enchantment handling for versions 1.13 to 1.20.4 has been changed. The "typeOfEnchantLevelValue" feature remains "short" for versions 1.8 to the latest. Because of this, the "typeOfEnchantLevelValue" condition has been changed.

[IKorzI]

Made with: PrismarineJS/minecraft-data#986

[extremeheat]

enchants really should not be a getter at all. Too much logic it's doing that should only need to be done on construction

[extremeheat]

CI failing

[IKorzI]

I apologize in advance.
I'm just getting started with pull requests so I might not understand some things.
At the moment, Cl errors are solved by replacing the string

} else if (typeOfEnchantLevelValue === 'short' && enchantNbtKey === 'Enchantments') {

with the string

} else if (typeOfEnchantLevelValue === 'string' && enchantNbtKey === 'Enchantments') {

The typeOfValueForEnchantLevel feature had the value string for versions 1.13 and higher.
That's why there was a condition typeOfEnchantLevelValue === 'string'

But I removed it and therefore changed the condition to typeOfEnchantLevelValue === 'short'

PrismarineJS/minecraft-data@ddd0c39

The tests apparently also use the typeOfValueForEnchantLevel feature with the value string

Do I need to return the old condition typeOfEnchantLevelValue === 'string'?