[ISSUE]: Inventory peripherals are incompatible with CC:T's built-in implementation

[SquidDev]

Mod Version 1.0.1-beta

Describe the bug KubeJS+CC:Tweaked provides its own implementation of CC:T's inventory peripherals. However, this implementation has several incompatibilities with the original, meaning that programs that work with just CC:T do not work with KubeJS+CC:Tweaked when installed.

• Peripherals will always be called inventory rather than their block entity name (e.g. inventory_0 rather than minecraft:chest_0.
• Optional arguments are not correctly handled — pushItems("inventory_1", 1, nil, 1) should run fine, but instead throws an error that the limit parameter must be a number.

This problem is exacerbated by the fact that KubeJS provides its implementation of the generic peripherals for all block entities, not just those where a Javascript method has been written.

To Reproduce/Expected behaviour

1. Place a computer and (full block) wired modem down, then place two chests on either side of the modem.
2. Right click the modem, observe that the peripherals are incorrectly called inventory_N, rather than minecraft:chest_N.
3. Write a program to move an item from slot 1 of the first inventory to slot 1 of the second (e.g. peripheral.wrap("inventory_0").pushItems("inventory_1", 1, nil, 1). This should run, but instead errors with "Limit must be an invalid integer".

Additional context Ideally this mod would sit on top of the generic peripheral system, rather than wholesale replacing it. That's currently not possible, but I'm definitely happy to discuss what changes would be needed on CC:T's side to make it so.

[wolfieboy09]

Question, what MC version are you on?

[SquidDev]

Forge 1.20.1

[wolfieboy09]

Thanks for that. Been talking with the other developer. We have located what the problem was, and we will look into fixing it when we have the time. We also really appreciate you being open to finding another way to make our peripherals co-exist with the generic ones.