Skip to content

fix(mcp): send search via POST and preflight oversized store payloads#19

Open
bharatindia123-a11y wants to merge 1 commit into
jcartu:mainfrom
bharatindia123-a11y:fix/mcp-post-search
Open

fix(mcp): send search via POST and preflight oversized store payloads#19
bharatindia123-a11y wants to merge 1 commit into
jcartu:mainfrom
bharatindia123-a11y:fix/mcp-post-search

Conversation

@bharatindia123-a11y
Copy link
Copy Markdown

@bharatindia123-a11y bharatindia123-a11y commented Apr 13, 2026

Summary

  • switch the MCP memory_search client from GET /search to POST /search with a JSON body
  • preserve limit clamping while avoiding long-query URL overflow / HTTP 414 failures
  • add an MCP-side preflight guard for memory_store content above 8000 chars so oversized requests fail clearly before transport
  • update MCP tests to verify POST search payloads, collection propagation, and the size guard

Why

The core Rasputin server already supports POST /search and intentionally enforces the 8000-char /commit limit. The correct fix in this repo is therefore in the MCP client layer rather than the server core.

Test Plan

  • PYTEST_DISABLE_PLUGIN_AUTOLOAD=1 python3 -m pytest -o addopts='' tests/test_mcp.py -q

🤖 This PR was created by an AI coding agent
🤖 This was written by an AI coding agent

@bharatindia123-a11y bharatindia123-a11y changed the title fix(mcp): use POST search and validate store size fix(mcp): send search via POST and preflight oversized store payloads Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant