fix(ci): resolve 3 remaining test failures from the wiki autonomy turn

After the v3.17.0 prep the test suite still had three classes of
failure on Linux CI. Each one is a real test/policy mismatch, not a
flake:

# tests_py/handlers/test_seed_project.py (3 tests)

The handler refuses to operate on transient roots
(``pytest-of-`` / ``/private/var/folders/`` / ``/var/folders/`` /
``.claude/worktrees/``) to keep test runs and worktree creation from
polluting the wiki. The test fixture uses ``tmp_path`` which IS a
transient root — so the handler returns ``seeded: False`` before
the codepath under test ever runs.

Added an autouse fixture at the module top that stubs
``is_transient_seed_root`` to return ``False`` for every test in the
module. The tests are about the per-domain purge contract (issue
#16), not the transient-root gate; the stub keeps them focused.

# tests_py/handlers/test_open_visualization.py
# (test_opens_browser_at_legacy_url_when_extras_missing)

The test expected the handler to open a legacy URL when
``_prepare_layout`` returns ``igraph_missing``. That fallback path
no longer exists — the user direction 2026-05-17 was that the graph
build must NEVER block the MCP tool launch OR the interface load,
so ``_prepare_layout`` is no longer called from ``handler()``. The
handler now always opens ``?viz=tilemap``.

Renamed the test to ``test_opens_browser_at_tilemap_url_unconditionally``
and rewrote it to assert the current behaviour: tilemap URL always
opens, regardless of any (irrelevant) layout state. Kept the
docstring explaining the history so the next reader knows why.

# tests_py/core/test_wiki_sync_routing.py
# (test_file_documentation_does_not_route_to_notes)

Same policy clash as the two ``test_wiki_classifier.py`` tests I
fixed in an earlier commit: ``codebase`` / ``code-reference`` tagged
content used to route to ``reference/``; the 2026-05-17 policy
moved those tags into ``_AUDIT_TAGS`` (Gate -1 rejection) so the
wiki layer never re-accumulates per-file extractor dumps. The
autonomous worker now produces curated structural pages instead.

Renamed the test to ``test_file_documentation_is_rejected_from_wiki``
and assert ``build_from_memory`` returns None for this tag set.

Lint + format clean; 18-test subset (the three previously-failing
suites) passes locally on Python 3.10.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: cdeust

tests_py/core/test_wiki_sync_routing.py

@@ -165,14 +165,15 @@ def test_each_new_page_gets_a_distinct_id() -> None:
     assert id_a != id_b
 
 
-def test_file_documentation_does_not_route_to_notes() -> None:
-    """Task #8 fix: pages tagged as code references must not land in notes/.
-
-    Before ADR-2244, file documentation produced by codebase_analyze
-    ended up at notes/<domain>/<id>-file-*.md because the old
-    _KIND_TO_DIR had no 'file' mapping. The classifier now marks the
-    provenance as auto-generated and routes to reference/ via the
-    modern kind.
+def test_file_documentation_is_rejected_from_wiki() -> None:
+    """Policy 2026-05-17 (superseded ADR-2244 Phase 6 admission):
+    ``codebase`` / ``code-reference`` tags mark per-file extractor
+    output that bloats the wiki — they stay in PG memory only.
+    Coverage of the codebase flows through the structural scope
+    pages (architecture / services / api / data-flow / code-walkthrough
+    per project) the autonomous worker authors — not via per-file
+    auto-generated dumps. ``build_from_memory`` returns None so no
+    wiki page is written.
     """
     content = (
         "Decision: this module parses a single source file via tree-sitter "
@@ -186,10 +187,8 @@ def test_file_documentation_does_not_route_to_notes() -> None:
         tags=["code-reference", "codebase"],
         domain="cortex",
     )
-    assert result is not None
-    rel, md = result
-    assert not rel.startswith("notes/"), (
-        f"file-documentation still routing to notes/: {rel!r}"
+    assert result is None, (
+        "codebase/code-reference tagged content must be rejected from the "
+        "wiki — it lives in PG memory only; the autonomous worker "
+        "produces curated structural pages instead"
     )
-    fm = _parse_frontmatter(md)
-    assert fm["provenance"] == "auto-generated"

tests_py/handlers/test_open_visualization.py

@@ -94,8 +94,15 @@ def test_opens_browser_at_tilemap_url(self):
             asyncio.run(handler({}))
         mock_open.assert_called_once_with("http://localhost:5555/?viz=tilemap")
 
-    def test_opens_browser_at_legacy_url_when_extras_missing(self):
-        """``viz-tile`` extras unavailable falls back to the legacy URL."""
+    def test_opens_browser_at_tilemap_url_unconditionally(self):
+        """Handler always opens the tilemap URL — graph build happens on
+        demand via the UI's Graph button, not on launch.
+
+        Previously a fallback path opened the legacy URL when
+        ``_prepare_layout`` reported ``igraph_missing``; that branch
+        was removed when the on-launch layout precomputation was
+        dropped (2026-05). The handler now returns immediately after
+        opening ``?viz=tilemap`` regardless of any layout state."""
         with (
             patch(
                 "mcp_server.handlers.open_visualization.launch_server",
@@ -104,11 +111,7 @@ def test_opens_browser_at_legacy_url_when_extras_missing(self):
             patch(
                 "mcp_server.handlers.open_visualization.open_in_browser",
             ) as mock_open,
-            patch(
-                "mcp_server.handlers.open_visualization._prepare_layout",
-                return_value={"status": "error", "reason": "igraph_missing"},
-            ),
         ):
             result = asyncio.run(handler({}))
-        mock_open.assert_called_once_with("http://localhost:5555")
-        assert "viz-tile" in result["message"]
+        mock_open.assert_called_once_with("http://localhost:5555/?viz=tilemap")
+        assert "tilemap" in result["message"]

tests_py/handlers/test_seed_project.py

@@ -13,10 +13,37 @@
 import asyncio
 from pathlib import Path
 
+import pytest
+
 from mcp_server.handlers.seed_project import handler
 from mcp_server.infrastructure.memory_store import MemoryStore
 
 
+@pytest.fixture(autouse=True)
+def _allow_transient_seed_roots(monkeypatch):
+    """Disable the transient-root guard for this module's tests.
+
+    The handler refuses to operate on paths matching ``pytest-of-`` /
+    ``/private/var/folders/`` / ``/var/folders/`` so test runs and
+    worktree creation never pollute the wiki. But this module's tests
+    use ``tmp_path`` (which IS a transient root) by design — they're
+    proving the per-domain purge contract, not validating the
+    transient-root gate. Stub the predicate to ``False`` for the
+    duration of each test so the handler proceeds into the codepath
+    under test.
+    """
+    monkeypatch.setattr(
+        "mcp_server.handlers.seed_project_constants.is_transient_seed_root",
+        lambda _path: False,
+    )
+    # Also patch the import name as seen by ``seed_project`` itself,
+    # since the handler imports the predicate at function-call time.
+    import mcp_server.handlers.seed_project as sp
+
+    if hasattr(sp, "is_transient_seed_root"):
+        monkeypatch.setattr(sp, "is_transient_seed_root", lambda _path: False)
+
+
 def _make_repo(tmp_path: Path, name: str) -> Path:
     """Build a minimal repo skeleton sufficient for seed_project to find content."""
     root = tmp_path / name