nothumanallowed 13.5.177 → 13.5.179

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nothumanallowed",
-  "version": "13.5.177",
+  "version": "13.5.179",
   "description": "NotHumanAllowed — 38 AI agents, 80 tools, Studio (visual agentic workflows). Email, calendar, browser automation, screen capture, canvas, cron/heartbeat, Alexandria E2E messaging, GitHub, Notion, Slack, voice chat, free AI (Liara), 28 languages. Zero-dependency CLI.",
   "type": "module",
   "bin": {

package/src/commands/ui.mjs

@@ -696,7 +696,7 @@ export async function cmdUI(args) {
 
       // GET /api/weather?location=<city> — live weather via wttr.in (no API key)
       if (method === 'GET' && pathname === '/api/weather') {
-        const loc = (urlObj.searchParams.get('location') || config.location || '').trim();
+        const loc = (url.searchParams.get('location') || config.location || '').trim();
         if (!loc) { sendJSON(res, 400, { error: 'location required' }); return; }
         try {
           const wttrRes = await fetch(`https://wttr.in/${encodeURIComponent(loc)}?format=j1`, {
@@ -715,8 +715,12 @@ export async function cmdUI(args) {
             desc: d.hourly?.[4]?.weatherDesc?.[0]?.value || '',
             rain: d.hourly?.[4]?.chanceofrain || '0',
           }));
+          // Use the user-provided location as display name — wttr.in nearest_area
+          // often resolves to a nearby suburb (e.g. "Modena" → "Sant'Agnese") which
+          // is confusing. The weather data itself is correct regardless.
+          const displayCity = loc.includes(',') ? loc.split(',')[0].trim() : loc;
           sendJSON(res, 200, {
-            city: area?.areaName?.[0]?.value || loc,
+            city: displayCity,
             country: area?.country?.[0]?.value || '',
             tempC: cur.temp_C,
             feelsC: cur.FeelsLikeC,

package/src/constants.mjs

@@ -5,7 +5,7 @@ import { fileURLToPath } from 'url';
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
-export const VERSION = '13.5.177';
+export const VERSION = '13.5.179';
 export const BASE_URL = 'https://nothumanallowed.com/cli';
 export const API_BASE = 'https://nothumanallowed.com/api/v1';
 

package/src/services/web-ui.mjs

@@ -287,33 +287,37 @@ function apiPatch(p){return fetch(API+p,{method:'PATCH'}).then(function(r){retur
 
 // ---- LOAD DATA ----
 function loadWeather(){
-  // Try browser geolocation first, fall back to IP geolocation, then saved config
   var savedLoc=localStorage.getItem('nha_weather_location');
   function fetchWeather(loc){
     apiGet('/api/weather?location='+encodeURIComponent(loc)).then(function(r){
       if(r&&r.tempC){dash.weather=r;dashLoaded.weather=true;if(currentView==='dashboard')render();}
-    }).catch(function(){});
+      else{dashLoaded.weather=true;if(currentView==='dashboard')render();}
+    }).catch(function(){dashLoaded.weather=true;if(currentView==='dashboard')render();});
+  }
+  function ipFallback(){
+    fetch('https://ipapi.co/json/').then(function(r){return r.json();}).then(function(d){
+      var city=d.city||'';
+      if(city){localStorage.setItem('nha_weather_location',city);fetchWeather(city);}
+      else{dashLoaded.weather=true;render();}
+    }).catch(function(){dashLoaded.weather=true;render();});
   }
+  // If location already known, use it directly
   if(savedLoc){fetchWeather(savedLoc);return;}
+  // Try browser geolocation — pass lat,lng directly to wttr.in (no reverse geocoding needed)
   if(navigator.geolocation){
     navigator.geolocation.getCurrentPosition(function(pos){
-      // Reverse geocode via nominatim (free, no key)
-      fetch('https://nominatim.openstreetmap.org/reverse?lat='+pos.coords.latitude+'&lon='+pos.coords.longitude+'&format=json',{headers:{'User-Agent':'nha-cli/1.0'}})
-        .then(function(r){return r.json();})
-        .then(function(d){
-          var city=d.address&&(d.address.city||d.address.town||d.address.village||d.address.county||'');
-          if(city){localStorage.setItem('nha_weather_location',city);fetchWeather(city);}
+      var latLng=pos.coords.latitude.toFixed(4)+','+pos.coords.longitude.toFixed(4);
+      fetchWeather(latLng);
+      // Also resolve city name for display — save for next time
+      fetch('https://nominatim.openstreetmap.org/reverse?lat='+pos.coords.latitude+'&lon='+pos.coords.longitude+'&format=json&accept-language=en&zoom=10')
+        .then(function(r){return r.json();}).then(function(d){
+          // zoom=10 = city level — avoids resolving to suburbs/villages
+          var city=d.address&&(d.address.city||d.address.town||d.address.county||'');
+          if(city)localStorage.setItem('nha_weather_location',city);
         }).catch(function(){});
-    },function(){
-      // Geoloc denied — IP fallback
-      fetch('https://ipapi.co/json/').then(function(r){return r.json();}).then(function(d){
-        var city=d.city||'';if(city){localStorage.setItem('nha_weather_location',city);fetchWeather(city);}
-      }).catch(function(){});
-    },{timeout:5000});
+    },function(){ipFallback();},{timeout:6000,maximumAge:300000});
   } else {
-    fetch('https://ipapi.co/json/').then(function(r){return r.json();}).then(function(d){
-      var city=d.city||'';if(city){localStorage.setItem('nha_weather_location',city);fetchWeather(city);}
-    }).catch(function(){});
+    ipFallback();
   }
 }