fix: skip valkey-glide on Windows

docs/VALKEY_SEARCH_INTEGRATION.md

@@ -31,8 +31,10 @@ Both are compatible with LangBot.
 
 ## Installation
 
-Valkey Search support is included when you install LangBot — the `valkey-glide` dependency is
-declared in `pyproject.toml`. To install manually:
+Valkey Search support is included automatically on Linux and macOS. The official `valkey-glide`
+client does not currently publish a Windows package, so LangBot skips this optional dependency on
+Windows; LangBot remains usable there, but the Valkey Search backend is unavailable. To install the
+client manually on a supported platform:
 
 ```bash
 pip install 'valkey-glide>=2.4.1,<3.0.0'

pyproject.toml

@@ -80,7 +80,7 @@ dependencies = [
     "pgvector>=0.4.1",
     "botocore>=1.42.39",
     "litellm>=1.0.0",
-    "valkey-glide>=2.4.1,<3.0.0",
+    "valkey-glide>=2.4.1,<3.0.0; sys_platform != 'win32'", # No Windows wheels are published
 ]
 keywords = [
     "bot",

src/langbot/pkg/vector/vdbs/valkey_search.py

@@ -119,7 +119,8 @@ def supported_search_types(cls) -> list[SearchType]:
     def __init__(self, ap: app.Application):
         if not VALKEY_SEARCH_AVAILABLE:
             raise ImportError(
-                "valkey-glide is not installed. Install it with: pip install 'valkey-glide>=2.4.1,<3.0.0'"
+                'valkey-glide is not installed or is unavailable on this platform. '
+                "On Linux or macOS, install it with: pip install 'valkey-glide>=2.4.1,<3.0.0'"
             )
 
         self.ap = ap

tests/integration/vector/test_valkey_search.py

@@ -77,11 +77,12 @@ async def backend(valkey_config):
         ValkeySearchVectorDatabase,
         VALKEY_SEARCH_AVAILABLE,
     )
-    from glide import ft
 
     if not VALKEY_SEARCH_AVAILABLE:
         pytest.skip('valkey-glide not installed')
 
+    from glide import ft
+
     ap = _make_ap(valkey_config)
     db = ValkeySearchVectorDatabase(ap)
     client = await db._ensure_client()

tests/unit_tests/vector/test_valkey_search_filter.py

@@ -357,10 +357,13 @@ def _fake_credentials(**kwargs):
             cred_calls.append(kwargs)
             return ('CRED', kwargs)
 
-        monkeypatch.setattr(mod, 'GlideClient', _FakeClient)
-        monkeypatch.setattr(mod, 'ServerCredentials', _fake_credentials)
-        monkeypatch.setattr(mod, 'GlideClientConfiguration', lambda **kw: kw)
-        monkeypatch.setattr(mod, 'NodeAddress', lambda *a, **k: ('node', a, k))
+        # These names are absent when the optional valkey-glide dependency is
+        # unavailable (for example, on Windows), so allow the test doubles to
+        # create them on the module.
+        monkeypatch.setattr(mod, 'GlideClient', _FakeClient, raising=False)
+        monkeypatch.setattr(mod, 'ServerCredentials', _fake_credentials, raising=False)
+        monkeypatch.setattr(mod, 'GlideClientConfiguration', lambda **kw: kw, raising=False)
+        monkeypatch.setattr(mod, 'NodeAddress', lambda *a, **k: ('node', a, k), raising=False)
         return backend, created, cred_calls, warnings
 
     async def test_username_without_password_fails_closed(self, monkeypatch):