improve state tracking for icon changes

[tstromberg]

No description provided.

[kusari-inspector]

Kusari Analysis Results:

Caution

Flagged Issues Detected
These changes contain flagged issues that may introduce security risks.

While dependency analysis shows no security concerns with pinned versions or exposed secrets, code analysis identified a medium severity nil pointer dereference vulnerability in cmd/goose/github.go at line 212 that could lead to runtime panics. This poses a potential denial of service risk that should be addressed before merging. The issue has a clear mitigation path with enhanced error handling and additional safety checks for response pointer dereferencing.

Note

View full detailed analysis result for more information on the output and the checks that were run.

Required Code Mitigations

Add additional safety checks before dereferencing the response pointer to prevent potential nil pointer panics

• Location: cmd/goose/github.go:212

• Potential Code Fix:

if retryErr != nil {
			// Enhanced error handling with specific cases
			if resp != nil && *resp != nil {
				const (
					httpStatusUnauthorized  = 401
					httpStatusForbidden     = 403
					httpStatusUnprocessable = 422
				)
				switch (*resp).StatusCode {

---

@kusari-inspector rerun - Trigger a re-analysis of this PR
@kusari-inspector feedback [your message] - Send feedback to our AI and team
See Kusari's documentation for setup and configuration.
Commit: bdba1d6, performed at: 2025-09-23T23:13:38Z

Found this helpful? Give it a 👍 or 👎 reaction!

[kusari-inspector]

Issue: Add additional safety checks before dereferencing the response pointer to prevent potential nil pointer panics

Recommended Code Changes:

if retryErr != nil {
			// Enhanced error handling with specific cases
			if resp != nil && *resp != nil {
				const (
					httpStatusUnauthorized  = 401
					httpStatusForbidden     = 403
					httpStatusUnprocessable = 422
				)
				switch (*resp).StatusCode {