@artyfacts/claude 1.3.3 → 1.3.5

package/dist/cli.js

@@ -789,12 +789,13 @@ function configureMcp(apiKey, baseUrl) {
     const result = (0, import_child_process2.spawnSync)("claude", [
       "mcp",
       "add",
+      "artyfacts",
+      // Name first!
       "-s",
       "user",
       // Available across all projects
-      `-e`,
+      "-e",
       `ARTYFACTS_API_KEY=${apiKey}`,
-      "artyfacts",
       "--",
       "npx",
       "-y",
@@ -896,7 +897,7 @@ async function checkAndClaimTasks(baseUrl, apiKey, agentId, activeTasks, executo
     }
     console.log(`\u{1F4EC} Found ${tasks.length} claimable task(s)`);
     for (const task of tasks) {
-      if (activeTasks.has(task.section_id)) {
+      if (activeTasks.has(task.id)) {
         continue;
       }
       console.log(`
@@ -919,11 +920,11 @@ async function checkAndClaimTasks(baseUrl, apiKey, agentId, activeTasks, executo
       });
       const taskUuid = claimData.task?.id || task.id;
       console.log(`  \u{1F4CE} Using task UUID: ${taskUuid} (from queue: ${task.id})`);
-      activeTasks.add(task.section_id);
+      activeTasks.add(task.id);
       console.log("  \u2713 Claimed!");
       if (dryRun) {
         console.log("  \u{1F4CB} Dry run - not executing");
-        activeTasks.delete(task.section_id);
+        activeTasks.delete(task.id);
         continue;
       }
       console.log("  \u2192 Executing with Claude...");
@@ -960,7 +961,7 @@ async function checkAndClaimTasks(baseUrl, apiKey, agentId, activeTasks, executo
       } catch (error) {
         console.error(`  \u2192 \u274C Error:`, error instanceof Error ? error.message : error);
       } finally {
-        activeTasks.delete(task.section_id);
+        activeTasks.delete(task.id);
       }
       break;
     }

package/dist/cli.mjs

@@ -30,12 +30,13 @@ function configureMcp(apiKey, baseUrl) {
     const result = spawnSync("claude", [
       "mcp",
       "add",
+      "artyfacts",
+      // Name first!
       "-s",
       "user",
       // Available across all projects
-      `-e`,
+      "-e",
       `ARTYFACTS_API_KEY=${apiKey}`,
-      "artyfacts",
       "--",
       "npx",
       "-y",
@@ -137,7 +138,7 @@ async function checkAndClaimTasks(baseUrl, apiKey, agentId, activeTasks, executo
     }
     console.log(`\u{1F4EC} Found ${tasks.length} claimable task(s)`);
     for (const task of tasks) {
-      if (activeTasks.has(task.section_id)) {
+      if (activeTasks.has(task.id)) {
         continue;
       }
       console.log(`
@@ -160,11 +161,11 @@ async function checkAndClaimTasks(baseUrl, apiKey, agentId, activeTasks, executo
       });
       const taskUuid = claimData.task?.id || task.id;
       console.log(`  \u{1F4CE} Using task UUID: ${taskUuid} (from queue: ${task.id})`);
-      activeTasks.add(task.section_id);
+      activeTasks.add(task.id);
       console.log("  \u2713 Claimed!");
       if (dryRun) {
         console.log("  \u{1F4CB} Dry run - not executing");
-        activeTasks.delete(task.section_id);
+        activeTasks.delete(task.id);
         continue;
       }
       console.log("  \u2192 Executing with Claude...");
@@ -201,7 +202,7 @@ async function checkAndClaimTasks(baseUrl, apiKey, agentId, activeTasks, executo
       } catch (error) {
         console.error(`  \u2192 \u274C Error:`, error instanceof Error ? error.message : error);
       } finally {
-        activeTasks.delete(task.section_id);
+        activeTasks.delete(task.id);
       }
       break;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@artyfacts/claude",
-  "version": "1.3.3",
+  "version": "1.3.5",
   "description": "Claude adapter for Artyfacts - Execute tasks using Claude Code CLI",
   "main": "dist/index.js",
   "module": "dist/index.mjs",

package/src/cli.ts

@@ -50,12 +50,13 @@ function configureMcp(apiKey: string, baseUrl: string): boolean {
   
   try {
     // Add MCP server with environment variable
-    // Format: claude mcp add -e KEY=value name -- command
+    // Format: claude mcp add <name> -s user -e KEY=value -- command
+    // (name must come before options!)
     const result = spawnSync('claude', [
       'mcp', 'add',
+      'artyfacts',  // Name first!
       '-s', 'user',  // Available across all projects
-      `-e`, `ARTYFACTS_API_KEY=${apiKey}`,
-      'artyfacts',
+      '-e', `ARTYFACTS_API_KEY=${apiKey}`,
       '--',
       'npx', '-y', '@artyfacts/mcp-server',
     ], {
@@ -244,7 +245,8 @@ async function checkAndClaimTasks(
 
     // Process first available task
     for (const task of tasks) {
-      if (activeTasks.has(task.section_id)) {
+      // Use task.id (UUID) for deduplication - matches SSE taskId
+      if (activeTasks.has(task.id)) {
         continue; // Already processing
       }
 
@@ -280,12 +282,12 @@ async function checkAndClaimTasks(
       console.log(`  📎 Using task UUID: ${taskUuid} (from queue: ${task.id})`);  // DEBUG
 
       // Successfully claimed - now execute
-      activeTasks.add(task.section_id);
+      activeTasks.add(task.id);
       console.log('  ✓ Claimed!');
 
       if (dryRun) {
         console.log('  📋 Dry run - not executing');
-        activeTasks.delete(task.section_id);
+        activeTasks.delete(task.id);
         continue;
       }
 
@@ -323,7 +325,7 @@ async function checkAndClaimTasks(
       } catch (error) {
         console.error(`  → ❌ Error:`, error instanceof Error ? error.message : error);
       } finally {
-        activeTasks.delete(task.section_id);
+        activeTasks.delete(task.id);
       }
 
       // Only process one task at a time for now