fix: explicit type annotations for throttle callbacks broken by @octokit/plugin-retry v8.1.0

[Copilot]

@octokit/plugin-retry v8.1.0 changed its module augmentation target from @octokit/core to @octokit/core/types (a distinct TypeScript module path), breaking contextual type inference for the throttle.onRateLimit and throttle.onSecondaryRateLimit callback parameters in GithubClient.

Changes

• src/client/github_client.ts: Add explicit type annotations to throttle callback parameters; extract a shared ThrottleRequestOptions type alias to avoid duplication.

// Before – inference broke, parameters became implicit `any`
onRateLimit: (retryAfter, options, _octokit, retryCount) => { ... }

// After – explicit annotations restore type safety
type ThrottleRequestOptions = { method: string; url: string };
onRateLimit: (retryAfter: number, options: ThrottleRequestOptions, _octokit: unknown, retryCount: number) => { ... }

{ method: string; url: string } is a structural supertype of Required<EndpointDefaults>, so it satisfies the contra-variant parameter check under strictFunctionTypes.

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[Kesin11]

Thanks to implement workaround copilot. However, it might fix octokit/plugin-retry.js#666