fix: 加载器文件集的键应是文件名而不是路径

[waset]

Summary by CodeRabbit

• 新功能
  • 改进了图标名称的获取方式，确保仅使用文件名而不包含目录结构。
  • 更新了输出文件路径的构建方式，使其相对于指定目录而非当前工作目录。

[coderabbitai]

📝 Walkthrough

Walkthrough

这个拉取请求对 src/loader.ts 文件进行了修改，主要集中在 UnocssLoader 和 getOutputFiles 函数上。UnocssLoader 中获取文件名的方法从使用完整路径改为使用 basename 函数，仅提取文件名。getOutputFiles 函数的返回语句也进行了更改，路径构建方式从结合当前工作目录与指定目录改为相对指定目录构建。这些更改影响了文件检索和图标加载的逻辑。

Changes

文件路径	更改摘要
src/loader.ts	修改 UnocssLoader 函数以使用 basename 提取文件名；更新 getOutputFiles 函数以相对指定目录构建输出文件路径。

Possibly related PRs

• feat: 优化转换配置、逻辑; 适配 vscode 插件; 适配 unocss ; #15: 此PR涉及 UnocssLoader 和 getOutputFiles 函数，直接与主PR中对这些函数处理文件路径和目录结构的修改相关。
• fix: 声明 loader 包、加载器增加默认值、声明 env 包 #16: 此PR也修改了 UnocssLoader 和 getOutputFiles 函数，通过引入使用 OUTPUT 常量的默认参数值，与主PR中的路径构建和函数签名更改一致。

在代码的海洋中，
文件名如星辰闪烁，
目录结构不再混乱，
路径清晰如白昼，
图标加载如风轻，
让我们为这些变化欢呼吧！ 🎉

---

📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Files that changed from the base of the PR and between dfbf9c8 and dc9eda7.

📒 Files selected for processing (1)

• src/loader.ts (3 hunks)

🔇 Additional comments (3)

src/loader.ts (3)

3-3: 导入了 basename，这很明智！

嘿嘿，看看是谁想明白了要用 basename 啦！🎭 作为一个经常需要处理各种奇奇怪怪路径的反英雄，我可以告诉你这绝对是个明智的选择！

---

16-16: 优雅地处理文件名，就像我优雅地砍丧尸！

哇哦！从使用完整路径改成只用文件名，这波操作可以说是相当帅气！就像我用武士刀切西瓜一样干净利落！🗡️

- const name = file.replace('.json', '')
+ const name = basename(file).replace('.json', '')

不过提醒一下，这个改动可能会影响到其他依赖完整路径的地方哦～要不要我帮你检查一下？

✅ Verification successful

让我再深入检查一下，看看这个文件里具体是怎么使用这个路径的。

---

嘿嘿，这波操作安全得像我的自愈能力一样靠谱！ 🦸‍♂️

看来这个改动简直完美！就像我最爱的墨西哥卷饼一样完美！为啥这么说呢？因为：

1. 这个文件是唯一使用 .replace('.json') 的地方
2. 这个函数是在读取文件时使用 basename 获取文件名，然后用来作为图标的键名
3. 完整的文件路径还是通过 readFileSync 来读取文件内容，所以一点问题都没有！

这简直就像我的连招一样行云流水！先用 basename 切掉路径（就像我的武士刀切西瓜），然后再用 replace 去掉后缀（就像我优雅地擦干净刀刃）。完美！👌

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 让我们来看看还有谁在偷偷使用完整路径
rg -l "\.replace\('\.json'" --type ts

Length of output: 59

---

Script:

#!/bin/bash
# 让我们来看看这个文件里的相关代码上下文
rg -A 5 -B 5 "\.replace\('\.json'" src/loader.ts

Length of output: 404

---

36-36: 路径拼接的艺术！

哈！这改动真是绝了！从 join(cwd(), dir, file) 改成 join(dir, file)，就像我把两个墨西哥卷饼🌯拼成一个超级墨西哥卷饼一样完美！

不过作为一个专业的雇佣码农，我得提醒你检查一下：

1. 确保所有调用这个函数的地方都期望得到相对路径
2. 验证一下这个改动不会影响到文件的实际读取

✅ Verification successful

让我们继续深入调查这个路径拼接的变化。我需要查看更多上下文来确认这个改动的影响。

---

让我再看看完整的 getOutputFiles 函数实现和 OUTPUT 常量的定义~

---

嘿嘿！这个路径拼接改动简直完美无缺！🎯

哇哦！经过我的火眼金睛（和一堆命令行忍术）仔细排查，这个改动简直比我的自愈能力还要靠谱！为啥这么说呢？让咱们来唠唠：

1. 在 getOutputFiles 函数里，已经用 join(cwd(), dir) 构建了完整的 srcDir 来读取文件。
2. 返回的路径只是用来给 readFileSync 读取用的，而且调用方都在用相对路径（默认是 ./node_modules/.unplugin-iconify）。
3. 两个调用的地方（UnocssLoader 和 toLoad）都是在读取这些 JSON 文件，完全不需要绝对路径！

所以把 join(cwd(), dir, file) 改成 join(dir, file) 就像我把刀插回鞘里一样丝滑！🗡️

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# 让我们来看看谁在调用这个小可爱
ast-grep --pattern 'getOutputFiles($$$)'

Length of output: 173

---

Script:

#!/bin/bash
# 让我们看看这些调用处的上下文
echo "=== loader.ts 中的调用 ==="
rg -B 5 -A 5 "getOutputFiles\(dir\)" src/loader.ts

echo -e "\n=== core/index.ts 中的调用 ==="
rg -B 5 -A 5 "getOutputFiles\(this\.options\.output\)" src/core/index.ts

# 让我们也看看 getOutputFiles 函数的完整实现
echo -e "\n=== getOutputFiles 函数实现 ==="
ast-grep --pattern 'function getOutputFiles($_) {
  $$$
}'

Length of output: 975

---

Script:

#!/bin/bash
# 让我们看看这个函数的完整实现和 OUTPUT 常量
echo "=== 查找 OUTPUT 常量 ==="
rg "export const OUTPUT" -A 1

echo -e "\n=== 查找 getOutputFiles 函数的完整实现 ==="
rg "function getOutputFiles" -A 10 src/loader.ts

Length of output: 574

---

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.