deepanalysts 0.9.3__tar.gz → 0.9.4__tar.gz

{deepanalysts-0.9.3 → deepanalysts-0.9.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deepanalysts
-Version: 0.9.3
+Version: 0.9.4
 Summary: LangChain/LangGraph middleware for building AI agents with memory, skills, and filesystem support
 Author-email: Ganchuluun Narantsatsralt <tsatsralt@swifttech.cloud>
 License: MIT

{deepanalysts-0.9.3 → deepanalysts-0.9.4}/deepanalysts/middleware/filesystem.py

@@ -520,6 +520,13 @@ def _supports_execution(backend: BackendProtocol) -> bool:
     return isinstance(backend, SandboxBackendProtocol) or hasattr(backend, "execute")
 
 
+_MEDIA_BLOCK_TYPES = frozenset({"image", "image_url", "audio", "video", "file", "input_image"})
+
+
+def _is_media_block(block: object) -> bool:
+    return isinstance(block, dict) and block.get("type") in _MEDIA_BLOCK_TYPES
+
+
 TOO_LARGE_TOOL_MSG = """Tool result too large. The full result of this tool call {tool_call_id} was saved to the filesystem at: {file_path}
 Read it back with `read_file`, one slice at a time, using `offset` and `limit`
 (for example offset=0, limit=100 for the first 100 lines). Do not read the whole
@@ -1157,18 +1164,33 @@ class FilesystemMiddleware(AgentMiddleware):
         return await handler(request)
 
     def _extract_text_content(self, content: object) -> str:
-        """Extract text from message content, handling multimodal content."""
+        """Extract text from message content, handling multimodal content.
+
+        Media blocks (image/audio/video/file) never count as text: a chart tool
+        returning ``[image, text]`` used to be stringified whole, so its base64
+        crossed the eviction limit and the model never saw the image.
+        """
         if isinstance(content, str):
             return content
-        if isinstance(content, list) and len(content) == 1:
-            item = content[0]
-            if isinstance(item, str):
-                return item
-            if isinstance(item, dict) and item.get("type") == "text":
-                return item.get("text", "")
+        if isinstance(content, list):
+            parts: list[str] = []
+            for item in content:
+                if isinstance(item, str):
+                    parts.append(item)
+                elif isinstance(item, dict) and item.get("type") == "text":
+                    parts.append(item.get("text", ""))
+                elif not _is_media_block(item):
+                    parts.append(str(item))
+            return "\n".join(parts)
         # Fallback: stringify complex content
         return str(content)
 
+    @staticmethod
+    def _evicted_content(message: ToolMessage, notice: str) -> str | list:
+        """Replacement content: the eviction notice, plus any media blocks kept as-is."""
+        media = [b for b in message.content if _is_media_block(b)] if isinstance(message.content, list) else []
+        return [*media, {"type": "text", "text": notice}] if media else notice
+
     def _process_large_message(
         self,
         message: ToolMessage,
@@ -1186,10 +1208,13 @@ class FilesystemMiddleware(AgentMiddleware):
         # Use smart preview showing head and tail instead of just first lines
         content_sample = create_content_preview(content)
         processed_message = ToolMessage(
-            TOO_LARGE_TOOL_MSG.format(
-                tool_call_id=message.tool_call_id,
-                file_path=file_path,
-                content_sample=content_sample,
+            self._evicted_content(
+                message,
+                TOO_LARGE_TOOL_MSG.format(
+                    tool_call_id=message.tool_call_id,
+                    file_path=file_path,
+                    content_sample=content_sample,
+                ),
             ),
             tool_call_id=message.tool_call_id,
         )
@@ -1273,10 +1298,13 @@ class FilesystemMiddleware(AgentMiddleware):
         # Use smart preview showing head and tail instead of just first lines
         content_sample = create_content_preview(content)
         processed_message = ToolMessage(
-            TOO_LARGE_TOOL_MSG.format(
-                tool_call_id=message.tool_call_id,
-                file_path=file_path,
-                content_sample=content_sample,
+            self._evicted_content(
+                message,
+                TOO_LARGE_TOOL_MSG.format(
+                    tool_call_id=message.tool_call_id,
+                    file_path=file_path,
+                    content_sample=content_sample,
+                ),
             ),
             tool_call_id=message.tool_call_id,
         )

{deepanalysts-0.9.3 → deepanalysts-0.9.4}/deepanalysts.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: deepanalysts
-Version: 0.9.3
+Version: 0.9.4
 Summary: LangChain/LangGraph middleware for building AI agents with memory, skills, and filesystem support
 Author-email: Ganchuluun Narantsatsralt <tsatsralt@swifttech.cloud>
 License: MIT

{deepanalysts-0.9.3 → deepanalysts-0.9.4}/deepanalysts.egg-info/SOURCES.txt

@@ -47,6 +47,7 @@ tests/test_prompt_sections.py
 tests/test_protocol_bridge.py
 tests/test_regression_files_delta.py
 tests/test_regression_fs_backport.py
+tests/test_regression_media_tool_results.py
 tests/test_regression_middleware_tools.py
 tests/test_regression_new_api.py
 tests/test_regression_prompt_contracts.py

{deepanalysts-0.9.3 → deepanalysts-0.9.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "deepanalysts"
-version = "0.9.3"
+version = "0.9.4"
 description = "LangChain/LangGraph middleware for building AI agents with memory, skills, and filesystem support"
 readme = "README.md"
 license = { text = "MIT" }

deepanalysts-0.9.4/tests/test_regression_media_tool_results.py

@@ -0,0 +1,65 @@
+"""Regression: tool results carrying images were evicted as text (2026-10-13).
+
+`_extract_text_content` fell back to `str(content)` for any multi-block result,
+so a chart tool returning `[image, text]` counted its base64 PNG (~300K chars)
+as text, crossed the eviction limit, and the model got a file path + a base64
+preview instead of the image — no agent ever saw a chart.
+
+Locks:
+1. Only text blocks count toward the eviction size; media blocks never do.
+2. A small text part next to an image is NOT evicted; the message is untouched.
+3. When the text part itself is oversized, only the text is evicted and the
+   media blocks are kept on the replacement message.
+"""
+
+import pytest
+from langchain_core.messages import ToolMessage
+
+from deepanalysts.backends.state import StateBackend
+from deepanalysts.middleware.filesystem import FilesystemMiddleware
+
+pytestmark = pytest.mark.anyio
+
+B64 = "iVBORw0KGgo" + "A" * 400_000
+IMAGE = {"type": "image", "base64": B64, "mime_type": "image/png"}
+
+
+@pytest.fixture
+def rt(state_runtime):
+    return state_runtime()
+
+
+def _mw():
+    return FilesystemMiddleware(backend=lambda r: StateBackend(r), tool_token_limit_before_evict=5000)
+
+
+def _chart(text: str) -> ToolMessage:
+    return ToolMessage(content=[IMAGE, {"type": "text", "text": text}], tool_call_id="c1", name="view_chart")
+
+
+def test_media_blocks_do_not_count_as_text(rt):
+    assert _mw()._extract_text_content(_chart("Chart: BTC/USD 4h").content) == "Chart: BTC/USD 4h"
+
+
+def test_small_text_with_image_is_not_evicted(rt):
+    msg = _chart("Chart: BTC/USD 4h")
+    out = _mw()._intercept_large_tool_result(msg, rt)
+    assert out is msg
+
+
+def test_oversized_text_is_evicted_but_image_is_kept(rt):
+    msg = _chart("x" * 30_000)
+    out = _mw()._intercept_large_tool_result(msg, rt)
+    replacement = out.update["messages"][0] if hasattr(out, "update") else out
+    blocks = replacement.content
+    assert isinstance(blocks, list)
+    assert any(isinstance(b, dict) and b.get("type") == "image" for b in blocks)
+    text = " ".join(b.get("text", "") for b in blocks if isinstance(b, dict) and b.get("type") == "text")
+    assert "/large_tool_results/" in text
+    assert B64 not in text
+
+
+async def test_async_path_keeps_small_chart_results(rt):
+    msg = _chart("Chart: BTC/USD 4h")
+    out = await _mw()._aintercept_large_tool_result(msg, rt)
+    assert out is msg