From 46c2fcc56885f8232d9c655a0a58793ebc309bea Mon Sep 17 00:00:00 2001
From: hqhq1025 <1506751656@qq.com>
Date: Sun, 19 Apr 2026 05:00:32 +0800
Subject: [PATCH] fix(desktop): i18n ErrorBoundary fallback strings

---
 .../renderer/src/components/ErrorBoundary.tsx | 78 ++++++++++++-------
 .../src/components/InlineCommentComposer.tsx  |  3 +-
 packages/i18n/src/locales/en.json             |  8 ++
 packages/i18n/src/locales/zh-CN.json          |  8 ++
 4 files changed, 66 insertions(+), 31 deletions(-)

diff --git a/apps/desktop/src/renderer/src/components/ErrorBoundary.tsx b/apps/desktop/src/renderer/src/components/ErrorBoundary.tsx
index 45f1715a9..320409d5b 100644
--- a/apps/desktop/src/renderer/src/components/ErrorBoundary.tsx
+++ b/apps/desktop/src/renderer/src/components/ErrorBoundary.tsx
@@ -11,6 +11,7 @@
  * preview / topbar) so a single crash never blanks the entire window.
  */
 
+import { useT } from '@open-codesign/i18n';
 import { Button } from '@open-codesign/ui';
 import { Component, type ErrorInfo, type ReactNode } from 'react';
 
@@ -79,41 +80,58 @@ export class ErrorBoundary extends Component<ErrorBoundaryProps, ErrorBoundarySt
     if (this.props.fallback) {
       return this.props.fallback({ error, reset: this.reset });
     }
-    const scope = this.props.scope ?? 'this view';
     return (
-      <div className="h-full w-full flex items-center justify-center p-6 bg-[var(--color-background)]">
-        <div className="max-w-md w-full rounded-[var(--radius-2xl)] border border-[var(--color-border)] bg-[var(--color-surface)] shadow-[var(--shadow-card)] p-6">
-          <div className="text-xs uppercase tracking-wide text-[var(--color-error)] font-semibold mb-2">
-            {scope} crashed
-          </div>
-          <h2 className="text-lg font-semibold text-[var(--color-text-primary)] mb-1">
-            {error.name}: {error.message}
-          </h2>
-          <p className="text-sm text-[var(--color-text-secondary)] mb-4">
-            The rest of the app is still running. Reload this view, or copy the stack to file a bug.
-          </p>
-          <pre className="text-[11px] leading-snug font-mono text-[var(--color-text-muted)] bg-[var(--color-surface-active)] border border-[var(--color-border-muted)] rounded-[var(--radius-md)] p-3 max-h-40 overflow-auto whitespace-pre-wrap">
-            {error.stack ?? '(no stack)'}
-          </pre>
-          <div className="mt-4 flex gap-2 justify-end">
-            <Button
-              type="button"
-              variant="secondary"
-              size="md"
-              onClick={() => void this.copyStack()}
-            >
-              Copy stack
-            </Button>
-            <Button type="button" size="md" onClick={this.reset}>
-              Reload
-            </Button>
-          </div>
-        </div>
-      </div>
+      <ErrorBoundaryFallback
+        error={error}
+        scope={this.props.scope}
+        onReset={this.reset}
+        onCopyStack={() => void this.copyStack()}
+      />
     );
   }
 }
 
+interface ErrorBoundaryFallbackProps {
+  error: Error;
+  scope?: string | undefined;
+  onReset: () => void;
+  onCopyStack: () => void;
+}
+
+function ErrorBoundaryFallback({
+  error,
+  scope,
+  onReset,
+  onCopyStack,
+}: ErrorBoundaryFallbackProps): ReactNode {
+  const t = useT();
+  const scopeLabel = scope ?? t('errorBoundary.scopeFallback');
+  return (
+    <div className="h-full w-full flex items-center justify-center p-6 bg-[var(--color-background)]">
+      <div className="max-w-md w-full rounded-[var(--radius-2xl)] border border-[var(--color-border)] bg-[var(--color-surface)] shadow-[var(--shadow-card)] p-6">
+        <div className="text-xs uppercase tracking-wide text-[var(--color-error)] font-semibold mb-2">
+          {t('errorBoundary.crashedSuffix', { scope: scopeLabel })}
+        </div>
+        <h2 className="text-lg font-semibold text-[var(--color-text-primary)] mb-1">
+          {error.name}: {error.message}
+        </h2>
+        <p className="text-sm text-[var(--color-text-secondary)] mb-4">{t('errorBoundary.body')}</p>
+        <pre className="text-[11px] leading-snug font-mono text-[var(--color-text-muted)] bg-[var(--color-surface-active)] border border-[var(--color-border-muted)] rounded-[var(--radius-md)] p-3 max-h-40 overflow-auto whitespace-pre-wrap">
+          {error.stack ?? t('errorBoundary.noStack')}
+        </pre>
+        <div className="mt-4 flex gap-2 justify-end">
+          <Button type="button" variant="secondary" size="md" onClick={onCopyStack}>
+            {t('errorBoundary.copyStack')}
+          </Button>
+          <Button type="button" size="md" onClick={onReset}>
+            {t('errorBoundary.reload')}
+          </Button>
+        </div>
+      </div>
+    </div>
+  );
+}
+
 function ErrorBoundaryChildren({ children }: { children: ReactNode }): ReactNode {
   return children;
 }
diff --git a/apps/desktop/src/renderer/src/components/InlineCommentComposer.tsx b/apps/desktop/src/renderer/src/components/InlineCommentComposer.tsx
index 3735dff6a..880ee337d 100644
--- a/apps/desktop/src/renderer/src/components/InlineCommentComposer.tsx
+++ b/apps/desktop/src/renderer/src/components/InlineCommentComposer.tsx
@@ -33,7 +33,8 @@ function InlineCommentComposerCard({ selectedElement }: InlineCommentComposerCar
         <div className="min-w-0">
           <div className="inline-flex items-center gap-2 text-[var(--text-xs)] font-medium text-[var(--color-text-primary)]">
             <MessageSquareText className="h-4 w-4 text-[var(--color-accent)]" />
-            {t('inlineComment.title')} <code className="text-[var(--text-xs)]">{selectedElement.tag}</code>
+            {t('inlineComment.title')}{' '}
+            <code className="text-[var(--text-xs)]">{selectedElement.tag}</code>
           </div>
           <p
             className="mt-1 truncate text-[var(--text-xs)] text-[var(--color-text-muted)]"
diff --git a/packages/i18n/src/locales/en.json b/packages/i18n/src/locales/en.json
index 146b86125..a354ab0c8 100644
--- a/packages/i18n/src/locales/en.json
+++ b/packages/i18n/src/locales/en.json
@@ -372,6 +372,14 @@
     "enterBothModels": "Both model fields are required",
     "ollamaComingSoon": "Ollama integration is coming in v0.2"
   },
+  "errorBoundary": {
+    "scopeFallback": "this view",
+    "crashedSuffix": "{{scope}} crashed",
+    "body": "The rest of the app is still running. Reload this view, or copy the stack to file a bug.",
+    "noStack": "(no stack)",
+    "copyStack": "Copy stack",
+    "reload": "Reload"
+  },
   "errors": {
     "generic": "Something went wrong.",
     "providerAuthMissing": "No API key configured. Open Settings to add one.",
diff --git a/packages/i18n/src/locales/zh-CN.json b/packages/i18n/src/locales/zh-CN.json
index e3dbcffd4..d85a3e0ea 100644
--- a/packages/i18n/src/locales/zh-CN.json
+++ b/packages/i18n/src/locales/zh-CN.json
@@ -371,6 +371,14 @@
     "enterBothModels": "两个模型字段均为必填",
     "ollamaComingSoon": "Ollama 集成将在 v0.2 中支持"
   },
+  "errorBoundary": {
+    "scopeFallback": "这个视图",
+    "crashedSuffix": "{{scope}}崩溃了",
+    "body": "应用其余部分仍在运行。可以重新加载该视图，或复制堆栈用于反馈问题。",
+    "noStack": "（无堆栈信息）",
+    "copyStack": "复制堆栈",
+    "reload": "重新加载"
+  },
   "errors": {
     "generic": "出错了。",
     "providerAuthMissing": "还没配置 API Key。打开设置去添加一个。",