doover-cli 0.6.3__tar.gz → 0.6.4__tar.gz

{doover_cli-0.6.3 → doover_cli-0.6.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: doover-cli
-Version: 0.6.3
+Version: 0.6.4
 Summary: CLI for Doover
 License-File: LICENSE
 Requires-Python: >=3.11

{doover_cli-0.6.3 → doover_cli-0.6.4}/src/doover_cli/__init__.py

@@ -1,7 +1,7 @@
 from doover_cli.renderer import Renderer
 from typing import Annotated, Optional
 
-__version__ = "0.6.3"
+__version__ = "0.6.4"
 
 import click
 import typer

{doover_cli-0.6.3 → doover_cli-0.6.4}/src/doover_cli/apps/apps.py

@@ -299,6 +299,21 @@ def _resolve_application_id(client, app_config, *, staging: bool) -> int | None:
     return None
 
 
+def _image_package_uri(app_config) -> str | None:
+    """The ECR image a processor deploys from, or None if it ships a package.zip.
+
+    A processor whose dependencies cannot fit Lambda's 250MB unzipped zip limit declares
+    `PackageType: "Image"` in `lambda_config` and names an image instead. Such an app has
+    no source archive at all, so the whole build-and-upload-a-zip path has to be skipped:
+    there is no ./build.sh to run, and the control plane rejects a processor_source
+    upload for it.
+    """
+    lambda_config = getattr(app_config, "lambda_config", None) or {}
+    if lambda_config.get("PackageType") != "Image":
+        return None
+    return (lambda_config.get("Code") or {}).get("ImageUri")
+
+
 def _publish_processor_package(
     client,
     app_id: int,
@@ -1172,20 +1187,32 @@ def publish(
             rich.print("[green]Widget uploaded.[/green]")
 
     if app_config.type in PACKAGE_APP_TYPES:
-        if build_package:
-            print("\nBuilding package.zip for upload...")
-            shell_run("./build.sh", cwd=root_fp)
-        else:
+        image_uri = _image_package_uri(app_config)
+        if image_uri:
+            # Nothing to build or upload: the deployable is already in ECR, and the
+            # release below is what points the function at it.
             print(
-                "\nSkipping package.zip build (--no-build-package); using existing package.zip."
+                f"\nProcessor deploys from a container image; skipping package.zip.\n"
+                f"  image: {image_uri}\n"
+                f"Push the image before releasing, or the release will deploy whatever "
+                f"that tag currently points at."
             )
-        print("Uploading package.zip to Doover...")
-        processor_response = _publish_processor_package(
-            client,
-            application_id,
-            root_fp,
-        )
-        print("Done!")
+            processor_response = None
+        else:
+            if build_package:
+                print("\nBuilding package.zip for upload...")
+                shell_run("sh ./build.sh", cwd=root_fp)
+            else:
+                print(
+                    "\nSkipping package.zip build (--no-build-package); using existing package.zip."
+                )
+            print("Uploading package.zip to Doover...")
+            processor_response = _publish_processor_package(
+                client,
+                application_id,
+                root_fp,
+            )
+            print("Done!")
         renderer.render(processor_response or response)
         if release:
             ctx.invoke(

{doover_cli-0.6.3 → doover_cli-0.6.4}/tests/test_apps.py

@@ -706,7 +706,7 @@ def test_app_publish_processor_type_without_ui_skips_ui_export(monkeypatch, tmp_
 
     assert result.exit_code == 0
     assert "export_ui_called" not in captured
-    assert captured["build_script"] == ("./build.sh", tmp_path)
+    assert captured["build_script"] == ("sh ./build.sh", tmp_path)
     assert captured["processor_source"] == ("303", {"file": package_fp})
     # publish uploads the package but no longer cuts a version (that's `release`)
     assert "processor_version" not in captured
@@ -981,7 +981,7 @@ def test_app_publish_processor_builds_package_without_release(monkeypatch, tmp_p
     result = runner.invoke(app, ["app", "publish", str(tmp_path), "--no-release"])
 
     assert result.exit_code == 0
-    assert captured["build_script"] == ("./build.sh", tmp_path)
+    assert captured["build_script"] == ("sh ./build.sh", tmp_path)
     assert captured["application_id"] == "303"
     assert captured["body"] == {"file": package_fp}
     # publish uploads the package; the version is cut by `doover app release`