From 2776a5c1b208e3b5189d2fe6cf58762160664a05 Mon Sep 17 00:00:00 2001
From: pkuzic <p.kuzickins@gmail.com>
Date: Sat, 26 Sep 2026 16:21:45 +0100
Subject: [PATCH] MONKEY (fix-wind): log the live foliageWind tier; test the
 player-path rows

The review report (grass and tree sway not moving in the play build) did not
reproduce: the release/play exe publishes grass and tree strength 1 with
foliageWind absent from the config, and capture pairs show sway. The tier is
now logged at boot and on every change so a player run shows what the
receivers were given, and a test holds the no-pin default path: moving
travel, grass and tree rows set, tier 0 clearing them.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
---
 crates/benilla-world/src/wind/mod.rs | 42 ++++++++++++++++++++++++++++
 1 file changed, 42 insertions(+)

diff --git a/crates/benilla-world/src/wind/mod.rs b/crates/benilla-world/src/wind/mod.rs
index 668e0b5f..71fe4b45 100644
--- a/crates/benilla-world/src/wind/mod.rs
+++ b/crates/benilla-world/src/wind/mod.rs
@@ -190,7 +190,19 @@ fn update_wind(
     mut frame: ResMut<MonkeyFrame>,
     mut clock: Local<WindClock>,
     mut offset: Local<Option<f64>>,
+    mut logged_tier: Local<Option<u8>>,
 ) {
+    // MONKEY (fix-wind): name the live tier in the log (boot and every change), so a player run
+    // shows what the foliage receivers were given.
+    if *logged_tier != Some(quality.0) {
+        *logged_tier = Some(quality.0);
+        info!(
+            "MONKEY wind: foliageWind tier {} (grass {}, trees {})",
+            quality.0,
+            quality.0 >= 1,
+            quality.0 >= 2
+        );
+    }
     let offset = *offset.get_or_insert_with(capture_wind_offset);
     let seconds = time.elapsed_secs_f64() + offset;
     let storm = storm_blend(weather.sky_density);
@@ -331,6 +343,36 @@ mod tests {
         }
     }
 
+    /// MONKEY (fix-wind): the player path — no capture pin, the registered tier — publishes live
+    /// grass and tree rows every frame, and the travel keeps moving; tier 0 clears them.
+    #[test]
+    fn default_tier_publishes_moving_grass_and_tree_rows() {
+        use bevy::ecs::system::RunSystemOnce;
+        use std::time::Duration;
+        assert_eq!(FoliageWind::REGISTERED, 2);
+        let mut world = World::new();
+        world.insert_resource(Time::<()>::default());
+        world.insert_resource(WeatherState::default());
+        world.insert_resource(FoliageWind(FoliageWind::REGISTERED));
+        world.insert_resource(WindField::default());
+        world.insert_resource(MonkeyFrame::default());
+        let mut travel = Vec::new();
+        for _ in 0..3 {
+            world
+                .resource_mut::<Time>()
+                .advance_by(Duration::from_millis(500));
+            world.run_system_once(update_wind).unwrap();
+            let rows = world.resource::<MonkeyFrame>().pack(0.5, 0.0);
+            assert_eq!(rows[5][1..], [1.0, 1.0, 1.0], "sway, grass, tree strength");
+            travel.push(rows[5][0]);
+        }
+        assert!(travel[0] < travel[1] && travel[1] < travel[2], "{travel:?}");
+        world.insert_resource(FoliageWind(0));
+        world.run_system_once(update_wind).unwrap();
+        let rows = world.resource::<MonkeyFrame>().pack(0.5, 0.0);
+        assert_eq!(rows[5][1..], [0.0, 0.0, 0.0]);
+    }
+
     #[test]
     fn travel_rate_stays_bounded_late_in_a_session() {
         // MONKEY (fix-wind): the published travel advances at the wind speed, never speed x age.