Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Greptile Summary

This PR fixes a critical race condition in the map viewport handling where repeatedly switching tabs with no GNSS or offline map caused the viewport to jump from (0,0) zoom 2.0 to a full-world default view. The fix prevents the onCameraIdle listener from persisting stale default viewport data before the positioning logic has a chance to run and establish an intentional camera position.

Key changes:

• isInitialPositionSet guard added to block premature viewport saves
• LaunchedEffect(mapLibreMap) resets the guard whenever the MapView is recreated
• All positioning paths explicitly set the guard to reflect whether a meaningful position was used
• clearCameraPosition() method clears the (0,0) fallback so it doesn't persist and block future GPS/default-region centering

Testing: The fix correctly handles tab switches, config changes, GNSS availability transitions, and offline map region changes without regressing the viewport behavior.

Confidence Score: 5/5

• Safe to merge. The fix correctly addresses the race condition with no functional issues or regressions.
• The PR successfully fixes the described race condition where onCameraIdle could save stale default viewport data before intentional positioning. The implementation correctly gates the save with isInitialPositionSet, resets the guard on MapView recreation via LaunchedEffect, and all positioning paths properly manage the guard state. The clearCameraPosition() addition correctly prevents (0,0) fallback from persisting. Code is straightforward, well-integrated, and handles all documented test scenarios.
• No files require special attention.

_{Last reviewed commit: ef405a1}