speccrew 0.7.39 → 0.7.41

package/.speccrew/skills/speccrew-fd-api-contract/workflow.agentflow.xml

@@ -160,7 +160,7 @@ ${api_list.error_codes}</field>
     <block type="gateway" id="G4" mode="exclusive" desc="Update progress based on mode">
       <branch test="${feature_id} != null" name="Feature mode">
         <block type="task" id="B10" action="run-script" desc="Update workflow progress for feature">
-          <field name="command">node "${workspace_path}/scripts/update-progress.js" update-task --file "${iteration_path}/WORKFLOW-PROGRESS.json" --task-id "${feature_id}" --status confirmed --outputs '["02.feature-design/${feature_id}-${feature_name}-feature-spec.md","03.api-contract/${feature_id}-${feature_name}-api-contract.md"]'</field>
+          <field name="command">node "${workspace_path}/scripts/update-progress.js" update-task --file "${iteration_path}/WORKFLOW-PROGRESS.json" --stage 03_api_contract --task-id "${feature_id}" --status confirmed --outputs '["02.feature-design/${feature_id}-${feature_name}-feature-spec.md","03.api-contract/${feature_id}-${feature_name}-api-contract.md"]'</field>
         </block>
       </branch>
       <branch default="true" name="Module mode">

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "speccrew",
-  "version": "0.7.39",
+  "version": "0.7.41",
   "description": "Spec-Driven Development toolkit for AI-powered IDEs",
   "author": "charlesmu99",
   "repository": {

package/workspace-template/scripts/update-progress.js

@@ -152,7 +152,7 @@ function acquireLock(filePath) {
                 try {
                     const lockStat = fs.statSync(lockPath);
                     const ageSeconds = (Date.now() - lockStat.mtimeMs) / 1000;
-                    if (ageSeconds > 120) {
+                    if (ageSeconds > 30) {
                         console.error(`Warning: Stale lock file detected (age: ${Math.round(ageSeconds)}s), removing: ${lockPath}`);
                         fs.unlinkSync(lockPath);
                         // Do not consume retry count, continue to next loop attempt to acquire lock
@@ -161,10 +161,32 @@ function acquireLock(filePath) {
                 } catch (statErr) {
                     // Lock file was deleted during stat, continue retrying
                 }
+            } else {
+                // Non-EEXIST error (EACCES, EPERM, etc.) - log for debugging
+                if (retryCount === 0) {
+                    console.error(`Warning: Lock creation failed with ${error.code} for: ${lockPath}`);
+                }
             }
             retryCount++;
             if (retryCount >= maxRetries) {
-                throw new Error(`Failed to acquire file lock for '${filePath}' after ${maxRetries} attempts`);
+                // Force acquire as last resort
+                try {
+                    // Remove any existing lock file regardless of age
+                    if (fs.existsSync(lockPath)) {
+                        fs.unlinkSync(lockPath);
+                    }
+                    // Brief random delay to avoid thundering herd on force acquire
+                    const forceDelay = Math.floor(Math.random() * 500);
+                    const forceStart = Date.now();
+                    while (Date.now() - forceStart < forceDelay) {}
+
+                    const fd = fs.openSync(lockPath, 'wx');
+                    fs.closeSync(fd);
+                    console.error(`Warning: Lock acquired by force after ${maxRetries} retries for: ${filePath}`);
+                    return lockPath;
+                } catch (forceErr) {
+                    throw new Error(`Failed to acquire file lock for '${filePath}' after ${maxRetries} attempts (force acquire also failed: ${forceErr.code})`);
+                }
             }
             // Retry with jitter to avoid thundering herd
             const delay = 200 + Math.floor(Math.random() * 300);
@@ -630,13 +652,16 @@ function cmdUpdateTask(args) {
                 outputError(`Stage not found: ${args.stage}`);
             }
             targetStage = data.stages[args.stage];
-            if (!targetStage.features || !Array.isArray(targetStage.features)) {
-                outputError(`Stage has no features array: ${args.stage}`);
+            if (!targetStage.features) {
+                targetStage.features = [];
             }
             taskArray = targetStage.features;
             taskIndex = taskArray.findIndex(t => t.id === args.taskId);
             if (taskIndex === -1) {
-                outputError(`Task not found in stage ${args.stage}: ${args.taskId}`);
+                // UPSERT: Task not found, create new entry
+                console.error(`Info: Task ${args.taskId} not found in stage ${args.stage}, creating new entry`);
+                taskArray.push({ id: args.taskId, status: 'pending' });
+                taskIndex = taskArray.length - 1;
             }
         } else {
             // Flat structure: data.tasks