Fix 429 rate limit handling and context key type mismatch in login #480 (#481)

fix: handle 429 rate limit and context key type mismatch in login issue #480

Author: rudSarkar

cmd/auth.go

@@ -47,7 +47,7 @@ func loginCmd() *cobra.Command {
 		Use:   "login",
 		Short: "Login to the App Store",
 		RunE: func(cmd *cobra.Command, args []string) error {
-			interactive, _ := cmd.Context().Value(interactiveKey).(bool)
+			interactive := cmd.Context().Value(interactiveKey).(bool)
 
 			if password == "" && !interactive {
 				return errors.New("password is required when not running in interactive mode; use the \"--password\" flag")

pkg/http/client.go

@@ -160,6 +160,10 @@ func (c *client[R]) handleXMLResponse(res *http.Response) (Result[R], error) {
 		return Result[R]{}, fmt.Errorf("failed to read response body: %w", err)
 	}
 
+	if res.StatusCode == http.StatusTooManyRequests {
+		return Result[R]{}, fmt.Errorf("rate limited by Apple (HTTP %d): %s", res.StatusCode, strings.TrimSpace(string(body)))
+	}
+
 	var data R
 
 	normalizedBody := normalizeXMLPlistBody(body)