Resolve CI lint regressions in CVE-2019-9583 retry path

[Copilot]

The latest workflow runs failed due to XO lint errors introduced in the CVE-2019-9583 hardening changes, not due to test logic regressions. This update keeps the retry behavior intact while aligning the implementation with repository lint rules.

• Lint-compliant retry error code lookup

  • Switched retryable error code storage from Array to Set.
  • Replaced .includes(...) with .has(...) to satisfy unicorn/prefer-set-has.

• Control-flow simplification

  • Removed an unnecessary else branch after early return in the request callback to satisfy no-else-return.
  • Preserved existing behavior for retryable errors and vulnerable redirect detection.

const retryableErrorCodes = new Set(['ECONNRESET', 'ETIMEDOUT', 'ESOCKETTIMEDOUT']);

if (error) {
  return retryCallback(error);
}

if (typeof (response.headers.location) !== 'undefined' &&
    response.headers.location.includes('sid=@')) {
  return retryCallback(new Error(...));
}

---

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.