todo-agent 0.2.8__py3-none-any.whl → 0.3.1__py3-none-any.whl

todo_agent/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '0.2.8'
-__version_tuple__ = version_tuple = (0, 2, 8)
+__version__ = version = '0.3.1'
+__version_tuple__ = version_tuple = (0, 3, 1)
 
 __commit_id__ = commit_id = None

todo_agent/core/todo_manager.py

@@ -20,6 +20,7 @@ class TodoManager:
         context: Optional[str] = None,
         due: Optional[str] = None,
         recurring: Optional[str] = None,
+        duration: Optional[str] = None,
     ) -> str:
         """Add new task with explicit project/context parameters."""
         # Validate and sanitize inputs
@@ -78,7 +79,35 @@ class TodoManager:
                     if interval < 1:
                         raise ValueError("Interval must be at least 1.")
                 except ValueError:
-                    raise ValueError(f"Invalid interval '{parts[2]}'. Must be a positive integer.")
+                    raise ValueError(
+                        f"Invalid interval '{parts[2]}'. Must be a positive integer."
+                    )
+
+        if duration is not None:
+            # Validate duration format (e.g., "30m", "2h", "1d")
+            if not duration or not isinstance(duration, str):
+                raise ValueError("Duration must be a non-empty string.")
+
+            # Check if duration ends with a valid unit
+            if not any(duration.endswith(unit) for unit in ["m", "h", "d"]):
+                raise ValueError(
+                    f"Invalid duration format '{duration}'. Must end with m (minutes), h (hours), or d (days)."
+                )
+
+            # Extract the numeric part and validate it
+            value = duration[:-1]
+            if not value:
+                raise ValueError("Duration value cannot be empty.")
+
+            try:
+                # Check if the value is a valid positive number
+                numeric_value = float(value)
+                if numeric_value <= 0:
+                    raise ValueError("Duration value must be positive.")
+            except ValueError:
+                raise ValueError(
+                    f"Invalid duration value '{value}'. Must be a positive number."
+                )
 
         # Build the full task description with priority, project, and context
         full_description = description
@@ -98,6 +127,9 @@ class TodoManager:
         if recurring:
             full_description = f"{full_description} {recurring}"
 
+        if duration:
+            full_description = f"{full_description} duration:{duration}"
+
         self.todo_shell.add(full_description)
         return f"Added task: {full_description}"
 
@@ -167,6 +199,114 @@ class TodoManager:
         result = self.todo_shell.remove_priority(task_number)
         return f"Removed priority from task {task_number}: {result}"
 
+    def set_due_date(self, task_number: int, due_date: str) -> str:
+        """
+        Set or update due date for a task by intelligently rewriting it.
+
+        Args:
+            task_number: The task number to modify
+            due_date: Due date in YYYY-MM-DD format, or empty string to remove due date
+
+        Returns:
+            Confirmation message with the updated task
+        """
+        # Validate due date format only if not empty
+        if due_date.strip():
+            try:
+                datetime.strptime(due_date, "%Y-%m-%d")
+            except ValueError:
+                raise ValueError(
+                    f"Invalid due date format '{due_date}'. Must be YYYY-MM-DD."
+                )
+
+        result = self.todo_shell.set_due_date(task_number, due_date)
+        if due_date.strip():
+            return f"Set due date {due_date} for task {task_number}: {result}"
+        else:
+            return f"Removed due date from task {task_number}: {result}"
+
+    def set_context(self, task_number: int, context: str) -> str:
+        """
+        Set or update context for a task by intelligently rewriting it.
+
+        Args:
+            task_number: The task number to modify
+            context: Context name (without @ symbol), or empty string to remove context
+
+        Returns:
+            Confirmation message with the updated task
+        """
+        # Validate context name if not empty
+        if context.strip():
+            # Remove any existing @ symbols to prevent duplication
+            clean_context = context.strip().lstrip("@")
+            if not clean_context:
+                raise ValueError(
+                    "Context name cannot be empty after removing @ symbol."
+                )
+
+        result = self.todo_shell.set_context(task_number, context)
+        if context.strip():
+            clean_context = context.strip().lstrip("@")
+            return f"Set context @{clean_context} for task {task_number}: {result}"
+        else:
+            return f"Removed context from task {task_number}: {result}"
+
+    def set_project(self, task_number: int, projects: list) -> str:
+        """
+        Set or update projects for a task by intelligently rewriting it.
+
+        Args:
+            task_number: The task number to modify
+            projects: List of project operations. Each item can be:
+                     - "project" (add project)
+                     - "-project" (remove project)
+                     - Empty string removes all projects
+
+        Returns:
+            Confirmation message with the updated task
+        """
+        # Validate project names if not empty
+        if projects:
+            for project in projects:
+                if project.strip() and not project.startswith("-"):
+                    # Remove any existing + symbols to prevent duplication
+                    clean_project = project.strip().lstrip("+")
+                    if not clean_project:
+                        raise ValueError(
+                            "Project name cannot be empty after removing + symbol."
+                        )
+                elif project.startswith("-"):
+                    clean_project = project[1:].strip().lstrip("+")
+                    if not clean_project:
+                        raise ValueError(
+                            "Project name cannot be empty after removing - and + symbols."
+                        )
+
+        result = self.todo_shell.set_project(task_number, projects)
+
+        if not projects:
+            return f"No project changes made to task {task_number}: {result}"
+        else:
+            # Build operation description
+            operations = []
+            for project in projects:
+                if not project.strip():
+                    # Empty string is a NOOP - skip
+                    continue
+                elif project.startswith("-"):
+                    clean_project = project[1:].strip().lstrip("+")
+                    operations.append(f"removed +{clean_project}")
+                else:
+                    clean_project = project.strip().lstrip("+")
+                    operations.append(f"added +{clean_project}")
+
+            if not operations:
+                return f"No project changes made to task {task_number}: {result}"
+            else:
+                operation_desc = ", ".join(operations)
+                return f"Updated projects for task {task_number} ({operation_desc}): {result}"
+
     def list_projects(self, **kwargs: Any) -> str:
         """List all available projects in todo.txt."""
         result = self.todo_shell.list_projects()
@@ -263,4 +403,6 @@ class TodoManager:
     def get_current_datetime(self, **kwargs: Any) -> str:
         """Get the current date and time."""
         now = datetime.now()
-        return f"Current date and time: {now.strftime('%Y-%m-%d %H:%M:%S')} ({now.strftime('%A, %B %d, %Y at %I:%M %p')})"
+        week_number = now.isocalendar()[1]
+        timezone = now.astimezone().tzinfo
+        return f"Current date and time: {now.strftime('%Y-%m-%d %H:%M:%S')} {timezone} ({now.strftime('%A, %B %d, %Y at %I:%M %p')}) - Week {week_number}"

todo_agent/infrastructure/inference.py

@@ -4,6 +4,7 @@ LLM inference engine for todo.sh agent.
 
 import os
 import time
+from datetime import datetime
 from typing import Any, Dict, Optional
 
 try:
@@ -71,9 +72,9 @@ class Inference:
         tools_section = self._generate_tools_section()
 
         # Get current datetime for interpolation
-        from datetime import datetime
-
-        current_datetime = datetime.now().strftime("%Y-%m-%d %H:%M:%S")
+        now = datetime.now()
+        timezone_info = time.tzname[time.daylight]
+        current_datetime = f"{now.strftime('%Y-%m-%d %H:%M:%S')} {timezone_info}"
 
         # Get calendar output
         from .calendar_utils import get_calendar_output

todo_agent/infrastructure/openrouter_client.py

@@ -72,26 +72,32 @@ class OpenRouterClient(LLMClient):
             f"Token usage - Prompt: {prompt_tokens}, Completion: {completion_tokens}, Total: {total_tokens}"
         )
 
-        # Log tool call details if present
-        if response.get("choices"):
-            choice = response["choices"][0]
-            if "message" in choice and "tool_calls" in choice["message"]:
-                tool_calls = choice["message"]["tool_calls"]
-                self.logger.info(f"Response contains {len(tool_calls)} tool calls")
-
-                # Log thinking content (response body) if present
-                content = choice["message"].get("content", "")
-                if content and content.strip():
-                    self.logger.info(f"LLM thinking before tool calls: {content}")
-
-                for i, tool_call in enumerate(tool_calls):
-                    tool_name = tool_call.get("function", {}).get("name", "unknown")
-                    self.logger.info(f"  Tool call {i + 1}: {tool_name}")
-            elif "message" in choice and "content" in choice["message"]:
-                content = choice["message"]["content"]
-                self.logger.debug(
-                    f"Response contains content: {content[:100]}{'...' if len(content) > 100 else ''}"
-                )
+        # Extract and log choice details
+        choices = response.get("choices", [])
+        if not choices:
+            return
+
+        choice = choices[0]
+        message = choice.get("message", {})
+
+        # Always log reasoning and content if present
+        reasoning = message.get("reasoning", "")
+        if reasoning:
+            self.logger.info(f"LLM reasoning: {reasoning}")
+
+        content = message.get("content", "")
+        if content:
+            self.logger.info(f"LLM content: {content}")
+
+        # Handle tool calls
+        tool_calls = message.get("tool_calls", [])
+        if tool_calls:
+            self.logger.info(f"Response contains {len(tool_calls)} tool calls")
+
+            # Log each tool call
+            for i, tool_call in enumerate(tool_calls, 1):
+                tool_name = tool_call.get("function", {}).get("name", "unknown")
+                self.logger.info(f"  Tool call {i}: {tool_name}")
 
         self.logger.debug(f"Raw response: {json.dumps(response, indent=2)}")