Skip to content

支持按安装位置关联 Skill 仓库来源 - #3056

Open
130040167 wants to merge 8 commits into
BigPizzaV3:mainfrom
130040167:codex/skill-source-association-review
Open

130040167 wants to merge 8 commits into
BigPizzaV3:mainfrom
130040167:codex/skill-source-association-review

Conversation

@130040167

@130040167 130040167 commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

问题

同一个 Skill 可能安装在多个 Agent 的目录中,也可能对应同一 GitHub 仓库里的不同子目录。仅按名称识别来源会把独立安装混在一起,关联仓库后也可能丢失用户已有的分组。

修改

  • 在每个实际安装位置提供来源匹配和手动关联;同仓库有多个同名 Skill 时,可分别选择仓库内路径。
  • 提供批量扫描、进度、暂停和继续;关闭详情面板后扫描继续运行,暂停时仍可使用已发现的候选。
  • 候选按内容匹配和名称线索区分;确认关联只记录来源,不移动文件,也不接管更新。
  • 只将 Skill 自身 Git 仓库的 remote 作为线索;内容不同的关联不建立更新基线,也不立即提示可更新。
  • 同仓库的跨 Agent 安装合并展示,保留各位置身份及真正的冲突;关联后迁移已有分组身份。

依赖与审核顺序

本 PR 是共享 Skill 发现与启停 #3045 的后续,该前置项已合入。普通 GitHub 原位更新及备份 #3057 依赖本项;当前 main 文件列表为本项增量,也可使用 130040167#2 查看独立差异。

验证

  • 当前提交 7318b3fc:Skill 核心并行测试 202 项通过、5 项按环境忽略;来源匹配定向测试 20 项通过。
  • 此前同一分支的 Skill 前端测试 82 项、TypeScript 检查通过;本次修复只涉及核心来源匹配。
  • 后续更新分支此前的隔离管理器在等待 WebView 调试连接时超时,尚未完成真实管理页自动验收。

@130040167
130040167 marked this pull request as ready for review November 11, 2026 20:58
@BigPizzaV3

BigPizzaV3 commented Nov 12, 2026 •

Copy link
Copy Markdown
Owner

感谢拆分!#3045 已合入,本 PR 和 main 合并无冲突,不需要 rebase,当前文件列表就是本项增量(18 个文件,+3848/-102)。本地复核:Skill 核心 200 项通过,生命周期等集成用例通过,前端测试与 TypeScript 检查通过。来源关联需要预览加确认,并校验路径身份和内容快照,不会静默改动文件,这点很好。

两处小建议,不阻塞:

  1. discover_skill_source_evidence 里的 git -C <dir> remote get-url origin 会向上找到父目录的仓库(比如 Skill 放在 dotfiles 仓库里),把无关的 remote 当成线索。可以先确认 git rev-parse --show-toplevel 就是该目录,再采用。
  2. 按名称或手动关联时,content_hash 记成本地哈希,关联后会马上显示「有更新」,下一步的原位更新就可能把本地内容换成另一个仓库的内容。建议内容不一致的关联先不标记可更新,或者在界面上明确提示「内容不同」。

CI 的运行卡在 GitHub 队列里(同一时段其他运行也一样),我们取消不掉,麻烦你推下一个提交时它会重新跑。

@130040167

Copy link
Copy Markdown
Contributor Author

感谢复核,已在 7318b3fc 处理:

  1. 读取 Git remote 前先检查仓库顶层目录与 Skill 原目录是否指向同一位置;父级 dotfiles 仓库的 remote 不再作为此 Skill 的线索。
  2. 按名称或手动关联时,只有本地内容与选定仓库内容一致才建立更新基线。内容不一致仍可关联来源,但不会立刻显示可更新;后续确认内容一致时可建立基线。关联本身仍不接管或改动文件。

补充了父级与自身仓库、名称及手动匹配、单位置及普通关联的回归。当前提交 Skill 核心并行测试 202 项通过、5 项按环境忽略,来源匹配定向测试 20 项通过。新提交已触发三平台 CI。

This branch has not been deployed

No deployments
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