chore: removing click.launch

It now uses `webbrowser.open` directly and includes more robust error handling to gracefully manage potential exceptions when launching the browser.

Removes the `click.launch` fallback, as it was causing issues.

Author: aorumbayev

src/algokit/cli/explore.py

@@ -99,10 +99,11 @@ def get_explore_url(network: str) -> str:
 def explore_command(network: str) -> None:
     url = get_explore_url(network)
     logger.info(f"Opening {network} explorer in your default browser")
+    logger.info(f"URL: {url}")
 
-    if is_wsl():
-        import webbrowser
+    import webbrowser
 
+    if is_wsl():
         warning = (
             "Unable to open browser from WSL environment.\n"
             "Ensure 'wslu' is installed: (https://wslutiliti.es/wslu/install.html),\n"
@@ -111,15 +112,13 @@ def explore_command(network: str) -> None:
         try:
             if not webbrowser.open(url):
                 logger.warning(warning)
-        except Exception as e:
-            logger.warning(warning, exc_info=e)
+        except Exception:
+            logger.warning(warning)
     else:
+        # https://github.com/pallets/click/issues/2868 restore click.launch once bug is fixed
         try:
-            click.launch(url)
-        except Exception:
-            # https://github.com/pallets/click/issues/2868
-            try:
-                if not webbrowser.open(url):
-                    logger.warning(f"Failed to open browser. Please open this URL manually: {url}")
-            except Exception:
+            if not webbrowser.open(url):
                 logger.warning(f"Failed to open browser. Please open this URL manually: {url}")
+        except Exception as e:
+            logger.debug("Error opening browser", exc_info=e)
+            logger.warning(f"Failed to open browser. Please open this URL manually: {url}")

tests/explore/test_explore.py

@@ -11,7 +11,7 @@
 
 @pytest.mark.parametrize("command", ["", "localnet", "testnet", "mainnet"])
 def test_explore(command: str, mocker: MockerFixture) -> None:
-    launch_mock = mocker.patch("click.launch")
+    launch_mock = mocker.patch("webbrowser.open")
     result = invoke(f"explore {command}")
 
     assert result.exit_code == 0
@@ -24,7 +24,7 @@ def test_explore(command: str, mocker: MockerFixture) -> None:
 def test_explore_wsl_exception(mocker: MockerFixture, caplog: pytest.LogCaptureFixture) -> None:
     command = "localnet"
     mocker.patch("algokit.cli.explore.is_wsl", return_value=True)
-    mocker.patch("webbrowser.open", side_effect=Exception("Test Exception"))
+    mocker.patch("webbrowser.open", return_value=False)
 
     with caplog.at_level(logging.WARNING):
         result = invoke(f"explore {command}")
@@ -36,7 +36,6 @@ def test_explore_wsl_exception(mocker: MockerFixture, caplog: pytest.LogCaptureF
 def test_explore_webbrowser_exception(mocker: MockerFixture, caplog: pytest.LogCaptureFixture) -> None:
     command = "localnet"
     mocker.patch("algokit.cli.explore.is_wsl", return_value=False)
-    mocker.patch("click.launch", side_effect=Exception("Click Exception"))
     mocker.patch("webbrowser.open", side_effect=Exception("Webbrowser Exception"))
 
     with caplog.at_level(logging.WARNING):

tests/explore/test_explore.test_explore.localnet.approved.txt

@@ -1,4 +1,5 @@
 Opening localnet explorer in your default browser
+URL: https://explore.algokit.io/localnet
 ----
 launch args:
 ----

tests/explore/test_explore.test_explore.mainnet.approved.txt

@@ -1,4 +1,5 @@
 Opening mainnet explorer in your default browser
+URL: https://explore.algokit.io/mainnet
 ----
 launch args:
 ----

tests/explore/test_explore.test_explore.testnet.approved.txt

@@ -1,4 +1,5 @@
 Opening testnet explorer in your default browser
+URL: https://explore.algokit.io/testnet
 ----
 launch args:
 ----