localcode 0.3.2__tar.gz → 0.3.3__tar.gz

{localcode-0.3.2/src/localcode.egg-info → localcode-0.3.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: localcode
-Version: 0.3.2
+Version: 0.3.3
 Summary: High-performance AI coding on consumer hardware.
 Author: LocalCode contributors
 License-Expression: Apache-2.0

{localcode-0.3.2 → localcode-0.3.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "localcode"
-version = "0.3.2"
+version = "0.3.3"
 description = "High-performance AI coding on consumer hardware."
 readme = "README.md"
 requires-python = ">=3.10"

{localcode-0.3.2 → localcode-0.3.3}/src/localcode/__init__.py

@@ -1,3 +1,3 @@
 __all__ = ["__version__"]
 
-__version__ = "0.3.2"
+__version__ = "0.3.3"

{localcode-0.3.2 → localcode-0.3.3}/src/localcode/agent/context.py

@@ -246,10 +246,9 @@ def _semantic_tool_summary(content: str) -> str:
     )
     if is_error:
         # Self-conditioning (arXiv:2509.09677): echoing an OLD error's text
-        # pushes the model to repeat the mistake (scale-independent). Recent
-        # errors stay intact (kept above by keep_recent) so it can fix the
-        # immediate problem; older ones drop to a neutral note. The progress
-        # ledger still records the failure fact, minus the conditioning text.
+        # pushes the model to repeat the mistake. Recent errors stay intact
+        # (kept above by keep_recent) so it can fix the immediate problem;
+        # older ones drop to a neutral note (the ledger still records the fact).
         return "[an earlier tool call errored and was handled — details dropped]"
     if facts:
         return (
@@ -725,9 +724,10 @@ def build_progress_ledger(
         return list(dict.fromkeys(s for s in seq if s))
 
     lines = [
-        "## Work already done this task — build on it. Do NOT re-read a file or "
-        "re-run a command listed below unless you have CHANGED it since; you "
-        "already have the result.",
+        "## A log of YOUR OWN tool calls so far this turn — NOT the user's work "
+        "and NOT pre-existing files on disk (leftover files you did not create are "
+        "not your progress or your task). Don't re-read/re-run anything below "
+        "unless you changed it; keep making NEW progress toward the user's goal.",
     ]
     rd = _uniq(files_read)
     if rd:

{localcode-0.3.2 → localcode-0.3.3}/src/localcode/agent/recovery.py

@@ -390,9 +390,11 @@ def churn_nudge_for(signal: ChurnSignal) -> str:
         )
     # INVESTIGATION_SPIN
     return (
-        "SYSTEM: You've spent several rounds reading and searching files "
-        "without taking any concrete action. You're investigating in circles. "
-        "Take a concrete action NOW: make a specific edit, run the build/tests, "
-        "or — if you genuinely lack information only the user has — ask ONE "
-        "focused question. Do not read or grep more files this round."
+        "SYSTEM: Several rounds of reading/listing/searching without WRITING "
+        "anything — you're going in circles. STOP exploring. If the task is to "
+        "build something, create the target file and write real code THIS round "
+        "with write_file/edit_file; do NOT read, ls, find, cat, or grep again. "
+        "Leftover files on disk are NOT your progress (may be an unrelated run) — "
+        "only files YOU write count. If you truly lack info only the user has, "
+        "ask ONE focused question; otherwise start writing now."
     )

{localcode-0.3.2 → localcode-0.3.3}/src/localcode/tools/bash.py

@@ -711,6 +711,50 @@ def _normalize_repo_root_variants(cmd: str, repo: str) -> str:
     return pattern.sub(repo, cmd)
 
 
+def _quoting_error_hint(output: str) -> str:
+    """Turn a raw shell quoting failure into actionable guidance.
+
+    A path with spaces or parentheses (e.g. `Qwen 3.6 35B-A3B (Q8)`) that
+    isn't quoted breaks bash with `syntax error near unexpected token` — an
+    error a model can't act on, so it retries the same broken shape in a loop
+    (observed exactly this). Tell it precisely how to fix it.
+    """
+    low = output.lower()
+    if "syntax error near unexpected token" not in low and "unexpected eof" not in low:
+        return ""
+    return (
+        "HINT: a path or argument with spaces or parentheses wasn't quoted, so "
+        "the shell mis-parsed it. Wrap paths in SINGLE quotes — e.g. "
+        "ls -la '/Users/you/My Dir (v2)'. Simpler and safer: use list_files or "
+        "read_file with the path as an argument — they take the raw path and "
+        "never need shell escaping."
+    )
+
+
+def _redirect_shell_dir_listing(cmd: str) -> str:
+    """Route `ls <path>` to list_files, which can't break on spaces/parens.
+
+    Mirrors the `cat → read_file` guard. Only fires for a plain `ls` with a
+    PATH argument (no pipes/redirs/chains) — bare `ls`/`ls -la` in the cwd
+    can't hit the quoting failure, so it's left to run.
+    """
+    _cd, body = _extract_leading_cd(cmd)
+    b = body.strip()
+    if any(sep in b for sep in ("|", ">", "<", "&&", "||", ";", "$(", "`")):
+        return ""
+    # Path arg must NOT start with '-' (else `ls -la` reads its own flags as a path).
+    m = re.match(r"^ls((?:\s+-[a-zA-Z]+)*)\s+([^-\s].*)$", b)
+    if m is None:
+        return ""  # bare `ls`/`ls -la` (no path) — harmless, let it run
+    path = m.group(2).strip().strip('"').strip("'")
+    return (
+        f"REJECTED: use list_files(path='{path}') to inspect a directory, not "
+        "bash `ls <path>`. list_files takes the path as an argument, so it never "
+        "breaks on spaces or parentheses in a folder name — which is what fails "
+        "with bash here."
+    )
+
+
 def execute(ctx: ToolContext, args: dict) -> str:
     cmd = _normalize_repo_root_variants(args["command"], str(ctx.repo))
     # Block ANY use of process-attaching debuggers — lldb / dtrace /
@@ -776,6 +820,9 @@ def execute(ctx: ToolContext, args: dict) -> str:
     file_read_redirect = _redirect_shell_file_read(cmd, str(repo))
     if file_read_redirect:
         return file_read_redirect
+    dir_listing_redirect = _redirect_shell_dir_listing(cmd)
+    if dir_listing_redirect:
+        return dir_listing_redirect
     file_write_redirect = _redirect_shell_file_write(cmd, str(repo))
     if file_write_redirect:
         return file_write_redirect
@@ -969,6 +1016,11 @@ def execute(ctx: ToolContext, args: dict) -> str:
             pass
         if r.returncode != 0:
             output = f"[exit code {r.returncode}]\n{output}"
+            # Shell quoting failure (unquoted spaces/parens in a path) → give
+            # the model an actionable fix instead of a raw syntax error it loops on.
+            _qh = _quoting_error_hint(output)
+            if _qh:
+                output = _qh + "\n\n" + output
         # AirPlay-collision detector: if the model curls localhost:5000
         # or :7000, those are squatted by macOS AirPlay Receiver /
         # AirTunes — bare `curl -s` sees a 200 with empty body and

{localcode-0.3.2 → localcode-0.3.3}/src/localcode/tui/screens/chat.py

@@ -411,12 +411,16 @@ class _ChatTextArea(TextArea):
         & .text-area--cursor-line {
             background: ansi_default;
         }
-        /* The caret. `reverse` swaps the cell's fg/bg using the terminal's
-           OWN colors, so the cursor is a light block on a dark terminal and
-           a dark block on a light one — no hardcoded hex that fights the
-           ansi_default palette (which rendered as a near-black block). */
+        /* The caret. textual-ansi reports dark=False, so TextArea's own
+           `&:light .text-area--cursor` rule (background: $foreground 70%) wins
+           over a plain override and renders a dark/black block. Force it with
+           !important: color+background = ansi_default (terminal fg/bg) plus
+           `reverse` swaps them, giving a light block on a dark terminal and a
+           dark block on a light one — terminal-adaptive, no hardcoded hex. */
         & .text-area--cursor {
-            text-style: reverse;
+            color: ansi_default !important;
+            background: ansi_default !important;
+            text-style: reverse !important;
         }
         &:focus {
             background-tint: transparent 0%;

{localcode-0.3.2 → localcode-0.3.3/src/localcode.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: localcode
-Version: 0.3.2
+Version: 0.3.3
 Summary: High-performance AI coding on consumer hardware.
 Author: LocalCode contributors
 License-Expression: Apache-2.0

{localcode-0.3.2 → localcode-0.3.3}/tests/test_context.py

@@ -169,7 +169,9 @@ def test_progress_ledger_dedups_and_shows_outcomes():
         files_read=["a.ts", "b.ts", "a.ts", "a.ts"],  # a.ts re-read
         budget_chars=3500,
     )
-    assert "build on it" in led.lower() and "do not re-read" in led.lower()
+    # Header frames the ledger as the assistant's OWN actions (not the user's
+    # work / pre-existing files) so the model doesn't treat stale dirs as its task.
+    assert "your own tool calls" in led.lower() and "new progress" in led.lower()
     assert led.count("a.ts") == 1  # deduped
     assert "src/types.ts" in led
     assert "ok npm i" in led and "x npm run build" in led  # outcomes flagged