Skip to content

完善共享 Skill 发现、来源关联与更新 - #3032

Closed
130040167 wants to merge 13 commits into
BigPizzaV3:mainfrom
130040167:codex/skill-shared-sources-pr
Closed

130040167 wants to merge 13 commits into
BigPizzaV3:mainfrom
130040167:codex/skill-shared-sources-pr

Conversation

@130040167

Copy link
Copy Markdown
Contributor

问题

共享目录中的 Skill 可能被误判为未启用,来源会被误记为 cc-switch;同一仓库提供给不同 Agent 的同名 Skill 也可能显示为冲突,且无法分别选择仓库路径。

修改

  • 发现共享 Skill 并按原生配置读取和切换启用状态,保留不同 Agent 的实际安装位置。
  • 合并同仓库的跨 Agent 展示,支持按独立安装位置关联仓库内的不同 Skill 路径;共用原件的安装位置共用关联。
  • 恢复仓库来源匹配,支持后台扫描、暂停和继续;关联来源与接管更新分开确认。
  • 对普通 GitHub Skill 提供原位更新预览、备份和恢复,保留独立修改的冲突保护;REA 继续手动维护。
  • 区分受控源与备份,展示备份路径和文件修改时间,并减少重复扫描和等待。

验证

  • 前端完整测试、TypeScript 检查和 Vite 构建通过(Claude Code 外部 CLI 验证按环境跳过)。
  • Rust Skill 回归 267 项通过、5 项按环境条件忽略;拓展只读接口回归 8 项通过。
  • 两个无头页面夹具通过,覆盖多 Agent 分别选择仓库路径和唯一候选默认选择。
  • Windows release 构建通过。隔离管理器的 WebView 调试端口未监听,原生页面业务断言尚未执行。

当前上游的 full_mode_launch_strips_inherited_proxy_vars 测试在 Windows 上因环境变量名称大小写断言失败,可独立复现,与本次 Skill 改动无关。

@BigPizzaV3

Copy link
Copy Markdown
Owner

感谢投入!这个 PR 有 1.2 万行(skills.rs 一个文件就 +4871 行),内容是共享 Skill 的发现、跨 Agent 合并展示、仓库来源匹配、原位更新加备份恢复、REA 更新检查,还夹带了 Impeccable 版本展示。这些都是新功能,一次性很难评审。

请先处理:

  1. Linux CI 失败是本 PR 自己的问题。 rea_update.rs:363 的测试 probe_rejects_mismatched_scope_and_reports_each_skill_version 调用了 command.env_clear(),把 PATH 清空了,所以 Linux 上找不到 node。请保留 PATH,或者改用 node 的绝对路径。(Windows 上的失败是 main 已有的问题,已经修复。)
  2. 拆分。 建议按功能拆成几个独立的 PR:① 共享 Skill 的发现与合并展示;② 来源关联;③ GitHub Skill 原位更新与备份恢复;④ REA / Impeccable。每个都附上对应的测试,我们逐个评审。

@130040167

130040167 commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor Author

感谢审查,已经开始拆分。

  • 共享 Skill 的发现、启停与展示已单独提交为 修复共享 Skill 发现与启停 #3045,并附上对应测试。
  • 拆分后的 PR 不再包含 REA 和 Impeccable 的专门处理,原 PR 中导致 Linux CI 失败的 REA 专用测试也不会进入拆分 PR。
  • 来源关联、GitHub 原位更新与备份恢复会继续拆成独立增量;来源关联本身不会接管本地文件的更新。

原 PR 先保留,待拆分完成后再处理,避免遗漏已有功能。

@130040167

Copy link
Copy Markdown
Contributor Author

已按功能拆分:

  1. 共享 Skill 发现、启停与展示:修复共享 Skill 发现与启停 #3045。
  2. 按安装位置关联仓库来源:来源关联增量。
  3. 普通 GitHub Skill 原位更新与备份恢复:更新与备份增量。

后两个 PR 依次以前一个功能分支为基准,只展示对应增量。由于当前账号无法在上游创建中间基准分支,待前置 PR 合入后会将后续增量分别提交到上游。

拆分结果不包含 REA 和 Impeccable 专用处理。原 PR 由上述增量替代,关闭以避免重复评审。

@130040167 130040167 closed this Nov 11, 2026
@130040167

Copy link
Copy Markdown
Contributor Author

来源关联增量的 Windows、macOS 和 Linux 检查现均已通过(Linux 重跑后通过)。更新与备份增量已补充备份读取错误处理并重新运行三平台检查。

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants