@homenshum/convex-mcp-nodebench 0.9.0 → 0.9.1

package/dist/index.js

@@ -79,7 +79,7 @@ for (const tool of ALL_TOOLS) {
 // ── Server setup ────────────────────────────────────────────────────
 const server = new Server({
     name: "convex-mcp-nodebench",
-    version: "0.9.0",
+    version: "0.9.1",
 }, {
     capabilities: {
         tools: {},

package/dist/tools/architectTools.js

@@ -444,24 +444,15 @@ function inferConvexStrategy(signature) {
         return "Wrap optional fields with v.optional() — avoids runtime errors on missing fields";
     if (/v\.union/i.test(lower))
         return "Use v.union() for discriminated types — add a 'type' field as discriminator for type safety";
-    // Function types
-    if (/internalmutation/i.test(lower))
-        return "Create internalMutation — only callable by other server functions, not from client. Import from _generated/server";
-    if (/internalaction/i.test(lower))
-        return "Create internalAction — for server-side-only work like API calls. Cannot access ctx.db directly, use ctx.runMutation/ctx.runQuery";
-    if (/internalquery/i.test(lower))
-        return "Create internalQuery — server-side-only reads. Not subscribable from client";
-    if (/httpaction/i.test(lower))
-        return "Create httpAction in http.ts — register with httpRouter.route({ method, path, handler })";
-    if (/mutation\b/i.test(lower))
-        return "Create public mutation — add args validator, return validator, and auth check (ctx.auth.getUserIdentity)";
-    if (/action\b/i.test(lower))
-        return "Create public action — for external API calls. Wrap in try/catch, use ctx.runMutation for DB writes";
-    if (/query\b/i.test(lower))
-        return "Create public query — reactive subscription from client. Must be deterministic, no side effects";
-    // Data access
+    // Data access (BEFORE generic function types — ctx.db.query must match before query)
     if (/ctx\.db\.query/i.test(lower))
         return "Add ctx.db.query('table') — chain .withIndex() for indexed lookups (avoid full-table .filter())";
+    if (/ctx\.db\.insert/i.test(lower))
+        return "Use ctx.db.insert('table', { ...fields }) — within a mutation for transactional guarantee";
+    if (/ctx\.db\.patch/i.test(lower))
+        return "Use ctx.db.patch(id, { ...fields }) — partial update, only specified fields change";
+    if (/ctx\.db\.delete/i.test(lower))
+        return "Use ctx.db.delete(id) — remember to clean up related records and storage files";
     if (/withindex/i.test(lower))
         return "Use .withIndex('indexName', q => q.eq('field', value)) — ensure index exists in schema.ts first";
     if (/withsearchindex/i.test(lower))
@@ -470,30 +461,31 @@ function inferConvexStrategy(signature) {
         return "Add .collect() to get all results — WARNING: add .take(limit) for bounded queries to avoid scanning entire table";
     if (/\.paginate\b/i.test(lower))
         return "Use .paginate(opts) with paginationOptsValidator in args — return { page, isDone, continueCursor }";
-    if (/ctx\.db\.insert/i.test(lower))
-        return "Use ctx.db.insert('table', { ...fields }) — within a mutation for transactional guarantee";
-    if (/ctx\.db\.patch/i.test(lower))
-        return "Use ctx.db.patch(id, { ...fields }) — partial update, only specified fields change";
-    if (/ctx\.db\.delete/i.test(lower))
-        return "Use ctx.db.delete(id) — remember to clean up related records and storage files";
-    // Auth
+    // Cross-function calls (BEFORE generic function types — ctx.runMutation must match before mutation)
+    if (/ctx\.runmutation|runmutation/i.test(lower))
+        return "Use ctx.runMutation(internal.module.functionName, args) — prefer internal functions for server-to-server mutation calls";
+    if (/ctx\.runquery|runquery/i.test(lower))
+        return "Use ctx.runQuery(internal.module.functionName, args) — prefer internal functions for server-to-server query calls";
+    if (/ctx\.runaction|runaction/i.test(lower))
+        return "Use ctx.runAction(internal.module.functionName, args) — prefer internal functions for server-to-server action calls";
+    // Auth (BEFORE generic function types)
     if (/getuseridentity/i.test(lower))
         return "Add `const identity = await ctx.auth.getUserIdentity()` — throw if null for protected endpoints";
     if (/identity.*check|if.*identity/i.test(lower))
         return "Add identity null check after getUserIdentity() — return 401/throw for unauthenticated users";
-    // Scheduler
+    // Scheduler (BEFORE generic function types)
     if (/scheduler.*runafter/i.test(lower))
         return "Add ctx.scheduler.runAfter(delayMs, api.module.functionName, args) — use at least 1s delay for retries, add termination condition for loops";
     if (/scheduler.*runat/i.test(lower))
         return "Add ctx.scheduler.runAt(timestamp, api.module.functionName, args) — use for one-time future execution";
-    // Storage
+    // Storage (BEFORE generic function types)
+    if (/storage.*generateuploadurl/i.test(lower))
+        return "Use ctx.storage.generateUploadUrl() in a mutation — return URL to client for direct upload";
     if (/storage.*store/i.test(lower))
         return "Use ctx.storage.store(blob) — returns storageId, save it to a document for later retrieval";
     if (/storage.*geturl/i.test(lower))
         return "Use ctx.storage.getUrl(storageId) — returns URL or null, always null-check the result";
-    if (/storage.*generateuploadurl/i.test(lower))
-        return "Use ctx.storage.generateUploadUrl() in a mutation — return URL to client for direct upload";
-    // Client-side
+    // Client-side (BEFORE generic function types — useQuery before query)
     if (/usequery/i.test(lower))
         return "Add useQuery(api.module.queryName, args) — reactive subscription, re-renders on data change";
     if (/usemutation/i.test(lower))
@@ -502,9 +494,21 @@ function inferConvexStrategy(signature) {
         return "Add useAction(api.module.actionName) — returns async function, use for external API calls";
     if (/convexprovider|convexreactclient/i.test(lower))
         return "Wrap app with <ConvexProvider client={convex}> — create client with new ConvexReactClient(url)";
-    // Cross-function calls
-    if (/runmutation|runquery|runaction/i.test(lower))
-        return "Use ctx.runMutation/ctx.runQuery/ctx.runAction for cross-function calls — prefer internal functions for server-to-server";
+    // Function types (GENERIC — must come AFTER all specific patterns above)
+    if (/internalmutation/i.test(lower))
+        return "Create internalMutation — only callable by other server functions, not from client. Import from _generated/server";
+    if (/internalaction/i.test(lower))
+        return "Create internalAction — for server-side-only work like API calls. Cannot access ctx.db directly, use ctx.runMutation/ctx.runQuery";
+    if (/internalquery/i.test(lower))
+        return "Create internalQuery — server-side-only reads. Not subscribable from client";
+    if (/httpaction/i.test(lower))
+        return "Create httpAction in http.ts — register with httpRouter.route({ method, path, handler })";
+    if (/mutation\b/i.test(lower))
+        return "Create public mutation — add args validator, return validator, and auth check (ctx.auth.getUserIdentity)";
+    if (/action\b/i.test(lower))
+        return "Create public action — for external API calls. Wrap in try/catch, use ctx.runMutation for DB writes";
+    if (/query\b/i.test(lower))
+        return "Create public query — reactive subscription from client. Must be deterministic, no side effects";
     return "Inject this pattern into the appropriate Convex file — check existing patterns with convex_scan_capabilities first";
 }
 function inferTargetFile(signature) {

package/dist/tools/reportingTools.js

@@ -75,7 +75,7 @@ function buildSarif(projectDir, auditTypes, limit) {
                 tool: {
                     driver: {
                         name: "convex-mcp-nodebench",
-                        version: "0.9.0",
+                        version: "0.9.1",
                         informationUri: "https://www.npmjs.com/package/@homenshum/convex-mcp-nodebench",
                         rules: [...rulesMap.values()],
                     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@homenshum/convex-mcp-nodebench",
-  "version": "0.9.0",
+  "version": "0.9.1",
   "description": "Convex-specific MCP server applying NodeBench self-instruct diligence patterns to Convex development. Schema audit, function compliance, deployment gates, persistent gotcha DB, and methodology guidance. Complements Context7 (raw docs) and official Convex MCP (deployment introspection) with structured verification workflows.",
   "type": "module",
   "bin": {