Skip to content

Commit f8cd3c2

Browse files
QUSETIONSclaude
andcommitted
fix(ci): resolve all ruff E/F lint errors (CI was red on every run)
- F401: remove unused imports (agent_loop cybernetic controllers, input_handler time, etc.) - F821: add TYPE_CHECKING-guarded type imports (Store/AppState, SystemState, AgentMetricsCollector, ContextCyberneticsOrchestrator) — annotation-only, no runtime/circular-import impact - F841: remove dead local assignments in agent_loop - E741: rename ambiguous 'l' loop vars to 'line'/'part' CI ruff step (ruff check --select=E,F) now passes; full suite green (1150 passed). Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent 62094d3 commit f8cd3c2

34 files changed

Lines changed: 2180 additions & 1563 deletions

conftest.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,10 @@
55
import pytest
66

77

8+
def pytest_configure(config: pytest.Config) -> None:
9+
config.addinivalue_line("markers", "benchmark: mark test as a benchmark test.")
10+
11+
812
# These root-level scripts are manual smoke/integration utilities from earlier
913
# development rounds. Normal pytest coverage lives under tests/.
1014
collect_ignore = [

minicode/agent_intelligence.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,11 @@
33
from typing import Any
44

55

6+
from typing import TYPE_CHECKING
7+
8+
if TYPE_CHECKING:
9+
from minicode.agent_metrics import AgentMetricsCollector
10+
611
class ErrorCategory(Enum):
712
NETWORK = "network"
813
PERMISSION = "permission"

minicode/agent_loop.py

Lines changed: 191 additions & 51 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)