fix(collaboration,i18n): @object-ui/collaboration 整包接入 i18n,CommentThread 不再硬编码英文 (objectstack#5506) - #3438
Merged
Conversation
…read 不再硬编码英文 (objectstack#5506)
`@object-ui/collaboration` 此前只依赖 `@object-ui/types`,所有用户可见文案都是
英文字面量 —— zh 会话下是一个中文控制台里套着一个英文评论区:"3 comments"、
"Reply"、"Resolve"、"just now"、"Add a comment... (use @ to mention)"。
按 PM 裁定走方案 A(整包一次接入),不做单串补丁、不做 formatter prop 兜底:
包内新增唯一翻译入口 `useCollaborationTranslation` / `COLLAB_DEFAULT_TRANSLATIONS`,
底层是 `createSafeTranslation`(data-table / form / filter-builder 既有模式)。
有 provider 时跟随会话语言,无 provider 时落到 defaults map 的英文 ——
后者不是锦上添花:CommentThread 是可独立使用的导出组件,宿主可能整棵树都没挂
I18nProvider,把字面量塞进没有兜底的 t() 正是把标签变成 `collaboration.reply`
这种裸 key 的方式。
单据列了 13 处,逐行复核实际是 **20 处**。原 sweep 漏掉的 7 处:
`{n}h ago`、`{n}d ago`、`(edited)`、评论条数、`Oldest`/`Newest` 排序项、
`Replying to {name}...`、发送按钮 `Send`。20 处全部入键 —— 漏掉任何一处
都会发出一个"翻译了九成"的评论区。
其中两处除了没翻译,还多带一层缺陷:被编译进组件的不只是词,还有**复数规则**。
- 头部是 `` `${n} comment${n !== 1 ? 's' : ''}` ``;
- 表情回应 tooltip 是 `` n === 1 ? '1 reaction' : `${n} reactions` ``。
两处产出的**英文都是对的** —— 这不是 #3423 在页签徽标上修的那个 "1 items" 缺陷 ——
但选哪一种形式是硬写进渲染路径的英文语法。任何语言都无法套用自己的规则:
ru 需要三种形式、ja 一种都不需要,无论语言包里写什么都表达不出来。
两处都改用本仓既有的**双 key** 约定(`collaboration.commentCount`/`commentCountOne`、
`collaboration.reactionCount`/`reactionCountOne`),不用 i18next 的 `_one`/`_other`
后缀:zh/ja/ko 没有独立单数形式,那几个包会合理地缺 `_one` 半边,
all-locales-key-parity 会把它读成丢键。count 以**字符串**插值,
i18next 因此跳过自己的复数解析(needsPluralHandling 对字符串 count 为 false),
选单复数的始终是双 key 方案。
reaction tooltip 用**专属**键对,没有复用 `detail.reactionCount`:后者插值
`{{emoji}}`,而这个调用点的 emoji 就是按钮本身的可见内容,没有值可以传给占位符,
复用会在每种语言下把字面量 `{{emoji}}` 留在无障碍名里。
相对时间按词条级处理:分/时/天分档原样不动,没有引入任何日期库。
`>= 7d` 分支仍走运行时自己的 `toLocaleDateString()` —— 它本来就不是硬编码英文,
而把它钉到会话语言另有失败模式(不认识的 tag 抛 RangeError 落进外层 catch,
反而会渲染出原始 ISO 串),另行跟踪。
`Save` / `Cancel` / `Edit` / `Delete` 读共享的 `common` 命名空间,没有在
`collaboration` 下再拼一遍:它们是通用动作词,十个包里早就有译文,
再写一份只是多一处要同步的地方。真正新增的 21 条 key 十个语言包全补真实译文。
测试方向(先预测,再按实跑结果订正 —— 有一条预测错了,如实记下而不是抹掉):
- 有 provider 的文件:把 CommentThread.tsx 回滚到 origin/main、保留新测试后,
zh/de/ja 共 13 个用例全红,4 个 en 用例全绿。en 两侧都绿正是目的,不是漏测:
英文文案必须原样存活,翻转反而说明文案被改动了。
- **预测错的那一条**:原以为 en 单数用例("1 comment")改前就红,即假定头部像
#3423 里的页签徽标那样只拼了个 `s`、根本没有单数分支。实跑证明不是 ——
origin/main 两处都写了单数分支,英文输出本来就正确。所以它和其余 en 用例
一样两侧都绿,真正钉住"复数规则不能编译进组件"的是 de 用例。
- 无 provider 的文件:**所有**断言两侧都绿。这就是它要钉的不变量(英文兜底不许被
t() 吃掉);它能抓的是本次改动自己引入的破坏 —— 组件里接了某个 key 而
COLLAB_DEFAULT_TRANSLATIONS 里漏了或拼错了,只有这个文件会红。
无 provider 的断言必须单独成文件:`createI18n` 调 `instance.use(initReactI18next)`,
该实例会被注册成 react-i18next 的模块级默认实例,卸载和 cleanup() 都不撤销;
同一文件里只要先挂过一次 I18nProvider,之后所有"无 provider"渲染都会静默落到
那个语言上(踩坑记录见 PR #3423)。
Fixes #3424
Refs objectstack-ai/objectstack#5506
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GTRjn8xBqp75dk7kFupVRt
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
Contributor
✅ Console Performance Budget
📦 Bundle Size Report
Size Limits
|
This was referenced Aug 6, 2026
yinlianghui
marked this pull request as ready for review
August 6, 2026 04:42
This was referenced Aug 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #3424
Refs objectstack-ai/objectstack#5506
@object-ui/collaboration此前只依赖@object-ui/types,所有用户可见文案都是英文字面量 —— zh 会话下是一个中文控制台里套着一个英文评论区:"3 comments"、"Reply"、"Resolve"、"just now"、"Add a comment... (use @ to mention)"。按 PM 裁定走方案 A(整包一次接入),不做单串补丁、不做 formatter prop 兜底。
做了什么
packages/collaboration/package.json加@object-ui/i18n: workspace:*,pnpm-lock.yaml只多出 importers 里的三行。packages/collaboration/src/useCollaborationTranslation.ts——useCollaborationTranslation/COLLAB_DEFAULT_TRANSLATIONS,底层是createSafeTranslation(data-table / form / filter-builder 既有模式)。有 provider 时跟随会话语言,无 provider 时落到 defaults map 的英文。defaults map 那一半不是锦上添花:CommentThread 是可独立使用的导出组件,宿主可能整棵树都没挂 provider,把字面量塞进没有兜底的
t()正是把标签变成collaboration.reply这种裸 key 的方式,而且是静默的 —— 没有 i18next 实例时,fallbackLng: 'en'根本没有机会触发。实际是 20 处,不是 13 处
单据列了 13 处,逐行复核实际是 20 处。原 sweep 漏掉的 7 处:
{n}h ago、{n}d ago、(edited)、评论条数、Oldest/Newest排序项、Replying to {name}...、发送按钮Send。20 处全部入键 —— 漏掉任何一处都会发出一个"翻译了九成"的评论区。两处的复数规则也被编译进了组件
`${n} comment${n !== 1 ? 's' : ''}`n === 1 ? '1 reaction' : `${n} reactions`两处产出的英文都是对的 —— 这不是 #3423 在页签徽标上修的那个 "1 items" 缺陷 —— 但选哪一种形式是硬写进渲染路径的英文语法。任何语言都无法套用自己的规则:ru 需要三种形式、ja 一种都不需要,无论语言包里写什么都表达不出来。
两处都改用本仓既有的双 key 约定(
collaboration.commentCount/commentCountOne、collaboration.reactionCount/reactionCountOne),⛔ 不用 i18next 的_one/_other后缀:zh/ja/ko 没有独立单数形式,那几个包会合理地缺_one半边,all-locales-key-parity会把它读成丢键。count 以字符串插值,i18next 因此跳过自己的复数解析,选单复数的始终是双 key 方案。约束逐条对照
detail.reactionCount—— 后者插值{{emoji}},而这个调用点的 emoji 就是按钮本身的可见内容,没有值可以传给占位符,复用会在每种语言下把字面量{{emoji}}留在无障碍名里。测试里有一条专门扫所有title属性,断言不含{{。comment-thread-no-provider-fallback.test.tsx从不 import I18nProvider。createI18n调instance.use(initReactI18next),该实例会被注册成 react-i18next 的模块级默认实例,卸载和cleanup()都不撤销。>= 7d分支仍走运行时自己的toLocaleDateString()—— 它本来就不是硬编码英文,而把它钉到会话语言另有失败模式(不认识的 tag 抛 RangeError 落进外层 catch,反而会渲染出原始 ISO 串),另行跟踪。Save/Cancel/Edit/Delete读共享的common命名空间,没有在collaboration下再拼一遍:它们是通用动作词,十个包里早就有译文,再写一份只是多一处要同步的地方。测试方向:先预测,再按实跑结果订正
有一条预测错了,如实记在这里和两个测试文件的头注释里,而不是抹掉。
把
CommentThread.tsx回滚到origin/main、保留新测试后:s、根本没有单数分支。实跑证明不是:origin/main两处都写了单数分支,英文输出本来就正确。所以它和其余 en 用例一样两侧都绿,真正钉住"复数规则不能编译进组件"的是 de 用例。COLLAB_DEFAULT_TRANSLATIONS里漏了或拼错了,只有这个文件会红。跑过的命令(worktree 仓根,
flock+--maxWorkers=2):vitest run packages/collaboration packages/i18n→ 25 files / 287 tests passed(含all-locales-key-parity、en-zh-key-parity)pnpm --filter @object-ui/collaboration --filter @object-ui/i18n type-check→ Done(collaboration 含tsc -p tsconfig.test.json)eslint触碰到的文件 → 0 errors(仅useRealtimeSubscription.ts等既有 warning)pnpm check:control-bytes→ OK;另对每个改动文件做了grep -naP自查CommentThread使用者;app-shell 那四个vi.mock('@object-ui/collaboration')的用例照跑,25 passedGenerated by Claude Code