Skip to content

Commit cf0a7cd

Browse files
alpslaclaude
andcommitted
fix(v9): Use category instead of status for EXISTING_REST check
The IssueStatus type is 'new' | 'existing' | 'resolved' which doesn't include EXISTING_REST. The category property set during two-branch comparison has the correct values (NEW, EXISTING_MODIFIED, EXISTING_REST, RESOLVED) to determine if an issue needs AI processing. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent 0723fe0 commit cf0a7cd

1 file changed

Lines changed: 5 additions & 3 deletions

File tree

packages/agents/src/two-branch/analyzers/v9-report-formatter.ts

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -668,9 +668,11 @@ ${Object.entries(categoryCounts).map(([cat, count]) =>
668668

669669
// Only generate new fix if:
670670
// 1. No pre-existing fix, AND
671-
// 2. Issue is actionable (NEW or EXISTING_MODIFIED)
672-
const isActionable = issue.status !== 'EXISTING_REST' &&
673-
(issue.status === 'new' || issue.status === 'modified' || !issue.status);
671+
// 2. Issue is actionable (NEW or EXISTING_MODIFIED based on category)
672+
// Note: category is set during two-branch comparison and indicates actionability
673+
const issueCategory = (issue as any).category as string | undefined;
674+
const isActionable = issueCategory !== 'EXISTING_REST' && issueCategory !== 'RESOLVED' &&
675+
(issueCategory === 'NEW' || issueCategory === 'EXISTING_MODIFIED' || !issueCategory);
674676

675677
if (!fixSuggestion && isActionable) {
676678
fixSuggestion = await this.generateDynamicFix(issue);

0 commit comments

Comments
 (0)