Handle JSONDecodeError in is_server_or_retry_error function

[danielchalef]

Important

Add json.decoder.JSONDecodeError handling to is_server_or_retry_error in client.py for retry logic.

• Error Handling:
  • Update is_server_or_retry_error in client.py to handle json.decoder.JSONDecodeError.
  • Ensures JSON decoding errors are treated as retryable errors.

^{This description was created by for 1d840bf. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 1d840bf in 54 seconds

More details

• Looked at 13 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 2 drafted comments based on config settings.

1. graphiti_core/llm_client/client.py:38

• Draft comment:
  Consider adding a comment explaining why JSONDecodeError is considered a retryable error. This will help future developers understand the reasoning behind this decision.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR adds JSONDecodeError to the list of exceptions that trigger a retry. This is a logical decision if JSON decoding errors are expected to be transient and recoverable by retrying the request.

2. graphiti_core/llm_client/client.py:37

• Draft comment:
  The function name is_server_or_retry_error could be more descriptive. Consider renaming it to something like is_retryable_error to better convey its purpose.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The function name should be more descriptive to indicate its purpose clearly.

Workflow ID: wflow_2mYHx9sigH9iSGBx

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.