Add test verifying transient network errors ARE retried

Complements the "does not retry certificate errors" test by explicitly
verifying that transient errors DO trigger retry logic and eventually succeed.

Tests 5 key transient error types from the PR:
- ETIMEDOUT (timeout)
- ECONNRESET (connection reset)
- socket hang up
- premature close
- getaddrinfo ENOTFOUND (DNS failure)

Each error is tested to ensure:
1. The error triggers a retry (mockRequestFn called 2x)
2. The request eventually succeeds after retry

This confirms the distinction: transient errors retry, permanent errors don't.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: amcaplan

packages/cli-kit/src/private/node/api.test.ts

@@ -372,6 +372,54 @@ describe('retryAwareRequest', () => {
       expect(mockRequestFn).toHaveBeenCalledTimes(1)
     }
   })
+
+  test('retries transient network errors and eventually succeeds', async () => {
+    const transientErrors = [
+      {error: 'ETIMEDOUT', description: 'timeout'},
+      {error: 'ECONNRESET', description: 'connection reset'},
+      {error: 'socket hang up', description: 'socket hang up'},
+      {error: 'premature close', description: 'premature close'},
+      {error: 'getaddrinfo ENOTFOUND', description: 'DNS failure'},
+    ]
+
+    for (const {error: transientError, description} of transientErrors) {
+      const mockRequestFn = vi
+        .fn()
+        .mockImplementationOnce(() => {
+          throw new Error(transientError)
+        })
+        .mockImplementationOnce(() => {
+          return Promise.resolve({
+            status: 200,
+            data: {success: true},
+            headers: new Headers(),
+          })
+        })
+
+      const mockScheduleDelayFn = vi.fn((fn, _delay) => fn())
+
+      const result = retryAwareRequest(
+        {
+          request: mockRequestFn,
+          url: 'https://example.com/graphql.json',
+          useNetworkLevelRetry: true,
+          maxRetryTimeMs: 2000,
+        },
+        undefined,
+        {defaultDelayMs: 10, scheduleDelay: mockScheduleDelayFn},
+      )
+
+      await vi.runAllTimersAsync()
+
+      await expect(result).resolves.toEqual({
+        headers: expect.anything(),
+        status: 200,
+        data: {success: true},
+      })
+      expect(mockRequestFn, `${description} should trigger retry`).toHaveBeenCalledTimes(2)
+      // scheduleDelay might not always be called depending on timing, so we just verify the retry happened
+    }
+  })
 })
 
 describe('isTransientNetworkError', () => {