@opengsd/gsd-pi 1.3.0 → 1.4.0

package/integrations/hermes/tests/test_read_cli_contract.py

@@ -0,0 +1,50 @@
+"""Contract test: gsd read progress against minimal-project fixture."""
+
+from __future__ import annotations
+
+import json
+import os
+import subprocess
+from pathlib import Path
+
+import pytest
+
+FIXTURE = Path(__file__).parent / "fixtures" / "minimal-project"
+REPO_ROOT = Path(__file__).resolve().parents[3]
+GSD_CLI = os.environ.get("GSD_CLI_PATH", "node")
+GSD_LOADER = os.environ.get(
+    "GSD_LOADER_PATH", str(REPO_ROOT / "dist" / "loader.js")
+)
+
+
+def _gsd_cmd(*args: str) -> list[str]:
+    if GSD_CLI == "node" or GSD_CLI.endswith("node"):
+        return [GSD_CLI, GSD_LOADER, *args]
+    return [GSD_CLI, *args]
+
+
+def test_read_cli_progress_fixture() -> None:
+    if not Path(GSD_LOADER).exists():
+        pytest.skip(f"GSD loader not built: {GSD_LOADER}")
+
+    result = subprocess.run(
+        _gsd_cmd(
+            "read",
+            "progress",
+            "--json",
+            "--project",
+            str(FIXTURE.resolve()),
+        ),
+        capture_output=True,
+        text=True,
+        check=False,
+        timeout=30,
+        cwd=str(REPO_ROOT),
+    )
+    assert result.returncode == 0, result.stderr or result.stdout
+    envelope = json.loads(result.stdout)
+    assert envelope["integration_version"] == 1
+    assert envelope["kind"] == "progress"
+    data = envelope["data"]
+    assert data.get("activeMilestone", {}).get("id") == "M001"
+    assert "Hermes Integration" in (data.get("activeMilestone", {}).get("title") or "")

package/integrations/hermes/tests/test_register.py

@@ -0,0 +1,56 @@
+"""Registration integration tests."""
+
+from __future__ import annotations
+
+import asyncio
+from pathlib import Path
+from typing import Any, Callable
+from unittest.mock import MagicMock
+
+import open_gsd_hermes
+from open_gsd_hermes.config import GsdConfig
+from open_gsd_hermes.types import ProgressSnapshot
+
+
+class FakeContext:
+    def __init__(self) -> None:
+        self.commands: dict[str, Callable[..., Any]] = {}
+        self.hooks: dict[str, Callable[..., Any]] = {}
+
+    def register_command(self, name: str, handler: Callable[..., Any]) -> None:
+        self.commands[name] = handler
+
+    def register_hook(self, name: str, handler: Callable[..., Any]) -> None:
+        self.hooks[name] = handler
+
+    def register_memory_provider(self, _provider: Any) -> None:
+        pass
+
+    def dispatch_tool(self, _name: str, _arguments: dict[str, Any]) -> None:
+        pass
+
+
+def test_pre_llm_call_uses_bound_session_project(tmp_path: Path, monkeypatch) -> None:
+    default_project = tmp_path / "default"
+    bound_project = tmp_path / "bound"
+    (default_project / ".gsd").mkdir(parents=True)
+    (bound_project / ".gsd").mkdir(parents=True)
+    client = MagicMock()
+    client.progress.return_value = ProgressSnapshot(phase="execute")
+    ctx = FakeContext()
+
+    monkeypatch.setenv("HERMES_SESSION_KEY", "agent:main:cli:direct:local")
+    monkeypatch.setattr(
+        open_gsd_hermes,
+        "load_config",
+        lambda: GsdConfig(default_project=str(default_project)),
+    )
+    monkeypatch.setattr(open_gsd_hermes, "GsdMcpClient", lambda _config: client)
+
+    open_gsd_hermes.register(ctx)
+    bind_result = asyncio.run(ctx.commands["gsd"](f"bind {bound_project}"))
+    hook_result = ctx.hooks["pre_llm_call"]()
+
+    assert bind_result == f"Bound to `{bound_project.resolve()}`"
+    assert hook_result["context"].startswith("## GSD Project Snapshot")
+    client.progress.assert_called_once_with(str(bound_project.resolve()))

package/integrations/hermes/tests/test_snapshot.py

@@ -0,0 +1,53 @@
+"""Golden snapshot test for pre_llm_call formatting."""
+
+from __future__ import annotations
+
+from pathlib import Path
+
+from open_gsd_hermes.formatting import format_ref
+from open_gsd_hermes.snapshot import format_snapshot
+from open_gsd_hermes.types import ProgressSnapshot
+
+GOLDEN = Path(__file__).parent / "golden" / "snapshot.txt"
+
+
+def test_snapshot_golden() -> None:
+    progress = ProgressSnapshot(
+        active_milestone={"id": "M001", "title": "Hermes Integration"},
+        active_slice={"id": "S01", "title": "Gateway MVP"},
+        active_task={"id": "T01", "title": "Plugin scaffold"},
+        phase="execute",
+        milestones={"total": 1, "done": 0, "active": 1, "pending": 0, "parked": 0},
+        slices={"total": 0, "done": 0, "active": 0, "pending": 0},
+        tasks={"total": 0, "done": 0, "pending": 0},
+        requirements={"active": 2, "validated": 0, "deferred": 0, "outOfScope": 0},
+        blockers=[],
+        next_action="Run contract tests for binding and snapshot golden.",
+    )
+    rendered = format_snapshot(progress)
+    expected = GOLDEN.read_text(encoding="utf-8").strip()
+    # Phase maps "execution" -> execute in MCP; golden uses execute
+    assert rendered.strip() == expected
+
+
+def test_snapshot_omits_duplicate_id_when_title_missing() -> None:
+    progress = ProgressSnapshot(
+        active_milestone={"id": "M001"},
+        active_slice={"id": "S01", "title": ""},
+        active_task={"id": "T01", "title": "Plugin scaffold"},
+        phase="execute",
+    )
+
+    rendered = format_snapshot(progress)
+
+    assert "Active milestone: M001" in rendered
+    assert "Active slice: S01" in rendered
+    assert "M001: M001" not in rendered
+    assert "S01: S01" not in rendered
+
+
+def test_format_ref_supports_snapshot_and_transition_styles() -> None:
+    ref = {"id": "M001", "title": "Hermes Integration"}
+
+    assert format_ref(ref) == "M001: Hermes Integration"
+    assert format_ref(ref, include_title=False) == "M001"