As part of the logic to decide if an error is retryable, isRetryableError() looks into the human-readable message of the error to look for a bunch of substrings. These substrings include common phrases such as "network error". This can easily lead to false positives, like one I encountered where it considered this error as retryable, leading to time wasted debugging an error that should've been trivial. This should be replaced with a more correct alternative that, ideally, doesn't look into a human readable message.