terradev-cli 5.4.0__tar.gz → 5.4.2__tar.gz

{terradev_cli-5.4.0 → terradev_cli-5.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: terradev-cli
-Version: 5.4.0
+Version: 5.4.2
 Summary: An imperative command-line-interface for AI workload orchestration.
 Author-email: Terradev Team <team@terradev.com>
 Maintainer-email: Terradev Team <team@terradev.com>
@@ -93,7 +93,7 @@ Requires-Dist: redis>=5.0.0; extra == "enterprise"
 Requires-Dist: PyJWT>=2.8.0; extra == "enterprise"
 Dynamic: license-file
 
-# Terradev CLI v5.3.8
+# Terradev CLI v5.4.2
 
 **An imperative command-line-interface for AI workload orchestration.**
 
@@ -111,6 +111,19 @@ Continued focus on lower cost, faster provisioning, and topology-aware execution
 
 Model agnostic. Dataset agnostic. GPU agnostic. Provider agnostic. The only thing Terradev is not agnostic about is correctness: it enforces topology, idempotency, and sequencing.
 
+**NOTES ON 5.4.2**
+
+Fixed **credential type checking bug** in `is_first_time_user()` and onboarding:
+
+- **Bug**: `is_first_time_user()` and onboarding code called `.lower()` on credential values without checking if they were strings. The credentials file uses nested dict format `{"runpod": {"api_key": "..."}}`, so when iterating over values, some are dicts, not strings. This caused `AttributeError: 'dict' object has no attribute 'lower'`.
+
+- **Fix**:
+  - Added `isinstance(value, str)` check before calling `.lower()` in `is_first_time_user()` (line 119)
+  - Added `isinstance(existing_value, str)` check before calling `.lower()` in onboarding (line 764)
+  - The `_provider_creds()` method already correctly handles both flat and nested credential formats
+
+- **Impact**: `terradev quote` and other commands that check credentials now work correctly with the nested dict credential format.
+
 **NOTES ON 5.3.9**
 
 Added **LoRAX (LoRA eXchange) integration** and **HuggingFace PEFT import** for production-grade multi-LoRA inference serving:

{terradev_cli-5.4.0/terradev_cli → terradev_cli-5.4.2}/README.md

@@ -1,4 +1,4 @@
-# Terradev CLI v5.3.8
+# Terradev CLI v5.4.2
 
 **An imperative command-line-interface for AI workload orchestration.**
 
@@ -16,6 +16,19 @@ Continued focus on lower cost, faster provisioning, and topology-aware execution
 
 Model agnostic. Dataset agnostic. GPU agnostic. Provider agnostic. The only thing Terradev is not agnostic about is correctness: it enforces topology, idempotency, and sequencing.
 
+**NOTES ON 5.4.2**
+
+Fixed **credential type checking bug** in `is_first_time_user()` and onboarding:
+
+- **Bug**: `is_first_time_user()` and onboarding code called `.lower()` on credential values without checking if they were strings. The credentials file uses nested dict format `{"runpod": {"api_key": "..."}}`, so when iterating over values, some are dicts, not strings. This caused `AttributeError: 'dict' object has no attribute 'lower'`.
+
+- **Fix**:
+  - Added `isinstance(value, str)` check before calling `.lower()` in `is_first_time_user()` (line 119)
+  - Added `isinstance(existing_value, str)` check before calling `.lower()` in onboarding (line 764)
+  - The `_provider_creds()` method already correctly handles both flat and nested credential formats
+
+- **Impact**: `terradev quote` and other commands that check credentials now work correctly with the nested dict credential format.
+
 **NOTES ON 5.3.9**
 
 Added **LoRAX (LoRA eXchange) integration** and **HuggingFace PEFT import** for production-grade multi-LoRA inference serving:

{terradev_cli-5.4.0 → terradev_cli-5.4.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "terradev-cli"
-version = "5.4.0"
+version = "5.4.2"
 description = "An imperative command-line-interface for AI workload orchestration."
 readme = "README.md"
 license = {text = "Apache-2.0"}

{terradev_cli-5.4.0 → terradev_cli-5.4.2/terradev_cli}/README.md

@@ -1,4 +1,4 @@
-# Terradev CLI v5.3.8
+# Terradev CLI v5.4.1
 
 **An imperative command-line-interface for AI workload orchestration.**
 
@@ -16,6 +16,33 @@ Continued focus on lower cost, faster provisioning, and topology-aware execution
 
 Model agnostic. Dataset agnostic. GPU agnostic. Provider agnostic. The only thing Terradev is not agnostic about is correctness: it enforces topology, idempotency, and sequencing.
 
+**NOTES ON 5.4.1**
+
+Fixed **credential persistence scope bug** in `terradev configure`:
+
+- **Bug**: `api` variable was only instantiated inside the `else` block (interactive mode), but `api.save_credentials()` was called unconditionally at line 1530. This caused `UnboundLocalError` when using `--provider` flag (single-provider mode), and would have overwritten credentials with empty data in interactive mode.
+
+- **Fix**:
+  - Moved `api = TerradevAPI()` to line 952 (before `if provider:` branching) to ensure it's always in scope
+  - Removed duplicate `api` instantiation from the `else` block
+  - Moved `api.save_credentials()` inside the `else` block (line 1531) - it only runs in interactive mode since single-provider mode saves directly to file at line 1186-1187
+
+- **Impact**: Both `terradev configure --provider <name>` and `terradev configure` (interactive) now correctly persist credentials to `~/.terradev/credentials.json`.
+
+**NOTES ON 5.4.0**
+
+Fixed **credential persistence bug** in `terradev configure` interactive mode:
+
+- **Bug**: Interactive mode (`terradev configure` without `--provider` flag) crashed with `NameError: name 'api' is not defined` at line 1515, preventing credentials from being saved to disk. This caused `terradev quote` to return empty results for all providers.
+
+- **Fix**: 
+  - Added `api = TerradevAPI()` initialization in interactive mode
+  - Added `api.load_credentials()` after `prompt_for_credentials()` to reload credentials from disk
+  - Fixed incorrectly nested Kubernetes configuration prompts (were inside "No providers configured" else block)
+  - Restored `api.save_credentials()` to save all credentials (provider + ML integrations + Kubernetes settings)
+
+- **Impact**: Users can now successfully configure providers in interactive mode and retrieve real-time pricing via `terradev quote`.
+
 **NOTES ON 5.3.9**
 
 Added **LoRAX (LoRA eXchange) integration** and **HuggingFace PEFT import** for production-grade multi-LoRA inference serving:

{terradev_cli-5.4.0 → terradev_cli-5.4.2}/terradev_cli/cli.py

@@ -116,7 +116,7 @@ class TerradevAPI:
         # Check if all credentials are still placeholder values
         placeholder_patterns = ["your_", "example_", "test_", "placeholder_", "xxx"]
         for key, value in self.credentials.items():
-            if value and not any(
+            if value and isinstance(value, str) and not any(
                 pattern in value.lower() for pattern in placeholder_patterns
             ):
                 return False  # Found a real credential
@@ -761,7 +761,7 @@ def run_interactive_onboarding(api: TerradevAPI):
         existing_value = api.credentials.get(
             f"{provider_key}_api_key"
         ) or api.credentials.get(f"{provider_key}_access_key_id")
-        if existing_value and not any(
+        if existing_value and isinstance(existing_value, str) and not any(
             pattern in existing_value.lower()
             for pattern in ["your_", "example_", "test_", "placeholder_", "xxx"]
         ):
@@ -947,6 +947,10 @@ def configure(provider):
       RunPod is the easiest to set up (5 minutes): terradev setup runpod --quick
     """
 
+    # Initialize API object for credential management
+    # Must be instantiated before branching to ensure it's in scope for save_credentials()
+    api = TerradevAPI()
+
     if provider:
         # Configure specific provider
         from terradev_cli.credential_prompt import prompt_for_credentials
@@ -1204,9 +1208,6 @@ def configure(provider):
         # Interactive configuration for all providers
         from terradev_cli.credential_prompt import prompt_for_credentials
 
-        # Initialize API object for credential management
-        api = TerradevAPI()
-
         configured_providers = prompt_for_credentials()
 
         # Reload credentials from file after prompt_for_credentials saves them
@@ -1525,13 +1526,14 @@ def configure(provider):
                 api.credentials["ray_head_node_ip"] = ray_head
             print("   Ray configured  distributed computing")
 
-    # Save all credentials (provider + integrations) to disk
-    api.save_credentials()
+        # Save all credentials (provider + integrations) to disk
+        # Only in interactive mode - single-provider mode saves directly to file
+        api.save_credentials()
 
-    print("\nCredentials saved successfully!")
-    print(f"Stored in: {api.credentials_file}")
-    print("Your keys are encrypted and stored locally only.")
-    print("\nOpen Source Mode: Unlimited access")
+        print("\nCredentials saved successfully!")
+        print(f"Stored in: {api.credentials_file}")
+        print("Your keys are encrypted and stored locally only.")
+        print("\nOpen Source Mode: Unlimited access")
 
 
 @cli.command()

{terradev_cli-5.4.0 → terradev_cli-5.4.2}/terradev_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: terradev-cli
-Version: 5.4.0
+Version: 5.4.2
 Summary: An imperative command-line-interface for AI workload orchestration.
 Author-email: Terradev Team <team@terradev.com>
 Maintainer-email: Terradev Team <team@terradev.com>
@@ -93,7 +93,7 @@ Requires-Dist: redis>=5.0.0; extra == "enterprise"
 Requires-Dist: PyJWT>=2.8.0; extra == "enterprise"
 Dynamic: license-file
 
-# Terradev CLI v5.3.8
+# Terradev CLI v5.4.2
 
 **An imperative command-line-interface for AI workload orchestration.**
 
@@ -111,6 +111,19 @@ Continued focus on lower cost, faster provisioning, and topology-aware execution
 
 Model agnostic. Dataset agnostic. GPU agnostic. Provider agnostic. The only thing Terradev is not agnostic about is correctness: it enforces topology, idempotency, and sequencing.
 
+**NOTES ON 5.4.2**
+
+Fixed **credential type checking bug** in `is_first_time_user()` and onboarding:
+
+- **Bug**: `is_first_time_user()` and onboarding code called `.lower()` on credential values without checking if they were strings. The credentials file uses nested dict format `{"runpod": {"api_key": "..."}}`, so when iterating over values, some are dicts, not strings. This caused `AttributeError: 'dict' object has no attribute 'lower'`.
+
+- **Fix**:
+  - Added `isinstance(value, str)` check before calling `.lower()` in `is_first_time_user()` (line 119)
+  - Added `isinstance(existing_value, str)` check before calling `.lower()` in onboarding (line 764)
+  - The `_provider_creds()` method already correctly handles both flat and nested credential formats
+
+- **Impact**: `terradev quote` and other commands that check credentials now work correctly with the nested dict credential format.
+
 **NOTES ON 5.3.9**
 
 Added **LoRAX (LoRA eXchange) integration** and **HuggingFace PEFT import** for production-grade multi-LoRA inference serving: