[API Node] Better execution error handling (#3587)

Co-authored-by: github-actions <[email protected]>

Author: huchenlei

src/components/dialog/content/ApiNodesSignInContent.vue

@@ -12,7 +12,7 @@
     <ApiNodesList :node-names="apiNodeNames" />
 
     <div class="flex justify-between items-center">
-      <Button :label="t('g.learnMore')" link />
+      <Button :label="t('g.learnMore')" link @click="handleLearnMoreClick" />
       <div class="flex gap-2">
         <Button
           :label="t('g.cancel')"
@@ -37,4 +37,8 @@ const { apiNodeNames, onLogin, onCancel } = defineProps<{
   onLogin?: () => void
   onCancel?: () => void
 }>()
+
+const handleLearnMoreClick = () => {
+  window.open('https://www.comfy.org/faq', '_blank')
+}
 </script>

src/components/dialog/content/SignInRequiredDialogContent.vue

@@ -1122,20 +1122,6 @@
       "signOut": "Log Out",
       "success": "Signed out successfully",
       "successDetail": "You have been signed out of your account."
-    },
-    "required": {
-      "signIn": {
-        "title": "Sign-In Required to Execute Workflow",
-        "message": "This workflow includes nodes that require an active account. Please log in or create one to continue.",
-        "hint": "To login go to: Settings > User > Login",
-        "action": "Open Settings to Login"
-      },
-      "credits": {
-        "title": "Credits Required to Execute Workflow",
-        "message": "This workflow includes nodes that require credits. Please add credits to your account to continue.",
-        "hint": "To add credits go to: Settings > User > Credits",
-        "action": "Open Settings to Add Credits"
-      }
     }
   },
   "validation": {

src/locales/en/main.json

@@ -36,20 +36,6 @@
       "termsText": "Al hacer clic en \"Siguiente\" o \"Registrarse\", aceptas nuestros",
       "title": "Inicia sesión en tu cuenta"
     },
-    "required": {
-      "credits": {
-        "action": "Abrir configuración para añadir créditos",
-        "hint": "Para añadir créditos ve a: Configuración > Usuario > Créditos",
-        "message": "Este flujo de trabajo incluye nodos que requieren créditos. Por favor, añade créditos a tu cuenta para continuar.",
-        "title": "Créditos requeridos para ejecutar el flujo de trabajo"
-      },
-      "signIn": {
-        "action": "Abrir configuración para iniciar sesión",
-        "hint": "Para iniciar sesión ve a: Configuración > Usuario > Iniciar sesión",
-        "message": "Este flujo de trabajo incluye nodos que requieren una cuenta activa. Por favor, inicia sesión o crea una para continuar.",
-        "title": "Inicio de sesión requerido para ejecutar el flujo de trabajo"
-      }
-    },
     "signOut": {
       "signOut": "Cerrar sesión",
       "success": "Sesión cerrada correctamente",

src/locales/es/main.json

@@ -36,20 +36,6 @@
       "termsText": "En cliquant sur \"Suivant\" ou \"S'inscrire\", vous acceptez nos",
       "title": "Connectez-vous à votre compte"
     },
-    "required": {
-      "credits": {
-        "action": "Ouvrir les paramètres pour ajouter des crédits",
-        "hint": "Pour ajouter des crédits, allez dans : Paramètres > Utilisateur > Crédits",
-        "message": "Ce workflow inclut des nœuds nécessitant des crédits. Veuillez ajouter des crédits à votre compte pour continuer.",
-        "title": "Crédits requis pour exécuter le workflow"
-      },
-      "signIn": {
-        "action": "Ouvrir les paramètres pour se connecter",
-        "hint": "Pour vous connecter, allez dans : Paramètres > Utilisateur > Connexion",
-        "message": "Ce workflow inclut des nœuds nécessitant un compte actif. Veuillez vous connecter ou en créer un pour continuer.",
-        "title": "Connexion requise pour exécuter le workflow"
-      }
-    },
     "signOut": {
       "signOut": "Se déconnecter",
       "success": "Déconnexion réussie",

src/locales/fr/main.json

@@ -36,20 +36,6 @@
       "termsText": "「次へ」または「サインアップ」をクリックすると、私たちの",
       "title": "アカウントにログインする"
     },
-    "required": {
-      "credits": {
-        "action": "設定を開いてクレジットを追加",
-        "hint": "クレジットを追加するには: 設定 > ユーザー > クレジット",
-        "message": "このワークフローにはクレジットが必要なノードが含まれています。続行するにはアカウントにクレジットを追加してください。",
-        "title": "ワークフロー実行にはクレジットが必要です"
-      },
-      "signIn": {
-        "action": "設定を開いてログイン",
-        "hint": "ログインするには: 設定 > ユーザー > ログイン",
-        "message": "このワークフローにはアクティブなアカウントが必要なノードが含まれています。続行するにはログインまたはアカウントを作成してください。",
-        "title": "ワークフロー実行にはサインインが必要です"
-      }
-    },
     "signOut": {
       "signOut": "ログアウト",
       "success": "正常にサインアウトしました",

src/locales/ja/main.json

@@ -36,20 +36,6 @@
       "termsText": "\"다음\" 또는 \"가입하기\"를 클릭하면 우리의",
       "title": "계정에 로그인"
     },
-    "required": {
-      "credits": {
-        "action": "설정에서 크레딧 추가 열기",
-        "hint": "크레딧을 추가하려면: 설정 > 사용자 > 크레딧 으로 이동하세요.",
-        "message": "이 워크플로에는 크레딧이 필요한 노드가 포함되어 있습니다. 계속하려면 계정에 크레딧을 추가하세요.",
-        "title": "워크플로 실행을 위한 크레딧 필요"
-      },
-      "signIn": {
-        "action": "설정에서 로그인 열기",
-        "hint": "로그인하려면: 설정 > 사용자 > 로그인 으로 이동하세요.",
-        "message": "이 워크플로에는 활성 계정이 필요한 노드가 포함되어 있습니다. 계속하려면 로그인하거나 계정을 생성하세요.",
-        "title": "워크플로 실행을 위한 로그인 필요"
-      }
-    },
     "signOut": {
       "signOut": "로그아웃",
       "success": "성공적으로 로그아웃되었습니다",

src/locales/ko/main.json

@@ -36,20 +36,6 @@
       "termsText": "Нажимая \"Далее\" или \"Зарегистрироваться\", вы соглашаетесь с нашими",
       "title": "Войдите в свой аккаунт"
     },
-    "required": {
-      "credits": {
-        "action": "Открыть настройки для пополнения кредитов",
-        "hint": "Чтобы пополнить кредиты, перейдите в: Настройки > Пользователь > Кредиты",
-        "message": "Этот рабочий процесс содержит узлы, для которых необходимы кредиты. Пожалуйста, пополните баланс, чтобы продолжить.",
-        "title": "Требуются кредиты для выполнения рабочего процесса"
-      },
-      "signIn": {
-        "action": "Открыть настройки для входа",
-        "hint": "Чтобы войти, перейдите в: Настройки > Пользователь > Вход",
-        "message": "Этот рабочий процесс содержит узлы, для которых необходима активная учетная запись. Пожалуйста, войдите или создайте учетную запись, чтобы продолжить.",
-        "title": "Требуется вход для выполнения рабочего процесса"
-      }
-    },
     "signOut": {
       "signOut": "Выйти",
       "success": "Вы успешно вышли из системы",

src/locales/ru/main.json

@@ -36,20 +36,6 @@
       "termsText": "点击“下一步”或“注册”即表示您同意我们的",
       "title": "登录您的账户"
     },
-    "required": {
-      "credits": {
-        "action": "打开设置进行充值",
-        "hint": "要充值，请前往：设置 > 用户 > 积分",
-        "message": "此工作流包含需要积分的节点。请为您的账户充值以继续。",
-        "title": "需要积分以执行工作流"
-      },
-      "signIn": {
-        "action": "打开设置进行登录",
-        "hint": "要登录，请前往：设置 > 用户 > 登录",
-        "message": "此工作流包含需要活跃账户的节点。请登录或创建账户以继续。",
-        "title": "需要登录以执行工作流"
-      }
-    },
     "signOut": {
       "signOut": "退出登录",
       "success": "成功退出登录",

src/locales/zh/main.json

@@ -674,13 +674,18 @@ export class ComfyApp {
 
     api.addEventListener('execution_error', ({ detail }) => {
       // Check if this is an auth-related error or credits-related error
-      if (detail.exception_message === 'Please login first to use this node.') {
-        useDialogService().showSignInRequiredDialog({ type: 'signIn' })
+      if (
+        detail.exception_message ===
+        'Unauthorized: Please login first to use this node.'
+      ) {
+        useDialogService().showApiNodesSignInDialog([detail.node_type])
       } else if (
         detail.exception_message ===
         'Payment Required: Please add credits to your account to use this node.'
       ) {
-        useDialogService().showSignInRequiredDialog({ type: 'credits' })
+        useDialogService().showTopUpCreditsDialog({
+          isInsufficientCredits: true
+        })
       } else {
         useDialogService().showExecutionErrorDialog(detail)
       }