fix(ci): 修复auto-merge语法bug - 逻辑抽取到独立脚本 #748

Merged

auto-approve-bot merged 5 commits from fix/auto-merge-syntax-bug into develop 2026-07-23 10:31:12 +08:00

Conversation 5 Commits 5 Files Changed 3

+297 -286

xiaoxia commented 2026-07-23 10:15:02 +08:00

Owner

Copy Link

Copy Source

问题

auto-merge workflow的bash脚本在act runner中报 syntax error near unexpected token 'fi'，本地bash -n检查无问题，疑似YAML内嵌大段bash导致act解析异常。

修复

将auto-approve和auto-merge的核心逻辑从workflow YAML中抽取为独立脚本：

• scripts/ci/auto_approve.sh
• scripts/ci/auto_merge.sh

workflow中改为单行调用，彻底规避YAML内嵌bash的转义/缩进/解析坑，也方便本地调试。

## 问题 auto-merge workflow的bash脚本在act runner中报 `syntax error near unexpected token 'fi'`，本地bash -n检查无问题，疑似YAML内嵌大段bash导致act解析异常。 ## 修复 将auto-approve和auto-merge的核心逻辑从workflow YAML中抽取为独立脚本： - `scripts/ci/auto_approve.sh` - `scripts/ci/auto_merge.sh` workflow中改为单行调用，彻底规避YAML内嵌bash的转义/缩进/解析坑，也方便本地调试。

xiaoxia added 3 commits 2026-07-23 10:15:02 +08:00

feat(ci): 抽取auto-approve逻辑到独立脚本 2cb2142255

feat(ci): 抽取auto-merge逻辑到独立脚本 d064c99213

fix(ci): auto-merge语法bug - 调用独立脚本替代内嵌大段bash daa4e80f53

xiaoxia added 1 commit 2026-07-23 10:15:04 +08:00

fix(ci): auto-merge语法bug - 调用独立脚本替代内嵌大段bash daa4e80f53

gitea-actions bot commented 2026-07-23 10:16:41 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-07-23 10:21:57 +08:00

fix(ci): 修复YAML格式 - on触发器被PyYAML转为true的bug c10eac868a

xiaoxia added 1 commit 2026-07-23 10:22:50 +08:00

fix(ci): auto-merge语法bug修复 - 抽取到独立脚本调用 ed45c25b9c

auto-approve-bot approved these changes 2026-07-23 10:29:48 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-23 10:29:48 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit 26cc6a88b7 into develop 2026-07-23 10:31:12 +08:00

auto-approve-bot deleted branch fix/auto-merge-syntax-bug 2026-07-23 10:31:12 +08:00

gitea-actions bot commented 2026-07-23 10:33:09 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

auto-approve-bot commented 2026-07-23 10:35:57 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #748

⚠️ 问题（0个需要修改）

无

💡 建议（3个可选）

1. scripts/ci/auto_approve.sh 第4-6行 & scripts/ci/auto_merge.sh 第4-6行：set -eu 重复声明

   • 两个脚本中均连续出现了两次 set -eu，这是代码重构时的复制粘贴残留，虽然不影响功能，但应删除重复行以保持整洁。

2. scripts/ci/auto_approve.sh & scripts/ci/auto_merge.sh：建议统一文件权限

   • 新增的脚本文件在 diff 中显示为 mode 100644（普通文本文件）。虽然 Workflow 中通过 bash scripts/... 调用不依赖执行权限，但作为可执行脚本，建议赋予 755 权限（chmod +x），符合惯例也方便本地直接运行测试。

3. scripts/ci/auto_approve.sh 与 scripts/ci/auto_merge.sh 逻辑一致性：BACKEND_COUNT 计算方式不统一

   • auto_approve.sh 使用 grep -cv 计算后端文件数，而 auto_merge.sh 使用 TOTAL - FRONTEND_COUNT 计算。
   • 虽然在大多数情况下结果一致，但在 FILES 为空或包含空行等边界情况下行为可能不同（例如空输入时 grep -cv 返回 1，而数学计算返回 0）。建议统一使用一种计算逻辑（推荐数学计算，更精确）。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-23 10:35:57 | 模型:}

## 代码审查结果 - PR #748 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（3个可选） 1. **scripts/ci/auto_approve.sh 第4-6行 & scripts/ci/auto_merge.sh 第4-6行**：`set -eu` 重复声明 - 两个脚本中均连续出现了两次 `set -eu`，这是代码重构时的复制粘贴残留，虽然不影响功能，但应删除重复行以保持整洁。 2. **scripts/ci/auto_approve.sh & scripts/ci/auto_merge.sh**：建议统一文件权限 - 新增的脚本文件在 diff 中显示为 `mode 100644`（普通文本文件）。虽然 Workflow 中通过 `bash scripts/...` 调用不依赖执行权限，但作为可执行脚本，建议赋予 `755` 权限（`chmod +x`），符合惯例也方便本地直接运行测试。 3. **scripts/ci/auto_approve.sh 与 scripts/ci/auto_merge.sh 逻辑一致性**：`BACKEND_COUNT` 计算方式不统一 - `auto_approve.sh` 使用 `grep -cv` 计算后端文件数，而 `auto_merge.sh` 使用 `TOTAL - FRONTEND_COUNT` 计算。 - 虽然在大多数情况下结果一致，但在 `FILES` 为空或包含空行等边界情况下行为可能不同（例如空输入时 `grep -cv` 返回 1，而数学计算返回 0）。建议统一使用一种计算逻辑（推荐数学计算，更精确）。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-23 10:35:57 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

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#748