slyplan-mcp 1.7.3 → 1.7.5

package/dist/cli.js

@@ -347,11 +347,10 @@ function mergeMcpJson(existing, auth) {
     else if (auth.refreshToken) {
         env.SLYPLAN_REFRESH_TOKEN = auth.refreshToken;
     }
-    result.mcpServers.slyplan = {
-        command: 'npx',
-        args: ['-y', 'slyplan-mcp@latest'],
-        env,
-    };
+    const isWindows = process.platform === 'win32';
+    result.mcpServers.slyplan = isWindows
+        ? { command: 'cmd', args: ['/c', 'npx', '-y', 'slyplan-mcp@latest'], env }
+        : { command: 'npx', args: ['-y', 'slyplan-mcp@latest'], env };
     return result;
 }
 // Hooks go in .claude/settings.json

package/dist/db.js

@@ -1,5 +1,25 @@
 import { supabase, getUserId } from './supabase.js';
 import { v4 as uuid } from 'uuid';
+// --- Helpers ---
+// Explicit column lists to reduce Supabase egress (avoid select('*'))
+const NODE_COLUMNS = 'id,parent_id,type,title,description,status,progress,sort_order,collapsed,metadata,created_by,created_at,updated_at';
+const LINK_COLUMNS = 'id,node_id,url,label,link_type,created_at';
+/** Batch-fetch links for multiple node IDs in a single query */
+async function batchGetLinks(nodeIds) {
+    const linksByNode = new Map();
+    if (nodeIds.length === 0)
+        return linksByNode;
+    const { data: linkRows } = await supabase
+        .from('links')
+        .select(LINK_COLUMNS)
+        .in('node_id', nodeIds);
+    for (const link of (linkRows || [])) {
+        const arr = linksByNode.get(link.node_id) || [];
+        arr.push(dbLinkToLink(link));
+        linksByNode.set(link.node_id, arr);
+    }
+    return linksByNode;
+}
 function dbNodeToTree(row, links, children) {
     return {
         id: row.id,
@@ -32,17 +52,17 @@ function dbLinkToLink(row) {
 export async function getTree(rootId, maxDepth) {
     const { data: allNodes, error: nodesErr } = await supabase
         .from('nodes')
-        .select('*')
+        .select(NODE_COLUMNS)
         .order('sort_order', { ascending: true });
     if (nodesErr)
         throw new Error(nodesErr.message);
     const { data: allLinks, error: linksErr } = await supabase
         .from('links')
-        .select('*');
+        .select(LINK_COLUMNS);
     if (linksErr)
         throw new Error(linksErr.message);
     const linksByNode = new Map();
-    for (const link of allLinks || []) {
+    for (const link of (allLinks || [])) {
         const arr = linksByNode.get(link.node_id) || [];
         arr.push(dbLinkToLink(link));
         linksByNode.set(link.node_id, arr);
@@ -88,16 +108,13 @@ export async function getTree(rootId, maxDepth) {
 export async function getNode(id, maxDepth = 1) {
     const { data: row, error } = await supabase
         .from('nodes')
-        .select('*')
+        .select(NODE_COLUMNS)
         .eq('id', id)
         .single();
     if (error || !row)
         return null;
-    const { data: linkRows } = await supabase
-        .from('links')
-        .select('*')
-        .eq('node_id', id);
-    const links = (linkRows || []).map(dbLinkToLink);
+    const linkMap = await batchGetLinks([id]);
+    const links = linkMap.get(id) || [];
     const children = maxDepth > 0 ? await getChildrenRecursive(id, maxDepth, 1) : [];
     let progress = row.progress;
     if (children.length > 0) {
@@ -108,15 +125,17 @@ export async function getNode(id, maxDepth = 1) {
 async function getChildrenRecursive(parentId, maxDepth, currentDepth) {
     const { data: childRows } = await supabase
         .from('nodes')
-        .select('*')
+        .select(NODE_COLUMNS)
         .eq('parent_id', parentId)
         .order('sort_order', { ascending: true });
+    const typedChildren = (childRows || []);
+    if (typedChildren.length === 0)
+        return [];
+    // Batch-fetch all links for this level in ONE query instead of N+1
+    const childIds = typedChildren.map(c => c.id);
+    const linkMap = await batchGetLinks(childIds);
     const children = [];
-    for (const child of (childRows || [])) {
-        const { data: childLinkRows } = await supabase
-            .from('links')
-            .select('*')
-            .eq('node_id', child.id);
+    for (const child of typedChildren) {
         const grandchildren = currentDepth < maxDepth
             ? await getChildrenRecursive(child.id, maxDepth, currentDepth + 1)
             : [];
@@ -124,7 +143,7 @@ async function getChildrenRecursive(parentId, maxDepth, currentDepth) {
         if (grandchildren.length > 0) {
             progress = Math.round(grandchildren.reduce((sum, c) => sum + c.progress, 0) / grandchildren.length);
         }
-        children.push(dbNodeToTree({ ...child, progress }, (childLinkRows || []).map(dbLinkToLink), grandchildren));
+        children.push(dbNodeToTree({ ...child, progress }, linkMap.get(child.id) || [], grandchildren));
     }
     return children;
 }
@@ -305,17 +324,11 @@ export async function getWorkMode() {
     const nodeIds = wmRows.map(r => r.node_id);
     const { data: nodeRows } = await supabase
         .from('nodes')
-        .select('*')
+        .select(NODE_COLUMNS)
         .in('id', nodeIds);
-    const results = [];
-    for (const row of (nodeRows || [])) {
-        const { data: linkRows } = await supabase
-            .from('links')
-            .select('*')
-            .eq('node_id', row.id);
-        results.push(dbNodeToTree(row, (linkRows || []).map(dbLinkToLink), []));
-    }
-    return results;
+    // Batch-fetch all links in ONE query instead of N+1
+    const linkMap = await batchGetLinks(nodeIds);
+    return (nodeRows || []).map(row => dbNodeToTree(row, linkMap.get(row.id) || [], []));
 }
 export async function addToWorkMode(nodeId) {
     const userId = getUserId();
@@ -469,7 +482,7 @@ export async function removeFromWorkMode(nodeId) {
 export async function searchNodes(query, filters) {
     let q = supabase
         .from('nodes')
-        .select('*')
+        .select(NODE_COLUMNS)
         .or(`title.ilike.%${query}%,description.ilike.%${query}%`)
         .order('sort_order', { ascending: true })
         .limit(50);
@@ -524,7 +537,7 @@ export async function deleteDependency(id) {
 export async function listDependencies(nodeIds) {
     const { data, error } = await supabase
         .from('node_dependencies')
-        .select('*')
+        .select('id,from_node_id,to_node_id,dependency_type,created_by,created_at')
         .or(`from_node_id.in.(${nodeIds.join(',')}),to_node_id.in.(${nodeIds.join(',')})`);
     if (error)
         throw new Error(error.message);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "slyplan-mcp",
-  "version": "1.7.3",
+  "version": "1.7.5",
   "description": "MCP server for Slyplan — visual project management via Claude",
   "type": "module",
   "bin": {