Describe the bug
One can simply wrap a heartbound chest like any other chest and pushItems items in and out of it.

To Reproduce

Expected behavior
The chest should not be wrap-able at all.

Minecraft version
Minecraft 1.20.1
Fabric 0.16.9

Mod version
Spectrum: 1.8.1
CC:Tweaked: 1.114.2

Does this work if someone else places the turtle? IIRC, turtles store some information about the player who placed them and can use it to act as that player.

Yes, it works on other people's chests

If that is the case I assume Turtles can also access chests that are using vanillas Lock Mechanism. Can you test that, please? In that case that will need to be fixed on Computercrafts side.

If that is the case I assume Turtles can also access chests that are using vanillas Lock Mechanism. Can you test that, please? In that case that will need to be fixed on Computercrafts side.

Indeed they can. Forwarding the issue, then.

Good to know, thank you!

I'll watch their issue. Once that is fixed they also will not be able to access heartbound chests anymore

So locked chests are a GUI-only mechanism — they don't defend against inserting/removing items via automation (hoppers, CC turtles, etc...).

The issue here is that:

• CC:T finds inventories using Fabric's ItemStorage block lookup API, via the null side. We choose to use the null side so that you can interact with the whole inventory at once, rather than just a given side.
• This gives us an InventoryStorageImpl — as this just targets a normal Inventory, rather than a SidedInventory.
• This means HeartboundChestBlockEntity's canInsert/canExtract are ignored.

I don't think any mod is really doing anything wrong here, just the combination of the three just ends up broken!

The "best" thing to do would be to change HeartboundChestBlockEntity to not implement Inventory in the first place, but obviously that's quite hard to given it inherits from SpectrumChestBlockEntity. The easiest fix might just be to register an empty storage for the block — something like this should work:

ItemStorage.SIDED.registerForBlockEntity((be, d) -> Storage.empty(), SpectrumBlockEntities.HEARTBOUND_CHEST);

Not a bad idea. Thanks!