Avoid double-including error message in retry when Mode.TOOLS

[ilyanekhay-uta]

fixes #513

---

	🚀 This PR description was created by Ellipsis for commit c88e1d5.

Summary:

This PR modifies the reask_messages function in instructor/retry.py to prevent double-inclusion of error messages when the mode is Mode.TOOLS.

Key points:

• Modified reask_messages function in instructor/retry.py.
• Changed if-else condition to avoid double-including error messages in Mode.TOOLS.
• Error message is now only included for the relevant mode.

---

Generated with ❤️ by ellipsis.dev

[ellipsis-dev]

👍 Looks good to me!

• Reviewed the entire pull request up to c88e1d5
• Looked at 24 lines of code in 1 files
• Took 40 seconds to review

More info

• Skipped 0 files when reviewing.
• Skipped posting 1 additional comments because they didn't meet confidence threshold of 85%.

1. instructor/retry.py:30:

• Assessed confidence : 0%
• Comment:
  This change in control flow seems to be in line with the PR's intent of avoiding double-including the error message in the retry when Mode.TOOLS. However, it would be good to confirm this by checking the usage of the reask_messages function in the codebase.
• Reasoning:
  The PR changes the control flow of the reask_messages function. Previously, if mode was Mode.TOOLS, the function would yield a message for each tool call, and then yield another message if mode was Mode.MD_JSON. Now, the function will only yield the Mode.MD_JSON message if mode is not Mode.TOOLS. This seems to be the intended behavior, as the PR title suggests that the error message was being included twice in the retry when mode was Mode.TOOLS. However, I would like to confirm this by checking the usage of the reask_messages function in the codebase.

Workflow ID: wflow_2n5BrUwzPKx9b4lm

---

Not what you expected? You can customize the content of the reviews using rules. Learn more here.