chore: code cleanup - formatting and linting fixes

- Run black formatter on all source and test files
- Fix ruff linting issues (remove unused imports, fix f-string)
- Fix unused variable in test_chat_config.py
- All linting and type checking passes

Author: jwesleye

src/basic_agent_chat_loop/chat_loop.py

@@ -867,7 +867,9 @@ async def _async_run(self):
                             continue
 
                         # List available prompt templates grouped by source
-                        templates_grouped = self.template_manager.list_templates_grouped()
+                        templates_grouped = (
+                            self.template_manager.list_templates_grouped()
+                        )
                         self.display_manager.display_templates(templates_grouped)
                         continue
 

src/basic_agent_chat_loop/components/display_manager.py

@@ -5,7 +5,6 @@
 """
 
 import time
-from pathlib import Path
 from typing import Any, Optional
 
 from .token_tracker import TokenTracker
@@ -293,7 +292,7 @@ def display_templates(self, templates_grouped: list):
             print("  ~/.prompts/")
             print("  ./.claude/commands/")
             print("  ~/.claude/commands/")
-            print(f"Example: ~/.prompts/review.md")
+            print("Example: ~/.prompts/review.md")
             return
 
         # Count total templates across all sources
@@ -314,7 +313,9 @@ def display_templates(self, templates_grouped: list):
             for name, desc in templates:
                 override_indicator = ""
                 if name in seen_templates:
-                    override_indicator = f" {Colors.DIM}(overrides previous){Colors.RESET}"
+                    override_indicator = (
+                        f" {Colors.DIM}(overrides previous){Colors.RESET}"
+                    )
                 else:
                     seen_templates.add(name)
 

src/basic_agent_chat_loop/components/template_manager.py

@@ -27,9 +27,9 @@ def __init__(self, prompts_dir: Optional[Path] = None):
         # Template directories in priority order (first = highest priority)
         # Priority: ~/.prompts > ./.claude/commands > ~/.claude/commands
         self.template_dirs = [
-            base_prompts_dir,                          # Legacy/base (highest priority)
-            Path.cwd() / ".claude" / "commands",       # Project-specific (medium)
-            Path.home() / ".claude" / "commands",      # User global (lowest priority)
+            base_prompts_dir,  # Legacy/base (highest priority)
+            Path.cwd() / ".claude" / "commands",  # Project-specific (medium)
+            Path.home() / ".claude" / "commands",  # User global (lowest priority)
         ]
 
         # Keep prompts_dir for backward compatibility (used for initialization)

tests/unit/test_chat_config.py

@@ -352,6 +352,7 @@ def test_get_config_reload(self, temp_config_file, tmp_path, monkeypatch):
         """Test that get_config can reload configuration."""
         # Clear global config state first
         import basic_agent_chat_loop.chat_config as chat_config_module
+
         chat_config_module._global_config = None
 
         # Create isolated home and cwd directories
@@ -364,7 +365,9 @@ def test_get_config_reload(self, temp_config_file, tmp_path, monkeypatch):
         monkeypatch.setattr(Path, "home", lambda: home_dir)
         monkeypatch.setattr(Path, "cwd", lambda: cwd_dir)
 
-        config1 = get_config()
+        # Get initial config to initialize singleton
+        get_config()
+        # Reload with temp config file
         config2 = get_config(temp_config_file, reload=True)
 
         # Verify reload loaded the temp config file

tests/unit/test_response_renderer.py

@@ -4,7 +4,6 @@
 Agent library handles all response rendering naturally.
 """
 
-import pytest
 
 from basic_agent_chat_loop.components.response_renderer import ResponseRenderer
 
@@ -28,7 +27,9 @@ def test_initialization(self):
 
     def test_initialization_with_different_agent_name(self):
         """Test initialization with different agent names."""
-        renderer = ResponseRenderer(agent_name="MyCustomAgent", colors_module=MockColors)
+        renderer = ResponseRenderer(
+            agent_name="MyCustomAgent", colors_module=MockColors
+        )
         assert renderer.agent_name == "MyCustomAgent"
 
     def test_initialization_with_special_characters_in_name(self):

tests/unit/test_template_manager.py

@@ -320,9 +320,7 @@ def test_list_templates_grouped(self, multi_dir_setup):
         assert "shared" in project_templates
 
         # Check user directory templates
-        user_group = next(
-            (g for g in grouped if g[0] == multi_dir_setup["user"]), None
-        )
+        user_group = next((g for g in grouped if g[0] == multi_dir_setup["user"]), None)
         assert user_group is not None
         user_templates = dict(user_group[1])
         assert "user_only" in user_templates