Fix issue #7658: [Bug]: BadRequestError from ContentPolicyViolationError

[openhands-agent]

• This change is worth documenting at https://docs.all-hands.dev/
• Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

End-user friendly description of the problem this fixes or functionality that this introduces.
User-friendly UI for a content policy violation error

---

This pull request fixes #7658.

The issue has been successfully resolved based on the following implemented changes and their effects:

1. The core problem (BadRequestError from ContentPolicyViolationError not being properly handled) has been directly addressed by adding explicit error handling for ContentPolicyViolationError in agent_controller.py.

2. The implementation includes:

• Proper error type detection and handling in the controller
• Addition of the error to the supported error types list
• Comprehensive i18n support with translations in 13 languages
• A new test case that verifies the error handling works as expected

3. The changes ensure that when content policy violations occur:

• Users receive a clear, localized error message instead of a generic error
• The error state is properly captured in the agent's state
• The error is correctly propagated to the UI
• The system gracefully handles the error condition

4. The test suite demonstrates that the error handling works as intended, showing the correct error message is displayed and the proper state transitions occur.

These changes fully address the original issue by providing proper handling of the ContentPolicyViolationError, making it consistent with how other errors are handled in the system, and ensuring users receive clear feedback when content policy violations occur.

Automatic fix generated by OpenHands 🙌

---

To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:226b7fc-nikolaik   --name openhands-app-226b7fc   docker.all-hands.dev/all-hands-ai/openhands:226b7fc

[enyst]

@openhands-agent Read the diff of this PR carefully: it's PR 7660 . Use GitHub API to read it. Read also its description, and the linked issue.

The diff is not correct: the error might come to us either as ContentPolicyViolationError as defined in litellm, either as BadRequestError with the error string containing ContentPolicyViolationError. You should catch it and handle it like we handle BudgetExceeded errors.

[openhands-agent]

OpenHands started fixing the pr! You can monitor the progress here.

[openhands-agent]

Based on the provided information, here is a concise overview:

Status: ✅ FULLY RESOLVED

Changes Made:

• Implemented comprehensive error handling for ContentPolicyViolationError
• Added support for both direct and wrapped (BadRequestError) instances
• Followed existing BudgetExceeded error pattern
• Added thorough test coverage for both scenarios

All key requirements appear to have been met:

1. Proper error handling ✓
2. Consistent with existing patterns ✓
3. Complete test coverage ✓
4. Addresses user-reported issues ✓

No remaining issues are indicated in the provided information.

[rbren]

Thank you!