Remove utils_tools and disable private_hf_repo_tool

- Delete utils_tools.py (datetime now provided in system prompt)
- Comment out private_hf_repo_tool registration (system prompt now
reliably instructs agent to push outputs to hub)

agent/core/tools.py

@@ -37,13 +37,12 @@ from agent.tools.github_read_file import (
 )
 from agent.tools.jobs_tool import HF_JOBS_TOOL_SPEC, hf_jobs_handler
 from agent.tools.plan_tool import PLAN_TOOL_SPEC, plan_tool_handler
-from agent.tools.private_hf_repo_tools import (
-    PRIVATE_HF_REPO_TOOL_SPEC,
-    private_hf_repo_handler,
-)
 
-# NOTE: Utils tool disabled - date/time now loaded into system prompt at initialization
-# from agent.tools.utils_tools import UTILS_TOOL_SPEC, utils_handler
+# NOTE: Private HF repo tool disabled - system prompt handles pushing to hub pretty well now
+# from agent.tools.private_hf_repo_tools import (
+#     PRIVATE_HF_REPO_TOOL_SPEC,
+#     private_hf_repo_handler,
+# )
 
 # Suppress aiohttp deprecation warning
 warnings.filterwarnings(
@@ -281,18 +280,12 @@ def create_builtin_tools() -> list[ToolSpec]:
             parameters=HF_JOBS_TOOL_SPEC["parameters"],
             handler=hf_jobs_handler,
         ),
-        ToolSpec(
-            name=PRIVATE_HF_REPO_TOOL_SPEC["name"],
-            description=PRIVATE_HF_REPO_TOOL_SPEC["description"],
-            parameters=PRIVATE_HF_REPO_TOOL_SPEC["parameters"],
-            handler=private_hf_repo_handler,
-        ),
-        # NOTE: Utils tool disabled - date/time now loaded into system prompt at initialization (less tool calls=more reliablity)
+        # NOTE: Private HF repo tool disabled - system prompt now reliably instructs agent to push to hub
         # ToolSpec(
-        #     name=UTILS_TOOL_SPEC["name"],
-        #     description=UTILS_TOOL_SPEC["description"],
-        #     parameters=UTILS_TOOL_SPEC["parameters"],
-        #     handler=utils_handler,
+        #     name=PRIVATE_HF_REPO_TOOL_SPEC["name"],
+        #     description=PRIVATE_HF_REPO_TOOL_SPEC["description"],
+        #     parameters=PRIVATE_HF_REPO_TOOL_SPEC["parameters"],
+        #     handler=private_hf_repo_handler,
         # ),
         # GitHub tools
         # NOTE: Github search code tool disabled - a bit buggy

agent/tools/utils_tools.py

@@ -1,203 +0,0 @@
-"""
-Utils Tools - General utility operations
-
-Provides system information like current date/time with timezone support.
-"""
-
-import zoneinfo
-from datetime import datetime
-from typing import Any, Dict, Literal
-
-from agent.tools.types import ToolResult
-
-# Operation names
-OperationType = Literal["get_datetime"]
-
-
-class UtilsTool:
-    """Tool for general utility operations."""
-
-    async def execute(self, params: Dict[str, Any]) -> ToolResult:
-        """Execute the specified utility operation."""
-        operation = params.get("operation")
-        args = params.get("args", {})
-
-        # If no operation provided, return usage instructions
-        if not operation:
-            return self._show_help()
-
-        # Normalize operation name
-        operation = operation.lower()
-
-        # Check if help is requested
-        if args.get("help"):
-            return self._show_operation_help(operation)
-
-        try:
-            # Route to appropriate handler
-            if operation == "get_datetime":
-                return await self._get_datetime(args)
-            else:
-                return {
-                    "formatted": f'Unknown operation: "{operation}"\n\n'
-                    "Available operations: get_datetime\n\n"
-                    "Call this tool with no operation for full usage instructions.",
-                    "totalResults": 0,
-                    "resultsShared": 0,
-                    "isError": True,
-                }
-
-        except Exception as e:
-            return {
-                "formatted": f"Error executing {operation}: {str(e)}",
-                "totalResults": 0,
-                "resultsShared": 0,
-                "isError": True,
-            }
-
-    def _show_help(self) -> ToolResult:
-        """Show usage instructions when tool is called with no arguments."""
-        usage_text = """# Utils Tool
-
-Utility operations for system information.
-
-## Available Commands
-
-- **get_datetime** - Get current date and time with timezone support
-
-## Examples
-
-### Get current date and time (Paris timezone by default)
-Call this tool with:
-```json
-{
-  "operation": "get_datetime",
-  "args": {}
-}
-```
-
-### Get current date and time in a specific timezone
-Call this tool with:
-```json
-{
-  "operation": "get_datetime",
-  "args": {
-    "timezone": "America/New_York"
-  }
-}
-```
-
-Common timezones: Europe/Paris, America/New_York, America/Los_Angeles, Asia/Tokyo, UTC
-
-## Tips
-
-- **Default timezone**: Paris (Europe/Paris)
-- **Date format**: dd-mm-yyyy
-- **Time format**: HH:MM:SS.mmm (24-hour format with milliseconds)
-- **Timezone names**: Use IANA timezone database names (e.g., "Europe/Paris", "UTC")
-"""
-        return {"formatted": usage_text, "totalResults": 1, "resultsShared": 1}
-
-    def _show_operation_help(self, operation: str) -> ToolResult:
-        """Show help for a specific operation."""
-        help_text = f"Help for operation: {operation}\n\nCall with appropriate arguments. Use the main help for examples."
-        return {"formatted": help_text, "totalResults": 1, "resultsShared": 1}
-
-    async def _get_datetime(self, args: Dict[str, Any]) -> ToolResult:
-        """Get current date and time with timezone support."""
-        timezone_name = args.get("timezone", "Europe/Paris")
-
-        try:
-            # Get timezone object
-            tz = zoneinfo.ZoneInfo(timezone_name)
-
-            # Get current datetime in specified timezone
-            now = datetime.now(tz)
-
-            # Format date as dd-mm-yyyy
-            date_str = now.strftime("%d-%m-%Y")
-
-            # Format time as HH:MM:SS.mmm
-            time_str = now.strftime("%H:%M:%S.%f")[
-                :-3
-            ]  # Remove last 3 digits to keep only milliseconds
-
-            # Get timezone abbreviation/offset
-            tz_offset = now.strftime("%z")
-            tz_name = now.strftime("%Z")
-
-            response = f"""✓ Current date and time
-
-**Date:** {date_str}
-**Time:** {time_str}
-**Timezone:** {timezone_name} ({tz_name}, UTC{tz_offset[:3]}:{tz_offset[3:]})
-
-**ISO Format:** {now.isoformat()}
-**Unix Timestamp:** {int(now.timestamp())}"""
-
-            return {"formatted": response, "totalResults": 1, "resultsShared": 1}
-
-        except zoneinfo.ZoneInfoNotFoundError:
-            return {
-                "formatted": f"Invalid timezone: {timezone_name}\n\n"
-                "Use IANA timezone database names like:\n"
-                "- Europe/Paris\n"
-                "- America/New_York\n"
-                "- Asia/Tokyo\n"
-                "- UTC\n\n"
-                "See: https://en.wikipedia.org/wiki/List_of_tz_database_time_zones",
-                "totalResults": 0,
-                "resultsShared": 0,
-                "isError": True,
-            }
-        except Exception as e:
-            return {
-                "formatted": f"Failed to get date/time: {str(e)}",
-                "totalResults": 0,
-                "resultsShared": 0,
-                "isError": True,
-            }
-
-
-# Tool specification for agent registration
-UTILS_TOOL_SPEC = {
-    "name": "utils",
-    "description": (
-        "System utility operations - currently provides date/time with timezone support. "
-        "**Use when:** (1) Need current date for logging/timestamps, (2) User asks 'what time is it', "
-        "(3) Need timezone-aware datetime for scheduling/coordination, (4) Creating timestamped filenames. "
-        "**Operation:** get_datetime with optional timezone parameter (default: Europe/Paris). "
-        "Returns: Date (dd-mm-yyyy), time (HH:MM:SS.mmm), timezone info, ISO format, Unix timestamp. "
-        "**Pattern:** utils get_datetime → use timestamp in filename/log → upload to hf_private_repos. "
-        "Supports IANA timezone names: 'Europe/Paris', 'America/New_York', 'Asia/Tokyo', 'UTC'."
-    ),
-    "parameters": {
-        "type": "object",
-        "properties": {
-            "operation": {
-                "type": "string",
-                "enum": ["get_datetime"],
-                "description": "Operation to execute. Valid values: [get_datetime]",
-            },
-            "args": {
-                "type": "object",
-                "description": (
-                    "Operation-specific arguments as a JSON object. "
-                    "For get_datetime: timezone (string, optional, default: Europe/Paris). "
-                    "Use IANA timezone names like 'America/New_York', 'Asia/Tokyo', 'UTC'."
-                ),
-                "additionalProperties": True,
-            },
-        },
-    },
-}
-
-
-async def utils_handler(arguments: Dict[str, Any]) -> tuple[str, bool]:
-    """Handler for agent tool router."""
-    try:
-        tool = UtilsTool()
-        result = await tool.execute(arguments)
-        return result["formatted"], not result.get("isError", False)
-    except Exception as e:
-        return f"Error executing Utils tool: {str(e)}", False