fix(cli): log panic in token refresh callback instead of silently discarding

1 file · +5 -1

M internal/adapters/in/cli/remote/client.go +5 -1


@@ -162,7 +162,11 @@ func (c Client) request(ctx context.Context, method, path string, body any) (h
162	162		if c.onTokenRefreshed != nil {
163	163		// Token persistence is non-fatal: a panic in the callback must not crash the request.
164	164		func() {
165		-	defer func() { recover() }() //nolint:errcheck
	165	+	defer func() {
	166	+	if r := recover(); r != nil {
	167	+	fmt.Fprintf(os.Stderr, "warning: token refresh callback panicked: %v\n", r)
	168	+	}
	169	+	}()
166	170		c.onTokenRefreshed(newToken)
167	171		}()
168	172		}

 		if c.onTokenRefreshed != nil {
 			// Token persistence is non-fatal: a panic in the callback must not crash the request.
 			func() {
-				defer func() { recover() }() //nolint:errcheck
+				defer func() {
+					if r := recover(); r != nil {
+						fmt.Fprintf(os.Stderr, "warning: token refresh callback panicked: %v\n", r)
+					}
+				}()
 				c.onTokenRefreshed(newToken)
 			}()
 		}