State: Add local_iteration attribute

[li-boxuan]

We have iteration that tracks OpenDevin progress. When OpenDevin exceeds config.max_iterations, it pauses execution.

The introduction of agent delegation mechanism essentially makes OpenDevin a multi-agentic system. Without this PR, every agent instance keeps track of their own "local" iterations. What this means is with max_iterations = 10, OpenDevin could speak to LLM for 10 * 10 = 100 times in the worst case (root agent delegates 10 times). This defeats the purpose of config.max_iterations.

This PR fixes this problem by introducing local_iteration that had the same behavior as prior iteration. The attribute iteration now becomes a global/shared value per task.

Do we really need configs for both max iterations per task and max iterations per subtask? I don't think so. Thus, this PR doesn't introduce any new config.

Closes #2121

[tobitege]

For usability purposes: how does the UI behave one the limit is reached?
What options does the user have then?

[li-boxuan]

For usability purposes: how does the UI behave one the limit is reached?
What options does the user have then?

It's as bad as the status quo. No functional change. The user needs to click on continue to proceed one more step.

I know you tried and it didn't work well for Claude, presumably because Claude was too conservative and cautious.

[tobitege]

I know you tried and it didn't work well for Claude, presumably because Claude was too conservative and cautious.

Yeh, we need to come up with a PR to make this a more graceful "ending" of a chat session.
Like give options like resetting the counter, or start a new session with a summary of the current one or something similar.
If the use isn't actually done with what they wanted to achieve, it's a bit frustrating hanging there with one-step iterations.

[tobitege]

Haven't run it myself, but lgtm.
Thanks!

[li-boxuan]

Yeh, we need to come up with a PR to make this a more graceful "ending" of a chat session.
Like give options like resetting the counter, or start a new session with a summary of the current one or something similar.

I am in favor of this idea. I don't want to take it though.

[neubig]

Tests are failing due to #2992