fix(service-error-classification): add TypeError as transient error (smithy-lang#1574)

* fix(service-error-classification): add TypeError as transient error

* chore(changeset): changeset

* fix(service-error-classification): add message checks

Author: siddsriv

.changeset/large-donkeys-notice.md

@@ -0,0 +1,5 @@
+---
+"@smithy/service-error-classification": patch
+---
+
+add browser connection issues as transient errors to retry on

packages/service-error-classification/src/index.ts

@@ -20,6 +20,28 @@ export const isClockSkewError = (error: SdkError) => CLOCK_SKEW_ERROR_CODES.incl
  */
 export const isClockSkewCorrectedError = (error: SdkError) => error.$metadata?.clockSkewCorrected;
 
+/**
+ *
+ * @internal
+ */
+export const isBrowserNetworkError = (error: SdkError) => {
+  const errorMessages = new Set([
+    "Failed to fetch", // Chrome
+    "NetworkError when attempting to fetch resource", // Firefox
+    "The Internet connection appears to be offline", // Safari 16
+    "Load failed", // Safari 17+
+    "Network request failed", // `cross-fetch`
+  ]);
+
+  const isValid = error && error instanceof TypeError;
+
+  if (!isValid) {
+    return false;
+  }
+
+  return errorMessages.has(error.message);
+};
+
 export const isThrottlingError = (error: SdkError) =>
   error.$metadata?.httpStatusCode === 429 ||
   THROTTLING_ERROR_CODES.includes(error.name) ||
@@ -36,6 +58,7 @@ export const isTransientError = (error: SdkError, depth = 0): boolean =>
   TRANSIENT_ERROR_CODES.includes(error.name) ||
   NODEJS_TIMEOUT_ERROR_CODES.includes((error as { code?: string })?.code || "") ||
   TRANSIENT_ERROR_STATUS_CODES.includes(error.$metadata?.httpStatusCode || 0) ||
+  isBrowserNetworkError(error) ||
   (error.cause !== undefined && depth <= 10 && isTransientError(error.cause, depth + 1));
 
 export const isServerError = (error: SdkError) => {