fix: allow structured leaf nodes without children#32
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the output schemas for the remnote_search and remnote_read_note tools to mark the children array of structured content nodes as optional, matching bridge responses that omit children for leaf nodes. It also updates the schema descriptions, adds a unit test to verify this behavior, and documents the change in CHANGELOG.md. There are no review comments, so I have no feedback to provide.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
What changed
Mark the
childrenarray optional for structured content nodes in theremnote_searchandremnote_read_noteoutput schemas. The property remains supported for non-leaf nodes.Why
The RemNote bridge intentionally omits
childrenfor leaf nodes, but the MCP server output schema required it. Strict MCP clients therefore rejected otherwise valid large structured responses with a schema error.Impact
childrennow validate correctly.children; recursive structured output is unchanged.Validation
children?in the structured output schema