Skip to content

Feat/max priority api and transpiler cleanup#9

Open
coolnether123 wants to merge 12 commits intoMainfrom
feat/max-priority-api-and-transpiler-cleanup
Open

Feat/max priority api and transpiler cleanup#9
coolnether123 wants to merge 12 commits intoMainfrom
feat/max-priority-api-and-transpiler-cleanup

Conversation

@coolnether123
Copy link
Owner

  • Fixed rulebuilder handling for priorities above 4
  • Clamped random rule assignment through the normal max-priority path
  • Kept the default max priority at 9 and made that default consistent
  • Replaced hardcoded enable-priority behavior with shared logic
  • Improved rulebuilder labels/tooltips for extended priorities
  • Added a public, documented, reflection-friendly priority API
  • Reduced noisy transpiler warnings and made snapshot behavior more selective
  • Updated cooperative patching and test harness code to use the new diagnostics flow

TreadTheDawnGames and others added 12 commits December 21, 2025 22:09
Integrated a Harmony-based transpiler/framework and refactored  manual priorities across the mod. Added a set of ModAPI.Harmony/Spine helper files (HarmonyBootstrap, HarmonyCompat, HarmonyHelper, HarmonyUtil, PatchRegistry and many transpiler helpers) and wired them into the project file. Refactored and extended priority logic: centralized remapping, tooltip and color logic, and replaced hardcoded vanilla max-priority constants with calls to a configurable GetMaxPriority(); added robust IL pattern-finding/replace helpers and safer transpiler usage. UI/drawing changes: CustomWorkBoxDrawer now supports compact priority rendering and Patch_WorkPriority_DoCell_Unified was updated to draw compact priorities, adjust caching/layout comments, and tidy hover/skill overlays. Minor cleanups in Pawn_WorkSettings prefix and project file updates; compiled assembly and pdb were updated.
Still need to make a proper vanilla default number for higher priority maxes.
@coolnether123 coolnether123 self-assigned this Mar 10, 2026
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