refactor: code splitting config

[xusd320]

Rename code splitting config structs, is an equivalent with before.

Summary by CodeRabbit

• Bug Fixes

  • 修正了配置类型名称中的拼写错误，确保功能正常。

• 新功能

  • 引入了新的测试模块，增强了对 URL 处理功能的验证。

• 重构

  • 多个枚举和结构体名称进行了重命名，以简化代码结构和配置机制。

[coderabbitai]

Walkthrough

该拉取请求主要涉及对多个文件中的类型和结构的重命名以及修正。transform.rs 文件中的 SimpilifyConfig 被更正为 SimplifyConfig。其他文件（如 code_splitting.rs、optimize_chunk.rs 和 ssu.rs）中，相关的枚举和结构也进行了重命名，以简化代码结构。此外，utils.rs 文件中新增了一个测试模块，用于验证 URL 处理功能。

Changes

文件路径	更改摘要
crates/mako/src/build/transform.rs	更正 SimpilifyConfig 为 SimplifyConfig。
crates/mako/src/config/code_splitting.rs, crates/mako/src/generate/optimize_chunk.rs, crates/mako/src/plugins/ssu.rs	重命名枚举和结构：OptimizeAllowChunks → AllowChunks，OptimizeChunkNameSuffixStrategy → ChunkNameSuffixStrategy，OptimizeChunkGroup → ChunkGroup，更新字段类型以使用新枚举。
crates/mako/src/utils.rs	新增测试模块 mod tests，包含测试函数 test_process_req_url。

Possibly related PRs

• fix: e2e code-splitting.complex #1392: 涉及配置相关的类型重命名，可能与主拉取请求的配置处理有关。
• feat: pass umi config mako #1394: 涉及配置处理的修改，可能与主拉取请求中的配置类型命名有关。
• refact: some confusing types and logic #1397: 包含对 ssu.rs 文件的更改，可能与整体配置和插件处理相关。
• fix(bundler-mako): config mako.plugins should work #1400: 涉及配置处理的修改，可能间接与主拉取请求相关。
• fix: not deasync small async chunk #1435: 涉及块处理和优化的更改，可能与主拉取请求的转换逻辑相关。
• fix(update): async module update miss async deps #1437: 依赖管理的更改可能与主拉取请求中的配置处理结构相关。
• refactor: avoid underscore prefix for chunk file name #1471: 块文件命名约定的更改可能是更广泛重构努力的一部分。
• perf: optimize group_chunks, speed up group_chunks by 500% #1475: 关注优化块处理，可能与主拉取请求的转换逻辑相关。
• refactor: merge source map, speed up generation by 800% #1509: 源映射处理的更改可能与主拉取请求中的配置处理相关。
• fix: merge mako config #1578: 直接修改 getMakoConfig 函数，与主拉取请求中的配置命名相关。
• Revert "fix: merge mako config" #1589: 撤销与配置合并相关的更改，可能与主拉取请求的配置类型命名有关。
• fix(bundler-mako): experimental config should be merged deeply #1617: 涉及合并实验性配置，相关于主拉取请求的配置处理。

Suggested reviewers

• stormslowly
• sorrycc

Poem

在草地上跳跃的小兔，
改名更正真是妙趣。
代码整齐如春花，
测试新增让人欢喜。
让我们一起欢庆吧，
代码更好，心情愉快！ 🐇✨

---

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?

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>, please review it.
  • Generate unit testing code for this file.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
  • @coderabbitai generate unit testing code for this file.
  • @coderabbitai modularize this function.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
  • @coderabbitai read src/utils.ts and generate unit testing code.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
  • @coderabbitai help me debug CodeRabbit configuration file.

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)

• @coderabbitai pause to pause the reviews on a PR.
• @coderabbitai resume to resume the paused reviews.
• @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
• @coderabbitai full review to do a full review from scratch and review all the files again.
• @coderabbitai summary to regenerate the summary of the PR.
• @coderabbitai resolve resolve all the CodeRabbit review comments.
• @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
• @coderabbitai help to get help.

Other keywords and placeholders

• Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
• Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
• Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• Please see the configuration documentation for more information.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[coderabbitai]

Actionable comments posted: 1

🧹 Outside diff range and nitpick comments (3)

crates/mako/src/utils.rs (1)

Line range hint 75-93: 测试模块添加得很好，建议小幅改进。

新增的测试模块对 process_req_url 函数进行了全面的测试，涵盖了多种公共路径配置。这些测试用例有助于确保函数在不同情况下的正确性。

建议：

1. 考虑添加一个测试用例，检查当请求 URL 不以公共路径开头时的行为。
2. 可以使用参数化测试来简化测试代码，提高可读性。

是否需要我为您提供参数化测试的示例代码？

crates/mako/src/generate/optimize_chunk.rs (2)

428-432: 枚举检查和块类型创建的更新是正确的

AllowChunks 枚举的更新在 apply_optimize_infos 方法中得到了正确的反映。确定块类型的逻辑变得更加简化和明确，这是一个很好的改进。

建议：考虑使用 match 语句来替代 if-else，可能会使代码更加清晰：

let info_chunk_type = match info.group_options.allow_chunks {
    AllowChunks::Async => ChunkType::Sync,
    _ => ChunkType::Entry(info_chunk_id.clone(), info.group_options.name.clone(), true)
};

这样可以使逻辑更加明确，并为将来可能的扩展提供更好的结构。

---

616-645: code_splitting_strategy_granular 函数中的类型和枚举更新正确

code_splitting_strategy_granular 函数中的结构体、字段和枚举变体的重命名是正确的。这些更改保持了代码的一致性，同时保留了原有的逻辑。

建议：考虑将 ChunkGroup 的创建逻辑抽取到单独的函数中，以提高代码的可读性和可维护性。例如：

fn create_framework_chunk_group(framework_packages: &[String]) -> ChunkGroup {
    ChunkGroup {
        name: "framework".to_string(),
        allow_chunks: AllowChunks::All,
        test: if framework_packages.is_empty() {
            Some("^$".to_string())
        } else {
            Some(format!(r#"[/\\]node_modules[/\\].*({})[/\\]"#, framework_packages.join("|")))
        },
        priority: -10,
        ..Default::default()
    }
}

这样可以使主函数更加简洁，并使每个 ChunkGroup 的创建逻辑更容易理解和维护。

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between 1d44b11 and 4e097b4.

📒 Files selected for processing (5)

• crates/mako/src/build/transform.rs (2 hunks)
• crates/mako/src/config/code_splitting.rs (4 hunks)
• crates/mako/src/generate/optimize_chunk.rs (8 hunks)
• crates/mako/src/plugins/ssu.rs (2 hunks)
• crates/mako/src/utils.rs (1 hunks)

✅ Files skipped from review due to trivial changes (1)

• crates/mako/src/build/transform.rs

🧰 Additional context used

🔇 Additional comments (11)

crates/mako/src/generate/optimize_chunk.rs (5)

10-12: 导入语句和结构定义的更新看起来不错！

这些更改与重命名的类型保持一致，提高了代码的一致性和可读性。重构后的命名更加清晰，有助于理解代码的结构和用途。

Also applies to: 21-21

---

173-173: 优化正则表达式创建方法，潜在的性能提升！

使用 create_cached_regex 替代直接创建正则表达式是一个很好的优化。这种方法可能会缓存编译后的正则表达式，从而在多次使用时提高性能。这是一个细微但有价值的改进。

---

Line range hint 332-369: 枚举变体名称更新正确

ChunkNameSuffixStrategy 枚举变体的重命名在 optimize_name_suffix 方法中得到了正确的反映。这些更改保持了代码的一致性，同时保留了原有的逻辑。重命名提高了代码的可读性，使意图更加明确。

---

514-521: check_chunk_type_allow 方法的更新正确且一致

AllowChunks 枚举的重命名在 check_chunk_type_allow 方法中得到了正确的反映。方法签名和枚举检查的更新保持了代码的一致性，同时保留了原有的逻辑。这些更改提高了代码的可读性和可维护性。

---

591-597: code_splitting_strategy_auto 函数中的结构体和字段名称更新正确

ChunkGroup 结构体及其字段的重命名在 code_splitting_strategy_auto 函数中得到了正确的反映。这些更改保持了代码的一致性，同时保留了原有的逻辑。重命名提高了代码的可读性，使代码结构更加清晰。

crates/mako/src/config/code_splitting.rs (4)

Line range hint 6-13: AllowChunks 枚举定义正确

AllowChunks 枚举添加了 Default、Eq、PartialEq 等派生属性，定义清晰明确。

---

65-71: ChunkNameSuffixStrategy 枚举定义正确

ChunkNameSuffixStrategy 枚举添加了 Eq、PartialEq 等派生属性，定义完整。

---

73-80: 更新 ChunkGroup 结构体

ChunkGroup 结构体重命名并更新了各字段类型，如将 allow_chunks 更改为使用新的 AllowChunks 枚举，使代码更简洁。

---

Line range hint 96-108: 更新 ChunkGroup 的默认实现

ChunkGroup 的 Default 实现已更新，确保新添加的字段如 allow_chunks、name_suffix 等得到正确初始化。

crates/mako/src/plugins/ssu.rs (2)

17-18: 确认类型导入的更新

导入的类型中，将 OptimizeChunkGroup 和 OptimizeAllowChunks 替换为 ChunkGroup 和 AllowChunks，确保新类型在代码中的使用正确且一致。

---

166-175: 验证 ChunkGroup 中 test 字段字符串模式的处理是否正确

将 test 字段从 Regex 更改为 String，需要确保代码中所有使用 test 字段的地方都已更新，且能正确解析和匹配字符串模式，以实现预期的文件路径匹配功能。

可以运行以下脚本来检查代码中对 test 字段的处理是否正确：

[coderabbitai]

💡 Codebase verification

⚠️ Potential issue

确认 test 字段类型更改未正确处理字符串到正则表达式的转换

在多个文件中，test 字段现在被赋值为 Option<String>，但未发现将字符串解析为正则表达式的实现逻辑。这可能导致潜在的运行时错误。

• crates/mako/src/config/code_splitting.rs
• crates/mako/src/generate/optimize_chunk.rs
• crates/mako/src/plugins/ssu.rs
• crates/mako/src/visitors/meta_url_replacer.rs
• crates/mako/src/visitors/async_module.rs
• crates/mako/src/generate/chunk_pot/util.rs

🔗 Analysis chain

注意 test 字段类型的更改

ChunkGroup 结构体中的 test 字段类型从 Option<Regex> 改为 Option<String>。请确保在代码中正确处理字符串到正则表达式的转换，以避免潜在的运行时错误。

请运行以下脚本以查找使用 test 字段的位置，验证是否已正确处理：

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 描述：查找所有使用 `test` 字段的地方，确保有字符串到正则表达式的解析逻辑。

# 测试：搜索对 `test` 字段的访问。期望：找到解析字符串为正则表达式的代码。
rg --type rust 'test\s*:' -A 5

Length of output: 4277

[codecov]

Codecov Report

Attention: Patch coverage is 2.50000% with 39 lines in your changes missing coverage. Please review.

Project coverage is 61.99%. Comparing base (1d44b11) to head (4e097b4).
Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
crates/mako/src/generate/optimize_chunk.rs	0.00%	24 Missing ⚠️
crates/mako/src/config/code_splitting.rs	0.00%	11 Missing ⚠️
crates/mako/src/plugins/ssu.rs	0.00%	4 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1629      +/-   ##
==========================================
+ Coverage   61.97%   61.99%   +0.02%     
==========================================
  Files         155      155              
  Lines       15467    15462       -5     
==========================================
  Hits         9585     9585              
+ Misses       5882     5877       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.