secufusion-mcp 1.0.16 → 1.0.17

package/index.js

@@ -842,26 +842,34 @@ server.tool("manage_project_spec", "Manages the .secufusion-project-spec.json fi
 // ─── Task management helpers ──────────────────────────────────────────────────
 const TASKS_DIR = ".secufusion/tasks";
 const REGISTRY_FILE = ".secufusion/registry.json";
-function taskDir(workItemId, title) {
-    const baseDir = resolve(TASKS_DIR);
-    const prefixes = [workItemId + "-", "WI-" + workItemId + "-"];
-    if (fs.existsSync(baseDir)) {
-        const folders = fs.readdirSync(baseDir);
-        // Find an existing folder that starts with the workItemId
-        const match = folders.find(f => f === workItemId ||
-            f === `WI-${workItemId}` ||
-            prefixes.some(p => f.startsWith(p)));
-        if (match) {
-            return resolve(`${TASKS_DIR}/${match}`);
-        }
-    }
-    // If no folder exists, generate a new one
-    if (title) {
-        const slug = title.toLowerCase().replace(/[^a-z0-9]+/g, '-').replace(/^-|-$/g, '').substring(0, 50);
-        return resolve(`${TASKS_DIR}/${workItemId}-${slug}`);
-    }
-    // Fallback if no title is provided and folder doesn't exist
-    return resolve(`${TASKS_DIR}/${workItemId}`);
+function resolveTaskFolder(workItemId, tasksRoot, registryData) {
+    const registry = registryData || readRegistry();
+    const entry = registry.find(t => t["work_item_id"] === workItemId);
+    // 1. Try registry first
+    if (entry && entry["folder"]) {
+        const folderPath = path.join(tasksRoot, entry["folder"]);
+        if (fs.existsSync(folderPath))
+            return folderPath;
+    }
+    // 2. Fallback: scan directory
+    if (fs.existsSync(tasksRoot)) {
+        const taskFolders = fs.readdirSync(tasksRoot);
+        const match = taskFolders.find(f => f.startsWith(workItemId + "-") || f === workItemId || f === `WI-${workItemId}`);
+        if (match)
+            return path.join(tasksRoot, match);
+    }
+    // 3. Not found
+    return null;
+}
+function generateNewTaskFolder(workItemId, title, tasksRoot) {
+    const slug = title.toLowerCase()
+        .replace(/[:\s]+/g, '-')
+        .replace(/[^a-z0-9-]+/g, '')
+        .replace(/-+/g, '-')
+        .replace(/^-|-$/g, '')
+        .slice(0, 50)
+        .replace(/-+$/, '');
+    return path.join(tasksRoot, `${workItemId}-${slug}`);
 }
 function readJson(filePath) {
     const raw = readFileSafe(filePath);
@@ -910,7 +918,17 @@ server.tool("manage_task", "Manages structured task records in .secufusion/tasks
     scenario_type: z.enum(["unit", "integration", "e2e", "manual"]).optional().describe("Test scenario type. Used with 'add_scenario'."),
 }, async ({ action, work_item_id, title, description, acceptance_criteria, pending_acs, completed_acs, next_step, tags, file_path, change_summary, decision, rationale, scenario, scenario_type }) => {
     const inputChars = JSON.stringify({ action, work_item_id, title, description, acceptance_criteria, pending_acs, completed_acs, next_step, tags, file_path, change_summary, decision, rationale, scenario, scenario_type }).length;
-    const dir = taskDir(work_item_id, action === "initialize" ? title : undefined);
+    const tasksRoot = resolve(TASKS_DIR);
+    let dir = resolveTaskFolder(work_item_id, tasksRoot);
+    if (action === "initialize") {
+        if (!title)
+            return appendTelemetry({ isError: true, content: [{ type: "text", text: "ERROR: 'title' is required for action=initialize." }] }, inputChars);
+        if (!dir)
+            dir = generateNewTaskFolder(work_item_id, title, tasksRoot);
+    }
+    if (!dir && action !== "initialize") {
+        return appendTelemetry({ isError: true, content: [{ type: "text", text: `Task WI-${work_item_id} not found in ${tasksRoot}. Has it been initialized?` }] }, inputChars);
+    }
     const specPath = path.join(dir, "spec.json");
     const progressPath = path.join(dir, "progress.json");
     const decisionsPath = path.join(dir, "decisions.json");
@@ -930,7 +948,7 @@ server.tool("manage_task", "Manages structured task records in .secufusion/tasks
             writeFile(filesPath, JSON.stringify([], null, 2));
             writeFile(scenariosPath, JSON.stringify([], null, 2));
         }
-        upsertRegistry({ work_item_id, title, status: "active", created_at: now, tags: tags || [] });
+        upsertRegistry({ work_item_id, title, status: "active", created_at: now, tags: tags || [], folder: path.basename(dir) });
         return appendTelemetry({ content: [{ type: "text", text: `✅ Task WI-${work_item_id} initialized.\n\n**Folder:** ${dir}\n**Title:** ${title}\n**ACs:** ${(acceptance_criteria || []).length}\n\n${JSON.stringify(spec, null, 2)}` }] }, inputChars);
     }
     if (!fs.existsSync(specPath)) {
@@ -1032,9 +1050,10 @@ server.tool("manage_task", "Manages structured task records in .secufusion/tasks
 // ─── Tool 6: get_task_history ─────────────────────────────────────────────────
 server.tool("get_task_history", "Retrieve the full history of a past task — spec, decisions, files touched, and scenarios. Use before starting similar work to understand how it was done before.", { work_item_id: z.string().describe("Work item ID of the past task to retrieve.") }, async ({ work_item_id }) => {
     const inputChars = JSON.stringify({ work_item_id }).length;
-    const dir = taskDir(work_item_id);
-    if (!fs.existsSync(dir)) {
-        return appendTelemetry({ isError: true, content: [{ type: "text", text: `Task WI-${work_item_id} not found in ${resolve(TASKS_DIR)}.` }] }, inputChars);
+    const tasksRoot = resolve(TASKS_DIR);
+    const dir = resolveTaskFolder(work_item_id, tasksRoot);
+    if (!dir) {
+        return appendTelemetry({ isError: true, content: [{ type: "text", text: `Task WI-${work_item_id} not found in ${tasksRoot}. Has it been initialized? Run manage_task action=initialize first.` }] }, inputChars);
     }
     const history = {
         spec: readJson(path.join(dir, "spec.json")),
@@ -1056,10 +1075,13 @@ server.tool("search_tasks", "Search all past tasks by keyword. Scans registry an
         return appendTelemetry({ content: [{ type: "text", text: "No tasks found in registry. Initialize tasks first." }] }, inputChars);
     const terms = keywords.toLowerCase().split(/\s+/).filter(Boolean);
     const matches = [];
+    const tasksRoot = resolve(TASKS_DIR);
     for (const entry of registry) {
         if (status_filter !== "any" && entry["status"] !== status_filter)
             continue;
-        const dir = taskDir(entry["work_item_id"]);
+        const dir = resolveTaskFolder(entry["work_item_id"], tasksRoot, registry);
+        if (!dir)
+            continue;
         const blobs = [
             JSON.stringify(entry),
             readFileSafe(path.join(dir, "spec.json")) || "",
@@ -1082,8 +1104,9 @@ server.tool("search_tasks", "Search all past tasks by keyword. Scans registry an
 // ─── Tool 8: get_pattern_from_task ────────────────────────────────────────────
 server.tool("get_pattern_from_task", "Extract reusable implementation patterns from a completed task — decisions made, files modified, and test scenarios used. Call this before implementing something similar to a past feature.", { work_item_id: z.string().describe("Work item ID of the past task to extract patterns from.") }, async ({ work_item_id }) => {
     const inputChars = JSON.stringify({ work_item_id }).length;
-    const dir = taskDir(work_item_id);
-    if (!fs.existsSync(dir))
+    const tasksRoot = resolve(TASKS_DIR);
+    const dir = resolveTaskFolder(work_item_id, tasksRoot);
+    if (!dir)
         return appendTelemetry({ isError: true, content: [{ type: "text", text: `Task WI-${work_item_id} not found. Cannot extract patterns.` }] }, inputChars);
     const spec = readJson(path.join(dir, "spec.json")) || {};
     const decisions = readJson(path.join(dir, "decisions.json")) || [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "secufusion-mcp",
-  "version": "1.0.16",
+  "version": "1.0.17",
   "type": "module",
   "description": "SecuFusion MCP server - developer workflow tooling with guardrails",
   "main": "index.js",