Resolve Issue: API Error Handling Improvement

[patched-codes]

This pull request from patched fixes issue.

---

• File changed: patchwork/common/client/llm/protocol.py

• File changed: patchwork/common/client/llm/google_.py

[patched-admin]

File Changed: patchwork/common/client/llm/google_.py

Rule 1: Do not ignore potential bugs in the code

Details: Changing the type hint from str to dict for response_format parameter could introduce potential type-related bugs. The parameter now accepts a broader type (dict) which might lead to runtime errors if the dictionary structure doesn't match what completion_create_params.ResponseFormat expects.

Affected Code Snippet:

response_format: dict | completion_create_params.ResponseFormat | NotGiven = NOT_GIVEN,

Start Line: 267
End Line: 267

File Changed: patchwork/common/client/llm/protocol.py

Rule 1: Do not ignore potential bugs in the code

Details: Potential bug risk identified. Changing the type hint from str to dict for response_format parameter could introduce runtime errors if the code expects string values in other parts of the codebase. The type change seems to broaden the accepted types without clear validation mechanisms.

Affected Code Snippet:

response_format: dict | completion_create_params.ResponseFormat | NotGiven = NOT_GIVEN,

Start Line: 141
End Line: 141

---

Rule 2: Do not overlook possible security vulnerabilities

Details: Security concern identified. Accepting a dictionary input instead of a string for response_format increases the attack surface. Without proper input validation, arbitrary dictionaries could potentially lead to injection vulnerabilities or unexpected behavior in the response formatting logic.

Affected Code Snippet:

response_format: dict | completion_create_params.ResponseFormat | NotGiven = NOT_GIVEN,

Start Line: 141
End Line: 141