claude-code-workflow 6.3.30 → 6.3.32

package/ccw/src/templates/dashboard-js/views/issue-manager.js

@@ -562,6 +562,9 @@ function renderQueueCard(queue, isActive) {
             <i data-lucide="git-merge" class="w-3 h-3"></i>
           </button>
         ` : ''}
+        <button class="btn-sm btn-danger" onclick="confirmDeleteQueue('${safeQueueId}')" title="${t('issues.deleteQueue') || 'Delete queue'}">
+          <i data-lucide="trash-2" class="w-3 h-3"></i>
+        </button>
       </div>
     </div>
   `;
@@ -619,6 +622,33 @@ async function deactivateQueue(queueId) {
   }
 }
 
+function confirmDeleteQueue(queueId) {
+  const msg = t('issues.confirmDeleteQueue') || 'Are you sure you want to delete this queue? This action cannot be undone.';
+  if (confirm(msg)) {
+    deleteQueue(queueId);
+  }
+}
+
+async function deleteQueue(queueId) {
+  try {
+    const response = await fetch('/api/queue/' + encodeURIComponent(queueId) + '?path=' + encodeURIComponent(projectPath), {
+      method: 'DELETE'
+    });
+    const result = await response.json();
+    if (result.success) {
+      showNotification(t('issues.queueDeleted') || 'Queue deleted successfully', 'success');
+      queueData.expandedQueueId = null;
+      await Promise.all([loadQueueData(), loadAllQueues()]);
+      renderIssueView();
+    } else {
+      showNotification(result.error || 'Failed to delete queue', 'error');
+    }
+  } catch (err) {
+    console.error('Failed to delete queue:', err);
+    showNotification('Failed to delete queue', 'error');
+  }
+}
+
 async function renderExpandedQueueView(queueId) {
   const safeQueueId = escapeHtml(queueId || '');
   // Fetch queue detail
@@ -1405,6 +1435,23 @@ function renderIssueDetailPanel(issue) {
           `).join('') : '<p class="text-sm text-muted-foreground">' + (t('issues.noTasks') || 'No tasks') + '</p>'}
         </div>
       </div>
+
+      <!-- Actions -->
+      <div class="detail-section issue-detail-actions">
+        <label class="detail-label">${t('issues.actions') || 'Actions'}</label>
+        <div class="flex gap-2 flex-wrap">
+          ${!issue._isArchived ? `
+            <button class="btn-secondary btn-sm" onclick="confirmArchiveIssue('${issue.id}')">
+              <i data-lucide="archive" class="w-4 h-4"></i>
+              ${t('issues.archive') || 'Archive'}
+            </button>
+          ` : ''}
+          <button class="btn-secondary btn-sm btn-danger" onclick="confirmDeleteIssue('${issue.id}', ${issue._isArchived || false})">
+            <i data-lucide="trash-2" class="w-4 h-4"></i>
+            ${t('issues.delete') || 'Delete'}
+          </button>
+        </div>
+      </div>
     </div>
   `;
 
@@ -1419,6 +1466,67 @@ function closeIssueDetail() {
   issueData.selectedIssue = null;
 }
 
+// ========== Issue Delete & Archive ==========
+function confirmDeleteIssue(issueId, isArchived) {
+  const msg = t('issues.confirmDeleteIssue') || 'Are you sure you want to delete this issue? This action cannot be undone.';
+  if (confirm(msg)) {
+    deleteIssue(issueId, isArchived);
+  }
+}
+
+async function deleteIssue(issueId, isArchived) {
+  try {
+    const response = await fetch('/api/issues/' + encodeURIComponent(issueId) + '?path=' + encodeURIComponent(projectPath), {
+      method: 'DELETE'
+    });
+    const result = await response.json();
+    if (result.success) {
+      showNotification(t('issues.issueDeleted') || 'Issue deleted successfully', 'success');
+      closeIssueDetail();
+      if (isArchived) {
+        issueData.historyIssues = issueData.historyIssues.filter(i => i.id !== issueId);
+      } else {
+        issueData.issues = issueData.issues.filter(i => i.id !== issueId);
+      }
+      renderIssueView();
+      updateIssueBadge();
+    } else {
+      showNotification(result.error || 'Failed to delete issue', 'error');
+    }
+  } catch (err) {
+    console.error('Failed to delete issue:', err);
+    showNotification('Failed to delete issue', 'error');
+  }
+}
+
+function confirmArchiveIssue(issueId) {
+  const msg = t('issues.confirmArchiveIssue') || 'Archive this issue? It will be moved to history.';
+  if (confirm(msg)) {
+    archiveIssue(issueId);
+  }
+}
+
+async function archiveIssue(issueId) {
+  try {
+    const response = await fetch('/api/issues/' + encodeURIComponent(issueId) + '/archive?path=' + encodeURIComponent(projectPath), {
+      method: 'POST'
+    });
+    const result = await response.json();
+    if (result.success) {
+      showNotification(t('issues.issueArchived') || 'Issue archived successfully', 'success');
+      closeIssueDetail();
+      await loadIssueData();
+      renderIssueView();
+      updateIssueBadge();
+    } else {
+      showNotification(result.error || 'Failed to archive issue', 'error');
+    }
+  } catch (err) {
+    console.error('Failed to archive issue:', err);
+    showNotification('Failed to archive issue', 'error');
+  }
+}
+
 function toggleSolutionExpand(solId) {
   const el = document.getElementById('solution-' + solId);
   if (el) {

package/ccw/src/tools/cli-executor-core.ts

@@ -160,7 +160,7 @@ interface ClaudeWithSettingsParams {
   prompt: string;
   settingsPath: string;
   endpointId: string;
-  mode: 'analysis' | 'write' | 'auto';
+  mode: 'analysis' | 'write' | 'auto' | 'review';
   workingDir: string;
   cd?: string;
   includeDirs?: string[];
@@ -351,12 +351,12 @@ type BuiltinCliTool = typeof BUILTIN_CLI_TOOLS[number];
 const ParamsSchema = z.object({
   tool: z.string().min(1, 'Tool is required'), // Accept any tool ID (built-in or custom endpoint)
   prompt: z.string().min(1, 'Prompt is required'),
-  mode: z.enum(['analysis', 'write', 'auto']).default('analysis'),
+  mode: z.enum(['analysis', 'write', 'auto', 'review']).default('analysis'),
   format: z.enum(['plain', 'yaml', 'json']).default('plain'), // Multi-turn prompt concatenation format
   model: z.string().optional(),
   cd: z.string().optional(),
   includeDirs: z.string().optional(),
-  timeout: z.number().default(0), // 0 = no internal timeout, controlled by external caller (e.g., bash timeout)
+  // timeout removed - controlled by external caller (bash timeout)
   resume: z.union([z.boolean(), z.string()]).optional(), // true = last, string = single ID or comma-separated IDs
   id: z.string().optional(), // Custom execution ID (e.g., IMPL-001-step1)
   noNative: z.boolean().optional(), // Force prompt concatenation instead of native resume
@@ -388,7 +388,7 @@ async function executeCliTool(
     throw new Error(`Invalid params: ${parsed.error.message}`);
   }
 
-  const { tool, prompt, mode, format, model, cd, includeDirs, timeout, resume, id: customId, noNative, category, parentExecutionId, outputFormat } = parsed.data;
+  const { tool, prompt, mode, format, model, cd, includeDirs, resume, id: customId, noNative, category, parentExecutionId, outputFormat } = parsed.data;
 
   // Validate and determine working directory early (needed for conversation lookup)
   let workingDir: string;
@@ -862,7 +862,6 @@ async function executeCliTool(
 
     let stdout = '';
     let stderr = '';
-    let timedOut = false;
 
     // Handle stdout
     child.stdout!.on('data', (data: Buffer) => {
@@ -924,18 +923,14 @@ async function executeCliTool(
       debugLog('CLOSE', `Process closed`, {
         exitCode: code,
         duration: `${duration}ms`,
-        timedOut,
         stdoutLength: stdout.length,
         stderrLength: stderr.length,
         outputUnitsCount: allOutputUnits.length
       });
 
       // Determine status - prioritize output content over exit code
-      let status: 'success' | 'error' | 'timeout' = 'success';
-      if (timedOut) {
-        status = 'timeout';
-        debugLog('STATUS', `Execution timed out after ${duration}ms`);
-      } else if (code !== 0) {
+      let status: 'success' | 'error' = 'success';
+      if (code !== 0) {
         // Non-zero exit code doesn't always mean failure
         // Check if there's valid output (AI response) - treat as success
         const hasValidOutput = stdout.trim().length > 0;
@@ -1169,25 +1164,8 @@ async function executeCliTool(
       reject(new Error(`Failed to spawn ${tool}: ${error.message}\n  Command: ${command} ${args.join(' ')}\n  Working Dir: ${workingDir}`));
     });
 
-    // Timeout handling (timeout=0 disables internal timeout, controlled by external caller)
-    let timeoutId: NodeJS.Timeout | null = null;
-    if (timeout > 0) {
-      timeoutId = setTimeout(() => {
-        timedOut = true;
-        child.kill('SIGTERM');
-        setTimeout(() => {
-          if (!child.killed) {
-            child.kill('SIGKILL');
-          }
-        }, 5000);
-      }, timeout);
-    }
-
-    child.on('close', () => {
-      if (timeoutId) {
-        clearTimeout(timeoutId);
-      }
-    });
+    // Timeout controlled by external caller (bash timeout)
+    // When parent process terminates, child will be cleaned up via process exit handler
   });
 }
 
@@ -1198,7 +1176,8 @@ export const schema: ToolSchema = {
 Modes:
 - analysis: Read-only operations (default)
 - write: File modifications allowed
-- auto: Full autonomous operations (codex only)`,
+- auto: Full autonomous operations (codex only)
+- review: Code review mode (codex uses 'codex review' subcommand, others accept but no operation change)`,
   inputSchema: {
     type: 'object',
     properties: {
@@ -1213,8 +1192,8 @@ Modes:
       },
       mode: {
         type: 'string',
-        enum: ['analysis', 'write', 'auto'],
-        description: 'Execution mode (default: analysis)',
+        enum: ['analysis', 'write', 'auto', 'review'],
+        description: 'Execution mode (default: analysis). review mode uses codex review subcommand for codex tool.',
         default: 'analysis'
       },
       model: {
@@ -1228,12 +1207,8 @@ Modes:
       includeDirs: {
         type: 'string',
         description: 'Additional directories (comma-separated). Maps to --include-directories for gemini/qwen, --add-dir for codex'
-      },
-      timeout: {
-        type: 'number',
-        description: 'Timeout in milliseconds (default: 0 = disabled, controlled by external caller)',
-        default: 0
       }
+      // timeout removed - controlled by external caller (bash timeout)
     },
     required: ['tool', 'prompt']
   }

package/ccw/src/tools/cli-executor-utils.ts

@@ -223,7 +223,21 @@ export function buildCommand(params: {
 
     case 'codex':
       useStdin = true;
-      if (nativeResume?.enabled) {
+      if (mode === 'review') {
+        // codex review mode: non-interactive code review
+        // Format: codex review [OPTIONS] [PROMPT]
+        args.push('review');
+        // Default to --uncommitted if no specific review target in prompt
+        args.push('--uncommitted');
+        if (model) {
+          args.push('-m', model);
+        }
+        // codex review uses positional prompt argument, not stdin
+        useStdin = false;
+        if (prompt) {
+          args.push(prompt);
+        }
+      } else if (nativeResume?.enabled) {
         args.push('resume');
         if (nativeResume.isLatest) {
           args.push('--last');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-code-workflow",
-  "version": "6.3.30",
+  "version": "6.3.32",
   "description": "JSON-driven multi-agent development framework with intelligent CLI orchestration (Gemini/Qwen/Codex), context-first architecture, and automated workflow execution",
   "type": "module",
   "main": "ccw/src/index.js",