fix(ci): 安全/风格检查 pip install 加固——防缓存损坏导致 CI 失败 #1584

Merged

xiaoxia merged 1 commits from fix/ci-cache-robustness into develop 2026-08-31 21:07:42 +08:00

Conversation 3 Commits 1 Files Changed 2

+29 -13

xiaoxia commented 2026-08-31 20:16:58 +08:00

Owner

Copy Link

Copy Source

问题

Gitea Actions cache 返回损坏归档时（cache EOF/deserialization failed），pip install pip-audit/detect-secrets/vulture 因 hash mismatch 直接 exit 1，导致 validate-security/validate-style 整体失败。

修复

• detect-secrets/pip-audit/vulture 安装加 --no-cache-dir
• pip-audit 安装失败后 graceful skip（本身是 advisory-only）
• detect-secrets 安装失败后重试一次 --no-binary
• vulture 安装失败后 skip 死代码检测（本身是 advisory-only）

影响

• validate-security：不再因 pip cache 损坏而整体失败
• validate-style：vulture 安装失败不再阻断 CI

## 问题 Gitea Actions cache 返回损坏归档时（cache EOF/deserialization failed），pip install pip-audit/detect-secrets/vulture 因 hash mismatch 直接 exit 1，导致 validate-security/validate-style 整体失败。 ## 修复 - detect-secrets/pip-audit/vulture 安装加 `--no-cache-dir` - pip-audit 安装失败后 graceful skip（本身是 advisory-only） - detect-secrets 安装失败后重试一次 `--no-binary` - vulture 安装失败后 skip 死代码检测（本身是 advisory-only） ## 影响 - validate-security：不再因 pip cache 损坏而整体失败 - validate-style：vulture 安装失败不再阻断 CI

xiaoxia added 1 commit 2026-08-31 20:16:59 +08:00

fix(ci): 安全/风格检查 pip install 加固——防缓存损坏导致 CI 失败 ... 72f47592a9

问题：Gitea Actions cache 返回损坏归档时（cache EOF/deserialization failed），
pip install pip-audit/detect-secrets/vulture 因 hash mismatch 直接 exit 1，
导致 validate-security/validate-style 整体失败。

修复：
- detect-secrets/pip-audit/vulture 安装加 --no-cache-dir
- pip-audit 安装失败后 graceful skip（本身是 advisory-only）
- detect-secrets 安装失败后重试一次 --no-binary
- vulture 安装失败后 skip 死代码检测（本身是 advisory-only）

auto-approve-bot commented 2026-08-31 20:24:16 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #1584

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

1. scripts/ci/validate_style.sh 第27-28行：异常处理逻辑不完整。当 pip install vulture 失败时，脚本仅打印了告警信息但并未真正跳过后续执行。第28行 vulture --version 仍会执行并报错 "command not found"。虽然 set +e 不会导致脚本退出，但这会产生误导性的错误日志，违背了"skipping"的初衷。建议参照 validate_security.sh 中 pip-audit 的处理方式，增加 command -v vulture 的存在性检查。

💡 建议（1个可选）

1. scripts/ci/validate_security.sh 第86行：代码冗余。在安装失败的错误处理块中执行 pip-audit --version 2>/dev/null || true 没有实际意义（因为安装失败时该命令肯定不可用），且后续第90行已经做了正确的存在性检查。建议删除该行以保持逻辑清晰。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-08-31 12:24:16 | 模型:}

## 代码审查结果 - PR #1584 ### ⚠️ 问题（1个需要修改） 1. **scripts/ci/validate_style.sh 第27-28行**：异常处理逻辑不完整。当 `pip install vulture` 失败时，脚本仅打印了告警信息但并未真正跳过后续执行。第28行 `vulture --version` 仍会执行并报错 "command not found"。虽然 `set +e` 不会导致脚本退出，但这会产生误导性的错误日志，违背了"skipping"的初衷。建议参照 `validate_security.sh` 中 `pip-audit` 的处理方式，增加 `command -v vulture` 的存在性检查。 ### 💡 建议（1个可选） 1. **scripts/ci/validate_security.sh 第86行**：代码冗余。在安装失败的错误处理块中执行 `pip-audit --version 2>/dev/null || true` 没有实际意义（因为安装失败时该命令肯定不可用），且后续第90行已经做了正确的存在性检查。建议删除该行以保持逻辑清晰。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ✅ 性能无明显问题 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-08-31 12:24:16 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

gitea-actions bot commented 2026-08-31 20:25:17 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia merged commit 72f47592a9 into develop 2026-08-31 21:07:42 +08:00

gitea-actions bot commented 2026-08-31 21:13:51 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1584 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 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#1584