Retry transient server errors for theme API reads - #8894

@@ -1,14 +1,24 @@
 import { Headers } from 'form-data';
+import { ClientError } from 'graphql-request';
 export type API = 'admin' | 'storefront-renderer' | 'partners' | 'business-platform' | 'app-management';
 export declare const allAPIs: API[];
-export type NetworkRetryBehaviour = {
+interface CommonRetryBehaviour {
+    recordCommandRetries?: boolean;
+    /**
+     * Retry, with backoff, requests that fail with a transient server-side error: a GraphQL
+     * `INTERNAL_SERVER_ERROR` (usually on an HTTP 200) or an HTTP 502/503/504.
+     *
+     * Only enable this for requests that are safe to repeat. GraphQL requests only honour it for
+     * queries; mutations are never retried on these errors.
+     */
+    retryTransientServerErrors?: boolean;
+}
+export type NetworkRetryBehaviour = ({
     useNetworkLevelRetry: true;
     maxRetryTimeMs: number;
-    recordCommandRetries?: boolean;
-} | {
+} & CommonRetryBehaviour) | ({
     useNetworkLevelRetry: false;
-    recordCommandRetries?: boolean;
-};
+} & CommonRetryBehaviour);
 type RequestOptions<T> = {
     request: () => Promise<T>;
     url: string;
@@ -52,6 +62,15 @@ export declare function isNetworkError(error: unknown): boolean;
  * @returns A boolean indicating if the request was aborted.
  */
 export declare function isAbortedFetchError(error: unknown): boolean;
+/**
+ * Checks whether an error is a transient server-side failure that is likely to succeed if retried:
+ * a GraphQL `INTERNAL_SERVER_ERROR` (Shopify APIs usually return it on an HTTP 200), or an HTTP
+ * 502, 503 or 504.
+ *
+ * @param error - The error to check.
+ * @returns Whether the error is a transient server-side failure.
+ */
+export declare function isTransientServerError(error: unknown): error is ClientError;
 export declare function simpleRequestWithDebugLog<T extends {
     headers: Headers;
     status: number;

@@ -51,11 +51,11 @@ export declare const upgradeJsonOutputSchema: import("../json-output-schema.js")
     scope: zod.ZodEnum<["global", "local"]>;
 }, "strict", zod.ZodTypeAny, {
     status: "skipped";
-    scope: "local" | "global";
     reason: "development" | "local-autoupgrade" | "dependency-not-found";
+    scope: "local" | "global";
 }, {
     status: "skipped";
-    scope: "local" | "global";
     reason: "development" | "local-autoupgrade" | "dependency-not-found";
+    scope: "local" | "global";
 }>]>>;
 export type UpgradeResult = InferJsonOutputSchema<typeof upgradeJsonOutputSchema>;
\ No newline at end of file

@@ -36,4 +36,11 @@ export declare function isPermissionCode(code: string | undefined): boolean;
  * @param errors - The `errors` value from a GraphQL response.
  * @returns True when a rate-limit code is present in any error.
  */
-export declare function hasRateLimitCode(errors: unknown): boolean;
\ No newline at end of file
+export declare function hasRateLimitCode(errors: unknown): boolean;
+/**
+ * Whether any GraphQL error carries a transient server-side failure code.
+ *
+ * @param errors - The `errors` value from a GraphQL response.
+ * @returns True when a transient server code is present in any error.
+ */
+export declare function hasTransientServerCode(errors: unknown): boolean;
\ No newline at end of file