@tiens.nguyen/gonext-local-worker 1.0.193 → 1.0.195

package/gonext_agent_chat.py

@@ -1800,6 +1800,41 @@ def _ws_for_path(path: str):
     return None
 
 
+def _workspace_overview(roots, max_entries: int = 40) -> str:
+    """Cheap, LOCAL top-level listing of each registered workspace root — a plain
+    os.listdir(), no recursion, no network call, no model call. Used two ways: (1)
+    injected into the task prompt so the agent has upfront grounding that the workspace
+    is non-empty WITHOUT spending its first (possibly failing) tool-call round-trip just
+    discovering that; (2) appended to the degrade-to-plain-reply message so even a TOTAL
+    coding-model outage still shows the user their files are actually there — this is
+    the one piece of workspace context that stays available no matter how badly the
+    model-calling side of the agent is failing, since it never touches a model at all."""
+    import os
+    parts = []
+    for w in roots:
+        root = w["path"]
+        try:
+            entries = sorted(
+                e for e in os.listdir(root) if e not in _RAG_SKIP_DIRS
+            )
+        except OSError as e:
+            parts.append(f"{w['name']} ({root}): could not list ({e})")
+            continue
+        if not entries:
+            parts.append(f"{w['name']} ({root}): EMPTY — no files or folders found.")
+            continue
+        shown = entries[:max_entries]
+        labeled = [
+            f"{e}/" if os.path.isdir(os.path.join(root, e)) else e for e in shown
+        ]
+        more = f" …(+{len(entries) - max_entries} more)" if len(entries) > max_entries else ""
+        parts.append(
+            f"{w['name']} ({root}) — {len(entries)} top-level item(s): "
+            + ", ".join(labeled) + more
+        )
+    return "\n".join(parts)
+
+
 def _ws_write_allowed(path: str) -> str:
     """Resolve `path` for WRITING: must be inside a registered workspace and never
     inside its .git/ internals (agents must not corrupt version control)."""
@@ -2347,8 +2382,15 @@ def run_agent_chat(cfg):
         "If the user adds information, rag_add(source_url=url, text=...).\n"
     ) if _RAG_AVAILABLE else ""
     _ws_names = ", ".join(f"{w['name']} = {w['path']}" for w in _WS_ROOTS)
+    # Upfront, ZERO-COST (no tool call, no model round-trip) top-level listing — so the
+    # agent already knows the workspace is non-empty and roughly what's in it before its
+    # first step, instead of having to spend (and risk losing to a timeout) a whole
+    # round-trip just calling list_dir to discover that.
+    _ws_overview = _workspace_overview(_WS_ROOTS) if _WS_AVAILABLE else ""
     _ws_tool_block = (
         f"  WORKSPACES (local code folders you may READ and EDIT): {_ws_names}\n"
+        f"  Current contents:\n"
+        + "\n".join(f"    {line}" for line in _ws_overview.splitlines()) + "\n"
         "    - list_dir(path) / read_text_file(path) — browse and read files.\n"
         "    - grep_repo(pattern, path='', glob='') — search code, returns file:line hits. "
         "Use this to find the EXACT lines to change.\n"
@@ -3929,7 +3971,24 @@ def run_agent_chat(cfg):
             # a fabricated "final" answer here instead would poison every future turn's
             # prompt with raw exception text (this is exactly what caused task #35).
             raise e
-        _emit({"type": "final", "text": fallback})
+        # _plain_reply has NO tool/file/HTTP access whatsoever — that's by design (it's a
+        # bare chat completion). Without a note here, its honest "I don't have access to
+        # your workspace/that page/etc." reads to the user as a PERMISSIONS bug, when the
+        # real cause is that the tool-capable agent loop failed upstream (e.g. the coding
+        # model timed out) and we silently swapped in a capability-limited fallback. Make
+        # that swap visible so the user can tell "genuinely no access" apart from
+        # "the real attempt never got to run" (seen live: workspace summarize request →
+        # coding model timed out 3x on a slow/unreachable remote Ollama box → silent
+        # fallback claimed "no access to your workspace").
+        note = f"⚠️ I couldn't finish the full investigation ({_clip(str(e), 120)}) — here's what I can say without it:\n\n{fallback}"
+        # Append the workspace's ACTUAL top-level contents (computed locally, no model
+        # call — see _workspace_overview) so the user sees their files are really there
+        # even during a total coding-model outage, instead of trusting a plain-chat
+        # model's guess that "there's no code or project" (seen live — false: the
+        # workspace had real content the whole time, the model just never reached it).
+        if _WS_AVAILABLE:
+            note += f"\n\n(Your registered workspace:\n{_workspace_overview(_WS_ROOTS)})"
+        _emit({"type": "final", "text": note})
 
 
 def _log(text: str):

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tiens.nguyen/gonext-local-worker",
-  "version": "1.0.193",
+  "version": "1.0.195",
   "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",