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

{minitest_cli-0.8.2 → minitest_cli-0.8.3}/AGENTS.md

@@ -49,7 +49,7 @@ uv add <package>                                   # Add new dependency (always
 ### Output Convention
 - `--json` flag: JSON to stdout, diagnostics to stderr
 - Without `--json`: human-friendly rich tables to stdout, diagnostics to stderr
-- Exit codes: 0=Success, 1=General error, 2=Auth error, 3=Network/API error, 4=Not found
+- Exit codes: 0=Success, 1=General error, 2=Auth error, 3=Network/API error, 4=Not found, 5=Build invalid
 
 ### No Interactive Prompts
 - All input via flags, env vars, or stdin

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: minitest-cli
-Version: 0.8.2
+Version: 0.8.3
 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.2 → minitest_cli-0.8.3}/pyproject.toml

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

{minitest_cli-0.8.2 → minitest_cli-0.8.3}/src/minitest_cli/commands/build.py

@@ -14,6 +14,7 @@ from minitest_cli.commands.build_helpers import (
     format_build_row,
     format_pagination_info,
     handle_response_error,
+    print_validation_warnings,
     resolve_app,
     run_api_call,
     upload_status_message,
@@ -79,6 +80,7 @@ def upload(
     if json_mode:
         output(result_dict, json_mode=True)
     else:
+        print_validation_warnings(result.validation_warnings)
         print_success(f"Build uploaded: {file.name} ({resolved_platform})")
         output(result_dict, json_mode=False)
 

{minitest_cli-0.8.2 → minitest_cli-0.8.3}/src/minitest_cli/commands/build_helpers.py

@@ -12,10 +12,13 @@ import typer
 from minitest_cli.core.app_context import resolve_app_id
 from minitest_cli.core.config import Settings
 from minitest_cli.models import BuildListResponse, BuildResponse
-from minitest_cli.utils.output import print_error
+from minitest_cli.utils.output import err_console, print_error
 
 EXIT_NETWORK_ERROR = 3
 EXIT_NOT_FOUND = 4
+EXIT_BUILD_INVALID = 5
+
+BUILD_INVALID_ERROR_CODE = "build_invalid"
 
 # ---------------------------------------------------------------------------
 # Context accessors
@@ -93,11 +96,44 @@ def handle_response_error(resp: httpx.Response, *, resource: str = "Build") -> N
     if resp.status_code == 404:
         print_error(f"{resource} not found: {extract_detail(resp)}")
         raise typer.Exit(code=EXIT_NOT_FOUND)
+    if resp.status_code == 422 and _try_handle_build_invalid(resp):
+        raise typer.Exit(code=EXIT_BUILD_INVALID)
     if resp.status_code >= 400:
         print_error(f"API error ({resp.status_code}): {extract_detail(resp)}")
         raise typer.Exit(code=EXIT_NETWORK_ERROR)
 
 
+def _try_handle_build_invalid(resp: httpx.Response) -> bool:
+    """If response is a build-invalid error, render issues and return True."""
+    try:
+        body = resp.json()
+    except Exception:  # noqa: BLE001
+        return False
+    if not isinstance(body, dict) or body.get("error_code") != BUILD_INVALID_ERROR_CODE:
+        return False
+    issues = body.get("issues") or []
+    print_error("Build rejected: failed validation for virtual-device execution.")
+    for issue in issues:
+        if not isinstance(issue, dict):
+            continue
+        code = issue.get("code", "unknown")
+        message = issue.get("message", "")
+        err_console.print(f"  [red]✖[/red] {code}: {message}")
+    return True
+
+
+def print_validation_warnings(warnings: list[dict] | None) -> None:
+    """Print non-fatal validation warnings to stderr."""
+    if not warnings:
+        return
+    for warning in warnings:
+        if not isinstance(warning, dict):
+            continue
+        code = warning.get("code", "unknown")
+        message = warning.get("message", "")
+        err_console.print(f"  [yellow]⚠[/yellow] {code}: {message}")
+
+
 def run_api_call[T](coro: Coroutine[object, object, T]) -> T:
     """Run an async API coroutine, catching network errors → exit 3."""
     try:

{minitest_cli-0.8.2 → minitest_cli-0.8.3}/src/minitest_cli/models/build.py

@@ -13,6 +13,7 @@ class BuildResponse(CamelModel):
     original_name: str
     size_bytes: int | None = None
     created_at: datetime
+    validation_warnings: list[dict] | None = None
 
 
 class BuildListResponse(CamelModel):

{minitest_cli-0.8.2 → minitest_cli-0.8.3}/tests/test_build_commands.py

@@ -125,6 +125,30 @@ class TestHandleResponseError:
             handle_response_error(resp)
         assert exc_info.value.exit_code == 3
 
+    def test_422_build_invalid_exits_5(self) -> None:
+        resp = _mock_response(
+            422,
+            {
+                "error_code": "build_invalid",
+                "issues": [
+                    {
+                        "code": "android.no_compatible_abi",
+                        "message": "APK does not include x86_64 ABI.",
+                        "detail": {},
+                    },
+                ],
+            },
+        )
+        with pytest.raises(Exit) as exc_info:
+            handle_response_error(resp)
+        assert exc_info.value.exit_code == 5
+
+    def test_422_other_falls_through(self) -> None:
+        resp = _mock_response(422, {"detail": "validation error"})
+        with pytest.raises(Exit) as exc_info:
+            handle_response_error(resp)
+        assert exc_info.value.exit_code == 3
+
 
 class TestFormatPaginationInfo:
     def test_with_total_shows_page_and_range(self) -> None:
@@ -279,6 +303,55 @@ class TestUploadCommand:
 
         assert result.exit_code == 3
 
+    def test_upload_build_invalid_422_exits_5_and_prints_issues(self, tmp_path: Path) -> None:
+        build_file = tmp_path / "bad.apk"
+        build_file.write_bytes(b"data")
+        settings = _make_settings(tmp_path)
+        body = {
+            "error_code": "build_invalid",
+            "issues": [
+                {
+                    "code": "android.no_compatible_abi",
+                    "message": "APK does not include x86_64 ABI.",
+                    "detail": {"abis": ["arm64-v8a"]},
+                },
+                {
+                    "code": "android.test_only_apk",
+                    "message": "APK is marked test-only.",
+                    "detail": {},
+                },
+            ],
+        }
+        client = _mock_upload_client(_mock_response(422, body))
+
+        with patch("minitest_cli.commands.build.ApiClient", return_value=client):
+            result = _run_with_context(["upload", str(build_file)], settings)
+
+        assert result.exit_code == 5
+        combined = result.output + (result.stderr if result.stderr_bytes else "")
+        assert "android.no_compatible_abi" in combined or "no_compatible_abi" in combined
+
+    def test_upload_with_validation_warnings_succeeds_and_prints_them(self, tmp_path: Path) -> None:
+        build_file = tmp_path / "warn.apk"
+        build_file.write_bytes(b"data")
+        settings = _make_settings(tmp_path)
+        body = {
+            **_UPLOAD_RESPONSE,
+            "validationWarnings": [
+                {
+                    "code": "android.unsigned_apk",
+                    "message": "APK is not signed.",
+                    "detail": {},
+                },
+            ],
+        }
+        client = _mock_upload_client(_mock_response(200, body))
+
+        with patch("minitest_cli.commands.build.ApiClient", return_value=client):
+            result = _run_with_context(["upload", str(build_file)], settings, json_mode=False)
+
+        assert result.exit_code == 0
+
     def test_upload_requires_auth(self, tmp_path: Path) -> None:
         build_file = tmp_path / "test.apk"
         build_file.write_bytes(b"data")

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

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