[WC-3450] Complete LeafletMap MobX migration with ViewModel

[iobuhov]

Pull request type

Refactoring (e.g. file rename, variable rename, etc.)

---

Description

This PR completes the MobX migration for the LeafletMap component by:

Key Changes:

• ✨ Created LeafletMapViewModel to handle all map logic with MobX reactions
• 🔨 Simplified LeafletMap.tsx component (removed 184 lines) to focus on rendering
• 📦 Extracted marker creation logic into src/utils/leaflet-markers.ts
• 🏗️ Set up proper DI with useLeafletMapVM injection hook
• 🧪 Updated tests to work with new ViewModel architecture
• 🎨 Fixed all linting issues (import ordering, return types)

Architecture:

• ViewModel (LeafletMapViewModel): Manages map instance, tile layer, markers, and reactivity
• Component (LeafletMap): Pure presentational component using ref callback for setup
• Services: Integrated with LocationResolverService and CurrentLocationService
• Reactive updates: MobX reaction syncs markers when locations change

Testing:

• ✅ All 90 tests passing
• ✅ 87% code coverage maintained
• ✅ Linting clean (no errors or warnings)

Impact:

• 18 files changed
• 311 insertions(+), 284 deletions(-)
• Better separation of concerns
• More maintainable and testable code

This PR builds on the existing MobX infrastructure and completes the migration started in the parent branch.

---

What should be covered while testing?

1. Map Rendering: Verify the map renders correctly with markers
2. Marker Updates: Check that markers update reactively when data changes
3. Auto Zoom: Test automatic zoom behavior with multiple markers
4. Manual Zoom: Verify manual zoom controls work correctly
5. Current Location: Test current location marker appears and updates
6. Map Interactions: Drag, scroll, and zoom controls function properly
7. Different Providers: Test with both OpenStreetMap and other providers

Test in Mendix Studio Pro:

• Add Maps widget to a page with data source
• Verify markers appear at correct locations
• Test adding/removing markers dynamically
• Check zoom and pan behavior
• Verify no console errors or memory leaks

---

🤖 Generated with Claude Code

[r0b1n]

When expression is still loading, value is going to be undefined for a bit bfore taking final shape, it shouldn't fall back to apiKey when apiKeyExp is not undefined. The sign that expression is not configured is that whole apiKeyExp is undefined, not just value.