vantage-peers-mcp 2.7.0 → 2.7.1

package/CHANGELOG.md

@@ -1,5 +1,40 @@
 # Changelog
 
+## [2.7.1] — 2026-06-14 — Day 101 FIX-B mcpError() → mcpConvexError() sweep (task k1744wk2gfgqt2gdqh41d4r91h88n410)
+
+Patch-level fix to make every tool wrapper surface structured ConvexError
+diagnostics instead of opaque `"Server Error [Request ID: ...]"` strings.
+
+**Problem (Day 101):** when a Convex mutation/query threw a `ConvexError`
+(e.g. `TASK_START_BLOCKED`, `COMPLETION_NOTE_REQUIRED`) or an
+`ArgumentValidationError`, the tool wrapper called
+`mcpError(error.message ?? String(error))` which returned a plain
+`Error: <message>` text payload. The MCP client (Claude Code) then often
+re-displayed this as a generic `Server Error [Request ID:...]`, masking the
+actual root cause and triggering misdiagnosis sprees (Pi msg
+`jn7c5tfj0347vaenyqgrezehk188mr11` mistakenly attributed the symptoms to
+a backend regression that did not exist — see Sigma report msg
+`jn773p8qnfp6ycb1f7gajhs8vn88m11a`).
+
+**Fix:** sweep all 99 occurrences of
+`return mcpError(error.message ?? String(error))` →
+`return mcpConvexError(error)`. The `mcpConvexError` helper (already
+present at `src/tools.ts:535`, shipped 2.4.x) parses ConvexError messages
+into a structured JSON `{ code, message, path, hint }` payload that
+surfaces the actual error code (`ArgumentValidationError`,
+`ConvexError`, `AuthorizationError`, `SchemaValidationError`, etc.)
+along with a path and a concise hint for ID-table mismatches.
+
+Total counts post-sweep: `grep mcpConvexError(` → 102 ; `grep
+mcpError(error.message` → 0.
+
+Type check: `npx tsc --noEmit` exits 0.
+
+Mission/refs: `k575kc1ryps0n8br95jw3q7d0x88m2v9` MCP CRUD Baseline Standard,
+task FIX-B `k1744wk2gfgqt2gdqh41d4r91h88n410`. Co-shipped with hook
+`enforce-friction-field.py` v1.1.0 STDERR clarification (Day 101 task
+FIX-A `k17ads7kh7qk7yxgfe0dh73ggx88ny9f`).
+
 ## [2.7.0] — 2026-06-13 — Day 100 get_by_id surface Phase 2b (task k172735brsw6bc3j2dkkkfxqrx88kkjq)
 
 Phase 2b wires 2 MCP wrappers calling the Phase 2a Convex queries deployed

package/dist/src/tools.js

@@ -849,7 +849,7 @@ export function registerTools(server, convex, oauthCtx) {
                 createdBy,
                 errorMessage: error?.message ?? String(error),
             });
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── soft_delete_memory ──────────────────────────────────────────────────────
@@ -882,7 +882,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_memory ──────────────────────────────────────────────────────────────
@@ -912,7 +912,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── recall ──────────────────────────────────────────────────────────────────
@@ -967,7 +967,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── text_search ─────────────────────────────────────────────────────────────
@@ -1013,7 +1013,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── hybrid_search ───────────────────────────────────────────────────────────
@@ -1070,7 +1070,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── store_episode ───────────────────────────────────────────────────────────
@@ -1124,7 +1124,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_profile ─────────────────────────────────────────────────────────────
@@ -1157,7 +1157,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── update_profile ──────────────────────────────────────────────────────────
@@ -1215,7 +1215,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_memories ───────────────────────────────────────────────────────────
@@ -1316,7 +1316,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── send_message ────────────────────────────────────────────────────────────
@@ -1394,7 +1394,7 @@ export function registerTools(server, convex, oauthCtx) {
                 channel,
                 errorMessage: error?.message ?? String(error),
             });
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── check_messages ──────────────────────────────────────────────────────────
@@ -1473,7 +1473,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── mark_as_read ────────────────────────────────────────────────────────────
@@ -1519,7 +1519,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── delete_message ──────────────────────────────────────────────────────────
@@ -1558,7 +1558,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── set_summary ─────────────────────────────────────────────────────────────
@@ -1597,7 +1597,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_peers ──────────────────────────────────────────────────────────────
@@ -1683,7 +1683,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_messages ───────────────────────────────────────────────────────────
@@ -1774,7 +1774,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_broadcast_status ───────────────────────────────────────────────────
@@ -1826,7 +1826,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── create_task ─────────────────────────────────────────────────────────────
@@ -1903,7 +1903,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_tasks ──────────────────────────────────────────────────────────────
@@ -2014,7 +2014,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── update_task ─────────────────────────────────────────────────────────────
@@ -2099,7 +2099,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── complete_task ───────────────────────────────────────────────────────────
@@ -2140,7 +2140,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── start_task ──────────────────────────────────────────────────────────────
@@ -2177,7 +2177,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── checkout_task ───────────────────────────────────────────────────────────
@@ -2215,7 +2215,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── delete_task ─────────────────────────────────────────────────────────────
@@ -2252,7 +2252,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── block_task ──────────────────────────────────────────────────────────────
@@ -2301,7 +2301,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── add_task_dependency ─────────────────────────────────────────────────────
@@ -2346,7 +2346,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_tasks_by_mission ───────────────────────────────────────────────────
@@ -2429,7 +2429,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── create_mission ──────────────────────────────────────────────────────────
@@ -2490,7 +2490,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_missions ───────────────────────────────────────────────────────────
@@ -2588,7 +2588,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_mission ─────────────────────────────────────────────────────────────
@@ -2618,7 +2618,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── update_mission ──────────────────────────────────────────────────────────
@@ -2675,7 +2675,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── update_mission_status ───────────────────────────────────────────────────
@@ -2710,7 +2710,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── write_diary ─────────────────────────────────────────────────────────────
@@ -2766,7 +2766,7 @@ export function registerTools(server, convex, oauthCtx) {
                 orchestrator,
                 errorMessage: error?.message ?? String(error),
             });
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_diary ───────────────────────────────────────────────────────────────
@@ -2809,7 +2809,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_diaries ────────────────────────────────────────────────────────────
@@ -2902,7 +2902,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── create_briefing_note ────────────────────────────────────────────────────
@@ -3041,7 +3041,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_briefing_notes ─────────────────────────────────────────────────────
@@ -3140,7 +3140,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── register_component ──────────────────────────────────────────────────────
@@ -3202,7 +3202,7 @@ export function registerTools(server, convex, oauthCtx) {
                 createdBy,
                 errorMessage: error?.message ?? String(error),
             });
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_components ─────────────────────────────────────────────────────────
@@ -3280,7 +3280,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_component ───────────────────────────────────────────────────────────
@@ -3312,7 +3312,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── update_component ────────────────────────────────────────────────────────
@@ -3361,7 +3361,7 @@ export function registerTools(server, convex, oauthCtx) {
                 componentId,
                 errorMessage: error?.message ?? String(error),
             });
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── delete_component ────────────────────────────────────────────────────────
@@ -3390,7 +3390,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── search_components ───────────────────────────────────────────────────────
@@ -3441,7 +3441,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── create_recurring_task ───────────────────────────────────────────────────
@@ -3500,7 +3500,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_recurring_tasks ────────────────────────────────────────────────────
@@ -3573,7 +3573,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── pause_recurring_task ────────────────────────────────────────────────────
@@ -3600,7 +3600,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── resume_recurring_task ───────────────────────────────────────────────────
@@ -3627,7 +3627,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── delete_recurring_task ───────────────────────────────────────────────────
@@ -3654,7 +3654,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── update_recurring_task ───────────────────────────────────────────────────
@@ -3700,7 +3700,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── create_mandate ──────────────────────────────────────────────────────────
@@ -3759,7 +3759,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── accept_mandate ──────────────────────────────────────────────────────────
@@ -3794,7 +3794,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── update_mandate ──────────────────────────────────────────────────────────
@@ -3838,7 +3838,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── settle_mandate ──────────────────────────────────────────────────────────
@@ -3875,7 +3875,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── validate_mandate_spending ───────────────────────────────────────────────
@@ -3902,7 +3902,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_mandates ───────────────────────────────────────────────────────────
@@ -3988,7 +3988,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── create_bu ───────────────────────────────────────────────────────────────
@@ -4058,7 +4058,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── update_bu ───────────────────────────────────────────────────────────────
@@ -4127,7 +4127,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_bu ──────────────────────────────────────────────────────────────────
@@ -4157,7 +4157,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_bus ────────────────────────────────────────────────────────────────
@@ -4237,7 +4237,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── delete_bu ───────────────────────────────────────────────────────────────
@@ -4271,7 +4271,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── add_repo_mapping ────────────────────────────────────────────────────────
@@ -4319,7 +4319,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_repo_mappings ──────────────────────────────────────────────────────
@@ -4393,7 +4393,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── remove_repo_mapping ─────────────────────────────────────────────────────
@@ -4427,7 +4427,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_issues ─────────────────────────────────────────────────────────────
@@ -4547,7 +4547,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_issue ───────────────────────────────────────────────────────────────
@@ -4581,7 +4581,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── update_issue_status ─────────────────────────────────────────────────────
@@ -4621,7 +4621,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── link_commit_to_issue ────────────────────────────────────────────────────
@@ -4659,7 +4659,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── verify_issue ────────────────────────────────────────────────────────────
@@ -4695,7 +4695,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── issue_stats ─────────────────────────────────────────────────────────────
@@ -4728,7 +4728,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── create_fix_pattern ──────────────────────────────────────────────────────
@@ -4786,7 +4786,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── add_fix_attempt ─────────────────────────────────────────────────────────
@@ -4827,7 +4827,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── validate_fix ────────────────────────────────────────────────────────────
@@ -4861,7 +4861,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── search_fix_patterns ─────────────────────────────────────────────────────
@@ -4912,7 +4912,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_fix_patterns ───────────────────────────────────────────────────────
@@ -5001,7 +5001,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── link_issue_to_pattern ───────────────────────────────────────────────────
@@ -5035,7 +5035,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_mission_template ────────────────────────────────────────────────────
@@ -5063,7 +5063,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── update_mission_template ─────────────────────────────────────────────────
@@ -5131,7 +5131,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── instantiate_template_into_mission ───────────────────────────────────────
@@ -5192,7 +5192,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── add_deployment ──────────────────────────────────────────────────────────
@@ -5242,7 +5242,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── remove_deployment ───────────────────────────────────────────────────────
@@ -5274,7 +5274,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── list_errors ─────────────────────────────────────────────────────────────
@@ -5353,7 +5353,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_error ───────────────────────────────────────────────────────────────
@@ -5383,7 +5383,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── whoami ──────────────────────────────────────────────────────────────────
@@ -5512,7 +5512,7 @@ export function registerTools(server, convex, oauthCtx) {
         repo: z.string(),
         orchestrator: z.string(),
         project: z.string().optional(),
-    }, async ({ repo, orchestrator, project }) => {
+    }, { readOnlyHint: false, openWorldHint: false, destructiveHint: false, title: "Register repo mapping (alias)" }, async ({ repo, orchestrator, project }) => {
         const masterDenied = guardMasterOnly("register_repo_mapping");
         if (masterDenied)
             return masterDenied;
@@ -5525,7 +5525,7 @@ export function registerTools(server, convex, oauthCtx) {
             return { content: [{ type: "text", text: JSON.stringify(result, null, 2) }] };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // delete_repo_mapping → was remove_repo_mapping [ALIAS of remove_repo_mapping — C0.3 master-only]
@@ -5533,7 +5533,7 @@ export function registerTools(server, convex, oauthCtx) {
         "WHEN: use when a repo is archived or its events should no longer generate VP notifications. " +
         "EXAMPLE: delete_repo_mapping repo='vantageos-agency/vantage-peers'.", {
         repo: z.string(),
-    }, async ({ repo }) => {
+    }, { readOnlyHint: false, openWorldHint: false, destructiveHint: true, title: "Delete repo mapping (alias)" }, async ({ repo }) => {
         const masterDenied = guardMasterOnly("delete_repo_mapping");
         if (masterDenied)
             return masterDenied;
@@ -5542,7 +5542,7 @@ export function registerTools(server, convex, oauthCtx) {
             return { content: [{ type: "text", text: JSON.stringify(result, null, 2) }] };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // register_deployment → was add_deployment [ALIAS of add_deployment — C0.1 master-only]
@@ -5554,7 +5554,7 @@ export function registerTools(server, convex, oauthCtx) {
         deployKeyEnvVar: z.string(),
         githubRepo: z.string(),
         orchestrator: z.string(),
-    }, async ({ name, deploymentUrl, deployKeyEnvVar, githubRepo, orchestrator }) => {
+    }, { readOnlyHint: false, openWorldHint: false, destructiveHint: false, title: "Register deployment (alias)" }, async ({ name, deploymentUrl, deployKeyEnvVar, githubRepo, orchestrator }) => {
         const masterDenied = guardMasterOnly("register_deployment");
         if (masterDenied)
             return masterDenied;
@@ -5569,7 +5569,7 @@ export function registerTools(server, convex, oauthCtx) {
             return { content: [{ type: "text", text: JSON.stringify(result, null, 2) }] };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // delete_deployment → was remove_deployment [ALIAS of remove_deployment — C0.1 master-only]
@@ -5577,7 +5577,7 @@ export function registerTools(server, convex, oauthCtx) {
         "WHEN: use when a deployment is retired or moved to a different monitoring config. " +
         "EXAMPLE: delete_deployment name='vantage-prod'.", {
         name: z.string(),
-    }, async ({ name }) => {
+    }, { readOnlyHint: false, openWorldHint: false, destructiveHint: true, title: "Delete deployment (alias)" }, async ({ name }) => {
         const masterDenied = guardMasterOnly("delete_deployment");
         if (masterDenied)
             return masterDenied;
@@ -5586,7 +5586,7 @@ export function registerTools(server, convex, oauthCtx) {
             return { content: [{ type: "text", text: JSON.stringify(result, null, 2) }] };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // check_mandate_spending → was validate_mandate_spending (not C0-gated — read-only check)
@@ -5595,7 +5595,7 @@ export function registerTools(server, convex, oauthCtx) {
         "EXAMPLE: check_mandate_spending mandateId='j57aaaaa...' proposedAmount=500.", {
         mandateId: z.string(),
         proposedAmount: z.number(),
-    }, async ({ mandateId, proposedAmount }) => {
+    }, { readOnlyHint: true, openWorldHint: false, destructiveHint: false, title: "Check mandate spending (alias)" }, async ({ mandateId, proposedAmount }) => {
         try {
             const result = await convex.query("mandates:validateSpending", {
                 mandateId,
@@ -5604,7 +5604,7 @@ export function registerTools(server, convex, oauthCtx) {
             return { content: [{ type: "text", text: JSON.stringify(result, null, 2) }] };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // check_fix → was validate_fix [ALIAS of validate_fix — C0.5 master-only]
@@ -5613,7 +5613,7 @@ export function registerTools(server, convex, oauthCtx) {
         "EXAMPLE: check_fix patternId='j57aaaaa...' validatedFix='Add suppressHydrationWarning to date elements'.", {
         patternId: z.string(),
         validatedFix: z.string(),
-    }, async ({ patternId, validatedFix }) => {
+    }, { readOnlyHint: false, openWorldHint: false, destructiveHint: false, title: "Check/validate fix (alias)" }, async ({ patternId, validatedFix }) => {
         const masterDenied = guardMasterOnly("check_fix");
         if (masterDenied)
             return masterDenied;
@@ -5625,7 +5625,7 @@ export function registerTools(server, convex, oauthCtx) {
             return { content: [{ type: "text", text: JSON.stringify(result, null, 2) }] };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // create_fix_attempt → was add_fix_attempt (not C0-gated — uses guardFrom(createdBy))
@@ -5638,7 +5638,7 @@ export function registerTools(server, convex, oauthCtx) {
         why: z.string().optional(),
         commitSha: z.string().optional(),
         createdBy: z.string(),
-    }, async ({ patternId, description, worked, why, commitSha, createdBy }) => {
+    }, { readOnlyHint: false, openWorldHint: false, destructiveHint: false, title: "Create fix attempt (alias)" }, async ({ patternId, description, worked, why, commitSha, createdBy }) => {
         const fromDenied = guardFrom(createdBy);
         if (fromDenied)
             return fromDenied;
@@ -5654,7 +5654,7 @@ export function registerTools(server, convex, oauthCtx) {
             return { content: [{ type: "text", text: JSON.stringify(result, null, 2) }] };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // create_task_dependency → was add_task_dependency (not C0-gated — uses callerOrchestrator auth)
@@ -5664,7 +5664,7 @@ export function registerTools(server, convex, oauthCtx) {
         taskId: z.string(),
         dependsOn: z.array(z.string()),
         callerOrchestrator: z.string(),
-    }, async ({ taskId, dependsOn, callerOrchestrator }) => {
+    }, { readOnlyHint: false, openWorldHint: false, destructiveHint: false, title: "Create task dependency (alias)" }, async ({ taskId, dependsOn, callerOrchestrator }) => {
         try {
             const result = await convex.mutation("tasks:update", {
                 taskId,
@@ -5674,7 +5674,7 @@ export function registerTools(server, convex, oauthCtx) {
             return { content: [{ type: "text", text: JSON.stringify(result, null, 2) }] };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // update_summary → was set_summary (not C0-gated — uses orchestratorId auth)
@@ -5684,7 +5684,7 @@ export function registerTools(server, convex, oauthCtx) {
         orchestratorId: z.string(),
         instanceId: z.string().optional(),
         summary: z.string(),
-    }, async ({ orchestratorId, instanceId, summary }) => {
+    }, { readOnlyHint: false, openWorldHint: false, destructiveHint: false, title: "Update summary (alias)" }, async ({ orchestratorId, instanceId, summary }) => {
         try {
             const result = await convex.mutation("profiles:updateDynamic", {
                 orchestratorId,
@@ -5694,7 +5694,7 @@ export function registerTools(server, convex, oauthCtx) {
             return { content: [{ type: "text", text: JSON.stringify(result, null, 2) }] };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // create_diary → was write_diary (not C0-gated — uses guardFrom(author) when present)
@@ -5705,7 +5705,7 @@ export function registerTools(server, convex, oauthCtx) {
         orchestrator: z.string(),
         content: z.string(),
         author: z.string().optional(),
-    }, async ({ date, orchestrator, content, author }) => {
+    }, { readOnlyHint: false, openWorldHint: false, destructiveHint: false, title: "Create diary entry (alias)" }, async ({ date, orchestrator, content, author }) => {
         assertContentSize(content, "content");
         if (author !== undefined) {
             const fromDenied = guardFrom(author);
@@ -5722,7 +5722,7 @@ export function registerTools(server, convex, oauthCtx) {
             return { content: [{ type: "text", text: JSON.stringify(result, null, 2) }] };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_task ────────────────────────────────────────────────────────────────
@@ -5749,7 +5749,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_fix_pattern ─────────────────────────────────────────────────────────
@@ -5775,7 +5775,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_mandate ─────────────────────────────────────────────────────────────
@@ -5801,7 +5801,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_repo_mapping ────────────────────────────────────────────────────────
@@ -5832,7 +5832,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_message ─────────────────────────────────────────────────────────────
@@ -5861,7 +5861,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
     // ── get_recurring_task ──────────────────────────────────────────────────────
@@ -5890,7 +5890,7 @@ export function registerTools(server, convex, oauthCtx) {
             };
         }
         catch (error) {
-            return mcpError(error.message ?? String(error));
+            return mcpConvexError(error);
         }
     });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "vantage-peers-mcp",
-  "version": "2.7.0",
+  "version": "2.7.1",
   "description": "MCP server for VantagePeers — shared memory, messaging, and task coordination for AI agent teams",
   "type": "module",
   "main": "./dist/server.js",