datawhale/whale-town-end
3
4
Fork 2
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor：将 ZulipAccountsModule 改为全局单例模块 #51

Merged
moyin merged 2 commits from docs/ai-reading-guide-20260115 into main 2026-01-15 15:00:15 +08:00
Conversation 0 Commits 2 Files Changed 4 +8 -8
moyin commented 2026-01-15 15:00:04 +08:00
Owner
Copy Link

合并请求：AI 阅读指南与代码规范优化

基本信息

项目 内容
分支名称 docs/ai-reading-guide-20260115
目标分支 main
创建日期 2026-01-15
提交数量 5

变更概述

本次合并主要包含以下改动：

  1. 文档完善：补充 AI 阅读指南中的代码质量检查规范和提交规范
  2. 代码重构：将 ZulipAccountsModule 改为全局单例模块，消除重复导入
  3. 代码规范优化：优化 auth 模块和 login_core 模块的代码规范

提交记录

提交 类型 描述
d92a078 refactor 将 ZulipAccountsModule 改为全局单例模块
cde20c6 docs 补充合并文档不纳入Git提交的规范说明
a8de256 docs 添加异常处理完整性检查规范
9f4d291 style 优化 auth 模块代码规范
4f18f0f refactor 消除代码重复，提取手机号查找为私有方法

文件变更统计

 docs/ai-reading/step3-code-quality.md         | 232 +++
 docs/ai-reading/step7-code-commit.md          |  32 +
 src/app.module.ts                             |   5 +
 src/business/admin/admin.module.ts            |   4 +-
 src/business/auth/auth.module.ts              |   3 +-
 src/business/auth/register.service.spec.ts    |   7 +-
 src/business/auth/register.service.ts         |  16 +-
 src/business/zulip/zulip.module.ts            |   4 +-
 src/core/login_core/login_core.module.spec.ts |  36 +
 src/core/login_core/login_core.service.ts     |  50 +-
 10 files changed, 353 insertions(+), 36 deletions(-)

主要改动说明

1. ZulipAccountsModule 全局化重构

  • 在 AppModule 中统一导入 ZulipAccountsModule.forRoot()
  • 移除 admin.module、auth.module、zulip.module 中的重复导入
  • 添加数据库 charset: utf8mb4 配置，支持中文和 emoji

2. 文档更新

  • step3-code-quality.md：添加异常处理完整性检查规范
  • step7-code-commit.md：补充合并文档不纳入 Git 提交的规范说明

3. 代码规范优化

  • login_core.service.ts：提取手机号查找为私有方法，消除代码重复
  • register.service.ts：优化代码结构和异常处理

测试情况

  • 单元测试通过
  • 集成测试通过
  • 本地运行验证

审核清单

  • 代码符合项目规范
  • 文档更新完整
  • 无安全风险
  • 无性能问题

备注

本次合并不包含破坏性变更，可安全合并到主分支。

# 合并请求：AI 阅读指南与代码规范优化 ## 基本信息 | 项目 | 内容 | |------|------| | 分支名称 | `docs/ai-reading-guide-20260115` | | 目标分支 | `main` | | 创建日期 | 2026-01-15 | | 提交数量 | 5 | ## 变更概述 本次合并主要包含以下改动： 1. **文档完善**：补充 AI 阅读指南中的代码质量检查规范和提交规范 2. **代码重构**：将 ZulipAccountsModule 改为全局单例模块，消除重复导入 3. **代码规范优化**：优化 auth 模块和 login_core 模块的代码规范 ## 提交记录 | 提交 | 类型 | 描述 | |------|------|------| | d92a078 | refactor | 将 ZulipAccountsModule 改为全局单例模块 | | cde20c6 | docs | 补充合并文档不纳入Git提交的规范说明 | | a8de256 | docs | 添加异常处理完整性检查规范 | | 9f4d291 | style | 优化 auth 模块代码规范 | | 4f18f0f | refactor | 消除代码重复，提取手机号查找为私有方法 | ## 文件变更统计 ``` docs/ai-reading/step3-code-quality.md | 232 +++ docs/ai-reading/step7-code-commit.md | 32 + src/app.module.ts | 5 + src/business/admin/admin.module.ts | 4 +- src/business/auth/auth.module.ts | 3 +- src/business/auth/register.service.spec.ts | 7 +- src/business/auth/register.service.ts | 16 +- src/business/zulip/zulip.module.ts | 4 +- src/core/login_core/login_core.module.spec.ts | 36 + src/core/login_core/login_core.service.ts | 50 +- 10 files changed, 353 insertions(+), 36 deletions(-) ``` ## 主要改动说明 ### 1. ZulipAccountsModule 全局化重构 - 在 `AppModule` 中统一导入 `ZulipAccountsModule.forRoot()` - 移除 `admin.module`、`auth.module`、`zulip.module` 中的重复导入 - 添加数据库 `charset: utf8mb4` 配置，支持中文和 emoji ### 2. 文档更新 - `step3-code-quality.md`：添加异常处理完整性检查规范 - `step7-code-commit.md`：补充合并文档不纳入 Git 提交的规范说明 ### 3. 代码规范优化 - `login_core.service.ts`：提取手机号查找为私有方法，消除代码重复 - `register.service.ts`：优化代码结构和异常处理 ## 测试情况 - [ ] 单元测试通过 - [ ] 集成测试通过 - [ ] 本地运行验证 ## 审核清单 - [ ] 代码符合项目规范 - [ ] 文档更新完整 - [ ] 无安全风险 - [ ] 无性能问题 ## 备注 本次合并不包含破坏性变更，可安全合并到主分支。
moyin added 1 commit 2026-01-15 15:00:04 +08:00
refactor：将 ZulipAccountsModule 改为全局单例模块 d92a078fc7 
- 在 AppModule 中统一导入 ZulipAccountsModule.forRoot()
- 移除 admin.module、auth.module、zulip.module 中的重复导入
- 添加数据库 charset: utf8mb4 配置，支持中文和 emoji
moyin added 1 commit 2026-01-15 15:00:09 +08:00
Merge branch 'main' into docs/ai-reading-guide-20260115 6e7de1a11a
moyin merged commit 01787d701c into main 2026-01-15 15:00:15 +08:00
moyin deleted branch docs/ai-reading-guide-20260115 2026-01-15 15:00:15 +08:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
moyin
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: datawhale/whale-town-end#51
Delete Branch "docs/ai-reading-guide-20260115"

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?