Skip to content

fix(skill): 恢复 #170 技能 ZIP 缓存修复供重新审阅 - #178

Closed
3316891527 wants to merge 1 commit into
fix/revert-pr-170-176from
fix/reopen-pr-170-skill-zip-cache
Closed

3316891527 wants to merge 1 commit into
fix/revert-pr-170-176from
fix/reopen-pr-170-skill-zip-cache

Conversation

@3316891527

@3316891527 3316891527 commented Nov 9, 2026 •

Copy link
Copy Markdown
Owner

变更说明 / 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

  • 已记录核验结果和未运行项原因。
  • diff 与原修复完整一致,原分支和备份保留。
  • 待 revert: 撤销 PR #170–#176,恢复合入前代码 #177 获准合入后将目标改为 dev。
  • 本轮固定候选 assembleDebug、testDebugUnitTest 与 Candidate checks 实际成功。
  • 等待维护者明确批准合并。

本项已随 集成 PR #191 合入 dev,并经 发布同步 PR #193 同步到 main。原来源提交 56c2300725a41f799ec5b8edb1de64b3d0105db5 已核对保留。

本 PR 原目标为 fix/revert-pr-170-176;由于 dev 已包含全部来源提交,GitHub 拒绝将目标改到 dev。关闭此重复待合并条目,代码与原始审阅记录均保留。

@3316891527 3316891527 added the ci:full-android Require assembleDebug and Android JVM tests for this PR candidate label Nov 9, 2026
Reapply the complete original merge diff from PR #170 on the reviewed rollback base.
Original merge: 29f6728

Keep this work open for maintainer review. No merge or auto-merge without explicit permission.
@3316891527
3316891527 force-pushed the fix/reopen-pr-170-skill-zip-cache branch from bff89f6 to 56c2300 Compare November 9, 2026 03:16
3316891527 added a commit that referenced this pull request Nov 10, 2026
chore: integrate thirteen reviewed changes into dev
@3316891527 3316891527 closed this Nov 10, 2026
@3316891527
3316891527 deleted the fix/reopen-pr-170-skill-zip-cache branch November 10, 2026 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:full-android Require assembleDebug and Android JVM tests for this PR candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant