Skip to content

refactor(skill): 精简数据科学技能分发与版本维护 - #4717

Merged
xile611 merged 1 commit into
developfrom
codex/optimize-data-science-skill
Oct 19, 2026
Merged

xile611 merged 1 commit into
developfrom
codex/optimize-data-science-skill

Conversation

@xile611

@xile611 xile611 commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

💡 问题的背景与解决方案

数据科学 skill 升级图库版本时需要修改多处版本号,指南和变体示例依赖远端 Markdown/JSON,分发包还携带完整来源清单、包元数据及维护测试。此次将使用资源与仓库维护材料分离,降低升级成本,并验证完整复制到仓库外后仍可独立使用。

  • 使用 release.json 统一 VChart/statplots 的 2.2.0 版本入口,以自动生成的 snapshot.json 校验参考资源版本。
  • 指南、类型索引及 52 份变体示例全部使用包内路径;HTML 保留两个固定版本的 JavaScript CDN。
  • 移除 skill 内的 package.json,将完整 manifest、测试和维护说明移至 packages/vchart-statplots/skill-maintenance。
  • 同步器校验发布来源、资源 hash、本地引用闭包及变体配置,并在发布失败时恢复资源、外置清单和 controller。
  • 补充整包边界与仓库外隔离回归:六份示例生成后删除原始 CSV 再重放;删除技能副本后,独立 HTML 仍可通过真实 CDN 渲染。

🤔 这个分支是...

  • 重构
  • 依赖版本更新
  • 测试 case 更新
  • 文档更新

🔗 相关 issue / PR

无。

🐞 Bugserver 用例 id

不适用。本次覆盖为仓库本地测试。

📝 Changelog

Language Changelog
English Pin the data science skill to VChart/statplots 2.2.0, localize reference documentation and variant examples, and separate repository maintenance resources from the portable skill bundle.
Chinese 数据科学 skill 使用统一的 2.2.0 版本入口和本地参考资源,分离仓库维护材料,并验证仓库外独立生成、重放及渲染。

使用约束:需完整复制技能包;Python 3.11+,XLSX 可选 openpyxl;HTML 打开时仍需网络加载固定 CDN。维护端自定义资源目录必须同时显式指定外置 --manifest,不得放在资源目录中。

☑️ 自测

  • Python:28 项通过。
  • Node 资源、发布回滚及表格绑定:53 项通过。
  • 浏览器 catalog + portable:100 项通过,包含全部 28 份最小示例、52 份变体和真实 CDN。
  • 资源离线校验、技能结构校验、108 处本地 Markdown 链接及 git diff --check 通过。
  • 推送钩子 rush test --only tag:package:4 个包共 1241 项测试通过,3 个封装包无测试操作;存在测试输出警告。
  • VChart 全量视觉基线比较:264 / 264 通过,0 差异、0 执行错误、0 未完成。官方 develop 基线 6416af72b687040ac0e85aa20c13e0820802c5e9,本地提交 339eaa76bcbfdec97a09d180e3cd7faf6af48982,工作区干净。
  • 文档已更新;示例和类型来自对应版本发布物;Changelog 已在上述表格提供。

详细验收记录:整体审查与独立分发验证。该记录保存技能专项验收时的结果,全量视觉检查与推送钩子的最新结果以上述 PR 自测为准。


🚀 Summary

copilot:summary

🔍 Walkthrough

copilot:walkthrough

将 VChart/statplots CDN 升级到 2.2.0,以 release.json 统一人工版本入口,
通过自动生成的 snapshot.json 校验参考资源版本,避免多处修改版本号。

- 将指南、索引和 52 份变体示例收敛到包内资源,保留固定版本 JS CDN
- 移除 skill 中的包元数据,将完整 manifest、测试和维护说明移至仓库维护目录
- 完善同步发布失败回滚、来源完整性和本地引用闭包检查
- 清理分发缓存与开发配置,补充仓库外隔离生成、重放及独立 HTML 回归
- 收录实施计划与验收记录

验证:28 项 Python、53 项 Node、100 项浏览器测试通过;
资源离线校验、技能结构校验及 git diff --check 通过。
@github-actions github-actions Bot added the docs label Oct 19, 2026

@skie1997 skie1997 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已审查 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 尚在运行,合并前仍需确认最终检查结果。

@xile611
xile611 merged commit 02a4215 into develop Oct 19, 2026
11 of 14 checks passed
@xile611
xile611 deleted the codex/optimize-data-science-skill branch October 19, 2026 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants