Progress bar resets to 1/total_items on completion in run.py:146 (completed=True)

[himanshu231204]

Description

In openagent_eval/cli/commands/run.py at line 146, completed=True is passed to Progress.update(). In Rich's progress bar, True is coerced to 1 in a numeric context, causing the progress bar to drop from 100% to 1/total_items when the evaluation completes.

Location

openagent_eval/cli/commands/run.py, line 146

Current Code

progress.update(task, description="Complete!", completed=True)

Why This Is Wrong

• completed=True evaluates to 1 when used numerically
• The task's total is set to total_items (e.g., 50 items) at line 126
• Before this line, the bar correctly shows 50/50 (100%)
• After this line, the bar drops to 1/50 (2%) — it looks like the progress reset

Expected Fix

# Option A: Advance to completion properly
progress.update(task, description="Complete!", completed=total_items)

# Option B: Use advance + update
progress.update(task, description="Complete!")

Impact

• Visual bug: The progress bar visibly drops from 100% to near-zero right at the end
• This is especially noticeable for large evaluations with many items
• Creates a poor user experience — looks like something went wrong

Severity

High — Visible UI bug that affects all evaluation runs.

Steps to Reproduce

1. Create a config file with a dataset of 10+ items
2. Run oaeval run config.yaml
3. Observe the progress bar drops to a near-zero percentage when the evaluation completes and the description changes to "Complete!"

[himanshu231204]

/oc explain the issue

[opencode-agent]

Model not found: opencode/minimax-m2.5-free. Did you mean: minimax-m2.5, minimax-m2.7, deepseek-v4-flash-free?

opencode session  |  github run

[himanshu231204]

/oc explain the issue

[opencode-agent]

The bug is at openagent_eval/cli/commands/run.py:146 — but looking at the current code, it already reads completed=total_items, which is the correct fix. The issue description was written against an older version.

The original bug: completed=True was passed instead of completed=total_items. In Rich's Progress.update(), True is coerced to 1 in a numeric context, so a bar that showed 50/50 (100%) would drop to 1/50 (2%) the moment evaluation finished — a jarring visual glitch making it look like progress reset.

The fix (Option A from the issue) swaps True for total_items, keeping the bar at total_items/total_items (100%) on completion. This appears to already be applied in the working tree.

opencode session  |  github run