fix(config): remove default.yaml, which paired text prompts with the tool-call model

default.yaml told the model to answer in mswea_bash_command text blocks
and repeated that guidance in its format_error_template, but the default
model class only parses native tool calls, so a model that followed the
prompts could only loop through format errors into RepeatedFormatError.
hello_world.py (the only runtime consumer) now loads mini.yaml; tests
that used default.yaml as their text-format agent config now load
mini_textbased.yaml.

Author: EnesYilmazcode

src/minisweagent/config/README.md

@@ -1,7 +1,6 @@
 # Configs
 
 * `mini.yaml` - Default config for `mini`/`agents/interactive.py` agent.
-* `default.yaml` - Default config for the `default.py` agent.
 
 ## Benchmarks
 

src/minisweagent/config/default.yaml

@@ -32,7 +32,7 @@ def main(
     agent = DefaultAgent(
         LitellmModel(model_name=model_name),
         LocalEnvironment(),
-        **yaml.safe_load(Path(package_dir / "config" / "default.yaml").read_text())["agent"],
+        **yaml.safe_load(Path(package_dir / "config" / "mini.yaml").read_text())["agent"],
     )
     agent.run(task)
     return agent

src/minisweagent/run/hello_world.py

@@ -57,9 +57,9 @@ def is_observation_message(msg: dict) -> bool:
 
 
 @pytest.fixture
-def default_config():
-    """Load default agent config from config/default.yaml"""
-    config_path = Path("src/minisweagent/config/default.yaml")
+def textbased_config():
+    """Load text-based agent config from config/mini_textbased.yaml"""
+    config_path = Path("src/minisweagent/config/mini_textbased.yaml")
     with open(config_path) as f:
         config = yaml.safe_load(f)
     return config["agent"]
@@ -114,10 +114,10 @@ def make_response_api_model(
 
 
 @pytest.fixture(params=["text", "toolcall", "response_api"])
-def model_factory(request, default_config, toolcall_config):
+def model_factory(request, textbased_config, toolcall_config):
     """Parametrized fixture that returns (factory_fn, config) for all three model types."""
     if request.param == "text":
-        return make_text_model, default_config
+        return make_text_model, textbased_config
     elif request.param == "toolcall":
         return make_tc_model, toolcall_config
     else:  # response_api

tests/agents/test_default.py

@@ -102,9 +102,9 @@ def _make_model(outputs: list[tuple[str, list[dict]]], **kwargs) -> Deterministi
 
 
 @pytest.fixture
-def default_config():
-    """Load default agent config from config/default.yaml"""
-    config_path = Path("src/minisweagent/config/default.yaml")
+def textbased_config():
+    """Load text-based agent config from config/mini_textbased.yaml"""
+    config_path = Path("src/minisweagent/config/mini_textbased.yaml")
     with open(config_path) as f:
         config = yaml.safe_load(f)
     return config["agent"]
@@ -120,10 +120,10 @@ def toolcall_config():
 
 
 @pytest.fixture(params=["text", "toolcall", "response_api"])
-def model_factory(request, default_config, toolcall_config):
+def model_factory(request, textbased_config, toolcall_config):
     """Parametrized fixture that returns (factory_fn, config) for all three model types."""
     if request.param == "text":
-        return make_text_model, default_config
+        return make_text_model, textbased_config
     elif request.param == "toolcall":
         return make_tc_model, toolcall_config
     else:  # response_api

tests/agents/test_interactive.py

@@ -568,8 +568,7 @@ def test_output_file_is_created(tmp_path):
 
     # Create a temporary config file
     config_file = tmp_path / "test_config.yaml"
-    default_config_path = Path("src/minisweagent/config/default.yaml")
-    config_file.write_text(default_config_path.read_text())
+    config_file.write_text(Path("src/minisweagent/config/mini_textbased.yaml").read_text())
 
     with (
         patch("minisweagent.run.mini.configure_if_first_time"),

tests/run/test_cli_integration.py

@@ -11,13 +11,13 @@ def test_agent_save_includes_class_names():
     """Test that agent.save includes the full class names with import paths."""
     import yaml
 
-    config_path = Path("src/minisweagent/config/default.yaml")
+    config_path = Path("src/minisweagent/config/mini_textbased.yaml")
     with open(config_path) as f:
-        default_config = yaml.safe_load(f)["agent"]
+        agent_config = yaml.safe_load(f)["agent"]
 
     model = DeterministicModel(outputs=[make_output("echo 'test'", [])])
     env = LocalEnvironment()
-    agent = DefaultAgent(model, env, **default_config)
+    agent = DefaultAgent(model, env, **agent_config)
 
     agent.add_messages({"role": "system", "content": "test system message"})
     agent.add_messages({"role": "user", "content": "test user message"})
@@ -52,13 +52,13 @@ def test_agent_serialize():
     """Test that agent.serialize returns the correct structure."""
     import yaml
 
-    config_path = Path("src/minisweagent/config/default.yaml")
+    config_path = Path("src/minisweagent/config/mini_textbased.yaml")
     with open(config_path) as f:
-        default_config = yaml.safe_load(f)["agent"]
+        agent_config = yaml.safe_load(f)["agent"]
 
     model = DeterministicModel(outputs=[make_output("echo 'test'", [])])
     env = LocalEnvironment()
-    agent = DefaultAgent(model, env, **default_config)
+    agent = DefaultAgent(model, env, **agent_config)
 
     agent.add_messages({"role": "system", "content": "test system message"})
     agent.add_messages({"role": "user", "content": "test user message"})