refactor: clean up imports and improve default message limit

- Remove unused import of get_cancel_agent_key from config_commands.py since only the display name function is needed
- Increase default message limit from 100 to 1000 to provide more generous interaction capacity
- Fix trailing whitespace and formatting issues in test files for better code consistency
- These changes improve code cleanliness and provide a better default user experience with higher message limits

Author: mpfaffenberger

code_puppy/command_line/config_commands.py

@@ -46,7 +46,7 @@ def handle_show_command(command: str) -> bool:
         get_use_dbos,
         get_yolo_mode,
     )
-    from code_puppy.keymap import get_cancel_agent_key, get_cancel_agent_display_name
+    from code_puppy.keymap import get_cancel_agent_display_name
     from code_puppy.messaging import emit_info
 
     puppy_name = get_puppy_name()

code_puppy/config.py

@@ -1051,7 +1051,7 @@ def set_enable_dbos(enabled: bool) -> None:
     set_config_value("enable_dbos", "true" if enabled else "false")
 
 
-def get_message_limit(default: int = 100) -> int:
+def get_message_limit(default: int = 1000) -> int:
     """
     Returns the user-configured message/request limit for the agent.
     This controls how many steps/requests the agent can take.

tests/test_uvx_detection.py

@@ -46,7 +46,7 @@ def test_is_uvx_in_chain_with_uvx_exe(self):
     def test_is_uvx_in_chain_with_uv_exe_only(self):
         """Test _is_uvx_in_chain does NOT detect uv.exe alone (only uvx.exe)."""
         from code_puppy.uvx_detection import _is_uvx_in_chain
-        
+
         # uv.exe alone should NOT trigger detection - only uvx.exe has SIGINT issues
         chain = ["python.exe", "uv.exe", "powershell.exe"]
         assert _is_uvx_in_chain(chain) is False
@@ -84,6 +84,7 @@ def test_should_use_alternate_cancel_key_windows_uvx(self):
 
             # Re-import with mocks active
             import importlib
+
             import code_puppy.uvx_detection
 
             importlib.reload(code_puppy.uvx_detection)
@@ -101,6 +102,7 @@ def test_should_use_alternate_cancel_key_windows_uvx(self):
     def test_should_use_alternate_cancel_key_windows_no_uvx(self):
         """Test should_use_alternate_cancel_key returns False on Windows without uvx."""
         import importlib
+
         import code_puppy.uvx_detection
 
         importlib.reload(code_puppy.uvx_detection)
@@ -116,6 +118,7 @@ def test_should_use_alternate_cancel_key_windows_no_uvx(self):
     def test_should_use_alternate_cancel_key_linux_uvx(self):
         """Test should_use_alternate_cancel_key returns False on Linux even with uvx."""
         import importlib
+
         import code_puppy.uvx_detection
 
         importlib.reload(code_puppy.uvx_detection)
@@ -131,6 +134,7 @@ def test_should_use_alternate_cancel_key_linux_uvx(self):
     def test_should_use_alternate_cancel_key_macos_uvx(self):
         """Test should_use_alternate_cancel_key returns False on macOS even with uvx."""
         import importlib
+
         import code_puppy.uvx_detection
 
         importlib.reload(code_puppy.uvx_detection)
@@ -180,6 +184,7 @@ def test_get_parent_process_chain_psutil_available(self):
         # This test runs with actual psutil if installed
         try:
             import psutil  # noqa: F401
+
             from code_puppy.uvx_detection import _get_parent_process_chain_psutil
 
             chain = _get_parent_process_chain_psutil()
@@ -241,6 +246,7 @@ def test_keymap_returns_ctrl_k_on_windows_uvx(self):
         ):
             # Import fresh to pick up the mock
             import importlib
+
             import code_puppy.keymap
 
             importlib.reload(code_puppy.keymap)
@@ -258,6 +264,7 @@ def test_keymap_respects_config_when_not_uvx(self):
             patch("code_puppy.config.get_value", return_value="ctrl+q"),
         ):
             import importlib
+
             import code_puppy.keymap
 
             importlib.reload(code_puppy.keymap)
@@ -275,6 +282,7 @@ def test_keymap_returns_default_when_not_uvx_and_no_config(self):
             patch("code_puppy.config.get_value", return_value=None),
         ):
             import importlib
+
             import code_puppy.keymap
 
             importlib.reload(code_puppy.keymap)
@@ -289,6 +297,7 @@ class TestMockedProcessChain:
     def test_uvx_detection_with_mocked_chain(self):
         """Test uvx detection with a simulated uvx process chain."""
         import importlib
+
         import code_puppy.uvx_detection
 
         importlib.reload(code_puppy.uvx_detection)
@@ -309,6 +318,7 @@ def test_uvx_detection_with_mocked_chain(self):
     def test_no_uvx_detection_with_mocked_chain(self):
         """Test no uvx detection with a non-uvx process chain."""
         import importlib
+
         import code_puppy.uvx_detection
 
         importlib.reload(code_puppy.uvx_detection)
@@ -329,6 +339,7 @@ def test_no_uvx_detection_with_mocked_chain(self):
     def test_uvx_detection_caching(self):
         """Test that uvx detection result is cached."""
         import importlib
+
         import code_puppy.uvx_detection
 
         importlib.reload(code_puppy.uvx_detection)
@@ -354,4 +365,4 @@ def counting_get_chain():
             assert result1 is True
             assert result2 is True
             # Should only call the chain function once due to caching
-            assert call_count == 1
+            assert call_count == 1