fix(ci): use github.ref_name for tag detection (gitea.ref not available in runner) #1640

Merged

xiaoxia merged 1 commits from fix/ci-ref-name-tag-detection into develop 2026-09-03 13:22:51 +08:00

Conversation 3 Commits 1 Files Changed 1

+3 -3

xiaoxia commented 2026-09-03 13:11:28 +08:00

Owner

Copy Link

Copy Source

问题

PR #1639 将 github.ref 改为 gitea.ref，但 runner 日志显示 gitea.ref 同样为空，Build Production 仍然被 skip。

Runner 日志：

expression `startsWith(gitea.ref, refs/tags/v) || ...` evaluated to false

根因

Gitea Actions act runner v0.6.1 对 tag push 事件不设置 github.ref 也不设置 gitea.ref，但 github.ref_name 是可用的（在 runner 日志中可见其被正确引用）。

修复

将 startsWith(gitea.ref, refs/tags/v) 改为 startsWith(github.ref_name, v)。

tag push 时 github.ref_name = v0.1.135（不含 refs/tags/ 前缀），所以用 startsWith(..., v) 即可匹配。

## 问题 PR #1639 将 `github.ref` 改为 `gitea.ref`，但 runner 日志显示 `gitea.ref` 同样为空，Build Production 仍然被 skip。 Runner 日志： ``` expression `startsWith(gitea.ref, refs/tags/v) || ...` evaluated to false ``` ## 根因 Gitea Actions act runner v0.6.1 对 tag push 事件不设置 `github.ref` 也不设置 `gitea.ref`，但 `github.ref_name` 是可用的（在 runner 日志中可见其被正确引用）。 ## 修复 将 `startsWith(gitea.ref, refs/tags/v)` 改为 `startsWith(github.ref_name, v)`。 tag push 时 `github.ref_name` = `v0.1.135`（不含 `refs/tags/` 前缀），所以用 `startsWith(..., v)` 即可匹配。

xiaoxia added 1 commit 2026-09-03 13:11:29 +08:00

fix(ci): use github.ref_name instead of gitea.ref for tag detection 7da3e9ce47

auto-approve-bot commented 2026-09-03 13:12:55 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

• 是否存在阻塞级问题：是
• 阻塞级问题数量：1 个

📊 审查概览

• 整体评价：需修改
• 建议级问题数量：1 个

🔴 阻塞级问题（必须修复）

1. [.gitea/workflows/ci-pipeline.yml: 1465, 1615, 1747] 条件判断逻辑变更导致分支误触发
   • 问题类型：逻辑bug
   • 问题描述：原逻辑 startsWith(gitea.ref, 'refs/tags/v') 明确限定了引用必须以 refs/tags/v 开头，即只匹配以 v 开头的 Tag。修改后使用 github.ref_name（短引用名）并仅判断是否以 v 开头，丢失了对引用类型的校验。这会导致名为 verify-fix、version-update 等以 v 开头的分支也会触发生产构建和部署，造成严重的误操作和资源浪费。
   • 修改建议：应增加对引用类型的判断，例如恢复使用 startsWith(github.ref, 'refs/tags/v')（使用完整引用），或修改为 (startsWith(github.ref_name, 'v') && github.ref_type == 'tag')。

💡 改进建议（不阻塞合并）

1. [.gitea/workflows/ci-pipeline.yml: 全局] 上下文变量混用风险
   • 具体内容：文件位于 .gitea/workflows 目录下，但代码中混用了 gitea（如 concurrency.group 中）和 github（如 if 条件中）上下文变量。虽然 Gitea Actions 兼容 GitHub 语法，但建议统一使用 gitea 上下文变量（如 gitea.ref_name, gitea.event_name）以确保在 Gitea 环境下的最佳兼容性和明确性，除非该工作流需要同时在 GitHub 上运行。

✅ 良好实践

无

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-09-03 05:12:55 | 模型:}

### 【阻塞级判定】 - 是否存在阻塞级问题：是 - 阻塞级问题数量：1 个 ### 📊 审查概览 - 整体评价：需修改 - 建议级问题数量：1 个 ### 🔴 阻塞级问题（必须修复） 1. **[.gitea/workflows/ci-pipeline.yml: 1465, 1615, 1747] 条件判断逻辑变更导致分支误触发** - 问题类型：逻辑bug - 问题描述：原逻辑 `startsWith(gitea.ref, 'refs/tags/v')` 明确限定了引用必须以 `refs/tags/v` 开头，即只匹配以 `v` 开头的 Tag。修改后使用 `github.ref_name`（短引用名）并仅判断是否以 `v` 开头，丢失了对引用类型的校验。这会导致名为 `verify-fix`、`version-update` 等以 `v` 开头的分支也会触发生产构建和部署，造成严重的误操作和资源浪费。 - 修改建议：应增加对引用类型的判断，例如恢复使用 `startsWith(github.ref, 'refs/tags/v')`（使用完整引用），或修改为 `(startsWith(github.ref_name, 'v') && github.ref_type == 'tag')`。 ### 💡 改进建议（不阻塞合并） 1. **[.gitea/workflows/ci-pipeline.yml: 全局] 上下文变量混用风险** - 具体内容：文件位于 `.gitea/workflows` 目录下，但代码中混用了 `gitea`（如 `concurrency.group` 中）和 `github`（如 `if` 条件中）上下文变量。虽然 Gitea Actions 兼容 GitHub 语法，但建议统一使用 `gitea` 上下文变量（如 `gitea.ref_name`, `gitea.event_name`）以确保在 Gitea 环境下的最佳兼容性和明确性，除非该工作流需要同时在 GitHub 上运行。 ### ✅ 良好实践 无 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-09-03 05:12:55 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

gitea-actions bot commented 2026-09-03 13:14:22 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

项目	详情
PR号	#1640
预览链接	https://pr-1640.preview.xiaoxiajianji.com
API环境	staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1640 | | 预览链接 | [https://pr-1640.preview.xiaoxiajianji.com](https://pr-1640.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

xiaoxia merged commit 4290f7d19f into develop 2026-09-03 13:22:51 +08:00

xiaoxia deleted branch fix/ci-ref-name-tag-detection 2026-09-03 13:22:51 +08:00

gitea-actions bot commented 2026-09-03 13:23:05 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #1640 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #1640 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1640