test: update tests to match removed auto_save config option

- Remove auto_save parameter from DisplayManager tests
- Remove auto_save assertions from ChatConfig tests
- Remove auto_save from ChatLoop initialization tests
- Remove auto_save prompts from config wizard tests
- Update display banner test to expect 'always enabled' message
- All 318 tests now passing

Author: jwesleye

tests/unit/test_chat_config.py

@@ -42,7 +42,6 @@ def agent_override_config(tmp_path):
   'Test Agent':
     features:
       show_tokens: true
-      auto_save: true
   'Another Agent':
     behavior:
       max_retries: 10
@@ -59,7 +58,6 @@ def test_defaults_loaded(self):
         config = ChatConfig()
 
         assert config.get("features.show_tokens") is True
-        assert config.get("features.auto_save") is True
         assert config.get("features.rich_enabled") is True
         assert config.get("behavior.max_retries") == 3
         assert config.get("ui.show_banner") is True
@@ -117,7 +115,6 @@ def test_get_agent_override(self, agent_override_config):
 
         # Agent-specific override
         assert config.get("features.show_tokens", agent_name="Test Agent") is True
-        assert config.get("features.auto_save", agent_name="Test Agent") is True
 
         # Different agent
         assert config.get("behavior.max_retries", agent_name="Another Agent") == 10
@@ -160,7 +157,6 @@ def test_get_section_with_agent_override(self, agent_override_config):
         # Agent-specific section
         agent_features = config.get_section("features", agent_name="Test Agent")
         assert agent_features["show_tokens"] is True
-        assert agent_features["auto_save"] is True
 
     def test_get_colors_section_decodes_escapes(self, tmp_path, monkeypatch):
         """Test that colors section contains valid color values."""
@@ -202,13 +198,13 @@ def test_set_agent_specific_value(self):
         """Test setting agent-specific configuration."""
         config = ChatConfig()
 
-        config.set("features.auto_save", False, agent_name="My Agent")
+        config.set("features.show_tokens", False, agent_name="My Agent")
 
         # Global should remain unchanged (default is True now)
-        assert config.get("features.auto_save") is True
+        assert config.get("features.show_tokens") is True
 
         # Agent-specific should be set
-        assert config.get("features.auto_save", agent_name="My Agent") is False
+        assert config.get("features.show_tokens", agent_name="My Agent") is False
 
 
 class TestConfigMerging:

tests/unit/test_chat_loop_utils.py

@@ -93,7 +93,6 @@ def test_initialization_with_defaults(self, mock_agent):
             assert chat_loop.retry_delay == 2.0
             assert chat_loop.timeout == 120.0
             assert chat_loop.spinner_style == "dots"
-            assert chat_loop.auto_save is False
             assert chat_loop.show_metadata is True
             assert chat_loop.show_thinking is True
             assert chat_loop.show_duration is True
@@ -107,7 +106,6 @@ def test_initialization_with_config(self, mock_agent, tmp_path):
             "behavior.retry_delay": 3.0,
             "behavior.timeout": 180.0,
             "behavior.spinner_style": "bouncingBall",
-            "features.auto_save": True,
             "features.show_metadata": False,
             "features.show_tokens": True,
             "features.rich_enabled": False,
@@ -130,7 +128,6 @@ def test_initialization_with_config(self, mock_agent, tmp_path):
         assert chat_loop.retry_delay == 3.0
         assert chat_loop.timeout == 180.0
         assert chat_loop.spinner_style == "bouncingBall"
-        assert chat_loop.auto_save is True
         assert chat_loop.show_metadata is False
         assert chat_loop.show_tokens is True
         assert chat_loop.use_rich is False  # Rich disabled by config

tests/unit/test_config_wizard.py

@@ -34,7 +34,6 @@ def temp_config_file(tmp_path):
   reset: '\\033[0m'
 
 features:
-  auto_save: false
   show_tokens: true
   show_metadata: true
   rich_enabled: true
@@ -351,7 +350,6 @@ def test_configure_features_all_defaults(self, mock_input, wizard):
         wizard._configure_features()
 
         assert "features" in wizard.config
-        assert "auto_save" in wizard.config["features"]
         assert "show_tokens" in wizard.config["features"]
         assert "show_metadata" in wizard.config["features"]
         assert "rich_enabled" in wizard.config["features"]
@@ -360,11 +358,11 @@ def test_configure_features_all_defaults(self, mock_input, wizard):
     @patch("builtins.input")
     def test_configure_features_custom_values(self, mock_input, wizard):
         """Test configuring features with custom values."""
-        mock_input.side_effect = ["y", "y", "n", "n", "y"]
+        # Removed auto_save, so now only 4 prompts instead of 5
+        mock_input.side_effect = ["y", "n", "n", "y"]
 
         wizard._configure_features()
 
-        assert wizard.config["features"]["auto_save"] is True
         assert wizard.config["features"]["show_tokens"] is True
         assert wizard.config["features"]["show_metadata"] is False
         assert wizard.config["features"]["rich_enabled"] is False
@@ -562,7 +560,7 @@ def test_write_config_global_new_file(
         monkeypatch.setattr(Path, "home", lambda: tmp_path)
         wizard.config = {
             "colors": {"user": "bright_white"},
-            "features": {"auto_save": False},
+            "features": {"show_tokens": False},
             "ui": {},
             "audio": {"enabled": True, "notification_sound": None},
             "behavior": {},
@@ -666,7 +664,7 @@ def test_generate_yaml_includes_all_sections(self, wizard):
         """Test that generated YAML includes all config sections."""
         wizard.config = {
             "colors": {"user": "bright_white"},
-            "features": {"auto_save": False},
+            "features": {"show_tokens": False},
             "ui": {"show_banner": True},
             "audio": {"enabled": True, "notification_sound": None},
             "behavior": {"max_retries": 3},
@@ -820,7 +818,6 @@ def test_reset_global_config_with_confirmation(
         content = result.read_text()
         assert "bright_white" in content
         assert "bright_blue" in content
-        assert "auto_save: false" in content
         assert "show_tokens: false" in content
 
     @patch("builtins.input")

tests/unit/test_display_manager.py

@@ -48,7 +48,6 @@ def display_with_config():
     return DisplayManager(
         agent_name="Test Agent",
         agent_description="Test description",
-        auto_save=True,
         config=config,
     )
 
@@ -68,7 +67,6 @@ def test_initialization_defaults(self):
         assert display.show_banner is True
         assert display.show_metadata is False
         assert display.use_rich is False
-        assert display.auto_save is False
         assert display.config is None
         assert display.status_bar is None
 
@@ -85,15 +83,13 @@ def test_initialization_with_all_params(self):
             show_banner=False,
             show_metadata=True,
             use_rich=True,
-            auto_save=True,
             config=config,
             status_bar=status_bar,
         )
 
         assert display.show_banner is False
         assert display.show_metadata is True
         assert display.use_rich is True
-        assert display.auto_save is True
         assert display.config is config
         assert display.status_bar is status_bar
 
@@ -156,7 +152,7 @@ def test_display_banner_with_config(self, display_with_config, capsys):
         captured = capsys.readouterr()
 
         assert "Configuration loaded" in captured.out
-        assert "Auto-save: enabled" in captured.out
+        assert "Auto-save: always enabled" in captured.out
 
 
 class TestDisplayHelp:
@@ -269,11 +265,12 @@ def test_display_info_with_config(self, display_with_config, capsys):
         assert "Configuration file support" in captured.out
 
     def test_display_info_with_auto_save(self, display_with_config, capsys):
-        """Test info display with auto-save enabled."""
+        """Test info display shows auto-save always enabled."""
         display_with_config.display_info()
         captured = capsys.readouterr()
 
-        assert "Auto-save conversations" in captured.out
+        # Auto-save is now always enabled, so should always appear
+        assert "Auto-save conversations after each message" in captured.out
 
 
 class TestDisplaySessionSummary: