localcode 0.3.34__tar.gz → 0.3.35__tar.gz

{localcode-0.3.34/src/localcode.egg-info → localcode-0.3.35}/PKG-INFO

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

{localcode-0.3.34 → localcode-0.3.35}/pyproject.toml

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

{localcode-0.3.34 → localcode-0.3.35}/src/localcode/thinking.py

@@ -80,17 +80,15 @@ def should_use_thinking(
     if policy in {"0", "false", "no", "off", "none", ""}:
         return False
     if policy in {"1", "true", "yes", "on"}:
-        # `on` forces reasoning on genuine decision rounds — but still skips the
-        # channel on rounds we've explicitly marked mechanical (post-write
-        # implement / verify / running / complete): there is nothing to reason
-        # about there, and forcing the degeneration-prone channel onto a
-        # mechanical round is what lets a small quantized model spiral into a
-        # repetition loop. Gating loop INCIDENCE here at the policy layer
-        # composes with the sampler thinking-budget + periodicity detector +
-        # no-think retry, which catch any residual. Reasoning stages and
-        # un-staged rounds still think, so `on` still means "reason deeply".
-        stage = (task_stage or "").strip().lower()
-        return stage not in _NO_THINKING_STAGES
+        # `on` FORCES thinking on every round — it respects the user's explicit
+        # opt-in and never second-guesses it by stage. Stage is inferred from the
+        # last completed action, not what the next round must decide, so a round
+        # after a read/edit during `implement` may genuinely need reasoning;
+        # gating it off here would silently break the contract. Reasoning-loop
+        # prevention lives in the layers that don't lie about intent: the
+        # server-side thinking budget, the periodicity detector, and the
+        # verified no-thinking retry. Use `auto` for stage-aware reasoning.
+        return True
     if policy == "legacy":
         return runtime_mode.endswith("-think")
     if policy == "auto":

{localcode-0.3.34 → localcode-0.3.35/src/localcode.egg-info}/PKG-INFO

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

{localcode-0.3.34 → localcode-0.3.35}/tests/test_execution_policies.py

@@ -43,16 +43,15 @@ def test_deterministic_completion_requires_evidence_and_stops_when_done():
     assert "Stop" in done.correction
 
 
-def test_on_mode_skips_mechanical_stages():
-    """`on` forces reasoning on decision/un-staged rounds but skips the
-    degeneration-prone channel on explicitly-mechanical stages — bounding
-    reasoning-loop incidence at the policy layer without breaking the opt-in."""
+def test_on_mode_forces_thinking_on_every_stage():
+    """`on` means ALWAYS on — it respects the user's explicit opt-in and never
+    gates by stage (stage describes the last completed action, not what the next
+    round must decide). Stage-aware reasoning is `auto`'s job; loop prevention is
+    the server budget + detector + no-think retry, not a policy-layer gate."""
     from localcode.thinking import should_use_thinking as s
-    # Mechanical stages: thinking off even under `on`.
-    for stage in ("implement", "verify", "running", "complete", "verified"):
-        assert s("", "on", goal_type="build_app", task_stage=stage) is False, stage
-    # Decision / reasoning stages and un-staged rounds: still think.
-    for stage in ("plan", "discover", "repair", "scaffolding", ""):
+    for stage in ("implement", "verify", "running", "complete", "verified",
+                  "plan", "discover", "repair", "scaffolding", ""):
         assert s("", "on", goal_type="build_app", task_stage=stage) is True, stage
-    # `off` is still unconditionally off.
+    # `auto`, by contrast, DOES gate mechanical stages off.
+    assert s("", "auto", goal_type="build_app", task_stage="implement") is False
     assert s("", "off", goal_type="build_app", task_stage="plan") is False