Skip to content

fix: correct typos in Python, Chinese, Japanese and Korean locales#1774

Merged
SengokuCola merged 1 commit into
Mai-with-u:mainfrom
jmclulu:fix/typos
May 31, 2026
Merged

fix: correct typos in Python, Chinese, Japanese and Korean locales#1774
SengokuCola merged 1 commit into
Mai-with-u:mainfrom
jmclulu:fix/typos

Conversation

@jmclulu
Copy link
Copy Markdown
Contributor

@jmclulu jmclulu commented May 30, 2026

  • Fix "emoji_registed" → "emoji_registered" in mmipkg_tool.py
  • Fix duplicated Chinese character "的的" → "的" in README_CN.md
  • Fix 9 Japanese locale errors (mixed Chinese/Japanese chars, typos)
  • Fix 9 Korean locale errors (misspellings and garbled characters)
  • ✅ 接受:与main直接相关的Bug修复:提交到dev分支
  • 新增功能类pr需要经过issue提前讨论,否则不会被合并
  • 🌐 i18n 提醒:除 bootstrap 或紧急修复外,请不要把非 zh-CN 目标翻译作为常规 GitHub 编辑面;常规翻译以 Crowdin -> l10n_* PR 回流为准,详见 docs/i18n.md

请填写以下内容

(删除掉中括号内的空格,并替换为小写的x

    • main 分支 禁止修改,请确认本次提交的分支 不是 main 分支
    • 我确认我阅读了贡献指南
    • 本次更新类型为:BUG修复
    • 本次更新类型为:功能新增
    • 本次更新是否经过测试
    • 如果本次修改涉及 src/A_memorix,我确认已阅读 src/A_memorix/MODIFICATION_POLICY.md,不涉及则无需勾选
  1. 请填写破坏性更新的具体内容(如有):
  2. 请简要说明本次更新的内容和目的:

其他信息

  • 关联 Issue:Close #
  • 截图/GIF
  • 附加信息:

Summary by CodeRabbit

发布说明

  • Bug Fixes

    • 修正了日文和韩文用户界面中的多处文案拼写和措辞错误
    • 修复了导入工具中的拼写问题
  • Documentation

    • 修正了中文文档中的重复字表述错误

Review Change Stack

- Fix "emoji_registed" → "emoji_registered" in mmipkg_tool.py
- Fix duplicated Chinese character "的的" → "的" in README_CN.md
- Fix 9 Japanese locale errors (mixed Chinese/Japanese chars, typos)
- Fix 9 Korean locale errors (misspellings and garbled characters)
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 30, 2026

总览

本PR包含跨多个文件的批量字错修正和本地化文案调整。更新了日文和韩文UI文案的字词错误,修正了中文文档的重复字问题,以及Python脚本中的目录路径拼写错误。

改动

本地化与拼写修正

Layer / File(s) 摘要
日文本地化文案纠错
dashboard/src/i18n/locales/ja.json
修正auth模块验证提示(verifyingLabelverifyEnter)、安全相关文案(securityTip1/2/5)、动画相关描述(disableAnimDesc/Detail/Btn)、许可证说明(licenseDesc)、重启提示(restartingDesc)和搜索插件说明(pluginsDesc)中的日文字错和用词。
韩文本地化文案纠错
dashboard/src/i18n/locales/ko.json
修正韩文拼写和措辞,涉及外观主题(accentPrimary/Hint)、动画描述(enableAnimationsDesc/disableAnimDetail)、行高(lineHeightCompact)、模糊标签(blurLabel)、侧边栏(sidebarWidthLabel/bgSidebar)和无障碍跳转文案(skipToContent)。
文档和代码拼写修正
docs/README_CN.md, scripts/mmipkg_tool.py
修正Chinese README中"其他人的的说话风格"的重复字错误,以及mmipkg_tool.py中两处默认输出目录路径从emoji_registed纠正为emoji_registered

预估代码审查工作量

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive PR描述包含了详细的修复内容列表,但缺少模板中多个必填项的勾选,且部分字段未完整填写。 建议补充填写模板中的所有勾选项(1-5项),明确说明测试状态,并完善第7项的详细说明。
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR标题清晰准确地概括了主要变更,涵盖了所有修复内容(Python、中文、日文和韩文的拼写错误)。
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@scripts/mmipkg_tool.py`:
- Line 456: 默认输出目录使用了 "data/emoji_registered",与 WebUI 的约定 EMOJI_REGISTERED_DIR /
EMOJI_DIR ("data/emoji") 不一致;请在 scripts/mmipkg_tool.py 中将涉及默认导出/输出目录的变量和参数(例如
output_dir、default_output_dir 或函数中默认参数)从 "data/emoji_registered" 改为
"data/emoji",确保所有出现此默认值的位置都统一为 "data/emoji"(包括文件内第一个赋值和任何后续默认参数或常量引用),以对齐
src/webui/routers/emoji/support.py 的目录约定并避免导入/注册遗漏。
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: efaddd9d-135d-4637-894d-a7a289fa23f0

📥 Commits

Reviewing files that changed from the base of the PR and between 864ea04 and dca192b.

📒 Files selected for processing (4)
  • dashboard/src/i18n/locales/ja.json
  • dashboard/src/i18n/locales/ko.json
  • docs/README_CN.md
  • scripts/mmipkg_tool.py

Comment thread scripts/mmipkg_tool.py
# 如果未指定输出目录,使用默认的已注册表情包目录
if output_dir is None:
output_dir = os.path.join(PROJECT_ROOT, "data", "emoji_registed")
output_dir = os.path.join(PROJECT_ROOT, "data", "emoji_registered")
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major | 🏗️ Heavy lift

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# 验证项目中注册表情包的实际存储目录

# 搜索 EMOJI_REGISTERED_DIR 或 EMOJI_DIR 的定义
echo "=== 查找表情包目录定义 ==="
rg -n "EMOJI_REGISTERED_DIR|EMOJI_DIR\s*=" --type=py

# 搜索 data/emoji_registered 的使用
echo -e "\n=== 查找 data/emoji_registered 的使用 ==="
rg -n "emoji_registered" --type=py -C2

# 搜索 data/emoji 目录的使用(排除 emoji_registered)
echo -e "\n=== 查找 data/emoji 目录的使用(排除 _registered) ==="
rg -n 'data[/\\]emoji(?!_)' --type=py -C2

Repository: Mai-with-u/MaiBot

Length of output: 1499


脚本默认输出目录与 WebUI 表情目录约定不一致:建议对齐 data/emoji

  • scripts/mmipkg_tool.py 默认导入/输出目录写为 data/emoji_registered(lines 456、999)
  • WebUI 侧表情“已注册/目录”约定为 data/emojisrc/webui/routers/emoji/support.pyEMOJI_REGISTERED_DIR = "data/emoji"EMOJI_DIR = EMOJI_REGISTERED_DIR
  • 建议将 mmipkg_tool.py 的默认 output_dir/default_output_dir 改为 data/emoji,避免后续注册/维护/清理按 data/emoji 查找导致导入结果异常或遗漏
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@scripts/mmipkg_tool.py` at line 456, 默认输出目录使用了 "data/emoji_registered",与
WebUI 的约定 EMOJI_REGISTERED_DIR / EMOJI_DIR ("data/emoji") 不一致;请在
scripts/mmipkg_tool.py 中将涉及默认导出/输出目录的变量和参数(例如 output_dir、default_output_dir
或函数中默认参数)从 "data/emoji_registered" 改为 "data/emoji",确保所有出现此默认值的位置都统一为
"data/emoji"(包括文件内第一个赋值和任何后续默认参数或常量引用),以对齐 src/webui/routers/emoji/support.py
的目录约定并避免导入/注册遗漏。

@SengokuCola SengokuCola merged commit dd2d2a8 into Mai-with-u:main May 31, 2026
1 check passed
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.

2 participants