Skip to content

Bump HarmonyX to 2.16.0#1091

Open
BennyDioxide wants to merge 8 commits intoLavaGang:alpha-developmentfrom
BennyDioxide:chore/bump-harmonyx
Open

Bump HarmonyX to 2.16.0#1091
BennyDioxide wants to merge 8 commits intoLavaGang:alpha-developmentfrom
BennyDioxide:chore/bump-harmonyx

Conversation

@BennyDioxide
Copy link
Copy Markdown

@BennyDioxide BennyDioxide commented Feb 7, 2026

Benefits: osx-arm64 (Apple Silicon) support, .Net 10

Problems to be addressed:
HarmonyX supports osx-arm64 via MonoMod/MonoMod#241, so MonoMod should be updated as well. But since they do a great amount of rewrite, things would be hugely complicated:

  • The entire MonoMod.RuntimeDetour system has been rewrited, making detours no longer have OnDetour to hook on and PatchShield and InstancePatchFix thus stop blocking RuntimeDetour hooks.
  • MonoMod.RuntimeDetour.DetourHelper is gone, DetourHelper::Pin and DetourHelper::GetNativeStart no longer works
  • MonoMod.RuntimeDetour now bundles its own MonoMod.Iced through MonoMod.Core, making lots of conflicts

Along with related dependencies
@HerpDerpinstine HerpDerpinstine added Improvement New feature or request Work-in-Progress This issue or pull request is being worked on currently labels Feb 8, 2026
@BennyDioxide
Copy link
Copy Markdown
Author

In newer MonoMod.RuntimeDetour, it appears to be new events too hook into, (Detour/ILHook)(Applied/Undone).
Unfortunately, the so called events are always called after the detour graph manipulation, so we'll have to manually patch the Apply methods.
Also, Detour is now gone, so we don't have to patch them as well.

As DetourHelper is gone, and they removed the Process.Start
check from platform detection as well, this can be removed.
The MonoMod API changed a lot, which's the time to remove back compatibilities
The upstream Harmony moved it to internal
New MonoMod decided to bundle their own version of Iced of whatever reason
@BennyDioxide BennyDioxide marked this pull request as ready for review February 13, 2026 12:48
@HerpDerpinstine HerpDerpinstine added this to the 0.7.3 milestone Feb 25, 2026
@HerpDerpinstine HerpDerpinstine added Under Review Currently being tested and discussed/looked at further and removed Work-in-Progress This issue or pull request is being worked on currently labels Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Improvement New feature or request Under Review Currently being tested and discussed/looked at further

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants