From 6215dd378e4c59947f7db412e615b05f2f3cbcb5 Mon Sep 17 00:00:00 2001
From: icarus <eurfelux@gmail.com>
Date: Sun, 24 Aug 2025 18:57:24 +0800
Subject: [PATCH] =?UTF-8?q?refactor(OCR=E8=AE=BE=E7=BD=AE):=20=E6=B7=BB?=
 =?UTF-8?q?=E5=8A=A0=E9=94=99=E8=AF=AF=E8=BE=B9=E7=95=8C=E5=A4=84=E7=90=86?=
 =?UTF-8?q?=E5=B9=B6=E7=A7=BB=E9=99=A4=E6=97=A0=E7=94=A8=E6=B3=A8=E9=87=8A?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

在OCR设置组件中添加ErrorBoundary以处理潜在错误
移除OcrTesseractSettings中的TODO注释
---
 .../settings/DocProcessSettings/OcrProviderSettings.tsx      | 3 ++-
 .../src/pages/settings/DocProcessSettings/OcrSettings.tsx    | 5 +++--
 .../settings/DocProcessSettings/OcrTesseractSettings.tsx     | 1 -
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/src/renderer/src/pages/settings/DocProcessSettings/OcrProviderSettings.tsx b/src/renderer/src/pages/settings/DocProcessSettings/OcrProviderSettings.tsx
index a9ba128d7a..501b8d792f 100644
--- a/src/renderer/src/pages/settings/DocProcessSettings/OcrProviderSettings.tsx
+++ b/src/renderer/src/pages/settings/DocProcessSettings/OcrProviderSettings.tsx
@@ -1,4 +1,5 @@
 // import { loggerService } from '@logger'
+import { ErrorBoundary } from '@renderer/components/ErrorBoundary'
 import { isBuiltinOcrProvider, OcrProvider } from '@renderer/types'
 import { getOcrProviderLogo } from '@renderer/utils/ocr'
 import { Avatar, Divider, Flex } from 'antd'
@@ -35,7 +36,7 @@ const OcrProviderSettings = ({ provider }: Props) => {
         </Flex>
       </SettingTitle>
       <Divider style={{ width: '100%', margin: '10px 0' }} />
-      {getProviderSettings()}
+      <ErrorBoundary>{getProviderSettings()}</ErrorBoundary>
     </>
   )
 }
diff --git a/src/renderer/src/pages/settings/DocProcessSettings/OcrSettings.tsx b/src/renderer/src/pages/settings/DocProcessSettings/OcrSettings.tsx
index 9ad2d111ad..2a829e7925 100644
--- a/src/renderer/src/pages/settings/DocProcessSettings/OcrSettings.tsx
+++ b/src/renderer/src/pages/settings/DocProcessSettings/OcrSettings.tsx
@@ -1,4 +1,5 @@
 import { PictureOutlined } from '@ant-design/icons'
+import { ErrorBoundary } from '@renderer/components/ErrorBoundary'
 import { useTheme } from '@renderer/context/ThemeProvider'
 import { useAppSelector } from '@renderer/store'
 import { OcrProvider } from '@renderer/types'
@@ -26,7 +27,7 @@ const OcrSettings: FC = () => {
   ]
 
   return (
-    <>
+    <ErrorBoundary>
       <SettingGroup theme={themeMode}>
         <SettingTitle>{t('settings.tool.ocr.title')}</SettingTitle>
         <SettingDivider />
@@ -35,7 +36,7 @@ const OcrSettings: FC = () => {
       <SettingGroup theme={themeMode}>
         <OcrProviderSettings provider={provider} />
       </SettingGroup>
-    </>
+    </ErrorBoundary>
   )
 }
 export default OcrSettings
diff --git a/src/renderer/src/pages/settings/DocProcessSettings/OcrTesseractSettings.tsx b/src/renderer/src/pages/settings/DocProcessSettings/OcrTesseractSettings.tsx
index 7e94a31194..8c85cee8bc 100644
--- a/src/renderer/src/pages/settings/DocProcessSettings/OcrTesseractSettings.tsx
+++ b/src/renderer/src/pages/settings/DocProcessSettings/OcrTesseractSettings.tsx
@@ -13,7 +13,6 @@ export const OcrTesseractSettings = () => {
   const { t } = useTranslation()
   const { provider } = useOcrProvider(BuiltinOcrProviderIds.tesseract)
 
-  // TODO: use error boundary
   if (!isOcrTesseractProvider(provider)) {
     throw new Error('Not tesseract provider.')
   }