fix: CI test failures and Sentry-flagged engine bootstrap crash

- Add duckdb to dev dependencies so CI installs it (fixes 19 test failures)
- Add pytest skip markers for tests requiring sqlguard (proprietary Rust
  extension not available in CI) and boto3 (optional warehouse dependency)
- Wrap ensureEngine() in try-catch in resolvePython() to show a clear error
  message instead of crashing the CLI on bootstrap failure (Sentry review)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: anandgupta42

packages/altimate-code/src/bridge/client.ts

@@ -58,7 +58,14 @@ export namespace Bridge {
 
     // 4. Production: uv-managed engine
     const { ensureEngine, enginePythonPath } = await import("./engine")
-    await ensureEngine()
+    try {
+      await ensureEngine()
+    } catch (err) {
+      throw new Error(
+        `Failed to bootstrap Python engine: ${err instanceof Error ? err.message : String(err)}. ` +
+          `Set ALTIMATE_CLI_PYTHON to a Python 3.10+ interpreter to skip automatic bootstrap.`,
+      )
+    }
     return enginePythonPath()
   }
 

packages/altimate-engine/pyproject.toml

@@ -52,6 +52,7 @@ tunneling = ["sshtunnel>=0.4", "paramiko>=3.0"]
 dev = [
     "pytest>=7.0",
     "ruff>=0.4",
+    "duckdb>=0.9",
 ]
 
 [tool.hatch.build.targets.wheel]

packages/altimate-engine/tests/test_cost_gate.py

@@ -7,6 +7,14 @@
 
 from altimate_engine.ci.cost_gate import scan_files, _has_jinja, _split_statements
 
+try:
+    import sqlguard
+    _HAS_SQLGUARD = True
+except ImportError:
+    _HAS_SQLGUARD = False
+
+_needs_sqlguard = pytest.mark.skipif(not _HAS_SQLGUARD, reason="sqlguard not installed")
+
 
 # ---------------------------------------------------------------------------
 # Helpers
@@ -73,6 +81,7 @@ def test_clean_file_passes(self):
         finally:
             os.unlink(path)
 
+    @_needs_sqlguard
     def test_cartesian_product_has_warnings(self):
         """CROSS JOIN produces lint warnings (SELECT *, missing aliases, no LIMIT)."""
         path = _write_temp_sql("SELECT * FROM a CROSS JOIN b")
@@ -133,6 +142,7 @@ def test_multiple_files_mixed(self):
             os.unlink(clean_path)
             os.unlink(warn_path)
 
+    @_needs_sqlguard
     def test_multiple_statements_in_file(self):
         """Multiple statements: lint runs on each; warnings don't fail the gate."""
         path = _write_temp_sql("SELECT 1; SELECT * FROM a CROSS JOIN b;")

packages/altimate-engine/tests/test_enterprise_connectors.py

@@ -6,6 +6,12 @@
 
 from altimate_engine.connections import ConnectionRegistry
 
+try:
+    import boto3
+    _HAS_BOTO3 = True
+except ImportError:
+    _HAS_BOTO3 = False
+
 
 @pytest.fixture(autouse=True)
 def reset_registry():
@@ -37,6 +43,7 @@ def test_inherits_postgres(self):
 
         assert issubclass(RedshiftConnector, PostgresConnector)
 
+    @pytest.mark.skipif(not _HAS_BOTO3, reason="boto3 not installed")
     def test_iam_role_requires_cluster_id(self):
         from altimate_engine.connectors.redshift import RedshiftConnector
 

packages/altimate-engine/tests/test_feedback_store.py

@@ -7,6 +7,14 @@
 
 from altimate_engine.sql.feedback_store import FeedbackStore, _regex_strip_literals
 
+try:
+    import sqlguard
+    _HAS_SQLGUARD = True
+except ImportError:
+    _HAS_SQLGUARD = False
+
+_needs_sqlguard = pytest.mark.skipif(not _HAS_SQLGUARD, reason="sqlguard not installed")
+
 
 @pytest.fixture
 def store(tmp_path):
@@ -325,6 +333,7 @@ def test_tier4_with_window_functions(self, store):
         windowed = store.predict("SELECT id, ROW_NUMBER() OVER (ORDER BY id) FROM t")
         assert windowed["predicted_bytes"] >= base["predicted_bytes"]
 
+    @_needs_sqlguard
     def test_tier4_cross_join_high_complexity(self, store):
         """CROSS JOINs should significantly increase the estimate."""
         simple = store.predict("SELECT id FROM t")

packages/altimate-engine/tests/test_server.py

@@ -5,6 +5,14 @@
 from altimate_engine.server import dispatch, handle_line
 from altimate_engine.models import JsonRpcRequest
 
+try:
+    import sqlguard
+    _HAS_SQLGUARD = True
+except ImportError:
+    _HAS_SQLGUARD = False
+
+_needs_sqlguard = pytest.mark.skipif(not _HAS_SQLGUARD, reason="sqlguard not installed")
+
 
 class TestDispatch:
     def test_ping(self):
@@ -14,6 +22,7 @@ def test_ping(self):
         assert "version" in response.result
         assert response.error is None
 
+    @_needs_sqlguard
     def test_sql_analyze(self):
         request = JsonRpcRequest(
             method="sql.analyze",
@@ -83,6 +92,7 @@ def test_warehouse_list(self):
         assert response.error is None
         assert "warehouses" in response.result
 
+    @_needs_sqlguard
     def test_sql_format(self):
         request = JsonRpcRequest(
             method="sql.format",