Skip to content

Fix headless callbacks - #16

Merged
Paliak merged 1 commit into
Paliak:tests-branchfrom
AdamZ-8113:fix-headless-callbacks
Sep 13, 2026
Merged

Paliak merged 1 commit into
Paliak:tests-branchfrom
AdamZ-8113:fix-headless-callbacks

Conversation

@AdamZ-8113

Copy link
Copy Markdown

Description of the problem being solved:

This fixes the mainObject nil error during headless startup. HeadlessWrapper.lua still had old callback functions that overrode the ones loaded from _SimpleGraphic.def.lua. Those functions were setting the old local variables, while the newer wrapper was reading mainObject and callbackTable.

Steps taken to verify a working solution:

Tested locally against tests-branch at 59c25d23 using the official PoB test container.

  • Confirmed the startup error before the fix and successful startup afterward.
  • Loaded and calculated all 500 builds from the live pob.codes API in batches of 50. Also checked callback registration and dispatch.
  • Ran the existing unit tests: 583 passed, with one failure in the trade-query 401 response test.

Link to a build that showcases this PR:

Before screenshot:

After screenshot:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants