create-claude-workspace 2.3.36 → 2.3.38

package/dist/scheduler/loop.mjs

@@ -600,10 +600,13 @@ async function runTaskPipeline(task, workerId, agents, deps) {
                 model: getAgentModel(pipeline.assignedAgent, agents, task),
             }, state, task.id, logger, onMessage, onSpawnStart, onSpawnEnd, mcpServers);
         }
+        // Check if changes contain actual code (affects QA + review decisions)
+        const postImplFiles = getChangedFiles(worktreePath);
+        const codeChanged = hasCodeChanges(postImplFiles);
         // STEP 3: QA (E2E tests, integration tests, acceptance criteria verification)
-        // Only for tasks that need it — skip for pure refactoring, config changes, etc.
+        // Skip for non-code changes (docs, config, assets) and pure refactoring tasks.
         if (!shouldSkip('test')) {
-            const needsQA = task.type !== 'fullstack' || !isRefactoringTask(task);
+            const needsQA = codeChanged && (task.type !== 'fullstack' || !isRefactoringTask(task));
             if (needsQA) {
                 pipeline.step = 'test';
                 appendEvent(projectDir, createEvent('step_changed', { taskId: task.id, step: 'test' }));
@@ -630,11 +633,11 @@ async function runTaskPipeline(task, workerId, agents, deps) {
                 }
             }
             else {
-                logger.info(`[${task.id}] Skipping QA step (refactoring/config task)`);
+                logger.info(`[${task.id}] Skipping QA step (${codeChanged ? 'refactoring/config task' : 'no code changes'})`);
             }
         } // end if !shouldSkip('test')
-        // STEP 4: Review
-        if (!shouldSkip('review')) {
+        // STEP 4: Review (skip for non-code changes — nothing to review)
+        if (!shouldSkip('review') && codeChanged) {
             pipeline.step = 'review';
             appendEvent(projectDir, createEvent('step_changed', { taskId: task.id, step: 'review' }));
             const reviewRouting = await orchestrator.routeTask(task, 'review', agents);
@@ -677,7 +680,10 @@ async function runTaskPipeline(task, workerId, agents, deps) {
                     }
                 }
             }
-        } // end if !shouldSkip('review')
+        }
+        else if (!shouldSkip('review') && !codeChanged) {
+            logger.info(`[${task.id}] Skipping review (no code changes)`);
+        } // end review
         // STEP 5: Commit
         if (!shouldSkip('commit')) {
             pipeline.step = 'commit';
@@ -879,6 +885,21 @@ function isRefactoringTask(task) {
     return lower.includes('refactor') || lower.includes('config') || lower.includes('cleanup')
         || lower.includes('rename') || lower.includes('migrate') || lower.includes('upgrade');
 }
+/** Code file extensions that warrant QA and review. */
+const CODE_EXTENSIONS = new Set([
+    '.ts', '.tsx', '.js', '.jsx', '.mts', '.mjs', '.cts', '.cjs',
+    '.vue', '.svelte', '.py', '.rs', '.go', '.java', '.kt',
+    '.css', '.scss', '.sass', '.less',
+    '.html', '.htm',
+    '.sql',
+]);
+/** Check if changed files contain actual code (not just docs/config/assets). */
+function hasCodeChanges(changedFiles) {
+    return changedFiles.some(f => {
+        const ext = f.slice(f.lastIndexOf('.')).toLowerCase();
+        return CODE_EXTENSIONS.has(ext);
+    });
+}
 // ─── Orphaned worktree recovery ───
 /** Mark issue as done on the platform after a successful recovery merge */
 export function closeRecoveredIssue(projectDir, branch, logger) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-claude-workspace",
-  "version": "2.3.36",
+  "version": "2.3.38",
   "author": "",
   "repository": {
     "type": "git",