@tiens.nguyen/gonext-local-worker 1.0.74 → 1.0.76

package/gonext_agent_chat.py

@@ -117,9 +117,31 @@ def _summarise_step(step_log):
     return label + (" | ".join(parts) if parts else "thinking…")
 
 
+# Keywords that strongly indicate the user wants to make an HTTP/network request,
+# regardless of what the final output is (time, text, data, etc.).
+_AGENT_KEYWORDS = re.compile(
+    r"\b("
+    r"request|fetch|call|hit|ping|curl|wget|GET|POST|PUT|DELETE|PATCH"
+    r"|api|endpoint|url|http|https"
+    r"|external\s+source|external\s+api|external\s+service"
+    r"|web\s+service|rest\s+api|rest\s+call"
+    r"|download|scrape|crawl"
+    r")\b",
+    re.IGNORECASE,
+)
+
+
 def _route(task_text: str, base_url: str, api_key: str, model_id: str) -> bool:
-    """Ask the local model if this task needs an HTTP request.
-    Returns True → run full agent; False → plain chat reply."""
+    """Decide if the task needs the HTTP agent (True) or a plain chat reply (False).
+
+    Fast-path: if the user explicitly mentions network/request keywords → agent.
+    Otherwise: ask the model to classify.
+    """
+    # Fast-path: explicit HTTP/network intent overrides the model classifier.
+    if _AGENT_KEYWORDS.search(task_text):
+        _log(f"router → YES (keyword match)")
+        return True
+
     try:
         from openai import OpenAI
         client = OpenAI(base_url=base_url, api_key=api_key or "local",
@@ -128,10 +150,13 @@ def _route(task_text: str, base_url: str, api_key: str, model_id: str) -> bool:
             model=model_id,
             messages=[
                 {"role": "system", "content": (
-                    "You are a task classifier. Reply with YES or NO only, no punctuation."
+                    "You are a task classifier. Reply YES or NO only, no punctuation.\n"
+                    "Answer YES if the task requires fetching data from an external network source "
+                    "(URL, API, website, web service, or any remote server).\n"
+                    "Answer NO if it can be solved entirely with Python stdlib, math, or is just conversation."
                 )},
                 {"role": "user", "content": (
-                    f"Does this task require making an HTTP request to a URL or API?\n\n"
+                    f"Does this task require fetching data from an external network source?\n\n"
                     f"Task: {task_text}\n\nYES or NO:"
                 )},
             ],
@@ -139,7 +164,7 @@ def _route(task_text: str, base_url: str, api_key: str, model_id: str) -> bool:
             temperature=0,
         )
         answer = (resp.choices[0].message.content or "").strip().upper()
-        _log(f"router → {answer!r}")
+        _log(f"router → {answer!r} (model)")
         return answer.startswith("Y")
     except Exception as e:  # noqa: BLE001
         _log(f"router error: {e} — defaulting to agent")
@@ -246,10 +271,17 @@ def run_agent_chat(cfg):
         "You have ONE built-in function: `http_request(method, url, headers='', body='')`. "
         "It returns a STRING with the HTTP status and body. "
         "Call it, then immediately call `final_answer(response)` inside the same code block.\n"
-        "Do NOT put final_answer outside the code block.\n\n"
+        "IMPORTANT RULES:\n"
+        "- Python's `datetime` module is available — use it for date/time tasks, no HTTP needed.\n"
+        "- If http_request returns an error starting with 'Error:', the URL failed. "
+        "Do NOT retry the same URL. Try a DIFFERENT URL or API, or use Python stdlib.\n"
+        "- Do NOT put final_answer outside the code block.\n\n"
     )
     task_with_hint = tool_hint + "Task: " + task_text
 
+    # Track URLs that have already failed so we don't retry dead endpoints across steps.
+    _failed_urls: set = set()
+
     @tool
     def http_request(method: str, url: str, headers: str = "", body: str = "") -> str:
         """Perform an HTTP request and return the status code and body preview.
@@ -266,11 +298,25 @@ def run_agent_chat(cfg):
                 parsed_headers = json.loads(headers)
             except Exception:  # noqa: BLE001
                 pass
-        # Retry once on timeout — gorok tunnels can be flaky on the first attempt.
+        url_key = f"{method.upper()}:{url}"
+        if url_key in _failed_urls:
+            msg = f"Error: {url} already failed — try a different URL or use Python stdlib."
+            _emit({"type": "step", "text": f"HTTP {method.upper()} {url} → (skipped, already failed)"})
+            _log(f"http_request skipped (already failed): {url_key}")
+            return msg
         result = _http_request_impl(method, url, parsed_headers, body or None)
         if result.startswith("Error:"):
+            # Retry once for flaky connections (e.g. gorok tunnels).
             _log(f"http_request retry {method.upper()} {url}")
             result = _http_request_impl(method, url, parsed_headers, body or None)
+        if result.startswith("Error:"):
+            # Both attempts failed — mark URL as dead so model tries something else.
+            _failed_urls.add(url_key)
+            result = (
+                f"{result}\n"
+                "Note: This URL failed twice. Do NOT retry it. "
+                "Try a DIFFERENT URL or use Python's datetime/math/etc. module instead."
+            )
         status_line = result.split("\n")[0][:150] if result else "no response"
         _emit({"type": "step", "text": f"HTTP {method.upper()} {url} → {status_line}"})
         _log(f"http_request {method.upper()} {url} → {result[:80]}")

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiens.nguyen/gonext-local-worker",
-  "version": "1.0.74",
+  "version": "1.0.76",
   "description": "Polls GoNext cloud API for async local LLM jobs and runs them against Ollama/OpenAI-compatible servers on this Mac",
   "type": "module",
   "license": "MIT",