Fixed circular imports by moving StrategyResult to base

reasoning/__init__.py

@@ -26,45 +26,69 @@ Learning & Adaptation:
 16. Monetization Strategies
 """
 
-from .base import ReasoningStrategy
+# Base types and classes
+from .base import ReasoningStrategy, StrategyResult
+
+# Core reasoning strategies
+from .chain_of_thought import ChainOfThoughtStrategy
+from .tree_of_thoughts import TreeOfThoughtsStrategy
+from .recursive import RecursiveReasoning
+from .analogical import AnalogicalReasoning
+from .meta_learning import MetaLearningStrategy
+from .local_llm import LocalLLMStrategy
+
+# Advanced reasoning strategies
 from .multimodal import MultiModalReasoning
 from .bayesian import BayesianReasoning
 from .quantum import QuantumReasoning
 from .neurosymbolic import NeurosymbolicReasoning
 from .emergent import EmergentReasoning
-from .meta_learning import MetaLearningStrategy
-from .chain_of_thought import ChainOfThoughtStrategy
-from .tree_of_thoughts import TreeOfThoughtsStrategy
-from .recursive import RecursiveReasoning
-from .analogical import AnalogicalReasoning
 from .specialized import SpecializedReasoning
-from .local_llm import LocalLLMStrategy
+
+# Domain-specific strategies
 from .market_analysis import MarketAnalysisStrategy
 from .portfolio_optimization import PortfolioOptimizationStrategy
 from .venture_strategies import VentureStrategy
 from .monetization import MonetizationStrategy
-from .unified_engine import UnifiedReasoningEngine, StrategyType, StrategyResult, UnifiedResult
+
+# Model integrations
+from .groq_strategy import GroqStrategy
+
+# Unified engine
+from .unified_engine import UnifiedReasoningEngine, StrategyType, UnifiedResult
 
 __all__ = [
+    # Base types
     'ReasoningStrategy',
+    'StrategyResult',
+    
+    # Core reasoning
+    'ChainOfThoughtStrategy',
+    'TreeOfThoughtsStrategy',
+    'RecursiveReasoning',
+    'AnalogicalReasoning',
+    'MetaLearningStrategy',
+    'LocalLLMStrategy',
+    
+    # Advanced reasoning
     'MultiModalReasoning',
     'BayesianReasoning',
     'QuantumReasoning',
     'NeurosymbolicReasoning',
     'EmergentReasoning',
-    'MetaLearningStrategy',
-    'ChainOfThoughtStrategy',
-    'TreeOfThoughtsStrategy',
-    'RecursiveReasoning',
-    'AnalogicalReasoning',
     'SpecializedReasoning',
-    'LocalLLMStrategy',
+    
+    # Domain-specific
     'MarketAnalysisStrategy',
     'PortfolioOptimizationStrategy',
     'VentureStrategy',
     'MonetizationStrategy',
+    
+    # Model integrations
+    'GroqStrategy',
+    
+    # Unified engine
     'UnifiedReasoningEngine',
     'StrategyType',
-    'StrategyResult',
     'UnifiedResult'
 ]

reasoning/unified_engine.py

@@ -10,7 +10,7 @@ import asyncio
 from collections import defaultdict
 import numpy as np
 
-from .base import ReasoningStrategy
+from .base import ReasoningStrategy, StrategyResult
 from .groq_strategy import GroqStrategy
 from .chain_of_thought import ChainOfThoughtStrategy
 from .tree_of_thoughts import TreeOfThoughtsStrategy
@@ -60,18 +60,6 @@ class StrategyType(str, Enum):
     VENTURE = "venture"
     VENTURE_TYPE = "venture_type"
 
-@dataclass
-class StrategyResult:
-    """Result from a reasoning strategy."""
-    strategy_type: StrategyType
-    success: bool
-    answer: Optional[str]
-    confidence: float
-    reasoning_trace: List[Dict[str, Any]]
-    metadata: Dict[str, Any]
-    performance_metrics: Dict[str, Any]
-    timestamp: datetime = field(default_factory=datetime.now)
-
 @dataclass
 class UnifiedResult:
     """Combined result from multiple strategies."""