@masslessai/push-todo 4.0.0 → 4.0.2

package/lib/daemon.js

@@ -105,6 +105,17 @@ const taskStderrBuffer = new Map(); // displayNumber -> lines[]
 const taskProjectPaths = new Map(); // displayNumber -> projectPath
 let daemonStartTime = null;
 
+// ==================== Utilities ====================
+
+function parseJsonField(value) {
+  if (!value) return [];
+  if (Array.isArray(value)) return value;
+  if (typeof value === 'string') {
+    try { return JSON.parse(value); } catch { return []; }
+  }
+  return [];
+}
+
 // ==================== Logging ====================
 
 function rotateLogs() {
@@ -1000,6 +1011,24 @@ async function markTaskAsCompleted(displayNumber, taskId, comment) {
   }
 }
 
+async function hasApprovedConfirmation(displayNumber) {
+  try {
+    const response = await apiRequest(`synced-todos?display_number=${displayNumber}`);
+    if (!response.ok) return false;
+    const data = await response.json();
+    const todos = data.todos || [];
+    if (todos.length === 0) return false;
+    const events = parseJsonField(todos[0].executionEventsJson);
+    for (let i = events.length - 1; i >= 0; i--) {
+      if (events[i].type === 'confirmation_approved') return true;
+      if (events[i].type === 'confirmation_rejected') return false;
+    }
+    return false;
+  } catch {
+    return false;
+  }
+}
+
 /**
  * Auto-heal: detect if a previous execution already completed work for this task.
  * Checks for existing branch commits and PRs to avoid redundant re-execution.
@@ -1386,10 +1415,37 @@ async function executeTask(task) {
 
   taskProjectPaths.set(displayNumber, projectPath);
 
+  // Build attachment context for the prompt (screenshots, links)
+  let attachmentContext = '';
+  try {
+    const links = parseJsonField(task.linkAttachmentsJson || task.linkAttachments || task.link_attachments);
+    const screenshots = parseJsonField(task.screenshotAttachmentsJson || task.screenshotAttachments || task.screenshot_attachments);
+    const contextApp = task.contextApp || task.context_app || null;
+
+    const parts = [];
+    if (contextApp) {
+      parts.push(`Context app: ${contextApp}`);
+    }
+    if (links.length > 0) {
+      parts.push('Links:\n' + links.map(l => `  - ${l.url}${l.title ? ` (${l.title})` : ''}`).join('\n'));
+    }
+    if (screenshots.length > 0) {
+      parts.push('Screenshots:\n' + screenshots.map(s =>
+        `  - ${s.imageFilename || s.image_filename}${s.sourceApp ? ` (from ${s.sourceApp})` : ''}`
+      ).join('\n'));
+    }
+
+    if (parts.length > 0) {
+      attachmentContext = '\n\nAttachments:\n' + parts.join('\n');
+    }
+  } catch {
+    // Non-critical — continue without attachment context
+  }
+
   // Build prompt
   const prompt = `Work on Push task #${displayNumber}:
 
-${content}
+${content}${attachmentContext}
 
 IMPORTANT:
 1. If you need to understand the codebase, start by reading the CLAUDE.md file if it exists.
@@ -1405,6 +1461,7 @@ IMPORTANT:
     'Bash(git *)',
     'Bash(npm *)', 'Bash(npx *)', 'Bash(yarn *)',
     'Bash(python *)', 'Bash(python3 *)', 'Bash(pip *)', 'Bash(pip3 *)',
+    'Bash(push-todo *)',
     'Task'
   ].join(',');
 
@@ -1614,15 +1671,36 @@ async function handleTaskCompletion(displayNumber, exitCode) {
       }
     }
 
-    // Auto-complete task after successful merge (configurable, default ON)
+    // Auto-complete task (configurable, default ON)
     const taskId = info.taskId;
-    if (getAutoCompleteEnabled() && merged && taskId) {
-      const comment = semanticSummary
-        ? `${semanticSummary} (${durationStr} on ${machineName})`
-        : `Completed in ${durationStr} on ${machineName}`;
-      const completed = await markTaskAsCompleted(displayNumber, taskId, comment);
-      if (!completed) {
-        logError(`Task #${displayNumber}: Failed to mark as completed — status is session_finished but not completed`);
+    let autoCompleted = false;
+
+    if (getAutoCompleteEnabled() && taskId) {
+      // Path 1: PR merge (code tasks)
+      if (merged) {
+        const comment = semanticSummary
+          ? `${semanticSummary} (${durationStr} on ${machineName})`
+          : `Completed in ${durationStr} on ${machineName}`;
+        autoCompleted = await markTaskAsCompleted(displayNumber, taskId, comment);
+        if (!autoCompleted) {
+          logError(`Task #${displayNumber}: Failed to mark as completed after merge`);
+        }
+      }
+
+      // Path 2: Confirmation approval (content tasks — tweets, emails, etc.)
+      // If user approved a confirmation AND Claude exited cleanly, the task is done.
+      if (!autoCompleted && !merged) {
+        const approved = await hasApprovedConfirmation(displayNumber);
+        if (approved) {
+          log(`Task #${displayNumber}: user-approved confirmation detected, auto-completing`);
+          const comment = semanticSummary
+            ? `${semanticSummary} (${durationStr} on ${machineName})`
+            : `Confirmed and completed in ${durationStr} on ${machineName}`;
+          autoCompleted = await markTaskAsCompleted(displayNumber, taskId, comment);
+          if (!autoCompleted) {
+            logError(`Task #${displayNumber}: Failed to mark as completed after confirmation`);
+          }
+        }
       }
     }
 
@@ -1631,7 +1709,7 @@ async function handleTaskCompletion(displayNumber, exitCode) {
       summary,
       completedAt: new Date().toISOString(),
       duration,
-      status: merged ? 'completed' : 'session_finished',
+      status: autoCompleted ? 'completed' : 'session_finished',
       prUrl,
       sessionId
     });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@masslessai/push-todo",
-  "version": "4.0.0",
+  "version": "4.0.2",
   "description": "Voice tasks from Push iOS app for Claude Code",
   "type": "module",
   "bin": {