Skip to content

Fix/i18n keycheck flatten - #17

Open
snqig wants to merge 3 commits into
mainfrom
fix/i18n-keycheck-flatten
Open

snqig wants to merge 3 commits into
mainfrom
fix/i18n-keycheck-flatten

Conversation

@snqig

@snqig snqig commented Sep 9, 2026

Copy link
Copy Markdown
Owner

PR 标题

变更说明

简要描述本次 PR 的改动内容。

变更类型

  • 新功能(feat)
  • Bug 修复(fix)
  • 重构(refactor)
  • 文档(docs)
  • 测试(test)
  • 构建/CI(build/ci)
  • 其他(chore)

关联 Issue

Closes #

事件发布检查清单

本次变更若涉及领域事件(Domain Event)发布,请逐项确认;Transactional Outbox 模式要求聚合写入与事件写入必须在同一事务内。

  • 本次变更不涉及领域事件发布(如勾选此项可跳过下列项)
  • 聚合写入与 persistEvents() 在同一 transaction()
  • 事务提交后调用 order.clearDomainEvents()
  • Handler 实现了幂等性(INSERT IGNORE + affectedRows 检查,或类似机制)
  • 已补充/更新对应事件的单元测试或集成测试
  • 已更新 docs/印前模块文档.md / 生产模块文档.md / 打样模块文档.md 中的事件列表(如适用)

测试清单

  • 单元测试通过 pnpm test:unit:run
  • 集成测试通过(如适用)
  • E2E 测试通过(如适用)
  • 覆盖率未下降 pnpm test:coverage

通用检查清单

  • 代码通过 pnpm lint
  • 类型检查通过 pnpm ts-check
  • 提交信息遵循 Conventional Commits 规范
  • 无硬编码密钥或敏感信息

Breaking Changes

  • 本次变更包含 Breaking Change

部署注意事项

  • 需要 DB migration
  • 需要新增/修改环境变量
  • 需要调整 Redis 配置
  • 需要数据回填脚本

zh-TW: 补齐缺失的 93 个键(含 Error 命名空间,经 opencc 转繁体)

en/vi: 新增 Error 命名空间 36 键翻译 + 补齐 3 个非 Error 缺失键

en: prune 49 个代码未引用的死键(别名键,已 verify_en_extra 校验 0 引用)

src/i18n/request.ts: 落地 fallbackLocale 深合并回退(阶段1)

孤儿 key 精扫: diagnose_i18n_keys + i18n-key-check --unused(zh-CN 5646 孤儿键待清理;226 缺失含 73 嵌套误报)
从 git 历史父版本恢复 48 个键的原始中文(46 直接恢复 + 2 消歧:k_1cy20k0=新建评审、k_1wtck8n=余料)。

其余 79 个为 Dcprint 自描述英文键(cardSelected/scanQR/uvInk 等)、Common.activate/deactivate、Hr.fillMonth、SalesReturn.selectOrder,按键名+代码上下文推断中文;仅 k_1gfnr28(成本表空态)、k_sfsp66(仓库名称) 为上下文推断待复核。

验证:i18n-key-check 代码→zh-CN 真实扁平缺失键归零(剩余 73 为脚本浅校验嵌套键误报,非真实缺口);CRLF 行尾保留;JSON 可解析。
loadMessages 改为扁平化点键(新增 flatten 辅助函数),Monitoring/trace 等嵌套命名空间的 73 个键不再误报为 missing(实测 73→0);--unused 颗粒度同步精细化(5646→5649,真正嵌套键仅 ~71 个,浅校验对 unused 虚高仅 ~3)。注:审计报告原推“--unused 降为真实 5081”系与 measure-2026-09-09.mjs 口径混淆的误判,本脚本扁平化后实测忠实值为 5649。
@vercel

vercel Bot commented Sep 9, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
vnerp Error Error Sep 9, 2026 1:41am UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant