Update geocoding service to use Google Maps API and enhance permissions in settings

.claude/settings.local.json

@@ -4,7 +4,13 @@
       "WebSearch",
       "Bash(python app.py:*)",
       "WebFetch(domain:github.com)",
-      "Bash(pip show:*)"
+      "Bash(pip show:*)",
+      "Bash(python test_google_geocoding.py:*)",
+      "Bash(pip install:*)",
+      "Bash(find:*)",
+      "Bash(netstat:*)",
+      "Bash(findstr:*)",
+      "Bash(taskkill:*)"
     ],
     "deny": [],
     "ask": []

.env.example

@@ -5,8 +5,9 @@ AI_PROVIDER=anthropic
 ANTHROPIC_API_KEY=your_anthropic_api_key_here
 GOOGLE_API_KEY=your_google_api_key_here
 
-# HERE Maps API Key (for geocoding)
-HERE_API_KEY=your_here_api_key_here
+# Google Maps API Key (for geocoding)
+# Note: This can be the same as GOOGLE_API_KEY if Maps API is enabled
+GOOGLE_MAPS_API_KEY=your_google_maps_api_key_here
 
 # PostgreSQL Database Configuration
 DB_HOST=localhost

chat/geocoding.py

@@ -1,10 +1,10 @@
 """
 Geocoding service for FleetMind
-Handles address validation with HERE API and smart mock fallback
+Handles address validation with Google Maps API and smart mock fallback
 """
 
 import os
-import requests
+import googlemaps
 import logging
 from typing import Dict, Optional
 
@@ -36,16 +36,23 @@ CITY_COORDINATES = {
 
 
 class GeocodingService:
-    """Handle address geocoding with HERE API and smart mock fallback"""
+    """Handle address geocoding with Google Maps API and smart mock fallback"""
 
     def __init__(self):
-        self.here_api_key = os.getenv("HERE_API_KEY", "")
-        self.use_mock = not self.here_api_key or self.here_api_key.startswith("your_")
+        self.google_maps_key = os.getenv("GOOGLE_MAPS_API_KEY", "")
+        self.use_mock = not self.google_maps_key or self.google_maps_key.startswith("your_")
 
         if self.use_mock:
-            logger.info("Geocoding: Using mock (HERE_API_KEY not configured)")
+            logger.info("Geocoding: Using mock (GOOGLE_MAPS_API_KEY not configured)")
+            self.gmaps_client = None
         else:
-            logger.info("Geocoding: Using HERE Maps API")
+            try:
+                self.gmaps_client = googlemaps.Client(key=self.google_maps_key)
+                logger.info("Geocoding: Using Google Maps API")
+            except Exception as e:
+                logger.error(f"Failed to initialize Google Maps client: {e}")
+                self.use_mock = True
+                self.gmaps_client = None
 
     def geocode(self, address: str) -> Dict:
         """
@@ -61,40 +68,36 @@ class GeocodingService:
             return self._geocode_mock(address)
         else:
             try:
-                return self._geocode_here(address)
+                return self._geocode_google(address)
             except Exception as e:
-                logger.error(f"HERE API failed: {e}, falling back to mock")
+                logger.error(f"Google Maps API failed: {e}, falling back to mock")
                 return self._geocode_mock(address)
 
-    def _geocode_here(self, address: str) -> Dict:
-        """Real HERE API geocoding"""
-        url = "https://geocode.search.hereapi.com/v1/geocode"
-
-        params = {
-            "q": address,
-            "apiKey": self.here_api_key
-        }
-
-        response = requests.get(url, params=params, timeout=10)
-        response.raise_for_status()
+    def _geocode_google(self, address: str) -> Dict:
+        """Real Google Maps API geocoding"""
+        try:
+            # Call Google Maps Geocoding API
+            result = self.gmaps_client.geocode(address)
 
-        data = response.json()
+            if not result:
+                # No results found, fall back to mock
+                logger.warning(f"Google Maps API found no results for: {address}")
+                return self._geocode_mock(address)
 
-        if not data.get("items"):
-            # No results found, fall back to mock
-            logger.warning(f"HERE API found no results for: {address}")
-            return self._geocode_mock(address)
+            # Get first result
+            first_result = result[0]
+            location = first_result['geometry']['location']
 
-        # Get first result
-        item = data["items"][0]
-        position = item["position"]
+            return {
+                "lat": location['lat'],
+                "lng": location['lng'],
+                "formatted_address": first_result.get('formatted_address', address),
+                "confidence": "high (Google Maps API)"
+            }
 
-        return {
-            "lat": position["lat"],
-            "lng": position["lng"],
-            "formatted_address": item.get("address", {}).get("label", address),
-            "confidence": "high (HERE API)"
-        }
+        except Exception as e:
+            logger.error(f"Google Maps geocoding error: {e}")
+            raise
 
     def _geocode_mock(self, address: str) -> Dict:
         """
@@ -126,6 +129,6 @@ class GeocodingService:
     def get_status(self) -> str:
         """Get geocoding service status"""
         if self.use_mock:
-            return "⚠️ Using mock geocoding (add HERE_API_KEY for real)"
+            return "⚠️ Using mock geocoding (add GOOGLE_MAPS_API_KEY for real)"
         else:
-            return "✅ HERE Maps API connected"
+            return "✅ Google Maps API connected"

requirements.txt

@@ -15,6 +15,7 @@ psycopg2-binary>=2.9.9
 # API Clients
 requests>=2.31.0
 httpx>=0.27.1
+googlemaps>=4.10.0
 
 # Utilities
 python-dotenv>=1.0.0