fix(ci): restore unit test stability and lint compliance

Author: kargarisaac

src/lerim/agents/tools.py

@@ -1065,6 +1065,21 @@ def prune_history_processor(
 	return messages
 
 
+# Test/compat helper: construct a minimal context wrapper exposing ctx.deps.
+def build_test_ctx(
+	memory_root: Path,
+	trace_path: Path | None = None,
+	run_folder: Path | None = None,
+):
+	"""Return a lightweight RunContext-like object for direct tool calls in tests."""
+	deps = ExtractDeps(
+		memory_root=Path(memory_root),
+		trace_path=Path(trace_path) if trace_path else None,
+		run_folder=Path(run_folder) if run_folder else None,
+	)
+	return SimpleNamespace(deps=deps)
+
+
 # ── Legacy compatibility shim (DSPy maintain/ask agents only) ────────────
 
 

src/lerim/server/api.py

@@ -862,7 +862,7 @@ def api_queue_jobs(
             selected = _resolve_selected_projects(
                 config=config, scope="project", project=project
             )
-        except ValueError as exc:
+        except ValueError:
             # Backward-compatible fallback for HTTP/API callers that still
             # pass free-form project substrings.
             selected = []

tests/unit/test_agent.py

@@ -3,8 +3,6 @@
 from __future__ import annotations
 
 from dataclasses import replace
-from types import SimpleNamespace
-
 import httpx
 import pytest
 from openai import RateLimitError

tests/unit/test_extract_single_pass.py

@@ -13,7 +13,6 @@
 
 import inspect
 from dataclasses import fields
-from pathlib import Path
 
 from pydantic_ai.models.test import TestModel
 
@@ -26,15 +25,11 @@
 )
 from lerim.agents.tools import (
 	ExtractDeps,
-	Finding,
 	compute_request_budget,
-	context_pressure_injector,
 	edit,
 	grep,
 	note,
-	notes_state_injector,
 	prune,
-	prune_history_processor,
 	read,
 	verify_index,
 	write,

tests/unit/test_runtime_orchestrator.py

@@ -114,8 +114,6 @@ def call(model):
 		assert seen == ["primary"]
 
 	def test_retry_transient_error_same_model(self, tmp_path, monkeypatch):
-		import lerim.server.runtime as runtime_mod
-
 		monkeypatch.setattr(time, "sleep", lambda *_: None)
 		rt = _build_runtime(tmp_path, monkeypatch)
 		attempts = 0

tests/unit/test_tools.py

@@ -1343,12 +1343,12 @@ def test_low_pressure_status_only(self, tmp_path):
 			p.content for m in out
 			for p in m.parts if isinstance(p, SystemPromptPart)
 		]
-		assert any("CONTEXT:" in l for l in labels)
+		assert any("CONTEXT:" in line for line in labels)
 		# No pressure wording
-		for l in labels:
-			if "CONTEXT:" in l:
-				assert "pressure" not in l.lower()
-				assert "HARD" not in l
+		for line in labels:
+			if "CONTEXT:" in line:
+				assert "pressure" not in line.lower()
+				assert "HARD" not in line
 
 	def test_soft_pressure_label(self, tmp_path):
 		"""60-80% triggers soft pressure wording suggesting prune()."""
@@ -1366,10 +1366,10 @@ def test_soft_pressure_label(self, tmp_path):
 			p.content for m in messages
 			for p in m.parts if isinstance(p, SystemPromptPart)
 		]
-		ctx_labels = [l for l in labels if "CONTEXT:" in l]
+		ctx_labels = [line for line in labels if "CONTEXT:" in line]
 		assert ctx_labels, "Expected at least one CONTEXT: label"
-		assert any("soft pressure" in l for l in ctx_labels)
-		assert not any("HARD PRESSURE" in l for l in ctx_labels)
+		assert any("soft pressure" in line for line in ctx_labels)
+		assert not any("HARD PRESSURE" in line for line in ctx_labels)
 
 	def test_hard_pressure_label(self, tmp_path):
 		"""Above 80% triggers HARD PRESSURE wording demanding prune()."""
@@ -1385,9 +1385,9 @@ def test_hard_pressure_label(self, tmp_path):
 			p.content for m in messages
 			for p in m.parts if isinstance(p, SystemPromptPart)
 		]
-		ctx_labels = [l for l in labels if "CONTEXT:" in l]
+		ctx_labels = [line for line in labels if "CONTEXT:" in line]
 		assert ctx_labels
-		assert any("HARD PRESSURE" in l for l in ctx_labels)
+		assert any("HARD PRESSURE" in line for line in ctx_labels)
 
 
 # ---------------------------------------------------------------------------
@@ -1414,7 +1414,7 @@ def test_empty_notes_shows_zero(self, tmp_path):
 			p.content for m in messages
 			for p in m.parts if isinstance(p, SystemPromptPart)
 		]
-		assert any("NOTES:" in l and "0 findings" in l for l in labels)
+		assert any("NOTES:" in line and "0 findings" in line for line in labels)
 
 	def test_notes_grouped_by_theme(self, tmp_path):
 		"""deps.notes with 8 findings across 3 themes produces a label listing all three."""
@@ -1438,7 +1438,7 @@ def test_notes_grouped_by_theme(self, tmp_path):
 			p.content for m in messages
 			for p in m.parts if isinstance(p, SystemPromptPart)
 		]
-		notes_line = next((l for l in labels if l.startswith("NOTES:")), None)
+		notes_line = next((line for line in labels if line.startswith("NOTES:")), None)
 		assert notes_line is not None
 		assert "8 findings" in notes_line
 		assert "3 theme" in notes_line
@@ -1465,7 +1465,7 @@ def test_reads_live_state_not_snapshot(self, tmp_path):
 			p.content for m in messages_1
 			for p in m.parts if isinstance(p, SystemPromptPart)
 		]
-		line_1 = next((l for l in labels_1 if l.startswith("NOTES:")), None)
+		line_1 = next((line for line in labels_1 if line.startswith("NOTES:")), None)
 		assert line_1 is not None
 		assert "1 theme" in line_1
 
@@ -1479,7 +1479,7 @@ def test_reads_live_state_not_snapshot(self, tmp_path):
 			p.content for m in messages_2
 			for p in m.parts if isinstance(p, SystemPromptPart)
 		]
-		line_2 = next((l for l in labels_2 if l.startswith("NOTES:")), None)
+		line_2 = next((line for line in labels_2 if line.startswith("NOTES:")), None)
 		assert line_2 is not None
 		assert "2 theme" in line_2
 		assert "theme2" in line_2