Bump version to 1.3.6

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: jwesleye

CHANGELOG.md

@@ -7,6 +7,16 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.3.6] - 2024-12-24
+
+### Fixed
+- **Harmony Auto-Detection** - Fixed harmony processor not auto-detecting gpt-oss models
+  - Now properly calls `HarmonyProcessor.detect_harmony_agent()` during initialization
+  - Enhanced detection to check direct string attributes on agent (model, model_id, model_name)
+  - Updated agent_loader to handle agents where model is stored as direct string attribute
+  - Added comprehensive debug logging to show detection process
+  - Resolves issue where "openai/gpt-oss-120b" models weren't being auto-detected
+
 ## [1.3.5] - 2024-12-24
 
 ### Fixed

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "basic-agent-chat-loop"
-version = "1.3.5"
+version = "1.3.6"
 description = "Feature-rich interactive CLI for AI agents with token tracking, prompt templates, aliases, and configuration"
 readme = "README.md"
 requires-python = ">=3.9"

src/basic_agent_chat_loop/__init__.py

@@ -4,7 +4,7 @@
 agent aliases, and extensive configuration options.
 """
 
-__version__ = "1.3.5"
+__version__ = "1.3.6"
 
 from .chat_config import ChatConfig
 from .chat_loop import ChatLoop

src/basic_agent_chat_loop/chat_loop.py

@@ -603,7 +603,13 @@ def __init__(
         harmony_enabled_config_raw = (
             self.config.get("harmony.enabled", None) if self.config else None
         )
-        uses_harmony = self.agent_metadata.get("uses_harmony", False)
+
+        # Auto-detect harmony support
+        # Check both metadata (for loaded agents) and agent object itself
+        # (for dynamic agents)
+        uses_harmony_metadata = self.agent_metadata.get("uses_harmony", False)
+        uses_harmony_detected = HarmonyProcessor.detect_harmony_agent(self.agent)
+        uses_harmony = uses_harmony_metadata or uses_harmony_detected
 
         # Normalize config value to handle string values from YAML
         # "auto"/None → None (auto-detect)
@@ -615,7 +621,9 @@ def __init__(
 
         logger.debug(f"Harmony config value (raw): {harmony_enabled_config_raw!r}")
         logger.debug(f"Harmony config value (normalized): {harmony_enabled_config!r}")
-        logger.debug(f"Auto-detected harmony: {uses_harmony}")
+        logger.debug(f"Harmony from metadata: {uses_harmony_metadata}")
+        logger.debug(f"Harmony from detection: {uses_harmony_detected}")
+        logger.debug(f"Auto-detected harmony (combined): {uses_harmony}")
 
         # Determine if harmony should be enabled
         # None = auto-detect, True = force enable, False = force disable

src/basic_agent_chat_loop/components/agent_loader.py

@@ -209,19 +209,27 @@ def extract_agent_metadata(agent: Any) -> dict[str, Any]:
     metadata["uses_harmony"] = HarmonyProcessor.detect_harmony_agent(agent)
 
     # Try to extract model information
-    if hasattr(agent, "model"):
-        model = agent.model
+    model_id = None
 
-        # Try multiple attribute names for model ID
-        model_id = None
+    # First check if agent.model, agent.model_id, or agent.model_name are direct strings
+    for attr in ["model", "model_id", "model_name"]:
+        if hasattr(agent, attr):
+            value = getattr(agent, attr)
+            if value and isinstance(value, str):
+                model_id = value
+                break
+
+    # If not found as direct string, check agent.model as an object
+    if not model_id and hasattr(agent, "model"):
+        model = agent.model
 
         # Check for Strands-style config dict first
         if hasattr(model, "config") and isinstance(model.config, dict):
             model_id = model.config.get("model_id")
 
-        # Fall back to checking various attributes
+        # Fall back to checking various attributes on the model object
         if not model_id:
-            for attr in ["model_id", "model", "model_name", "_model_id", "name"]:
+            for attr in ["model_id", "model", "model_name", "_model_id", "name", "id"]:
                 if hasattr(model, attr):
                     model_id = getattr(model, attr)
                     if model_id and model_id != "Unknown":

src/basic_agent_chat_loop/components/harmony_processor.py

@@ -107,17 +107,33 @@ def detect_harmony_agent(agent: Any) -> bool:
 
         # Strategy 3: Check model name/id for gpt-oss or harmony
         model_indicators = []
+
+        # Check direct string attributes on agent
+        for attr in ["model", "model_id", "model_name"]:
+            if hasattr(agent, attr):
+                model_value = getattr(agent, attr)
+                if model_value and isinstance(model_value, str):
+                    model_indicators.append(model_value.lower())
+                    logger.debug(f"Found model string on agent.{attr}: {model_value}")
+
+        # Check nested attributes on agent.model object
         if hasattr(agent, "model"):
             model = agent.model
-            for attr in ["model_id", "model", "model_name", "name"]:
-                if hasattr(model, attr):
-                    model_value = getattr(model, attr)
-                    if model_value:
-                        model_indicators.append(str(model_value).lower())
+            # If model itself is a string, we already captured it above
+            if not isinstance(model, str):
+                for attr in ["model_id", "model", "model_name", "name", "id"]:
+                    if hasattr(model, attr):
+                        model_value = getattr(model, attr)
+                        if model_value:
+                            model_indicators.append(str(model_value).lower())
+                            logger.debug(
+                                f"Found model on agent.model.{attr}: {model_value}"
+                            )
 
+        # Check if any indicator contains gpt-oss or harmony
         for indicator in model_indicators:
             if "gpt-oss" in indicator or "harmony" in indicator:
-                logger.info(f"Agent model contains harmony indicator: {indicator}")
+                logger.info(f"✓ Agent model contains harmony indicator: {indicator}")
                 return True
 
         # Strategy 4: Check agent class name
@@ -132,6 +148,13 @@ def detect_harmony_agent(agent: Any) -> bool:
             logger.info("Agent has harmony-specific methods")
             return True
 
+        # No harmony indicators found
+        logger.debug(
+            f"No harmony indicators found. Checked: "
+            f"attributes (uses_harmony, harmony_encoding), "
+            f"model indicators: {model_indicators}, "
+            f"class: {agent.__class__.__name__}"
+        )
         return False
 
     def process_response(