Skip to content

fix(app-shell): AI 卡片的出站文本随会话语言取值,不再读 UI 包 (#3896) - #3921

Merged
yinlianghui merged 1 commit into
mainfrom
claude/issue-3896-outbound-conversation-language
Aug 9, 2026
Merged

fix(app-shell): AI 卡片的出站文本随会话语言取值,不再读 UI 包 (#3896)#3921
yinlianghui merged 1 commit into
mainfrom
claude/issue-3896-outbound-conversation-language

Conversation

@yinlianghui

Copy link
Copy Markdown
Collaborator

Fixes #3896

origin/main @ 2937bcf7db0da9d27cd4f330a7033d2b380d2335 切出。

缺陷:规则只实现了一半

#772 / #2884 立下的规则写在 AiChatPage 的门上 —— 出站文本随会话语言,渲染标签随 UI 语言。三条出站文本的写法是 convZh ? '中文字面量' : t('console.ai.…'),而 t() 读的是 UI 包,于是「非中文」那一支答的是 UI 语言、不是英文;第四条 planAnswerMessage 连门都没过。两个方向各一个已实测后果:

修法:一个会话语言取值器,四个站点

新增 packages/app-shell/src/console/ai/outboundAgentText.ts:中文会话取 zh 包该 key,其余会话取 en 包该 key,永不读 UI 包;planAnswerMessage 纳入同门({{question}}/{{option}} 占位符照填)。AiChatPageconvZh 现在只有一个消费者 —— 这个取值器的桥。三个控制台 AI 界面(/ai 页、ChatDockStudioAiCopilot)都挂同一个 ChatPane,一并生效。

为什么直接读两个包而不是 t(key, { lng }):i18next 的答案取决于宿主 app 装了哪些 bundle 以及 fallbackLng —— 一次 zh 查询在缺 zh bundle 时会静默回落到 en,那正是本单要修的「错语言」bug 换个触发条件重生。取值器里按语言分的兜底表被钉成与两个包逐字节一致,所以万一某个包不再定义该 key,中文会话兜底到中文,绝不兜到英文。

顺带的活性收益:今天 zh 包这四个值对中文会话是读不到的(中文会话走硬编码字面量),唯一读到它们的路径就是那个 bug;修完之后它们成了中文会话的正规来源。

钉子

反向验证(预判先写,再跑)

把四个站点还原成修前写法(取值器与全部测试保持已提交状态):

组合 / 钉子 预判 实测
zh UI + 英文会话(三条 *Message) —— 收到 zh 包的中文,见下方 diff
en UI + 英文会话 / de UI + 英文会话 绿 绿
en UI + 中文会话(planAnswerMessage)
de UI + 中文会话(planAnswerMessage)
zh UI + 中文会话 绿(不变量半边) 绿
console-namespace-3546 源码钉 —— convZh 代码行 3 → 6
outboundAgentText.test.ts 绿 绿
#3837 那条「三条出站文本」旧钉 绿 绿

最后两行是刻意预判成绿的,也是本单为什么必须单独立钉:取值器单测钉的是取值器的契约、不是它的采用;#3837 那条旧钉只测了 en UI + 中文会话和 de UI + 英文会话两格,对本缺陷天生盲。合计 4 红 / 66。

判别性最强的那格:

- Expected                                            + Received
- "planApproveMessage": "Looks good — build it as proposed."
+ "planApproveMessage": "确认,开始搭建。"
- "changesConfirmMessage": "Confirm — apply the change you just proposed."
+ "changesConfirmMessage": "确认修改,应用你刚才提议的改动。"

测试证据

  • pnpm --filter '@object-ui/app-shell^...' build → EXIT=0
  • 消费半径全量(packages/app-shell/src/console/ai + packages/i18n,含四个被改的钉子文件):48 files / 682 tests passed
  • 全仓 turbo run type-check --concurrency=2:78 successful, 78 total
  • node scripts/check-control-bytes.mjs:OK(3797 tracked text files);改动文件另跑控制字节自扫,零命中
  • eslint 改动文件:0 errors(新增两个文件零告警)

未动的东西

标签仍随 UI 语言,zh/en 语言包的值一字未改(所以 check:i18n-en-drift 不涉及),de.ts 未碰(#3876 在飞),content/docs/releases/ 未碰。changeset:@object-ui/app-shell patch。


Generated by Claude Code

#772 / #2884 立下的规则写在 `AiChatPage` 的门上:出站文本随**会话**语言,渲染
标签随 UI 语言。规则只实现了一半 —— 三条出站文本写作
`convZh ? '<中文字面量>' : t('console.ai.…')`,而 `t()` 读的是 **UI 包**,于是
「非中文」那一支答的是 UI 语言、不是英文;第四条 `planAnswerMessage` 连门都没过。

两个方向各一个已实测后果:

- **zh 控制台 + 英文会话发出中文。** zh 包定义了全部四个 key,所以 `t()` 是命中
  而非落到英文 defaultValue:点 Build it,进英文线程的是 `确认,开始搭建。`。cloud
  确认门(`service-ai-studio` `confirm-gate.ts` `APPROVAL_RE`)两种语言都认,构建
  照走,agent 随后把整条线程转成中文 —— 即 #2884 的症状、触发方向相反。
- **一键回答芯片两个方向都发 UI 语言。** 完全未门控:中文会话里发出
  `For "…", go with: …`(#772 开头那句抱怨的字面重演),zh 控制台下的英文会话里
  发出中文句子。

四个站点现在统一走一个取值器 `outboundAgentText.ts`:中文会话取 `zh` 包该 key,
其余会话取 `en` 包该 key,永不读 UI 包。三个控制台 AI 界面(`/ai` 页、chat dock、
Studio copilot)都挂同一个 `ChatPane`,一并生效。

直接读两个包而不用 `t(key, { lng })` 是刻意的:i18next 的答案取决于宿主 app 装了
哪些 bundle 以及 `fallbackLng`,而 zh 查询静默回落到 en 正是上面那个错语言 bug。
取值器内的兜底表按语言分,并被钉成与两个包逐字节一致 —— 万一某个包不再定义该 key,
中文会话兜底到中文,绝不兜到英文。

标签未动:仍随 UI 语言。#3837 那条「`*Label` 漂回会话门就红」的钉子现在同时守住
反向 —— 出站 `*Message` 被重新塞回 `t()` 也会红。

Co-authored-by: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GTRjn8xBqp75dk7kFupVRt
@github-actions github-actions Bot added the tests label Aug 9, 2026
@vercel

vercel Bot commented Aug 9, 2026

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
objectui Ignored Ignored Aug 9, 2026 2:49am

Request Review

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

✅ Console Performance Budget

Metric Value Budget
Main entry (gzip) 28.1 KB 350 KB
Entry file index-DZfVsTSU.js
Status PASS

📦 Bundle Size Report

Package Size Gzipped
app-shell (index.js) 8.66KB 3.13KB
app-shell (runtime-config.js) 7.42KB 2.32KB
app-shell (types.js) 0.01KB 0.04KB
app-shell (urlParams.js) 7.57KB 2.97KB
auth (AuthContext.js) 0.31KB 0.24KB
auth (AuthGuard.js) 1.17KB 0.53KB
auth (AuthProvider.js) 22.10KB 4.37KB
auth (AuthShell.js) 3.49KB 1.40KB
auth (ForgotPasswordForm.js) 12.21KB 3.45KB
auth (LoginForm.js) 18.13KB 5.39KB
auth (PreviewBanner.js) 0.90KB 0.50KB
auth (RegisterForm.js) 6.64KB 2.21KB
auth (SocialSignInButtons.js) 9.60KB 3.89KB
auth (UserMenu.js) 3.40KB 1.22KB
auth (auth-gate-events.js) 1.29KB 0.66KB
auth (authStyles.js) 5.04KB 1.72KB
auth (createAuthClient.js) 35.76KB 9.11KB
auth (createAuthenticatedFetch.js) 4.37KB 1.69KB
auth (index.js) 2.35KB 1.07KB
auth (org-roles.js) 6.66KB 2.78KB
auth (phone-identifier.js) 1.11KB 0.66KB
auth (types.js) 0.59KB 0.35KB
auth (useAuth.js) 4.91KB 0.87KB
auth (useIsWorkspaceAdmin.js) 1.61KB 0.85KB
collaboration (CommentThread.js) 26.07KB 7.56KB
collaboration (LiveCursors.js) 3.17KB 1.27KB
collaboration (PresenceAvatars.js) 6.49KB 2.64KB
collaboration (PresenceProvider.js) 2.79KB 1.13KB
collaboration (index.js) 1.65KB 0.73KB
collaboration (useCollaborationTranslation.js) 6.05KB 2.52KB
collaboration (useCommentSearch.js) 1.98KB 0.88KB
collaboration (useConflictResolution.js) 7.75KB 1.86KB
collaboration (useMentionNotifications.js) 1.81KB 0.68KB
collaboration (usePresence.js) 6.33KB 1.84KB
collaboration (useRealtimeSubscription.js) 7.91KB 2.01KB
components (index.js) 482.39KB 106.34KB
core (index.js) 2.96KB 1.13KB
create-plugin (index.js) 10.08KB 3.26KB
data-objectstack (index.js) 139.61KB 35.99KB
fields (index.js) 230.97KB 56.74KB
i18n (LocalizationContext.js) 1.76KB 0.96KB
i18n (currency.js) 1.22KB 0.64KB
i18n (i18n.js) 4.32KB 1.77KB
i18n (index.js) 2.65KB 1.06KB
i18n (pickLocalized.js) 1.70KB 0.83KB
i18n (provider.js) 9.48KB 3.27KB
i18n (useObjectLabel.js) 27.59KB 6.63KB
i18n (useSafeTranslation.js) 4.52KB 1.96KB
layout (index.js) 38.53KB 10.71KB
mobile (MobileProvider.js) 0.92KB 0.49KB
mobile (ResponsiveContainer.js) 0.94KB 0.38KB
mobile (breakpoints.js) 1.51KB 0.70KB
mobile (createOfflineDataSource.js) 5.61KB 1.74KB
mobile (index.js) 1.50KB 0.62KB
mobile (offlineQueue.js) 3.91KB 1.35KB
mobile (pwa.js) 0.97KB 0.49KB
mobile (serviceWorker.js) 1.48KB 0.62KB
mobile (serviceWorkerSource.js) 3.41KB 1.48KB
mobile (useBreakpoint.js) 1.54KB 0.65KB
mobile (useGesture.js) 6.96KB 1.98KB
mobile (useOfflineSync.js) 1.99KB 0.72KB
mobile (usePullToRefresh.js) 2.53KB 0.85KB
mobile (useResponsive.js) 0.71KB 0.42KB
mobile (useResponsiveConfig.js) 1.36KB 0.63KB
mobile (useSpecGesture.js) 4.32KB 1.64KB
mobile (useTouchTarget.js) 1.01KB 0.54KB
permissions (MePermissionsProvider.js) 8.75KB 3.06KB
permissions (PermissionContext.js) 0.31KB 0.25KB
permissions (PermissionGuard.js) 0.89KB 0.45KB
permissions (PermissionProvider.js) 3.67KB 1.12KB
permissions (evaluator.js) 4.41KB 1.44KB
permissions (index.js) 0.91KB 0.41KB
permissions (store.js) 0.91KB 0.42KB
permissions (useFieldPermissions.js) 1.28KB 0.52KB
permissions (usePermissions.js) 1.55KB 0.71KB
plugin-ai (index.js) 15.71KB 3.79KB
plugin-calendar (index.js) 44.98KB 12.37KB
plugin-charts (index.js) 61.04KB 17.31KB
plugin-chatbot (index.js) 180.33KB 42.79KB
plugin-dashboard (index.js) 117.21KB 30.27KB
plugin-designer (index.js) 210.51KB 42.51KB
plugin-detail (index.js) 236.63KB 59.02KB
plugin-editor (index.js) 2.46KB 1.10KB
plugin-form (index.js) 112.10KB 27.10KB
plugin-gantt (index.js) 162.55KB 39.57KB
plugin-grid (index.js) 187.63KB 49.66KB
plugin-kanban (index.js) 48.30KB 13.28KB
plugin-list (index.js) 105.12KB 25.48KB
plugin-map (index.js) 16.81KB 5.24KB
plugin-markdown (index.js) 13.72KB 4.69KB
plugin-report (index.js) 40.58KB 10.58KB
plugin-timeline (index.js) 25.76KB 7.33KB
plugin-tree (index.js) 8.50KB 2.88KB
plugin-view (index.js) 84.03KB 20.55KB
providers (DataSourceProvider.js) 0.75KB 0.39KB
providers (MetadataProvider.js) 1.37KB 0.59KB
providers (ThemeProvider.js) 1.90KB 0.85KB
providers (UploadProvider.js) 11.71KB 3.53KB
providers (index.js) 0.44KB 0.22KB
providers (types.js) 0.01KB 0.04KB
react-runtime (index.js) 5.67KB 2.37KB
react (LazyPluginLoader.js) 3.77KB 1.33KB
react (SchemaRenderer.js) 19.28KB 6.38KB
react (data-invalidation.js) 5.05KB 2.08KB
react (index.js) 1.02KB 0.55KB
react (spec-input.js) 0.20KB 0.18KB
sdui-parser (codegen.js) 4.09KB 1.74KB
sdui-parser (index.js) 4.47KB 2.03KB
sdui-parser (parse.js) 10.04KB 2.82KB
sdui-parser (types.js) 0.29KB 0.24KB
sdui-parser (validate.js) 4.69KB 1.48KB
types (ai.js) 0.20KB 0.17KB
types (api-types.js) 0.20KB 0.18KB
types (app.js) 2.87KB 0.99KB
types (base.js) 0.20KB 0.18KB
types (blocks.js) 0.20KB 0.18KB
types (complex.js) 0.20KB 0.18KB
types (crud.js) 0.20KB 0.18KB
types (data-display.js) 0.20KB 0.18KB
types (data-protocol.js) 0.20KB 0.19KB
types (data.js) 0.20KB 0.18KB
types (designer.js) 1.87KB 0.85KB
types (disclosure.js) 0.20KB 0.18KB
types (error-code.js) 1.54KB 0.88KB
types (feedback.js) 0.20KB 0.18KB
types (field-types.js) 0.20KB 0.18KB
types (form.js) 0.20KB 0.18KB
types (http-retry.js) 4.32KB 2.02KB
types (index.js) 2.71KB 1.34KB
types (layout.js) 0.20KB 0.18KB
types (managed-by.js) 0.19KB 0.18KB
types (mobile.js) 2.59KB 1.31KB
types (navigation.js) 0.20KB 0.18KB
types (objectql.js) 0.20KB 0.18KB
types (overlay.js) 0.20KB 0.18KB
types (permissions.js) 0.20KB 0.18KB
types (plugin-scope.js) 0.20KB 0.18KB
types (record-components.js) 0.20KB 0.19KB
types (record-semantics.js) 1.28KB 0.67KB
types (registry.js) 0.20KB 0.18KB
types (reports.js) 0.20KB 0.18KB
types (spec-report.js) 5.05KB 1.93KB
types (system-fields.js) 3.33KB 1.54KB
types (theme.js) 0.20KB 0.18KB
types (ui-action.js) 3.40KB 1.71KB
types (views.js) 0.20KB 0.18KB
types (widget.js) 0.20KB 0.18KB

Size Limits

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

@yinlianghui
yinlianghui marked this pull request as ready for review August 9, 2026 02:55
@yinlianghui
yinlianghui added this pull request to the merge queue Aug 9, 2026
Merged via the queue into main with commit 99ba5fb Aug 9, 2026
21 checks passed
@yinlianghui
yinlianghui deleted the claude/issue-3896-outbound-conversation-language branch August 9, 2026 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

2 participants