ellf-cli 18.0.28__tar.gz → 18.0.30__tar.gz

{ellf_cli-18.0.28/ellf_cli.egg-info → ellf_cli-18.0.30}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ellf-cli
-Version: 18.0.28
+Version: 18.0.30
 Summary: Ellf Command Line Interface
 Home-page: https://prodi.gy
 Author: ExplosionAI GmbH

{ellf_cli-18.0.28 → ellf_cli-18.0.30}/ellf_cli/commands/actions.py

@@ -220,8 +220,10 @@ def info(
         as_json=as_json,
         select=select,
         stats=_stats(res),
-        # Superseded by the stats block; --json still carries both.
-        table_exclude=["stats", "related_tasks"],
+        # Noise in a table; --json and an explicit --select still carry them.
+        table_exclude=(
+            None if select else ["stats", "related_tasks", "created_by_user"]
+        ),
     )
     return res
 

{ellf_cli-18.0.28 → ellf_cli-18.0.30}/ellf_cli/commands/agents.py

@@ -3,6 +3,7 @@ from collections.abc import Sequence
 from typing import Any, Optional, Union
 from uuid import UUID
 
+from pydantic import ValidationError
 from radicli import Arg
 from wasabi import msg
 
@@ -13,6 +14,7 @@ from ..errors import (
     BrokerError,
     CLIError,
     EllfError,
+    EllfErrors,
     HTTPXErrors,
     JobStartRejectedError,
 )
@@ -296,17 +298,25 @@ def info(
         as_json=as_json,
         select=select,
         stats=_stats(res),
-        # Superseded by the stats block; --json still carries the ids.
-        table_exclude=["related_tasks"],
+        # Noise in a table; --json and an explicit --select still carry them.
+        table_exclude=None if select else ["related_tasks", "created_by_user"],
     )
     return res
 
 
 def _stats(agent: AgentDetail) -> dict[str, Any]:
-    return {
+    stats = {
         "tasks": len(agent.related_tasks),
         "last_started": agent.last_execution_started,
     }
+    try:
+        counts = get_auth_state().client.agent.annotation_counts(agent.id)
+    except (EllfErrors.EllfError, ValidationError, *HTTPXErrors):
+        # Stats are decorative; a mismatched PAM schema shouldn't fail `info`.
+        return stats
+    stats["annotations"] = counts.annotations
+    stats["sessions"] = counts.sessions
+    return stats
 
 
 @cli.subcommand(

{ellf_cli-18.0.28 → ellf_cli-18.0.30}/ellf_cli/commands/projects.py

@@ -1,5 +1,6 @@
 import builtins
 import logging
+import sys
 from collections.abc import Sequence
 from typing import Any, Optional, Union
 from uuid import UUID
@@ -19,6 +20,7 @@ from ellf_pam_sdk.models import (
 from ..auth import AuthState
 from ..cli import cli
 from ..errors import BrokerError, CLIError, EllfErrors, HTTPXErrors
+from ..interactivity import isatty
 from ..messages import Messages
 from ..query import resolve_project, resolve_project_id
 from ..ui import print_info_table, print_mutation_result, print_table_with_select
@@ -186,7 +188,9 @@ def update(
 )
 def delete(name_or_id: Union[str, UUID], as_json: bool = False) -> UUID:
     """Delete a project"""
-    project_id = resolve_project_id(name_or_id)
+    project = resolve_project(name_or_id)
+    project_id = project.id
+    _confirm_delete(project)
     auth = get_auth_state()
     _stop_project_jobs(auth, project_id)
     try:
@@ -205,6 +209,20 @@ def delete(name_or_id: Union[str, UUID], as_json: bool = False) -> UUID:
     return project_id
 
 
+def _confirm_delete(project: ProjectDetail) -> None:
+    """Type the project name back to confirm.
+
+    Refused without a terminal, so a script or an agent fails instead of
+    hanging on a prompt nothing will answer.
+    """
+    if not isatty(sys.stdin) or not isatty(sys.stdout):
+        raise CLIError(Messages.E193.format(name=project.name))
+    msg.warn(Messages.T071.format(name=project.name))
+    typed = input(Messages.project_delete_confirm).strip()
+    if typed != project.name:
+        raise CLIError(Messages.E194.format(name=project.name))
+
+
 def _stop_project_jobs(auth: AuthState, project_id: UUID) -> None:
     """Stop all cluster jobs for a project's tasks, actions, and agents."""
     job_ids: builtins.list[tuple[UUID, broker_models.JobType]] = []

{ellf_cli-18.0.28 → ellf_cli-18.0.30}/ellf_cli/commands/services.py

@@ -212,7 +212,13 @@ def info(
         except (BrokerError, *HTTPXErrors) as e:
             msg.warn(f"Could not fetch live status from the cluster: {e}")
     print_info_table(
-        res, exclude=["plan"], as_json=as_json, select=select, stats=_stats(res)
+        res,
+        exclude=["plan"],
+        as_json=as_json,
+        select=select,
+        stats=_stats(res),
+        # Noise in a table; --json and an explicit --select still carry it.
+        table_exclude=None if select else ["created_by_user"],
     )
     return res
 

{ellf_cli-18.0.28 → ellf_cli-18.0.30}/ellf_cli/commands/tasks.py

@@ -29,7 +29,6 @@ from ..query import (
     stop_job,
 )
 from ..ui import (
-    format_duration,
     print_args_table,
     print_info_table,
     print_logs,
@@ -225,8 +224,8 @@ def info(
         as_json=as_json,
         select=select,
         stats=_stats(res, as_json=as_json),
-        # Superseded by the stats block; --json still carries the ids.
-        table_exclude=["related_actions"],
+        # Noise in a table; --json and an explicit --select still carry them.
+        table_exclude=None if select else ["related_actions", "created_by_user"],
     )
     return res
 
@@ -239,14 +238,13 @@ def _stats(task: TaskDetail, *, as_json: bool) -> dict[str, Any]:
         # Stats are decorative. A PAM release whose stats schema differs from
         # this CLI's (older, or newer) should not take down `info`.
         return {"related_actions": len(task.related_actions)}
-    duration = stats.duration or 0
     return {
+        "annotations": stats.annotations,
         "annotators": stats.annotators,
         "agents": stats.agents or 0,
         "sessions": stats.sessions,
         "live_sessions": stats.live_sessions,
-        # Whole seconds for --json, as PAM reports it; readable in the table.
-        "duration": duration if as_json else format_duration(duration),
+        "duration": stats.duration,
         "related_actions": len(task.related_actions),
     }
 

{ellf_cli-18.0.28 → ellf_cli-18.0.30}/ellf_cli/commands/users.py

@@ -1,9 +1,10 @@
 import builtins
 from collections.abc import Sequence
 from datetime import datetime, timezone
-from typing import Literal, Optional, Union
+from typing import Any, Literal, Optional, Union
 from uuid import UUID
 
+from pydantic import ValidationError
 from radicli import Arg
 from wasabi import msg
 
@@ -25,7 +26,11 @@ from ..cli import cli
 from ..errors import CLIError, EllfErrors, HTTPXErrors
 from ..messages import Messages
 from ..query import _try_uuid, resolve_task
-from ..ui import print_info_table, print_mutation_result, print_table_with_select
+from ..ui import (
+    print_info_table,
+    print_mutation_result,
+    print_table_with_select,
+)
 from ._state import get_auth_state
 
 # Mirrors the role model of the web app's members page: a user is an
@@ -177,7 +182,7 @@ def info(
 ) -> UserDetail:
     """Get detailed info for a user"""
     res = _resolve_user(email_name_or_id)
-    print_info_table(res, as_json=as_json, select=select)
+    _print_user(res, select=select, as_json=as_json)
     return res
 
 
@@ -197,10 +202,43 @@ def whoami(
     """Show the currently authenticated user"""
     auth = get_auth_state()
     res = auth.client.user.read(id=auth.user_id)
-    print_info_table(res, as_json=as_json, select=select)
+    _print_user(res, select=select, as_json=as_json)
     return res
 
 
+def _print_user(
+    user: UserDetail, *, select: Optional[builtins.list[str]], as_json: bool
+) -> None:
+    """The shared body of `users info` and `users whoami`."""
+    print_info_table(
+        user,
+        as_json=as_json,
+        select=select,
+        stats=_stats(user, as_json=as_json),
+        # Noise in a table; --json and an explicit --select still carry them.
+        table_exclude=None if select else ["groups", "tags"],
+    )
+
+
+def _stats(user: UserDetail, *, as_json: bool) -> dict[str, Any]:
+    """This member's org-wide annotation stats, for the info block."""
+    client = get_auth_state().client
+    try:
+        stats = client.user.stats(user.id)
+    except (EllfErrors.EllfError, ValidationError, *HTTPXErrors):
+        # Stats are decorative; a mismatched PAM schema shouldn't fail `info`.
+        return {}
+    return {
+        "projects": stats.projects,
+        "tasks": stats.tasks,
+        "sessions": stats.sessions,
+        "annotations": stats.annotations,
+        "duration": stats.duration,
+        "last_annotated": stats.last_annotated,
+        "last_active": stats.last_active,
+    }
+
+
 @cli.subcommand(
     "users",
     "update",

{ellf_cli-18.0.28 → ellf_cli-18.0.30}/ellf_cli/ellf.json

@@ -1,7 +1,7 @@
 {
   "prog": "ellf",
   "help": "Ellf Command Line Interface.",
-  "version": "18.0.27",
+  "version": "18.0.29",
   "extra_key": "_extra",
   "commands": {
     "actions": {

{ellf_cli-18.0.28 → ellf_cli-18.0.30}/ellf_cli/ellf_skills/skills/ellf-prodigy/assets/templates/template_agent_recipe.py

@@ -68,8 +68,8 @@ def create_llm_client(api_key: str, model_name: str):
 #
 # The SDK reads poll_interval and idle_shutdown_seconds from the recipe
 # parameters automatically — you don't need to handle them yourself. The
-# same goes for icon: this is the avatar used in the app wherever the agent's 
-# annotations are attributed to it. Pick the default that suits your agent — the 
+# same goes for icon: this is the avatar used in the app wherever the agent's
+# annotations are attributed to it. Pick the default that suits your agent — the
 # model provider it calls, or what it does — from the set in props.icon.
 #
 # Agents are created and managed like tasks and actions:

{ellf_cli-18.0.28 → ellf_cli-18.0.30}/ellf_cli/messages.py

@@ -142,6 +142,9 @@ ellf secret create my-credentials OPENAI_API_KEY="sk-..." CUSTOM_SECRET=-
     T068 = "Select your plan here:\n{url}"
     T069 = "Your free trial has ended"
     T070 = "Today is the last day of your free trial"
+    T071 = "This deletes {name} and everything in it: tasks, actions, agents, services, project plans and annotations. Any annotated datasets stay on your cluster."
+
+    project_delete_confirm = "Type the project name to confirm: "
 
     # CLI docstrings for placeholders (parent commands with no functions)
     doc_todos = "Manage todos in your Ellf projects"
@@ -302,6 +305,8 @@ ellf secret create my-credentials OPENAI_API_KEY="sk-..." CUSTOM_SECRET=-
     E190 = "No project set, so there is nothing to list {plural} for. Pass --project, or set a default with `ellf config project`"
     E191 = "Could not create {noun} {name}: input validation failed"
     E192 = "Annotators can't log in via the CLI. Ask an admin of your organization '{org}' to upgrade your role to developer, then run `{command}` again."
+    E193 = "Deleting a project has to be confirmed via a terminal. Run `ellf projects delete {name}` again in an interactive terminal session."
+    E194 = "Wrong project name, so {name} was not deleted"
 
 
 #  fmt: on

{ellf_cli-18.0.28 → ellf_cli-18.0.30}/ellf_cli/ui.py

@@ -150,21 +150,6 @@ def _json_default(value: Any) -> Any:
     return str(value)
 
 
-def format_duration(seconds: int) -> str:
-    """A span of seconds as ``1h 30m``, ``45m`` or ``0m``.
-
-    Annotation time is reported in whole seconds, but nobody reads a session
-    to the second: minutes are the finest unit shown.
-    """
-    minutes, _ = divmod(max(0, int(seconds)), 60)
-    hours, minutes = divmod(minutes, 60)
-    if hours and minutes:
-        return f"{hours}h {minutes}m"
-    if hours:
-        return f"{hours}h"
-    return f"{minutes}m"
-
-
 def print_info_table(
     item: BaseModel,
     select: list[str] | None = None,

ellf_cli-18.0.30/ellf_cli/version/18.0.28_2026-09-21-09-44-01_patch_4430a721-0b3c-4f6a-b9bc-a53e667161bb

@@ -0,0 +1,3 @@
+18.0.28
+patch
+4430a721-0b3c-4f6a-b9bc-a53e667161bb

ellf_cli-18.0.30/ellf_cli/version/18.0.29_2026-09-21-10-58-07_patch_83b2b942-60a4-4bef-afd1-99690ba52b69

@@ -0,0 +1,3 @@
+18.0.29
+patch
+83b2b942-60a4-4bef-afd1-99690ba52b69

{ellf_cli-18.0.28 → ellf_cli-18.0.30/ellf_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ellf-cli
-Version: 18.0.28
+Version: 18.0.30
 Summary: Ellf Command Line Interface
 Home-page: https://prodi.gy
 Author: ExplosionAI GmbH

{ellf_cli-18.0.28 → ellf_cli-18.0.30}/ellf_cli.egg-info/SOURCES.txt

@@ -281,6 +281,8 @@ ellf_cli/version/18.0.24_2026-09-20-15-49-27_patch_c1222081-ff44-4f20-aea2-5840b
 ellf_cli/version/18.0.25_2026-09-21-06-11-40_patch_6df46dd9-e7d3-4e71-bec4-e5c93b4ac863
 ellf_cli/version/18.0.26_2026-09-21-07-54-11_patch_cc52dd36-5eba-4094-a1b0-d6f79f103441
 ellf_cli/version/18.0.27_2026-09-21-08-36-49_patch_a25fa125-9bcd-4317-8da5-851c1c329631
+ellf_cli/version/18.0.28_2026-09-21-09-44-01_patch_4430a721-0b3c-4f6a-b9bc-a53e667161bb
+ellf_cli/version/18.0.29_2026-09-21-10-58-07_patch_83b2b942-60a4-4bef-afd1-99690ba52b69
 ellf_cli/version/18.0.2_2026-09-13-06-36-33_patch_176bb944-1c49-4dd3-a001-8fce908825a0
 ellf_cli/version/18.0.3_2026-09-13-12-32-46_patch_58e5ce3a-f2a7-4789-b2cd-5cc75c72b6ec
 ellf_cli/version/18.0.4_2026-09-13-12-45-14_patch_b6c79d55-8f85-4f47-a304-df52b122c38c
@@ -291,6 +293,7 @@ ellf_cli/version/18.0.7_2026-09-13-17-06-08_patch_428af161-0b27-43c2-8751-e5f787
 ellf_cli/version/18.0.9_2026-09-14-12-37-42_patch_b2ca0529-4b67-4fa4-8fdc-5f5e9f5d5166
 ellf_cli/version/__init__.py
 ellf_cli/version/read_version.py
+tests/test_agents_info_stats.py
 tests/test_appdirs.py
 tests/test_auth.py
 tests/test_config.py
@@ -312,6 +315,7 @@ tests/test_infra_state_bucket.py
 tests/test_infra_terraform_init.py
 tests/test_invalid_secrets.py
 tests/test_job_list_project_filter.py
+tests/test_jobs_info_hides_creator.py
 tests/test_jobs_list_unmatched.py
 tests/test_key_pair.py
 tests/test_login.py
@@ -320,6 +324,7 @@ tests/test_main.py
 tests/test_org_select.py
 tests/test_plans.py
 tests/test_projects.py
+tests/test_projects_delete_confirm.py
 tests/test_publish_build_poll.py
 tests/test_publish_data.py
 tests/test_publish_import_root.py
@@ -336,5 +341,6 @@ tests/test_ui.py
 tests/test_ui_extras.py
 tests/test_usage_counts.py
 tests/test_users_assign.py
+tests/test_users_info_stats.py
 tests/test_users_list.py
 tests/test_util.py

ellf_cli-18.0.30/tests/test_agents_info_stats.py

@@ -0,0 +1,69 @@
+"""`agents info` reports what the agent has annotated, as the app's table does."""
+
+from datetime import datetime, timezone
+from types import SimpleNamespace
+from typing import Any, cast
+from uuid import uuid4
+
+import pytest
+
+from ellf_cli.commands import agents as agents_cmd
+from ellf_cli.errors import EllfErrors
+from ellf_pam_sdk.models import AgentAnnotationStats, AgentDetail
+
+STARTED = datetime(2026, 9, 20, 8, 46, tzinfo=timezone.utc)
+
+
+@pytest.fixture
+def stats_of(monkeypatch):
+    """Build an agent's stats block against a fixed annotation-counts reply."""
+
+    def _run(counts: Any, tasks: int = 2) -> dict[str, Any]:
+        agent_id = uuid4()
+        # _stats reads three attributes; a full AgentDetail would need the
+        # cluster and recipe plumbing none of them touch.
+        agent = cast(
+            AgentDetail,
+            SimpleNamespace(
+                id=agent_id,
+                related_tasks=[object()] * tasks,
+                last_execution_started=STARTED,
+            ),
+        )
+
+        def _counts(_id):
+            if isinstance(counts, Exception):
+                raise counts
+            return counts
+
+        auth = SimpleNamespace(
+            client=SimpleNamespace(agent=SimpleNamespace(annotation_counts=_counts))
+        )
+        monkeypatch.setattr(agents_cmd, "get_auth_state", lambda: auth)
+        return agents_cmd._stats(agent)
+
+    return _run
+
+
+def test_annotations_and_sessions_join_the_block(stats_of):
+    counts = AgentAnnotationStats(agent_id=uuid4(), annotations=450, sessions=3)
+    assert stats_of(counts) == {
+        "tasks": 2,
+        "last_started": STARTED,
+        "annotations": 450,
+        "sessions": 3,
+    }
+
+
+def test_unreported_annotations_pass_through_as_none(stats_of):
+    """None means no cluster has reported, which is not a zero."""
+    counts = AgentAnnotationStats(agent_id=uuid4(), annotations=None, sessions=0)
+    assert stats_of(counts)["annotations"] is None
+
+
+def test_counts_failure_leaves_the_rest_of_the_block(stats_of):
+    """Stats are decorative: the counts are lost, not the command."""
+    assert stats_of(EllfErrors.EllfError("boom")) == {
+        "tasks": 2,
+        "last_started": STARTED,
+    }

ellf_cli-18.0.30/tests/test_jobs_info_hides_creator.py

@@ -0,0 +1,59 @@
+"""`<job> info` keeps `created_by_user` out of the default table."""
+
+from types import SimpleNamespace
+from typing import Any
+
+import pytest
+
+from ellf_cli.commands import actions as actions_cmd
+from ellf_cli.commands import agents as agents_cmd
+from ellf_cli.commands import services as services_cmd
+from ellf_cli.commands import tasks as tasks_cmd
+
+# module, the name it resolves the job with, and what it hides by default
+JOBS = [
+    (tasks_cmd, "resolve_task", ["related_actions", "created_by_user"]),
+    (actions_cmd, "resolve_action", ["stats", "related_tasks", "created_by_user"]),
+    (agents_cmd, "resolve_agent", ["related_tasks", "created_by_user"]),
+    (services_cmd, "resolve_service", ["created_by_user"]),
+]
+
+
+@pytest.fixture
+def shown(monkeypatch):
+    """Run one job's `info` and capture what it asks to be printed."""
+
+    def _run(module: Any, resolver: str, **kwargs: Any) -> dict[str, Any]:
+        captured: dict[str, Any] = {}
+        job = SimpleNamespace(id="job-id")
+        status = SimpleNamespace(state=SimpleNamespace(value="ready"))
+        auth = SimpleNamespace(
+            broker_client=SimpleNamespace(
+                jobs=SimpleNamespace(get_status=lambda _id: status)
+            )
+        )
+        monkeypatch.setattr(module, "get_auth_state", lambda: auth)
+        monkeypatch.setattr(module, resolver, lambda *a, **k: job)
+        monkeypatch.setattr(module, "_stats", lambda *a, **k: {})
+        monkeypatch.setattr(
+            module,
+            "print_info_table",
+            lambda item, **kw: captured.update(item=item, **kw),
+        )
+        module.info("whatever", **kwargs)
+        return captured
+
+    return _run
+
+
+@pytest.mark.parametrize("module,resolver,hidden", JOBS)
+def test_creator_is_hidden_from_the_default_table(shown, module, resolver, hidden):
+    assert shown(module, resolver)["table_exclude"] == hidden
+
+
+@pytest.mark.parametrize("module,resolver,hidden", JOBS)
+def test_naming_your_own_columns_keeps_them(shown, module, resolver, hidden):
+    """table_exclude would otherwise beat the include and empty the table."""
+    res = shown(module, resolver, select=["created_by_user"])
+    assert res["table_exclude"] is None
+    assert res["select"] == ["created_by_user"]

ellf_cli-18.0.30/tests/test_projects_delete_confirm.py

@@ -0,0 +1,101 @@
+"""`projects delete` must be confirmed by hand, and must fail without a TTY."""
+
+from datetime import datetime, timezone
+from uuid import uuid4
+
+import pytest
+
+from ellf_cli.commands import projects as projects_cmd
+from ellf_cli.errors import CLIError
+from ellf_pam_sdk.models import ProjectDetail
+
+NAME = "my-project"
+
+
+def _project(name: str = NAME) -> ProjectDetail:
+    now = datetime.now(timezone.utc)
+    return ProjectDetail(
+        id=uuid4(),
+        created=now,
+        updated=now,
+        org_id=uuid4(),
+        name=name,
+        slug=name,
+        description="",
+    )
+
+
+class _Delete:
+    """Calls `projects delete` under a chosen set of terminals, and carries
+    the record of how far it got -- which the tests that expect a refusal
+    have to read off the fixture, since the call raises before returning."""
+
+    def __init__(self, monkeypatch, reached: list[str]):
+        self._monkeypatch = monkeypatch
+        self.reached = reached
+
+    def __call__(self, *, stdin_tty: bool, stdout_tty: bool, typed: str = NAME):
+        self._monkeypatch.setattr(
+            projects_cmd,
+            "isatty",
+            lambda stream: (
+                stdin_tty if stream is projects_cmd.sys.stdin else stdout_tty
+            ),
+        )
+        self._monkeypatch.setattr("builtins.input", lambda _prompt="": typed)
+        return projects_cmd.delete(NAME), self.reached
+
+
+@pytest.fixture
+def run(monkeypatch):
+    """Call `projects delete`, recording whether it got past the prompt."""
+    project = _project()
+    reached: list[str] = []
+
+    monkeypatch.setattr(projects_cmd, "resolve_project", lambda _: project)
+    monkeypatch.setattr(
+        projects_cmd,
+        "_stop_project_jobs",
+        lambda *a, **k: reached.append("stopped"),
+    )
+    monkeypatch.setattr(projects_cmd, "get_auth_state", lambda: _FakeAuth(reached))
+    monkeypatch.setattr(projects_cmd, "print_mutation_result", lambda *a, **k: None)
+    return _Delete(monkeypatch, reached)
+
+
+class _FakeAuth:
+    def __init__(self, reached):
+        self.client = self
+        self.project = self
+        self._reached = reached
+
+    def delete(self, id):  # noqa: A002 - matches the SDK's signature
+        self._reached.append("deleted")
+
+
+def test_refuses_without_a_terminal(run):
+    with pytest.raises(CLIError, match="has to be confirmed via a terminal"):
+        run(stdin_tty=False, stdout_tty=True)
+    assert run.reached == []
+
+
+def test_refuses_when_stdout_is_piped(run):
+    with pytest.raises(CLIError, match="has to be confirmed via a terminal"):
+        run(stdin_tty=True, stdout_tty=False)
+    assert run.reached == []
+
+
+def test_refuses_a_name_that_does_not_match(run):
+    with pytest.raises(CLIError, match="Wrong project name"):
+        run(stdin_tty=True, stdout_tty=True, typed="My-Project")
+    assert run.reached == []
+
+
+def test_deletes_once_the_name_is_typed(run):
+    _, reached = run(stdin_tty=True, stdout_tty=True, typed=NAME)
+    assert reached == ["stopped", "deleted"]
+
+
+def test_accepts_the_name_with_stray_whitespace(run):
+    _, reached = run(stdin_tty=True, stdout_tty=True, typed=f"  {NAME}\n")
+    assert reached == ["stopped", "deleted"]

{ellf_cli-18.0.28 → ellf_cli-18.0.30}/tests/test_ui_extras.py

@@ -14,7 +14,6 @@ from pydantic import BaseModel
 
 from ellf_cli.ui import (
     dicts_to_table,
-    format_duration,
     is_silent,
     isatty,
     print_args_table,
@@ -215,21 +214,6 @@ class TestPrintInfoTable:
         assert json.loads(capsys.readouterr().out)["age"] == 30
 
 
-class TestFormatDuration:
-    def test_minutes_are_the_finest_unit(self):
-        assert format_duration(0) == "0m"
-        assert format_duration(59) == "0m"
-        assert format_duration(180) == "3m"
-        assert format_duration(5400) == "1h 30m"
-
-    def test_whole_hours_drop_the_minutes(self):
-        assert format_duration(7200) == "2h"
-        assert format_duration(3600 * 1824 + 24 * 60) == "1824h 24m"
-
-    def test_never_negative(self):
-        assert format_duration(-30) == "0m"
-
-
 class TestPrintTableWithSelect:
     def test_basic(self, capsys):
         items = [_SampleModel(name="Alice", age=30)]