ouroboros-ai 0.26.3.dev2__tar.gz → 0.26.3.dev3__tar.gz

{ouroboros_ai-0.26.3.dev2 → ouroboros_ai-0.26.3.dev3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ouroboros-ai
-Version: 0.26.3.dev2
+Version: 0.26.3.dev3
 Summary: Specification-first workflow engine for AI coding agents. Works with Claude Code and Codex CLI.
 Author-email: Q00 <jqyu.lee@gmail.com>
 License-File: LICENSE

{ouroboros_ai-0.26.3.dev2 → ouroboros_ai-0.26.3.dev3}/src/ouroboros/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.26.3.dev2'
-__version_tuple__ = version_tuple = (0, 26, 3, 'dev2')
+__version__ = version = '0.26.3.dev3'
+__version_tuple__ = version_tuple = (0, 26, 3, 'dev3')
 
 __commit_id__ = commit_id = None

{ouroboros_ai-0.26.3.dev2 → ouroboros_ai-0.26.3.dev3}/src/ouroboros/mcp/tools/evaluation_handlers.py

@@ -685,7 +685,7 @@ class LateralThinkHandler:
             )
 
         persona_str = arguments.get("persona", "contrarian")
-        failed_attempts_raw = arguments.get("failed_attempts", [])
+        failed_attempts_raw = arguments.get("failed_attempts") or []
 
         # Convert string to ThinkingPersona enum
         try:

{ouroboros_ai-0.26.3.dev2 → ouroboros_ai-0.26.3.dev3}/tests/unit/mcp/tools/test_definitions.py

@@ -44,6 +44,7 @@ from ouroboros.orchestrator.adapter import (
     DELEGATED_PARENT_SESSION_ID_ARG,
 )
 from ouroboros.orchestrator.session import SessionTracker
+from ouroboros.resilience.lateral import ThinkingPersona
 
 
 def create_mock_live_ambiguity_score(
@@ -1015,6 +1016,77 @@ metadata:
 """
 
 
+class TestLateralThinkHandler:
+    """Test LateralThinkHandler argument normalization."""
+
+    async def test_handle_treats_null_failed_attempts_as_empty(self) -> None:
+        """Explicit null from MCP clients should behave like an omitted optional array."""
+        handler = LateralThinkHandler()
+
+        mock_lateral_result = MagicMock(
+            approach_summary="Try a different angle",
+            prompt="Consider an alternative path",
+            questions=("What assumption can you invert?",),
+            persona=MagicMock(value="contrarian"),
+        )
+        mock_thinker = MagicMock()
+        mock_thinker.generate_alternative.return_value = Result.ok(mock_lateral_result)
+
+        with patch(
+            "ouroboros.resilience.lateral.LateralThinker",
+            return_value=mock_thinker,
+        ):
+            result = await handler.handle(
+                {
+                    "problem_context": "tool crashes when optional arg is null",
+                    "current_approach": "call ouroboros_lateral_think without failed_attempts",
+                    "failed_attempts": None,
+                }
+            )
+
+        assert result.is_ok
+        mock_thinker.generate_alternative.assert_called_once_with(
+            persona=ThinkingPersona.CONTRARIAN,
+            problem_context="tool crashes when optional arg is null",
+            current_approach="call ouroboros_lateral_think without failed_attempts",
+            failed_attempts=(),
+        )
+
+    async def test_handle_filters_falsey_failed_attempts_entries(self) -> None:
+        """Falsy entries should be dropped while valid entries are stringified."""
+        handler = LateralThinkHandler()
+
+        mock_lateral_result = MagicMock(
+            approach_summary="Try a different angle",
+            prompt="Consider an alternative path",
+            questions=("What assumption can you invert?",),
+            persona=MagicMock(value="architect"),
+        )
+        mock_thinker = MagicMock()
+        mock_thinker.generate_alternative.return_value = Result.ok(mock_lateral_result)
+
+        with patch(
+            "ouroboros.resilience.lateral.LateralThinker",
+            return_value=mock_thinker,
+        ):
+            result = await handler.handle(
+                {
+                    "problem_context": "problem",
+                    "current_approach": "approach",
+                    "persona": "architect",
+                    "failed_attempts": ["first", None, "", 7],
+                }
+            )
+
+        assert result.is_ok
+        mock_thinker.generate_alternative.assert_called_once_with(
+            persona=ThinkingPersona.ARCHITECT,
+            problem_context="problem",
+            current_approach="approach",
+            failed_attempts=("first", "7"),
+        )
+
+
 class TestMeasureDriftHandler:
     """Test MeasureDriftHandler class."""