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

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

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

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

@@ -525,26 +525,62 @@ 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);
+              }
+            }
+          }
+
+          // Step 1: For each parent, add ALL missing children to subTaskIds in one update
+          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(`subTaskIds: added ${missingIds.size} children to "${parent.title}"`);
+            } catch (e) {
+              errors.push(`failed to fix subTaskIds for "${taskMap.get(parentId)?.title}": ${e.message}`);
+            }
+          }
+
+          // Step 2: Remove subtasks from project taskIds — tasks created without parentId
+          // get added to project.taskIds by SP; later setting parentId doesn't remove them,
+          // causing double-display (root AND nested). Fix: remove any task with parentId from
+          // its project's taskIds, grouped by project to avoid multiple updates per project.
+          const projects = await PluginAPI.getAllProjects();
+          for (const project of projects) {
+            const subtaskIdsInProject = (project.taskIds || []).filter(id => {
+              const t = taskMap.get(id);
+              return t && t.parentId;
+            });
+            if (subtaskIdsInProject.length > 0) {
+              try {
+                const cleaned = (project.taskIds || []).filter(id => {
+                  const t = taskMap.get(id);
+                  return !t || !t.parentId;
+                });
+                await PluginAPI.updateProject(project.id, { taskIds: cleaned });
+                fixes.push(`project taskIds: removed ${subtaskIdsInProject.length} subtasks from "${project.title}"`);
+              } catch (e) {
+                errors.push(`failed to fix taskIds for project "${project.title}": ${e.message}`);
               }
             }
           }
-          result = { fixed: fixes.length, fixes };
+
+          result = { fixed: fixes.length, fixes, errors };
           break;
         }
 

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

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