Skip to content

Remove max_repair_rounds plumbing#402

Open
zhongkaifu wants to merge 4 commits intomainfrom
codex/remove-max_repair_rounds-and-its-callers
Open

Remove max_repair_rounds plumbing#402
zhongkaifu wants to merge 4 commits intomainfrom
codex/remove-max_repair_rounds-and-its-callers

Conversation

@zhongkaifu
Copy link
Copy Markdown
Owner

Motivation

  • The planner no longer exposes a configurable max_repair_rounds surface, so call sites and docs that passed or referenced this parameter were removed to keep the API consistent.
  • Simplify CLI and example scripts by relying on the planner's internal repair behavior rather than plumbing an external round limit.

Description

  • Removed the --max-repair-rounds CLI argument and stopped passing max_repair_rounds into update_workflow_with_two_pass in update_workflow.py.
  • Removed max_repair_rounds arguments passed from example and server code by updating build_workflow.py and webapp/server.py to call plan_workflow_with_two_pass / update_workflow_with_two_pass without the deprecated parameter.
  • Updated tests/test_update_workflow_planner.py to match the new planner signature and revised the README text to drop the max_repair_rounds reference.

Testing

  • No automated tests were executed as part of this change (tests were updated to reflect the API change but test suite was not run).

Codex Task

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