Repository navigation
refactor(skill): 精简数据科学技能分发与版本维护 - #4717
Merged
Merged
Conversation
将 VChart/statplots CDN 升级到 2.2.0,以 release.json 统一人工版本入口, 通过自动生成的 snapshot.json 校验参考资源版本,避免多处修改版本号。 - 将指南、索引和 52 份变体示例收敛到包内资源,保留固定版本 JS CDN - 移除 skill 中的包元数据,将完整 manifest、测试和维护说明移至仓库维护目录 - 完善同步发布失败回滚、来源完整性和本地引用闭包检查 - 清理分发缓存与开发配置,补充仓库外隔离生成、重放及独立 HTML 回归 - 收录实施计划与验收记录 验证:28 项 Python、53 项 Node、100 项浏览器测试通过; 资源离线校验、技能结构校验及 git diff --check 通过。
skie1997
reviewed
Oct 19, 2026
skie1997
left a comment
Contributor
There was a problem hiding this comment.
已审查 339eaa7,本轮未发现需要修改的确定性代码缺陷。
重点检查了 release/snapshot/manifest 版本一致性、指南与变体本地路径、维护资源迁出后的引用、发布失败时资源/清单/controller 的恢复,以及仓库外生成和无原始 CSV 的重放。变更未涉及 VChart 渲染源码。
独立验证(PR 精确源码导出至临时目录,未修改原分支):
- Node 资源、发布回滚及绑定测试:53 / 53 通过。
- Python 测试:27 项通过,1 项 XLSX 测试因当前 Python 3.11 未安装可选 openpyxl 跳过;包含六份示例在隔离环境中复制生成、删除 CSV 后重放。
- 从真实 npm 发布物同步到独立目录成功,148 份资源、外置 manifest 与 PR 内容逐字节一致,controller 与维护源一致。
sync-visualization-skill.mjs --check离线校验通过。
本轮未独立重跑 Playwright 浏览器测试和 VChart 全量视觉回归,不将 PR 自测中的 100 / 264 项计作本轮实测。查看时部分 CI 尚在运行,合并前仍需确认最终检查结果。
skie1997
approved these changes
Oct 19, 2026
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.
💡 问题的背景与解决方案
数据科学 skill 升级图库版本时需要修改多处版本号,指南和变体示例依赖远端 Markdown/JSON,分发包还携带完整来源清单、包元数据及维护测试。此次将使用资源与仓库维护材料分离,降低升级成本,并验证完整复制到仓库外后仍可独立使用。
release.json统一 VChart/statplots 的 2.2.0 版本入口,以自动生成的snapshot.json校验参考资源版本。package.json,将完整 manifest、测试和维护说明移至packages/vchart-statplots/skill-maintenance。🤔 这个分支是...
🔗 相关 issue / PR
无。
🐞 Bugserver 用例 id
不适用。本次覆盖为仓库本地测试。
📝 Changelog
使用约束:需完整复制技能包;Python 3.11+,XLSX 可选 openpyxl;HTML 打开时仍需网络加载固定 CDN。维护端自定义资源目录必须同时显式指定外置
--manifest,不得放在资源目录中。☑️ 自测
git diff --check通过。rush test --only tag:package:4 个包共 1241 项测试通过,3 个封装包无测试操作;存在测试输出警告。6416af72b687040ac0e85aa20c13e0820802c5e9,本地提交339eaa76bcbfdec97a09d180e3cd7faf6af48982,工作区干净。详细验收记录:整体审查与独立分发验证。该记录保存技能专项验收时的结果,全量视觉检查与推送钩子的最新结果以上述 PR 自测为准。
🚀 Summary
copilot:summary
🔍 Walkthrough
copilot:walkthrough