Skip to content

feat: MCP: add minimal response mode#27

Merged
spokV merged 3 commits intoagentic-box:devfrom
justinTM:codex/minimal-similar-response
Mar 13, 2026
Merged

feat: MCP: add minimal response mode#27
spokV merged 3 commits intoagentic-box:devfrom
justinTM:codex/minimal-similar-response

Conversation

@justinTM
Copy link
Copy Markdown

@justinTM justinTM commented Mar 10, 2026

Helps with token efficiency when used as an MCP server.

Response payloads were hitting ~1k characters when metadata was relatively large.

tests/conftest.py: shared with #28

Justin Mai and others added 2 commits March 9, 2026 19:56
Add an optional response_mode argument to memory_create so callers can
request a token-efficient response that returns only the created memory id.
The default full payload remains unchanged.

- validate response_mode against full|minimal
- return only the created memory id in minimal mode
- cover the minimal contract with a small fixture-based pytest

Co-Authored-By: OpenAI Codex <noreply@openai.com>
Keep minimal memory_create responses compact while preserving similar-memory details when suggest_similar=true. Add focused pytest coverage for the id-only case, the preserved similar payload, and the disabled-suggestions path.

Co-Authored-By: OpenAI Codex <noreply@openai.com>
@justinTM justinTM changed the base branch from main to dev March 10, 2026 19:45
Copy link
Copy Markdown
Collaborator

@spokV spokV left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM — clean implementation. Minimal response mode preserves important signals (duplicate warnings, similar memories) while cutting payload size. Good test coverage.

@spokV spokV merged commit 5a15a26 into agentic-box:dev Mar 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.

2 participants