@allpepper/task-orchestrator 1.0.3 → 1.1.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@allpepper/task-orchestrator",
-  "version": "1.0.3",
+  "version": "1.1.0",
   "description": "MCP server for hierarchical task orchestration with SQLite persistence",
   "type": "module",
   "main": "src/index.ts",

package/src/repos/base.ts

@@ -149,3 +149,19 @@ export function countTasksByProject(projectId: string): TaskCounts {
   }
   return { total, byStatus };
 }
+
+export function countFeaturesByProject(projectId: string): number {
+  const row = queryOne<{ count: number }>(
+    'SELECT COUNT(*) as count FROM features WHERE project_id = ?',
+    [projectId]
+  );
+  return row?.count ?? 0;
+}
+
+export function countTasksByFeatureId(featureId: string): number {
+  const row = queryOne<{ count: number }>(
+    'SELECT COUNT(*) as count FROM tasks WHERE feature_id = ?',
+    [featureId]
+  );
+  return row?.count ?? 0;
+}

package/src/repos/features.ts

@@ -343,16 +343,51 @@ export function updateFeature(
 /**
  * Delete a feature
  */
-export function deleteFeature(id: string): Result<boolean> {
+export function deleteFeature(id: string, options?: { cascade?: boolean }): Result<boolean> {
   try {
+    const cascade = options?.cascade ?? false;
+
+    // Check if feature exists
+    const exists = queryOne<{ id: string }>('SELECT id FROM features WHERE id = ?', [id]);
+
+    if (!exists) {
+      throw new NotFoundError('Feature', id);
+    }
+
+    // Count children
+    const taskCounts = countTasksByFeature(id);
+    const taskCount = taskCounts.total;
+
+    // If children exist and no cascade, return error with counts
+    if (taskCount > 0 && !cascade) {
+      return err(
+        `Cannot delete feature: contains ${taskCount} task${taskCount > 1 ? 's' : ''}. Use cascade: true to delete all.`,
+        'HAS_CHILDREN'
+      );
+    }
+
     const result = transaction(() => {
-      // Check if feature exists
-      const exists = queryOne<{ id: string }>('SELECT id FROM features WHERE id = ?', [id]);
+      if (cascade) {
+        // Get all task IDs for this feature
+        const taskIds = queryAll<{ id: string }>(
+          'SELECT id FROM tasks WHERE feature_id = ?',
+          [id]
+        );
 
-      if (!exists) {
-        throw new NotFoundError('Feature', id);
+        // Delete each task's dependencies, sections, and tags
+        for (const task of taskIds) {
+          execute('DELETE FROM dependencies WHERE from_task_id = ? OR to_task_id = ?', [task.id, task.id]);
+          execute('DELETE FROM sections WHERE entity_type = ? AND entity_id = ?', [EntityType.TASK, task.id]);
+          deleteTags(task.id, EntityType.TASK);
+        }
+
+        // Delete all tasks for this feature
+        execute('DELETE FROM tasks WHERE feature_id = ?', [id]);
       }
 
+      // Delete feature sections
+      execute('DELETE FROM sections WHERE entity_type = ? AND entity_id = ?', [EntityType.FEATURE, id]);
+
       // Delete associated tags
       deleteTags(id, EntityType.FEATURE);
 

package/src/repos/projects.ts

@@ -1,6 +1,6 @@
-import { queryOne, queryAll, execute, generateId, now, buildSearchVector, loadTags, saveTags, deleteTags, ok, err, buildPaginationClause, countTasksByProject, type TaskCounts } from './base';
+import { queryOne, queryAll, execute, generateId, now, buildSearchVector, loadTags, saveTags, deleteTags, ok, err, buildPaginationClause, countTasksByProject, countFeaturesByProject, type TaskCounts } from './base';
 import type { Project, Result } from '../domain/types';
-import { ProjectStatus, NotFoundError, ConflictError, ValidationError } from '../domain/types';
+import { ProjectStatus, NotFoundError, ConflictError, ValidationError, EntityType } from '../domain/types';
 import { transaction } from '../db/client';
 import { isValidTransition, getAllowedTransitions, isTerminalStatus } from '../services/status-validator';
 
@@ -221,20 +221,75 @@ export function updateProject(
   }
 }
 
-export function deleteProject(id: string): Result<boolean> {
+export function deleteProject(id: string, options?: { cascade?: boolean }): Result<boolean> {
   try {
-    const result = transaction(() => {
-      const existing = queryOne<ProjectRow>(
-        'SELECT id FROM projects WHERE id = ?',
-        [id]
+    const cascade = options?.cascade ?? false;
+
+    // Check if project exists
+    const existing = queryOne<ProjectRow>(
+      'SELECT id FROM projects WHERE id = ?',
+      [id]
+    );
+
+    if (!existing) {
+      throw new NotFoundError('Project', id);
+    }
+
+    // Count children
+    const featureCount = countFeaturesByProject(id);
+    const taskCounts = countTasksByProject(id);
+    const taskCount = taskCounts.total;
+
+    // If children exist and no cascade, return error with counts
+    if ((featureCount > 0 || taskCount > 0) && !cascade) {
+      const parts: string[] = [];
+      if (featureCount > 0) parts.push(`${featureCount} feature${featureCount > 1 ? 's' : ''}`);
+      if (taskCount > 0) parts.push(`${taskCount} task${taskCount > 1 ? 's' : ''}`);
+      return err(
+        `Cannot delete project: contains ${parts.join(' and ')}. Use cascade: true to delete all.`,
+        'HAS_CHILDREN'
       );
+    }
 
-      if (!existing) {
-        throw new NotFoundError('Project', id);
+    const result = transaction(() => {
+      if (cascade) {
+        // Get all task IDs for this project
+        const taskIds = queryAll<{ id: string }>(
+          'SELECT id FROM tasks WHERE project_id = ?',
+          [id]
+        );
+
+        // Delete each task's dependencies, sections, and tags
+        for (const task of taskIds) {
+          execute('DELETE FROM dependencies WHERE from_task_id = ? OR to_task_id = ?', [task.id, task.id]);
+          execute('DELETE FROM sections WHERE entity_type = ? AND entity_id = ?', [EntityType.TASK, task.id]);
+          deleteTags(task.id, EntityType.TASK);
+        }
+
+        // Delete all tasks for this project
+        execute('DELETE FROM tasks WHERE project_id = ?', [id]);
+
+        // Get all feature IDs for this project
+        const featureIds = queryAll<{ id: string }>(
+          'SELECT id FROM features WHERE project_id = ?',
+          [id]
+        );
+
+        // Delete each feature's sections and tags
+        for (const feature of featureIds) {
+          execute('DELETE FROM sections WHERE entity_type = ? AND entity_id = ?', [EntityType.FEATURE, feature.id]);
+          deleteTags(feature.id, EntityType.FEATURE);
+        }
+
+        // Delete all features for this project
+        execute('DELETE FROM features WHERE project_id = ?', [id]);
       }
 
-      // Delete tags first
-      deleteTags(id, 'PROJECT');
+      // Delete project sections
+      execute('DELETE FROM sections WHERE entity_type = ? AND entity_id = ?', [EntityType.PROJECT, id]);
+
+      // Delete project tags
+      deleteTags(id, EntityType.PROJECT);
 
       // Delete project
       execute('DELETE FROM projects WHERE id = ?', [id]);

package/src/tools/manage-container.ts

@@ -52,6 +52,7 @@ export function registerManageContainerTool(server: McpServer): void {
       complexity: z.number().int().optional(),
       tags: z.string().optional(),
       version: z.number().int().optional(),
+      cascade: z.boolean().optional().describe('For delete operation: if true, deletes all child entities (features, tasks) recursively. Required when deleting a project or feature that has children.'),
     },
     async (params) => {
       try {
@@ -235,11 +236,13 @@ export function registerManageContainerTool(server: McpServer): void {
             };
           }
 
+          const cascadeOption = params.cascade ? { cascade: true } : undefined;
+
           let result;
           if (containerType === 'project') {
-            result = deleteProject(id);
+            result = deleteProject(id, cascadeOption);
           } else if (containerType === 'feature') {
-            result = deleteFeature(id);
+            result = deleteFeature(id, cascadeOption);
           } else {
             result = deleteTask(id);
           }