edsger 0.41.2 → 0.41.3

package/dist/commands/workflow/executors/phase-executor.js

@@ -130,9 +130,7 @@ async function validateAndLogChecklists(options, name, checklistContext, feature
     }
 }
 // Higher-order function for phase execution
-export const createPhaseRunner = (phaseConfig) => async (options, config
-// eslint-disable-next-line complexity -- orchestration function with sequential phase setup, execution, and result processing
-) => {
+export const createPhaseRunner = (phaseConfig) => async (options, config) => {
     const { featureId, verbose } = options;
     const { name, execute } = phaseConfig;
     // Track phase duration for logging

package/dist/commands/workflow/phase-orchestrator.js

@@ -36,9 +36,7 @@ const logAndMarkPhaseCompleted = async (result, verbose) => {
  * Orchestrate phase execution based on execution mode
  * Routes to appropriate phase sequence based on mode (only_*, from_*, full_pipeline)
  */
-export const runPipelineByMode = async (options, config, executionMode
-// eslint-disable-next-line complexity -- orchestration function routing many execution modes to phase sequences
-) => {
+export const runPipelineByMode = async (options, config, executionMode) => {
     const { featureId, verbose } = options;
     if (verbose) {
         logInfo(`🚀 Starting pipeline with mode: ${executionMode} for feature: ${featureId}`);

package/dist/index.js

@@ -16,10 +16,10 @@ import { runConfigGet, runConfigList, runConfigSet, runConfigUnset, } from './co
 import { runGrowthAnalysis } from './commands/growth-analysis/index.js';
 import { runInit } from './commands/init/index.js';
 import { runIntelligence } from './commands/intelligence/index.js';
+import { runPRResolve } from './commands/pr-resolve/index.js';
+import { runPRReview } from './commands/pr-review/index.js';
 import { runRefactor } from './commands/refactor/refactor.js';
 import { runTaskWorker } from './commands/task-worker/index.js';
-import { runPRReview } from './commands/pr-review/index.js';
-import { runPRResolve } from './commands/pr-resolve/index.js';
 import { runWorkflow } from './commands/workflow/index.js';
 import { logError, logInfo } from './utils/logger.js';
 // Get package.json version dynamically

package/dist/phases/app-store-generation/index.js

@@ -13,9 +13,7 @@ function truncateKeywords(keywords) {
     }
     return keywords.slice(0, 100).replace(/,[^,]*$/, '');
 }
-export const generateAppStoreAssets = async (options, config
-// eslint-disable-next-line complexity -- orchestration function with sequential asset generation steps
-) => {
+export const generateAppStoreAssets = async (options, config) => {
     const { productId, targetStore, screenshotsOnly, listingsOnly, verbose } = options;
     if (verbose) {
         logInfo(`Starting app store generation for product: ${productId}`);

package/dist/phases/branch-planning/index.js

@@ -116,9 +116,7 @@ async function persistBranches(sortedBranches, featureId, verbose) {
         }
     }
 }
-export const planFeatureBranches = async (options, config
-// eslint-disable-next-line complexity -- orchestration function with context fetching, agent execution, and result processing
-) => {
+export const planFeatureBranches = async (options, config) => {
     const { featureId, verbose, replaceExisting } = options;
     if (verbose) {
         logInfo(`Starting branch planning for feature ID: ${featureId}`);

package/dist/phases/bug-fixing/analyzer.js

@@ -32,9 +32,7 @@ async function* prompt(bugFixPrompt) {
         setTimeout(res, 10000);
     });
 }
-export const fixTestFailures = async (options, config
-// eslint-disable-next-line complexity -- orchestration function with agent setup, message processing, and result extraction
-) => {
+export const fixTestFailures = async (options, config) => {
     const { featureId, testErrors, attemptNumber = 1, verbose } = options;
     if (verbose) {
         logInfo(`Starting bug fixing for feature ID: ${featureId} (Attempt ${attemptNumber})`);

package/dist/phases/code-implementation/index.js

@@ -170,9 +170,7 @@ message, lastAssistantResponse, featureId, verbose
     }
     return null;
 }
-export const implementFeatureCode = async (options, config, checklistContext
-// eslint-disable-next-line complexity -- orchestration function coordinating full code implementation lifecycle
-) => {
+export const implementFeatureCode = async (options, config, checklistContext) => {
     const { featureId, verbose, baseBranch = 'main' } = options;
     if (verbose) {
         logInfo(`Starting code implementation for feature ID: ${featureId}`);

package/dist/phases/code-refine/index.js

@@ -41,9 +41,7 @@ export const MAX_REFINE_ITERATIONS = 10;
  * Similar to technical-design, this includes an iterative improvement cycle:
  * refine → verification → improve → re-refine (if needed)
  */
-export const refineCodeFromPRFeedback = async (options, config, checklistContext
-// eslint-disable-next-line complexity -- orchestration function coordinating PR feedback refinement lifecycle
-) => {
+export const refineCodeFromPRFeedback = async (options, config, checklistContext) => {
     const { featureId, githubToken, verbose } = options;
     if (verbose) {
         logInfo(`Starting code refine for feature ID: ${featureId}`);

package/dist/phases/code-review/index.js

@@ -67,9 +67,7 @@ baseBranchInfo, baseBranchForRebase, originalBaseBranchForRebase, verbose) {
 /**
  * Main code review function
  */
-export const reviewPullRequest = async (options, config, checklistContext
-// eslint-disable-next-line complexity -- orchestration function coordinating full code review lifecycle
-) => {
+export const reviewPullRequest = async (options, config, checklistContext) => {
     const { featureId, githubToken, verbose } = options;
     if (verbose) {
         logInfo(`Starting code review for feature ID: ${featureId}`);

package/dist/phases/code-testing/analyzer.js

@@ -31,9 +31,7 @@ async function* prompt(testingPrompt) {
         setTimeout(res, 10000);
     });
 }
-export const writeCodeTests = async (options, config
-// eslint-disable-next-line complexity -- orchestration function with context fetching, agent execution, and test result processing
-) => {
+export const writeCodeTests = async (options, config) => {
     const { featureId, verbose } = options;
     if (verbose) {
         logInfo(`Starting code testing phase for feature ID: ${featureId}`);

package/dist/phases/feature-analysis/index.js

@@ -6,9 +6,7 @@ import { executeAnalysisQuery, parseAnalysisResult } from './agent.js';
 import { prepareAnalysisContext } from './context.js';
 import { buildAnalysisResult, deleteArtifacts, deleteSpecificArtifacts, getAllDraftArtifactIds, resetReadyArtifactsToDraft, saveAnalysisArtifactsAsDraft, updateArtifactsToReady, } from './outcome.js';
 import { createFeatureAnalysisSystemPrompt } from './prompts.js';
-export const analyseFeature = async (options, config, checklistContext
-// eslint-disable-next-line complexity -- orchestration function with context assembly, agent execution, and result processing
-) => {
+export const analyseFeature = async (options, config, checklistContext) => {
     const { featureId, verbose } = options;
     if (verbose) {
         logInfo(`Starting feature analysis for feature ID: ${featureId}`);

package/dist/phases/functional-testing/analyzer.js

@@ -326,9 +326,7 @@ async function saveTestResults(featureId, testStatus, structuredTestResult, last
     }
     return testReportResult;
 }
-export const runFunctionalTesting = async (options, config, checklistContext
-// eslint-disable-next-line complexity -- orchestration function coordinating functional test execution and result processing
-) => {
+export const runFunctionalTesting = async (options, config, checklistContext) => {
     const { featureId, verbose } = options;
     if (verbose) {
         logInfo(`Starting functional testing for feature ID: ${featureId}`);

package/dist/phases/growth-analysis/index.js

@@ -49,9 +49,7 @@ contentSuggestions) {
     }
     return plans;
 }
-export const analyseGrowth = async (options, config
-// eslint-disable-next-line complexity -- orchestration function with context assembly, agent execution, and result processing
-) => {
+export const analyseGrowth = async (options, config) => {
     const { productId, verbose, guidance, analysisId } = options;
     if (verbose) {
         logInfo(`Starting growth analysis for product ID: ${productId}`);

package/dist/phases/pr-execution/index.js

@@ -2,7 +2,7 @@ import { query } from '@anthropic-ai/claude-agent-sdk';
 import { execSync } from 'child_process';
 import { DEFAULT_MODEL } from '../../constants.js';
 import { logFeaturePhaseEvent } from '../../services/audit-logs.js';
-import { getPullRequests } from '../../services/pull-requests.js';
+import { getPullRequests, } from '../../services/pull-requests.js';
 import { getCurrentBranch, returnToMainBranch, } from '../../utils/git-branch-manager.js';
 import { logDebug, logError, logInfo } from '../../utils/logger.js';
 import { fetchPRExecutionContext } from './context.js';
@@ -98,6 +98,18 @@ export const executeFeaturePRs = async (options, config) => {
             }
         }
         // ======================================
+        // Filter to only the next eligible PR
+        // ======================================
+        // Only process the first pending PR whose predecessor is merged/closed.
+        // Already-executed PRs (branch_created, pr_opened, etc.) are included for sync.
+        activePullRequests = filterToEligiblePRs(activePullRequests, verbose);
+        if (activePullRequests.length === 0) {
+            if (verbose) {
+                logInfo('No eligible PRs to process — waiting for current PR to be merged or closed.');
+            }
+            return buildNoChangeResult(featureId, context.pullRequests.length);
+        }
+        // ======================================
         // Agent: Create/Update branches
         // ======================================
         if (verbose) {
@@ -285,6 +297,52 @@ export const executeFeaturePRs = async (options, config) => {
         return buildExecutionErrorResult(featureId, errorMessage);
     }
 };
+/**
+ * Filter PRs to only include the next eligible pending PR.
+ *
+ * - Already-executed PRs (branch_created, pr_opened, in_review, approved) are
+ *   kept for incremental sync.
+ * - Merged/closed PRs are excluded (no branch work needed).
+ * - Only the first pending PR is included, and only if the PR immediately
+ *   before it (by sequence) is merged or closed (or it's the first PR).
+ * - All other pending PRs are blocked until the current one completes.
+ */
+function filterToEligiblePRs(pullRequests, verbose) {
+    const sorted = [...pullRequests].sort((a, b) => a.sequence - b.sequence);
+    const eligible = [];
+    let foundPendingToProcess = false;
+    let blockedCount = 0;
+    for (let i = 0; i < sorted.length; i++) {
+        const pr = sorted[i];
+        if (pr.status === 'merged' || pr.status === 'closed') {
+            // Done — no branch work needed
+            continue;
+        }
+        if (pr.status !== 'pending') {
+            // Already executed but not done — include for incremental sync
+            eligible.push(pr);
+            continue;
+        }
+        // PR is pending
+        if (foundPendingToProcess) {
+            blockedCount++;
+            continue;
+        }
+        // Check if the previous PR (by sequence) is merged or closed
+        const prevPR = i > 0 ? sorted[i - 1] : null;
+        if (!prevPR || prevPR.status === 'merged' || prevPR.status === 'closed') {
+            eligible.push(pr);
+        }
+        else {
+            blockedCount++;
+        }
+        foundPendingToProcess = true;
+    }
+    if (blockedCount > 0 && verbose) {
+        logInfo(`⏳ ${blockedCount} pending PR(s) waiting for previous PR to be merged or closed`);
+    }
+    return eligible;
+}
 /**
  * Execute an agent query for branch creation/update
  */

package/dist/phases/pr-resolve/prompts.js

@@ -73,8 +73,9 @@ export function createResolveUserPrompt(unresolvedThreads) {
     let commentIndex = 0;
     for (const thread of unresolvedThreads) {
         const firstComment = thread.comments.nodes[0];
-        if (!firstComment)
+        if (!firstComment) {
             continue;
+        }
         commentIndex++;
         const commentId = `comment_${commentIndex}`;
         commentIdToThreadId.set(commentId, thread.id);

package/dist/phases/pr-resolve/workspace.d.ts

@@ -18,7 +18,7 @@ export declare function prepareWorkspace(owner: string, repo: string, headRef: s
 /**
  * Push changes from workspace back to remote.
  */
-export declare function pushChanges(repoPath: string, headRef: string, token: string, verbose?: boolean): boolean;
+export declare function pushChanges(repoPath: string, headRef: string, token: string, _verbose?: boolean): boolean;
 /**
  * Check if there are uncommitted changes in the workspace.
  */

package/dist/phases/pr-resolve/workspace.js

@@ -98,7 +98,7 @@ export function prepareWorkspace(owner, repo, headRef, prNumber, token, verbose)
 /**
  * Push changes from workspace back to remote.
  */
-export function pushChanges(repoPath, headRef, token, verbose) {
+export function pushChanges(repoPath, headRef, token, _verbose) {
     const gitCredentialArgs = buildCredentialArgs(token);
     try {
         execFileSync('git', [...gitCredentialArgs, 'push', 'origin', headRef], {

package/dist/phases/pr-review/__tests__/review-comments.test.js

@@ -17,11 +17,13 @@ function mapCommentsToReviewPayload(agentComments, files) {
     const result = [];
     for (const comment of agentComments) {
         const lineToPosition = fileLineToPosition.get(comment.file);
-        if (!lineToPosition)
+        if (!lineToPosition) {
             continue;
+        }
         const positionResult = findClosestPosition(comment.line, lineToPosition);
-        if (!positionResult)
+        if (!positionResult) {
             continue;
+        }
         let body = comment.comment;
         if (positionResult.actualLine !== comment.line) {
             body = `**Note**: Comment originally for line ${comment.line}, adjusted to line ${positionResult.actualLine} (nearest line in diff).\n\n${body}`;

package/dist/phases/pr-review/index.js

@@ -14,9 +14,7 @@ import { createStandaloneReviewSystemPrompt, createStandaloneReviewUserPrompt, }
 /**
  * Review a standalone PR and post comments to GitHub.
  */
-export async function reviewStandalonePR(options
-// eslint-disable-next-line complexity
-) {
+export async function reviewStandalonePR(options) {
     const { pullRequestUrl, githubToken, verbose, prId } = options;
     logInfo(`Starting standalone PR review: ${pullRequestUrl}`);
     try {

package/dist/phases/pr-shared/agent-utils.js

@@ -24,7 +24,6 @@ export async function* createPromptGenerator(prompt) {
 /**
  * Extract text content from assistant message content array.
  */
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function extractTextFromContent(
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 content, verbose) {

package/dist/phases/pr-shared/context.d.ts

@@ -3,8 +3,8 @@
  * Reuses GitHub API utilities from code-review/context.ts without feature dependencies.
  */
 import { type PRCommit, type PRData, type PRFile } from '../code-review/context.js';
-export { parsePullRequestUrl } from '../code-review/context.js';
 export type { PRCommit, PRData, PRFile } from '../code-review/context.js';
+export { parsePullRequestUrl } from '../code-review/context.js';
 export interface StandalonePRContext {
     pullRequestUrl: string;
     pullRequestNumber: number;

package/dist/phases/pr-splitting/index.js

@@ -27,9 +27,7 @@ async function* prompt(analysisPrompt) {
  * then uses AI to produce a PR split plan saved to the database.
  * Human review is expected before running the pr-execution phase.
  */
-export const splitFeatureIntoPRs = async (options, config
-// eslint-disable-next-line complexity -- orchestration function with context fetching, agent execution, and result processing
-) => {
+export const splitFeatureIntoPRs = async (options, config) => {
     const { featureId, verbose, replaceExisting } = options;
     if (verbose) {
         logInfo(`Starting PR splitting for feature ID: ${featureId}`);

package/dist/phases/technical-design/index.js

@@ -24,9 +24,7 @@ async function* prompt(analysisPrompt) {
         setTimeout(res, 10000);
     });
 }
-export const generateTechnicalDesign = async (options, config, checklistContext
-// eslint-disable-next-line complexity -- orchestration function with context assembly, design generation, and verification
-) => {
+export const generateTechnicalDesign = async (options, config, checklistContext) => {
     const { featureId, verbose } = options;
     if (verbose) {
         logInfo(`Starting technical design generation for feature ID: ${featureId}`);

package/dist/phases/test-cases-analysis/index.js

@@ -6,9 +6,7 @@ import { executeTestCasesAnalysisQuery, parseAnalysisResult } from './agent.js';
 import { prepareTestCasesAnalysisContext } from './context.js';
 import { buildTestCasesAnalysisResult, deleteSpecificTestCases, deleteTestCaseArtifacts, getAllDraftTestCaseIds, resetReadyTestCasesToDraft, saveTestCasesAsDraft, updateTestCasesToReady, } from './outcome.js';
 import { createTestCasesAnalysisSystemPrompt } from './prompts.js';
-export const analyseTestCases = async (options, config, checklistContext
-// eslint-disable-next-line complexity -- orchestration function with context assembly, agent execution, and result processing
-) => {
+export const analyseTestCases = async (options, config, checklistContext) => {
     const { featureId, verbose } = options;
     if (verbose) {
         logInfo(`Starting test cases analysis for feature ID: ${featureId}`);

package/dist/phases/user-stories-analysis/index.js

@@ -6,9 +6,7 @@ import { executeUserStoriesAnalysisQuery, parseAnalysisResult, } from './agent.j
 import { prepareUserStoriesAnalysisContext } from './context.js';
 import { buildUserStoriesAnalysisResult, deleteSpecificUserStories, deleteUserStoryArtifacts, getAllDraftUserStoryIds, resetReadyUserStoriesToDraft, saveUserStoriesAsDraft, updateUserStoriesToReady, } from './outcome.js';
 import { createUserStoriesAnalysisSystemPrompt } from './prompts.js';
-export const analyseUserStories = async (options, config, checklistContext
-// eslint-disable-next-line complexity -- orchestration function with context assembly, agent execution, and result processing
-) => {
+export const analyseUserStories = async (options, config, checklistContext) => {
     const { featureId, verbose } = options;
     if (verbose) {
         logInfo(`Starting user stories analysis for feature ID: ${featureId}`);

package/package.json

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