nextmv 1.4.1.dev3__tar.gz → 1.4.1.dev4__tar.gz

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: nextmv
-Version: 1.4.1.dev3
+Version: 1.4.1.dev4
 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.1.dev4/nextmv/__about__.py

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

{nextmv-1.4.1.dev3 → nextmv-1.4.1.dev4}/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
+
+                return profile_env
 
-        if self.profile is not None and self.profile != "":
-            return self.profile
+        if self.profile is not None:
+            profile = self.profile.strip()
+            if profile != "":
+                if profile.lower() == "default":
+                    return None
+
+                return profile
 
         return None
 
@@ -583,24 +601,24 @@ class Client:
         str
             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 != "":
+        if self.url is not None and self.url != "":
+            url = self.url
+        elif url_key_env is not None and url_key_env != "":
+            url = url_key_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.1.dev4}/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/__about__.py

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