From 56a4bd112ba87e8d1bcac9563f21b10323f82f61 Mon Sep 17 00:00:00 2001 From: "claude[bot]" <41898282+claude[bot]@users.noreply.github.com> Date: Mon, 23 Mar 2026 09:47:16 +0000 Subject: [PATCH] fix: add 11 missing write operations to exhaustive MCP bridge test The exhaustive_write_operations_not_read_only test was missing 11 write operations defined in core-types.ts, reducing its regression-prevention value. Added: selection.delete, block.toggle, table.addRowBefore, table.addRowAfter, table.deleteRow, table.addColumnBefore, table.addColumnAfter, table.deleteColumn, table.toggleHeaderRow, smartInsert, and insertMedia. Closes #515 Co-Authored-By: Claude Opus 4.6 --- src-tauri/src/mcp_bridge/state.rs | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/src-tauri/src/mcp_bridge/state.rs b/src-tauri/src/mcp_bridge/state.rs index 6e8da188..668df4a1 100644 --- a/src-tauri/src/mcp_bridge/state.rs +++ b/src-tauri/src/mcp_bridge/state.rs @@ -289,6 +289,7 @@ mod tests { "document.replaceInSource", "document.setContent", // Selection/cursor mutations + "selection.delete", "selection.replace", "selection.set", "cursor.setPosition", @@ -310,10 +311,18 @@ mod tests { // Block operations "block.insertHorizontalRule", "block.setType", + "block.toggle", // Table operations + "table.addColumnAfter", + "table.addColumnBefore", + "table.addRowAfter", + "table.addRowBefore", "table.batchModify", "table.delete", + "table.deleteColumn", + "table.deleteRow", "table.insert", + "table.toggleHeaderRow", // Mutation/batch operations "mutation.applyDiff", "mutation.batchEdit", @@ -345,6 +354,9 @@ mod tests { "workspace.saveDocumentAs", // Genie invocation (side-effecting) "genies.invoke", + // Smart/media insert + "smartInsert", + "insertMedia", // VMark-specific commands "vmark.cjkPunctuationConvert", "vmark.cjkSpacingFix",