Skip to content

fix(app-shell): localize popup-blocked toast and SSO spinner tab copy via i18n (#3321) - #3328

Merged
xuyushun441-sys merged 2 commits into
mainfrom
claude/issue-3321-popup-copy-english
Aug 4, 2026
Merged

fix(app-shell): localize popup-blocked toast and SSO spinner tab copy via i18n (#3321)#3328
xuyushun441-sys merged 2 commits into
mainfrom
claude/issue-3321-popup-copy-english

Conversation

@xuyushun441-sys

@xuyushun441-sysxuyushun441-sys commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Fixes#3321

问题

consoleServerAction.tsopensInNewTab 编排里硬编码了中英双语文案,违反 AGENTS.md Commandment #-1(代码库纯英文)。四处文案:

  1. SSO spinner tab 标题:正在打开… Opening…
  2. SSO spinner tab 正文:正在为你打开环境…
  3. popup-blocked toast 标题:浏览器拦截了弹窗 / Popup blocked
  4. toast 描述 点击在新标签页打开环境 + action label 打开环境

方案(采用 issue 评论 PM 预裁决的首选方案:t 经 options 注入,非底线纯英文字面量方案——i18n 键位与基建顺畅,无需降级)

  • t 经 options 注入createConsoleServerActionHandler 新增可选 t?: ConsoleServerActionTranslate(i18next 风格 t(key, englishDefault))。包装器是普通函数不能调 hook,两个调用点(useConsoleActionRuntimeRecordDetailView)都有 hook 上下文,用 useObjectTranslation().t 经 env ref 注入——语言切换即时生效,且不打断 handler 实例的稳定性(in-flight guard 依赖同一实例)。
  • 英文默认值兜底:不传 t(测试 / standalone)时全部落到英文默认值;代码里不再有任何非英文文案。
  • 中文进 i18n 资源文件:新增 console.serverAction.* 五个键(openingTitle / openingBody / popupBlockedTitle / popupBlockedDescription / popupBlockedAction),11 个 locale 包全量补齐(en/zh/de/es/fr/ja/ko/pt/ru/ar,all-locales-key-parity 强制对齐)。
  • 顺手一处安全加固:locale 字符串写入 spinner document 前做最小 HTML 转义(escapeHtml),有测试钉住。
  • 措辞去掉「环境」特指:包装器是通用基建(任何 opensInNewTab server action 都走它),原中文「打开环境」是 cloud SSO 场景专属词汇;新文案改为通用表述("Open in new tab" 等),避免把 cloud 词汇固化进 app-shell。如维护者希望保留「环境」措辞,改 locale 资源文件即可,不用动代码。

验证

命令结果
落点 + 调用点 + parity 测试仓根 pnpm exec vitest run --maxWorkers=2 consoleServerAction / actions-envelope.ratchet / useConsoleActionRuntime / RecordDetailView.modalDispatch / all-locales-key-parity / en-zh-key-parity6 files, 83 tests 全绿(14.75s)
新增文案测试同上(含 4 个新测试:英文默认值、注入 t 走键位(tab + toast)、HTML 转义、无 CJK pin)绿
破坏验证临时把 toast 标题与 spinner 正文改回中文 → 恰好 2 个文案 pin 测试红opens lazily…toast fallbackno CJK in code,2 failed / 12 passed)→ 还原 → 14/14 绿
lint(改动文件)pnpm exec eslint <改动文件>0 errors(193 个既有 no-explicit-any warning,均非新增行)
type-check(i18n + app-shell)+ 本地全量被容器重启打断(排队等共享验证锁时容器第三次重启,PM 裁定不再重跑本地全量),交由本 PR 的 CI 把关⚠️ CI 把关

Changeset

@object-ui/app-shell + @object-ui/i18n 均 patch(fixed group 联动,无 major)。

🤖 Generated with Claude Code

https://claude.ai/code/session_01NVPjPzmmAJ2Ngtvgg5MSRa

… via i18n (#3321)
The opensInNewTab choreography in consoleServerAction.ts shipped hard-coded
bilingual Chinese/English copy, violating AGENTS.md Commandment #-1. The four
strings (spinner tab title + body, popup-blocked toast title/description/
action label) now route through new console.serverAction.* keys in
@object-ui/i18n (all eleven locale packs, key parity kept), injected via an
optional i18next-style t option on createConsoleServerActionHandler from the
two hook-context call sites; English defaults apply when t is omitted. Locale
strings are HTML-escaped before entering the spinner document.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NVPjPzmmAJ2Ngtvgg5MSRa
@vercel

vercelBot commented Aug 4, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
ProjectDeploymentActionsUpdated (UTC)
objectuiIgnoredIgnoredAug 4, 2026 12:55am

Request Review

@github-actions

Copy link
Copy Markdown
Contributor

✅ Console Performance Budget

MetricValueBudget
Main entry (gzip)28.1 KB350 KB
Entry fileindex-BXVhb_qJ.js
StatusPASS

📦 Bundle Size Report

PackageSizeGzipped
app-shell (index.js)8.47KB3.09KB
app-shell (runtime-config.js)7.42KB2.32KB
app-shell (types.js)0.01KB0.04KB
app-shell (urlParams.js)7.57KB2.97KB
auth (AuthContext.js)0.31KB0.24KB
auth (AuthGuard.js)1.17KB0.53KB
auth (AuthProvider.js)22.10KB4.37KB
auth (AuthShell.js)3.49KB1.40KB
auth (ForgotPasswordForm.js)12.21KB3.45KB
auth (LoginForm.js)18.13KB5.39KB
auth (PreviewBanner.js)0.90KB0.50KB
auth (RegisterForm.js)6.64KB2.21KB
auth (SocialSignInButtons.js)9.60KB3.89KB
auth (UserMenu.js)3.40KB1.22KB
auth (auth-gate-events.js)1.29KB0.66KB
auth (authStyles.js)5.04KB1.72KB
auth (createAuthClient.js)35.76KB9.11KB
auth (createAuthenticatedFetch.js)4.37KB1.69KB
auth (index.js)2.35KB1.07KB
auth (org-roles.js)6.66KB2.78KB
auth (phone-identifier.js)1.11KB0.66KB
auth (types.js)0.59KB0.35KB
auth (useAuth.js)4.91KB0.87KB
auth (useIsWorkspaceAdmin.js)1.61KB0.85KB
collaboration (CommentThread.js)18.38KB4.49KB
collaboration (LiveCursors.js)3.17KB1.27KB
collaboration (PresenceAvatars.js)3.65KB1.42KB
collaboration (PresenceProvider.js)2.79KB1.13KB
collaboration (index.js)1.25KB0.53KB
collaboration (useCommentSearch.js)1.98KB0.88KB
collaboration (useConflictResolution.js)7.75KB1.86KB
collaboration (useMentionNotifications.js)1.81KB0.68KB
collaboration (usePresence.js)6.33KB1.84KB
collaboration (useRealtimeSubscription.js)7.91KB2.01KB
components (index.js)476.66KB104.69KB
core (index.js)2.25KB0.80KB
create-plugin (index.js)9.28KB2.98KB
data-objectstack (index.js)136.23KB34.75KB
fields (index.js)224.99KB55.14KB
i18n (LocalizationContext.js)1.76KB0.96KB
i18n (currency.js)1.22KB0.64KB
i18n (i18n.js)4.32KB1.77KB
i18n (index.js)2.46KB0.96KB
i18n (pickLocalized.js)1.70KB0.83KB
i18n (provider.js)5.37KB1.72KB
i18n (useObjectLabel.js)26.14KB6.07KB
i18n (useSafeTranslation.js)3.26KB1.44KB
layout (index.js)38.53KB10.71KB
mobile (MobileProvider.js)0.92KB0.49KB
mobile (ResponsiveContainer.js)0.94KB0.38KB
mobile (breakpoints.js)1.51KB0.70KB
mobile (createOfflineDataSource.js)5.61KB1.74KB
mobile (index.js)1.50KB0.62KB
mobile (offlineQueue.js)3.91KB1.35KB
mobile (pwa.js)0.97KB0.49KB
mobile (serviceWorker.js)1.48KB0.62KB
mobile (serviceWorkerSource.js)3.41KB1.48KB
mobile (useBreakpoint.js)1.54KB0.65KB
mobile (useGesture.js)6.96KB1.98KB
mobile (useOfflineSync.js)1.99KB0.72KB
mobile (usePullToRefresh.js)2.53KB0.85KB
mobile (useResponsive.js)0.71KB0.42KB
mobile (useResponsiveConfig.js)1.36KB0.63KB
mobile (useSpecGesture.js)4.05KB1.53KB
mobile (useTouchTarget.js)1.01KB0.54KB
permissions (MePermissionsProvider.js)8.75KB3.06KB
permissions (PermissionContext.js)0.31KB0.25KB
permissions (PermissionGuard.js)0.89KB0.45KB
permissions (PermissionProvider.js)3.67KB1.12KB
permissions (evaluator.js)4.41KB1.44KB
permissions (index.js)0.91KB0.41KB
permissions (store.js)0.91KB0.42KB
permissions (useFieldPermissions.js)1.28KB0.52KB
permissions (usePermissions.js)1.55KB0.71KB
plugin-ai (index.js)15.71KB3.79KB
plugin-calendar (index.js)44.98KB12.37KB
plugin-charts (index.js)60.54KB17.13KB
plugin-chatbot (index.js)180.09KB42.72KB
plugin-dashboard (index.js)112.01KB28.86KB
plugin-designer (index.js)210.51KB42.51KB
plugin-detail (index.js)230.56KB56.80KB
plugin-editor (index.js)2.46KB1.10KB
plugin-form (index.js)111.54KB26.97KB
plugin-gantt (index.js)162.25KB39.55KB
plugin-grid (index.js)185.08KB49.04KB
plugin-kanban (index.js)47.89KB13.18KB
plugin-list (index.js)104.94KB25.32KB
plugin-map (index.js)16.81KB5.24KB
plugin-markdown (index.js)13.72KB4.69KB
plugin-report (index.js)40.55KB10.59KB
plugin-timeline (index.js)25.76KB7.33KB
plugin-tree (index.js)8.34KB2.82KB
plugin-view (index.js)83.67KB20.43KB
providers (DataSourceProvider.js)0.75KB0.39KB
providers (MetadataProvider.js)1.37KB0.59KB
providers (ThemeProvider.js)1.90KB0.85KB
providers (UploadProvider.js)11.71KB3.53KB
providers (index.js)0.44KB0.22KB
providers (types.js)0.01KB0.04KB
react-runtime (index.js)5.67KB2.37KB
react (LazyPluginLoader.js)3.77KB1.33KB
react (SchemaRenderer.js)19.28KB6.38KB
react (data-invalidation.js)5.05KB2.08KB
react (index.js)1.02KB0.55KB
react (spec-input.js)0.20KB0.18KB
sdui-parser (codegen.js)4.09KB1.74KB
sdui-parser (index.js)4.47KB2.03KB
sdui-parser (parse.js)10.04KB2.82KB
sdui-parser (types.js)0.29KB0.24KB
sdui-parser (validate.js)4.69KB1.48KB
types (ai.js)0.20KB0.17KB
types (api-types.js)0.20KB0.18KB
types (app.js)2.87KB0.99KB
types (base.js)0.20KB0.18KB
types (blocks.js)0.20KB0.18KB
types (complex.js)0.20KB0.18KB
types (crud.js)0.20KB0.18KB
types (data-display.js)0.20KB0.18KB
types (data-protocol.js)0.20KB0.19KB
types (data.js)0.20KB0.18KB
types (designer.js)1.87KB0.85KB
types (disclosure.js)0.20KB0.18KB
types (error-code.js)1.54KB0.88KB
types (feedback.js)0.20KB0.18KB
types (field-types.js)0.20KB0.18KB
types (form.js)0.20KB0.18KB
types (http-retry.js)4.32KB2.02KB
types (index.js)2.46KB1.21KB
types (layout.js)0.20KB0.18KB
types (managed-by.js)0.19KB0.18KB
types (mobile.js)0.20KB0.18KB
types (navigation.js)0.20KB0.18KB
types (objectql.js)0.20KB0.18KB
types (overlay.js)0.20KB0.18KB
types (permissions.js)0.20KB0.18KB
types (plugin-scope.js)0.20KB0.18KB
types (record-components.js)0.20KB0.19KB
types (record-semantics.js)1.28KB0.67KB
types (registry.js)0.20KB0.18KB
types (reports.js)0.20KB0.18KB
types (spec-report.js)5.05KB1.93KB
types (system-fields.js)3.33KB1.54KB
types (theme.js)0.20KB0.18KB
types (ui-action.js)3.40KB1.71KB
types (views.js)0.20KB0.18KB
types (widget.js)0.20KB0.18KB

Size Limits

  • ✅ Core packages should be < 50KB gzipped
  • ✅ Component packages should be < 100KB gzipped
  • ⚠️ Plugin packages should be < 150KB gzipped

@xuyushun441-sys
xuyushun441-sys marked this pull request as ready for review August 4, 2026 01:48
@xuyushun441-sys
xuyushun441-sys added this pull request to the merge queueAug 4, 2026
Merged via the queue into main with commit a415684Aug 4, 2026
14 of 16 checks passed
@xuyushun441-sys
xuyushun441-sys deleted the claude/issue-3321-popup-copy-english branch August 4, 2026 01:48
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Popup-blocked toast and SSO spinner tab ship bilingual Chinese/English copy, violating the English-only commandment

2 participants

@xuyushun441-sys@claude