@masslessai/push-todo 3.7.1 → 3.7.2

package/lib/api.js

@@ -298,28 +298,6 @@ export async function registerProject(gitRemote, keywords = [], description = ''
   return true;
 }
 
-/**
- * Validate machine registration.
- *
- * @param {string} machineId - Machine identifier
- * @returns {Promise<Object>} Validation result
- */
-export async function validateMachine(machineId) {
-  const response = await apiRequest('validate-machine', {
-    method: 'POST',
-    body: JSON.stringify({
-      machine_id: machineId
-    })
-  });
-
-  if (!response.ok) {
-    const text = await response.text();
-    throw new Error(`Machine validation failed: ${text}`);
-  }
-
-  const data = await response.json();
-  return data;
-}
 
 /**
  * Get the current CLI version from the server.

package/lib/cli.js

@@ -89,7 +89,6 @@ ${bold('CONNECT OPTIONS:')}
   --check-version                  Check for updates (JSON output)
   --update                         Update to latest version
   --validate-key                   Validate API key (JSON output)
-  --validate-machine               Validate machine registration (JSON output)
   --validate-project               Validate project registration (JSON output)
   --store-e2ee-key <key>           Import E2EE encryption key
   --description <text>             Project description (with connect)
@@ -134,7 +133,6 @@ const options = {
   'check-version': { type: 'boolean' },
   'update': { type: 'boolean' },
   'validate-key': { type: 'boolean' },
-  'validate-machine': { type: 'boolean' },
   'validate-project': { type: 'boolean' },
   'store-e2ee-key': { type: 'string' },
   'description': { type: 'string' }

package/lib/connect.js

@@ -618,30 +618,6 @@ async function validateApiKeyStatus() {
   return { status: 'invalid', message: result.reason };
 }
 
-/**
- * Validate machine registration.
- */
-async function validateMachineStatus() {
-  const machineId = getMachineId();
-  const machineName = getMachineName();
-
-  try {
-    const result = await api.validateMachine(machineId);
-    return {
-      status: 'valid',
-      machineId,
-      machineName,
-      ...result
-    };
-  } catch (error) {
-    return {
-      status: 'error',
-      machineId,
-      machineName,
-      message: error.message
-    };
-  }
-}
 
 /**
  * Validate project registration (full validation with warnings).
@@ -742,11 +718,7 @@ function validateProjectStatus() {
  * Get device name for registration.
  */
 function getDeviceName() {
-  try {
-    return require('os').hostname() || 'Unknown Device';
-  } catch {
-    return 'Unknown Device';
-  }
+  return getMachineName() || 'Unknown Device';
 }
 
 /**
@@ -1090,13 +1062,6 @@ export async function runConnect(options = {}) {
     return;
   }
 
-  // Handle --validate-machine (JSON output)
-  if (options['validate-machine'] || options.validateMachine) {
-    const result = await validateMachineStatus();
-    console.log(JSON.stringify(result, null, 2));
-    return;
-  }
-
   // Handle --validate-project (JSON output)
   if (options['validate-project'] || options.validateProject) {
     const result = validateProjectStatus();
@@ -1191,13 +1156,8 @@ export async function runConnect(options = {}) {
         }
       }
 
-      // Validate and show machine ID
-      const machineInfo = await validateMachineStatus();
-      if (machineInfo.status === 'valid') {
-        console.log(`  Machine: ${machineInfo.machineName}`);
-      } else {
-        console.log(`  ⚠️  Machine ID: ${machineInfo.message}`);
-      }
+      // Show machine info
+      console.log(`  Machine: ${getMachineName()}`);
       console.log('  ' + '='.repeat(40));
       console.log('');
 
@@ -1274,13 +1234,8 @@ export async function runConnect(options = {}) {
     }
   }
 
-  // Validate and show machine ID
-  const machineInfo = await validateMachineStatus();
-  if (machineInfo.status === 'valid') {
-    console.log(`  Machine: ${machineInfo.machineName}`);
-  } else {
-    console.log(`  ⚠️  Machine ID: ${machineInfo.message}`);
-  }
+  // Show machine info
+  console.log(`  Machine: ${getMachineName()}`);
   console.log('  ' + '='.repeat(40));
   console.log('');
   console.log('  Your iOS app will sync this automatically.');
@@ -1297,7 +1252,6 @@ export {
   checkVersion,
   doUpdate,
   validateApiKeyStatus,
-  validateMachineStatus,
   validateProjectStatus,
   validateProjectInfo,
   setupE2EE,

package/lib/daemon-health.js

@@ -197,7 +197,7 @@ function getInstalledVersion() {
  * Auto-restarts daemon if version mismatch detected.
  *
  * This prevents stale daemons from running after npm package updates.
- * See: /docs/20260204_daemon_heartbeat_status_indicator_implementation_plan.md
+ * See: /docs/20260204_daemon_heartbeat_status_indicator_implementation_plan.md (machine_registry table)
  */
 export function ensureDaemonRunning() {
   const status = getDaemonStatus();

package/lib/daemon.js

@@ -297,8 +297,8 @@ async function fetchQueuedTasks() {
     const projects = getListedProjects();
     const gitRemotes = Object.keys(projects);
 
-    // Add heartbeat headers for daemon status tracking
-    // See: /docs/20260204_daemon_heartbeat_status_indicator_implementation_plan.md
+    // Add machine registry headers for daemon status tracking
+    // See: /docs/20260204_daemon_heartbeat_status_indicator_implementation_plan.md (machine_registry table)
     const heartbeatHeaders = {};
     if (machineId && gitRemotes.length > 0) {
       heartbeatHeaders['X-Machine-Id'] = machineId;
@@ -711,6 +711,43 @@ function extractSessionIdFromStdout(proc, buffer) {
   return null;
 }
 
+// ==================== Semantic Summary Extraction ====================
+
+/**
+ * Generate a semantic summary by resuming the Claude session in --print mode.
+ * Claude already has full context of what it did — we just ask it to summarize.
+ * Uses execFileSync (not execSync) to avoid shell injection.
+ *
+ * @param {string} worktreePath - Path to the git worktree where Claude ran
+ * @param {string} sessionId - Claude session ID
+ * @returns {string|null} A short semantic summary, or null if extraction fails
+ */
+function extractSemanticSummary(worktreePath, sessionId) {
+  if (!worktreePath || !sessionId) return null;
+
+  try {
+    const result = execFileSync('claude', [
+      '--resume', sessionId,
+      '--print',
+      'Summarize what you accomplished in 1-2 sentences. Be specific about outcomes (what was built, fixed, drafted), not process. If you failed or got stuck, explain what went wrong. If no code changes were made, say so. Do not use markdown.'
+    ], {
+      cwd: worktreePath,
+      timeout: 30000,
+      stdio: ['ignore', 'pipe', 'pipe']
+    });
+
+    const summary = result.toString().trim();
+    if (summary && summary.length > 0) {
+      // Cap at 300 chars to keep it concise
+      return summary.length > 300 ? summary.slice(0, 297) + '...' : summary;
+    }
+    return null;
+  } catch (error) {
+    log(`Semantic summary extraction failed: ${error.message}`);
+    return null;
+  }
+}
+
 // ==================== Task Execution ====================
 
 function updateTaskDetail(displayNumber, updates) {
@@ -918,24 +955,32 @@ function handleTaskCompletion(displayNumber, exitCode) {
 
   log(`Task #${displayNumber} completed with code ${exitCode} (${duration}s)`);
 
-  if (exitCode === 0) {
-    // Extract session ID
-    const buffer = taskStdoutBuffer.get(displayNumber) || [];
-    const sessionId = extractSessionIdFromStdout(taskInfo.process, buffer);
+  // Extract session ID for both success and failure (needed for AI summary)
+  const buffer = taskStdoutBuffer.get(displayNumber) || [];
+  const sessionId = extractSessionIdFromStdout(taskInfo.process, buffer);
+  const worktreePath = getWorktreePath(displayNumber, projectPath);
+  const durationStr = duration < 60 ? `${duration}s` : `${Math.floor(duration / 60)}m ${duration % 60}s`;
+  const machineName = getMachineName() || 'Mac';
 
-    if (sessionId) {
-      log(`Task #${displayNumber} session_id: ${sessionId}`);
-    } else {
-      log(`Task #${displayNumber} could not extract session_id`);
-    }
+  if (sessionId) {
+    log(`Task #${displayNumber} session_id: ${sessionId}`);
+  } else {
+    log(`Task #${displayNumber} could not extract session_id`);
+  }
 
+  if (exitCode === 0) {
     // Auto-create PR first so we can include it in the summary
     const prUrl = createPRForTask(displayNumber, summary, projectPath);
 
-    // Build execution summary for Supabase (shown in iOS timeline)
-    const durationStr = duration < 60 ? `${duration}s` : `${Math.floor(duration / 60)}m ${duration % 60}s`;
-    const machineName = getMachineName() || 'Mac';
-    let executionSummary = `Ran for ${durationStr} on ${machineName}.`;
+    // Ask Claude to summarize what it accomplished
+    const semanticSummary = extractSemanticSummary(worktreePath, sessionId);
+
+    // Combine: semantic summary first (what), then machine metadata (how)
+    let executionSummary = '';
+    if (semanticSummary) {
+      executionSummary = semanticSummary + '\n';
+    }
+    executionSummary += `Ran for ${durationStr} on ${machineName}.`;
     if (prUrl) {
       executionSummary += ` PR: ${prUrl}`;
     }
@@ -966,7 +1011,13 @@ function handleTaskCompletion(displayNumber, exitCode) {
     });
   } else {
     const stderr = taskInfo.process.stderr?.read()?.toString() || '';
-    const errorMsg = `Exit code ${exitCode}: ${stderr.slice(0, 200)}`;
+
+    // Ask Claude to explain what went wrong (if session exists)
+    const failureSummary = extractSemanticSummary(worktreePath, sessionId);
+    const errorMsg = failureSummary
+      ? `${failureSummary}\nExit code ${exitCode}. Ran for ${durationStr} on ${machineName}.`
+      : `Exit code ${exitCode}: ${stderr.slice(0, 200)}`;
+
     updateTaskStatus(displayNumber, 'failed', { error: errorMsg });
 
     if (NOTIFY_ON_FAILURE) {

package/lib/index.js

@@ -39,7 +39,6 @@ export {
   updateTaskExecution,
   validateApiKey,
   registerProject,
-  validateMachine,
   getLatestVersion
 } from './api.js';
 

package/lib/machine-id.js

@@ -9,7 +9,7 @@
  * File location: ~/.config/push/machine_id
  */
 
-import { readFileSync, writeFileSync, mkdirSync, existsSync } from 'fs';
+import { readFileSync, writeFileSync, mkdirSync, existsSync, unlinkSync } from 'fs';
 import { homedir, hostname, platform, release, version, arch } from 'os';
 import { join, dirname } from 'path';
 import { randomUUID } from 'crypto';
@@ -90,7 +90,6 @@ export function getMachineInfo() {
 export function resetMachineId() {
   if (existsSync(MACHINE_ID_FILE)) {
     try {
-      const { unlinkSync } = require('fs');
       unlinkSync(MACHINE_ID_FILE);
     } catch {
       // Ignore errors

package/lib/utils/format.js

@@ -148,6 +148,16 @@ export function formatTaskForDisplay(task) {
     lines.push('**Status:** Active');
   }
 
+  // Show execution summary (semantic + machine metadata)
+  const execSummary = task.executionSummary || task.execution_summary;
+  if (execSummary) {
+    lines.push('');
+    lines.push('### What was done');
+    for (const line of execSummary.split('\n').filter(Boolean)) {
+      lines.push(`> ${line}`);
+    }
+  }
+
   // Show session resume hint for any task with a session ID
   const sessionId = task.executionSessionId || task.execution_session_id;
   if (sessionId && displayNum) {

package/package.json

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