nextmv 1.4.1.dev3__tar.gz → 1.4.2.dev3__tar.gz

{nextmv-1.4.1.dev3 → nextmv-1.4.2.dev3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nextmv
-Version: 1.4.1.dev3
+Version: 1.4.2.dev3
 Summary: The all-purpose Python SDK for Nextmv
 Project-URL: Homepage, https://www.nextmv.io
 Project-URL: Documentation, https://nextmv-py.docs.nextmv.io/en/latest/nextmv/

nextmv-1.4.2.dev3/nextmv/__about__.py

@@ -0,0 +1 @@
+__version__ = "v1.4.2.dev3"

{nextmv-1.4.1.dev3 → nextmv-1.4.2.dev3}/nextmv/cloud/application/__init__.py

@@ -811,6 +811,9 @@ class Application(
 
         if manifest.configuration is not None and manifest.configuration.options is not None:
             options = manifest.configuration.options.to_dict()
+            # Ignore local_only options since they are not meant to be used on the platform
+            if "items" in options:
+                options["items"] = [item for item in options["items"] if not item.get("local_only", False)]
             if "format" in options and isinstance(options["format"], list):
                 # the endpoint expects a dictionary with a template key having a list of strings
                 # the app.yaml however defines format as a list of strings, so we need to convert it here

{nextmv-1.4.1.dev3 → nextmv-1.4.2.dev3}/nextmv/cloud/client.py

@@ -24,6 +24,7 @@ import requests
 import yaml
 from requests.adapters import HTTPAdapter, Retry
 
+from nextmv import deprecated
 from nextmv._serialization import deflated_serialize_json
 
 _MAX_LAMBDA_PAYLOAD_SIZE: int = 500 * 1024 * 1024
@@ -197,12 +198,10 @@ class Client:
     backoff_max: float = 60
     """Maximum backoff time to use for requests to the Nextmv Cloud API, in
     seconds."""
-    configuration_file: str = "~/.nextmv/config.yaml"
+    configuration_file: str | None = None
     """
     Deprecated. This attribute is no longer being used. Use the `profile`
     attribute to specify different configurations instead.
-
-    Path to the configuration file used by the Nextmv CLI.
     """
     headers: dict[str, str] | None = None
     """Headers to use for requests to the Nextmv Cloud API."""
@@ -255,11 +254,14 @@ class Client:
         Raises
         ------
         ValueError
-            If `api_key` is an empty string.
-            If no API key is found in any of the lookup locations.
-            If a profile is specified via `NEXTMV_PROFILE` but not found in
-            the configuration file.
-            If `apikey` is not found in the configuration file for the
+            If no API key is found after checking the constructor argument,
+            the ``NEXTMV_API_KEY`` environment variable, and the
+            configuration file (for the resolved profile or the default
+            profile). A ``None`` or empty ``api_key`` is treated as unset
+            and causes the lookup to fall through to the next source.
+            If a profile is specified via ``NEXTMV_PROFILE`` or the
+            ``profile`` attribute but is not found in the configuration file.
+            If ``apikey`` is not found in the configuration file for the
             selected profile.
         """
 
@@ -268,6 +270,12 @@ class Client:
         self.api_key = self.__resolve_api_key(profile)
         self.__set_headers_api_key(self.api_key)
 
+        if self.configuration_file is not None and self.configuration_file != "":
+            deprecated(
+                name="Client.configuration_file",
+                reason="`Client.configuration_file` is deprecated, use `Client.profile` to work with another profile",
+            )
+
     def request(
         self,
         method: str,
@@ -553,11 +561,21 @@ class Client:
             The resolved profile name, or `None` if no profile is set.
         """
         profile_env = os.getenv("NEXTMV_PROFILE")
-        if profile_env is not None and profile_env != "":
-            return profile_env
+        if profile_env is not None:
+            profile_env = profile_env.strip()
+            if profile_env != "":
+                if profile_env.lower() == "default":
+                    return None
 
-        if self.profile is not None and self.profile != "":
-            return self.profile
+                return profile_env
+
+        if self.profile is not None:
+            profile = self.profile.strip()
+            if profile != "":
+                if profile.lower() == "default":
+                    return None
+
+                return profile
 
         return None
 
@@ -584,23 +602,22 @@ class Client:
             The resolved API endpoint URL.
         """
         if self.url is not None and self.url != "":
-            return self.url
-
-        url_key_env = os.getenv("NEXTMV_ENDPOINT")
-        if url_key_env is not None and url_key_env != "":
-            return url_key_env
-
-        if profile is not None and profile != "":
+            url = self.url
+        elif (url_env := os.getenv("NEXTMV_ENDPOINT")):
+            url = url_env
+        elif profile is not None and profile != "":
             url = _retrieve_endpoint_from_config(profile)
-            return url
-
-        # The fallback behavior is to attempt to retrieve the default endpoint
-        # from the config file. If everything fails, we return the hardcoded
-        # default endpoint.
-        try:
-            url = _retrieve_endpoint_from_config()
-        except (RuntimeError, ValueError):
-            url = "https://api.cloud.nextmv.io"
+        else:
+            # The fallback behavior is to attempt to retrieve the default endpoint
+            # from the config file. If everything fails, we return the hardcoded
+            # default endpoint.
+            try:
+                url = _retrieve_endpoint_from_config()
+            except (RuntimeError, ValueError):
+                url = "https://api.cloud.nextmv.io"
+
+        if not url.startswith("https://") and not url.startswith("http://"):
+            url = f"https://{url}"
 
         return url
 

{nextmv-1.4.1.dev3 → nextmv-1.4.2.dev3}/nextmv/manifest.py

@@ -511,6 +511,8 @@ class ManifestOption(BaseModel):
         The description of the option.
     required : bool, default=False
         Whether the option is required or not.
+    local_only : bool, default=False
+        Whether the option is only used locally and should not be sent to Nextmv Cloud.
     additional_attributes : Optional[dict[str, Any]], default=None
         Optional additional attributes for the option. The Nextmv Cloud may
         perform validation on these attributes. For example, the maximum
@@ -551,11 +553,28 @@ class ManifestOption(BaseModel):
     """The description of the option"""
     required: bool = False
     """Whether the option is required or not"""
+    local_only: bool = False
+    """Whether the option is only used locally and should not be sent to Nextmv Cloud."""
     additional_attributes: dict[str, Any] | None = None
     """Optional additional attributes for the option."""
     ui: ManifestOptionUI | None = None
     """Optional UI attributes for the option."""
 
+    def model_post_init(self, __context) -> None:
+        """
+        Validations done after parsing the model.
+
+        Raises
+        ------
+        ValueError
+            If validation fails. A descriptive error message is provided in this case.
+        """
+
+        # It does not make sense to define options as required AND them being local only,
+        # since this would make Platform runs via UI cumbersome to impossible.
+        if self.required and self.local_only:
+            raise ValueError(f"Option '{self.name}' cannot be both required and local only.")
+
     @classmethod
     def from_option(cls, option: Option) -> "ManifestOption":
         """

{nextmv-1.4.1.dev3 → nextmv-1.4.2.dev3}/tests/cli/test_mcp.py

@@ -362,7 +362,7 @@ class TestGetClient(unittest.TestCase):
 
     @patch.dict("os.environ", {}, clear=True)
     @patch("nextmv.cli.mcp.tools._helpers.Client", side_effect=Exception("no config"))
-    def test_get_client_no_key_no_config_raises(self, mock_build):
+    def test_get_client_no_key_no_config_raises(self, mock_client):
         """Test that _get_client raises when no API key or config is available."""
         from nextmv.cli.mcp.server import _get_client
 

{nextmv-1.4.1.dev3 → nextmv-1.4.2.dev3}/tests/cloud/app.yaml

@@ -64,6 +64,12 @@ configuration:
           control_type: select
           hidden_from:
             - operator
+      - name: a local only parameter
+        option_type: string
+        default: local_default
+        description: a local only description
+        required: false
+        local_only: true
   content:
     format: "multi-file"
     multi-file:

{nextmv-1.4.1.dev3 → nextmv-1.4.2.dev3}/tests/test_manifest.py

@@ -189,7 +189,7 @@ class TestManifest(unittest.TestCase):
     def test_extract_options(self):
         manifest = Manifest.from_yaml("tests/cloud")
         options = manifest.extract_options(should_parse=False)
-        self.assertEqual(len(options.options), 5)
+        self.assertEqual(len(options.options), 6)
 
         found = {
             "string": False,
@@ -500,6 +500,57 @@ class TestManifestOption(unittest.TestCase):
                 self.assertEqual(option.required, manifest_option.required)
 
 
+class TestManifestOptionLocalOnly(unittest.TestCase):
+    def test_local_only_defaults_to_false(self):
+        option = ManifestOption(name="my_option", option_type="string", default="val")
+        self.assertFalse(option.local_only)
+
+    def test_local_only_can_be_set_to_true(self):
+        option = ManifestOption(name="my_option", option_type="int", default=0, local_only=True)
+        self.assertTrue(option.local_only)
+
+    def test_required_and_local_only_raises(self):
+        with self.assertRaises(ValueError) as ctx:
+            ManifestOption(name="my_option", option_type="string", default="val", required=True, local_only=True)
+        self.assertIn("my_option", str(ctx.exception))
+        self.assertIn("required", str(ctx.exception).lower())
+        self.assertIn("local only", str(ctx.exception).lower())
+
+    def test_local_only_round_trips_through_dict(self):
+        original = ManifestOption(name="my_option", option_type="float", default=1.5, local_only=True)
+        as_dict = original.to_dict()
+        self.assertTrue(as_dict["local_only"])
+        restored = ManifestOption.from_dict(as_dict)
+        self.assertTrue(restored.local_only)
+
+    def test_non_local_only_round_trips_through_dict(self):
+        original = ManifestOption(name="my_option", option_type="bool", default=False, local_only=False)
+        as_dict = original.to_dict()
+        self.assertFalse(as_dict["local_only"])
+        restored = ManifestOption.from_dict(as_dict)
+        self.assertFalse(restored.local_only)
+
+    def test_local_only_loaded_from_yaml(self):
+        manifest = Manifest.from_yaml("tests/cloud")
+        items = manifest.configuration.options.items
+        local_only_items = [item for item in items if item.local_only]
+        self.assertEqual(len(local_only_items), 1)
+        item = local_only_items[0]
+        self.assertEqual(item.name, "a local only parameter")
+        self.assertEqual(item.option_type, "string")
+        self.assertEqual(item.default, "local_default")
+        self.assertFalse(item.required)
+        self.assertTrue(item.local_only)
+
+    def test_other_options_are_not_local_only(self):
+        manifest = Manifest.from_yaml("tests/cloud")
+        items = manifest.configuration.options.items
+        non_local_only_items = [item for item in items if not item.local_only]
+        # All items except the one explicitly marked local_only should have local_only=False.
+        for item in non_local_only_items:
+            self.assertFalse(item.local_only, msg=f"Option '{item.name}' should not be local_only")
+
+
 class TestWriteSampleManifest(unittest.TestCase):
     def test_writes_python_manifest(self):
         with tempfile.TemporaryDirectory() as dirpath:

nextmv-1.4.1.dev3/nextmv/__about__.py

@@ -1 +0,0 @@
-__version__ = "v1.4.1.dev3"