Skip to content

Conversation

BrickPi
Copy link
Contributor

@BrickPi BrickPi commented Jun 24, 2025

A couple of improvements to mod loading in preparation for main repo ModMenu work.

  • Moves generic modloading logic from headless simulation to GameManager
  • Loads all mod descriptors present on disk (so that we can present them to the user in an in-game mod menu)
  • Loads selected mods + dependencies, with improved error handling and (simple) cycle prevention.
  • Allows users to specify full name or shorthand mod name when loading mods on command line ("HPM" and "Historical Project Mod 0.4.6" are both valid, for instance)

@BrickPi BrickPi added enhancement New feature or request topic:core labels Jun 24, 2025
@BrickPi BrickPi force-pushed the load-mod-descriptors branch from 3740914 to 606edda Compare June 24, 2025 21:49
@BrickPi BrickPi force-pushed the load-mod-descriptors branch from 606edda to 9586a84 Compare June 25, 2025 14:12
@BrickPi BrickPi force-pushed the load-mod-descriptors branch 2 times, most recently from a55d5b1 to 7f50a20 Compare July 6, 2025 13:19
@BrickPi BrickPi force-pushed the load-mod-descriptors branch 2 times, most recently from 0941b0e to d9a3105 Compare July 6, 2025 16:24
@BrickPi BrickPi force-pushed the load-mod-descriptors branch from d9a3105 to bbfc74d Compare July 7, 2025 11:34
@BrickPi BrickPi force-pushed the load-mod-descriptors branch from bbfc74d to 7c23bd2 Compare July 7, 2025 16:47
Spartan322
Spartan322 previously approved these changes Jul 7, 2025
@Spartan322 Spartan322 dismissed their stale review July 7, 2025 17:10

I get six errors on running it.

wvpm
wvpm previously approved these changes Jul 14, 2025
@BrickPi BrickPi force-pushed the load-mod-descriptors branch from 8fafde8 to 39ef1a5 Compare July 15, 2025 09:42
wvpm
wvpm previously approved these changes Jul 15, 2025
@wvpm wvpm force-pushed the load-mod-descriptors branch from 39ef1a5 to 7621ade Compare July 31, 2025 07:43
@wvpm wvpm force-pushed the load-mod-descriptors branch 2 times, most recently from a47befc to aca26a7 Compare July 31, 2025 07:54
@wvpm wvpm force-pushed the load-mod-descriptors branch from aca26a7 to 00fa204 Compare July 31, 2025 07:56
@wvpm wvpm added this to the 0.9.0 milestone Aug 3, 2025
@wvpm wvpm merged commit c5dfb95 into master Aug 3, 2025
16 checks passed
@wvpm wvpm deleted the load-mod-descriptors branch August 3, 2025 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request topic:core
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants