modal 1.6.2.dev1__tar.gz → 1.6.2.dev3__tar.gz

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: modal
-Version: 1.6.2.dev1
+Version: 1.6.2.dev3
 Summary: Python client library for Modal
 Author-email: Modal Labs <support@modal.com>
 License-Expression: Apache-2.0

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/modal/_environments.py

@@ -54,7 +54,7 @@ class _EnvironmentManager:
         modal.Environment.objects.create("my-environment")
         ```
         """
-        check_environment_name(name)
+        check_environment_name(name, allow_periods=False)
 
         client = await _Client.from_env() if client is None else client
         request = api_pb2.EnvironmentCreateRequest(name=name, is_managed=restricted)
@@ -464,7 +464,7 @@ class _Environment(_Object, type_prefix="en"):
         client: _Client | None = None,
     ) -> "_Environment":
         """Look up an Environment object using its name."""
-        check_environment_name(name)
+        check_environment_name(name, allow_periods=True)
         return _Environment._get_or_create(
             name=name,
             repr=f"Environment.from_name({name!r})",

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/modal/_output/status.py

@@ -20,6 +20,11 @@ def _get_annotation_for_web_url(url_info: api_pb2.WebUrlInfo) -> str:
     return suffix
 
 
+def _format_url(url: str) -> str:
+    # OSC 8 hyperlinks stay clickable when the URL wraps across lines
+    return f"[dim underline][link={url}]{url}[/link][/dim underline]"
+
+
 class FunctionCreationStatus:
     """Context manager for tracking and displaying function creation progress."""
 
@@ -58,9 +63,9 @@ class FunctionCreationStatus:
             suffix = _get_annotation_for_web_url(url_info)
 
             self.status_row.finish(f"Created {self.type} [green]{self.name}[/green]")
-            self.status_row.details(f"[dim underline]{web_url}[/dim underline]{auth_suffix}{suffix}")
+            self.status_row.details(f"{_format_url(web_url)}{auth_suffix}{suffix}")
             for custom_domain in self.response.function.custom_domain_info:
-                self.status_row.details(f"Custom domain: [dim underline]{custom_domain.url}[/dim underline]")
+                self.status_row.details(f"Custom domain: {_format_url(custom_domain.url)}")
 
         elif self.response.function.flash_service_urls:
             # Despite the gRPC API types, Servers only have one URL
@@ -68,7 +73,7 @@ class FunctionCreationStatus:
             auth_suffix = " 🔑" if authenticated else " [yellow](unauthenticated)[/yellow]"
             self.status_row.finish(f"Created {self.type} [green]{self.name}[/green]")
             url = self.response.function.flash_service_urls[0]
-            self.status_row.details(f"[dim underline]{url}[/dim underline]{auth_suffix}")
+            self.status_row.details(f"{_format_url(url)}{auth_suffix}")
 
         else:
             self.status_row.finish(f"Created {self.type} [green]{self.name}[/green].")
@@ -81,10 +86,8 @@ class FunctionCreationStatus:
                         auth_suffix = " 🔑" if authenticated else " [yellow](unauthenticated)[/yellow]"
                         self.status_row.details(
                             f"{method_definition.function_name} -> "
-                            f"[dim underline]{method_definition.web_url}[/dim underline]{auth_suffix}{suffix}"
+                            f"{_format_url(method_definition.web_url)}{auth_suffix}{suffix}"
                         )
                         for custom_domain in method_definition.custom_domain_info:
                             indent = len(method_definition.function_name) * " "
-                            self.status_row.details(
-                                f"{indent} -> [dim underline]{custom_domain.url}[/dim underline]{auth_suffix}"
-                            )
+                            self.status_row.details(f"{indent} -> {_format_url(custom_domain.url)}{auth_suffix}")

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/modal/_utils/name_utils.py

@@ -27,13 +27,15 @@ def is_valid_object_name(name: str) -> bool:
     )
 
 
-def is_valid_environment_name(name: str) -> bool:
+def is_valid_environment_name(name: str, *, allow_periods: bool) -> bool:
     # first char is alnum, the rest allows other chars
     # The "en-" prefix is reserved for environment IDs, so that references
     # to environments can be resolved by either name or ID unambiguously.
+    # New names cannot contain periods, but some existing Environments have them and must stay resolvable.
+    rest_chars = r"a-zA-Z0-9-_." if allow_periods else r"a-zA-Z0-9-_"
     return (
         len(name) <= 64
-        and re.match(r"^[a-zA-Z0-9][a-zA-Z0-9-_.]+$", name) is not None
+        and re.match(rf"^[a-zA-Z0-9][{rest_chars}]*$", name) is not None
         and not name.lower().startswith("en-")
     )
 
@@ -67,12 +69,12 @@ def check_object_name(name: str, object_type: str) -> None:
         raise InvalidError(message)
 
 
-def check_environment_name(name: str) -> None:
+def check_environment_name(name: str, *, allow_periods: bool) -> None:
     message = (
         f"Invalid Environment name: '{name}'."
-        "\n\nEnvironment names can only start with alphanumeric characters,"
-        " may contain only alphanumeric characters, dashes, periods, and underscores,"
-        " must be shorter than 64 characters, and cannot start with 'en-'."
+        "\n\nEnvironment names must be 1-64 characters, can only start with alphanumeric characters,"
+        " may contain only alphanumeric characters, dashes, and underscores,"
+        " and cannot start with 'en-'."
     )
-    if not is_valid_environment_name(name):
+    if not is_valid_environment_name(name, allow_periods=allow_periods):
         raise InvalidError(message)

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/modal/_workspace.py

@@ -495,7 +495,8 @@ class _WorkspaceSettingsManager:
     """mdmd:namespace"""
 
     @classmethod
-    def valid_settings(cls):
+    def _valid_settings(cls):
+        """mdmd:hidden"""
         return ("default-environment", "image-builder-version")
 
     def __init__(self, workspace: _Workspace):
@@ -554,4 +555,4 @@ class _WorkspaceSettingsManager:
             case "default-environment":
                 await self._set_default_environment(value)
             case _:
-                raise ValueError(f"Unknown setting {name!r}. Valid settings: {', '.join(self.valid_settings())}")
+                raise ValueError(f"Unknown setting {name!r}. Valid settings: {', '.join(self._valid_settings())}")

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/modal/cli/app.py

@@ -225,12 +225,12 @@ async def rollover(
             await _stop_and_wait_for_containers(client, app_id, response.deployed_at, env)
         except Exception as exc:
             warnings.warn(f"App updated successfully, but containers did not all terminate. {exc}", UserWarning)
-            output_mgr.print(f"\nView Deployment: [magenta]{response.url}[/magenta]")
+            output_mgr.print(f"\nView Deployment: [magenta][link={response.url}]{response.url}[/link][/magenta]")
             sys.exit(1)
 
     duration = time.monotonic() - t0
     output_mgr.step_completed(f"Rollover completed in {duration:.3f}s with {strategy} strategy! 🎉")
-    output_mgr.print(f"\nView Deployment: [magenta]{response.url}[/magenta]")
+    output_mgr.print(f"\nView Deployment: [magenta][link={response.url}]{response.url}[/link][/magenta]")
 
 
 @app_cli.command("promote", panel="Management", no_args_is_help=True, hidden=True)
@@ -281,7 +281,7 @@ async def promote(
 
     output_mgr = OutputManager.get()
     output_mgr.print(f"[green]✓[/green] Promoted App to v{version_number}!")
-    output_mgr.print(f"\nView Deployment: [magenta]{resp.url}[/magenta]")
+    output_mgr.print(f"\nView Deployment: [magenta][link={resp.url}]{resp.url}[/link][/magenta]")
 
 
 @app_cli.command(
@@ -356,10 +356,10 @@ async def rollback(
             warnings.warn(
                 f"App rollback executed successfully, but containers did not all terminate. {exc}", UserWarning
             )
-            output_mgr.print(f"\nView Deployment: [magenta]{resp.url}[/magenta]")
+            output_mgr.print(f"\nView Deployment: [magenta][link={resp.url}]{resp.url}[/link][/magenta]")
             sys.exit(1)
     output_mgr.print("[green]✓[/green] Deployment rollback successful!")
-    output_mgr.print(f"\nView Deployment: [magenta]{resp.url}[/magenta]")
+    output_mgr.print(f"\nView Deployment: [magenta][link={resp.url}]{resp.url}[/link][/magenta]")
 
 
 @app_cli.command("stop", panel="Management", no_args_is_help=True)

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/modal/cli/endpoint.py

@@ -260,7 +260,7 @@ async def list_(*, json: bool = False, env: Optional[str] = None):
 @click.option(
     "--name",
     default=None,
-    help="Endpoint name. If not provided, a default will be derived from the model name.",
+    help="Endpoint name. Defaults to inference-{number} if not provided.",
 )
 @click.option(
     "--model",
@@ -410,6 +410,8 @@ async def create(
 
     output = OutputManager.get()
     output.print(f"[green]✓[/green] Endpoint '{resp.name}' ({resp.endpoint_id}) was created and started provisioning.")
+    if resp.model_name:
+        output.print(f'  → Send [cyan]"model": "{resp.model_name}"[/cyan] in requests to the Endpoint.')
     if resp.endpoint_page_url:
         output.print(f"  → View progress at [magenta]{resp.endpoint_page_url}[/magenta].")
         output.print("  → The Endpoint will also appear in [cyan]modal endpoint list[/cyan].")
@@ -517,6 +519,7 @@ async def info(
                 {
                     "name": endpoint_info.name,
                     "endpoint_id": endpoint_id,
+                    "model_name": endpoint_info.model_name or None,
                     "repo_id": endpoint_info.repo_id,
                     "revision": endpoint_info.revision or None,
                     "volume_id": endpoint_info.volume_id or None,
@@ -547,6 +550,8 @@ async def info(
     header.add_row(Text("Endpoint:"), endpoint_text)
     header.add_row(Text("Endpoint ID:"), Text(endpoint_id))
     header.add_row(Text("State:"), Text(status))
+    if endpoint_info.model_name:
+        header.add_row(Text("Model name:"), Text(endpoint_info.model_name))
     model_text = Text(endpoint_info.repo_id)
     if endpoint_info.revision:
         model_text.append(f"@{endpoint_info.revision[:7]}")

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/modal/cli/environment.py

@@ -87,7 +87,7 @@ def create(
     default_role: str | None = None,
     public: bool = False,
 ):
-    check_environment_name(name)
+    check_environment_name(name, allow_periods=False)
     Environment.objects.create(
         name,
         restricted=restricted,
@@ -116,8 +116,8 @@ def update(
     if set_name is None and set_web_suffix is None:
         raise UsageError("You need to at least one new property (using --set-name or --set-web-suffix)")
 
-    if set_name:
-        check_environment_name(set_name)
+    if set_name and set_name != current_name:
+        check_environment_name(set_name, allow_periods=False)
 
     environments.update_environment(current_name, new_name=set_name, new_web_suffix=set_web_suffix)
     rich.print("[green]✓[/green] Environment updated")

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/modal/client.pyi

@@ -80,7 +80,7 @@ class _Client:
         server_url: str,
         client_type: int,
         credentials: typing.Optional[tuple[str, str]],
-        version: str = "1.6.2.dev1",
+        version: str = "1.6.2.dev3",
         *,
         oauth_credentials: typing.Optional[_OAuthCredentials] = None,
     ):
@@ -290,7 +290,7 @@ class Client:
         server_url: str,
         client_type: int,
         credentials: typing.Optional[tuple[str, str]],
-        version: str = "1.6.2.dev1",
+        version: str = "1.6.2.dev3",
         *,
         oauth_credentials: typing.Optional[_OAuthCredentials] = None,
     ):

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/modal/experimental/_flash_container_entry.py

@@ -41,6 +41,7 @@ class _FlashContainerEntry:
                     self.flash_manager = server_forward()
                 else:
                     try:
+                        # Only rank 0 of a clustered replica exposes the HTTP endpoint through Flash.
                         rank = get_current_cluster_context().rank
                         if rank != 0:
                             return

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/modal/functions.pyi

@@ -613,7 +613,7 @@ class Function(
 
     _call_generator: ___call_generator_spec
 
-    class __remote_spec(typing_extensions.Protocol[ReturnType_INNER, P_INNER]):
+    class __remote_spec(typing_extensions.Protocol[P_INNER, ReturnType_INNER]):
         def __call__(self, /, *args: P_INNER.args, **kwargs: P_INNER.kwargs) -> ReturnType_INNER:
             """Calls the function remotely, executing it with the given arguments and returning the execution's result.
 
@@ -638,7 +638,7 @@ class Function(
             """
             ...
 
-    remote: __remote_spec[modal._functions.ReturnType, modal._functions.P]
+    remote: __remote_spec[modal._functions.P, modal._functions.ReturnType]
 
     class __remote_gen_spec(typing_extensions.Protocol):
         def __call__(self, /, *args, **kwargs) -> typing.Generator[typing.Any, None, None]:
@@ -688,7 +688,7 @@ class Function(
         """
         ...
 
-    class ___experimental_spawn_spec(typing_extensions.Protocol[ReturnType_INNER, P_INNER]):
+    class ___experimental_spawn_spec(typing_extensions.Protocol[P_INNER, ReturnType_INNER]):
         def __call__(self, /, *args: P_INNER.args, **kwargs: P_INNER.kwargs) -> FunctionCall[ReturnType_INNER]:
             """[Experimental] Calls the function with the given arguments, without waiting for the results.
 
@@ -721,7 +721,7 @@ class Function(
             """
             ...
 
-    _experimental_spawn: ___experimental_spawn_spec[modal._functions.ReturnType, modal._functions.P]
+    _experimental_spawn: ___experimental_spawn_spec[modal._functions.P, modal._functions.ReturnType]
 
     class ___spawn_map_inner_spec(typing_extensions.Protocol[P_INNER]):
         def __call__(self, /, *args: P_INNER.args, **kwargs: P_INNER.kwargs) -> None: ...
@@ -729,7 +729,7 @@ class Function(
 
     _spawn_map_inner: ___spawn_map_inner_spec[modal._functions.P]
 
-    class __spawn_spec(typing_extensions.Protocol[ReturnType_INNER, P_INNER]):
+    class __spawn_spec(typing_extensions.Protocol[P_INNER, ReturnType_INNER]):
         def __call__(self, /, *args: P_INNER.args, **kwargs: P_INNER.kwargs) -> FunctionCall[ReturnType_INNER]:
             """Calls the function with the given arguments, without waiting for the results.
 
@@ -762,7 +762,7 @@ class Function(
             """
             ...
 
-    spawn: __spawn_spec[modal._functions.ReturnType, modal._functions.P]
+    spawn: __spawn_spec[modal._functions.P, modal._functions.ReturnType]
 
     @property
     def _raw_f_(self) -> collections.abc.Callable[..., typing.Any]: ...

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/modal/runner.py

@@ -697,7 +697,7 @@ async def _deploy_app(
     else:
         output_mgr.step_completed(f"App deployed in {t:.3f}s! 🎉")
 
-    output_mgr.print(f"\nView Deployment: [magenta]{app_url}[/magenta]")
+    output_mgr.print(f"\nView Deployment: [magenta][link={app_url}]{app_url}[/link][/magenta]")
     return DeployResult(
         app_id=running_app.app_id,
         app_page_url=running_app.app_page_url,

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/modal/workspace.pyi

@@ -631,7 +631,9 @@ class WorkspaceSettingsManager:
         ...
 
     @classmethod
-    def valid_settings(cls): ...
+    def _valid_settings(cls):
+        """mdmd:hidden"""
+        ...
 
     class __list_spec(typing_extensions.Protocol):
         def __call__(self, /):

{modal-1.6.2.dev1 → modal-1.6.2.dev3}/modal.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: modal
-Version: 1.6.2.dev1
+Version: 1.6.2.dev3
 Summary: Python client library for Modal
 Author-email: Modal Labs <support@modal.com>
 License-Expression: Apache-2.0