Skip to content

fix: 按 PR 与运行隔离 COS 预览路径 - #40

Merged
tiye merged 1 commit into
mainfrom
codex/map-cos-pr-isolation
Oct 4, 2026
Merged

tiye merged 1 commit into
mainfrom
codex/map-cos-pr-isolation

Conversation

@tiye

@tiye tiye commented Oct 4, 2026 •

Copy link
Copy Markdown
Member

改动

原 workflow 所有 PR 共用 repository/pr/,不同预览会覆盖同一路径。本次改为 pr////,Vite base、COS prefix 与 public-base-url 继续共用同一输出。上传队列按 PR 编号隔离,生产路径和原 rsync 目标不变。

只改 workflow 与 README。沿用 COS Action v1.2.0 内置 verify,不加上传校验脚本、依赖、测试文件或 concurrently;保留现有构建门禁、测试产物复用、过期部署检查与 secrets 策略。保留已审阅 Action SHA,本次不改变供应链 pin 策略。

验证

  • YAML 解析、全部 run 块 bash 语法、git diff --check 通过。
  • 执行实际 asset-path 脚本,确认生产前缀不变,两个 PR、运行与重跑共四个预览路径互不冲突,Vite/CDN 输出一致。
  • 同 head 的 CI 37179687879 已成功完成严格依赖/类型检查、构建和 COS Action 内置公开 verify。日志确认隔离前缀 pr/40/37179687879/1/,HTML 765、CSS 318、JS 308212 字节三文件校验成功;生产服务器步骤按 PR 跳过,不代表生产或现场浏览器验收。本地未访问 COS、生产服务器或 secrets。

本 PR 不更改 Calcit 0.27.0 及模块版本,不声称完成 0.28 类型迁移或生产部署。

@tiye
tiye merged commit 646d3cd into main Oct 4, 2026
2 checks passed
@tiye
tiye deleted the codex/map-cos-pr-isolation branch October 4, 2026 05:36
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