Skip to content

Remove max_repair_rounds parameter and its callers#403

Merged
zhongkaifu merged 14 commits intomainfrom
codex/remove-max_repair_rounds-and-its-callers-9sd4c5
Jan 19, 2026
Merged

Remove max_repair_rounds parameter and its callers#403
zhongkaifu merged 14 commits intomainfrom
codex/remove-max_repair_rounds-and-its-callers-9sd4c5

Conversation

@zhongkaifu
Copy link
Copy Markdown
Owner

Motivation

  • Remove the external max_repair_rounds knob from CLI, helper and web entrypoints to simplify the planner API and avoid exposing an extra tuning parameter.
  • Treat repair iteration limits as an internal implementation detail rather than a caller-provided argument.

Description

  • Drop the --max-repair-rounds CLI argument and stop passing max_repair_rounds into update_workflow_with_two_pass in update_workflow.py.
  • Remove the max_repair_rounds argument from the plan_workflow_with_two_pass call in the interactive helper in build_workflow.py.
  • Stop passing max_repair_rounds from the web API dispatch logic in webapp/server.py.
  • Update the test tests/test_update_workflow_planner.py to stop supplying the removed argument, and adjust the README wording to refer to repair loop behavior without referencing the removed parameter.

Testing

  • The unit test tests/test_update_workflow_planner.py was updated to remove the now-unused max_repair_rounds argument; no automated test suite was executed as part of this change.
  • No automated tests were run in the rollout (no failures reported).

Codex Task

@zhongkaifu zhongkaifu merged commit fa1e79b into main Jan 19, 2026
1 check failed
@zhongkaifu zhongkaifu deleted the codex/remove-max_repair_rounds-and-its-callers-9sd4c5 branch January 19, 2026 06:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant