Fix logo. Fix driver import fail formatting

Author: Maxteabag

.gitignore

@@ -54,5 +54,6 @@ assets/*
 !assets/favorites/
 assets/favorites/*
 !assets/favorites/logo.png
+!assets/favorites/logo_sqlit.png
 .worktrees/
 .obsidian/

assets/favorites/logo_sqlit.png

@@ -3,13 +3,12 @@
 from __future__ import annotations
 
 import importlib
+import os
 from abc import ABC, abstractmethod
 from dataclasses import dataclass
 from pathlib import Path
 from typing import TYPE_CHECKING, Any
 
-from rich.markup import escape
-
 if TYPE_CHECKING:
     from ...config import ConnectionConfig
 
@@ -91,6 +90,18 @@ def import_driver_module(
     package_name: str | None,
 ) -> Any:
     """Import a driver module, raising MissingDriverError with detail if it fails."""
+    # Mock flag to force driver import error for testing
+    if os.environ.get("SQLIT_MOCK_DRIVER_ERROR") and extra_name and package_name:
+        from ...db.exceptions import MissingDriverError
+
+        raise MissingDriverError(
+            driver_name,
+            extra_name,
+            package_name,
+            module_name=module_name,
+            import_error=f"No module named '{module_name}'",
+        )
+
     if not extra_name or not package_name:
         return importlib.import_module(module_name)
 
@@ -1039,9 +1050,7 @@ def _create_driver_import_error_hint(driver_name: str, extra_name: str, package_
     from ...install_strategy import detect_strategy
 
     strategy = detect_strategy(extra_name=extra_name, package_name=package_name)
-    instructions = escape(strategy.manual_instructions)
     return (
         f"{driver_name} driver not found.\n\n"
-        f"To connect to {driver_name}, run:\n\n"
-        f"[bold]{instructions}[/bold]\n"
+        f"{strategy.manual_instructions}\n"
     )

sqlit/db/adapters/base.py

@@ -37,7 +37,7 @@ def _is_pipx() -> bool:
     pipx_override = os.environ.get("SQLIT_MOCK_PIPX", "").strip().lower()
     if pipx_override in {"1", "true", "yes", "pipx"}:
         return True
-    if pipx_override in {"0", "false", "no", "pip", "unknown"}:
+    if pipx_override in {"0", "false", "no", "pip", "unknown", "no-pip"}:
         return False
 
     exe = sys.executable.lower()
@@ -70,6 +70,8 @@ def _pep668_externally_managed() -> bool:
 
 
 def _pip_available() -> bool:
+    if os.environ.get("SQLIT_MOCK_PIPX", "").strip().lower() == "no-pip":
+        return False
     return importlib.util.find_spec("pip") is not None
 
 
@@ -154,6 +156,18 @@ def _format_manual_instructions(package_name: str, reason: str) -> str:
 
 def detect_strategy(*, extra_name: str, package_name: str) -> InstallStrategy:
     """Detect the best installation strategy for optional driver dependencies."""
+    # When mocking driver errors, also force the no-pip path to show full instructions
+    if os.environ.get("SQLIT_MOCK_DRIVER_ERROR"):
+        return InstallStrategy(
+            kind="no-pip",
+            can_auto_install=False,
+            manual_instructions=_format_manual_instructions(
+                package_name,
+                "pip is not available for this Python interpreter.",
+            ),
+            reason_unavailable="pip is not available.",
+        )
+
     if _is_unknown_install():
         return InstallStrategy(
             kind="unknown",

sqlit/install_strategy.py

@@ -4,7 +4,6 @@
 
 from collections.abc import Callable
 
-from rich.markup import escape
 from textual.app import ComposeResult
 from textual.binding import Binding
 from textual.containers import VerticalScroll
@@ -94,7 +93,7 @@ def compose(self) -> ComposeResult:
             )
 
             with VerticalScroll(id="package-scroll"):
-                yield Static(escape(self._instructions_text.strip()), id="package-script")
+                yield Static(self._instructions_text.strip(), id="package-script")
 
     def on_mount(self) -> None:
         self.query_one("#package-scroll", VerticalScroll).focus()