fix: add idx for task ordering, tests

[greysonlalonde]

fixes #3953

---

Note

Use 1-indexed task_number to append plans to matching tasks, with warnings for duplicates/missing, and expand tests to cover ordering and validation.

• Planning integration:
  • Update Crew._handle_crew_planning to append plans by task_number mapping instead of positional zip.
  • Log warnings for duplicate task_number (use first) and for missing plans; preserve original descriptions.
• Models:
  • Change PlanPerTask to include required task_number: int (>=1) and keep task, plan fields.
• Defaults:
  • Set planning agent LLM default to "gpt-4o-mini" in CrewPlanner.
• Tests:
  • Add integration tests ensuring out-of-order, missing, and duplicate task_number plans are handled correctly.
  • Enforce task_number validation and adjust mocks for knowledge/tools in task summaries.

^{Written by Cursor Bugbot for commit 3bc4b6e. This will update automatically on new commits. Configure here.}

[cursor]

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on December 28

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.