From 6274d77d1f3d7be1587ed40badfe2bb309a7e4b5 Mon Sep 17 00:00:00 2001
From: Jason Penilla <11360596+jpenilla@users.noreply.github.com>
Date: Mon, 19 Aug 2024 11:21:30 -0700
Subject: [PATCH] Document and improve mod compatibility

---
 README.md                                     |  8 +++++++
 .../resources/META-INF/neoforge.mods.toml     | 22 +++++++++++++++++++
 .../StateHolderMixin.java                     |  5 ++++-
 .../ZeroCollidingReferenceStateTable.java     |  3 ++-
 4 files changed, 36 insertions(+), 2 deletions(-)

diff --git a/README.md b/README.md
index 719a0f4a..e3741f06 100644
--- a/README.md
+++ b/README.md
@@ -12,5 +12,13 @@ patches. Listed below are notable patches:
  - Random ticking optimisations
  - [Starlight](https://github.com/PaperMC/Starlight/)
 
+## Mod Compatibility
+| Mod         | Compatibility                                                                                                                                                           |
+|-------------|-------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
+| FerriteCore | <details><summary>📝 requires config changes</summary>In `config/ferritecore-mixin.toml`:<br/>Set `replaceNeighborLookup` and `replacePropertyMap` to `false`</details> |
+| ModernFix   | <details><summary>📝 requires config changes</summary>In `config/modernfix-mixins.properties`:<br/>Add `mixin.bugfix.paper_chunk_patches=false`</details>               |
+| Lithium     | ❌ incompatible                                                                                                                                                          |
+| C2ME        | ❌ incompatible                                                                                                                                                          |
+
 ## Contact
 [Discord](https://discord.gg/tuinity)
diff --git a/neoforge/src/main/resources/META-INF/neoforge.mods.toml b/neoforge/src/main/resources/META-INF/neoforge.mods.toml
index 1c657985..68097c67 100644
--- a/neoforge/src/main/resources/META-INF/neoforge.mods.toml
+++ b/neoforge/src/main/resources/META-INF/neoforge.mods.toml
@@ -28,6 +28,28 @@ versionRange = "[1.21,1.21.2)"
 ordering = "NONE"
 side = "BOTH"
 
+[[dependencies.moonrise]]
+modId = "starlight"
+type = "incompatible"
+
+[[dependencies.moonrise]]
+modId = "lithium"
+type = "incompatible"
+
+[[dependencies.moonrise]]
+# unofficial lithium port
+modId = "canary"
+type = "incompatible"
+
+[[dependencies.moonrise]]
+# unofficial lithium port
+modId = "radium"
+type = "incompatible"
+
+[[dependencies.moonrise]]
+modId = "c2me"
+type = "incompatible"
+
 [[mixins]]
 config = "moonrise.mixins.json"
 
diff --git a/src/main/java/ca/spottedleaf/moonrise/mixin/blockstate_propertyaccess/StateHolderMixin.java b/src/main/java/ca/spottedleaf/moonrise/mixin/blockstate_propertyaccess/StateHolderMixin.java
index 6baa5176..c8377b84 100644
--- a/src/main/java/ca/spottedleaf/moonrise/mixin/blockstate_propertyaccess/StateHolderMixin.java
+++ b/src/main/java/ca/spottedleaf/moonrise/mixin/blockstate_propertyaccess/StateHolderMixin.java
@@ -65,6 +65,8 @@ private void init(final CallbackInfo ci) {
             )
     )
     private void loadTable(final Map<Map<Property<?>, Comparable<?>>, S> map, final CallbackInfo ci) {
+        // Uses #entrySet() instead of #values() for ModernFix compat (until when/if they implement #values() on their map) (also in ZCRST#loadInTable)
+
         if (this.optimisedTable.isLoaded()) {
             ci.cancel();
             return;
@@ -72,7 +74,8 @@ private void loadTable(final Map<Map<Property<?>, Comparable<?>>, S> map, final
         this.optimisedTable.loadInTable(map);
 
         // de-duplicate the tables
-        for (final S value : map.values()) {
+        for (final Map.Entry<Map<Property<?>, Comparable<?>>, S> entry : map.entrySet()) {
+            final S value = entry.getValue();
             ((StateHolderMixin<O, S>)(Object)(StateHolder<O, S>)value).optimisedTable = this.optimisedTable;
         }
 
diff --git a/src/main/java/ca/spottedleaf/moonrise/patches/blockstate_propertyaccess/util/ZeroCollidingReferenceStateTable.java b/src/main/java/ca/spottedleaf/moonrise/patches/blockstate_propertyaccess/util/ZeroCollidingReferenceStateTable.java
index 1660f7bd..f5611fe5 100644
--- a/src/main/java/ca/spottedleaf/moonrise/patches/blockstate_propertyaccess/util/ZeroCollidingReferenceStateTable.java
+++ b/src/main/java/ca/spottedleaf/moonrise/patches/blockstate_propertyaccess/util/ZeroCollidingReferenceStateTable.java
@@ -78,7 +78,8 @@ public void loadInTable(final Map<Map<Property<?>, Comparable<?>>, S> universe)
 
         this.lookup = (S[])new StateHolder[universe.size()];
 
-        for (final S value : universe.values()) {
+        for (final Map.Entry<Map<Property<?>, Comparable<?>>, S> entry : universe.entrySet()) {
+            final S value = entry.getValue();
             if (value == null) {
                 continue;
             }