fix: panic with inner client (#714)

related to this thread in slack:
https://chainguard-dev.slack.com/archives/C02SD39C6BW/p1738862337902599

---------

Signed-off-by: hectorj2f <hector@chainguar.dev>
Signed-off-by: Hector Fernandez <hector@chainguard.dev>
Co-authored-by: hectorj2f <hector@chainguar.dev>

modules/github-bots/sdk/github.go

@@ -40,6 +40,7 @@ func NewGitHubClient(ctx context.Context, org, repo, policyName string, opts ...
 	})
 
 	client := GitHubClient{
+		inner:   github.NewClient(oauth2.NewClient(ctx, ts)),
 		ts:      ts,
 		bufSize: 1024 * 1024, // 1MB buffer for requests
 		org:     org,
@@ -50,11 +51,6 @@ func NewGitHubClient(ctx context.Context, org, repo, policyName string, opts ...
 		opt(&client)
 	}
 
-	// if client has not been configured withOpts then create a new client
-	if client.inner == nil {
-		client.inner = github.NewClient(oauth2.NewClient(ctx, ts))
-	}
-
 	return client
 }
 

modules/github-bots/sdk/github_test.go

@@ -78,6 +78,7 @@ func TestGitHubClientConfiguration(t *testing.T) {
 					"test-org",
 					"test-repo",
 					"test-policy",
+					WithSecondaryRateLimitWaiter(),
 					WithClient(testClient),
 				)