localcode 0.3.2__tar.gz → 0.3.4__tar.gz

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

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

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

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

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

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

{localcode-0.3.2 → localcode-0.3.4}/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.4}/src/localcode/agent/recovery.py

@@ -365,34 +365,36 @@ def churn_nudge_for(signal: ChurnSignal) -> str:
     Names the concrete subject (file path / command) + count so the model
     sees what it's thrashing on, then tells it the ONE thing to do instead.
     """
+    # NOTE: these are FORWARD-ONLY imperatives. They deliberately contain NO
+    # self-referential loop language ("you're going in circles", "you've
+    # rewritten X N times", "thrashing", "stop planning"). A small model
+    # parrots whatever we inject: injecting "you're going in circles" as a
+    # user-role message makes it reply "You're right, I've been going in
+    # circles" — which then sits in its own context and self-conditions the
+    # loop deeper (arXiv:2509.09677). So we tell it the NEXT concrete action
+    # and nothing about the failure it just had.
     if signal.mode is ChurnMode.FILE_REWRITE:
         return (
-            f"SYSTEM: You've rewritten {signal.subject} {signal.count} times "
-            "this turn. Stop rewriting it. Read the ACTUAL error output from "
-            "the last failure, identify the specific line/cause, and make ONE "
-            "targeted edit_file change to fix that — do not overwrite the whole "
-            "file again."
+            f"SYSTEM: {signal.subject} is written — move on to the next file. "
+            "If it has a specific error, make ONE targeted edit_file change to "
+            "the exact line; otherwise create the next file the task needs."
         )
     if signal.mode is ChurnMode.COMMAND_FAILURE:
         return (
-            f"SYSTEM: `{signal.subject}` has failed {signal.count} times this "
-            "turn. Re-running it will not help. Read its error output line by "
-            "line, fix the ROOT CAUSE (a missing dependency, a syntax error in "
-            "a config/source file, a wrong path), and only then run it again."
+            f"SYSTEM: `{signal.subject}` won't succeed as-is. Read its error "
+            "output, fix the root cause (missing dependency, a syntax error in a "
+            "config/source file, a wrong path) with an edit, then run it once."
         )
     if signal.mode is ChurnMode.PLANNING_SPIN:
         return (
-            "SYSTEM: You've spent several rounds planning and re-deriving the "
-            "same approach without changing a single file or running a build. "
-            "You've planned enough. Take ONE concrete action NOW: create or "
-            "edit the most important file for the next step, then build/run to "
-            "verify it. Do not restate the plan — execute the first step of it."
+            "SYSTEM: Take the next concrete action now — create or edit the next "
+            "file the task needs, then run the build to verify. Write code, not "
+            "a plan."
         )
     # 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: Take a concrete action now — create the next file the task "
+        "needs and write real code with write_file/edit_file. Files already on "
+        "disk that you did not create are not part of your task. If you truly "
+        "lack information only the user has, ask ONE focused question."
     )

{localcode-0.3.2 → localcode-0.3.4}/src/localcode/runtime.py

@@ -168,6 +168,21 @@ def _log_disconnect_context(stage: str, error: Any, *, mid_stream: bool = False)
     diag["stage"] = stage
     diag["mid_stream"] = mid_stream
     diag["error"] = err_str[:500]
+    # Capture the SERVER's own last words. llama-server prints the real cause
+    # (ggml/Metal allocation failure, GGML_ASSERT, OOM) to server.log right
+    # before it dies; without this we only see httpx's "connection dropped"
+    # and are left guessing. The tail is the actual diagnosis.
+    _server_tail = ""
+    try:
+        from .paths import global_state_dir
+        _lp = global_state_dir() / "server.log"
+        if _lp.exists():
+            _lines = _lp.read_text(errors="replace").splitlines()
+            _server_tail = "\n".join(_lines[-15:])[-1500:]
+    except Exception:
+        _server_tail = ""
+    if _server_tail:
+        diag["server_log_tail"] = _server_tail
     try:
         from .server_manager import _lifecycle_log as _ll
         _ll("server_disconnect", **diag)
@@ -186,6 +201,8 @@ def _log_disconnect_context(stage: str, error: Any, *, mid_stream: bool = False)
                 f"pressure_kill={diag.get('pressure_kill')} "
                 f"running={diag.get('running')} mid_stream={mid_stream} "
                 f"free_mb={diag.get('free_mb')} error={err_str[:300]}\n"
+                + (f"  server.log tail:\n    "
+                   + _server_tail.replace("\n", "\n    ") + "\n" if _server_tail else "")
             )
     except Exception:
         pass
@@ -2064,6 +2081,25 @@ class LocalCodeRuntimeGateway(_DiffusionMixin):
             # revert this hunk and accept occasional rephrase as a
             # model property of Qwen3.6 IQ2_M.
             payload["repeat_penalty"] = 1.05
+            # Forward the EOS-NEUTRAL anti-loop samplers that were previously
+            # computed into `opts` and then DROPPED (they only went to the dead
+            # ollama-shaped payload below). DRY penalises repeated N-GRAMS —
+            # not single tokens — so unlike repeat_penalty it does NOT suppress
+            # the EOS/sentence-end tokens that caused the 2026-04-29 paraphrase
+            # regression; min_p trims the low-probability tail that token-
+            # collapse loops feed on. llama-server has DRY OFF by default, so
+            # NOT forwarding these is a real root cause of the repeat-collapse
+            # loops. We deliberately do NOT re-add the aggressive repeat_penalty
+            # /top_k/top_p bundle (that was the regression). A/B back to
+            # temperature-only with LOCALCODE_SAMPLER_MINIMAL=1.
+            import os as _os
+            if _os.environ.get("LOCALCODE_SAMPLER_MINIMAL") != "1":
+                payload["dry_multiplier"] = opts["dry_multiplier"]
+                payload["dry_base"] = opts["dry_base"]
+                payload["dry_allowed_length"] = opts["dry_allowed_length"]
+                payload["dry_penalty_last_n"] = opts["dry_penalty_last_n"]
+                if opts.get("min_p", 0):
+                    payload["min_p"] = opts["min_p"]
             if "num_predict" in opts:
                 _np = opts["num_predict"]
                 # llama-server treats max_tokens=-1 as "use default", which on
@@ -2080,8 +2116,11 @@ class LocalCodeRuntimeGateway(_DiffusionMixin):
             # outbound chat-completions payload so we can verify DRY is
             # actually being forwarded. Set LOCALCODE_DEBUG_SAMPLERS=1
             # to enable. Self-disables after one log to keep noise down.
-            import os, logging
-            if os.environ.get("LOCALCODE_DEBUG_SAMPLERS") == "1" and not getattr(self, "_logged_samplers", False):
+            import logging
+            # Always log the ACTUAL forwarded samplers once per session (was
+            # opt-in) so we can correlate looping with the real server config —
+            # the "control this better" record. Cheap: fires exactly once.
+            if not getattr(self, "_logged_samplers", False):
                 sampler_keys = (
                     "temperature", "top_p", "top_k", "min_p",
                     "repeat_penalty", "repeat_last_n",

{localcode-0.3.2 → localcode-0.3.4}/src/localcode/server_manager.py

@@ -71,12 +71,9 @@ from .paths import (
     lifecycle_log_path,
 )
 
-# PID file stays GLOBAL — only one llama-server runs on the machine
-# (single port, single GPU memory budget). When you `cd` between
-# projects mid-session, the running server stays serving you. The
-# per-project lifecycle log records a fresh `server_started` entry
-# on next launch from the new project. See paths.py for the full
-# global-vs-project split rationale.
+# PID file stays GLOBAL — only one llama-server runs per machine (single port,
+# single GPU budget), so it keeps serving you when you `cd` between projects.
+# See paths.py for the global-vs-project split rationale.
 PID_FILE = server_pid_file()
 DEFAULT_PORT = 8081
 # Preferred-range scan (8081-8099) before falling through to an OS-assigned
@@ -177,13 +174,9 @@ def _rewrite_port_arg(cmd: list[str], port: int) -> list[str]:
 
 STUCK_SERVER_MARKER = stuck_server_marker_path()
 
-# Persistent (append-only) lifecycle log for every server start, stop,
-# pressure-kill, and recovery event. Distinct from the per-session
-# `~/.local/share/localcode/server.log` (which the setup screen
-# overwrites with `open(..., "w")` on every launch and so loses
-# inter-session context). Per-project (lives at
-# `<project_root>/.localcode/lifecycle.log`) so each project has its
-# own diagnostic timeline.
+# Persistent (append-only) lifecycle log for every server start/stop/kill/
+# recovery event. Per-project (`<project_root>/.localcode/lifecycle.log`) so
+# each project keeps its own diagnostic timeline across sessions.
 LIFECYCLE_LOG = lifecycle_log_path()
 
 
@@ -400,12 +393,11 @@ class ServerManager:
             had_prior = self._process is not None
             _prev_model = self._model_path  # capture before shutdown clears it
             self._shutdown_locked()
-            # Wait for the OLD server's memory to actually release before
-            # spawning the new one: the kernel may take 1-3 s to free wired
-            # Metal memory after the child reaps, and spawning the new ~10 GB
-            # server in that window double-commits and trips the pressure
-            # monitor on a 16 GB Mac. Best-effort — targets ~11 GB free, times
-            # out at 8 s, and proceeds anyway (refusing would be worse).
+            # Wait for the OLD server's memory to release before spawning the
+            # new one: the kernel takes 1-3 s to free wired Metal memory after
+            # the child reaps, and spawning into that window double-commits and
+            # trips the pressure monitor on a 16 GB Mac. Best-effort — targets
+            # ~11 GB free, times out at 8 s, proceeds anyway.
             if had_prior:
                 # Only a genuine model change is "model_switch"; a same-model
                 # relaunch (crash/disconnect recovery) is "server_restart".
@@ -430,20 +422,35 @@ class ServerManager:
             # AGX driver may read the env at first Metal touch — setting it in
             # the parent env before spawn is the belt-and-suspenders approach.
             env.setdefault("AGX_RELAX_CDM_CTXSTORE_TIMEOUT", "1")
+            # Capture the server's OWN stdout/stderr to server.log (this runtime
+            # path used to send both to /dev/null, discarding llama-server's
+            # final ggml/Metal error on the recurring -9). Append per-spawn.
+            log_fh = None
+            try:
+                from .paths import global_state_dir
+                log_fh = open(global_state_dir() / "server.log", "a", buffering=1)
+                log_fh.write(f"\n===== llama-server spawn (port {port}) =====\n")
+            except Exception:
+                log_fh = None
+            try:  # close any prior spawn's handle before replacing it
+                if getattr(self, "_log_fh", None) is not None:
+                    self._log_fh.close()
+            except Exception:
+                pass
+            self._log_fh = log_fh
             self._process = subprocess.Popen(
                 cmd,
-                stdout=subprocess.DEVNULL,
-                stderr=subprocess.DEVNULL,
+                stdout=(log_fh or subprocess.DEVNULL),
+                stderr=subprocess.STDOUT if log_fh else subprocess.DEVNULL,
                 start_new_session=True,  # process group leader → killpg works
                 env=env,
             )
             self._model_path = model_path
             self._port = port
 
-            # Layer 1: kernel-enforced memory ceiling via jetsam. If
-            # llama-server tries to wire more than the budget allows,
-            # macOS kernel kills it BEFORE the vm_fault wait-chain can
-            # form. This is the hard backstop — sanctioned Apple API.
+            # Layer 1: kernel-enforced memory ceiling via jetsam — the hard
+            # backstop (sanctioned Apple API) that kills llama-server before it
+            # can push the system over the edge.
             try:
                 from .memory_guard import (
                     set_jetsam_highwater, recommended_jetsam_limit_mb,
@@ -467,14 +474,10 @@ class ServerManager:
             except Exception:
                 self._pressure_thread = None
             self._write_pid_file(self._process.pid)
-            # Log the FULL command we launched with so future debugging
-            # can answer "what flags was the server using during this
-            # session?" — needed because subtle flags like
-            # `--lookup-cache-dynamic` or `--spec-type ngram-mod` cause
-            # repetition pathologies that look like model bugs but are
-            # actually decode-time speculative-cache feedback loops.
-            # Truncated to 4000 chars to keep the event small; that's
-            # enough for ~80 args.
+            # Log the FULL launch command so debugging can answer "what flags
+            # was the server using?" — subtle flags (e.g. --spec-type ngram-mod)
+            # cause repetition pathologies that look like model bugs. Truncated
+            # to 4000 chars (~80 args) to keep the event small.
             _flags_str = " ".join(str(c) for c in cmd)[:4000]
             _lifecycle_log("server_started", pid=self._process.pid, port=port,
                            model=Path(model_path).name,

{localcode-0.3.2 → localcode-0.3.4}/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.4}/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.4/src/localcode.egg-info}/PKG-INFO

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

{localcode-0.3.2 → localcode-0.3.4}/tests/test_churn_detection.py

@@ -80,8 +80,10 @@ def test_investigation_spin_outranks_planning_spin():
 def test_planning_spin_nudge_is_actionable():
     sig = detect_churn({}, {}, 0, planning_streak=CHURN_PLANNING_STREAK_LIMIT)
     text = churn_nudge_for(sig)
-    assert "planned enough" in text.lower()
-    assert "concrete action" in text.lower() or "execute" in text.lower()
+    # Forward-only imperative: tells it the next action, with NO echo-able
+    # self-critical loop language ("planned enough") a small model would parrot.
+    assert "concrete action" in text.lower() or "write code" in text.lower()
+    assert "planned enough" not in text.lower() and "circles" not in text.lower()
 
 
 def test_command_failure_takes_precedence_over_file_rewrite():

{localcode-0.3.2 → localcode-0.3.4}/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