superproductivity-mcp 1.2.6.dev7__tar.gz → 1.2.6.dev8__tar.gz

{superproductivity_mcp-1.2.6.dev7 → superproductivity_mcp-1.2.6.dev8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: superproductivity-mcp
-Version: 1.2.6.dev7
+Version: 1.2.6.dev8
 Summary: Super Productivity MCP server for Claude Desktop integration
 Project-URL: Homepage, https://github.com/ben-elliot-nice/superproductivity-mcp
 License-File: LICENSE.txt

{superproductivity_mcp-1.2.6.dev7 → superproductivity_mcp-1.2.6.dev8}/plugin/manifest.json

@@ -2,7 +2,7 @@
   "id": "sp-mcp-bridge",
   "name": "MCP Bridge",
   "description": "Bridge between Super Productivity and MCP (Model Context Protocol) servers for Claude Desktop integration",
-  "version": "1.2.6.dev7",
+  "version": "1.2.6.dev8",
   "manifestVersion": 1,
   "author": "Super Productivity Team",
   "homepage": "https://github.com/johannesjo/super-productivity",

{superproductivity_mcp-1.2.6.dev7 → superproductivity_mcp-1.2.6.dev8}/plugin/plugin.js

@@ -525,26 +525,38 @@ class MCPBridgePlugin {
           const allTasks = await PluginAPI.getTasks();
           const taskMap = new Map(allTasks.map(t => [t.id, t]));
           const fixes = [];
+          const errors = [];
+
+          // Build a map of parentId → [childIds that have parentId set but are missing from parent's subTaskIds]
+          // Root cause: concurrent create_tasks calls race on subTaskIds updates — last write wins,
+          // leaving orphaned children with parentId set but not in parent's subTaskIds array.
+          const missingFromParent = new Map(); // parentId → Set of childIds to add
           for (const task of allTasks) {
-            // Fix 1: child is in parent's subTaskIds but has no parentId set
-            for (const childId of (task.subTaskIds || [])) {
-              const child = taskMap.get(childId);
-              if (child && !child.parentId) {
-                await PluginAPI.updateTask(childId, { parentId: task.id });
-                fixes.push(`set parentId on "${child.title}" → parent "${task.title}"`);
-              }
-            }
-            // Fix 2: child has parentId set but is missing from parent's subTaskIds
             if (task.parentId) {
               const parent = taskMap.get(task.parentId);
               if (parent && !(parent.subTaskIds || []).includes(task.id)) {
-                const updated = [...(parent.subTaskIds || []), task.id];
-                await PluginAPI.updateTask(task.parentId, { subTaskIds: updated });
-                fixes.push(`added "${task.title}" to subTaskIds of parent "${parent.title}"`);
+                if (!missingFromParent.has(task.parentId)) {
+                  missingFromParent.set(task.parentId, new Set());
+                }
+                missingFromParent.get(task.parentId).add(task.id);
               }
             }
           }
-          result = { fixed: fixes.length, fixes };
+
+          // For each parent, do ONE update with ALL missing children at once to avoid race conditions
+          for (const [parentId, missingIds] of missingFromParent) {
+            const parent = taskMap.get(parentId);
+            try {
+              const updated = [...(parent.subTaskIds || []), ...missingIds];
+              await PluginAPI.updateTask(parentId, { subTaskIds: updated });
+              const childTitles = [...missingIds].map(id => taskMap.get(id)?.title || id);
+              fixes.push(`added ${missingIds.size} missing children to "${parent.title}": ${childTitles.join(', ')}`);
+            } catch (e) {
+              errors.push(`failed to fix parent "${parent?.title}": ${e.message}`);
+            }
+          }
+
+          result = { fixed: fixes.length, fixes, errors };
           break;
         }
 

{superproductivity_mcp-1.2.6.dev7 → superproductivity_mcp-1.2.6.dev8}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "superproductivity-mcp"
-version = "1.2.6.dev7"
+version = "1.2.6.dev8"
 description = "Super Productivity MCP server for Claude Desktop integration"
 readme = "README.md"
 requires-python = ">=3.10"