Retry code search API rate limits in github/github link check (#64410)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: docs-bot

src/workflows/secondary-ratelimit-retry.ts

@@ -20,7 +20,7 @@ export async function octoSecondaryRatelimitRetry<T>(
     } catch (error) {
       if (
         (isRequestError(error, 403) || isRequestError(error, 429)) &&
-        /You have exceeded a secondary rate limit/.test(error.message)
+        /You have exceeded a secondary rate limit|API rate limit exceeded/.test(error.message)
       ) {
         const retryAfterMs = getRetryAfterMs(error)
         if (retryAfterMs !== null && retryAfterMs > MAX_RETRY_AFTER_MS) {
@@ -46,7 +46,14 @@ export async function octoSecondaryRatelimitRetry<T>(
 
 function getRetryAfterMs(error: object): number | null {
   const headers = (error as { response?: { headers?: Record<string, unknown> } }).response?.headers
-  const value = headers?.['retry-after']
+  let value = headers?.['retry-after']
+  if (value === undefined) {
+    // The code search API's 30 requests/minute limit only sends a reset time.
+    const reset = Number(headers?.['x-ratelimit-reset'])
+    if (Number.isFinite(reset) && reset > 0) {
+      value = String(Math.max(0, Math.ceil(reset - Date.now() / 1000)) + 1)
+    }
+  }
   if (typeof value !== 'string' && typeof value !== 'number') return null
   const seconds = Number(value)
   if (String(value).trim() === '' || !Number.isFinite(seconds) || seconds < 0) return null

src/workflows/tests/secondary-ratelimit-retry.ts

@@ -32,6 +32,23 @@ describe('octoSecondaryRatelimitRetry', () => {
     expect(fn).toHaveBeenCalledTimes(2)
   })
 
+  test('retries a search API rate limit using x-ratelimit-reset', async () => {
+    vi.useFakeTimers()
+    vi.setSystemTime(new Date('2026-01-01T00:00:00Z'))
+    const reset = String(Date.parse('2026-01-01T00:00:10Z') / 1000)
+    const fn = failOnceThen(
+      makeError(403, 'API rate limit exceeded for user ID 1.', { 'x-ratelimit-reset': reset }),
+      'ok',
+    )
+    const promise = octoSecondaryRatelimitRetry(fn, { attempts: 2, sleepTime: 0 })
+
+    await vi.advanceTimersByTimeAsync(10_999)
+    expect(fn).toHaveBeenCalledTimes(1)
+    await vi.advanceTimersByTimeAsync(1)
+    expect(fn).toHaveBeenCalledTimes(2)
+    await expect(promise).resolves.toBe('ok')
+  })
+
   test('does not retry a 429 without the secondary rate limit message', async () => {
     const error = makeError(429, 'Too Many Requests')
     const fn = vi.fn().mockRejectedValue(error)