fix: add default console url value (#490)

Author: thiagobomfin-galileo

src/galileo/__future__/configuration.py

@@ -7,6 +7,7 @@
 
 from galileo.__future__.shared.exceptions import ConfigurationError
 from galileo.config import GalileoPythonConfig
+from galileo.constants import DEFAULT_CONSOLE_URL
 from galileo.utils.log_config import enable_console_logging as _enable_console_logging
 from galileo.utils.log_config import get_logger
 
@@ -76,7 +77,10 @@ def parse_log_level(value: str) -> str:
         sensitive=True,
     ),
     ConfigKey(
-        name="console_url", env_var="GALILEO_CONSOLE_URL", description="URL of the Galileo console", required=True
+        name="console_url",
+        env_var="GALILEO_CONSOLE_URL",
+        description="URL of the Galileo console",
+        default=DEFAULT_CONSOLE_URL,
     ),
     ConfigKey(
         name="openai_api_key",
@@ -379,11 +383,6 @@ def connect(cls) -> None:
         """Validate configuration and connect to Galileo."""
         cls._load_env_file()
 
-        if not cls.console_url:
-            raise ConfigurationError(
-                "Galileo console URL is required. Set Configuration.console_url or GALILEO_CONSOLE_URL."
-            )
-
         if not cls.galileo_api_key:
             raise ConfigurationError(
                 "Galileo API key is required. Set Configuration.galileo_api_key or GALILEO_API_KEY."

src/galileo/config.py

@@ -4,13 +4,14 @@
 
 from pydantic_core import Url
 
+from galileo.constants import DEFAULT_CONSOLE_URL
 from galileo_core.schemas.base_config import GalileoConfig
 
 
 class GalileoPythonConfig(GalileoConfig):
     # Config file for this project.
     config_filename: str = "galileo-python-config.json"
-    console_url: Url = "https://app.galileo.ai"
+    console_url: Url = DEFAULT_CONSOLE_URL
 
     _instance: ClassVar[Optional["GalileoPythonConfig"]] = None
 

src/galileo/constants/__init__.py

@@ -5,6 +5,7 @@
 DEFAULT_MODE = "batch"
 
 DEFAULT_API_URL = "https://api.galileo.ai/"
+DEFAULT_CONSOLE_URL = "https://app.galileo.ai/"
 
 # HTTP header prefix for all Galileo headers
 GALILEO_HEADER_PREFIX = "X-Galileo"
@@ -14,6 +15,7 @@
 
 __all__ = (
     "DEFAULT_API_URL",
+    "DEFAULT_CONSOLE_URL",
     "DEFAULT_LOG_STREAM_NAME",
     "DEFAULT_MODE",
     "DEFAULT_PROJECT_NAME",

src/galileo/experiments.py

@@ -144,7 +144,7 @@ def run(
 
         _logger.debug(f"job: {job}")
 
-        link = f"{self.config.console_url}/project/{project_obj.id}/experiments/{experiment_obj.id}"
+        link = f"{str(self.config.console_url).rstrip('/')}/project/{project_obj.id}/experiments/{experiment_obj.id}"
         message = f"Experiment {experiment_obj.name} has started and is currently processing. Results will be available at {link}"
         _logger.info(message)
 
@@ -208,7 +208,7 @@ def logged_process_func(row: DatasetRecord) -> Callable:
 
         _logger.info(f" {len(results)} rows processed for experiment {experiment_obj.name}.")
 
-        link = f"{self.config.console_url}/project/{project_obj.id}/experiments/{experiment_obj.id}"
+        link = f"{str(self.config.console_url).rstrip('/')}/project/{project_obj.id}/experiments/{experiment_obj.id}"
         message = f"Experiment {experiment_obj.name} has completed and results are available at {link}"
         _logger.info(message)
 

tests/future/test_configuration.py

@@ -281,25 +281,29 @@ def test_connect_succeeds_with_valid_configuration(
         assert "Validating Galileo configuration" in logs
         assert "Successfully connected to Galileo" in logs
 
-    def test_connect_fails_without_console_url(
-        self, monkeypatch: pytest.MonkeyPatch, reset_configuration: None, capture_logs: tuple[logging.Logger, StringIO]
+    def test_connect_passes_without_console_url(
+        self,
+        mock_api_endpoints: MagicMock,
+        monkeypatch: pytest.MonkeyPatch,
+        reset_configuration: None,
+        capture_logs: tuple[logging.Logger, StringIO],
     ) -> None:
-        """Test connect() raises ConfigurationError when console URL is missing."""
+        """Test successful connection with valid API key and should default to default console URL."""
         _, log_stream = capture_logs
 
-        # Set only API key
+        # Set valid configuration
         monkeypatch.setenv("GALILEO_API_KEY", "valid-key")
 
-        # Should raise ConfigurationError
-        with pytest.raises(ConfigurationError) as exc_info:
-            Configuration.connect()
+        # Connect should succeed without raising
+        Configuration.connect()
 
-        # Verify error message provides helpful guidance
-        assert "console url is required" in str(exc_info.value).lower()
-        assert "configuration.console_url" in str(exc_info.value).lower()
+        # Verify logging occurred (no print statements)
+        logs = log_stream.getvalue()
+        assert "Validating Galileo configuration" in logs
+        assert "Successfully connected to Galileo" in logs
 
-        # Note: Early validation failures don't produce logs since they occur
-        # before the logger.info call in connect()
+        # Verify console URL is set to default
+        assert Configuration.console_url == "https://app.galileo.ai/"
 
     def test_connect_fails_without_api_key(
         self, monkeypatch: pytest.MonkeyPatch, reset_configuration: None, capture_logs: tuple[logging.Logger, StringIO]

tests/test_experiments.py

@@ -572,6 +572,43 @@ def test_run_experiment_without_metrics(
             prompt_settings=ANY,
         )
 
+    @pytest.mark.parametrize("console_url", ["http://localtest:8088", "http://localtest:8088/"])
+    @travel(datetime(2012, 1, 1), tick=False)
+    @patch.object(galileo.datasets.Datasets, "get")
+    @patch.object(galileo.jobs.Jobs, "create")
+    @patch.object(galileo.experiments.Experiments, "create", return_value=experiment_response())
+    @patch.object(galileo.experiments.Experiments, "get", return_value=experiment_response())
+    @patch.object(galileo.experiments.Projects, "get_with_env_fallbacks", return_value=project())
+    def test_run_experiment_link_no_double_slash(
+        self,
+        mock_get_project: Mock,
+        mock_get_experiment: Mock,
+        mock_create_experiment: Mock,
+        mock_create_job: Mock,
+        mock_get_dataset: Mock,
+        console_url: str,
+        dataset_content: DatasetContent,
+    ) -> None:
+        # Given: a console_url with or without a trailing slash
+        mock_create_job.return_value = MagicMock()
+        mock_config = MagicMock()
+        mock_config.console_url = console_url
+
+        # When: running an experiment
+        with patch("galileo.experiments.GalileoPythonConfig.get", return_value=mock_config):
+            result = run_experiment(
+                "test_experiment",
+                project="awesome-new-project",
+                dataset_id=str(UUID(int=0)),
+                prompt_template=prompt_template(),
+            )
+
+        # Then: the link does not contain double slashes after the protocol
+        assert result is not None
+        link = result["link"]
+        assert "//project" not in link
+        assert f"/project/{project().id}/experiments/{experiment_response().id}" in link
+
     @travel(datetime(2012, 1, 1), tick=False)
     @patch.object(galileo.datasets.Datasets, "get")
     @patch.object(galileo.jobs.Jobs, "create")