Merge pull request #7145 from Shopify/04-01-add_user_aborted_a_request_to_errormessageimpliesenvironmentissue

Add user aborted a request to errorMessageImpliesEnvironmentIssue

Author: isaacroldan

packages/cli-kit/src/public/node/error.test.ts

@@ -70,4 +70,8 @@ describe('shouldReportErrorAsUnexpected helper', () => {
   test('returns false for errors that imply environment issues', () => {
     expect(shouldReportErrorAsUnexpected(new Error('EPERM: operation not permitted, scandir'))).toBe(false)
   })
+
+  test('returns false for user-aborted requests', () => {
+    expect(shouldReportErrorAsUnexpected(new Error('The user aborted a request.'))).toBe(false)
+  })
 })

packages/cli-kit/src/public/node/error.ts

@@ -233,6 +233,7 @@ function errorMessageImpliesEnvironmentIssue(message: string): boolean {
     'Client network socket disconnected before secure TLS connection was established',
     'spawn EPERM',
     'socket hang up',
+    'The user aborted a request.',
   ]
   const anyMatches = environmentIssueMessages.some((issueMessage) => message.includes(issueMessage))
   return anyMatches