From 45061a788834fda572a56fc5e6c48c30c43cd1cb Mon Sep 17 00:00:00 2001
From: Sun Haoyuan <58774225+Shy7777@users.noreply.github.com>
Date: Tue, 22 Sep 2026 15:34:47 +0800
Subject: [PATCH] Python: preserve Redis memory vectorizer data types

---
 .../_context_provider.py                      |  8 +++-
 .../tests/test_context_provider_edges.py      | 39 +++++++++++++++++++
 2 files changed, 45 insertions(+), 2 deletions(-)

diff --git a/python/packages/redis/agent_framework_redis/_context_provider.py b/python/packages/redis/agent_framework_redis/_context_provider.py
index 57fe4f97fd8..4886731c99f 100644
--- a/python/packages/redis/agent_framework_redis/_context_provider.py
+++ b/python/packages/redis/agent_framework_redis/_context_provider.py
@@ -12,7 +12,6 @@
 import sys
 from typing import TYPE_CHECKING, Any, ClassVar, Literal
 
-import numpy as np
 from agent_framework import Message
 from agent_framework._sessions import AgentSession, ContextProvider, SessionContext
 from agent_framework._telemetry import mark_feature_used
@@ -23,6 +22,7 @@
 from redisvl.index import AsyncSearchIndex
 from redisvl.query import AggregateHybridQuery, TextQuery
 from redisvl.query.filter import FilterExpression, Tag
+from redisvl.redis.utils import array_to_buffer
 from redisvl.utils.token_escaper import TokenEscaper
 from redisvl.utils.vectorize import BaseVectorizer
 
@@ -341,7 +341,11 @@ async def _add(
                 text_list, batch_size=len(text_list)
             )
             for i, d in enumerate(prepared):
-                vec = np.asarray(embeddings[i], dtype=np.float32).tobytes()
+                # aembed_many returns lists unless as_buffer is enabled.
+                vec = array_to_buffer(
+                    embeddings[i],  # pyright: ignore[reportArgumentType]
+                    dtype=self.redis_vectorizer.dtype,
+                )
                 field_name: str = self.vector_field_name
                 d[field_name] = vec
 
diff --git a/python/packages/redis/tests/test_context_provider_edges.py b/python/packages/redis/tests/test_context_provider_edges.py
index 36146f52019..ea6287ffe37 100644
--- a/python/packages/redis/tests/test_context_provider_edges.py
+++ b/python/packages/redis/tests/test_context_provider_edges.py
@@ -9,7 +9,9 @@
 from unittest.mock import AsyncMock, MagicMock, patch
 
 import pytest
+from agent_framework import AgentResponse, AgentSession, Message, SessionContext
 from agent_framework.exceptions import IntegrationInvalidRequestException
+from redisvl.redis.utils import buffer_to_array
 from redisvl.utils.vectorize import BaseVectorizer
 
 from agent_framework_redis._context_provider import RedisContextProvider
@@ -170,6 +172,43 @@ async def test_add_vectorizes_documents_and_applies_defaults(
     vectorizer.aembed_many.assert_awaited_once_with(["first", "second"], batch_size=2)
 
 
+@pytest.mark.parametrize("dtype, expected_size", [("float16", 4), ("float32", 8), ("float64", 16), ("bfloat16", 4)])
+async def test_after_run_stores_vectors_in_configured_dtype(
+    dtype: str,
+    expected_size: int,
+    mock_index: AsyncMock,
+) -> None:
+    vectorizer = MagicMock(spec=BaseVectorizer)
+    vectorizer.dims = 2
+    vectorizer.dtype = dtype
+    embeddings = [[1.5, -2.25], [3.125, 4.5]]
+    vectorizer.aembed_many = AsyncMock(return_value=embeddings)
+    provider = RedisContextProvider(
+        user_id="user-1",
+        redis_vectorizer=vectorizer,
+        vector_field_name="embedding",
+        redis_index=mock_index,
+    )
+    session = AgentSession(session_id="session-1")
+    context = SessionContext(
+        session_id=session.session_id,
+        input_messages=[Message(role="user", contents=["I prefer tea"])],
+    )
+    context._response = AgentResponse(messages=[Message(role="assistant", contents=["I'll remember that"])])
+
+    await provider.after_run(agent=MagicMock(), session=session, context=context, state={})
+
+    vector_field = next(field for field in provider.schema_dict["fields"] if field["name"] == "embedding")
+    assert vector_field["attrs"]["datatype"] == dtype
+    mock_index.load.assert_awaited_once()
+    loaded_docs = mock_index.load.await_args.args[0]
+    assert [doc["content"] for doc in loaded_docs] == ["I prefer tea", "I'll remember that"]
+    for doc, expected_embedding in zip(loaded_docs, embeddings, strict=True):
+        assert len(doc["embedding"]) == expected_size
+        assert buffer_to_array(doc["embedding"], dtype) == expected_embedding
+    vectorizer.aembed_many.assert_awaited_once_with(["I prefer tea", "I'll remember that"], batch_size=2)
+
+
 async def test_redis_search_requires_non_empty_text(
     patch_index: MagicMock,  # noqa: ARG001
 ) -> None: