Skip to content

ci: 视觉对比改用 src 模式避免环境变量干扰 - #36

Merged
flycran merged 2 commits into
mainfrom
ci/visual-compare-src-mode
Nov 6, 2026
Merged

flycran merged 2 commits into
mainfrom
ci/visual-compare-src-mode

Conversation

@flycran

@flycran flycran commented Nov 6, 2026

Copy link
Copy Markdown
Owner

问题

PR #35 的视觉对比在 head 侧生成失败。dist 模式通过 @actions/core 的 INPUT_* 和生产的 run() 生成 SVG,贡献数据依赖对 gh-calendar 请求的拦截注入;一旦作业环境带有 GITHUB_TOKEN(mise-action 等会导出),新版实现会优先走官方 GraphQL 渠道,绕过拦截并拿到实时数据——轻则对比数据漂移,重则请求失败直接中断。

变更

  • visual-compare.yml 的比较步骤从 --via dist 改为 --via src
  • src 模式把 calendarList 直接透传给 gensvg,不经过 run()/INPUT_*,不受作业环境变量影响

为什么安全

  • 视觉对比的 base 始终是最新 main,calendarList 自 v3.3.3 引入,当前 main 已支持
  • 对不支持的旧 ref,工具会把网络访问变成显式失败并提示改用 --via dist,不会误报视觉差异

dist 模式经 @actions/core 的 INPUT_* 与生产 run() 生成,贡献数据依赖 fetch 拦截;作业环境里的 GITHUB_TOKEN 会让被测版本改走官方渠道,绕过拦截并拿到实时数据。src 模式直接用 calendarList 注入固定数据,不受环境影响。base 始终是最新 main,v3.3.3 起支持 calendarList,更早版本会由工具显式报错提示。
@flycran
flycran merged commit 22dbd26 into main Nov 6, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant