Issue with basic reviewer (infinite loop while handling error)

[benjaminpillot]

Hi there,

Hope it will help, returning to lattereview, I faced a "review error" using LiteLLM provider along with the GPT API (not able to parse the JSON file), but then fell into a "retrying infinite loop", which ended consuming API credits. Anyway, I dug further and found in the review_item function (line 253 to 275) in basic_reviewer.py that while num_tried is initialized to 0 (line 257), it is never iterated later, so it turns into an infinite loop. It is just a very little fix, but it may save people some time and money ;)

Thank you again for this very nice package.

Benjamin

[PouriaRouzrokh]

Thanks for catching this. I believe this should have been fixed now. Could you please confirm?

[benjaminpillot]

I think the pypi package has not been rebuilt (still version 1.1.0 for may 2025), so for the moment it only works through manual installation of the git repository.

[PouriaRouzrokh]

Should be fixed now