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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: superproductivity-mcp
-Version: 1.2.6.dev6
+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.dev6 → 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.dev6",
+  "version": "1.2.6.dev8",
   "manifestVersion": 1,
   "author": "Super Productivity Team",
   "homepage": "https://github.com/johannesjo/super-productivity",

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

@@ -485,32 +485,20 @@ class MCPBridgePlugin {
           
         case 'addTask':
           if (command.data.parentId) {
-            // SP's addTask double-writes when parentId is supplied at creation time.
-            // Strategy: create WITHOUT parentId, then set parentId via updateTask (hides
-            // child from root), then add to parent's subTaskIds (produces nested display).
             const { parentId, title, ...rest } = command.data;
-            // Strip SP scheduling/tag syntax from title at creation if present, to prevent
-            // SP from parsing @project/#tag/+parent during addTask. Restore via update.
+            // Strip SP scheduling/tag syntax (@, #, +) from title at creation to prevent
+            // SP parsing it during addTask. Restore via a separate updateTask on title only.
             const hasSyntax = /[@#+]/.test(title);
             const createTitle = hasSyntax
               ? title.replace(/@\w+/g, '').replace(/#\w+/g, '').replace(/\+\w+/g, '').trim()
               : title;
-            // Create without parentId to avoid SP's addTask double-write bug.
             await this.log(`Creating subtask under ${parentId}`, 'debug');
-            const subtaskId = await PluginAPI.addTask({ ...rest, title: createTitle });
-            // Step 1: restore title syntax if stripped, and set parentId via updateTask.
-            // Setting parentId via updateTask (not addTask) avoids the double-write bug
-            // while still hiding the child from SP's root list.
-            const updatePayload = hasSyntax ? { parentId, title } : { parentId };
-            await PluginAPI.updateTask(subtaskId, updatePayload);
-            // Step 2: add child to parent's subTaskIds so SP nests it correctly.
-            const allTasks = await PluginAPI.getTasks();
-            const parentTask = allTasks.find(t => t.id === parentId);
-            if (parentTask) {
-              const existing = parentTask.subTaskIds || [];
-              if (!existing.includes(subtaskId)) {
-                await PluginAPI.updateTask(parentId, { subTaskIds: [...existing, subtaskId] });
-              }
+            // Pass parentId to addTask directly — SP creates a true subtask at creation
+            // time and handles hierarchy correctly. updateTask({ parentId }) creates a
+            // root-level cross-reference label instead of a true nested subtask.
+            const subtaskId = await PluginAPI.addTask({ ...rest, title: createTitle, parentId });
+            if (hasSyntax) {
+              await PluginAPI.updateTask(subtaskId, { title });
             }
             result = subtaskId;
           } else {
@@ -537,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.dev6 → superproductivity_mcp-1.2.6.dev8}/pyproject.toml

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