来源
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 作为参数传入。
来源
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 作为参数传入。