Skip to content

Commit bbfb20d

Browse files
committed
chore: aligned review retry terminology
1 parent 7911443 commit bbfb20d

3 files changed

Lines changed: 5 additions & 5 deletions

File tree

api/src/orchestrator/orchestrator.service.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,7 @@ function setup(overrides: { job?: Record<string, unknown> } = {}) {
133133
persist: resolved(undefined),
134134
threshold: 85,
135135
maxPasses: 5,
136-
maxParseRetries: 2,
136+
maxRetries: 2,
137137
};
138138

139139
const verify = {

api/src/orchestrator/orchestrator.service.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1569,12 +1569,12 @@ export class OrchestratorService {
15691569
// After MAX_REVIEW_RETRIES in a row the model is unlikely to self-correct, so fall through
15701570
// to the draft-PR path rather than burning the entire pass budget on malformed output.
15711571
const unparseablePasses = priorUnparseable + (parsed === null ? 1 : 0);
1572-
const maxParseRetries = this.review.maxParseRetries;
1572+
const maxRetries = this.review.maxRetries;
15731573

15741574
// Only revise when the review produced parseable, actionable issues.
15751575
// If the output couldn't be parsed, retry the review so the agent gets another
15761576
// chance to emit valid JSON — but cap consecutive unparseable retries.
1577-
if (pass < maxPasses && (parsed !== null || unparseablePasses < maxParseRetries)) {
1577+
if (pass < maxPasses && (parsed !== null || unparseablePasses < maxRetries)) {
15781578
if (parsed !== null) {
15791579
await this.jobs.transition(jobId, 'REVISING', {
15801580
reason: `addressing review pass ${pass}`,
@@ -1589,7 +1589,7 @@ export class OrchestratorService {
15891589
}
15901590

15911591
const reason =
1592-
unparseablePasses >= maxParseRetries
1592+
unparseablePasses >= maxRetries
15931593
? `Self-review produced unparseable output ${unparseablePasses} times in a row. Opening a draft PR for human review.`
15941594
: `Self-review didn't reach the confidence threshold after ${pass} passes (best ${result.confidence}/100). Opening a draft PR for human review.`;
15951595

api/src/review/review.service.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ export class ReviewService {
5656
}
5757

5858
/** Max consecutive unparseable review passes tolerated before bailing to a draft PR. */
59-
get maxParseRetries(): number {
59+
get maxRetries(): number {
6060
return this.config.get('MAX_REVIEW_RETRIES');
6161
}
6262

0 commit comments

Comments
 (0)