fix(tests): update test imports for renamed ExecutionResult types

Fixed broken imports after ExecutionResult consolidation:
- streaming/__init__.py: export StreamingExecutionResult
- integration/__init__.py: export ToolExecutionResult
- Updated 6 test files to use correct type names:
  - SafeExecutionResult (from safe_executor)
  - StreamingExecutionResult (from streaming)
  - ToolExecutionResult (from integration)

All 5791 tests now collect without errors.
106 critical tests passing.

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

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

Author: JuanCS-Dev

.qwen/index/index.json

@@ -15,20 +15,20 @@
 
 from .safety import SafetyValidator, safety_validator
 from .session import Session, SessionManager, session_manager
-from .shell_bridge import ShellBridge, ExecutionResult, shell_bridge
+from .shell_bridge import ShellBridge, ToolExecutionResult, shell_bridge
 
 __all__ = [
     # Safety
     "SafetyValidator",
     "safety_validator",
-    
+
     # Session
     "Session",
     "SessionManager",
     "session_manager",
-    
+
     # Shell Bridge
     "ShellBridge",
-    "ExecutionResult",
+    "ToolExecutionResult",
     "shell_bridge",
 ]

jdev_cli/integration/__init__.py

@@ -1,6 +1,6 @@
 """Reactive TUI and async streaming components."""
 
-from .executor import AsyncCommandExecutor, ExecutionResult
+from .executor import AsyncCommandExecutor, StreamingExecutionResult
 from .renderer import (
     ReactiveRenderer,
     ConcurrentRenderer,
@@ -16,7 +16,7 @@
 
 __all__ = [
     'AsyncCommandExecutor',
-    'ExecutionResult',
+    'StreamingExecutionResult',
     'ReactiveRenderer',
     'ConcurrentRenderer',
     'RenderEvent',

jdev_cli/streaming/__init__.py

@@ -5,7 +5,7 @@
 Tests real behavior, not imaginary features.
 
 Implementation details (READ FROM FILE):
-- execute() is async, returns ExecutionResult
+- execute() is async, returns SafeExecutionResult
 - is_command_allowed() returns Tuple[bool, str]
 - Whitelist: pytest, ruff, git, ls, du, pwd, whoami, etc
 - Dangerous patterns: rm, sudo, pipes |, &&, ||, ;, $(), backticks
@@ -18,7 +18,7 @@
 from pathlib import Path
 from jdev_tui.core.safe_executor import (
     SafeCommandExecutor,
-    ExecutionResult,
+    SafeExecutionResult,
     AllowedCommand,
     CommandCategory,
     get_safe_executor,

tests/security/test_safe_executor_real.py

@@ -16,7 +16,7 @@
     SafetyValidator,
     SessionManager,
     ShellBridge,
-    ExecutionResult,
+    ToolExecutionResult,
 )
 from jdev_cli.core.parser import ResponseParser
 

tests/test_phase2_integration.py

@@ -4,7 +4,7 @@
 import asyncio
 from jdev_cli.streaming import (
     AsyncCommandExecutor,
-    ExecutionResult,
+    StreamingExecutionResult,
     ReactiveRenderer,
     RenderEvent,
     RenderEventType,

tests/test_streaming.py

@@ -16,7 +16,7 @@
 
 from jdev_tui.core.safe_executor import (
     SafeCommandExecutor,
-    ExecutionResult,
+    SafeExecutionResult,
     AllowedCommand,
     CommandCategory,
     get_safe_executor,
@@ -266,12 +266,12 @@ def test_get_allowed_commands_by_category_returns_dict(
         assert expected_categories.issubset(set(by_category.keys()))
 
 
-class TestExecutionResultDataclass:
-    """Tests for ExecutionResult dataclass."""
+class TestSafeExecutionResultDataclass:
+    """Tests for SafeExecutionResult dataclass."""
 
     def test_success_result_creation(self) -> None:
         """Test creating a successful result."""
-        result = ExecutionResult(
+        result = SafeExecutionResult(
             success=True,
             exit_code=0,
             stdout="output",
@@ -286,7 +286,7 @@ def test_success_result_creation(self) -> None:
 
     def test_failure_result_creation(self) -> None:
         """Test creating a failure result."""
-        result = ExecutionResult(
+        result = SafeExecutionResult(
             success=False,
             exit_code=1,
             stdout="",

tests/unit/core/test_safe_executor.py

@@ -31,7 +31,7 @@
 
 from jdev_tui.core.safe_executor import (
     SafeCommandExecutor,
-    ExecutionResult,
+    SafeExecutionResult,
     AllowedCommand,
     CommandCategory,
     get_safe_executor,

tests/unit/core/test_safe_executor_coverage.py

@@ -16,7 +16,7 @@
 from jdev_tui.core.safe_executor import (
     SafeCommandExecutor,
     AllowedCommand,
-    ExecutionResult,
+    SafeExecutionResult,
     CommandCategory,
 )
 
@@ -337,7 +337,7 @@ class TestExecutionResultProperties:
     @settings(max_examples=50)
     def test_execution_result_creation(self, success, exit_code, stdout, stderr, command):
         """Property: ExecutionResult handles various inputs."""
-        result = ExecutionResult(
+        result = SafeExecutionResult(
             success=success,
             exit_code=exit_code,
             stdout=stdout,