nextmv 1.0.0.dev3__py3-none-any.whl → 1.0.0.dev5__py3-none-any.whl

nextmv/cli/configuration/config.py

@@ -7,7 +7,8 @@ from typing import Any
 
 import yaml
 
-from nextmv.cli.message import error
+from nextmv.cli.confirm import get_confirmation
+from nextmv.cli.message import error, success, warning
 from nextmv.cloud.account import Account
 from nextmv.cloud.application import Application
 from nextmv.cloud.client import Client
@@ -59,6 +60,18 @@ def save_config(config: dict[str, Any]) -> None:
         yaml.safe_dump(config, file)
 
 
+def non_profile_keys() -> set[str]:
+    """
+    Returns the set of keys that are not profile names in the configuration.
+
+    Returns
+    -------
+    set[str]
+        The set of non-profile keys.
+    """
+    return {API_KEY_KEY, ENDPOINT_KEY}
+
+
 def build_client(profile: str | None = None) -> Client:
     """
     Builds a `cloud.Client` using the API key and endpoint for the given
@@ -91,23 +104,35 @@ def build_client(profile: str | None = None) -> Client:
 
     if profile is not None:
         if profile not in config:
-            error(f"Profile [magenta]{profile}[/magenta] does not exist.")
+            error(
+                f"Profile [magenta]{profile}[/magenta] does not exist. "
+                "Create it using [code]nextmv configuration create[/code] with the --profile option."
+            )
 
         api_key = config[profile].get(API_KEY_KEY)
         if api_key is None or api_key == "":
-            error(f"API key for profile [magenta]{profile}[/magenta] is not set or is empty.")
+            error(
+                f"API key for profile [magenta]{profile}[/magenta] is not set or is empty. "
+                "Set it using [code]nextmv configuration create[/code] with the --profile and --api-key options."
+            )
 
         endpoint = config[profile].get(ENDPOINT_KEY)
         if endpoint is None or endpoint == "":
-            error(f"Endpoint for profile [magenta]{profile}[/magenta] is not set or is empty.")
+            error(
+                f"Endpoint for profile [magenta]{profile}[/magenta] is not set or is empty. "
+                "Please run [code]nextmv configuration create[/code]."
+            )
     else:
         api_key = config.get(API_KEY_KEY)
         if api_key is None or api_key == "":
-            error("Default API key is not set or is empty.")
+            error(
+                "Default API key is not set or is empty. "
+                "Please run [code]nextmv configuration create[/code] with the --api-key option."
+            )
 
         endpoint = config.get(ENDPOINT_KEY)
         if endpoint is None or endpoint == "":
-            error("Default endpoint is not set or is empty.")
+            error("Default endpoint is not set or is empty. Please run [code]nextmv configuration create[/code].")
 
     return Client(api_key=api_key, url=f"https://{endpoint}")
 
@@ -137,13 +162,21 @@ def build_app(app_id: str, profile: str | None = None) -> Application:
     """
     client = build_client(profile)
     exists = Application.exists(client=client, id=app_id)
-    if not exists:
+    if exists:
+        return Application(client=client, id=app_id)
+
+    warning(f"Application with ID [magenta]{app_id}[/magenta] does not exist.")
+    should_create = get_confirmation(f"Do you want to create a new application with ID [magenta]{app_id}[/magenta]?")
+    if not should_create:
         error(
-            f"Application with ID [magenta]{app_id}[/magenta] does not exist. "
-            "Use [code]nextmv cloud app create[/code] to create a new application."
+            f"Application with ID [magenta]{app_id}[/magenta] was not created and does not exist. "
+            "Use [code]nextmv cloud app create[/code] to create a new app."
         )
 
-    return Application(client=client, id=app_id)
+    app = Application.new(client=client, id=app_id, name=app_id)
+    success(f"Application with ID and name [magenta]{app_id}[/magenta] created successfully.")
+
+    return app
 
 
 def build_account(account_id: str | None = None, profile: str | None = None) -> Account:

nextmv/cli/configuration/create.py

@@ -58,14 +58,14 @@ def create(
     [bold][underline]Examples[/underline][/bold]
 
     - Default configuration.
-        $ [green]nextmv configuration create --api-key NEXTMV_API_KEY[/green]
+        $ [dim]nextmv configuration create --api-key NEXTMV_API_KEY[/dim]
 
     - Configure a profile named [magenta]hare[/magenta].
-        $ [green]nextmv configuration create --api-key NEXTMV_API_KEY --profile hare[/green]
+        $ [dim]nextmv configuration create --api-key NEXTMV_API_KEY --profile hare[/dim]
     """
 
     if profile is not None and profile.strip().lower() == "default":
-        error("[code]default[/code] is a reserved profile name.")
+        error("[magenta]default[/magenta] is a reserved profile name.")
 
     endpoint = str(endpoint)
     if endpoint.startswith("https://"):

nextmv/cli/configuration/delete.py

@@ -5,9 +5,9 @@ This module defines the configuration delete command for the Nextmv CLI.
 from typing import Annotated
 
 import typer
-from rich.prompt import Confirm
 
 from nextmv.cli.configuration.config import load_config, save_config
+from nextmv.cli.confirm import get_confirmation
 from nextmv.cli.message import error, info, success
 
 # Set up subcommand application.
@@ -36,25 +36,25 @@ def delete(
     ] = False,
 ) -> None:
     """
-    Delete a profile from the configuration. Use the [code]--yes[/code]
-    flag to skip the confirmation prompt.
+    Delete a profile from the configuration.
+
+    Use the --yes flag to skip the confirmation prompt.
 
     [bold][underline]Examples[/underline][/bold]
 
     - Delete a profile named [magenta]hare[/magenta].
-        $ [green]nextmv configuration delete --profile hare[/green]
+        $ [dim]nextmv configuration delete --profile hare[/dim]
 
     - Delete a profile named [magenta]hare[/magenta] without confirmation prompt.
-        $ [green]nextmv configuration delete --profile hare --yes[/green]
+        $ [dim]nextmv configuration delete --profile hare --yes[/dim]
     """
     config = load_config()
     if profile not in config:
         error(f"Profile [magenta]{profile}[/magenta] does not exist.")
 
     if not yes:
-        confirm = Confirm.ask(
+        confirm = get_confirmation(
             f"Are you sure you want to delete profile [magenta]{profile}[/magenta]? This action cannot be undone.",
-            default=False,
         )
 
         if not confirm:

nextmv/cli/configuration/list.py

@@ -6,7 +6,7 @@ import typer
 from rich.console import Console
 from rich.table import Table
 
-from nextmv.cli.configuration.config import API_KEY_KEY, ENDPOINT_KEY, load_config, obscure_api_key
+from nextmv.cli.configuration.config import API_KEY_KEY, ENDPOINT_KEY, load_config, non_profile_keys, obscure_api_key
 from nextmv.cli.message import error
 
 # Set up subcommand application.
@@ -22,7 +22,7 @@ def list() -> None:
     [bold][underline]Examples[/underline][/bold]
 
     - Show current configuration and all profiles.
-        $ [green]nextmv configuration list[/green]
+        $ [dim]nextmv configuration list[/dim]
     """
 
     config = load_config()
@@ -38,7 +38,7 @@ def list() -> None:
 
     for k, v in config.items():
         # Skip default configuration.
-        if k in {API_KEY_KEY, ENDPOINT_KEY}:
+        if k in non_profile_keys():
             continue
 
         profile = {

nextmv/cli/confirm.py

@@ -0,0 +1,34 @@
+import sys
+
+from rich.prompt import Confirm
+
+
+def get_confirmation(msg: str, default: bool = False) -> bool:
+    """
+    Method to get a yes/no confirmation from the user.
+
+    Parameters
+    ----------
+    msg : str
+        The message to display to the user.
+    default : bool, optional
+        The default value if the user just presses Enter. Default is False.
+
+    Returns
+    -------
+    bool
+        True if the user confirmed, False otherwise.
+    """
+
+    # If this is not an interactive terminal, do not ask for confirmation, to
+    # avoid hanging indefinitely waiting for a user response.
+    if not sys.stdin.isatty():
+        return default
+
+    return Confirm.ask(
+        msg,
+        default=default,
+        case_sensitive=False,
+        show_default=True,
+        show_choices=True,
+    )

nextmv/cli/main.py

@@ -13,22 +13,25 @@ about the features used here. An example of Rich markup can be found in the
 epilog of the Typer application defined below.
 """
 
-import os
 import sys
 from typing import Annotated
 
 import rich
 import typer
-from rich.prompt import Confirm
+from typer import rich_utils
 
 from nextmv.cli.cloud import app as cloud_app
 from nextmv.cli.community import app as community_app
 from nextmv.cli.configuration import app as configuration_app
 from nextmv.cli.configuration.config import CONFIG_DIR, GO_CLI_PATH, load_config
+from nextmv.cli.confirm import get_confirmation
 from nextmv.cli.message import error, info, success, warning
 from nextmv.cli.version import app as version_app
 from nextmv.cli.version import version_callback
 
+# Disable dim text for the extended help of commands.
+rich_utils.STYLE_HELPTEXT = ""
+
 # Main CLI application.
 app = typer.Typer(
     help="The Nextmv Command Line Interface (CLI).",
@@ -66,6 +69,15 @@ def callback(
     environment.
     """
 
+    # Skip checks for help commands.
+    if "--help" in sys.argv or "-h" in sys.argv:
+        return
+
+    # Skip checks for certain commands.
+    ignored_commands = {"configuration", "version"}
+    if ctx.invoked_subcommand in ignored_commands:
+        return
+
     handle_go_cli()
     handle_config_existence(ctx)
 
@@ -80,19 +92,21 @@ def handle_go_cli() -> None:
 
     exists = go_cli_exists()
     if exists:
-        delete = Confirm.ask(
+        delete = get_confirmation(
             "Do you want to delete the [italic red]deprecated[/italic red] Nextmv CLI "
-            f"at [magenta]{GO_CLI_PATH}[/magenta] now?",
-            default=False,
+            f"at [magenta]{GO_CLI_PATH}[/magenta] now?"
         )
         if delete:
             remove_go_cli()
-        else:
-            info(
-                msg="You can delete the [italic red]deprecated[/italic red] Nextmv CLI later by removing "
-                f"[magenta]{GO_CLI_PATH}[/magenta]. Make sure you also clean up your [code]PATH[/code].",
-                emoji=":bulb:",
-            )
+            return
+
+        info(
+            msg="You can delete the [italic red]deprecated[/italic red] Nextmv CLI later by removing "
+            f"[magenta]{GO_CLI_PATH}[/magenta]. "
+            "Make sure you also clean up your [code]PATH[/code], "
+            f"by removing references to [magenta]{CONFIG_DIR}[/magenta] from it.",
+            emoji=":bulb:",
+        )
 
 
 def handle_config_existence(ctx: typer.Context) -> None:
@@ -105,10 +119,6 @@ def handle_config_existence(ctx: typer.Context) -> None:
         The Typer context object.
     """
 
-    ignored_commands = {"configuration", "version"}
-    if ctx.invoked_subcommand in ignored_commands:
-        return
-
     config = load_config()
     if config == {}:
         error("No configuration found. Please run [code]nextmv configuration create[/code].")
@@ -130,11 +140,9 @@ def go_cli_exists() -> bool:
     if exists:
         warning(
             "A [italic red]deprecated[/italic red] Nextmv CLI is installed at "
-            f"[magenta]{GO_CLI_PATH}[/magenta]. You must delete it to avoid conflicts."
+            f"[magenta]{GO_CLI_PATH}[/magenta]. You should delete it to avoid conflicts."
         )
 
-    check_config_in_path()
-
     return exists
 
 
@@ -145,24 +153,7 @@ def remove_go_cli() -> None:
 
     if GO_CLI_PATH.exists():
         GO_CLI_PATH.unlink()
-        success(f"Deleted deprecated [magenta]{GO_CLI_PATH}[/magenta].")
-
-    check_config_in_path()
-
-
-def check_config_in_path() -> None:
-    """
-    Check if the configuration directory is in the PATH and notify the user.
-    """
-
-    path_dirs = os.environ.get("PATH", "").split(os.pathsep)
-    config_dir_str = str(CONFIG_DIR)
-
-    if config_dir_str in path_dirs:
-        warning(
-            f"[magenta]{CONFIG_DIR}[/magenta] was found in your [code]PATH[/code]. "
-            f"You should remove any entries related to [magenta]{CONFIG_DIR}[/magenta] from your [code]PATH[/code]."
-        )
+        success(f"Deleted [italic red]deprecated[/italic red] [magenta]{GO_CLI_PATH}[/magenta].")
 
 
 def main() -> None:

nextmv/cli/message.py

@@ -31,7 +31,7 @@ def error(msg: str) -> None:
     if not msg.endswith("."):
         msg += "."
 
-    rich.print(f"[red]Error:[/red] {msg}", file=sys.stderr)
+    rich.print(f":x: [red]Error:[/red] {msg}", file=sys.stderr)
 
     raise typer.Exit(code=1)
 
@@ -67,7 +67,7 @@ def warning(msg: str) -> None:
     if not msg.endswith("."):
         msg += "."
 
-    rich.print(f":construction: {msg}", file=sys.stderr)
+    rich.print(f":construction: [yellow] Warning:[/yellow] {msg}", file=sys.stderr)
 
 
 def info(msg: str, emoji: str | None = None) -> None:

nextmv/cli/version.py

@@ -18,7 +18,7 @@ def version() -> None:
     [bold][underline]Examples[/underline][/bold]
 
     - Show the version.
-        $ [green]nextmv version[/green]
+        $ [dim]nextmv version[/dim]
     """
 
     version_callback(True)

nextmv/cloud/__init__.py

@@ -1,42 +1,5 @@
 """Functionality for interacting with the Nextmv Cloud."""
 
-# These imports are kept for backwards compatibility but the preferred import path is
-# from nextmv directly. These imports will be removed in a future release.
-from nextmv.manifest import MANIFEST_FILE_NAME as MANIFEST_FILE_NAME
-from nextmv.manifest import Manifest as Manifest
-from nextmv.manifest import ManifestBuild as ManifestBuild
-from nextmv.manifest import ManifestContent as ManifestContent
-from nextmv.manifest import ManifestContentMultiFile as ManifestContentMultiFile
-from nextmv.manifest import ManifestContentMultiFileInput as ManifestContentMultiFileInput
-from nextmv.manifest import ManifestContentMultiFileOutput as ManifestContentMultiFileOutput
-from nextmv.manifest import ManifestOption as ManifestOption
-from nextmv.manifest import ManifestPython as ManifestPython
-from nextmv.manifest import ManifestPythonModel as ManifestPythonModel
-from nextmv.manifest import ManifestRuntime as ManifestRuntime
-from nextmv.manifest import ManifestType as ManifestType
-from nextmv.polling import PollingOptions as PollingOptions
-from nextmv.polling import poll as poll
-from nextmv.run import ErrorLog as ErrorLog
-from nextmv.run import ExternalRunResult as ExternalRunResult
-from nextmv.run import Format as Format
-from nextmv.run import FormatInput as FormatInput
-from nextmv.run import FormatOutput as FormatOutput
-from nextmv.run import Metadata as Metadata
-from nextmv.run import RunConfiguration as RunConfiguration
-from nextmv.run import RunInformation as RunInformation
-from nextmv.run import RunLog as RunLog
-from nextmv.run import RunQueuing as RunQueuing
-from nextmv.run import RunResult as RunResult
-from nextmv.run import RunType as RunType
-from nextmv.run import RunTypeConfiguration as RunTypeConfiguration
-from nextmv.run import TrackedRun as TrackedRun
-from nextmv.run import TrackedRunStatus as TrackedRunStatus
-from nextmv.run import run_duration as run_duration
-from nextmv.safe import safe_id as safe_id
-from nextmv.safe import safe_name_and_id as safe_name_and_id
-from nextmv.status import Status as Status
-from nextmv.status import StatusV2 as StatusV2
-
 from .acceptance_test import AcceptanceTest as AcceptanceTest
 from .acceptance_test import AcceptanceTestResults as AcceptanceTestResults
 from .acceptance_test import Comparison as Comparison
@@ -52,7 +15,6 @@ from .acceptance_test import MetricToleranceType as MetricToleranceType
 from .acceptance_test import MetricType as MetricType
 from .acceptance_test import ResultStatistics as ResultStatistics
 from .acceptance_test import StatisticType as StatisticType
-from .acceptance_test import ToleranceType as ToleranceType
 from .account import Account as Account
 from .account import AccountMember as AccountMember
 from .account import Queue as Queue

nextmv/cloud/acceptance_test.py

@@ -13,7 +13,7 @@ StatisticType : Enum
     Type of statistical process for collapsing multiple values of a metric.
 Comparison : Enum
     Comparison operators to use for comparing two metrics.
-ToleranceType : Enum
+MetricToleranceType : Enum
     Type of tolerance used for a metric.
 ExperimentStatus : Enum
     Status of an acceptance test experiment.
@@ -46,7 +46,6 @@ from enum import Enum
 
 from nextmv.base_model import BaseModel
 from nextmv.cloud.batch_experiment import ExperimentStatus
-from nextmv.deprecated import deprecated
 
 
 class MetricType(str, Enum):
@@ -212,69 +211,6 @@ class Comparison(str, Enum):
     """Not equal to metric type."""
 
 
-class ToleranceType(str, Enum):
-    """
-    !!! warning
-        `ToleranceType` is deprecated, use `MetricToleranceType` instead.
-
-    Type of tolerance used for a metric.
-
-    You can import the `ToleranceType` class directly from `cloud`:
-
-    ```python
-    from nextmv.cloud import ToleranceType
-    ```
-
-    This enumeration defines the different types of tolerances that can be used
-    when comparing metrics in acceptance tests.
-
-    Attributes
-    ----------
-    undefined : str
-        Undefined tolerance type (empty string).
-    absolute : str
-        Absolute tolerance type, using a fixed value.
-    relative : str
-        Relative tolerance type, using a percentage.
-
-    Examples
-    --------
-    >>> from nextmv.cloud import ToleranceType
-    >>> tol_type = ToleranceType.absolute
-    >>> tol_type
-    <ToleranceType.absolute: 'absolute'>
-    """
-
-    undefined = ""
-    """ToleranceType is deprecated, please use MetricToleranceType instead.
-    Undefined tolerance type."""
-    absolute = "absolute"
-    """ToleranceType is deprecated, please use MetricToleranceType instead.
-    Absolute tolerance type."""
-    relative = "relative"
-    """ToleranceType is deprecated, please use MetricToleranceType instead.
-    Relative tolerance type."""
-
-
-# Override __getattribute__ to emit deprecation warnings when enum values are accessed
-_original_getattribute = ToleranceType.__class__.__getattribute__
-
-
-def _deprecated_getattribute(cls, name: str):
-    # Only emit deprecation warning if this is specifically the ToleranceType class
-    if cls is ToleranceType and name in ("undefined", "absolute", "relative"):
-        deprecated(
-            f"ToleranceType.{name}",
-            "ToleranceType is deprecated and will be removed in a future version. "
-            "Please use MetricToleranceType instead",
-        )
-
-    return _original_getattribute(cls, name)
-
-
-ToleranceType.__class__.__getattribute__ = _deprecated_getattribute
-
-
 class MetricToleranceType(str, Enum):
     """
     Type of tolerance used for a metric.

nextmv/cloud/account.py

@@ -21,7 +21,7 @@ from pydantic import AliasChoices, Field
 
 from nextmv.base_model import BaseModel
 from nextmv.cloud.client import Client
-from nextmv.status import Status, StatusV2
+from nextmv.status import StatusV2
 
 
 class QueuedRun(BaseModel):
@@ -57,8 +57,6 @@ class QueuedRun(BaseModel):
         ID of the application version used for the run.
     execution_class : str
         Execution class used for the run.
-    status : Status
-        Deprecated: use status_v2.
     status_v2 : StatusV2
         Status of the run.
 
@@ -74,7 +72,6 @@ class QueuedRun(BaseModel):
     ...     "application_instance_id": "appins-123456",
     ...     "application_version_id": "appver-123456",
     ...     "execution_class": "standard",
-    ...     "status": "RUNNING",
     ...     "status_v2": "RUNNING"
     ... })
     >>> print(queued_run.name)
@@ -99,8 +96,6 @@ class QueuedRun(BaseModel):
     """ID of the application version used for the run."""
     execution_class: str
     """Execution class used for the run."""
-    status: Status
-    """Deprecated: use status_v2."""
     status_v2: StatusV2
     """Status of the run."""