xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 16 Pull Requests 12 Actions 34.2k Packages Projects Releases 7 Wiki Activity

feat(ci): add vulture dead code detection (P2 - advisory mode) #259

Closed
xiaoxia wants to merge 3 commits from feat/ci-vulture-dead-code into develop
pull from: feat/ci-vulture-dead-code
merge into: xiaoxia:develop
Conversation 0 Commits 3 Files Changed 3
+125
xiaoxia commented 2026-07-13 15:20:53 +08:00
Owner
Copy Link
Copy Source

概述

在 validate Job 中接入 vulture 死代码检测工具，P2 优先级，告警模式（不阻断 CI）。

vulture 死代码检测（P2 - 告警模式）

  • 置信度阈值：80%（逐步调高）
  • 扫描范围：alembic / apps / packages / scripts
  • 排除：测试文件、迁移文件、文档、node_modules、site-packages
  • 白名单：框架自动调用的代码（FastAPI routes、SQLAlchemy models、Celery tasks、Alembic migrations 等）
  • 模式：告警模式，不阻断合并（先摸清死代码存量，后续逐步清理）

新增文件

  • vulture.conf — vulture 配置文件
  • vulture_whitelist.py — 框架/动态调用代码白名单

修改文件

  • .gitea/workflows/ci-cd.yml — validate Job 新增 Dead code detection 步骤

后续计划

  1. 运行 1-2 周后统计死代码存量
  2. 逐步提高置信度阈值到 90%+
  3. 分批清理确认的死代码
  4. 稳定后考虑改为阻断模式
## 概述 在 validate Job 中接入 vulture 死代码检测工具，P2 优先级，告警模式（不阻断 CI）。 ### vulture 死代码检测（P2 - 告警模式） - **置信度阈值：80%**（逐步调高） - **扫描范围**：`alembic` / `apps` / `packages` / `scripts` - **排除**：测试文件、迁移文件、文档、node_modules、site-packages - **白名单**：框架自动调用的代码（FastAPI routes、SQLAlchemy models、Celery tasks、Alembic migrations 等） - **模式**：告警模式，不阻断合并（先摸清死代码存量，后续逐步清理） ## 新增文件 - `vulture.conf` — vulture 配置文件 - `vulture_whitelist.py` — 框架/动态调用代码白名单 ## 修改文件 - `.gitea/workflows/ci-cd.yml` — validate Job 新增 Dead code detection 步骤 ## 后续计划 1. 运行 1-2 周后统计死代码存量 2. 逐步提高置信度阈值到 90%+ 3. 分批清理确认的死代码 4. 稳定后考虑改为阻断模式
xiaoxia added 3 commits 2026-07-13 15:20:53 +08:00
ci: add vulture dead code detection config 0eda7977b0
ci: add vulture whitelist for framework/dynamic code 7b2acc75ed
ci: add vulture dead code detection to validate job
CI/CD Pipeline / Deploy Production (pull_request) Failing after 1560h21m48s
Details
CI/CD Pipeline / Build Production Runtime Images (pull_request) Failing after 1560h21m50s
Details
CI/CD Pipeline / Validate Code Quality And Tests (pull_request) Failing after 33s
Details
CI/CD Pipeline / Unit Tests (pull_request) Failing after 33s
Details
CI/CD Pipeline / Integration Tests (pull_request) Failing after 24s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped
Details
CI/CD Pipeline / Build & Push Staging (Watchtower auto-deploy) (pull_request) Failing after 1559h50m15s
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Failing after 1559h50m13s
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Failing after 1559h50m11s
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Failing after 1559h50m13s
Details
9d69c71d77 
- Add vulture dead code scan step (P2 - advisory mode)
- Confidence threshold: 80%
- Whitelist for framework code (FastAPI, SQLAlchemy, Celery, etc.)
- Exclude tests, migrations, scripts, docs
- Advisory only, does not block CI
xiaoxia added 1 commit 2026-07-13 15:20:54 +08:00
ci: add vulture dead code detection to validate job
CI/CD Pipeline / Deploy Production (pull_request) Failing after 1560h21m48s
Details
CI/CD Pipeline / Build Production Runtime Images (pull_request) Failing after 1560h21m50s
Details
CI/CD Pipeline / Validate Code Quality And Tests (pull_request) Failing after 33s
Details
CI/CD Pipeline / Unit Tests (pull_request) Failing after 33s
Details
CI/CD Pipeline / Integration Tests (pull_request) Failing after 24s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped
Details
CI/CD Pipeline / Build & Push Staging (Watchtower auto-deploy) (pull_request) Failing after 1559h50m15s
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Failing after 1559h50m13s
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Failing after 1559h50m11s
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Failing after 1559h50m13s
Details
9d69c71d77 
- Add vulture dead code scan step (P2 - advisory mode)
- Confidence threshold: 80%
- Whitelist for framework code (FastAPI, SQLAlchemy, Celery, etc.)
- Exclude tests, migrations, scripts, docs
- Advisory only, does not block CI
xiaoxia closed this pull request 2026-07-15 07:39:14 +08:00
Some checks are pending
CI/CD Pipeline / Deploy Production (pull_request) Failing after 1560h21m48s
Details
CI/CD Pipeline / Build Production Runtime Images (pull_request) Failing after 1560h21m50s
Details
CI/CD Pipeline / Validate Code Quality And Tests (pull_request) Failing after 33s
Details
CI/CD Pipeline / Unit Tests (pull_request) Failing after 33s
Details
CI/CD Pipeline / Integration Tests (pull_request) Failing after 24s
Details
CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped
Details
CI/CD Pipeline / Build & Push Staging (Watchtower auto-deploy) (pull_request) Failing after 1559h50m15s
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Failing after 1559h50m13s
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Failing after 1559h50m11s
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Failing after 1559h50m13s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
architecture
架构设计
 backend
后端相关
 blocked
阻塞中
 bug
Bug修复
 ci-bug
CI本身的bug
 ci-optimization
CI优化类任务
 ci-pipeline
CI流水线/workflow相关
 cleanup
代码清理
 devops
运维/部署/CI
 documentation
文档相关
 feature
新功能
 frontend
前端相关
 P0
阻塞性，必须立即处理
 P1
高优先级
 P2
中优先级
 P3
低优先级/后考虑
 question
需要讨论确认
 ready
PR CI通过，可合并
 refactor
代码重构
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
auto-approve-bot frontend-dev xiaoxia
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#259
Delete Branch "feat/ci-vulture-dead-code"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?