golden

~~(marking as a draft because it definitely needs discussion + I am still not super happy with some of the implementation)~~ this still stands but it's a misuse of the...

Would it make sense to rename `AbstractInventory` -> `Inventory` and `Inventory` -> `ContainerInventory` (or something like that)? Now that AbstractInventory is much less abstract and in fact contains most of...

Can reproduce on latest commit, seems like it happens because the player logged off and thus has no instance. The correct behaviour here is probably to manually specify the instance....

`InventoryCloseEvent` already supports getting and setting a new inventory, so personally I think `InventorySwapEvent` can be removed and all usages here can be replaced with `InventoryCloseEvent`, since this would just...

I'm still not convinced that InventorySwapEvent is necessary (although I don't really have any authority so I don't know how much this matters). I feel like at the minimum, a...