Fix defer placement for response body closing

Copilot · SamMorrowDrums · Copilot · commit 6e0e72a5bc30 · 2026-01-19T21:29:30.000Z
Co-authored-by: SamMorrowDrums &lt;4811358+SamMorrowDrums@users.noreply.github.com&gt;
diff --git a/pkg/github/issues.go b/pkg/github/issues.go
@@ -859,8 +859,8 @@ func AddSubIssue(ctx context.Context, client *github.Client, owner string, repo
 
 	// Handle non-201 status codes after retries exhausted
 	if lastResp != nil && lastResp.StatusCode != http.StatusCreated {
-		body, err := io.ReadAll(lastResp.Body)
 		defer func() { _ = lastResp.Body.Close() }()
+		body, err := io.ReadAll(lastResp.Body)
 		if err != nil {
 			return nil, fmt.Errorf("failed to read response body: %w", err)
 		}

Original file line number	Diff line number	Diff line change
`@@ -859,8 +859,8 @@ func AddSubIssue(ctx context.Context, client *github.Client, owner string, repo`
`859`	`859`
`860`	`860`	`// Handle non-201 status codes after retries exhausted`
`861`	`861`	`if lastResp != nil && lastResp.StatusCode != http.StatusCreated {`
`862`		`- body, err := io.ReadAll(lastResp.Body)`
`863`	`862`	`defer func() { _ = lastResp.Body.Close() }()`
	`863`	`+ body, err := io.ReadAll(lastResp.Body)`
`864`	`864`	`if err != nil {`
`865`	`865`	`return nil, fmt.Errorf("failed to read response body: %w", err)`
`866`	`866`	`}`