feat: add tp task <id> top-level command for better UX

Migrate 'tp get task <task_id>' to direct 'tp task <task_id>' shortcut.
The get task sub-command is kept for backwards compatibility.

- Add task_command.py with top-level 'task' command
- Register in main.py as a top-level command
- Add unit tests (7 new tests, 100% passing)
- Bump version to 1.0.1

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: minhdqdev

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "todopro-cli"
-version = "1.0.0"
+version = "1.0.1"
 description = "A professional CLI for TodoPro task management system"
 readme = "README.md"
 requires-python = ">=3.12"

src/todopro_cli/commands/task_command.py

@@ -0,0 +1,32 @@
+"""Top-level `tp task <task_id>` command — quick shortcut to get task details."""
+
+from typing import Annotated
+
+import typer
+
+from todopro_cli.services.context_manager import get_strategy_context
+from todopro_cli.services.task_service import TaskService
+from todopro_cli.utils.task_helpers import resolve_task_id
+from todopro_cli.utils.ui.formatters import format_output
+
+from .decorators import command_wrapper
+
+app = typer.Typer()
+
+
+@app.command("task")
+@command_wrapper
+async def task_command(
+    task_id: Annotated[str, typer.Argument(help="Task ID or suffix")],
+    output: Annotated[
+        str, typer.Option("--output", "-o", help="Output format")
+    ] = "table",
+) -> None:
+    """Get task details by ID or suffix."""
+    strategy = get_strategy_context()
+    task_repo = strategy.task_repository
+    task_service = TaskService(task_repo)
+
+    resolved_id = await resolve_task_id(task_service, task_id)
+    task = await task_service.get_task(resolved_id)
+    format_output(task.model_dump(), output)

src/todopro_cli/main.py

@@ -14,6 +14,7 @@
 from .commands.encryption import app as encryption_app  # E2EE commands (setup, status, recover, rotate, show-recovery)
 # from .commands.export_command import app as export_app  # DISABLED: Duplicate of data.py export, references undefined factory
 from .commands.get_command import app as get_app
+from .commands.task_command import app as task_command_app
 # from .commands.import_command import app as import_app  # DISABLED: Duplicate of data.py import, references undefined factory
 from .commands.list_command import app as list_app
 from .commands.login_command import app as login_command_app
@@ -71,6 +72,7 @@
 app.add_typer(
     list_app, name="list", help="List resources (tasks, projects, labels, etc.)"
 )
+app.add_typer(task_command_app, name="", help="Get task details by ID or suffix")
 app.add_typer(get_app, name="get", help="Get resource details")
 app.add_typer(create_app, name="create", help="Create new resources")
 app.add_typer(update_resource_app, name="update", help="Update existing resources")

tests/commands/test_task_command.py

@@ -0,0 +1,119 @@
+"""Unit tests for the top-level `tp task <task_id>` command."""
+# pylint: disable=redefined-outer-name
+
+from datetime import datetime
+from unittest.mock import AsyncMock, MagicMock, patch
+
+import pytest
+from typer.testing import CliRunner
+
+from todopro_cli.commands.task_command import app
+from todopro_cli.models import Task
+
+runner = CliRunner()
+
+
+@pytest.fixture
+def mock_task():
+    """Create a mock task for testing."""
+    return Task(
+        id="task-abc123",
+        content="Write unit tests",
+        description="Cover the new task command",
+        project_id=None,
+        due_date=None,
+        priority=1,
+        is_completed=False,
+        labels=[],
+        contexts=[],
+        created_at=datetime(2024, 1, 1, 12, 0, 0),
+        updated_at=datetime(2024, 1, 1, 12, 0, 0),
+    )
+
+
+@pytest.fixture
+def mock_task_service(mock_task):
+    """Patch TaskService and strategy context for all task_command tests."""
+    service_mock = MagicMock()
+    service_mock.get_task = AsyncMock(return_value=mock_task)
+
+    strategy_mock = MagicMock()
+    strategy_mock.task_repository = MagicMock()
+
+    with (
+        patch(
+            "todopro_cli.commands.task_command.get_strategy_context",
+            return_value=strategy_mock,
+        ),
+        patch(
+            "todopro_cli.commands.task_command.TaskService",
+            return_value=service_mock,
+        ),
+        patch(
+            "todopro_cli.commands.decorators.get_config_service",
+            return_value=MagicMock(
+                config=MagicMock(
+                    get_current_context=MagicMock(
+                        return_value=MagicMock(type="local")
+                    )
+                )
+            ),
+        ),
+    ):
+        yield service_mock
+
+
+class TestTaskCommandHelp:
+    """Structural tests — no network/auth needed."""
+
+    def test_help_exits_zero(self):
+        """--help should succeed."""
+        result = runner.invoke(app, ["--help"])
+        assert result.exit_code == 0
+
+    def test_help_contains_description(self):
+        """Help text should describe the command."""
+        result = runner.invoke(app, ["--help"])
+        assert "Get task details" in result.stdout
+
+    def test_help_mentions_id_argument(self):
+        """Help text should mention the task_id argument."""
+        result = runner.invoke(app, ["--help"])
+        assert "task_id" in result.stdout.lower() or "TASK_ID" in result.stdout
+
+    def test_missing_task_id_exits_nonzero(self):
+        """Invoking without a task_id should exit with an error."""
+        result = runner.invoke(app, [])
+        assert result.exit_code != 0
+
+
+class TestTaskCommandSuccess:
+    """Functional tests for the happy path."""
+
+    @patch("todopro_cli.commands.task_command.resolve_task_id")
+    def test_get_task_by_suffix(self, mock_resolve, mock_task_service, mock_task):
+        """tp task <suffix> should resolve and display the task."""
+        mock_resolve.return_value = AsyncMock(return_value="task-abc123")()
+
+        result = runner.invoke(app, ["abc123"])
+
+        assert result.exit_code == 0
+        mock_task_service.get_task.assert_called_once()
+
+    @patch("todopro_cli.commands.task_command.resolve_task_id")
+    def test_get_task_by_full_id(self, mock_resolve, mock_task_service, mock_task):
+        """tp task <full-id> should work with a full UUID-style ID."""
+        mock_resolve.return_value = AsyncMock(return_value="task-abc123")()
+
+        result = runner.invoke(app, ["task-abc123"])
+
+        assert result.exit_code == 0
+
+    @patch("todopro_cli.commands.task_command.resolve_task_id")
+    def test_output_json_flag(self, mock_resolve, mock_task_service, mock_task):
+        """tp task <id> --output json should pass json format."""
+        mock_resolve.return_value = AsyncMock(return_value="task-abc123")()
+
+        result = runner.invoke(app, ["abc123", "--output", "json"])
+
+        assert result.exit_code == 0