fix: Weather connector output + debug tracing

- Weather context now shows city name header, fetch timestamp, hourly forecast
- Added city config field to Weather connector registry
- Debug logging in getActiveContext() traces enabled connectors and results
- Error catch now logs connector ID and error message

Author: ijbo

changelogs/CHANGELOG-weather-connector-fix.md

@@ -0,0 +1,39 @@
+# Weather Connector — Enhanced Output & Debug Tracing
+
+- Weather connector now outputs city name in header (`[Live Weather Report — Tokyo]`)
+- Added fetch timestamp to weather context for recency verification
+- Added hourly forecast (every 3 hours, next 24h) to weather context
+- Added `city` config field to Weather connector registry for user-customizable city label
+- Added debug logging to `getActiveContext()` — traces which connectors are enabled and their fetch results
+- Fixed: error catch in `getActiveContext` now logs failed connector ID and error message instead of silently swallowing
+
+---
+
+## Summary
+
+Improved the Weather connector's context output to be more explicit and model-friendly, added a city name config field, and added debug tracing to diagnose cases where weather data doesn't reach the AI model.
+
+---
+
+## 1. Weather Context Formatting
+**Files:** `js/connectors.js`
+**What:** Replaced terse `[Weather — Open-Meteo (lat:..., lon:...)]` header with `[Live Weather Report — Tokyo (via Open-Meteo API)]` plus fetch timestamp, location line, and hourly forecast block (8 data points at 3h intervals).
+**Impact:** Small models like Gemma 4 E2B can now find and reference weather data in multi-source context thanks to clearer headers.
+
+## 2. City Config Field
+**Files:** `js/connectors.js`
+**What:** Added `city` config field to the `openmeteo` registry entry. The value (default: `Tokyo`) appears in the weather report header. Users can customize this via the connector config modal.
+**Impact:** Weather reports are labeled with the user's city name, making the context more natural for AI grounding.
+
+## 3. Debug Tracing
+**Files:** `js/connectors.js`
+**What:** `getActiveContext()` now logs `[Connectors] Enabled connectors: [...]` and per-connector result previews (first 100 chars or NULL). Error catches log the connector ID and error message.
+**Impact:** Enables diagnosing "weather data missing from context" issues via browser console.
+
+---
+
+## Files Changed (1 total)
+
+| File | Lines Changed | Type |
+|------|:---:|------|
+| `js/connectors.js` | +34 −6 | Weather output enhancement, city field, debug logging |

js/connectors.js

@@ -147,6 +147,7 @@
             authType: 'none',
             badge: 'Free · No key',
             configFields: [
+                { key: 'city', label: 'City name', placeholder: 'Tokyo', hint: 'Name of your city (appears in weather report header)' },
                 { key: 'lat', label: 'Latitude', placeholder: '35.6762', hint: 'Your city latitude (e.g. Tokyo: 35.6762, NYC: 40.7128, London: 51.5074)' },
                 { key: 'lon', label: 'Longitude', placeholder: '139.6503', hint: 'Your city longitude (e.g. Tokyo: 139.6503, NYC: -74.0060, London: -0.1278)' }
             ],
@@ -422,6 +423,7 @@
     async function fetchWeatherContext(config) {
         var lat = parseFloat(config.lat) || 35.6762;  // default Tokyo
         var lon = parseFloat(config.lon) || 139.6503;
+        var city = config.city || 'Tokyo';
         try {
             var url = 'https://api.open-meteo.com/v1/forecast?latitude=' + lat + '&longitude=' + lon +
                 '&current=temperature_2m,relative_humidity_2m,apparent_temperature,precipitation,weather_code,wind_speed_10m' +
@@ -437,27 +439,46 @@
                 95:'Thunderstorm', 96:'Thunderstorm with hail', 99:'Thunderstorm with heavy hail'
             };
             var condition = wmo[cur.weather_code] || ('Code ' + cur.weather_code);
+            var fetchTime = new Date().toUTCString();
             var lines = [
-                '[Weather — Open-Meteo (lat:' + lat + ', lon:' + lon + ')]',
-                '  Condition: ' + condition,
+                '[Live Weather Report — ' + city + ' (via Open-Meteo API)]',
+                'Fetched: ' + fetchTime,
+                'Location: ' + city + ' (lat:' + lat + ', lon:' + lon + ')',
+                '',
+                'Current Conditions:',
+                '  Weather: ' + condition,
                 '  Temperature: ' + cur.temperature_2m + '°C (feels like ' + cur.apparent_temperature + '°C)',
                 '  Humidity: ' + cur.relative_humidity_2m + '%',
-                '  Wind: ' + cur.wind_speed_10m + ' km/h',
-                '  Precipitation: ' + cur.precipitation + ' mm'
+                '  Wind Speed: ' + cur.wind_speed_10m + ' km/h',
+                '  Precipitation: ' + cur.precipitation + ' mm',
             ];
+            // Add hourly forecast if available
+            if (data.hourly && data.hourly.temperature_2m) {
+                lines.push('');
+                lines.push('Hourly Forecast (next 24h):');
+                var hours = data.hourly.time || [];
+                var temps = data.hourly.temperature_2m || [];
+                for (var i = 0; i < Math.min(hours.length, 24); i += 3) {
+                    var hr = hours[i] ? hours[i].split('T')[1] || hours[i] : '??';
+                    lines.push('  ' + hr + ' → ' + temps[i] + '°C');
+                }
+            }
             return lines.join('\n');
         } catch (e) {
             return '[Weather] Failed to fetch: ' + e.message;
         }
     }
 
+
     // --- Get Combined Context from All Enabled Connectors ---
     async function getActiveContext(query) {
         var parts = [];
         var promises = [];
+        var enabledIds = [];
 
         Object.keys(REGISTRY).forEach(function (id) {
             if (!isEnabled(id)) return;
+            enabledIds.push(id);
             var token = getToken(id);
             var config = getConfig(id);
 
@@ -474,11 +495,18 @@
                 default: fetchPromise = Promise.resolve(null);
             }
 
-            promises.push(fetchPromise.catch(function (e) { return null; }));
+            promises.push(fetchPromise.catch(function (e) {
+                console.error('[Connectors] Fetch failed for', id, ':', e.message);
+                return null;
+            }));
         });
 
+        console.log('[Connectors] Enabled connectors:', enabledIds);
         var results = await Promise.all(promises);
-        results.forEach(function (r) { if (r) parts.push(r); });
+        results.forEach(function (r, i) {
+            console.log('[Connectors] Result', enabledIds[i], ':', r ? r.substring(0, 100) : 'NULL');
+            if (r) parts.push(r);
+        });
 
         if (parts.length === 0) return null;