fix(tools): raise ToolUsageError instead of bare raise in _original_tool_calling

[Aakash326]

Fixes #6430.

The bare raise in the non-dict-arguments branch of _original_tool_calling
had no active exception to re-raise, so Python raised
RuntimeError: No active exception to re-raise instead of a meaningful
ToolUsageError. Replaced it with an explicit raise ToolUsageError(...)
matching the sibling return statement and the pattern used elsewhere in
this method.

Added regression test covering both raise_error=True and raise_error=False
paths.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 9f3b20a3-8be6-4f93-99eb-3f8d8887e2de

📥 Commits

Reviewing files that changed from the base of the PR and between 7baf8f9 and 0bb488d.

📒 Files selected for processing (2)

• lib/crewai/src/crewai/tools/tool_usage.py
• lib/crewai/tests/tools/test_tool_usage.py

---

📝 Walkthrough

Walkthrough

Changes

Tool argument error handling

Layer / File(s)	Summary
Normalize invalid argument errors lib/crewai/src/crewai/tools/tool_usage.py, lib/crewai/tests/tools/test_tool_usage.py	_original_tool_calling raises ToolUsageError for non-dictionary arguments when requested, and regression tests verify both raised and returned errors use the localized message.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main fix in _original_tool_calling and matches the change set.
Description check	✅ Passed	The description accurately explains the bug fix and the added regression test.
Linked Issues check	✅ Passed	The change replaces the bare raise with ToolUsageError and preserves the non-raise_error path as required by #6430.
Out of Scope Changes check	✅ Passed	The PR stays focused on the reported bug fix and its regression test, with no obvious unrelated changes.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[ErenAta16]

Same one-line fix as #6486, #6512, #6511, #6499 — this is one of five open PRs (added this one to the tally) independently fixing the same bare-raise-outside-except bug in _original_tool_calling. Reproduced the bug standalone: the bare raise here raises RuntimeError: No active exception to reraise rather than the intended ToolUsageError, since it is outside any except block. This PR's fix is correct and equivalent to the others. Left a fuller comparison on #6486, which constructs the error once and reuses it for both branches rather than twice — noting the duplication here so only one lands.