refactor(#777): 修复Domain层依赖Adapters的DDD违规（4处） #789

Merged

xiaoxia merged 1 commits from refactor/fix-domain-adapters-dependency into develop 2026-07-23 23:54:56 +08:00

Conversation 2 Commits 1 Files Changed 11

+242 -63

xiaoxia commented 2026-07-23 22:46:10 +08:00

Owner

Copy Link

Copy Source

变更内容

代码质量专项P1：#777 修复Domain层依赖Adapters的DDD违规

4处违规点

1. domain/auth/email_service.py → 反向import adapters.smtp
2. domain/auth/session_store.py → 反向import adapters.redis
3. domain/auth/__init__.py → 反向import application.auth.jwt_service
4. domain/auth/__init__.py → 反向import application.auth.password_hasher

修复方案（DDD依赖倒置）

领域层定义端口（Port）：

• EmailServicePort(ABC) — 邮件服务接口
• SessionStorePort(ABC) — Session存储接口
• JWTServicePort(ABC) — JWT服务接口
• PasswordHasherPort(ABC) — 密码哈希接口
• PasswordValidatorPort(ABC) — 密码强度验证接口

外层实现端口：

• adapters.smtp.EmailService(EmailServicePort)
• adapters.redis.SessionStore(SessionStorePort)
• application.auth.JWTService(JWTServicePort)
• application.auth.PasswordHasher(PasswordHasherPort)
• application.auth.PasswordValidator(PasswordValidatorPort)

其他调整：

• EmailConfig 从adapters上移到domain层（纯值对象，无基础设施依赖）
• 清理 domain/auth/__init__.py，只导出领域端口接口
• 测试文件import路径修正（从adapters层导入具体实现）

验证

• 领域层零对外依赖（adapters/application）
• 全量单测：4383 passed, 8 skipped

## 变更内容 代码质量专项P1：#777 修复Domain层依赖Adapters的DDD违规 ### 4处违规点 1. `domain/auth/email_service.py` → 反向import adapters.smtp 2. `domain/auth/session_store.py` → 反向import adapters.redis 3. `domain/auth/__init__.py` → 反向import application.auth.jwt_service 4. `domain/auth/__init__.py` → 反向import application.auth.password_hasher ### 修复方案（DDD依赖倒置） **领域层定义端口（Port）：** - `EmailServicePort(ABC)` — 邮件服务接口 - `SessionStorePort(ABC)` — Session存储接口 - `JWTServicePort(ABC)` — JWT服务接口 - `PasswordHasherPort(ABC)` — 密码哈希接口 - `PasswordValidatorPort(ABC)` — 密码强度验证接口 **外层实现端口：** - `adapters.smtp.EmailService(EmailServicePort)` - `adapters.redis.SessionStore(SessionStorePort)` - `application.auth.JWTService(JWTServicePort)` - `application.auth.PasswordHasher(PasswordHasherPort)` - `application.auth.PasswordValidator(PasswordValidatorPort)` **其他调整：** - `EmailConfig` 从adapters上移到domain层（纯值对象，无基础设施依赖） - 清理 `domain/auth/__init__.py`，只导出领域端口接口 - 测试文件import路径修正（从adapters层导入具体实现） ### 验证 - 领域层零对外依赖（adapters/application） - 全量单测：4383 passed, 8 skipped

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

refactor(#777): 修复Domain层依赖Adapters的DDD违规（4处） ... 00baaf9e93

- 领域层定义端口接口（EmailServicePort/SessionStorePort/JWTServicePort/PasswordHasherPort）
  遵循DDD依赖倒置原则：领域层定义端口，外层实现端口
- EmailConfig从adapters上移到domain层（值对象，无基础设施依赖）
- Adapter实现类继承领域端口接口：
  Smtp EmailService(EmailServicePort)
  Redis SessionStore(SessionStorePort)
  JWTService(JWTServicePort)
  PasswordHasher(PasswordHasherPort)
- 清理domain/auth/__init__.py，移除对application/adapters的反向import
- 测试文件import路径修正，从adapters层导入具体实现
- 领域层零对外依赖（adapters/application），全量单测4383 passed

auto-approve-bot commented 2026-07-23 22:54:22 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #789

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

无

💡 建议（1个可选）

1. 类型注解风格不一致：在新增的领域层文件（email_service.py, session_store.py）中使用了内置泛型类型（如 list[dict], tuple[...]），而在 password_hasher.py 和 jwt_service.py 中使用了 typing 模块的类型（List, Dict, Tuple）。建议统一风格，考虑到项目可能需要兼容旧版本 Python 或保持一致性，建议统一使用 typing 模块中的类型（List, Dict, Tuple）。

---

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

---

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

## 代码审查结果 - PR #789 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（1个可选） 1. **类型注解风格不一致**：在新增的领域层文件（`email_service.py`, `session_store.py`）中使用了内置泛型类型（如 `list[dict]`, `tuple[...]`），而在 `password_hasher.py` 和 `jwt_service.py` 中使用了 `typing` 模块的类型（`List`, `Dict`, `Tuple`）。建议统一风格，考虑到项目可能需要兼容旧版本 Python 或保持一致性，建议统一使用 `typing` 模块中的类型（`List`, `Dict`, `Tuple`）。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ✅ 性能良好 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-23 14:54:22 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

xiaoxia merged commit d92909321c into develop 2026-07-23 23:54:56 +08:00

xiaoxia deleted branch refactor/fix-domain-adapters-dependency 2026-07-23 23:54:56 +08:00

gitea-actions bot commented 2026-07-23 23:55:04 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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