cowork-bot: versions command silent-green fix — _pip_version() helper distinguishes not-installed vs missing version metadata; explicit 'no version metadata'/'error checking' lines instead of silent skip; +5 tests (24 pass, ruff clean)

Author: Coding-Dev-Tools

src/devforge/cli.py

@@ -128,19 +128,35 @@ def show_versions(
     for t in targets:
         info = TOOLS[t]
         try:
-            result = subprocess.run(
-                [sys.executable, "-m", "pip", "show", info["package"]], capture_output=True, text=True
-            )
-            if result.returncode == 0:
-                for line in result.stdout.splitlines():
-                    if line.startswith("Version:"):
-                        ver = line.split(":", 1)[1].strip()
-                        console.print(f"[cyan]{t:8}[/cyan] v{ver}")
-                        break
-            else:
-                console.print(f"[dim]{t:8}[/dim] [red]not installed[/red]")
-        except Exception:
-            console.print(f"[dim]{t:8}[/dim] [red]error checking[/red]")
+            ver = _pip_version(info["package"])
+        except Exception as e:
+            console.print(f"[dim]{t:8}[/dim] [red]error checking ({e})[/red]")
+            continue
+        if ver is None:
+            console.print(f"[dim]{t:8}[/dim] [red]not installed[/red]")
+        elif ver == "":
+            # pip show succeeded but returned no Version metadata — never stay silent.
+            console.print(f"[dim]{t:8}[/dim] [yellow]installed, no version metadata[/yellow]")
+        else:
+            console.print(f"[cyan]{t:8}[/cyan] v{ver}")
+
+
+def _pip_version(package: str) -> str | None:
+    """Return the installed version of *package*, or None if not installed.
+
+    Returns "" when ``pip show`` succeeds but the output carries no
+    ``Version:`` line (broken metadata) so callers can distinguish it from a
+    clean not-installed result instead of silently printing nothing.
+    """
+    result = subprocess.run(
+        [sys.executable, "-m", "pip", "show", package], capture_output=True, text=True
+    )
+    if result.returncode != 0:
+        return None
+    for line in result.stdout.splitlines():
+        if line.startswith("Version:"):
+            return line.split(":", 1)[1].strip()
+    return ""
 
 
 def _is_tool_installed(module_name: str) -> bool:

tests/test_cli.py

@@ -3,7 +3,7 @@
 from __future__ import annotations
 
 from devforge import TOOLS, __version__
-from devforge.cli import _is_tool_installed, app
+from devforge.cli import _is_tool_installed, _pip_version, app
 from typer.testing import CliRunner
 from unittest import mock
 
@@ -176,3 +176,35 @@ def test_help(self):
         assert "tools" in result.stdout
         assert "versions" in result.stdout
         assert "guard" in result.stdout
+
+
+class TestPipVersionHelper:
+    @mock.patch("devforge.cli.subprocess.run")
+    def test_returns_version_line(self, mock_run):
+        """Parse Version: out of successful pip show output."""
+        mock_run.return_value = mock.MagicMock(returncode=0, stdout="Name: x\nVersion: 1.2.3\n")
+        assert _pip_version("x") == "1.2.3"
+
+    @mock.patch("devforge.cli.subprocess.run")
+    def test_not_installed_returns_none(self, mock_run):
+        mock_run.return_value = mock.MagicMock(returncode=1, stdout="", stderr="not found")
+        assert _pip_version("x") is None
+
+    @mock.patch("devforge.cli.subprocess.run")
+    def test_missing_metadata_returns_empty(self, mock_run):
+        """pip show success without a Version line must NOT look like not installed."""
+        mock_run.return_value = mock.MagicMock(returncode=0, stdout="Name: x\n")
+        assert _pip_version("x") == ""
+
+    @mock.patch("devforge.cli._pip_version", return_value="")
+    def test_versions_reports_missing_metadata(self, _mock):
+        """Silent-green regression guard: broken metadata gets an explicit line."""
+        result = runner.invoke(app, ["versions", "guard"])
+        assert result.exit_code == 0
+        assert "no version metadata" in result.stdout
+
+    @mock.patch("devforge.cli._pip_version", side_effect=OSError("boom"))
+    def test_versions_reports_error(self, _mock):
+        result = runner.invoke(app, ["versions", "guard"])
+        assert result.exit_code == 0
+        assert "error checking" in result.stdout