minitest-cli 0.8.0__tar.gz → 0.8.2__tar.gz

{minitest_cli-0.8.0 → minitest_cli-0.8.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: minitest-cli
-Version: 0.8.0
+Version: 0.8.2
 Summary: Minitest CLI – command-line interface for the Minitest testing platform
 Project-URL: Homepage, https://minitap.ai/
 Project-URL: Source, https://github.com/minitap-ai/minitest-cli

{minitest_cli-0.8.0 → minitest_cli-0.8.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "minitest-cli"
-version = "0.8.0"
+version = "0.8.2"
 description = "Minitest CLI – command-line interface for the Minitest testing platform"
 readme = "README.md"
 license = "MIT"

{minitest_cli-0.8.0 → minitest_cli-0.8.2}/src/minitest_cli/commands/auth.py

@@ -1,6 +1,7 @@
 """Authentication commands: login, logout, status."""
 
 import json
+import shutil
 import subprocess
 from datetime import UTC, datetime
 
@@ -20,19 +21,22 @@ from minitest_cli.utils.output import output, print_error, print_info, print_suc
 app = typer.Typer(name="auth", help="Authentication management.")
 
 SKILL_NAME = "minitest-cli"
-SKILL_INSTALL_CMD = "npx skills add minitap-ai/agent-skills --skill minitest-cli"
+SKILL_INSTALL_ARGS = ["skills", "add", "minitap-ai/agent-skills", "--skill", "minitest-cli"]
+SKILL_INSTALL_DISPLAY = "npx " + " ".join(SKILL_INSTALL_ARGS)
 
 
-def _is_skill_installed() -> bool:
-    """Check if the minitest-cli skill is installed via ``npx skills ls``.
+def _find_npx() -> str | None:
+    return shutil.which("npx")
+
 
-    Queries both project-level and global scopes so the detection stays in
-    sync with whatever directories the ``skills`` CLI manages.
-    """
+def _is_skill_installed() -> bool:
+    npx = _find_npx()
+    if npx is None:
+        return False
     for flags in (["--json"], ["--json", "-g"]):
         try:
             result = subprocess.run(
-                ["npx", "skills", "ls", *flags],
+                [npx, "skills", "ls", *flags],
                 capture_output=True,
                 text=True,
                 timeout=30,
@@ -42,7 +46,6 @@ def _is_skill_installed() -> bool:
                 if any(s.get("name") == SKILL_NAME for s in skills):
                     return True
         except (subprocess.TimeoutExpired, json.JSONDecodeError, FileNotFoundError, OSError):
-            # npx not available or unexpected output – fall through
             pass
     return False
 
@@ -81,14 +84,21 @@ def login() -> None:
             answer = "n"
             print()  # newline after ^C / ^D
         if answer in ("", "y", "yes"):
-            print_info("")
-            print_info(f"   Running: {SKILL_INSTALL_CMD}")
-            print_info("")
-            subprocess.run(SKILL_INSTALL_CMD.split(), check=False)
+            npx = _find_npx()
+            if npx is None:
+                print_info("")
+                print_error("   npx not found on PATH. Install Node.js, then run:")
+                print_info(f"   {SKILL_INSTALL_DISPLAY}")
+                print_info("")
+            else:
+                print_info("")
+                print_info(f"   Running: {SKILL_INSTALL_DISPLAY}")
+                print_info("")
+                subprocess.run([npx, *SKILL_INSTALL_ARGS], check=False)
         else:
             print_info("")
             print_info("   You can install it later with:")
-            print_info(f"   {SKILL_INSTALL_CMD}")
+            print_info(f"   {SKILL_INSTALL_DISPLAY}")
             print_info("")
 
 

{minitest_cli-0.8.0 → minitest_cli-0.8.2}/src/minitest_cli/core/token_exchange.py

@@ -2,6 +2,8 @@
 
 from __future__ import annotations
 
+import base64
+import json
 import sys
 import time
 from typing import Any, NoReturn
@@ -40,12 +42,21 @@ def parse_and_save_token_response(settings: Settings, data: dict[str, Any]) -> C
         if not isinstance(user, dict):
             user = {}
         expires_in = data.get("expires_in", 3600)
+
+        user_id = user.get("id", "")
+        email = user.get("email", "")
+
+        if not email or not user_id:
+            claims = _decode_jwt_claims(data["access_token"])
+            user_id = user_id or claims.get("sub", "")
+            email = email or claims.get("email", "")
+
         creds = Credentials(
             access_token=data["access_token"],
             refresh_token=data["refresh_token"],
             expires_at=time.time() + int(expires_in),
-            user_id=user.get("id", ""),
-            email=user.get("email", ""),
+            user_id=user_id,
+            email=email,
         )
         save_credentials(settings, creds)
         return creds
@@ -53,6 +64,16 @@ def parse_and_save_token_response(settings: Settings, data: dict[str, Any]) -> C
         return None
 
 
+def _decode_jwt_claims(token: str) -> dict[str, Any]:
+    """Decode JWT payload without verification. Returns empty dict on failure."""
+    try:
+        payload = token.split(".")[1]
+        payload += "=" * (-len(payload) % 4)
+        return json.loads(base64.urlsafe_b64decode(payload))
+    except Exception:  # noqa: BLE001
+        return {}
+
+
 def register_oauth_client(supabase_url: str, redirect_uri: str) -> str:
     """Dynamically register an OAuth2 client with Supabase and return the client_id."""
     register_url = f"{supabase_url}/auth/v1/oauth/clients/register"

{minitest_cli-0.8.0 → minitest_cli-0.8.2}/src/minitest_cli/utils/update_check.py

@@ -1,4 +1,4 @@
-"""PyPI version check – cached for 24 hours, non-blocking, max 2s timeout."""
+"""PyPI version check – cached for 1 hour, non-blocking, max 2s timeout."""
 
 import json
 import shutil
@@ -13,7 +13,7 @@ from minitest_cli.core.config import Settings
 from minitest_cli.utils.output import print_warning
 
 CACHE_FILE_NAME = ".last_update_check"
-CACHE_TTL_SECONDS = 86400  # 24 hours
+CACHE_TTL_SECONDS = 3600
 CHECK_TIMEOUT_SECONDS = 2.0
 PYPI_URL = "https://pypi.org/pypi/minitest-cli/json"
 

{minitest_cli-0.8.0 → minitest_cli-0.8.2}/tests/test_auth_commands.py

@@ -2,12 +2,16 @@
 
 import base64
 import json
+import os
+import stat
+import sys
 import time
 from unittest.mock import patch
 
 import typer
 from typer.testing import CliRunner
 
+from minitest_cli.commands.auth import SKILL_INSTALL_ARGS
 from minitest_cli.commands.auth import app as auth_app
 from minitest_cli.core.auth import Credentials, save_credentials
 from minitest_cli.core.config import Settings
@@ -55,13 +59,13 @@ def _patch_context(settings, json_mode=False):
     ]
 
 
-def _run_with_context(command, settings, json_mode=False, args=None):
+def _run_with_context(command, settings, json_mode=False, args=None, stdin=None):
     """Run an auth command with patched context."""
     patches = _patch_context(settings, json_mode)
     for p in patches:
         p.start()
     try:
-        result = runner.invoke(auth_app, args or [command])
+        result = runner.invoke(auth_app, args or [command], input=stdin)
     finally:
         for p in patches:
             p.stop()
@@ -85,6 +89,51 @@ class TestLoginCommand:
         assert result.exit_code == 0
         assert "test@example.com" in result.output
 
+    def test_login_skill_install_uses_resolved_npx_path(self, tmp_path, monkeypatch):
+        # Regression: bare "npx" passed to subprocess.run fails on Windows where
+        # npx is npx.cmd. Verify the real shutil.which lookup happens and its
+        # absolute result is what reaches subprocess as argv[0].
+        bin_dir = tmp_path / "bin"
+        bin_dir.mkdir()
+        npx_name = "npx.cmd" if sys.platform == "win32" else "npx"
+        npx_path = bin_dir / npx_name
+        npx_path.write_text("#!/bin/sh\nexit 0\n")
+        npx_path.chmod(npx_path.stat().st_mode | stat.S_IEXEC | stat.S_IXGRP | stat.S_IXOTH)
+        monkeypatch.setenv("PATH", str(bin_dir), prepend=os.pathsep)
+
+        settings = _make_settings(tmp_path)
+        creds = _make_credentials()
+
+        with (
+            patch("minitest_cli.commands.auth.oauth_pkce_login", return_value=creds),
+            patch("minitest_cli.commands.auth._is_skill_installed", return_value=False),
+            patch("minitest_cli.commands.auth.subprocess.run") as mock_run,
+        ):
+            result = _run_with_context("login", settings, stdin="y\n")
+
+        assert result.exit_code == 0
+        mock_run.assert_called_once()
+        argv = mock_run.call_args.args[0]
+        assert argv[0] != "npx"
+        assert os.path.isabs(argv[0])
+        assert os.path.samefile(argv[0], npx_path)
+        assert argv[1:] == SKILL_INSTALL_ARGS
+
+    def test_login_skill_install_skipped_when_npx_missing(self, tmp_path, monkeypatch):
+        monkeypatch.setenv("PATH", "")
+        settings = _make_settings(tmp_path)
+        creds = _make_credentials()
+
+        with (
+            patch("minitest_cli.commands.auth.oauth_pkce_login", return_value=creds),
+            patch("minitest_cli.commands.auth._is_skill_installed", return_value=False),
+            patch("minitest_cli.commands.auth.subprocess.run") as mock_run,
+        ):
+            result = _run_with_context("login", settings, stdin="y\n")
+
+        assert result.exit_code == 0
+        mock_run.assert_not_called()
+
 
 class TestLogoutCommand:
     def test_logout_exits_2_when_env_token_set(self, tmp_path):

{minitest_cli-0.8.0 → minitest_cli-0.8.2}/uv.lock

@@ -141,7 +141,7 @@ wheels = [
 
 [[package]]
 name = "minitest-cli"
-version = "0.8.0"
+version = "0.8.2"
 source = { editable = "." }
 dependencies = [
     { name = "httpx" },