feat: limit config_to_agent to core configuration keys

- Remove support for advanced Agent parameters in config_to_agent
- Only support: model, prompt, tools, name in configuration
- Advanced parameters can still be passed via kwargs
- Remove agent_id test and update function mapping
- Keep interface simple and focused on basic agent configuration

🤖 Assisted by Amazon Q Developer

Author: mrlee-amazon

REFACTOR_TODO.md

@@ -0,0 +1,127 @@
+# Agent Config Refactor TODO
+
+## Plan
+Refactor from class-based `AgentConfig` to function-based `config_to_agent` approach:
+
+### Current Interface
+```python
+from strands.experimental.agent_config import AgentConfig
+config = AgentConfig("/path/to/config.json")
+agent = config.to_agent()
+```
+
+### Target Interface
+```python
+from strands.experimental import config_to_agent
+agent = config_to_agent("/path/to/config.json")
+```
+
+## Tasks
+
+### 1. Core Function Implementation
+- [ ] Create `config_to_agent` function in `src/strands/experimental/agent_config.py`
+- [ ] Support both file path and dictionary input
+- [ ] Handle `file://` prefix for file paths
+- [ ] Parse JSON configuration files
+- [ ] Pass configuration directly to Agent constructor
+
+### 2. Tool Handling Simplification
+- [ ] Remove ToolRegistry usage - let Agent handle tools internally
+- [ ] Remove DEFAULT_TOOLS concept
+- [ ] Pass tools list directly from config to Agent
+- [ ] Remove `raise_exception_on_missing_tool` parameter (use Agent's default behavior)
+
+### 3. Configuration Handling
+- [ ] Only pass non-None config values to Agent constructor
+- [ ] Use Agent's default configs when values not specified
+- [ ] Support all Agent constructor parameters from config
+
+### 4. Module Exports
+- [ ] Update `src/strands/experimental/__init__.py` to export `config_to_agent`
+- [ ] Remove `AgentConfig` class export
+
+### 5. Tests Update
+- [ ] Refactor all tests to use `config_to_agent` function
+- [ ] Remove class-based test patterns
+- [ ] Update test imports
+- [ ] Simplify tool-related tests (no more ToolRegistry mocking)
+- [ ] Update file handling tests
+
+### 6. Cleanup
+- [ ] Remove `AgentConfig` class entirely
+- [ ] Remove unused imports and dependencies
+- [ ] Clean up any remaining ToolRegistry references
+
+## Progress Tracking
+- [x] Task 1: Core Function Implementation
+- [x] Task 2: Tool Handling Simplification  
+- [x] Task 3: Configuration Handling
+- [x] Task 4: Module Exports
+- [x] Task 5: Tests Update
+- [x] Task 6: Cleanup
+
+## Completed Work
+
+### ✅ Core Function Implementation
+- Created `config_to_agent` function in `src/strands/experimental/agent_config.py`
+- Supports both file path and dictionary input
+- Handles `file://` prefix for file paths
+- Parses JSON configuration files
+- Passes configuration directly to Agent constructor
+
+### ✅ Tool Handling Simplification
+- Removed ToolRegistry usage - Agent handles tools internally
+- Removed DEFAULT_TOOLS concept
+- Pass tools list directly from config to Agent
+- Removed `raise_exception_on_missing_tool` parameter (uses Agent's default behavior)
+
+### ✅ Configuration Handling
+- Only passes non-None config values to Agent constructor
+- Uses Agent's default configs when values not specified
+- Supports all Agent constructor parameters from config
+
+### ✅ Module Exports
+- Updated `src/strands/experimental/__init__.py` to export `config_to_agent`
+- Removed `AgentConfig` class export
+
+### ✅ Tests Update
+- Refactored all tests to use `config_to_agent` function
+- Removed class-based test patterns
+- Updated test imports
+- Simplified tool-related tests (no more ToolRegistry mocking)
+- Updated file handling tests
+- Fixed model attribute access (`agent.model.config["model_id"]`)
+
+### ✅ Cleanup
+- Removed `AgentConfig` class entirely
+- Removed unused imports and dependencies
+- Cleaned up ToolRegistry references
+
+## Final Interface
+
+### Current Interface (Experimental)
+```python
+from strands.experimental import config_to_agent
+
+agent = config_to_agent("/path/to/amazing-agent.json")
+agent("do the thing")
+```
+
+### Example Configuration
+```json
+{
+  "model": "anthropic.claude-3-5-sonnet-20241022-v2:0",
+  "prompt": "You are a helpful assistant",
+  "tools": ["strands_tools.file_read", "my_app.tools.cake_tool", "/path/to/another_tool.py"],
+  "name": "MyAgent",
+  "agent_id": "agent-123"
+}
+```
+
+## All Tasks Complete! ✅
+
+## Notes
+- Preserve support for both file paths and dictionary inputs
+- Let Agent class handle all tool loading and validation
+- Use Agent's default behavior for missing tools/configs
+- Keep the `file://` prefix functionality

src/strands/experimental/agent_config.py

@@ -66,13 +66,6 @@ def config_to_agent(config: Union[str, Dict[str, Any]], **kwargs) -> Agent:
         "prompt": "system_prompt", 
         "tools": "tools",
         "name": "name",
-        "agent_id": "agent_id",
-        "session_manager": "session_manager",
-        "conversation_manager": "conversation_manager",
-        "hooks": "hooks",
-        "callback_handler": "callback_handler",
-        "state": "state",
-        "trace_attributes": "trace_attributes",
     }
 
     # Only include non-None values from config

tests/strands/experimental/test_agent_config.py

@@ -109,12 +109,6 @@ def test_config_to_agent_with_name(self):
         agent = config_to_agent(config)
         assert agent.name == "TestAgent"
 
-    def test_config_to_agent_with_agent_id(self):
-        """Test config_to_agent handles agent_id."""
-        config = {"model": "test-model", "agent_id": "test-agent-123"}
-        agent = config_to_agent(config)
-        assert agent.agent_id == "test-agent-123"
-
     def test_config_to_agent_ignores_none_values(self):
         """Test that None values in config are ignored."""
         config = {"model": "test-model", "prompt": None, "name": None}