Skip to content

Tech review iteration loop #81

@mcljot

Description

@mcljot

Tech review consumes 1 million tokens for passes 2 and 3 over 2 tickets in a batch for limited benefit.

Tech review iteration loop — diminishing returns
Both tickets followed the same pattern: iteration 1 returned MEDIUM with critical+significant issues, fixes were applied, iteration 2 returned MEDIUM with only SME/process items remaining, iteration 3 returned MEDIUM with the same SME/process items.

Iterations 2 and 3 consumed ~250k+ tokens each ticket for essentially zero incremental improvement. The reviewer correctly identified in iteration 1 that the remaining issues "require SME input and cannot be fixed by editing docs."

Suggested fix: Add an early-exit condition to the orchestrator's iteration loop: if the reviewer reports MEDIUM with zero critical and zero significant issues, treat it as acceptable without further iterations. The current logic only checks the confidence label, not the severity breakdown.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions