Merge pull request #65 from pollen-robotics/64-fix-the-format-timestamp-function

.github/workflows/tests.yml

@@ -5,6 +5,7 @@ on:
 
 permissions:
   contents: read
+  actions: write
 
 concurrency:
   group: ${{ github.workflow }}-${{ github.ref }}
@@ -20,6 +21,11 @@ jobs:
       matrix:
         python-version: ["3.12"]
 
+    env:
+      HF_TOKEN: ${{ secrets.HF_TOKEN }}
+      HF_HUB_ETAG_TIMEOUT: "120"
+      HF_HUB_DOWNLOAD_TIMEOUT: "120"
+
     steps:
       - uses: actions/checkout@v4
 
@@ -29,11 +35,40 @@ jobs:
 
       - uses: astral-sh/setup-uv@v5
 
+      - name: Set HF_HOME
+        shell: bash
+        run: |
+          echo "HF_HOME=${RUNNER_TEMP}/.hf" >> "$GITHUB_ENV"
+          mkdir -p "${RUNNER_TEMP}/.hf"
+
+      - name: Cache Hugging Face hub
+        uses: actions/cache@v4
+        with:
+          path: ${{ runner.temp }}/.hf
+          key: hf-${{ runner.os }}-${{ hashFiles('uv.lock', 'pyproject.toml') }}
+          restore-keys: hf-${{ runner.os }}-
+
+      # test-only .env file
+      - name: Create test .env
+        run: |
+          printf "OPENAI_API_KEY=test-dummy\n" > .env
+
       - name: Install (locked)
-        env:
-          GIT_LFS_SKIP_SMUDGE: "1"
         run: |
           uv sync --frozen --group dev --extra all_vision
 
+      # Prefetch HF dataset to avoid download during test collection
+      - name: Prefetch HF dataset
+        run: |
+          uv run python - <<'PY'
+          from huggingface_hub import snapshot_download
+          snapshot_download(
+              repo_id="pollen-robotics/reachy-mini-emotions-library",
+              repo_type="dataset",
+              etag_timeout=120,
+              max_workers=4
+          )
+          PY
+
       - name: Run tests
         run: uv run pytest -q

src/reachy_mini_conversation_demo/openai_realtime.py

@@ -239,14 +239,12 @@ class OpenaiRealtimeHandler(AsyncStreamHandler):
             self.connection = None
 
     def format_timestamp(self) -> str:
-        """Format current timestamp with date, time and elapsed seconds."""
-        current_time = asyncio.get_event_loop().time()
-        elapsed_seconds = current_time - self.start_time
-        dt = datetime.fromtimestamp(current_time)
+        """Format current timestamp with date, time, and elapsed seconds."""
+        loop_time = asyncio.get_event_loop().time()  # monotonic
+        elapsed_seconds = loop_time - self.start_time
+        dt = datetime.now()  # wall-clock
         return f"[{dt.strftime('%Y-%m-%d %H:%M:%S')} | +{elapsed_seconds:.1f}s]"
 
-
-
     async def send_idle_signal(self, idle_duration: float) -> None:
         """Send an idle signal to the openai server."""
         logger.debug("Sending idle signal")

tests/test_openai_realtime.py

@@ -0,0 +1,29 @@
+import asyncio
+from datetime import datetime, timezone
+from unittest.mock import MagicMock
+
+from reachy_mini_conversation_demo.tools import ToolDependencies
+from reachy_mini_conversation_demo.openai_realtime import OpenaiRealtimeHandler
+
+
+def _build_handler(loop: asyncio.AbstractEventLoop) -> OpenaiRealtimeHandler:
+    asyncio.set_event_loop(loop)
+    deps = ToolDependencies(reachy_mini=MagicMock(), movement_manager=MagicMock())
+    return OpenaiRealtimeHandler(deps)
+
+
+def test_format_timestamp_uses_wall_clock() -> None:
+    """Test that format_timestamp uses wall clock time."""
+    loop = asyncio.new_event_loop()
+    try:
+        print("Testing format_timestamp...")
+        handler = _build_handler(loop)
+        formatted = handler.format_timestamp()
+        print(f"Formatted timestamp: {formatted}")
+    finally:
+        asyncio.set_event_loop(None)
+        loop.close()
+
+    # Extract year from "[YYYY-MM-DD ...]"
+    year = int(formatted[1:5])
+    assert year == datetime.now(timezone.utc).year