Clearing a barbarian camp on a tile you own leaves the "improvement" behind about unciv HOT 6 OPEN

Index: core/src/com/unciv/logic/map/TileMap.kt
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/core/src/com/unciv/logic/map/TileMap.kt b/core/src/com/unciv/logic/map/TileMap.kt
--- a/core/src/com/unciv/logic/map/TileMap.kt	(revision 6f11f311a8fdf28f61d0fb06e3494a81fddabe97)
+++ b/core/src/com/unciv/logic/map/TileMap.kt	(date 1705647615821)
@@ -60,6 +60,7 @@
     /** Attention: lateinit will _stay uninitialized_ while in MapEditorScreen! */
     @Transient
     lateinit var gameInfo: GameInfo
+    fun hasGameInfo() = ::gameInfo.isInitialized
 
     /** Keep a copy of the [Ruleset] object passed to setTransients, for now only to allow subsequent setTransients without. Copied on [clone]. */
     @Transient
Index: core/src/com/unciv/models/ruleset/unique/UniqueTriggerActivation.kt
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/core/src/com/unciv/models/ruleset/unique/UniqueTriggerActivation.kt b/core/src/com/unciv/models/ruleset/unique/UniqueTriggerActivation.kt
--- a/core/src/com/unciv/models/ruleset/unique/UniqueTriggerActivation.kt	(revision 6f11f311a8fdf28f61d0fb06e3494a81fddabe97)
+++ b/core/src/com/unciv/models/ruleset/unique/UniqueTriggerActivation.kt	(date 1705646254220)
@@ -605,10 +605,7 @@
                 for (explorableTile in explorableTiles) {
                     explorableTile.setExplored(civInfo, true)
                     positions += explorableTile.position
-                    if (explorableTile.improvement == null)
-                        civInfo.lastSeenImprovement.remove(explorableTile.position)
-                    else
-                        civInfo.lastSeenImprovement[explorableTile.position] = explorableTile.improvement!!
+                    civInfo.cache.updateLastSeenImprovement(explorableTile)
                 }
 
                 if (notification != null) {
Index: core/src/com/unciv/logic/map/tile/TileInfoImprovementFunctions.kt
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/core/src/com/unciv/logic/map/tile/TileInfoImprovementFunctions.kt b/core/src/com/unciv/logic/map/tile/TileInfoImprovementFunctions.kt
--- a/core/src/com/unciv/logic/map/tile/TileInfoImprovementFunctions.kt	(revision 6f11f311a8fdf28f61d0fb06e3494a81fddabe97)
+++ b/core/src/com/unciv/logic/map/tile/TileInfoImprovementFunctions.kt	(date 1705647830324)
@@ -128,7 +128,7 @@
             // Can only remove roads if that road is actually there
             RoadStatus.values().any { it.removeAction == improvement.name } -> tile.roadStatus.removeAction == improvement.name
             // Can only remove features or improvement if that feature/improvement is actually there
-            improvement.name.startsWith(Constants.remove) -> tile.terrainFeatures.any { Constants.remove + it == improvement.name } 
+            improvement.name.startsWith(Constants.remove) -> tile.terrainFeatures.any { Constants.remove + it == improvement.name }
                 || Constants.remove + tile.improvement == improvement.name
             // Can only build roads if on land and they are better than the current road
             RoadStatus.values().any { it.name == improvement.name } -> !tile.isWater
@@ -207,26 +207,26 @@
             }
         }
 
-        if (improvementObject != null && improvementObject.hasUnique(UniqueType.RemovesFeaturesIfBuilt)) {
-            // Remove terrainFeatures that a Worker can remove
-            // and that aren't explicitly allowed under the improvement
-            val removableTerrainFeatures = tile.terrainFeatures.filter { feature ->
-                val removingAction = "${Constants.remove}$feature"
+        if (improvementObject != null) {
+            if (improvementObject.hasUnique(UniqueType.RemovesFeaturesIfBuilt)) {
+                // Remove terrainFeatures that a Worker can remove
+                // and that aren't explicitly allowed under the improvement
+                val removableTerrainFeatures = tile.terrainFeatures.filter { feature ->
+                    val removingAction = "${Constants.remove}$feature"
 
-                removingAction in tile.ruleset.tileImprovements // is removable
-                    && !improvementObject.isAllowedOnFeature(feature) // cannot coexist
-            }
+                    removingAction in tile.ruleset.tileImprovements // is removable
+                        && !improvementObject.isAllowedOnFeature(feature) // cannot coexist
+                }
 
-            tile.setTerrainFeatures(tile.terrainFeatures.filterNot { it in removableTerrainFeatures })
-        }
+                tile.setTerrainFeatures(tile.terrainFeatures.filterNot { it in removableTerrainFeatures })
+            }
 
-        if (civToActivateBroaderEffects != null && improvementObject != null
-            && improvementObject.hasUnique(UniqueType.TakesOverAdjacentTiles)
-        )
-            takeOverTilesAround(civToActivateBroaderEffects, tile)
+            if (civToActivateBroaderEffects != null && improvementObject.hasUnique(UniqueType.TakesOverAdjacentTiles))
+                takeOverTilesAround(civToActivateBroaderEffects, tile)
 
-        if (civToActivateBroaderEffects != null && improvementObject != null)
-            triggerImprovementUniques(improvementObject, civToActivateBroaderEffects, unit)
+            if (civToActivateBroaderEffects != null)
+                triggerImprovementUniques(improvementObject, civToActivateBroaderEffects, unit)
+        }
 
         val city = tile.owningCity
         if (city != null) {
@@ -236,6 +236,14 @@
                 city.reassignPopulationDeferred()
             }
         }
+
+        // Update seen improvement for all humans that can see the tile
+        if (!tile.tileMap.hasGameInfo()) return // Mapgen calls us
+        for (civ in tile.tileMap.gameInfo.civilizations) {
+            if (!civ.isHuman()) continue
+            if (tile !in civ.viewableTiles) continue
+            civ.cache.updateLastSeenImprovement(tile)
+        }
     }
 
     private fun triggerImprovementUniques(
@@ -249,7 +257,7 @@
             }
             else UniqueTriggerActivation.triggerCivwideUnique(unique, civ, tile = tile)
 
-        if (unit != null){
+        if (unit != null) {
             for (unique in unit.getTriggeredUniques(UniqueType.TriggerUponBuildingImprovement)
                 .filter { improvement.matchesFilter(it.params[0]) })
                 UniqueTriggerActivation.triggerUnitwideUnique(unique, unit)
Index: core/src/com/unciv/logic/civilization/transients/CivInfoTransientCache.kt
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/core/src/com/unciv/logic/civilization/transients/CivInfoTransientCache.kt b/core/src/com/unciv/logic/civilization/transients/CivInfoTransientCache.kt
--- a/core/src/com/unciv/logic/civilization/transients/CivInfoTransientCache.kt	(revision 6f11f311a8fdf28f61d0fb06e3494a81fddabe97)
+++ b/core/src/com/unciv/logic/civilization/transients/CivInfoTransientCache.kt	(date 1705646565473)
@@ -187,15 +187,22 @@
         civInfo.viewableTiles = newViewableTiles // to avoid concurrent modification problems
     }
 
-    private fun updateLastSeenImprovements() {
-        if (civInfo.playerType == PlayerType.AI) return // don't bother for AI, they don't really use the info anyway
-
-        for (tile in civInfo.viewableTiles) {
-            if (tile.improvement == null)
-                civInfo.lastSeenImprovement.remove(tile.position)
-            else
-                civInfo.lastSeenImprovement[tile.position] = tile.improvement!!
-        }
+    /** This updates what improvement this civ sees on [tile]
+     *  ...unless the civ is an AI (this is purely visual, the AI will only use the real info)
+     *
+     *  Does **not** include the visibility check - that _must_ be ensured by the caller.
+     *  Mutates [Civilization.lastSeenImprovement].
+     */
+    fun updateLastSeenImprovement(tile: Tile) {
+        if (civInfo.playerType == PlayerType.AI) return
+        if (tile.improvement == null)
+            civInfo.lastSeenImprovement.remove(tile.position)
+        else
+            civInfo.lastSeenImprovement[tile.position] = tile.improvement!!
+    }
+
+    private fun updateLastSeenImprovements() {
+        for (tile in civInfo.viewableTiles) updateLastSeenImprovement(tile)
     }
 
     private fun discoverNaturalWonders() {