Fix book screens opening on server thread

docs/PORTING.md

@@ -80,6 +80,10 @@ all changes to its destination and custom name. Descriptive Book Age references
 now carry a display name alongside their stable UUID, with backward-compatible
 decoding for existing UUID-only data.
 
+Version 0.1.5 ensures the book interaction listener opens screens only for the
+logical-client event. This avoids calling the render system from the integrated
+server thread when a book is used in singleplayer.
+
 The preview code is separated behind a renderer interface and uses an isolated
 Ponder level. The first iteration supplies the animated loading boundary; the
 next iteration will add server-authoritative snapshot transport and populate

gradle.properties

@@ -33,7 +33,7 @@ mod_name=Rivencraft
 # The license of the mod. Review your options at https://choosealicense.com/. All Rights Reserved is the default.
 mod_license=LGPL-3.0-or-later
 # The mod version. See https://semver.org/
-mod_version=0.1.4
+mod_version=0.1.5
 # The group ID for the mod. It is only important when publishing as an artifact to a Maven repository.
 # This should match the base package used for the mod sources.
 # See https://maven.apache.org/guides/mini/guide-naming-conventions.html

src/main/java/net/dnorberg/rivencraft/client/ClientBookInteractionHandler.java

@@ -12,6 +12,12 @@ public final class ClientBookInteractionHandler {
     }
 
     public static void onRightClickItem(PlayerInteractEvent.RightClickItem event) {
+        // NeoForge posts this common event on both logical sides in an
+        // integrated game. Screens may only be created by the client copy.
+        if (!event.getEntity().level().isClientSide()) {
+            return;
+        }
+
         boolean linkingBook = event.getItemStack().is(ModItems.LINKING_BOOK.get());
         boolean descriptiveBook = event.getItemStack().is(ModItems.DESCRIPTIVE_BOOK.get());
         if (!linkingBook && !descriptiveBook) {