refactor: make format_duration a public function since it's used cross-module

The function was prefixed with _ (marking it private) but imported by cli.py
and both test files. Removing the underscore makes the API boundary honest.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: 2 people

src/ralphify/cli.py

@@ -19,7 +19,7 @@
 from ralphify._frontmatter import CHECK_MARKER, CONTEXT_MARKER, INSTRUCTION_MARKER, PROMPT_MARKER
 from ralphify.checks import discover_checks
 from ralphify.contexts import discover_contexts
-from ralphify.engine import RunConfig, RunState, _format_duration, run_loop
+from ralphify.engine import RunConfig, RunState, format_duration, run_loop
 from ralphify.instructions import discover_instructions
 from ralphify.prompts import discover_prompts, is_prompt_name, resolve_prompt_name
 from ralphify.detector import detect_project
@@ -296,7 +296,7 @@ def emit(self, event: Event) -> None:
 
     def _on_run_started(self, d: dict) -> None:
         if d.get("timeout"):
-            self._rprint(f"[dim]Timeout: {_format_duration(d['timeout'])} per iteration[/dim]")
+            self._rprint(f"[dim]Timeout: {format_duration(d['timeout'])} per iteration[/dim]")
         if d.get("checks"):
             self._rprint(f"[dim]Checks: {d['checks']} enabled[/dim]")
         if d.get("contexts"):

src/ralphify/engine.py

@@ -121,7 +121,7 @@ def consume_reload_request(self) -> bool:
         return False
 
 
-def _format_duration(seconds: float) -> str:
+def format_duration(seconds: float) -> str:
     """Format duration in human-readable form."""
     if seconds < 60:
         return f"{seconds:.1f}s"
@@ -270,7 +270,7 @@ def _execute_agent(
             log_file = _write_log(log_path_dir, iteration, e.stdout, e.stderr)
 
     elapsed = time.monotonic() - start
-    duration = _format_duration(elapsed)
+    duration = format_duration(elapsed)
 
     if returncode is None:
         event_type = EventType.ITERATION_TIMED_OUT

tests/test_cli.py

@@ -10,7 +10,7 @@
 from ralphify.contexts import Context, ContextResult
 from ralphify.cli import app, CONFIG_FILENAME
 from ralphify._templates import RALPH_TOML_TEMPLATE, PROMPT_TEMPLATE
-from ralphify.engine import _format_duration
+from ralphify.engine import format_duration
 
 runner = CliRunner()
 
@@ -531,18 +531,18 @@ def test_timeout_shows_in_header(self, mock_run, tmp_path, monkeypatch):
 
 class TestFormatDuration:
     def test_seconds(self):
-        assert _format_duration(5.3) == "5.3s"
-        assert _format_duration(0.1) == "0.1s"
-        assert _format_duration(59.9) == "59.9s"
+        assert format_duration(5.3) == "5.3s"
+        assert format_duration(0.1) == "0.1s"
+        assert format_duration(59.9) == "59.9s"
 
     def test_minutes(self):
-        assert _format_duration(60) == "1m 0s"
-        assert _format_duration(90.5) == "1m 30s"
-        assert _format_duration(3599) == "59m 59s"
+        assert format_duration(60) == "1m 0s"
+        assert format_duration(90.5) == "1m 30s"
+        assert format_duration(3599) == "59m 59s"
 
     def test_hours(self):
-        assert _format_duration(3600) == "1h 0m"
-        assert _format_duration(5400) == "1h 30m"
+        assert format_duration(3600) == "1h 0m"
+        assert format_duration(5400) == "1h 30m"
 
 
 def _setup_check(tmp_path, name="ruff-lint", command="ruff check .", enabled=True,

tests/test_engine.py

@@ -7,7 +7,7 @@
 from unittest.mock import patch
 
 from ralphify._events import EventType, NullEmitter, QueueEmitter
-from ralphify.engine import RunConfig, RunState, RunStatus, _format_duration, run_loop
+from ralphify.engine import RunConfig, RunState, RunStatus, format_duration, run_loop
 
 _MOCK_SUBPROCESS = "ralphify.engine.subprocess.run"
 
@@ -282,15 +282,15 @@ def stop_later():
 
 class TestFormatDuration:
     def test_seconds(self):
-        assert _format_duration(5.3) == "5.3s"
-        assert _format_duration(0.1) == "0.1s"
-        assert _format_duration(59.9) == "59.9s"
+        assert format_duration(5.3) == "5.3s"
+        assert format_duration(0.1) == "0.1s"
+        assert format_duration(59.9) == "59.9s"
 
     def test_minutes(self):
-        assert _format_duration(60) == "1m 0s"
-        assert _format_duration(90.5) == "1m 30s"
-        assert _format_duration(3599) == "59m 59s"
+        assert format_duration(60) == "1m 0s"
+        assert format_duration(90.5) == "1m 30s"
+        assert format_duration(3599) == "59m 59s"
 
     def test_hours(self):
-        assert _format_duration(3600) == "1h 0m"
-        assert _format_duration(5400) == "1h 30m"
+        assert format_duration(3600) == "1h 0m"
+        assert format_duration(5400) == "1h 30m"