fix:tab completion error with invalid tenant

Author: Samuel Wan

src/pybritive/completers/__init__.py

@@ -1,2 +1,22 @@
 # register the completion scripts since they are not required to be registered anywhere else
+import os
+from typing import Optional
+
 from . import bash_gte_42, powershell_completion  # noqa: F401
+
+
+def get_tenant_for_api_completion() -> Optional[str]:
+    """Resolve the tenant for API completion from environment or config."""
+    tenant = os.getenv('BRITIVE_TENANT')
+    if not tenant:
+        from pybritive.helpers.config import ConfigManager  # noqa: PLC0415
+
+        config = ConfigManager(None)
+        config.load()
+        if config.default_tenant and config.default_tenant in config.tenants:
+            tenant = config.tenants[config.default_tenant].get('name')
+    if not tenant:
+        return None
+    if not tenant.endswith('.britive-app.com'):
+        tenant = f'{tenant.strip()}.britive-app.com'
+    return tenant

src/pybritive/completers/api.py

@@ -4,13 +4,16 @@
 from britive.britive import Britive
 from click.shell_completion import CompletionItem
 
+from pybritive.completers import get_tenant_for_api_completion
+
 
 def api_completer(ctx, param, incomplete):
     # create an instance of the Britive class, so we can inspect it
-    # this doesn't need to actually connect to any tenant, and we couldn't even if we
-    # wanted to since when performing shell completion we have no tenant/token
-    # context in order to properly establish a connection.
-    b = Britive(token='ignore', tenant='britive.com', query_features=False)
+    # using the user's configured tenant to avoid DNS resolution issues
+    tenant = get_tenant_for_api_completion()
+    if not tenant:
+        return []
+    b = Britive(token='ignore', tenant=tenant, query_features=False)
 
     # parse the incomplete command, so we can determine where in the "hierarchy" we are
     # and what commands/subcommands the user should be presented with

src/pybritive/completers/api_command.py

@@ -4,14 +4,17 @@
 
 from britive.britive import Britive
 
+from pybritive.completers import get_tenant_for_api_completion
+
 
 def get_dynamic_method_parameters(method):
     try:
         # create an instance of the Britive class, so we can inspect it
-        # this doesn't need to actually connect to any tenant, and we couldn't even if we
-        # wanted to since when performing shell completion we have no tenant/token
-        # context in order to properly establish a connection.
-        b = Britive(token='ignore', tenant='britive.com', query_features=False)
+        # using the user's configured tenant to avoid DNS resolution issues
+        tenant = get_tenant_for_api_completion()
+        if not tenant:
+            return []
+        b = Britive(token='ignore', tenant=tenant, query_features=False)
 
         # parse the method, so we can determine where in the "hierarchy" we are
         # and what commands/subcommands the user should be presented with