fix(): dispose temp i18n bundles #1459
Conversation
总览这些更改主要涉及菜单配置和国际化处理的重构。在 变更
详细解析国际化处理重构此次更改的核心是引入
通过这些变更，开发团队简化了菜单国际化的处理逻辑，提高了代码的可维护性和清晰度。 Tip CodeRabbit's docstrings feature is now available as part of our Early Access Program! Simply use the command Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1459 +/- ##
=======================================
Coverage 85.42% 85.43%
=======================================
Files 524 524
Lines 15644 15650 +6
Branches 2347 2345 -2
=======================================
+ Hits 13364 13370 +6
- Misses 1829 1830 +1
+ Partials 451 450 -1
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (3)
bricks/nav/src/data-providers/get-menu-config-options.ts (1)
16-16: 文档注释中修正 I8N → I18N
此处仅涉及注释修正，可读性更好，保持与实际功能一致。bricks/nav/src/data-providers/shared/smartDisplayForMenuTitle.ts (1)
20-20: 修正文档中的 I8N → I18N
这属于文档完善，有助于新读者理解。bricks/nav/src/data-providers/get-menu-config-tree.ts (1)
78-78: 文档注释从 I8N 更正为 I18N
与整体代码中的多语言概念保持一致，减少理解歧义。
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
bricks/nav/src/data-providers/get-menu-config-options.ts(2 hunks)bricks/nav/src/data-providers/get-menu-config-tree.ts(4 hunks)bricks/nav/src/data-providers/shared/smartDisplayForMenuTitle.ts(2 hunks)
🧰 Additional context used
🪛 GitHub Check: codecov/patch
bricks/nav/src/data-providers/shared/smartDisplayForMenuTitle.ts
[warning] 130-130: bricks/nav/src/data-providers/shared/smartDisplayForMenuTitle.ts#L130
Added line #L130 was not covered by tests
🔇 Additional comments (9)
bricks/nav/src/data-providers/get-menu-config-options.ts (4)
3-6: 引入新方法提升可维护性
这些导入用以替换直接使用 i18n，令多语言逻辑集中在一个封装函数中，结构更加清晰。
23-24: 确认 dispose 调用时机
在调用 initializeMenuI18n 后，需要确保在组件或模块销毁时也会正确调用 dispose，以免资源泄漏。
26-29: 标签内容中同时包含多语言与 menuId 信息
将标题解析结果与 menu.menuId 拼接，可提升可读性和可调试性。
32-33: 及时释放多语言资源
在返回 options 前调用 dispose，遵守了资源管理规范，能有效避免内存占用。
bricks/nav/src/data-providers/shared/smartDisplayForMenuTitle.ts (2)
15-15: 新增 MenuRawData 类型引用
这一步为后续使用 menuList 提供了更准确的类型注释。
110-134: 封装多语言管理逻辑，注意测试覆盖
initializeMenuI18n 集中处理了资源绑定与释放，提升了代码可读性和可维护性。根据静态分析提示，第 130 行可能缺少测试。建议补充单测以确保覆写资源的清理逻辑被验证。
🧰 Tools
🪛 GitHub Check: codecov/patch
[warning] 130-130: bricks/nav/src/data-providers/shared/smartDisplayForMenuTitle.ts#L130
Added line #L130 was not covered by tests
bricks/nav/src/data-providers/get-menu-config-tree.ts (3)
4-7: 统一改用 initializeMenuI18n 进行多语言处理
摒弃在此处直接处理 i18n，改用封装函数后，逻辑更聚合、各函数角色更明晰。
116-116: 使用 menuWithI18n 推进多语言实例管理
在此将菜单与对应的多语言资源进行关联，简化后续调用逻辑。
159-160: 确保在函数结束时释放资源
对多语言资源做统一 dispose 处理，避免造成多余占用。
|
🚀 Deployed on https://docs-preview-1459--next-bricks.netlify.app |
|
📐🤏 Size check result (a4b8622...105c80e): error code: 502 |
依赖检查
组件之间的依赖声明，是微服务组件架构下的重要信息，请确保其正确性。
请勾选以下两组选项其中之一：
或者：
提交信息检查
Git 提交信息将决定包的版本发布及自动生成的 CHANGELOG，请检查工作内容与提交信息是否相符，并在以下每组选项中都依次确认。
破坏性变更：
feat作为提交类型。BREAKING CHANGE: 你的变更说明。新特性：
feat作为提交类型。问题修复：
fix作为提交类型。杂项工作：
即所有对下游使用者无任何影响、且没有必要显示在 CHANGELOG 中的改动，例如修改注释、测试用例、开发文档等：
chore,docs,test等作为提交类型。Summary by CodeRabbit
新功能
initializeMenuI18n函数，用于初始化菜单的国际化处理。文档
重构
getMenuConfigOptions和getMenuConfigTree函数以使用新的国际化逻辑，简化了菜单配置的国际化处理。