terradev-cli 5.4.4__tar.gz → 5.4.6__tar.gz

{terradev_cli-5.4.4 → terradev_cli-5.4.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: terradev-cli
-Version: 5.4.4
+Version: 5.4.6
 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.4.4
+# Terradev CLI v5.4.6
 
 **An imperative command-line-interface for AI workload orchestration.**
 
@@ -111,6 +111,28 @@ 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.6**
+
+Added **debug logging** to `_get_provider_quotes()` for troubleshooting credential and API issues:
+
+- **Purpose**: Added stderr debug output to log credentials being passed to each provider and the number of quotes returned. This helps diagnose why `terradev quote` returns empty results.
+
+- **Changes**:
+  - Added `print(f"[DEBUG] {provider_name} creds: {creds}", file=sys.stderr)` to log credentials
+  - Added `print(f"[DEBUG] {provider_name} returned {len(raw_quotes) if raw_quotes else 0} quotes", file=sys.stderr)` to log quote counts
+
+- **Impact**: Debug output appears in stderr when running `terradev quote`, helping identify credential issues, API failures, or empty responses from providers.
+
+**NOTES ON 5.4.5**
+
+Fixed **nested credential detection bug** in `is_first_time_user()`:
+
+- **Bug**: The `is_first_time_user()` function only checked top-level credential values, but credentials now use nested dict format `{"runpod": {"api_key": "rpa_..."}}` from `configure --provider`. When a value was a dict, it was skipped entirely, so the function always returned `True` (first time user) even when real credentials existed.
+
+- **Fix**: Added recursion into dict values to check nested credentials. The loop now handles both flat format `{"runpod_api_key": "rpa_..."}` and nested format `{"runpod": {"api_key": "rpa_..."}}`.
+
+- **Impact**: Users with nested credential format are now correctly identified as having real credentials, preventing unwanted onboarding prompts and enabling proper credential validation.
+
 **NOTES ON 5.4.4**
 
 Fixed **case-sensitivity bug** in placeholder detection in `_provider_creds()`:

{terradev_cli-5.4.4 → terradev_cli-5.4.6}/README.md

@@ -1,4 +1,4 @@
-# Terradev CLI v5.4.4
+# Terradev CLI v5.4.6
 
 **An imperative command-line-interface for AI workload orchestration.**
 
@@ -16,6 +16,28 @@ 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.6**
+
+Added **debug logging** to `_get_provider_quotes()` for troubleshooting credential and API issues:
+
+- **Purpose**: Added stderr debug output to log credentials being passed to each provider and the number of quotes returned. This helps diagnose why `terradev quote` returns empty results.
+
+- **Changes**:
+  - Added `print(f"[DEBUG] {provider_name} creds: {creds}", file=sys.stderr)` to log credentials
+  - Added `print(f"[DEBUG] {provider_name} returned {len(raw_quotes) if raw_quotes else 0} quotes", file=sys.stderr)` to log quote counts
+
+- **Impact**: Debug output appears in stderr when running `terradev quote`, helping identify credential issues, API failures, or empty responses from providers.
+
+**NOTES ON 5.4.5**
+
+Fixed **nested credential detection bug** in `is_first_time_user()`:
+
+- **Bug**: The `is_first_time_user()` function only checked top-level credential values, but credentials now use nested dict format `{"runpod": {"api_key": "rpa_..."}}` from `configure --provider`. When a value was a dict, it was skipped entirely, so the function always returned `True` (first time user) even when real credentials existed.
+
+- **Fix**: Added recursion into dict values to check nested credentials. The loop now handles both flat format `{"runpod_api_key": "rpa_..."}` and nested format `{"runpod": {"api_key": "rpa_..."}}`.
+
+- **Impact**: Users with nested credential format are now correctly identified as having real credentials, preventing unwanted onboarding prompts and enabling proper credential validation.
+
 **NOTES ON 5.4.4**
 
 Fixed **case-sensitivity bug** in placeholder detection in `_provider_creds()`:

{terradev_cli-5.4.4 → terradev_cli-5.4.6}/pyproject.toml

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

{terradev_cli-5.4.4 → terradev_cli-5.4.6}/terradev_cli/cli.py

@@ -116,10 +116,18 @@ 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 isinstance(value, str) and not any(
+            if isinstance(value, dict):
+                # Check nested credentials (from configure --provider)
+                for nested_key, nested_value in value.items():
+                    if nested_value and isinstance(nested_value, str) and not any(
+                        pattern in nested_value.lower()
+                        for pattern in placeholder_patterns
+                    ):
+                        return False  # Found a real nested credential
+            elif value and isinstance(value, str) and not any(
                 pattern in value.lower() for pattern in placeholder_patterns
             ):
-                return False  # Found a real credential
+                return False  # Found a real flat credential
 
         return True  # All credentials appear to be placeholders
 
@@ -479,9 +487,15 @@ class TerradevAPI:
 
             factory = ProviderFactory()
             creds = self._provider_creds(provider_name)
+            
+            # Debug: log credentials for troubleshooting
+            import sys
+            print(f"[DEBUG] {provider_name} creds: {creds}", file=sys.stderr)
+            
             provider = factory.create_provider(provider_name, creds)
             try:
                 raw_quotes = await provider.get_instance_quotes(gpu_type)
+                print(f"[DEBUG] {provider_name} returned {len(raw_quotes) if raw_quotes else 0} quotes", file=sys.stderr)
             finally:
                 # Close aiohttp session to avoid ResourceWarning
                 if provider.session:

{terradev_cli-5.4.4 → terradev_cli-5.4.6}/terradev_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: terradev-cli
-Version: 5.4.4
+Version: 5.4.6
 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.4.4
+# Terradev CLI v5.4.6
 
 **An imperative command-line-interface for AI workload orchestration.**
 
@@ -111,6 +111,28 @@ 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.6**
+
+Added **debug logging** to `_get_provider_quotes()` for troubleshooting credential and API issues:
+
+- **Purpose**: Added stderr debug output to log credentials being passed to each provider and the number of quotes returned. This helps diagnose why `terradev quote` returns empty results.
+
+- **Changes**:
+  - Added `print(f"[DEBUG] {provider_name} creds: {creds}", file=sys.stderr)` to log credentials
+  - Added `print(f"[DEBUG] {provider_name} returned {len(raw_quotes) if raw_quotes else 0} quotes", file=sys.stderr)` to log quote counts
+
+- **Impact**: Debug output appears in stderr when running `terradev quote`, helping identify credential issues, API failures, or empty responses from providers.
+
+**NOTES ON 5.4.5**
+
+Fixed **nested credential detection bug** in `is_first_time_user()`:
+
+- **Bug**: The `is_first_time_user()` function only checked top-level credential values, but credentials now use nested dict format `{"runpod": {"api_key": "rpa_..."}}` from `configure --provider`. When a value was a dict, it was skipped entirely, so the function always returned `True` (first time user) even when real credentials existed.
+
+- **Fix**: Added recursion into dict values to check nested credentials. The loop now handles both flat format `{"runpod_api_key": "rpa_..."}` and nested format `{"runpod": {"api_key": "rpa_..."}}`.
+
+- **Impact**: Users with nested credential format are now correctly identified as having real credentials, preventing unwanted onboarding prompts and enabling proper credential validation.
+
 **NOTES ON 5.4.4**
 
 Fixed **case-sensitivity bug** in placeholder detection in `_provider_creds()`: