fix(ci): install [codebase] extra + skip tree-sitter tests without it

CI failure on 197155e+dc129b9: regex fallback path in
codebase_parser.parse_file produces basename-only symbol names ("bar"
not "Foo.bar") because tree-sitter isn't installed in CI. The L6-ring
feature legitimately requires tree-sitter; the fallback is graceful
but cannot emit caller-qualified chains.

Two-part fix:

1. .github/workflows/ci.yml — install `.[dev,postgresql,codebase]`
   so CI actually exercises the feature with tree-sitter.

2. tests_py/infrastructure/test_workflow_graph_source_native_ast.py
   — mark every test that depends on dotted qnames / MEMBER_OF /
   CALLS with @requires_tree_sitter (`pytest.mark.skipif` on
   ast_parser.is_available()). Local-dev configs without the
   [codebase] extra now skip cleanly instead of asserting on
   unachievable shapes.

Tests that don't require tree-sitter (empty list, non-source skip,
file cap, unresolved-import dropped) stay unmarked and continue to
run in both configs.

Local: 16 tests pass as before. CI should now pass the
codebase-extra-backed tests and skip nothing.

Author: cdeust

.github/workflows/ci.yml

@@ -67,7 +67,7 @@ jobs:
           key: ${{ runner.os }}-hf-all-MiniLM-L6-v2
 
       - name: Install dependencies
-        run: pip install -e ".[dev,postgresql]"
+        run: pip install -e ".[dev,postgresql,codebase]"
 
       - name: Pre-download embedding model
         run: python -c "from sentence_transformers import SentenceTransformer; SentenceTransformer('all-MiniLM-L6-v2', device='cpu')"

tests_py/infrastructure/test_workflow_graph_source_native_ast.py

@@ -12,10 +12,23 @@
 
 import pytest
 
+from mcp_server.core.ast_parser import is_available as _ast_available
 from mcp_server.infrastructure.workflow_graph_source_native_ast import (
     WorkflowGraphNativeASTSource,
 )
 
+# Tree-sitter is an optional extra (``pip install -e ".[codebase]"``).
+# Without it, ``parse_file_ast`` falls back to the regex path in
+# ``codebase_parser.parse_file``, which produces basename-only symbol
+# names (e.g. ``"bar"``) and does not populate ``calls_per_function``.
+# The feature this module tests — dotted qnames, MEMBER_OF, and CALLS
+# chains — requires tree-sitter. Skip cleanly when it isn't installed
+# so CI configs that omit the [codebase] extra don't fail spuriously.
+requires_tree_sitter = pytest.mark.skipif(
+    not _ast_available(),
+    reason="tree-sitter not installed — install the [codebase] extra",
+)
+
 
 @pytest.fixture
 def source() -> WorkflowGraphNativeASTSource:
@@ -44,6 +57,7 @@ def test_empty_file_list_returns_empty(self, source):
     def test_unreadable_paths_are_skipped(self, source):
         assert source.load_symbols(["/no/such/file.py"]) == []
 
+    @requires_tree_sitter
     def test_python_class_and_method_emit_symbol_rows(self, tmp_path, source):
         path = _write(
             tmp_path,
@@ -77,6 +91,7 @@ def test_non_source_files_skipped(self, tmp_path, source):
 
 
 class TestLoadASTEdges:
+    @requires_tree_sitter
     def test_member_of_emitted_for_method(self, tmp_path, source):
         path = _write(
             tmp_path,
@@ -96,6 +111,7 @@ def test_member_of_emitted_for_method(self, tmp_path, source):
             assert e["confidence"] == 1.0
             assert "native-ast" in e["reason"]
 
+    @requires_tree_sitter
     def test_imports_resolve_to_sibling_symbol(self, tmp_path, source):
         _write(
             tmp_path,
@@ -137,8 +153,10 @@ def test_empty_file_list_returns_empty(self, source):
 class TestCallEdges:
     """Caller-qualified CALLS: ``src_name`` is the enclosing method's
     qualified name, so the L6 ring can render the full method-to-method
-    chain inside and across files."""
+    chain inside and across files. All tests here require tree-sitter
+    (the regex fallback cannot produce caller-qualified names)."""
 
+    @requires_tree_sitter
     def test_intra_file_method_to_method_call(self, tmp_path, source):
         path = _write(
             tmp_path,
@@ -168,6 +186,7 @@ def test_intra_file_method_to_method_call(self, tmp_path, source):
         assert e["confidence"] == 1.0
         assert "native-ast:call" in e["reason"]
 
+    @requires_tree_sitter
     def test_cross_file_call_resolved(self, tmp_path, source):
         _write(
             tmp_path,
@@ -208,6 +227,7 @@ def test_unresolved_callee_dropped(self, tmp_path, source):
         # "os", "path", "join" are all unknown — no edges emitted.
         assert not calls, f"expected no call edges, got {calls}"
 
+    @requires_tree_sitter
     def test_dedup_within_caller(self, tmp_path, source):
         """A caller that invokes the same callee N times yields one edge."""
         path = _write(
@@ -240,6 +260,7 @@ class TestEndToEndContract:
     ``symbol_id`` and returns early at the ``dst_id not in b._nodes``
     guard."""
 
+    @requires_tree_sitter
     def test_calls_edge_lands_in_builder_output(self, tmp_path, source):
         from mcp_server.core.workflow_graph_builder import (
             WorkflowGraphBuilder,
@@ -293,6 +314,7 @@ def test_calls_edge_lands_in_builder_output(self, tmp_path, source):
         )
         assert len(calls) == 1
 
+    @requires_tree_sitter
     def test_cross_file_calls_edge_lands(self, tmp_path, source):
         from mcp_server.core.workflow_graph_builder import (
             WorkflowGraphBuilder,