fix clearing metadata-only inspection errors

Author: ssfun

cliproxyapi-pro-core/patches/account_inspection_scheduler.go

@@ -2627,10 +2627,10 @@ func (s *accountInspectionScheduler) clearInspectionAuthError(ctx context.Contex
 		return
 	}
 	auth := s.h.authByIndex(account.AuthIndex)
-	if auth == nil || auth.Status != coreauth.StatusError || auth.LastError == nil {
+	if auth == nil {
 		return
 	}
-	if auth.LastError.Code != "inspection_http_error" && auth.LastError.Code != "inspection_probe_error" && auth.LastError.Code != "antigravity_deep_probe_error" && auth.LastError.Code != "token_refresh_error" {
+	if !isInspectionAuthErrorCode(authInspectionLastErrorCode(auth)) {
 		return
 	}
 	err := s.updateInspectionAuth(ctx, account.AuthIndex, func(auth *coreauth.Auth) {
@@ -2649,6 +2649,29 @@ func (s *accountInspectionScheduler) clearInspectionAuthError(ctx context.Contex
 	}
 }
 
+func authInspectionLastErrorCode(auth *coreauth.Auth) string {
+	if auth == nil {
+		return ""
+	}
+	if auth.LastError != nil {
+		return strings.TrimSpace(auth.LastError.Code)
+	}
+	raw, ok := auth.Metadata["last_error"].(map[string]any)
+	if !ok {
+		return ""
+	}
+	return stringFromAny(raw["code"])
+}
+
+func isInspectionAuthErrorCode(code string) bool {
+	switch strings.TrimSpace(code) {
+	case "inspection_http_error", "inspection_probe_error", "antigravity_deep_probe_error", "token_refresh_error":
+		return true
+	default:
+		return false
+	}
+}
+
 func (s *accountInspectionScheduler) syncInspectionAuthStatus(ctx context.Context, account accountInspectionAccount, status int) {
 	if isAccountErrorStatus(status) {
 		message := fmt.Sprintf("HTTP %d", status)

cliproxyapi-pro-core/patches/account_inspection_scheduler_test.go

@@ -341,6 +341,56 @@ func TestSyncInspectionAuthErrorPersistsLastErrorMetadata(t *testing.T) {
 	}
 }
 
+func TestClearInspectionAuthErrorClearsMetadataOnlyError(t *testing.T) {
+	manager := coreauth.NewManager(nil, nil, nil)
+	registered, err := manager.Register(context.Background(), &coreauth.Auth{
+		Provider:      "antigravity",
+		ID:            "antigravity-user",
+		FileName:      "antigravity-user.json",
+		Status:        coreauth.StatusActive,
+		StatusMessage: "",
+		Unavailable:   false,
+		Metadata: map[string]any{
+			"email": "user@example.com",
+			"last_error": map[string]any{
+				"code":        "inspection_probe_error",
+				"http_status": 0,
+				"message":     "antigravity quota unavailable",
+				"retryable":   false,
+			},
+		},
+	})
+	if err != nil {
+		t.Fatalf("Register auth error = %v", err)
+	}
+
+	scheduler := &accountInspectionScheduler{h: &Handler{authManager: manager}}
+	scheduler.clearInspectionAuthError(context.Background(), accountFromAuth(registered))
+
+	var got *coreauth.Auth
+	for _, auth := range manager.List() {
+		if auth.ID == registered.ID {
+			got = auth
+			break
+		}
+	}
+	if got == nil {
+		t.Fatal("updated auth not found")
+	}
+	if got.LastError != nil {
+		t.Fatalf("LastError = %#v, want nil", got.LastError)
+	}
+	if _, ok := got.Metadata["last_error"]; ok {
+		t.Fatalf("metadata last_error = %#v, want removed", got.Metadata["last_error"])
+	}
+	if got.Status != coreauth.StatusActive || got.StatusMessage != "" || got.Unavailable {
+		t.Fatalf("status = %q message=%q unavailable=%v, want active/empty/false", got.Status, got.StatusMessage, got.Unavailable)
+	}
+	if got.Metadata["email"] != "user@example.com" {
+		t.Fatalf("metadata email = %#v, want preserved", got.Metadata["email"])
+	}
+}
+
 func TestAutoActionConfirmationDelaysExecution(t *testing.T) {
 	scheduler := &accountInspectionScheduler{}
 	result := testInspectionResult("quota", accountInspectionActionDisable, false, nil, true, "")