fix: ensure base_url defaults correctly when not set

When base_url is not provided via config file or CORTEX_BASE_URL
environment variable, the CLI now properly defaults to
https://api.getcortexapp.com before attempting to strip the value.

Previously, when no config file existed, url would remain None and
cause an AttributeError when url.strip() was called. This affected
GitHub Actions workflows and other automated environments.

Changes:
- Add safety check to set default URL if None before strip operation
- Add test case to verify proper defaulting behavior

Fixes #151

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: jeff-schnitter

cortexapps_cli/cli.py

@@ -23,6 +23,8 @@
 import cortexapps_cli.commands.deploys as deploys
 import cortexapps_cli.commands.discovery_audit as discovery_audit
 import cortexapps_cli.commands.docs as docs
+import cortexapps_cli.commands.entity_relationships as entity_relationships
+import cortexapps_cli.commands.entity_relationship_types as entity_relationship_types
 import cortexapps_cli.commands.entity_types as entity_types
 import cortexapps_cli.commands.gitops_logs as gitops_logs
 import cortexapps_cli.commands.groups as groups
@@ -57,6 +59,8 @@
 app.add_typer(deploys.app, name="deploys")
 app.add_typer(discovery_audit.app, name="discovery-audit")
 app.add_typer(docs.app, name="docs")
+app.add_typer(entity_relationships.app, name="entity-relationships")
+app.add_typer(entity_relationship_types.app, name="entity-relationship-types")
 app.add_typer(entity_types.app, name="entity-types")
 app.add_typer(gitops_logs.app, name="gitops-logs")
 app.add_typer(groups.app, name="groups")
@@ -121,6 +125,10 @@ def global_callback(
             else:
                 url = "https://api.getcortexapp.com"
 
+    # Set default URL if not provided
+    if not url:
+        url = "https://api.getcortexapp.com"
+
     # strip any quotes or spaces from the api_key and url
     api_key = api_key.strip('"\' ')
     url = url.strip('"\' /')

tests/test_config_file.py

@@ -70,3 +70,28 @@ def test_config_file_base_url_env_var(monkeypatch, tmp_path):
     f.write_text(content)
     monkeypatch.setenv("CORTEX_BASE_URL", "https://api.getcortexapp.com")
     cli(["-c", str(f), "-t", "mySection", "entity-types", "list"])
+
+def test_no_base_url_defaults_correctly(monkeypatch, tmp_path):
+    """
+    Test that when base_url is not provided via config file or environment variable,
+    the CLI defaults to https://api.getcortexapp.com without crashing.
+
+    This reproduces the bug from issue #151 where url.strip() was called on None.
+    """
+    # Remove CORTEX_BASE_URL from environment
+    monkeypatch.delenv("CORTEX_BASE_URL", raising=False)
+
+    cortex_api_key = os.getenv('CORTEX_API_KEY')
+
+    # Create config file WITHOUT base_url
+    f = tmp_path / "cortex_config_no_base_url"
+    template = Template("""
+        [default]
+        api_key = "${cortex_api_key}"
+        """)
+    content = template.substitute(cortex_api_key=cortex_api_key)
+    f.write_text(content)
+
+    # This should not crash and should use the default URL
+    response = cli(["-c", str(f), "entity-types", "list"])
+    assert 'definitions' in response, "Should successfully call API with default URL"