Reduce logs slightly

Author: mezz

Library/src/main/java/mezz/jei/library/ingredients/IngredientManager.java

@@ -106,7 +106,7 @@ public <V> void addIngredientsAtRuntime(IIngredientType<V> ingredientType, Colle
 			String ingredientStrings = ingredients.stream()
 				.map(ingredientHelper::getResourceLocation)
 				.map(ResourceLocation::toString)
-				.collect(Collectors.joining("\n", "[","]"));
+				.collect(Collectors.joining(", ", "[","]"));
 			LOGGER.debug("Ingredients added at runtime: {}", ingredientStrings);
 		}
 

Library/src/main/java/mezz/jei/library/plugins/vanilla/ingredients/ItemStackListFactory.java

@@ -14,6 +14,7 @@
 import net.minecraft.core.Holder;
 import net.minecraft.core.RegistryAccess;
 import net.minecraft.core.registries.Registries;
+import net.minecraft.network.chat.contents.TranslatableContents;
 import net.minecraft.world.entity.player.Player;
 import net.minecraft.world.flag.FeatureFlagSet;
 import net.minecraft.world.item.CreativeModeTab;
@@ -104,7 +105,8 @@ public static List<ItemStack> create(StackHelper stackHelper, ItemStackHelper it
 			}
 
 			if (displayItems.isEmpty() && searchTabDisplayItems.isEmpty()) {
-				LOGGER.warn(
+				Level logLevel = isKnownEmptyTab(tab) ? Level.DEBUG : Level.WARN;
+				LOGGER.log(logLevel,
 					"Item Group has no display items and no search tab display items. " +
 					"Items from this group will be missing from the JEI ingredient list. {}",
 					tab.getDisplayName().getString()
@@ -141,6 +143,11 @@ public static List<ItemStack> create(StackHelper stackHelper, ItemStackHelper it
 		return itemList;
 	}
 
+	private static boolean isKnownEmptyTab(CreativeModeTab tab) {
+		return tab.getDisplayName().getContents() instanceof TranslatableContents translatableContents &&
+			translatableContents.getKey().equals("itemGroup.op");
+	}
+
 	private static void addFromTab(
 		Collection<ItemStack> tabDisplayItems,
 		String displayType,

NeoForge/src/main/java/mezz/jei/neoforge/startup/StartEventObserver.java

@@ -58,7 +58,7 @@ public void register(PermanentEventSubscriptions subscriptions) {
 
 		subscriptions.register(ClientPlayerNetworkEvent.LoggingOut.class, event -> {
 			if (event.getPlayer() != null) {
-				LOGGER.info("JEI StartEventObserver received {}", event.getClass());
+				logReceivedEvent(event);
 				transitionState(State.LISTENING);
 			}
 		});
@@ -80,7 +80,7 @@ public void register(PermanentEventSubscriptions subscriptions) {
 
 					LOGGER.error("""
 							A Screen is opening but JEI hasn't started yet.
-							Normally, JEI is started after {}.
+							Normally, JEI is started after these event have fired: {}.
 							Something has caused one or more of these events to fail, so JEI is starting very late.
 							Missing events: {}""",
 						requiredEventsString,
@@ -109,7 +109,7 @@ private <T extends Event> void onEvent(T event) {
 			LOGGER.info("JEI StartEventObserver received {} too early, ignoring", event.getClass());
 			return;
 		}
-		LOGGER.info("JEI StartEventObserver received {}", event.getClass());
+		logReceivedEvent(event);
 		Class<? extends Event> eventClass = event.getClass();
 		if (requiredEvents.contains(eventClass) &&
 			observedEvents.add(eventClass) &&
@@ -123,6 +123,10 @@ private <T extends Event> void onEvent(T event) {
 		}
 	}
 
+	private static <T extends Event> void logReceivedEvent(T event) {
+		LOGGER.debug("JEI StartEventObserver received event: {}", event.getClass());
+	}
+
 	@Nullable
 	private static Connection getCurrentConnection() {
 		Minecraft minecraft = Minecraft.getInstance();