Skip to content

Commit 0723fe0

Browse files
alpslaclaude
andcommitted
fix(v9): Skip AI calls in report formatter for EXISTING_REST issues
SESSION 92: Report formatter was making duplicate AI calls: - Now uses pre-generated fix from enrichWithAI if available - Skips AI generation for EXISTING_REST issues - Only generates new fixes for actionable issues without existing fixes This significantly reduces API calls by: 1. Avoiding duplicate AI calls for already-enriched issues 2. Skipping AI for EXISTING_REST (unchanged on both branches) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent cde12cb commit 0723fe0

1 file changed

Lines changed: 12 additions & 2 deletions

File tree

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

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -663,8 +663,18 @@ ${Object.entries(categoryCounts).map(([cat, count]) =>
663663
}
664664

665665
private async formatIssueWithEducation(issue: Issue, severity: string): Promise<string> {
666-
// Get dynamic fix suggestion from specialized agent
667-
const fixSuggestion = await this.generateDynamicFix(issue);
666+
// SESSION 92: Use pre-generated fix if available, skip AI for EXISTING_REST
667+
let fixSuggestion = (issue as any).fixSuggestion;
668+
669+
// Only generate new fix if:
670+
// 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);
674+
675+
if (!fixSuggestion && isActionable) {
676+
fixSuggestion = await this.generateDynamicFix(issue);
677+
}
668678

669679
let issueReport = `#### ${issue.title}
670680

0 commit comments

Comments
 (0)