Bug Fix Waypoints

Author: cev-api

src/main/java/net/wurstclient/hacks/WaypointsHack.java

@@ -67,6 +67,7 @@ public final class WaypointsHack extends Hack
 	private static final Pattern END_PORTAL_NAME_PATTERN =
 		Pattern.compile("^End Portal (\\d+)$", Pattern.CASE_INSENSITIVE);
 	private static final long PORTAL_RECORD_COOLDOWN_MS = 2000L;
+	private static final long OCCLUSION_CACHE_MS = 250L;
 	private static final String[] DEATH_MESSAGE_PHRASES =
 		{"was slain", "was shot", "was blown", "was killed", "was fireballed",
 			"was roasted", "was doomed", "was squashed", "was pummeled",
@@ -93,6 +94,7 @@ public final class WaypointsHack extends Hack
 	private final Set<UUID> knownDead = new HashSet<>();
 	// Guard to avoid handling our own injected chat messages
 	private boolean sendingOwnChat = false;
+	private final Map<UUID, OcclusionSample> occlusionCache = new HashMap<>();
 
 	private final SliderSetting waypointRenderDistance = new SliderSetting(
 		"Waypoint render distance", 127, 0, DISTANCE_SLIDER_INFINITE, 1,
@@ -232,6 +234,7 @@ protected void onEnable()
 		EVENTS.add(GUIRenderListener.class, this);
 		otherDeathCooldown.clear();
 		knownDead.clear();
+		occlusionCache.clear();
 	}
 
 	@Override
@@ -245,6 +248,7 @@ protected void onDisable()
 		EVENTS.remove(GUIRenderListener.class, this);
 		otherDeathCooldown.clear();
 		knownDead.clear();
+		occlusionCache.clear();
 	}
 
 	// Add a temporary waypoint that should not be persisted. Returns the
@@ -775,6 +779,13 @@ public void onRender(PoseStack matrices, float partialTicks)
 			return;
 
 		var list = new ArrayList<>(manager.all());
+		long nowMs = System.currentTimeMillis();
+		double configuredLabelRange = waypointRenderDistance.getValue();
+		boolean configuredInfiniteLabels =
+			configuredLabelRange >= DISTANCE_SLIDER_INFINITE;
+		double occlusionMaxDistSq =
+			configuredInfiniteLabels ? Double.POSITIVE_INFINITY
+				: Math.max(0.0, configuredLabelRange * configuredLabelRange);
 		double beaconRange = beaconRenderDistance.getValue();
 		boolean beaconInfinite = beaconRange >= DISTANCE_SLIDER_INFINITE;
 		boolean beaconsEnabled = beaconInfinite || beaconRange > 0.0;
@@ -792,20 +803,7 @@ public void onRender(PoseStack matrices, float partialTicks)
 
 			double distSq = MC.player.distanceToSqr(wp.getX() + 0.5,
 				wp.getY() + 0.5, wp.getZ() + 0.5);
-			boolean applyObscuredRules =
-				dimObscuredOnScreenWaypoints.isChecked()
-					|| iconOnlyForObscuredOnScreenWaypoints.isChecked();
-			boolean obscured = applyObscuredRules && isWaypointObscured(wp);
-			boolean directlyLooked =
-				obscured && isDirectlyLookingAtWaypoint(wp, 5.0);
-			boolean suppressDetails = obscured && !directlyLooked
-				&& iconOnlyForObscuredOnScreenWaypoints.isChecked();
 			int waypointColor = applyFade(w.getColor(), distSq);
-			if(obscured && !directlyLooked
-				&& dimObscuredOnScreenWaypoints.isChecked())
-			{
-				waypointColor = dimForObscured(waypointColor);
-			}
 			double dist = Math.sqrt(distSq);
 			double trd = waypointRenderDistance.getValue();
 			boolean infiniteLabels = trd >= DISTANCE_SLIDER_INFINITE;
@@ -858,6 +856,24 @@ public void onRender(PoseStack matrices, float partialTicks)
 
 			if(renderLabel)
 			{
+				boolean applyObscuredRules =
+					dimObscuredOnScreenWaypoints.isChecked()
+						|| iconOnlyForObscuredOnScreenWaypoints.isChecked();
+				int labelColor = waypointColor;
+				boolean suppressDetails = false;
+				if(applyObscuredRules)
+				{
+					boolean directlyLooked =
+						isDirectlyLookingAtWaypoint(wp, 5.0);
+					boolean obscured =
+						!directlyLooked && isWaypointObscuredCached(w, wp,
+							distSq, occlusionMaxDistSq, nowMs);
+					suppressDetails = obscured
+						&& iconOnlyForObscuredOnScreenWaypoints.isChecked();
+					if(obscured && dimObscuredOnScreenWaypoints.isChecked())
+						labelColor = dimForObscured(labelColor);
+				}
+				
 				String title = w.getName() == null ? "" : w.getName();
 				String icon = iconChar(w.getIcon());
 				if(suppressDetails)
@@ -944,11 +960,11 @@ public void onRender(PoseStack matrices, float partialTicks)
 				}
 				// Keep a constant 10px separation using local pixel offset
 				float sepPx = 10.0f;
-				drawWorldLabel(matrices, title, lx, ly, lz, waypointColor,
-					scale, -sepPx);
+				drawWorldLabel(matrices, title, lx, ly, lz, labelColor, scale,
+					-sepPx);
 				if(!suppressDetails)
 					drawWorldLabel(matrices, distanceText, lx, ly, lz,
-						waypointColor, (float)(scale * 0.9f), 0f);
+						labelColor, (float)(scale * 0.9f), 0f);
 			}
 		}
 	}
@@ -1650,13 +1666,40 @@ private boolean isWaypointObscured(BlockPos waypointPos)
 	{
 		if(MC.player == null || MC.level == null || waypointPos == null)
 			return false;
+			
+		// If the waypoint's chunk isn't loaded, LOS is unknown. Treat it as
+		// obscured to avoid far waypoints popping back to full text.
+		if(!MC.level.hasChunkAt(waypointPos))
+			return true;
 
 		Vec3 eyes = MC.player.getEyePosition(1.0F);
 		Vec3 target = new Vec3(waypointPos.getX() + 0.5,
 			waypointPos.getY() + 1.2, waypointPos.getZ() + 0.5);
 		return !BlockUtils.hasLineOfSight(eyes, target);
 	}
 
+	private boolean isWaypointObscuredCached(Waypoint waypoint,
+		BlockPos waypointPos, double distSq, double maxOcclusionDistSq,
+		long nowMs)
+	{
+		if(waypoint == null || waypointPos == null)
+			return false;
+		
+		// Respect user's waypoint render distance setting.
+		if(distSq > maxOcclusionDistSq)
+			return false;
+		
+		OcclusionSample cached = occlusionCache.get(waypoint.getUuid());
+		if(cached != null && cached.pos.equals(waypointPos)
+			&& nowMs - cached.timestampMs <= OCCLUSION_CACHE_MS)
+			return cached.obscured;
+		
+		boolean obscured = isWaypointObscured(waypointPos);
+		occlusionCache.put(waypoint.getUuid(),
+			new OcclusionSample(waypointPos.immutable(), obscured, nowMs));
+		return obscured;
+	}
+	
 	private boolean isDirectlyLookingAtWaypoint(BlockPos waypointPos,
 		double maxAngleDeg)
 	{
@@ -1702,6 +1745,20 @@ private static final class WaypointEntry
 		}
 	}
 
+	private static final class OcclusionSample
+	{
+		final BlockPos pos;
+		final boolean obscured;
+		final long timestampMs;
+		
+		OcclusionSample(BlockPos pos, boolean obscured, long timestampMs)
+		{
+			this.pos = pos;
+			this.obscured = obscured;
+			this.timestampMs = timestampMs;
+		}
+	}
+	
 	private enum PortalKind
 	{
 		NETHER,