zara-memory: require explicit boolean MCP tool result status #121

New issue

Closed

opened 2026-09-08 02:03:01 +00:00 by nsaspy · 0 comments

nsaspy commented

2026-09-08 02:03:01 +00:00

Owner

Problem

SymbolicMemoryMCP._call() only branches when result.isError is True. A malformed native MCP response can omit isError or provide a non-boolean value while still supplying structuredContent, and Zara will accept that response as successful provider evidence.

Required behavior

MCP tool results must carry an explicit boolean isError field;
missing or non-boolean status fails closed with SymbolicMemoryMCPError;
isError = true retains existing backend-error handling;
isError = false retains existing structured-evidence validation;
no provider action is reported successful from ambiguous status;
full compatibility/registry/plugin/Nix package gates stay green.

Advances provider-evidence integrity under #7.

## Problem `SymbolicMemoryMCP._call()` only branches when `result.isError is True`. A malformed native MCP response can omit `isError` or provide a non-boolean value while still supplying `structuredContent`, and Zara will accept that response as successful provider evidence. ## Required behavior - MCP tool results must carry an explicit boolean `isError` field; - missing or non-boolean status fails closed with `SymbolicMemoryMCPError`; - `isError = true` retains existing backend-error handling; - `isError = false` retains existing structured-evidence validation; - no provider action is reported successful from ambiguous status; - full compatibility/registry/plugin/Nix package gates stay green. Advances provider-evidence integrity under #7.

nsaspy closed this issue

2026-09-08 02:03:01 +00:00