CODEFIX-1426 Interactively list and assign issues to Remediation Agent#239
CODEFIX-1426 Interactively list and assign issues to Remediation Agent#239vnaskos-sonar wants to merge 6 commits intomasterfrom
Conversation
c146db0 to
0a5902b
Compare
| async function fetchEligibleIssues( | ||
| issuesClient: IssuesClient, | ||
| orgKey: string | undefined, | ||
| projectKey: string, | ||
| ): Promise<SonarQubeIssue[]> { | ||
| const result = await issuesClient.searchIssues({ | ||
| projects: projectKey, | ||
| organization: orgKey, | ||
| issueStatuses: 'OPEN,CONFIRMED', | ||
| fixableByAgent: true, | ||
| ps: MAX_PAGE_SIZE, | ||
| p: 1, | ||
| }); | ||
| return result.issues; |
There was a problem hiding this comment.
Missing test coverage: fetchEligibleIssues fetches only page 1 (p: 1, ps: MAX_PAGE_SIZE = 500). For a project with more than 500 agent-fixable issues the remaining issues are silently omitted — the user sees "500 eligible issues found" with no indication that the list is truncated. No test covers this truncation scenario.
Either add pagination or document the cap and emit a warning when issues.length === MAX_PAGE_SIZE.
- Mark as noise
0a5902b to
a77a508
Compare
SummaryThis PR adds a new What's new:
Implementation details:
What reviewers should knowStart with the command implementation:
Review the tests next:
Quick checklist for reviewers:
Watch for:
|
|



No description provided.