Skip to content

Fix MCP read errors and audit rejected calls - #604

Merged
WaylandYang merged 2 commits into
deeplethe:devfrom
Floating-Y:fix/603-mcp-read-follow-ups
Sep 12, 2026
Merged

WaylandYang merged 2 commits into
deeplethe:devfrom
Floating-Y:fix/603-mcp-read-follow-ups

Conversation

@Floating-Y

Copy link
Copy Markdown
Contributor

Database failures during entity resolution now return explicit MCP tool errors
instead of invalid/unknown entity responses. This also covers graph reads by UUID
and path searches, while preserving existing missing-entity and empty-result behavior.

  • Record calls rejected by check_call through the existing mcp.tool_called
    audit path, once per refused or executed call.
  • Add regression coverage for read failures, auditing, search limits,
    entity-valued qualifiers, and changes without until.
  • Add a dated revision to ADR 0020 and clarify that removing or renaming
    documented structuredContent fields requires a decision record.

No new dependencies or database migrations.

Validation:

  • Formatting and workspace Clippy passed.
  • Workspace tests against PostgreSQL: 613 passed, 0 failed, 1 ignored.
  • All 8 MCP tests passed.
  • Frontend production build passed.
  • Database tests used UTOPIA_TEST_REQUIRE_DB=1 and 127.0.0.1.

Closes #603.

Floating-Y and others added 2 commits September 11, 2026 09:47
Signed-off-by: Floating-Y <118035379+Floating-Y@users.noreply.github.com>
@WaylandYang
WaylandYang merged commit 537f723 into deeplethe:dev Sep 12, 2026
4 checks passed
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.

Structured MCP reads: follow-ups from the #601 review

2 participants