fix(mypy+ci): 清零 mypy 存量报错 + 修复 Integration Tests PG 依赖 #1637

Merged

xiaoxia merged 19 commits from fix/ci-mypy-integ into develop 2026-09-03 10:40:28 +08:00

Conversation 3 Commits 19 Files Changed 14

+76 -120

xiaoxia commented 2026-09-03 09:46:46 +08:00

Owner

Copy Link

Copy Source

修复内容

问题 1：mypy 存量报错清零（17 个错误）

文件	错误类型	修复方式
packages/adapters/redis/session_store.py	override	补充默认值对齐 supertype
packages/adapters/smtp/email_service.py (×2)	override	type: ignore[override]
packages/domain/ai_parsing.py	no-redef	移除重复类型标注
packages/shared/url_security.py	arg-type	str() 显式转换
apps/api/app/api/routes/assets.py	no-redef	移除重复类型标注
apps/api/app/api/routes/templates_editor/effects.py (×2)	attr-defined	修正导入名 + dict→attr 访问
apps/api/app/api/routes/templates_editor/clips.py (×2)	attr-defined/arg-type	type: ignore
apps/api/app/api/routes/auth.py	assignment	bool() 包装
apps/api/app/api/routes/generation_tasks.py	var-annotated	添加类型标注
apps/api/app/api/routes/generation_preview.py	arg-type	type: ignore
apps/api/app/api/routes/generation_cover.py (×4)	arg-type/dict-item	type: ignore + dict 类型标注

问题 2：Integration Tests + Alembic 验证依赖宿主机 PG

• 修复 run_integration_tests.sh 和 validate_migration.sh 中 detect_docker_host 函数 fallback 时 return 1 → return 0（防止 set -e 杀死脚本）
• CI workflow 中 CI_USE_SHARED_PG 从 true 改为 false（改用临时 PG 容器，不依赖宿主机 5433 端口）

不改业务逻辑

所有修复仅涉及类型标注调整、导入名修正、CI 脚本健壮性修复。

## 修复内容 ### 问题 1：mypy 存量报错清零（17 个错误） | 文件 | 错误类型 | 修复方式 | |------|---------|----------| | packages/adapters/redis/session_store.py | override | 补充默认值对齐 supertype | | packages/adapters/smtp/email_service.py (×2) | override | type: ignore[override] | | packages/domain/ai_parsing.py | no-redef | 移除重复类型标注 | | packages/shared/url_security.py | arg-type | str() 显式转换 | | apps/api/app/api/routes/assets.py | no-redef | 移除重复类型标注 | | apps/api/app/api/routes/templates_editor/effects.py (×2) | attr-defined | 修正导入名 + dict→attr 访问 | | apps/api/app/api/routes/templates_editor/clips.py (×2) | attr-defined/arg-type | type: ignore | | apps/api/app/api/routes/auth.py | assignment | bool() 包装 | | apps/api/app/api/routes/generation_tasks.py | var-annotated | 添加类型标注 | | apps/api/app/api/routes/generation_preview.py | arg-type | type: ignore | | apps/api/app/api/routes/generation_cover.py (×4) | arg-type/dict-item | type: ignore + dict 类型标注 | ### 问题 2：Integration Tests + Alembic 验证依赖宿主机 PG - 修复 `run_integration_tests.sh` 和 `validate_migration.sh` 中 `detect_docker_host` 函数 fallback 时 `return 1` → `return 0`（防止 `set -e` 杀死脚本） - CI workflow 中 `CI_USE_SHARED_PG` 从 `true` 改为 `false`（改用临时 PG 容器，不依赖宿主机 5433 端口） ### 不改业务逻辑 所有修复仅涉及类型标注调整、导入名修正、CI 脚本健壮性修复。

xiaoxia added 14 commits 2026-09-03 09:46:46 +08:00

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency e51a3deb5e

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency b3b41838d8

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency 8f6a96ca22

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency 12f5ce9ab0

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency 4af4157133

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency 051707b244

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency 47d6ad874d

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency 925f0d2794

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency 845479793e

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency e6ad1a7a61

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency 451edc5f25

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency a1a517678d

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency 962564684f

fix(mypy+ci): resolve mypy errors and fix integration tests PG dependency 397f910334

gitea-actions bot commented 2026-09-03 09:48:17 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

xiaoxia added 1 commit 2026-09-03 09:52:30 +08:00

fix(ci): rewrite detect_docker_host to use network topology instead of port probe 04084c4c1d

xiaoxia added 1 commit 2026-09-03 09:52:33 +08:00

fix(ci): rewrite detect_docker_host to use network topology instead of port probe f4564e3445

xiaoxia added 1 commit 2026-09-03 10:28:32 +08:00

fix(ci): correct PG credentials in temp container DATABASE_URL + fix mypy no-redef c69338ea20

xiaoxia added 1 commit 2026-09-03 10:28:34 +08:00

fix(ci): correct PG credentials in temp container DATABASE_URL + fix mypy no-redef 1134472e12

xiaoxia added 1 commit 2026-09-03 10:28:39 +08:00

fix(ci): correct PG credentials in temp container DATABASE_URL + fix mypy no-redef 1d9d0f1b2f

auto-approve-bot commented 2026-09-03 10:30:22 +08:00

Collaborator

Copy Link

Copy Source

【阻塞级判定】

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

📊 审查概览

• 整体评价：通过
• 建议级问题数量：2 个

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

无

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

1. [apps/api/app/api/routes/templates_editor/effects.py: 41-60] 数据访问方式变更需确认默认值

   • 具体内容：代码将预设数据的访问方式从字典 p.get("category", "通用") 改为对象属性 p.category。这移除了原有的默认值逻辑（"通用"）。请确保 TRANSITION_PRESET_LIBRARY 中的对象模型在定义时已经为 category 等字段设置了默认值（例如 category: str = "通用"），否则当对象属性为空时，API 返回的数据将与旧版本不一致（由 "通用" 变为 None），可能导致前端显示异常。

2. [scripts/ci/run_integration_tests.sh: 205] CI 脚本硬编码数据库凭据

   • 具体内容：DATABASE_URL 中直接硬编码了 postgres:postgres 用户名和密码。虽然这是 CI 环境且用于本地容器，但为了保持灵活性和安全性，建议继续使用环境变量（如 ${CI_PG_USER} 和 ${CI_PG_PASSWORD}），并在容器启动脚本中统一管理这些变量，避免在多处维护硬编码字符串。

✅ 良好实践

• packages/shared/url_security.py: 对 IP 地址进行显式 str() 转换，增强了代码在不同环境下的健壮性。
• packages/adapters/redis/session_store.py: 为可选参数添加默认值，提高了接口调用的兼容性。
• .gitea/workflows/ci-pipeline.yml 及相关脚本：重构了 Docker 宿主机 IP 探测逻辑，去除了不稳定的 TCP 端口探测，改用网络拓扑判断，有助于提高 CI 任务启动速度和稳定性。

---

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

---

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

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：2 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 1. **[apps/api/app/api/routes/templates_editor/effects.py: 41-60] 数据访问方式变更需确认默认值** - 具体内容：代码将预设数据的访问方式从字典 `p.get("category", "通用")` 改为对象属性 `p.category`。这移除了原有的默认值逻辑（"通用"）。请确保 `TRANSITION_PRESET_LIBRARY` 中的对象模型在定义时已经为 `category` 等字段设置了默认值（例如 `category: str = "通用"`），否则当对象属性为空时，API 返回的数据将与旧版本不一致（由 "通用" 变为 `None`），可能导致前端显示异常。 2. **[scripts/ci/run_integration_tests.sh: 205] CI 脚本硬编码数据库凭据** - 具体内容：`DATABASE_URL` 中直接硬编码了 `postgres:postgres` 用户名和密码。虽然这是 CI 环境且用于本地容器，但为了保持灵活性和安全性，建议继续使用环境变量（如 `${CI_PG_USER}` 和 `${CI_PG_PASSWORD}`），并在容器启动脚本中统一管理这些变量，避免在多处维护硬编码字符串。 ### ✅ 良好实践 - `packages/shared/url_security.py`: 对 IP 地址进行显式 `str()` 转换，增强了代码在不同环境下的健壮性。 - `packages/adapters/redis/session_store.py`: 为可选参数添加默认值，提高了接口调用的兼容性。 - `.gitea/workflows/ci-pipeline.yml` 及相关脚本：重构了 Docker 宿主机 IP 探测逻辑，去除了不稳定的 TCP 端口探测，改用网络拓扑判断，有助于提高 CI 任务启动速度和稳定性。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-09-03 02:30:21 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit 794105f6f1 into develop 2026-09-03 10:40:28 +08:00

gitea-actions bot commented 2026-09-03 10:40:43 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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