newcode 0.1.5__tar.gz → 0.1.7__tar.gz

{newcode-0.1.5 → newcode-0.1.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: newcode
-Version: 0.1.5
+Version: 0.1.7
 Summary: AI-powered code generation agent - a fedstew fork of code-puppy
 Project-URL: repository, https://github.com/janfeddersen-wq/new_code
 Project-URL: HomePage, https://github.com/janfeddersen-wq/new_code

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/__init__.py

@@ -2,7 +2,7 @@ import importlib.metadata
 
 # Version detection
 try:
-    _detected_version = importlib.metadata.version("code-puppy")
+    _detected_version = importlib.metadata.version("newcode")
     # Ensure we never end up with None or empty string
     __version__ = _detected_version if _detected_version else "0.0.0-dev"
 except Exception:

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/agents/agent_creator_agent.py

@@ -50,7 +50,7 @@ class AgentCreatorAgent(BaseAgent):
             uc_tools_section = "\n".join(uc_tools_info)
         else:
             uc_tools_section = (
-                "No custom UC tools created yet. Use Helios to create some!"
+                "No custom UC tools created yet."
             )
 
         # Load available models dynamically
@@ -163,7 +163,7 @@ Users can optionally pin a specific model to their agent to override the global
 - `invoke_agent` - Invoke other agents with specific prompts (recommended for agent managers)
 
 ### 🔧 **Universal Constructor Tools** (custom tools):
-- These are tools created by Helios or via the Universal Constructor
+- These are tools created via the Universal Constructor
 - They persist across sessions and can be bound to any agent
 - Use `universal_constructor(action="list")` to see available custom tools
 - Bind them by adding their full name to the agent's tools array

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/config.py

@@ -104,7 +104,7 @@ PACK_AGENT_NAMES = frozenset(
 )
 
 # Agents that require Universal Constructor to be enabled
-UC_AGENT_NAMES = frozenset(["helios"])
+UC_AGENT_NAMES = frozenset()
 
 
 def get_pack_agents_enabled() -> bool:
@@ -1469,8 +1469,6 @@ DEFAULT_BANNER_COLORS = {
     "universal_constructor": "dark_cyan",  # Teal - constructing tools
     # Browser/Terminal tools - same color as edit_file (gold)
     "terminal_tool": "dark_goldenrod",  # Gold - browser terminal operations
-    # MCP tools - distinct from builtin tools
-    "mcp_tool_call": "dark_cyan",  # Teal - external MCP tool calls
 }
 
 

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/hook_engine/__init__.py

@@ -1,8 +1,14 @@
 """Hook engine package."""
 
-from .engine import HookEngine
-from .models import HookConfig, EventData, ExecutionResult, ProcessEventResult, HookRegistry
 from . import aliases
+from .engine import HookEngine
+from .models import (
+    EventData,
+    ExecutionResult,
+    HookConfig,
+    HookRegistry,
+    ProcessEventResult,
+)
 
 __all__ = [
     "HookEngine",

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/hook_engine/aliases.py

@@ -19,30 +19,29 @@ Adding a new provider
 
 from typing import Dict, FrozenSet, Optional
 
-
 # ---------------------------------------------------------------------------
 # Claude Code  (Anthropic)
 # Source: `claude mcp serve` → tools/list  (verified against v2.1.52)
 # ---------------------------------------------------------------------------
 CLAUDE_CODE_ALIASES: Dict[str, str] = {
     # Shell execution
-    "Bash":             "agent_run_shell_command",
+    "Bash": "agent_run_shell_command",
     # File system — read
-    "Glob":             "list_files",
-    "Read":             "read_file",
-    "Grep":             "grep",
+    "Glob": "list_files",
+    "Read": "read_file",
+    "Grep": "grep",
     # File system — write
-    "Edit":             "edit_file",
-    "Write":            "edit_file",    # Write = full overwrite; same internal tool
+    "Edit": "edit_file",
+    "Write": "edit_file",  # Write = full overwrite; same tool in puppy
     # File system — delete
-    "Delete":           "delete_file",
+    "Delete": "delete_file",
     # User interaction
-    "AskUserQuestion":  "ask_user_question",
+    "AskUserQuestion": "ask_user_question",
     # Agent / task orchestration
-    "Task":             "invoke_agent",
+    "Task": "invoke_agent",
     # Skills
-    "Skill":            "activate_skill",
-    "ToolSearch":       "list_or_search_skills",
+    "Skill": "activate_skill",
+    "ToolSearch": "list_or_search_skills",
     # NOTE: the tools below have no direct code_puppy equivalent yet.
     # They are listed here for documentation and future mapping:
     #   "TaskOutput"     -> (no equivalent)
@@ -92,10 +91,10 @@ SWARM_ALIASES: Dict[str, str] = {
 # To disable a provider's aliases, remove its entry from this dict.
 # ---------------------------------------------------------------------------
 PROVIDER_ALIASES: Dict[str, Dict[str, str]] = {
-    "claude":  CLAUDE_CODE_ALIASES,
-    "gemini":  GEMINI_ALIASES,   # placeholder — empty until populated
-    "codex":   CODEX_ALIASES,    # placeholder — empty until populated
-    "swarm":   SWARM_ALIASES,    # placeholder — empty until populated
+    "claude": CLAUDE_CODE_ALIASES,
+    "gemini": GEMINI_ALIASES,  # placeholder — empty until populated
+    "codex": CODEX_ALIASES,  # placeholder — empty until populated
+    "swarm": SWARM_ALIASES,  # placeholder — empty until populated
 }
 
 
@@ -103,6 +102,7 @@ PROVIDER_ALIASES: Dict[str, Dict[str, str]] = {
 # Flattened lookup structures — built once at import time for O(1) access.
 # ---------------------------------------------------------------------------
 
+
 def _build_lookup() -> Dict[str, FrozenSet[str]]:
     """
     Return a dict mapping every known name (provider *and* internal) to the
@@ -152,4 +152,4 @@ def resolve_internal_name(provider_tool_name: str) -> Optional[str]:
         for pname, internal in provider_aliases.items():
             if pname.lower() == provider_tool_name.lower():
                 return internal
-    return None
+    return None

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/hook_engine/engine.py

@@ -6,17 +6,20 @@ import logging
 import time
 from typing import Any, Dict, List, Optional
 
+from .executor import execute_hooks_sequential, get_blocking_result
+from .matcher import matches
 from .models import (
     EventData,
-    ExecutionResult,
     HookConfig,
     HookRegistry,
     ProcessEventResult,
 )
-from .matcher import matches
-from .executor import execute_hook, execute_hooks_sequential, get_blocking_result
 from .registry import build_registry_from_config, get_registry_stats
-from .validator import validate_hooks_config, format_validation_report, get_config_suggestions
+from .validator import (
+    format_validation_report,
+    get_config_suggestions,
+    validate_hooks_config,
+)
 
 logger = logging.getLogger(__name__)
 
@@ -61,7 +64,9 @@ class HookEngine:
 
         try:
             self._registry = build_registry_from_config(config)
-            logger.info(f"Loaded hook configuration: {self._registry.count_hooks()} total hooks")
+            logger.info(
+                f"Loaded hook configuration: {self._registry.count_hooks()} total hooks"
+            )
         except Exception as e:
             if self.strict_validation:
                 raise ValueError(f"Failed to build hook registry: {e}") from e
@@ -83,21 +88,37 @@ class HookEngine:
         start_time = time.perf_counter()
 
         if not self._registry:
-            return ProcessEventResult(blocked=False, executed_hooks=0, results=[], total_duration_ms=0.0)
+            return ProcessEventResult(
+                blocked=False, executed_hooks=0, results=[], total_duration_ms=0.0
+            )
 
         all_hooks = self._registry.get_hooks_for_event(event_type)
 
         if not all_hooks:
             duration_ms = (time.perf_counter() - start_time) * 1000
-            return ProcessEventResult(blocked=False, executed_hooks=0, results=[], total_duration_ms=duration_ms)
+            return ProcessEventResult(
+                blocked=False,
+                executed_hooks=0,
+                results=[],
+                total_duration_ms=duration_ms,
+            )
 
-        matching_hooks = self._filter_hooks_by_matcher(all_hooks, event_data.tool_name, event_data.tool_args)
+        matching_hooks = self._filter_hooks_by_matcher(
+            all_hooks, event_data.tool_name, event_data.tool_args
+        )
 
         if not matching_hooks:
             duration_ms = (time.perf_counter() - start_time) * 1000
-            return ProcessEventResult(blocked=False, executed_hooks=0, results=[], total_duration_ms=duration_ms)
+            return ProcessEventResult(
+                blocked=False,
+                executed_hooks=0,
+                results=[],
+                total_duration_ms=duration_ms,
+            )
 
-        logger.debug(f"Processing {event_type}: {len(matching_hooks)} matching hook(s) for tool '{event_data.tool_name}'")
+        logger.debug(
+            f"Processing {event_type}: {len(matching_hooks)} matching hook(s) for tool '{event_data.tool_name}'"
+        )
 
         if sequential:
             results = await execute_hooks_sequential(
@@ -105,7 +126,10 @@ class HookEngine:
             )
         else:
             from .executor import execute_hooks_parallel
-            results = await execute_hooks_parallel(matching_hooks, event_data, self.env_vars)
+
+            results = await execute_hooks_parallel(
+                matching_hooks, event_data, self.env_vars
+            )
 
         for hook, result in zip(matching_hooks, results):
             if hook.once and result.success:
@@ -142,7 +166,9 @@ class HookEngine:
                 if matches(hook.matcher, tool_name, tool_args):
                     matching_hooks.append(hook)
             except Exception as e:
-                logger.error(f"Error matching hook '{hook.matcher}': {e}", exc_info=True)
+                logger.error(
+                    f"Error matching hook '{hook.matcher}': {e}", exc_info=True
+                )
         return matching_hooks
 
     def get_stats(self) -> Dict[str, Any]:

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/hook_engine/executor.py

@@ -20,8 +20,8 @@ import re
 import time
 from typing import Any, Dict, List, Optional
 
-from .models import HookConfig, EventData, ExecutionResult
 from .matcher import _extract_file_path
+from .models import EventData, ExecutionResult, HookConfig
 
 logger = logging.getLogger(__name__)
 
@@ -40,6 +40,7 @@ def _build_stdin_payload(event_data: EventData) -> bytes:
         "permission_mode": "default"
     }
     """
+
     def _make_serializable(obj: Any) -> Any:
         if isinstance(obj, dict):
             return {k: _make_serializable(v) for k, v in obj.items()}
@@ -192,7 +193,7 @@ def _substitute_variables(
     result = command
     for var, value in substitutions.items():
         result = result.replace(f"${{{var}}}", str(value))
-        result = re.sub(rf'\${re.escape(var)}(?=\W|$)', str(value), result)
+        result = re.sub(rf"\${re.escape(var)}(?=\W|$)", str(value), result)
     return result
 
 
@@ -228,16 +229,18 @@ async def execute_hooks_parallel(
     final_results = []
     for i, result in enumerate(results):
         if isinstance(result, Exception):
-            final_results.append(ExecutionResult(
-                blocked=False,
-                hook_command=hooks[i].command,
-                stdout="",
-                stderr=str(result),
-                exit_code=-1,
-                duration_ms=0.0,
-                error=f"Hook execution failed: {result}",
-                hook_id=hooks[i].id,
-            ))
+            final_results.append(
+                ExecutionResult(
+                    blocked=False,
+                    hook_command=hooks[i].command,
+                    stdout="",
+                    stderr=str(result),
+                    exit_code=-1,
+                    duration_ms=0.0,
+                    error=f"Hook execution failed: {result}",
+                    hook_id=hooks[i].id,
+                )
+            )
         else:
             final_results.append(result)
     return final_results

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/hook_engine/matcher.py

@@ -79,14 +79,25 @@ def _match_single(pattern: str, tool_name: str, tool_args: Dict[str, Any]) -> bo
 
 
 def _extract_file_path(tool_args: Dict[str, Any]) -> Optional[str]:
-    file_keys = ["file_path", "file", "path", "target", "input_file",
-                 "output_file", "source", "destination", "src", "dest", "filename"]
+    file_keys = [
+        "file_path",
+        "file",
+        "path",
+        "target",
+        "input_file",
+        "output_file",
+        "source",
+        "destination",
+        "src",
+        "dest",
+        "filename",
+    ]
     for key in file_keys:
         if key in tool_args:
             value = tool_args[key]
             if isinstance(value, str):
                 return value
-            if hasattr(value, '__fspath__'):
+            if hasattr(value, "__fspath__"):
                 return str(value)
     for value in tool_args.values():
         if isinstance(value, str) and _looks_like_file_path(value):
@@ -107,7 +118,7 @@ def _looks_like_file_path(value: str) -> bool:
 
 
 def _is_regex_pattern(pattern: str) -> bool:
-    regex_chars = ['^', '$', '.', '+', '?', '[', ']', '(', ')', '{', '}', '|', '\\']
+    regex_chars = ["^", "$", ".", "+", "?", "[", "]", "(", ")", "{", "}", "|", "\\"]
     return any(char in pattern for char in regex_chars)
 
 

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/hook_engine/models.py

@@ -23,6 +23,7 @@ class HookConfig:
         enabled: Whether this hook is enabled (default: True)
         id: Optional unique identifier for this hook
     """
+
     matcher: str
     type: Literal["command", "prompt"]
     command: str
@@ -37,7 +38,9 @@ class HookConfig:
             raise ValueError("Hook matcher cannot be empty")
 
         if self.type not in ("command", "prompt"):
-            raise ValueError(f"Hook type must be 'command' or 'prompt', got: {self.type}")
+            raise ValueError(
+                f"Hook type must be 'command' or 'prompt', got: {self.type}"
+            )
 
         if not self.command:
             raise ValueError("Hook command cannot be empty")
@@ -47,6 +50,7 @@ class HookConfig:
 
         if self.id is None:
             import hashlib
+
             content = f"{self.matcher}:{self.type}:{self.command}"
             self.id = hashlib.sha256(content.encode()).hexdigest()[:12]
 
@@ -62,6 +66,7 @@ class EventData:
         tool_args: Arguments passed to the tool
         context: Optional context metadata (result, duration, etc.)
     """
+
     event_type: str
     tool_name: str
     tool_args: Dict[str, Any] = field(default_factory=dict)
@@ -89,6 +94,7 @@ class ExecutionResult:
         error: Error message if execution failed
         hook_id: ID of the hook that was executed
     """
+
     blocked: bool
     hook_command: str
     stdout: str = ""
@@ -117,6 +123,7 @@ class ExecutionResult:
 @dataclass
 class HookGroup:
     """A group of hooks that share the same matcher."""
+
     matcher: str
     hooks: List[HookConfig] = field(default_factory=list)
 
@@ -128,6 +135,7 @@ class HookGroup:
 @dataclass
 class HookRegistry:
     """Registry of all hooks organized by event type."""
+
     pre_tool_use: List[HookConfig] = field(default_factory=list)
     post_tool_use: List[HookConfig] = field(default_factory=list)
     session_start: List[HookConfig] = field(default_factory=list)
@@ -163,8 +171,9 @@ class HookRegistry:
     @staticmethod
     def _normalize_event_type(event_type: str) -> str:
         import re
-        s1 = re.sub('(.)([A-Z][a-z]+)', r'\1_\2', event_type)
-        return re.sub('([a-z0-9])([A-Z])', r'\1_\2', s1).lower()
+
+        s1 = re.sub("(.)([A-Z][a-z]+)", r"\1_\2", event_type)
+        return re.sub("([a-z0-9])([A-Z])", r"\1_\2", s1).lower()
 
     def add_hook(self, event_type: str, hook: HookConfig) -> None:
         attr_name = self._normalize_event_type(event_type)
@@ -186,9 +195,17 @@ class HookRegistry:
     def count_hooks(self, event_type: Optional[str] = None) -> int:
         if event_type is None:
             total = 0
-            for attr in ['pre_tool_use', 'post_tool_use', 'session_start',
-                         'session_end', 'pre_compact', 'user_prompt_submit', 'notification',
-                         'stop', 'subagent_stop']:
+            for attr in [
+                "pre_tool_use",
+                "post_tool_use",
+                "session_start",
+                "session_end",
+                "pre_compact",
+                "user_prompt_submit",
+                "notification",
+                "stop",
+                "subagent_stop",
+            ]:
                 total += len(getattr(self, attr))
             return total
         attr_name = self._normalize_event_type(event_type)
@@ -200,6 +217,7 @@ class HookRegistry:
 @dataclass
 class ProcessEventResult:
     """Result from processing an event through the hook engine."""
+
     blocked: bool
     executed_hooks: int
     results: List[ExecutionResult]

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/hook_engine/registry.py

@@ -6,7 +6,7 @@ Builds and manages the HookRegistry from configuration dictionaries.
 
 import logging
 import re
-from typing import Any, Dict, List, Optional
+from typing import Any, Dict
 
 from .models import HookConfig, HookRegistry
 
@@ -86,8 +86,8 @@ def get_registry_stats(registry: HookRegistry) -> Dict[str, Any]:
     }
 
     def _to_attr(event_type: str) -> str:
-        s1 = re.sub('(.)([A-Z][a-z]+)', r'\1_\2', event_type)
-        return re.sub('([a-z0-9])([A-Z])', r'\1_\2', s1).lower()
+        s1 = re.sub("(.)([A-Z][a-z]+)", r"\1_\2", event_type)
+        return re.sub("([a-z0-9])([A-Z])", r"\1_\2", s1).lower()
 
     for event_type in SUPPORTED_EVENT_TYPES:
         attr = _to_attr(event_type)

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/hook_engine/validator.py

@@ -53,7 +53,9 @@ def validate_hooks_config(config: Dict[str, Any]) -> Tuple[bool, List[str]]:
 
         for i, group in enumerate(hook_groups):
             if not isinstance(group, dict):
-                errors.append(f"'{event_type}[{i}]' must be a dict with 'matcher' and 'hooks'")
+                errors.append(
+                    f"'{event_type}[{i}]' must be a dict with 'matcher' and 'hooks'"
+                )
                 continue
 
             if "matcher" not in group:
@@ -74,8 +76,9 @@ def validate_hooks_config(config: Dict[str, Any]) -> Tuple[bool, List[str]]:
     return len(errors) == 0, errors
 
 
-def _validate_hook(event_type: str, group_idx: int, hook_idx: int,
-                   hook: Any) -> List[str]:
+def _validate_hook(
+    event_type: str, group_idx: int, hook_idx: int, hook: Any
+) -> List[str]:
     errors: List[str] = []
     prefix = f"'{event_type}[{group_idx}].hooks[{hook_idx}]'"
 
@@ -86,7 +89,9 @@ def _validate_hook(event_type: str, group_idx: int, hook_idx: int,
     if not hook_type:
         errors.append(f"{prefix} missing required field 'type'")
     elif hook_type not in VALID_HOOK_TYPES:
-        errors.append(f"{prefix} invalid type '{hook_type}'. Must be one of: {', '.join(VALID_HOOK_TYPES)}")
+        errors.append(
+            f"{prefix} invalid type '{hook_type}'. Must be one of: {', '.join(VALID_HOOK_TYPES)}"
+        )
 
     if hook_type == "command" and not hook.get("command"):
         errors.append(f"{prefix} missing required field 'command' for type 'command'")
@@ -103,8 +108,9 @@ def _validate_hook(event_type: str, group_idx: int, hook_idx: int,
     return errors
 
 
-def format_validation_report(is_valid: bool, errors: List[str],
-                              suggestions: Optional[List[str]] = None) -> str:
+def format_validation_report(
+    is_valid: bool, errors: List[str], suggestions: Optional[List[str]] = None
+) -> str:
     lines = []
     if is_valid:
         lines.append("✓ Configuration is valid")
@@ -121,15 +127,12 @@ def format_validation_report(is_valid: bool, errors: List[str],
     return "\n".join(lines)
 
 
-def get_config_suggestions(config: Dict[str, Any],
-                           errors: List[str]) -> List[str]:
+def get_config_suggestions(config: Dict[str, Any], errors: List[str]) -> List[str]:
     suggestions: List[str] = []
 
     for error in errors:
         if "Unknown event type" in error:
-            suggestions.append(
-                "Valid event types are: " + ", ".join(VALID_EVENT_TYPES)
-            )
+            suggestions.append("Valid event types are: " + ", ".join(VALID_EVENT_TYPES))
             break
 
     if any("missing required field 'command'" in e for e in errors):

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/mcp_/managed_server.py

@@ -5,6 +5,7 @@ This module provides a managed wrapper around pydantic-ai MCP server classes
 that adds management capabilities while maintaining 100% compatibility.
 """
 
+import json
 import os
 import uuid
 from dataclasses import dataclass, field
@@ -24,6 +25,7 @@ from pydantic_ai.mcp import (
 
 from code_puppy.http_utils import create_async_client
 from code_puppy.mcp_.blocking_startup import BlockingMCPServerStdio
+from code_puppy.messaging import emit_info
 
 
 def _expand_env_vars(value: Any) -> Any:
@@ -80,14 +82,13 @@ async def process_tool_call(
     tool_args: dict[str, Any],
 ) -> ToolResult:
     """A tool call processor that passes along the deps."""
-    from rich.console import Console
-
-    from code_puppy.config import get_banner_color
-
-    console = Console()
-    color = get_banner_color("mcp_tool_call")
-    banner = f"[{color}]───[/{color}] [bold]MCP TOOL CALL[/bold] [{color}]───[/{color}]"
-    console.print(f"\n{banner} [bold cyan]{name}[/bold cyan]")
+    group_id = uuid.uuid4()
+    emit_info(
+        f"\nMCP Tool Call - {name}",
+        message_group=group_id,
+    )
+    emit_info("\nArgs:", message_group=group_id)
+    emit_info(json.dumps(tool_args, indent=2), message_group=group_id)
     return await call_tool(name, tool_args, {"deps": ctx.deps})
 
 

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/messaging/rich_renderer.py

@@ -480,7 +480,7 @@ class RichConsoleRenderer:
                     count += get_recursive_file_count(sub)
                 return count
 
-            indent = "    " * depth
+            indent = "    " * (depth + 1)
 
             # For root level, just show contents
             if dir_path == root_key:

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/plugins/agent_skills/skills_install_menu.py

@@ -586,8 +586,8 @@ class SkillsInstallMenu:
                 import termios
 
                 termios.tcflush(sys.stdin.fileno(), termios.TCIFLUSH)
-            except (ImportError, termios.error, OSError):
-                pass  # Windows or not a tty
+            except Exception:
+                pass  # ImportError on Windows, termios.error, or not a tty
 
             # Small delay to let terminal settle before any output
             time.sleep(0.1)

{newcode-0.1.5 → newcode-0.1.7}/code_puppy/plugins/agent_skills/skills_menu.py

@@ -468,8 +468,8 @@ class SkillsMenu:
                 import termios
 
                 termios.tcflush(sys.stdin.fileno(), termios.TCIFLUSH)
-            except (ImportError, termios.error, OSError):
-                pass  # Windows or not a tty
+            except Exception:
+                pass  # ImportError on Windows, termios.error, or not a tty
 
             # Small delay to let terminal settle before any output
             time.sleep(0.1)