Fixed majority of bad suggestions

cogs/timings_check.yml

@@ -14,22 +14,26 @@ servers:
     Consider using [Purpur](https://ci.pl3x.net/job/Purpur/), [Airplane](https://ci.tivy.ca/), or [Airplane-Purpur](https://ci.tivy.ca/view/Purpur/) for the closest replacements.
 plugins:
   paper:
+    LagAssist:
+      prefix: "✅"
+      value: |-
+        Thank you for choosing LagAssist as part of your plugins. We highly recommend configuring the plugin, as most benefits that LagAssist provides are disabled by default.
     ClearLag:
-      prefix: "❌"
+      prefix: "⚠️"
       value: |-
-        Plugins that claim to remove lag actually cause more lag. 
+        The auto-removal feature of ClearLag can actually cause more lag, especially when used to clear mobs that will try to re-spawn once they are cleared. Removing projectiles is unnecessary and may break minigames, so please remove that from your config. Clearing items, primed-tnt, falling-blocks, and experience orbs may cause player confusion and anger. Please disable. 
     NoMobLag:
       prefix: "❌"
       value: |-
-        Plugins that claim to remove lag actually cause more lag.
+        The plugin is currently abandoned. We recommend looking into alternatives.
     ServerBooster:
-      prefix: "❌"
+      prefix: "⚠️"
       value: |-
-        Plugins that claim to remove lag actually cause more lag. 
+        Disable automatic chunk optimization as it may cause more bad than good. Mob limiter can actually be useful, so look into it. Take care with it's custom AI behaviour changer as it's not something a plugin should handle (use Airplane or other spigot forks that do that instead)
     AntiLag:
       prefix: "❌"
       value: |-
-        Plugins that claim to remove lag actually cause more lag. 
+        Completely redundant plugin. It's item removal feature is already handled better, and more efficiently by spigot (no plugins needed), without annoying players in the process.
     BookLimiter:
       prefix: "❌"
       value: |-
@@ -43,34 +47,34 @@ plugins:
       value: |-
         VillagerOptimiser is not useful in 1.15 and above.
     StackMob:
-      prefix: "❌"
+      prefix: "⚠️"
       value: |-
-        Stacking mobs causes more lag.
+        Stacking mobs can often cause more lag than good, as it will prompt more mobs to spawn. Please couple with an entity limiter. Additionally, traditional stacker plugins use an unefficient stacking algorithm that can cause performance issues by itself.
     Stacker:
       prefix: "❌"
       value: |-
-        Stacking mobs causes more lag.
+        Stacking mobs can often cause more lag than good, as it will prompt more mobs to spawn. Please couple with an entity limiter. Additionally, traditional stacker plugins use an unefficient stacking algorithm that can cause performance issues by itself.
     MobStacker:
       prefix: "❌"
       value: |-
-        Stacking mobs causes more lag.
+        Stacking mobs can often cause more lag than good, as it will prompt more mobs to spawn. Please couple with an entity limiter. Additionally, traditional stacker plugins use an unefficient stacking algorithm that can cause performance issues by itself.
     WildStacker:
       prefix: "❌"
       value: |-
-        Stacking mobs causes more lag.
+        Stacking mobs can often cause more lag than good, as it will prompt more mobs to spawn. Please couple with an entity limiter. Additionally, traditional stacker plugins use an unefficient stacking algorithm that can cause performance issues by itself.
     FastAsyncWorldEdit:
-      prefix: "❌"
+      prefix: "⚠️"
       value: |-
-        FAWE has been known to cause issues.
+        FAWE has been known to cause issues. However, it is way more efficient.
         Consider replacing FAWE with [Worldedit](https://enginehub.org/worldedit/#downloads).
     IllegalStack:
       prefix: "❌"
       value: |-
         You probably don't need IllegalStack as Paper already fixes all dupe and crash bugs.
     ExploitFixer:
-      prefix: "❌"
+      prefix: "⚠️"
       value: |-
-        You probably don't need ExploitFixer as Paper already fixes all dupe and crash bugs.
+        ExploitFixer is known not to tackle 0 days and as such is not extremely useful. Please use good server management policies, and other plugins for fixing 0 days.
     EntityTrackerFixer:
       prefix: "❌"
       value: |-
@@ -87,7 +91,7 @@ plugins:
     PermissionsEx:
       prefix: "❌"
       value: |-
-        PermissionsEx is an outdated permission plugin. 
+        PermissionsEx is an outdated permission plugin. It also has performance issues!
         Consider replacing it with [LuckPerms](https://luckperms.net/download).
     bPermissions:
       prefix: "❌"
@@ -138,14 +142,20 @@ config:
     - expressions:
       - int(server_properties["network-compression-threshold"]) <= 256
       - spigot["settings"]["bungeecord"] == "false"
-      prefix: "❌"
-      value: "Increase this in server.properties. Recommended: 512."
+      prefix: "⚠️"
+      value: "Network compression means the threshold at which the server will compress packets. Less means less CPU usage at the cost of network usage for both you and your player. Recommended: 256 (DEFAULT), 512 (BETTER CPU USAGE)"
+    network-compression-threshold-bungee:
+    - expressions:
+      - int(server_properties["network-compression-threshold"]) <= 0
+      - spigot["settings"]["bungeecord"] == "true"
+      prefix: "⚠️"
+      value: "Despite community consensus, disabling network compression on bungee can cause major performance issues, especially when gaining many players on multiple dedicated servers. Recommended: 512."
   bukkit:
     chunk-gc.period-in-ticks:
     - expressions:
       - int(bukkit["chunk-gc"]["period-in-ticks"]) >= 600
       prefix: "❌"
-      value: "Decrease this in bukkit.yml.\nRecommended: 400."
+      value: "Decrease this in bukkit.yml.\nRecommended: 300."
     ticks-per.monster-spawns:
     - expressions:
       - int(bukkit["ticks-per"]["monster-spawns"]) == 1
@@ -261,7 +271,7 @@ config:
     - expressions:
       - int(spigot["world-settings"]["default"]["arrow-despawn-rate"]) >= 1200
       prefix: "❌"
-      value: "Decrease this in spigot.yml.\nRecommended: 300."
+      value: "Decrease this in spigot.yml.\nRecommended: 1200."
     merge-radius.item:
     - expressions:
       - float(spigot["world-settings"]["default"]["merge-radius"]["item"]) <= 2.5
@@ -373,12 +383,12 @@ config:
       - '"BlockBalls" not in plugins'
       - '"ArmorStandTools" not in plugins'
       prefix: "❌"
-      value: "Disable this in paper.yml."
+      value: "Armor stands rarely need ticking. It's recommended to disable it in paper.yml."
     per-player-mob-spawns:
     - expressions:
       - paper["world-settings"]["default"]["per-player-mob-spawns"] == "false"
-      prefix: "❌"
-      value: "Enable this in paper.yml."
+      prefix: "⚠️"
+      value: "This makes mob spawning configuration more intuitive."
     alt-item-despawn-rate.enabled:
     - expressions:
       - paper["world-settings"]["default"]["alt-item-despawn-rate"]["enabled"] == "false"