Repository navigation
fix(skill): 恢复 #170 技能 ZIP 缓存修复供重新审阅 - #178
Closed
3316891527 wants to merge 1 commit into
Closed
3316891527 wants to merge 1 commit into
3316891527 wants to merge 1 commit into
Conversation
2 of 5 tasks
3316891527
force-pushed
the
fix/reopen-pr-170-skill-zip-cache
branch
from
November 9, 2026 03:16
bff89f6 to
56c2300
Compare
3316891527
added a commit
that referenced
this pull request
Nov 10, 2026
4 of 5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
变更说明 / Description
修复技能仓库更新后仍复用旧 ZIP 缓存、删除后重装仍命中旧内容的问题。按 GitHub commit SHA 区分 ZIP,保存安装来源缓存键,在删除技能时失效对应缓存,并保留下载进度回调。
背景与动机 / Context and motivation
维护者要求撤销 #170–#176 并恢复 open 审阅。已合并的原 PR 无法在 GitHub 重开,本 PR 对应原 #170,保持 open;未经维护者明确允许不得合并或启用自动合并。
改动范围 / Changes
完整恢复 #170 的 3 文件合并补丁,包括 SkillRepoZipPoolManager、SkillRepository 和缓存回归测试。
兼容性与风险 / Compatibility and risks
缓存命中由仓库来源与提交版本决定,删除技能会失效其安装来源缓存。当前暂以撤销 PR #177 的分支为目标,确保只展示本项差异;待 #177 获准进入 dev 后,目标应改为 dev 并重新检查,不能合入临时基准分支。
关联 Issue / Related issue
原 #170;撤销与恢复审阅要求见 #177。
验证方式 / Verification
初次重建时已逐项核对原完整合并补丁。本轮在清理历史空白并补齐 CI 的共同回滚基准上重放该项,保持独立功能差异。当前 PR 启用 ci:full-android,要求固定合并候选实际运行 assembleDebug 与 :app:testDebugUnitTest,Candidate checks 必须检查两项真实成功。新的结果以本 PR Checks 为准;历史 CI 不代替本轮构建和测试。
证据 / Evidence
原实现与历史检查见 #170。本 PR 是重新审阅入口,历史检查不等同于当前候选已通过检查。
检查清单 / Checklist
本项已随 集成 PR #191 合入
dev,并经 发布同步 PR #193 同步到main。原来源提交56c2300725a41f799ec5b8edb1de64b3d0105db5已核对保留。本 PR 原目标为
fix/revert-pr-170-176;由于dev已包含全部来源提交,GitHub 拒绝将目标改到dev。关闭此重复待合并条目,代码与原始审阅记录均保留。