dagster-cloud-cli 1.13.7__tar.gz → 1.13.9__tar.gz

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/PKG-INFO

@@ -1,13 +1,13 @@
 Metadata-Version: 2.4
 Name: dagster-cloud-cli
-Version: 1.13.7
+Version: 1.13.9
 Project-URL: Homepage, https://github.com/dagster-io/dagster/tree/master/python_modules/libraries/dagster-cloud-cli
 Author-email: Dagster Labs <hello@dagsterlabs.com>
 License-Expression: Apache-2.0
 License-File: LICENSE
 Requires-Python: <3.15,>=3.10
 Requires-Dist: click
-Requires-Dist: dagster-shared==1.13.7
+Requires-Dist: dagster-shared==1.13.9
 Requires-Dist: github3-py
 Requires-Dist: jinja2
 Requires-Dist: packaging>=20.9

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/commands/ci/__init__.py

@@ -14,6 +14,7 @@ import typer
 import yaml
 from dagster_shared import check
 from dagster_shared.utils import remove_none_recursively
+from dagster_shared.yaml_utils import safe_load_yaml
 from jinja2 import TemplateSyntaxError
 from typer import Typer
 
@@ -362,11 +363,11 @@ def _create_context_from_values(values_list: list[str], values_file: str | None)
     values_file_path = pathlib.Path(values_file) if values_file else None
     if values_file_path:
         try:
-            with open(values_file_path) as f:
+            with open(values_file_path, encoding="utf-8") as f:
                 if values_file_path.suffix == ".json":
                     context.update(json.load(f))
                 elif values_file_path.suffix in [".yaml", ".yml"]:
-                    context.update(yaml.safe_load(f))
+                    context.update(safe_load_yaml(f))
                 else:
                     raise ui.error(f"Unsupported values file extension {values_file_path.suffix}")
         except Exception as err:
@@ -832,6 +833,7 @@ def _build_docker(
     name = location_state.location_name
     docker_utils.verify_docker()
     registry_info = utils.get_registry_info(url)
+    repo_location = name if registry_info.get("is_harbor") else None
 
     docker_image_tag = docker_utils.default_image_tag(
         location_state.deployment_name, name, location_state.build.commit_hash
@@ -853,15 +855,18 @@ def _build_docker(
         base_image=docker_base_image,
         dockerfile_path=dockerfile_path,
         use_editable_dagster=use_editable_dagster,
+        location_name=repo_location,
     )
     if retval != 0:
         raise ui.error(f"Failed to build docker image for location {name}")
 
-    retval = docker_utils.upload_image(docker_image_tag, registry_info)
+    retval = docker_utils.upload_image(docker_image_tag, registry_info, location_name=repo_location)
     if retval != 0:
         raise ui.error(f"Failed to upload docker image for location {name}")
 
-    image = f"{registry_info['registry_url']}:{docker_image_tag}"
+    image = docker_utils.full_image_ref(
+        registry_info["registry_url"], repo_location, docker_image_tag
+    )
     ui.print(f"Built and uploaded image {image} for location {name}")
 
     return state.DockerBuildOutput(image=image)

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/commands/ci/checks.py

@@ -36,13 +36,13 @@ def check_dagster_cloud_yaml(yaml_path: pathlib.Path) -> CheckResult:
         result.errors.append(f"No such file {yaml_path}")
         return result
 
-    yaml_text = yaml_path.read_text()
+    yaml_text = yaml_path.read_text(encoding="utf-8")
     if not yaml_text.strip():
         result.errors.append(f"Unexpected blank file {yaml_path}")
         return result
 
     try:
-        parsed = load_dagster_cloud_yaml(yaml_path.read_text())
+        parsed = load_dagster_cloud_yaml(yaml_path.read_text(encoding="utf-8"))
     except pydantic.ValidationError as err:
         for error in get_validation_errors(err):
             result.errors.append(error)

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/commands/deployment/__init__.py

@@ -1,7 +1,7 @@
 from pathlib import Path
 from typing import Annotated
 
-import yaml
+from dagster_shared.yaml_utils import safe_load_yaml
 from typer import Argument, Typer
 
 from dagster_cloud_cli import gql, ui
@@ -34,7 +34,7 @@ def set_from_filecommand(
 ):
     """Set the Dagster Cloud deployment settings from a YAML file."""
     with open(file_path, encoding="utf8") as f:
-        settings = {"settings": yaml.safe_load(f) or {}}
+        settings = {"settings": safe_load_yaml(f) or {}}
     with gql.graphql_client_from_url(url, api_token, deployment_name=deployment) as client:
         gql.set_deployment_settings(client, settings)
 

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/commands/deployment/alert_policies/commands.py

@@ -1,6 +1,6 @@
 from pathlib import Path
 
-import yaml
+from dagster_shared.yaml_utils import safe_load_yaml
 from typer import Option, Typer
 
 from dagster_cloud_cli import gql, ui
@@ -42,7 +42,7 @@ def sync_command(
     """Sync your YAML configured alert policies to Dagster Cloud."""
     with gql.graphql_client_from_url(url, api_token, deployment_name=deployment) as client:
         with open(str(alert_policies_file), encoding="utf8") as f:
-            config = yaml.load(f.read(), Loader=yaml.SafeLoader)
+            config = safe_load_yaml(f.read())
 
         try:
             alert_policies = gql.reconcile_alert_policies(client, config)

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/commands/organization/__init__.py

@@ -1,7 +1,7 @@
 from pathlib import Path
 from typing import Annotated
 
-import yaml
+from dagster_shared.yaml_utils import safe_load_yaml
 from typer import Argument, Typer
 
 from dagster_cloud_cli import gql, ui
@@ -43,7 +43,7 @@ def set_from_filecommand(
         url = gql.url_from_config(organization=organization)
 
     with open(file_path, encoding="utf8") as f:
-        settings = {"settings": yaml.safe_load(f) or {}}
+        settings = {"settings": safe_load_yaml(f) or {}}
     with gql.graphql_client_from_url(url, api_token) as client:
         gql.set_organization_settings(client, settings)
 

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/commands/serverless/__init__.py

@@ -212,7 +212,6 @@ def deploy_command(
     if not source_directory:
         raise ui.error("No source directory provided.")
 
-    _check_source_directory(source_directory)
     docker_utils.verify_docker()
 
     env_vars = kwargs.get("env", [])
@@ -221,21 +220,29 @@ def deploy_command(
     with gql.graphql_client_from_url(url, api_token, deployment_name=deployment) as client:
         ecr_info = gql.get_ecr_info(client)
         registry = ecr_info["registry_url"]
+        repo_location = location_name if ecr_info.get("is_harbor") else None
 
         image_tag = kwargs.get("image") or docker_utils.default_image_tag(
             deployment, location_name, kwargs.get("commit_hash")
         )
         retval = docker_utils.build_image(
-            source_directory, image_tag, ecr_info, env_vars, base_image, use_editable_dagster=False
+            source_directory,
+            image_tag,
+            ecr_info,
+            env_vars,
+            base_image,
+            use_editable_dagster=False,
+            location_name=repo_location,
         )
         if retval != 0:
             return
 
-        retval = docker_utils.upload_image(image_tag, ecr_info)
+        retval = docker_utils.upload_image(image_tag, ecr_info, location_name=repo_location)
         if retval != 0:
             return
 
-        location_args = {**kwargs, "image": f"{registry}:{image_tag}"}
+        full_image = docker_utils.full_image_ref(registry, repo_location, image_tag)
+        location_args = {**kwargs, "image": full_image}
         location_document = get_location_document(location_name, location_args)
         gql.add_or_update_code_location(client, location_document)
 
@@ -394,7 +401,7 @@ def build_python_dependencies(
     except pex_builder.deps.DepsBuildFailure as err:
         errors = [
             "Could not build dependencies for this project.",
-            f"Dependencies:\n {' '.join(open(requirements_path).readlines())}",
+            f"Dependencies:\n {' '.join(open(requirements_path, encoding='utf-8').readlines())}",
             err.format_error(),
         ]
         raise ui.error("\n".join(errors))

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/commands/workspace/__init__.py

@@ -3,9 +3,9 @@ from collections.abc import Mapping
 from pathlib import Path
 from typing import Any
 
-import yaml
 from dagster_shared import check
 from dagster_shared.serdes.serdes import deserialize_value
+from dagster_shared.yaml_utils import safe_load_yaml
 from typer import Argument, Option, Typer
 
 from dagster_cloud_cli import gql, ui
@@ -341,7 +341,7 @@ def execute_sync_command(
     client, workspace, location_load_timeout, agent_heartbeat_timeout, url: str | None = None
 ):
     with open(str(workspace), encoding="utf8") as f:
-        config = yaml.load(f.read(), Loader=yaml.SafeLoader)
+        config = safe_load_yaml(f.read())
         processed_config = format_workspace_config(config)
 
     try:

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/config/models.py

@@ -2,7 +2,7 @@
 
 from typing import Any
 
-import yaml
+from dagster_shared.yaml_utils import safe_load_yaml
 from pydantic import BaseModel, Field, field_validator, model_validator
 
 
@@ -73,4 +73,4 @@ class ProcessedDagsterCloudConfig(BaseModel, extra="forbid"):
 
 
 def load_dagster_cloud_yaml(text) -> DagsterCloudYaml:
-    return DagsterCloudYaml.model_validate(yaml.safe_load(text))
+    return DagsterCloudYaml.model_validate(safe_load_yaml(text))

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/config_utils.py

@@ -8,7 +8,6 @@ from typing import TYPE_CHECKING, Any
 if TYPE_CHECKING:
     from typer.models import OptionInfo
 
-import yaml
 from click import Context
 from dagster_shared.merger import deep_merge_dicts
 from dagster_shared.plus.config import (
@@ -17,6 +16,7 @@ from dagster_shared.plus.config import (
     get_dg_config_path,
 )
 from dagster_shared.utils import remove_none_recursively
+from dagster_shared.yaml_utils import safe_load_yaml
 from typer import Option
 
 from dagster_cloud_cli import gql, ui
@@ -443,7 +443,7 @@ def get_location_document(name: str | None, kwargs: dict[str, Any]) -> dict[str,
     location_doc_from_file = {}
     if location_file:
         with open(location_file, encoding="utf8") as f:
-            location_doc_from_file = yaml.safe_load(f.read())
+            location_doc_from_file = safe_load_yaml(f.read())
 
     if not location_file and not name:
         raise ui.error(

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/core/graphql_client.py

@@ -39,6 +39,9 @@ RETRY_STATUS_CODES = [
     409,
 ]
 
+# S3 returns sporadic HTTP 500 as a transient error per its retry guidance.
+PRESIGNED_URL_PUT_RETRY_STATUS_CODES = [500, *RETRY_STATUS_CODES]
+
 
 class DagsterCloudAgentHttpClient:
     def __init__(

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/core/pex_builder/deps.py

@@ -419,19 +419,15 @@ def get_pyproject_toml_deps(code_directory: str) -> list[str]:
     except Exception as e:
         raise ValueError(f"Error parsing pyproject.toml: {e}")
 
-    lines = []
-
     # Handle dependencies in [project] section (PEP 621)
     project_section = pyproject_data.get("project", {})
     dependencies = project_section.get("dependencies", [])
-    for dep in dependencies:
-        lines.append(str(dep))
+    lines = [str(dep) for dep in dependencies]
 
     # Handle optional dependencies in [project.optional-dependencies]
     optional_deps = project_section.get("optional-dependencies", {})
     for group_deps in optional_deps.values():
-        for dep in group_deps:
-            lines.append(str(dep))
+        lines.extend(str(dep) for dep in group_deps)
 
     # Handle legacy [tool.poetry.dependencies] for Poetry projects
     poetry_section = pyproject_data.get("tool", {}).get("poetry", {})

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/core/pex_builder/parse_workspace.py

@@ -1,7 +1,7 @@
 import os
 from dataclasses import dataclass
 
-import yaml
+from dagster_shared.yaml_utils import safe_load_yaml
 
 from dagster_cloud_cli import ui
 
@@ -20,7 +20,7 @@ def get_locations(dagster_cloud_yaml_file) -> list[Location]:
 
     with open(dagster_cloud_yaml_file, encoding="utf-8") as yaml_file:
         workspace_contents = yaml_file.read()
-        workspace_contents_yaml = yaml.safe_load(workspace_contents)
+        workspace_contents_yaml = safe_load_yaml(workspace_contents)
 
         locations = []
         for location in workspace_contents_yaml["locations"]:

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/core/pex_builder/pex_registry.py

@@ -5,8 +5,15 @@ from tempfile import TemporaryDirectory
 from typing import Any
 
 import requests
+from requests.adapters import HTTPAdapter
+from urllib3.util.retry import Retry
 
 from dagster_cloud_cli import gql, ui
+from dagster_cloud_cli.core.graphql_client import (
+    DEFAULT_BACKOFF_FACTOR,
+    DEFAULT_RETRIES,
+    PRESIGNED_URL_PUT_RETRY_STATUS_CODES,
+)
 
 GENERATE_PUT_URL_QUERY = """
 mutation GenerateServerlessPexUrlMutation($filenames: [String!]!) {
@@ -131,6 +138,18 @@ def upload_files(dagster_cloud_url: str, dagster_cloud_api_token: str, filepaths
         ui.error(f"Cannot upload files, did not get PUT urls for: {filenames}")
         return
 
+    session = requests.Session()
+    put_retry_adapter = HTTPAdapter(
+        max_retries=Retry(
+            total=DEFAULT_RETRIES,
+            backoff_factor=DEFAULT_BACKOFF_FACTOR,
+            status_forcelist=PRESIGNED_URL_PUT_RETRY_STATUS_CODES,
+            allowed_methods=["PUT"],
+        )
+    )
+    session.mount("https://", put_retry_adapter)
+    session.mount("http://", put_retry_adapter)
+
     # we expect response list to be in the same order as the request
     for _, filepath, url in zip(filenames, filepaths, urls):
         if not url:
@@ -139,7 +158,7 @@ def upload_files(dagster_cloud_url: str, dagster_cloud_api_token: str, filepaths
 
         ui.print(f"Uploading {filepath} ...")
         with open(filepath, "rb") as f:
-            response = requests.put(url, data=f)
+            response = session.put(url, data=f)
             if response.ok:
                 ui.print(f"Upload successful: {filepath}")
             else:

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/core/pex_builder/source.py

@@ -58,19 +58,14 @@ def _build_local_package(local_dir: str, build_dir: str, python_interpreter: str
         ]
         subprocess.run(command, check=True, cwd=local_dir)
     elif os.path.exists(os.path.join(local_dir, "pyproject.toml")):
-        ui.print(f"Building package at {local_dir!r} using {python_interpreter} -m pip install")
-        # Use pip install with --target to build and install the package to the build directory
-        # This handles pyproject.toml files properly using modern Python build standards
-        command = [
-            python_interpreter,
-            "-m",
-            "pip",
-            "install",
-            "--target",
-            build_dir,
-            "--no-deps",  # Don't install dependencies, just the package itself
-            ".",
-        ]
+        install_args = ["--target", build_dir, "--no-deps", "."]
+        uv_path = shutil.which("uv")
+        if uv_path:
+            ui.print(f"Building package at {local_dir!r} using uv pip install")
+            command = [uv_path, "pip", "install", "--python", python_interpreter, *install_args]
+        else:
+            ui.print(f"Building package at {local_dir!r} using {python_interpreter} -m pip install")
+            command = [python_interpreter, "-m", "pip", "install", *install_args]
         subprocess.run(command, check=True, cwd=local_dir)
     else:
         ui.warn(f"No setup.py or pyproject.toml found in {local_dir!r} - will not build.")

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/docker_utils.py

@@ -34,6 +34,26 @@ def _template_dockerfile(env_vars, custom_base_image=None) -> Generator[bytes, N
         yield bytes(dockerfile_content, "utf-8")
 
 
+def full_image_ref(
+    registry_url: str,
+    location_name: str | None,
+    image_tag: str,
+) -> str:
+    """Build the full Docker image reference.
+
+    For Harbor-backed registries the registry URL already contains a
+    deployment-scoped prefix (``host/project/depl-prefix``).  Each code
+    location gets its own repository nested under that prefix, so the
+    reference becomes ``host/project/depl-prefix/location:tag``.
+
+    For legacy ECR registries (or when *location_name* is ``None``) the
+    reference falls back to the previous ``registry:tag`` format.
+    """
+    if location_name:
+        return f"{registry_url}/{location_name}:{image_tag}"
+    return f"{registry_url}:{image_tag}"
+
+
 def build_image(
     source_directory,
     image: str,
@@ -42,8 +62,9 @@ def build_image(
     base_image: str | None,
     use_editable_dagster: bool,
     dockerfile_path: str | None = None,
+    location_name: str | None = None,
 ) -> int:
-    registry = registry_info["registry_url"]
+    full_ref = full_image_ref(registry_info["registry_url"], location_name, image)
 
     with ExitStack() as stack:
         if dockerfile_path:
@@ -58,7 +79,7 @@ def build_image(
             "build",
             source_directory,
             "-t",
-            f"{registry}:{image}",
+            full_ref,
             "-f",
             file_path,
             "--platform",
@@ -74,7 +95,7 @@ def build_image(
         return subprocess.run(cmd, input=build_input, check=True).returncode
 
 
-def upload_image(image, registry_info) -> int:
+def upload_image(image, registry_info, location_name: str | None = None) -> int:
     registry = registry_info["registry_url"]
     aws_token = registry_info["aws_auth_token"]
     if not registry or not aws_token:
@@ -89,9 +110,8 @@ def upload_image(image, registry_info) -> int:
         input=password.encode(),
         check=True,
     )
-    return subprocess.call(
-        ["docker", "push", f"{registry}:{image}"], stderr=sys.stderr, stdout=sys.stdout
-    )
+    full_ref = full_image_ref(registry, location_name, image)
+    return subprocess.call(["docker", "push", full_ref], stderr=sys.stderr, stdout=sys.stdout)
 
 
 def default_image_tag(deployment: str, location_name: str, commit_hash: str | None) -> str:

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/gql.py

@@ -715,6 +715,7 @@ query CliGetEcrInfo {
         awsRegion
         awsAuthToken
         registryAllowCustomBase
+        registryIsHarbor
         registryUrl
     }
 }
@@ -728,6 +729,7 @@ def get_ecr_info(client: DagsterCloudGraphQLClient) -> Any:
         "aws_region": data["serverless"]["awsRegion"],
         "aws_auth_token": data["serverless"]["awsAuthToken"],
         "allow_custom_base": data["serverless"]["registryAllowCustomBase"],
+        "is_harbor": data["serverless"].get("registryIsHarbor", False),
     }
 
 

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli/types.py

@@ -43,10 +43,7 @@ SubcommandTags = Enum(
 class CliEventTags:
     @classmethod
     def enums(cls):
-        attrs = []
-        for attr in cls.__dict__.values():
-            if isinstance(attr, EnumMeta):
-                attrs.append(attr)
+        attrs = [attr for attr in cls.__dict__.values() if isinstance(attr, EnumMeta)]
         return attrs
 
     @classmethod

dagster_cloud_cli-1.13.9/dagster_cloud_cli/version.py

@@ -0,0 +1 @@
+__version__ = "1.13.9"

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli_tests/commands/ci/test_ci_template.py

@@ -1,9 +1,9 @@
 import pathlib
 
-import yaml
 from dagster_cloud_cli.config.models import ProcessedDagsterCloudConfig
 from dagster_cloud_cli.entrypoint import app
 from dagster_cloud_cli_tests.conftest import create_template_file
+from dagster_shared.yaml_utils import safe_load_yaml
 from typer.testing import CliRunner
 
 DAGSTER_CLOUD_YAML_JINJA = """
@@ -71,7 +71,7 @@ def test_ci_template(temp_dir):
             ],
         )
         assert result.exit_code == 0
-        processed_config = ProcessedDagsterCloudConfig.model_validate(yaml.safe_load(result.stdout))
+        processed_config = ProcessedDagsterCloudConfig.model_validate(safe_load_yaml(result.stdout))
         assert len(processed_config.locations) == 1
         assert processed_config.locations[0].container_context is not None
         assert set(processed_config.locations[0].container_context["k8s"]["env_vars"]) == {

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli_tests/config/test_template_loader.py

@@ -1,6 +1,6 @@
-import yaml
 from dagster_cloud_cli.config.jinja_template_loader import JinjaTemplateLoader
 from dagster_cloud_cli_tests.conftest import create_template_file
+from dagster_shared.yaml_utils import safe_load_yaml
 
 
 def test_template_loader(tmpdir, monkeypatch):
@@ -41,5 +41,5 @@ def test_template_loader_to_yaml_filter(tmpdir):
 
     with create_template_file(tmpdir, "template.jinja", ARTICLES_TEMPLATE) as template_file:
         result = JinjaTemplateLoader().render(template_file, context)
-        roundtrip = yaml.safe_load(result)
+        roundtrip = safe_load_yaml(result)
         assert roundtrip == context

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli_tests/conftest.py

@@ -16,7 +16,7 @@ def temp_dir() -> Generator[str, Any, None]:
 @contextmanager
 def create_template_file(tmpdir: str, filename: str, text: str) -> Generator[str, Any, None]:
     file_path = os.path.join(tmpdir, filename)
-    with open(file_path, "w") as f:
+    with open(file_path, "w", encoding="utf-8") as f:
         f.write(text)
     yield file_path
 

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli_tests/test_check.py

@@ -84,7 +84,7 @@ def test_dagster_cloud_yaml_check() -> None:
         with tempfile.TemporaryDirectory() as tmpdir:
             os.mkdir(os.path.join(tmpdir, "subdir"))
             yaml_path = os.path.join(tmpdir, "dagster_cloud.yaml")
-            with open(yaml_path, "w") as f:
+            with open(yaml_path, "w", encoding="utf-8") as f:
                 f.write(text)
             runner = CliRunner()
             return runner.invoke(

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli_tests/test_ci_commands.py

@@ -51,7 +51,7 @@ def with_dagster_yaml(text):
         with tempfile.TemporaryDirectory() as tmpdir:
             os.mkdir(os.path.join(tmpdir, "subdir"))
             yaml_path = os.path.join(tmpdir, "dagster_cloud.yaml")
-            with open(yaml_path, "w") as f:
+            with open(yaml_path, "w", encoding="utf-8") as f:
                 f.write(text)
             os.chdir(tmpdir)
             yield tmpdir

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/dagster_cloud_cli_tests/test_pyproject_toml_support.py

@@ -163,8 +163,9 @@ dependencies = ["requests>=2.25.0"]
 
 
 @patch("subprocess.run")
-def test_build_local_package_with_pyproject_toml(mock_run, temp_dir):
-    """Test building a package with pyproject.toml."""
+@patch("shutil.which", return_value=None)
+def test_build_local_package_with_pyproject_toml(mock_which, mock_run, temp_dir):
+    """Test building a package with pyproject.toml falls back to pip when uv is not available."""
     mock_run.return_value = None
 
     pyproject_path = Path(temp_dir) / "pyproject.toml"
@@ -196,6 +197,42 @@ version = "0.1.0"
     assert command == expected_command
 
 
+@patch("subprocess.run")
+@patch("shutil.which", return_value="/usr/bin/uv")
+def test_build_local_package_with_pyproject_toml_uses_uv(mock_which, mock_run, temp_dir):
+    """Test building a package with pyproject.toml uses uv when available."""
+    mock_run.return_value = None
+
+    pyproject_path = Path(temp_dir) / "pyproject.toml"
+    pyproject_path.write_text("""
+[project]
+name = "test-package"
+version = "0.1.0"
+""")
+
+    build_dir = Path(temp_dir) / "build"
+    build_dir.mkdir()
+
+    source._build_local_package(temp_dir, str(build_dir), "python")  # noqa: SLF001
+
+    mock_run.assert_called_once()
+    args, _kwargs = mock_run.call_args
+    command = args[0]
+
+    expected_command = [
+        "/usr/bin/uv",
+        "pip",
+        "install",
+        "--python",
+        "python",
+        "--target",
+        str(build_dir),
+        "--no-deps",
+        ".",
+    ]
+    assert command == expected_command
+
+
 @patch("subprocess.run")
 def test_build_local_package_prefers_setup_py(mock_run, temp_dir):
     """Test that setup.py is preferred over pyproject.toml when both exist."""

{dagster_cloud_cli-1.13.7 → dagster_cloud_cli-1.13.9}/pyproject.toml

@@ -1,20 +1,16 @@
 [build-system]
-requires = [
-    "hatchling>=1.19.0",
-]
+requires = ["hatchling>=1.19.0"]
 build-backend = "hatchling.build"
 
 [project]
 name = "dagster-cloud-cli"
-version = "1.13.7"
+version = "1.13.9"
 requires-python = ">=3.10,<3.15"
 description = ""
-authors = [
-    { name = "Dagster Labs", email = "hello@dagsterlabs.com" },
-]
+authors = [{name = "Dagster Labs", email = "hello@dagsterlabs.com"}]
 license = "Apache-2.0"
 dependencies = [
-    "dagster-shared==1.13.7",
+    "dagster-shared==1.13.9",
     "packaging>=20.9",
     "questionary",
     "requests",
@@ -48,6 +44,5 @@ test = [
     "pip",
 ]
 
-[tool.uv.sources.dagster-shared]
-path = "../../../python_modules/libraries/dagster-shared"
-editable = true
+[tool.uv.sources]
+dagster-shared = { path = "../dagster-shared", editable = true }

dagster_cloud_cli-1.13.7/dagster_cloud_cli/version.py

@@ -1 +0,0 @@
-__version__ = "1.13.7"