edsger 0.19.18 → 0.20.0

package/dist/api/features/feature-utils.d.ts

@@ -1,4 +1,14 @@
 import { FeatureInfo } from '../../types/features.js';
+/**
+ * Claim the next available ready_for_ai feature for processing.
+ * Uses database-level locking (FOR UPDATE SKIP LOCKED) to prevent
+ * race conditions between multiple workers.
+ *
+ * @param productId - The product ID to claim a feature from
+ * @param verbose - Whether to log verbose output
+ * @returns The claimed feature or null if no features available
+ */
+export declare function claimNextFeature(productId: string, verbose?: boolean): Promise<FeatureInfo | null>;
 /**
  * Filter features by status
  */

package/dist/api/features/feature-utils.js

@@ -1,5 +1,39 @@
 import { logInfo, logError } from '../../utils/logger.js';
 import { callMcpEndpoint } from '../mcp-client.js';
+/**
+ * Claim the next available ready_for_ai feature for processing.
+ * Uses database-level locking (FOR UPDATE SKIP LOCKED) to prevent
+ * race conditions between multiple workers.
+ *
+ * @param productId - The product ID to claim a feature from
+ * @param verbose - Whether to log verbose output
+ * @returns The claimed feature or null if no features available
+ */
+export async function claimNextFeature(productId, verbose) {
+    if (verbose) {
+        logInfo(`Attempting to claim next ready_for_ai feature for product: ${productId}`);
+    }
+    try {
+        const result = (await callMcpEndpoint('features/claim', {
+            product_id: productId,
+        }));
+        if (result.feature) {
+            if (verbose) {
+                logInfo(`✅ Claimed feature: ${result.feature.name} (${result.feature.id})`);
+            }
+            return result.feature;
+        }
+        if (verbose) {
+            logInfo('No features available for processing');
+        }
+        return null;
+    }
+    catch (error) {
+        const errorMessage = error instanceof Error ? error.message : String(error);
+        logError(`Failed to claim feature: ${errorMessage}`);
+        throw error;
+    }
+}
 /**
  * Filter features by status
  */

package/dist/commands/workflow/processor.d.ts

@@ -32,7 +32,8 @@ export declare class WorkflowProcessor {
      */
     stop(): void;
     /**
-     * Process the next available feature using functional composition
+     * Process the next available feature using atomic claim mechanism
+     * Uses database-level locking to prevent race conditions between workers
      */
     private processNextFeature;
     /**

package/dist/commands/workflow/processor.js

@@ -3,14 +3,13 @@
  * Monitors for ready_for_ai features and processes them through the complete pipeline
  * Uses functional programming principles
  */
-import { getReadyForAIFeatures } from '../../api/features/index.js';
+import { claimNextFeature } from '../../api/features/index.js';
 import { runFeatureWorkflow } from './feature-coordinator.js';
 import { logInfo } from '../../utils/logger.js';
 // Import core modules
 import { createInitialState, updateFeatureState, createProcessingState, createCompletedState, createFailedState, calculateStats, } from './core/state-manager.js';
-import { findNextFeature } from './core/feature-filter.js';
 import { evaluatePipelineResults } from './core/pipeline-evaluator.js';
-import { logProcessingStart, logRetryInfo, logPipelineResults, logProcessorStart, logProcessorReady, logProcessorStop, logFeatureSuccess, logFeatureFailed, logFeatureError, logNoFeaturesFound, logAllFeaturesProcessed, logSkippingProcessing, logPollingError, logProcessNextFeatureError, } from './core/workflow-logger.js';
+import { logProcessingStart, logRetryInfo, logPipelineResults, logProcessorStart, logProcessorReady, logProcessorStop, logFeatureSuccess, logFeatureFailed, logFeatureError, logNoFeaturesFound, logSkippingProcessing, logPollingError, logProcessNextFeatureError, } from './core/workflow-logger.js';
 /**
  * Workflow processor using functional programming principles
  */
@@ -64,7 +63,8 @@ export class WorkflowProcessor {
         logProcessorStop();
     }
     /**
-     * Process the next available feature using functional composition
+     * Process the next available feature using atomic claim mechanism
+     * Uses database-level locking to prevent race conditions between workers
      */
     async processNextFeature() {
         try {
@@ -76,24 +76,17 @@ export class WorkflowProcessor {
                 }
                 return;
             }
-            // Fetch features using unified API
-            const features = await getReadyForAIFeatures(this.options.productId, this.options.verbose);
-            if (features.length === 0) {
+            // Atomically claim the next available feature
+            // This uses FOR UPDATE SKIP LOCKED to prevent multiple workers from claiming the same feature
+            const claimedFeature = await claimNextFeature(this.options.productId, this.options.verbose);
+            if (!claimedFeature) {
                 if (this.options.verbose) {
                     logNoFeaturesFound();
                 }
                 return;
             }
-            // Find next feature to process using pure functions
-            const nextFeature = findNextFeature(features, this.processedFeatures, this.options.maxRetries);
-            if (!nextFeature) {
-                if (this.options.verbose) {
-                    logAllFeaturesProcessed();
-                }
-                return;
-            }
-            // Process the feature
-            await this.processFeature(nextFeature);
+            // Process the claimed feature
+            await this.processFeature(claimedFeature);
         }
         catch (error) {
             logProcessNextFeatureError(error);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "edsger",
-  "version": "0.19.18",
+  "version": "0.20.0",
   "type": "module",
   "bin": {
     "edsger": "dist/index.js"