Skip to content

feat(appeng): integrate StackApi for full fluid and essentia support.#169

Open
hinyb wants to merge 8 commits intoGTNewHorizons:masterfrom
hinyb:appeng
Open

feat(appeng): integrate StackApi for full fluid and essentia support.#169
hinyb wants to merge 8 commits intoGTNewHorizons:masterfrom
hinyb:appeng

Conversation

@hinyb
Copy link

@hinyb hinyb commented Dec 28, 2025

This PR refactors the old logic via new StackAPI, providing full support for fluid and essentia, matching the esisting item support.

Breaking Changes
To maintain consistency across the API, update the parameter order of setPatternSlot to align with storeInterfacePattern and clearInterfacePattern

  • Old: function(slot:number, database:address, entry:number, size:number, index:number)
  • New: function(slot:number, index:number[, database:address, entry:number, size:number]):boolean

Changes:

  • Full fluid and essentia Support via the new StackAPI.
  • Add network_changed event subscription.
  • Fix thread-safety issues in allItems.

fix GTNewHorizons/GT-New-Horizons-Modpack#19874
fix GTNewHorizons/GT-New-Horizons-Modpack#19567
fix GTNewHorizons/GT-New-Horizons-Modpack#20839
fix GTNewHorizons/GT-New-Horizons-Modpack#23349

@Dream-Master Dream-Master added the 🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta label Dec 28, 2025
@hinyb

This comment was marked as resolved.

@hinyb hinyb changed the title feat(appeng): add allFluids and network_changed event subscription. feat(appeng): add allFluids and network_changed event subscription and fix thread-safety issues in allItems. Dec 31, 2025
@PLASMAchicken

This comment was marked as resolved.

@hinyb

This comment was marked as resolved.

@hinyb hinyb changed the title feat(appeng): add allFluids and network_changed event subscription and fix thread-safety issues in allItems. feat(appeng): add fluid support and network_changed event subscription and fix thread-safety issues in allItems. Jan 6, 2026
@PLASMAchicken
Copy link
Member

Possibly causes GTNewHorizons/GT-New-Horizons-Modpack#23349

@konradmoesch
Copy link

I have tested the build artifact and my crashes have gone away, no new issues.
Whats still needed to be done? Can I help w/ anything? Looks good to me

@Dream-Master
Copy link
Member

we need to find some one review it.
@konradmoesch thanks for testing

BREAKING CHANGE: update the parameter order of `setPatternSlot` to align with `storeInterfacePattern` and `clearInterfacePattern`
@hinyb hinyb changed the title feat(appeng): add fluid support and network_changed event subscription and fix thread-safety issues in allItems. feat(appeng): integrate StackApi for full fluid and essentia support. Feb 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚧 Testing on Zeta Do not merge yet, testing this PR on Zeta

Projects

None yet

4 participants