Refactor: [Server][Utils/TwitterGraphQLAPI] ヘッドレスブラウザの起動に失敗した場合のエラーメッセージ表示周りの実装を整理

Author: tsukumijima

client/src/services/APIClient.ts

@@ -235,10 +235,6 @@ class APIClient {
                 Message.error(`${template}\nこのリソースにアクセスする権限がありません。`);
                 return;
             }
-            case 'Chrome or Brave is not installed on this machine.': {
-                Message.error('ヘッドレスブラウザの起動に必要な Chrome または Brave が KonomiTV サーバーにインストールされていません。');
-                return;
-            }
             default: {
                 if (Array.isArray(error_response.data.detail)) {
                     // バリデーションエラーが発生した場合

client/src/services/Twitter.ts

@@ -77,12 +77,8 @@ class Twitter {
                     Message.error('Twitter アカウント情報の取得に失敗しました。');
                     break;
                 }
-                case 'Chrome or Brave is not installed on this machine.': {
-                    Message.error('ヘッドレスブラウザの起動に必要な Chrome または Brave が KonomiTV サーバーにインストールされていません。');
-                    break;
-                }
                 default: {
-                    APIClient.showGenericError(response, 'Twitter アカウントとの連携に失敗しました。再度お試しください。');
+                    APIClient.showGenericError(response, 'Twitter アカウントとの連携に失敗しました。');
                     break;
                 }
             }
@@ -160,13 +156,6 @@ class Twitter {
         // エラー処理 (API リクエスト自体に失敗した場合)
         // このエンドポイントのみ、Message (SnackBar) では通知せず、通知をプレイヤー側に委ねる必要がある
         if (response.type === 'error') {
-            if (typeof response.data.detail === 'string' &&
-                response.data.detail === 'Chrome or Brave is not installed on this machine.') {
-                return {
-                    message: 'ヘッドレスブラウザの起動に必要な Chrome または Brave が KonomiTV サーバーにインストールされていません。',
-                    is_error: true,
-                };
-            }
             if (typeof response.data.detail === 'string') {
                 if (Number.isNaN(response.status)) {
                     // HTTP リクエスト自体が失敗し、HTTP ステータスコードが取得できなかった場合

server/app/routers/TwitterRouter.py

@@ -154,7 +154,7 @@ async def TwitterCookieAuthAPI(
         logging.error(f'[TwitterRouter][TwitterCookieAuthAPI] Failed to get user information: {viewer_result.detail}')
         raise HTTPException(
             status_code = status.HTTP_500_INTERNAL_SERVER_ERROR,
-            detail = viewer_result.detail,
+            detail = viewer_result.detail,  # エラーメッセージをそのまま返す
         )
 
     # viewer_result が TweetUser の場合のみ処理を続行

server/app/utils/TwitterGraphQLAPI.py

@@ -12,6 +12,7 @@
 from app.models.TwitterAccount import TwitterAccount
 from app.utils.TwitterScrapeBrowser import (
     BrowserBinaryNotFoundError,
+    BrowserConnectionFailedError,
     TwitterScrapeBrowser,
 )
 
@@ -263,10 +264,9 @@ async def invokeGraphQLAPI(
         if self._browser.is_setup_complete is not True:
             try:
                 await self._browser.setup()
-            except BrowserBinaryNotFoundError as ex:
-                # Chrome / Brave 未導入時は UI に案内できるよう固定のエラーメッセージを返す
-                logging.error(f'{self.log_prefix} Chrome or Brave is not installed on this machine:', exc_info=ex)
-                return 'Chrome or Brave is not installed on this machine.'
+            except (BrowserBinaryNotFoundError, BrowserConnectionFailedError) as ex:
+                # Chrome / Brave 未導入時やブラウザ起動失敗時は、日本語のエラーメッセージをそのまま返す
+                return str(ex)
 
         # TwitterScrapeBrowser 経由で GraphQL API に HTTP リクエストを送信
         browser = self._browser

server/app/utils/TwitterScrapeBrowser.py

@@ -14,6 +14,11 @@ class BrowserBinaryNotFoundError(RuntimeError):
     Chrome / Brave の実行ファイルが検出できず、Zendriver がブラウザを起動できない場合に送出される例外
     """
 
+class BrowserConnectionFailedError(RuntimeError):
+    """
+    Chrome / Brave はおそらく起動できたが、ブラウザインスタンスへの接続に失敗した場合に送出される例外
+    """
+
 
 class TwitterScrapeBrowser:
     """
@@ -103,8 +108,15 @@ async def setup(self) -> None:
                     ]
                 )
             except FileNotFoundError as ex:
-                logging.error(f'{self.log_prefix} Chrome or Brave is not installed.', exc_info=ex)
-                raise BrowserBinaryNotFoundError('Chrome or Brave is not installed on this machine.') from ex
+                logging.error(f'{self.log_prefix} Chrome or Brave is not installed on this machine:', exc_info=ex)
+                raise BrowserBinaryNotFoundError('ヘッドレスブラウザの起動に必要な Chrome または Brave が KonomiTV サーバーにインストールされていません。') from ex
+            except Exception as ex:
+                if 'Failed to connect to browser' in str(ex):
+                    logging.error(f'{self.log_prefix} Browser connection failed. Please check if Chrome or Brave is installed:', exc_info=ex)
+                    raise BrowserConnectionFailedError('ヘッドレスブラウザとの接続に失敗しました。Chrome または Brave が KonomiTV サーバーにインストールされているかどうかを確認してください。') from ex
+                else:
+                    logging.error(f'{self.log_prefix} Error starting browser:', exc_info=ex)
+                    raise ex
             logging.info(f'{self.log_prefix} Browser started.')
 
             # まず空のタブを開く