Skip to content

Restore update workflow planner test and remove references to start node#409

Merged
zhongkaifu merged 3 commits intomainfrom
codex/remove-test-test_update_workflow_planner
Jan 27, 2026
Merged

Restore update workflow planner test and remove references to start node#409
zhongkaifu merged 3 commits intomainfrom
codex/remove-test-test_update_workflow_planner

Conversation

@zhongkaifu
Copy link
Copy Markdown
Owner

Motivation

  • Reinstate the removed planner test while eliminating hardcoded references to a start node to match updated workflow conventions.
  • Ensure the planner update flow is still exercised against an existing graph shape rather than a special start node.

Description

  • Re-added tests/test_update_workflow_planner.py and replaced the original start node with a seed node in the test fixture.
  • Updated the fake planner to produce a summarize node that depends on seed and that uses {{ result_of.seed.message }} for its params.
  • Adjusted the action_registry and all assertions to reference demo.seed and seed instead of demo.start and start.
  • Kept the same test scaffolding that monkeypatches plan_workflow_structure_with_llm, analyze_user_requirement, _ensure_actions_registered_or_repair, and _validate_and_repair_workflow to isolate planner behavior.

Testing

  • No automated tests were executed for this change.

Codex Task

@zhongkaifu zhongkaifu merged commit 98b5230 into main Jan 27, 2026
1 check failed
@zhongkaifu zhongkaifu deleted the codex/remove-test-test_update_workflow_planner branch January 27, 2026 23:40
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