Merge pull request #9 from Krigsexe/claude/refactor-ai-context-repo-01RAdfZ29yZdtD3CQQHCTL3Z

Claude/refactor ai context repo 01 r adf z29y zdt d3 cqqhctl3 z

Author: Krigsexe

.odin/AI_CHECKPOINT.json

@@ -7,7 +7,7 @@
     "file_count": 34,
     "sih_root": -5174629986412216191
   },
-  "last_backup": "/workspace/.odin/backups/20250824-013204",
+  "last_backup": "/home/user/AI-Context-Engineering/.odin/backups/20251125-142938",
   "sih_root": 5740354900026072187,
   "risk_profile": "low",
   "commit": null,

agents/cli.py

@@ -11,7 +11,6 @@
 import json
 import logging
 import os
-import sys
 from pathlib import Path
 from typing import Optional
 
@@ -125,7 +124,7 @@ def cmd_list(args):
             temp = object.__new__(agent_class)
             try:
                 desc = agent_class.description.fget(temp)
-            except:
+            except Exception:
                 desc = "No description"
             print(f"  {agent_name:20} - {desc}")
 

agents/cognitive/review.py

@@ -289,8 +289,8 @@ async def _assess_maintainability(
         # Calculate basic metrics
         lines = code.split('\n')
         total_lines = len(lines)
-        code_lines = len([l for l in lines if l.strip() and not l.strip().startswith('#')])
-        comment_lines = len([l for l in lines if l.strip().startswith('#')])
+        code_lines = len([line for line in lines if line.strip() and not line.strip().startswith('#')])
+        comment_lines = len([line for line in lines if line.strip().startswith('#')])
 
         # Function/class count (basic)
         function_count = len(re.findall(r'\bdef\s+\w+', code))

agents/execution/security.py

@@ -257,7 +257,8 @@ async def _security_review(
 
         code = input_data.get("code", "")
         language = input_data.get("language", "python")
-        security_requirements = input_data.get("security_requirements", [])
+        # Reserved for future use with custom security requirement checks
+        _security_requirements = input_data.get("security_requirements", [])
 
         if not code:
             return AgentResult(

agents/shared/base_agent.py

@@ -6,12 +6,10 @@
 # =============================================================================
 
 from __future__ import annotations
-import asyncio
 import logging
 import uuid
 from abc import ABC, abstractmethod
 from dataclasses import dataclass, field
-from datetime import datetime
 from enum import Enum
 from typing import Any, Callable, Dict, List, Optional, Type
 

agents/shared/integrity.py

@@ -62,8 +62,8 @@ def normalize_python_ast(source: str) -> str:
 
     Preserving only structural semantics.
     """
-    source = textwrap.dedent(source)
-    source = re.sub(r'^\s+', '', source, flags=re.MULTILINE)
+    # Only dedent and strip leading/trailing whitespace, preserving internal indentation
+    source = textwrap.dedent(source).strip()
     tree = ast.parse(source)
     return ast.dump(tree, annotate_fields=True, include_attributes=False)
 

agents/shared/llm_client.py

@@ -6,7 +6,7 @@
 
 import os
 import logging
-from typing import List, Dict, Any, Optional
+from typing import List, Dict, Any
 
 from .providers import (
     BaseLLMProvider,

agents/shared/message_bus.py

@@ -10,7 +10,7 @@
 import time
 import uuid
 import logging
-from dataclasses import dataclass, field, asdict
+from dataclasses import dataclass, field
 from typing import Any, Callable, Dict, List, Optional, AsyncIterator
 from enum import Enum
 

agents/shared/providers/anthropic.py

@@ -24,8 +24,8 @@ class AnthropicProvider(BaseLLMProvider):
 
     def __init__(
         self,
-        api_key: str = None,
-        model: str = None,
+        api_key: Optional[str] = None,
+        model: Optional[str] = None,
         timeout: int = 120
     ):
         self.api_key = api_key or os.getenv("ANTHROPIC_API_KEY")

agents/shared/providers/base.py

@@ -44,7 +44,7 @@ def generate(
         temperature: float = 0.2,
         max_tokens: int = 4096,
         stop: Optional[List[str]] = None,
-        **kwargs
+        **kwargs: Any
     ) -> LLMResponse:
         """
         Generate a response from the LLM.
@@ -91,7 +91,7 @@ def generate_with_retry(
         self,
         messages: List[Message],
         max_retries: int = 3,
-        **kwargs
+        **kwargs: Any
     ) -> LLMResponse:
         """
         Generate with automatic retry on failure.
@@ -107,15 +107,17 @@ def generate_with_retry(
         Raises:
             Exception: If all retries fail
         """
-        last_error = None
+        last_error: Optional[Exception] = None
         for attempt in range(max_retries):
             try:
                 return self.generate(messages, **kwargs)
             except Exception as e:
                 last_error = e
                 if attempt < max_retries - 1:
                     time.sleep(2 ** attempt)  # Exponential backoff
-        raise last_error
+        if last_error is not None:
+            raise last_error
+        raise RuntimeError("No retries attempted")
 
     def count_tokens(self, text: str) -> int:
         """