Skip to content

perf(extensions): reduce I/O amplification in extension loading #138

Description

@HayWolf

来源

PR #136 review 评估后续跟进项

涉及的 review comments

#9 — skills.rs:333 — enable/disable skill 触发 4 次完整 load_skills 扫描

update_skill_enabled -> lookup_skill_actual_scope(2 次 load_skills)+ set_skill_enabled -> 再次 lookup_skill_actual_scope(2 次)。建议构建内存索引 HashMap<String, ConfigScope> 缓存 skill 安装范围,按需刷新。

#19 — marketplace.rs:204 — install_item 触发完整 listing + git sync

安装单个 marketplace item 走完整 listing pipeline(git clone/pull + 目录扫描 + manifest 解析)。建议添加定向查询方法,按 id 在已缓存 source 中直接查找。

#20 — marketplace.rs:311 — load_installed_plugin_records 每个 source 重复查询

marketplace_items_for_source 每个 source 都调一次 load_installed_plugin_records,但 marketplace_list_items 已经查过。建议将已构建的 HashMap 作为参数传入。

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions