Skip to content

docs: 修复 VChart Skill 教程图片失效 (#4700) - #4706

Merged
xile611 merged 1 commit into
developfrom
codex/fix-skill-tutorial-images-4700
Oct 14, 2026
Merged

xile611 merged 1 commit into
developfrom
codex/fix-skill-tutorial-images-4700

Conversation

@xile611

@xile611 xile611 commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

《VChart Skill 使用》中英文教程的 5 张截图引用了已失效的签名地址,官网无法显示图片。本次恢复原文同一资源 ID 的 WebP 文件,随文档静态资源发布,并同步替换双语引用。

🤔 这个分支是...

  • Bug fix
  • 网站/文档更新

🔗 相关 issue 链接

Fixes #4700

💡 问题的背景&解决方案

  • 从教程参考原文恢复 5 张截图,保留原始内容、尺寸和 WebP 字节,总计约 250 KiB。
  • 将图片纳入 docs/public/vchart/guide/skill-usage/,中英文教程共 10 处引用改为 /vchart/guide/skill-usage/*.webp,访问不再依赖到期签名。
  • 教程正文及 alt 文本保持原样。修复计划、资源 ID、文件哈希和验证结果记录在 docs/superpowers/plans/2026-10-14-fix-skill-tutorial-images.md。

📝 Changelog

Language Changelog
🇺🇸 English Fix broken images in the Chinese and English VChart Skill usage tutorials by serving screenshots from project static assets.
🇨🇳 Chinese 修复中英文 VChart Skill 使用教程的图片失效问题,截图改用项目静态资源。

☑️ 自测

  • 文档已更新。
  • Demo、Ts 类型定义及包版本变更不需要。
  • Changelog 已在 PR 中提供,文档修复不涉及发布包变更。
  • 5 个文件的 WebP 容器长度及双语引用一致性检查通过。
  • 中英文页面各 5 张图片完整解码、显示边界与真实语言切换通过;5 个静态响应均为 200 image/webp,SHA-256 与原文件一致。
  • Node.js 22.23.3 下运行 node packages/vchart/scripts/visual-test.mjs,与官方 develop 704033074a29a77ac90ed7c045d4b108af36ef05 完整比较:266 passed,0 diff,0 error,0 not_run。
  • Node.js 22.23.3 下推送钩子 rush test --only tag:package 通过:136 个测试套件、1219 个断言(VChart 96/546、Statplots 39/672、VUtils Extension 1/1)。
  • git diff --check 通过。

默认 rush docs 存在已有的包子路径别名错误(index.ts/esm/...),未将其记录为通过。本次使用仓库外临时 Vite 配置限定别名精确匹配和依赖扫描入口,完成真实 React 文档页面验收;正式配置与依赖未修改。

官网需要文档站发布后再验收,本次本地验证不代表线上已经恢复。


🚀 Summary

copilot:summary

🔍 Walkthrough

copilot:walkthrough

@github-actions github-actions Bot added the docs label Oct 14, 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.

已审查当前提交 6fff2041c1a060800791f3c3d6212bb3812b504b,未发现本次改动引入的、需要修复的问题。

本次独立核对了完整 diff,并从该提交读取 5 张 WebP 完成解码和内容检查:文件名与安装、目录、生成图表、调整样式、修复配置的用途对应;尺寸、字节数和 SHA-256 与记录一致。中英文共 10 处引用均指向存在的静态文件,正文和 alt 文本保持不变,路径沿用已有 /vchart/guide/... 资源约定,git diff --check 通过。

CI 状态仍需单独处理:

  • size 检查:build/index.min.js 为 2.45 MB,超过 2 MB 限制。该检查针对 JS 产物,本 PR 未改动其源码、依赖或构建配置。
  • BugServer 检查:返回 totalCount=2093、successCount=2050,后续脚本以退出码 1 结束。目前没有证据将此失败归因于图片引用修改,仍需维护者确认。

本次没有重跑全量图表测试或复现 PR 描述中的浏览器验收。官网是否恢复显示,需要文档与静态资源发布后再确认;上述结论不代表 CI 全绿或线上已修复。

@xile611
xile611 merged commit c5ec728 into develop Oct 14, 2026
11 of 13 checks passed
@xile611
xile611 deleted the codex/fix-skill-tutorial-images-4700 branch October 14, 2026 06:28
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.

[Bug] 官网文档的图片挂了

2 participants