style: 修复scripts目录ruff F841/B007/F401/F541问题（9个文件） #822

Merged

xiaoxia merged 1 commits from fix/ruff-f841-b007-detector into develop 2026-07-24 12:49:07 +08:00

Conversation 2 Commits 1 Files Changed 9

+16 -23

xiaoxia commented 2026-07-24 12:30:27 +08:00

Owner

Copy Link

Copy Source

变更

修复scripts目录下ruff检测到的所有问题（18处），包括：

• F841 未使用变量（6处）
• F401 未使用import（10处）
• B007 未使用循环控制变量（1处）
• F541 f-string缺少占位符（1处）

原因

PR模式Code Quality只扫描apps/目录，scripts/的ruff问题漏网；push模式全量扫描scripts/导致红灯。

涉及文件

• scripts/ci/ci_health_report.py
• scripts/ci/ci_repeated_failure_detector.py
• scripts/ci/pr_auto_scan.py
• scripts/ci_trigger_monitor.py
• scripts/init_phase4_tasks.py
• scripts/init_tracker_data.py
• scripts/init_tracker_direct.py
• scripts/smoke_public_boundary_flow.py
• scripts/smoke_public_upload_flow.py

## 变更 修复scripts目录下ruff检测到的所有问题（18处），包括： - F841 未使用变量（6处） - F401 未使用import（10处） - B007 未使用循环控制变量（1处） - F541 f-string缺少占位符（1处） ## 原因 PR模式Code Quality只扫描apps/目录，scripts/的ruff问题漏网；push模式全量扫描scripts/导致红灯。 ## 涉及文件 - scripts/ci/ci_health_report.py - scripts/ci/ci_repeated_failure_detector.py - scripts/ci/pr_auto_scan.py - scripts/ci_trigger_monitor.py - scripts/init_phase4_tasks.py - scripts/init_tracker_data.py - scripts/init_tracker_direct.py - scripts/smoke_public_boundary_flow.py - scripts/smoke_public_upload_flow.py

xiaoxia added 1 commit 2026-07-24 12:30:27 +08:00

style: 修复scripts目录ruff F841/B007/F401/F541问题 7fb3de7be7

auto-approve-bot commented 2026-07-24 12:44:39 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #822

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

1. scripts/init_tracker_direct.py 第38行：删除了 milestone1_id = cursor.lastrowid 赋值语句。

   • 问题描述：在数据库初始化脚本中，milestone1_id 用于获取刚插入的里程碑 ID，通常用于后续将任务（如 auth_tasks）关联到该里程碑。删除此变量会导致后续逻辑无法获取外键 ID，极大概率导致数据插入失败或关联丢失。
   • 后果：脚本运行时可能抛出 NameError，或者任务数据无法正确关联到里程碑。

2. scripts/smoke_public_upload_flow.py 第78行：删除了 library_id = library["id"] 赋值语句。

   • 问题描述：在冒烟测试流程中，创建资源后通常需要提取其 ID 用于后续步骤（如上传文件）。删除 library_id 变量会导致后续操作无法引用该库 ID。
   • 后果：后续依赖 library_id 的代码将报错，导致测试流程中断。

3. scripts/smoke_public_boundary_flow.py 第47行：删除了 workspace = _json_or_raise(...) 的赋值结果。

   • 问题描述：丢弃了创建 workspace 的返回值。冒烟测试通常需要验证返回数据或使用其中的 ID 进行后续边界测试。
   • 后果：如果后续代码使用了 workspace 变量，将引发 NameError，导致测试无法继续。

4. scripts/smoke_public_upload_flow.py 第50行：删除了 workspace = _json_or_raise(...) 的赋值结果。

   • 问题描述：同上，丢弃了创建 workspace 的返回值，导致后续无法使用该 workspace 的上下文信息。
   • 后果：可能导致后续测试步骤因缺少变量而失败。

💡 建议（2个可选）

1. scripts/ci_trigger_monitor.py 第103行：删除了 gitea_url 变量的赋值。

   • 建议描述：请确认 gitea_url 未在后续的 content 字典构建或消息发送逻辑中使用。如果它是用于构建跳转链接的，删除它会导致通知链接缺失。

2. scripts/pr_auto_scan.py 第11行：删除了 import os 和 import sys。

   • 建议描述：请确认脚本中不再使用 sys.exit()、os.environ 或其他来自这两个模块的方法。如果只是误判为未使用，应恢复导入。

---

✅ 格式检查通过 | ❌ 逻辑审查需修改 | ⚠️ 建议关注性能

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-24 04:44:39 | 模型:}

## 代码审查结果 - PR #822 ### ⚠️ 问题（4个需要修改） 1. **scripts/init_tracker_direct.py 第38行**：删除了 `milestone1_id = cursor.lastrowid` 赋值语句。 - **问题描述**：在数据库初始化脚本中，`milestone1_id` 用于获取刚插入的里程碑 ID，通常用于后续将任务（如 `auth_tasks`）关联到该里程碑。删除此变量会导致后续逻辑无法获取外键 ID，极大概率导致数据插入失败或关联丢失。 - **后果**：脚本运行时可能抛出 `NameError`，或者任务数据无法正确关联到里程碑。 2. **scripts/smoke_public_upload_flow.py 第78行**：删除了 `library_id = library["id"]` 赋值语句。 - **问题描述**：在冒烟测试流程中，创建资源后通常需要提取其 ID 用于后续步骤（如上传文件）。删除 `library_id` 变量会导致后续操作无法引用该库 ID。 - **后果**：后续依赖 `library_id` 的代码将报错，导致测试流程中断。 3. **scripts/smoke_public_boundary_flow.py 第47行**：删除了 `workspace = _json_or_raise(...)` 的赋值结果。 - **问题描述**：丢弃了创建 workspace 的返回值。冒烟测试通常需要验证返回数据或使用其中的 ID 进行后续边界测试。 - **后果**：如果后续代码使用了 `workspace` 变量，将引发 `NameError`，导致测试无法继续。 4. **scripts/smoke_public_upload_flow.py 第50行**：删除了 `workspace = _json_or_raise(...)` 的赋值结果。 - **问题描述**：同上，丢弃了创建 workspace 的返回值，导致后续无法使用该 workspace 的上下文信息。 - **后果**：可能导致后续测试步骤因缺少变量而失败。 ### 💡 建议（2个可选） 1. **scripts/ci_trigger_monitor.py 第103行**：删除了 `gitea_url` 变量的赋值。 - **建议描述**：请确认 `gitea_url` 未在后续的 `content` 字典构建或消息发送逻辑中使用。如果它是用于构建跳转链接的，删除它会导致通知链接缺失。 2. **scripts/pr_auto_scan.py 第11行**：删除了 `import os` 和 `import sys`。 - **建议描述**：请确认脚本中不再使用 `sys.exit()`、`os.environ` 或其他来自这两个模块的方法。如果只是误判为未使用，应恢复导入。 --- ✅ 格式检查通过 | ❌ 逻辑审查需修改 | ⚠️ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-24 04:44:39 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 627bd5b6a2 into develop 2026-07-24 12:49:07 +08:00

gitea-actions bot commented 2026-07-24 12:51:16 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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