todo-agent 0.3.3__tar.gz → 0.3.6__tar.gz

{todo_agent-0.3.3 → todo_agent-0.3.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: todo-agent
-Version: 0.3.3
+Version: 0.3.6
 Summary: A natural language interface for todo.sh task management
 Author: codeprimate
 Maintainer: codeprimate

{todo_agent-0.3.3 → todo_agent-0.3.6}/tests/test_core/test_todo_manager.py

@@ -577,14 +577,14 @@ class TestTodoManager(unittest.TestCase):
             self.assertIn("2025-01-15 10:30:00", result)
             self.assertIn("Week 3", result)
 
-    def test_created_completed_task_basic(self):
+    def test_create_completed_task_basic(self):
         """Test creating and immediately completing a task."""
         # Mock the todo_shell methods
         self.todo_shell.add.return_value = "Task added"
         self.todo_shell.list_tasks.return_value = "1 Test task\n2 Another task"
         self.todo_shell.complete.return_value = "Task completed"
 
-        result = self.todo_manager.created_completed_task("Test task")
+        result = self.todo_manager.create_completed_task("Test task")
 
         # Verify the task was added
         self.todo_shell.add.assert_called_once_with("Test task")
@@ -593,13 +593,13 @@ class TestTodoManager(unittest.TestCase):
         # Verify the result message
         self.assertIn("Created and completed task: Test task", result)
 
-    def test_created_completed_task_with_project_and_context(self):
+    def test_create_completed_task_with_project_and_context(self):
         """Test creating and completing a task with project and context."""
         self.todo_shell.add.return_value = "Task added"
         self.todo_shell.list_tasks.return_value = "1 Test task\n2 Another task"
         self.todo_shell.complete.return_value = "Task completed"
 
-        result = self.todo_manager.created_completed_task(
+        result = self.todo_manager.create_completed_task(
             "Test task", project="work", context="office"
         )
 
@@ -609,35 +609,38 @@ class TestTodoManager(unittest.TestCase):
         self.todo_shell.complete.assert_called_once_with(1)
         self.assertIn("+work @office", result)
 
-    def test_created_completed_task_with_custom_date(self):
+    def test_create_completed_task_with_custom_date(self):
         """Test creating and completing a task with a custom completion date."""
-        self.todo_shell.add.return_value = "Task added"
-        self.todo_shell.list_tasks.return_value = "1 Test task"
-        self.todo_shell.complete.return_value = "Task completed"
+        self.todo_shell.addto.return_value = "Task added to done.txt"
 
-        result = self.todo_manager.created_completed_task(
+        result = self.todo_manager.create_completed_task(
             "Test task", completion_date="2025-01-10"
         )
 
-        # Verify the task was completed (should find task number 1 since it contains "Test task")
-        self.todo_shell.complete.assert_called_once_with(1)
+        # Verify the task was added directly to done.txt with the custom date
+        self.todo_shell.addto.assert_called_once_with(
+            "done.txt", "x 2025-01-10 Test task"
+        )
+        # Verify add and complete were NOT called when using custom date
+        self.todo_shell.add.assert_not_called()
+        self.todo_shell.complete.assert_not_called()
         self.assertIn("completed on 2025-01-10", result)
 
-    def test_created_completed_task_with_invalid_date(self):
+    def test_create_completed_task_with_invalid_date(self):
         """Test that invalid completion date raises ValueError."""
         with self.assertRaises(ValueError) as context:
-            self.todo_manager.created_completed_task(
+            self.todo_manager.create_completed_task(
                 "Test task", completion_date="invalid-date"
             )
         self.assertIn("Invalid completion date format", str(context.exception))
 
-    def test_created_completed_task_sanitizes_project_and_context(self):
+    def test_create_completed_task_sanitizes_project_and_context(self):
         """Test that project and context with existing symbols are properly sanitized."""
         self.todo_shell.add.return_value = "Task added"
         self.todo_shell.list_tasks.return_value = "1 Test task"
         self.todo_shell.complete.return_value = "Task completed"
 
-        result = self.todo_manager.created_completed_task(
+        result = self.todo_manager.create_completed_task(
             "Test task", project="+work", context="@office"
         )
 
@@ -645,25 +648,25 @@ class TestTodoManager(unittest.TestCase):
         self.todo_shell.add.assert_called_once_with("Test task +work @office")
         self.assertIn("+work @office", result)
 
-    def test_created_completed_task_with_empty_project_after_sanitization(self):
+    def test_create_completed_task_with_empty_project_after_sanitization(self):
         """Test that empty project after sanitization raises ValueError."""
         with self.assertRaises(ValueError) as context:
-            self.todo_manager.created_completed_task("Test task", project="+")
+            self.todo_manager.create_completed_task("Test task", project="+")
         self.assertIn("Project name cannot be empty", str(context.exception))
 
-    def test_created_completed_task_with_empty_context_after_sanitization(self):
+    def test_create_completed_task_with_empty_context_after_sanitization(self):
         """Test that empty context after sanitization raises ValueError."""
         with self.assertRaises(ValueError) as context:
-            self.todo_manager.created_completed_task("Test task", context="@")
+            self.todo_manager.create_completed_task("Test task", context="@")
         self.assertIn("Context name cannot be empty", str(context.exception))
 
-    def test_created_completed_task_no_tasks_after_addition(self):
+    def test_create_completed_task_no_tasks_after_addition(self):
         """Test that RuntimeError is raised when no tasks are found after addition."""
         self.todo_shell.add.return_value = "Task added"
         self.todo_shell.list_tasks.return_value = ""  # No tasks found
 
         with self.assertRaises(RuntimeError) as context:
-            self.todo_manager.created_completed_task("Test task")
+            self.todo_manager.create_completed_task("Test task")
         self.assertIn("Failed to add task", str(context.exception))
 
 

{todo_agent-0.3.3 → todo_agent-0.3.6}/tests/test_infrastructure/test_todo_shell.py

@@ -810,3 +810,16 @@ class TestTodoShell:
             # Should not duplicate +work, only add +new
             mock_replace.assert_called_once_with(1, "Test task +work +new @context")
             assert result == "Task updated"
+
+    def test_get_help_constructs_correct_command(self):
+        """Test getting help constructs the correct todo.sh command."""
+        with patch.object(
+            self.todo_shell, "execute", return_value="Todo.sh help output"
+        ) as mock_execute:
+            result = self.todo_shell.get_help()
+
+            # Verify the correct command was constructed
+            mock_execute.assert_called_once_with(
+                ["todo.sh", "help"], suppress_color=False
+            )
+            assert result == "Todo.sh help output"

{todo_agent-0.3.3 → todo_agent-0.3.6}/tests/test_interface/test_cli.py

@@ -377,6 +377,97 @@ class TestCLI:
                     expected_line in str(call) for call in mock_print.call_args_list
                 )
 
+    def test_todo_passthrough_command_success(self):
+        """Test successful todo.sh passthrough command execution."""
+        with patch("builtins.input", return_value="/add test task"), patch("sys.exit"):
+            # Mock the todo_shell execute method
+            mock_todo_shell = Mock()
+            mock_todo_shell.execute.return_value = "1 test task"
+
+            # Create CLI instance with mocked dependencies
+            with patch("todo_agent.interface.cli.Config"), patch(
+                "todo_agent.interface.cli.TodoShell", return_value=mock_todo_shell
+            ), patch("todo_agent.interface.cli.TodoManager"), patch(
+                "todo_agent.interface.cli.ToolCallHandler"
+            ), patch("todo_agent.interface.cli.Inference"), patch(
+                "todo_agent.interface.cli.Logger"
+            ), patch("todo_agent.interface.cli.Console") as mock_console:
+                CLI()
+
+                # Mock the console methods
+                mock_console.return_value.input.return_value = "/add test task"
+                mock_console.return_value.print = Mock()
+
+                # Test the passthrough logic directly
+                user_input = "/add test task"
+                if user_input.startswith("/"):
+                    todo_command = user_input[1:].strip()
+                    output = mock_todo_shell.execute(["todo.sh", *todo_command.split()])
+
+                    # Verify the command was executed correctly
+                    mock_todo_shell.execute.assert_called_once_with(
+                        ["todo.sh", "add", "test", "task"]
+                    )
+                    assert output == "1 test task"
+
+    def test_todo_passthrough_command_empty(self):
+        """Test todo.sh passthrough with empty command."""
+        with patch("builtins.input", return_value="/"), patch("sys.exit"):
+            # Mock the todo_shell execute method
+            mock_todo_shell = Mock()
+
+            # Create CLI instance with mocked dependencies
+            with patch("todo_agent.interface.cli.Config"), patch(
+                "todo_agent.interface.cli.TodoShell", return_value=mock_todo_shell
+            ), patch("todo_agent.interface.cli.TodoManager"), patch(
+                "todo_agent.interface.cli.ToolCallHandler"
+            ), patch("todo_agent.interface.cli.Inference"), patch(
+                "todo_agent.interface.cli.Logger"
+            ), patch("todo_agent.interface.cli.Console") as mock_console:
+                CLI()
+
+                # Mock the console methods
+                mock_console.return_value.input.return_value = "/"
+                mock_console.return_value.print = Mock()
+
+                # Test the passthrough logic with empty command
+                user_input = "/"
+                if user_input.startswith("/"):
+                    todo_command = user_input[1:].strip()
+                    if not todo_command:
+                        # Should handle empty command gracefully
+                        assert todo_command == ""
+
+    def test_todo_help_command(self):
+        """Test todo-help command execution."""
+        with patch("builtins.input", return_value="todo-help"), patch("sys.exit"):
+            # Mock the todo_shell get_help method
+            mock_todo_shell = Mock()
+            mock_todo_shell.get_help.return_value = "Todo.sh help output"
+
+            # Create CLI instance with mocked dependencies
+            with patch("todo_agent.interface.cli.Config"), patch(
+                "todo_agent.interface.cli.TodoShell", return_value=mock_todo_shell
+            ), patch("todo_agent.interface.cli.TodoManager"), patch(
+                "todo_agent.interface.cli.ToolCallHandler"
+            ), patch("todo_agent.interface.cli.Inference"), patch(
+                "todo_agent.interface.cli.Logger"
+            ), patch("todo_agent.interface.cli.Console") as mock_console:
+                CLI()
+
+                # Mock the console methods
+                mock_console.return_value.input.return_value = "todo-help"
+                mock_console.return_value.print = Mock()
+
+                # Test the todo-help command
+                user_input = "todo-help"
+                if user_input.lower() == "todo-help":
+                    help_output = mock_todo_shell.get_help()
+
+                    # Verify the help was retrieved
+                    mock_todo_shell.get_help.assert_called_once()
+                    assert help_output == "Todo.sh help output"
+
     def test_empty_input_handling(self):
         """Test that empty input is handled gracefully."""
         # This would be tested in the main run loop

{todo_agent-0.3.3 → todo_agent-0.3.6}/tests/test_interface/test_tools.py

@@ -108,16 +108,16 @@ class TestToolErrorHandling:
         assert "Permission denied" in result["user_message"]
         assert "check file permissions" in result["user_message"]
 
-    def test_created_completed_task_tool_execution(self):
-        """Test that the created_completed_task tool can be executed successfully."""
+    def test_create_completed_task_tool_execution(self):
+        """Test that the create_completed_task tool can be executed successfully."""
         # Mock the todo_manager method
-        self.mock_todo_manager.created_completed_task.return_value = (
+        self.mock_todo_manager.create_completed_task.return_value = (
             "Created and completed task: Test task (completed on 2025-01-15)"
         )
 
         tool_call = {
             "function": {
-                "name": "created_completed_task",
+                "name": "create_completed_task",
                 "arguments": '{"description": "Test task", "completion_date": "2025-01-15"}',
             },
             "id": "test_id",
@@ -128,10 +128,10 @@ class TestToolErrorHandling:
         assert result["error"] is False
         assert "Created and completed task: Test task" in result["output"]
         assert result["tool_call_id"] == "test_id"
-        assert result["name"] == "created_completed_task"
+        assert result["name"] == "create_completed_task"
 
         # Verify the method was called with correct arguments
-        self.mock_todo_manager.created_completed_task.assert_called_once_with(
+        self.mock_todo_manager.create_completed_task.assert_called_once_with(
             description="Test task", completion_date="2025-01-15"
         )
 

{todo_agent-0.3.3 → todo_agent-0.3.6}/todo_agent/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '0.3.3'
-__version_tuple__ = version_tuple = (0, 3, 3)
+__version__ = version = '0.3.6'
+__version_tuple__ = version_tuple = (0, 3, 6)
 
-__commit_id__ = commit_id = 'g4d82778ea'
+__commit_id__ = commit_id = 'g30b41eb2b'