EchoChat

wanglei/EchoChat

Fork 0

Commit Graph

Author	SHA1	Message	Date
bujinyuan	ea2bf96c2f	fix(meeting): Task 16 修复 code-reviewer 审计 P1 八项按 docs/reviews/2026-04-23-phase2e-2-code-review.md 清单落地 P1 批次： - P1-1 主持人转让原子性：MeetingParticipantDAO.TransferHost 内部事务合并 meeting_rooms.host_id 更新，service/lifecycle 移除冗余 UpdateHost - P1-2 ListChatMessages / ListMyMeetings 回归 DAO：新增 MeetingChatDAO.ListByRoomBefore 反向游标，service 移除 s.db 直查 - P1-3 ListMyMeetings N+1 放大优化：新增 MeetingParticipantDAO.ListJoinedRoomsByUser JOIN + DISTINCT 单次 SQL - P1-4 EndRoom 行锁 + 事务：s.db.Transaction 包裹 SELECT FOR UPDATE → 快照 → MarkEnded → LeaveAllActive；DAO 补 WithTx(*gorm.DB) 和 GetByIDForUpdate - P1-5 后台 goroutine trace_id 保留：新增 logs.DetachContext(ctx)；替换 meeting_service / meeting_signal_service 共 10 处 context.Background()； meeting_ws_handler 每条 WS 消息分配独立 trace_id - P1-6 _broadcastSelfState 重试：最多 2 次指数退避（700ms→2100ms），检测 localAudioEnabled/localVideoEnabled 已被后续动作覆盖时放弃旧 patch - P1-7 HandleHostGraceExpired / HandleEmptyRoomExpired 处理锁：独立 host_grace_handling:<code> / empty_ttl_handling:<code> SETNX + 60s TTL，消除 Redis 自然过期 + 本地 timer 到点 + DEL 返回 0 的盲区 - P1-8 pushExistingRoomState 同步化：OnRoomJoin 返回前完成补推，消除与 REST member.joined 并行造成的前端状态闪烁并附带修复 .gitignore 中 logs/ 规则误伤 pkg/logs/ 代码目录的历史遗留问题，将 logger.go / trace.go 正式纳入版本控制。验证：backend go build + go vet 通过；frontend npm run build:h5 通过。 Made-with: Cursor	2026-04-23 16:53:30 +08:00
bujinyuan	d22f61fb9b	feat(phase2e-2): Go meeting 模块 DDL + Model + DAO + service/controller/router 骨架（Task 3+4） Task 3 - 数据库与持久化层： - init.sql 追加 3 张表（meeting_rooms / meeting_participants / meeting_chats）+ 9 索引 + 全量 COMMENT - 新增 phase2e2_migration.sql 增量升级脚本（IF NOT EXISTS 幂等） - app/constants/meeting.go 统一会议常量（类型/状态/角色/结束原因/离会原因/默认配置/WS 事件 8 组） - app/meeting/model 3 个 model + GORM tag（TIMESTAMP(0) 对齐现有表风格） - app/meeting/dao 3 个 DAO 共 24 个方法： * meeting_room_dao：Create/Get/Exists/MarkStarted/MarkEnded（乐观锁）/UpdateHost/UpdateSettings/ListByHost/ListExpiredForCleanup * meeting_participant_dao：JoinRoom（事务内复用旧记录）/LeaveRoom（EXTRACT(EPOCH) DB 时间）/LeaveAllActive/TransferHost（事务）/FindActiveByUser（JOIN 单点校验）/列表计数更新 * meeting_chat_dao：Create/ListByRoom（游标）/DeleteByRoomIDs/CountByRoom - psql 真库跑通 CRUD + UNIQUE + CASCADE + 主持人转让事务 + duration=10s 精确匹配 8 场景 Task 4 - Go 侧 service/controller/router/wire 骨架： - app/meeting/service/interfaces.go：NotifyPusher / UserInfoResolver / OnlineChecker 三接口（解耦 notify/contact/ws） - app/meeting/service/meeting_service.go：MeetingService + 8 sentinel error + 17 个空方法（返回 ErrNotImplemented） - app/meeting/controller/meeting_controller.go：12 个 Gin handler + responseNotImplemented（501） - app/meeting/router.go：12 条路由挂 /api/v1/meeting/* 并套 jwtAuth（扁平化 router.go，与 group/notify 一致） - app/meeting/provider.go：MeetingSet = wire.NewSet(DAO×3, Service, Controller) - 全局 wire.go 挂入 MeetingSet + 3 条 wire.Bind（Notify/UserInfo/Online）；App struct/NewApp 加字段；router/router.go 挂载 - 存量修复：admin/provider.go 补齐 MessageManage{DAO,Service,Controller} 解决旧版 wire 重生成报 no provider found - 验证：go build ./... / go vet ./... / wire ./app/provider 零错误；GIN_MODE=debug 启动打印全部 12 条路由；无 token curl 3 条代表性路由均返回 401（JWT 中间件生效）关键风格修正（偏离草案）： - 常量归入 app/constants/meeting.go 单文件（与 group.go / notify.go 同构），非草案 app/meeting/constants/ - TIMESTAMP(0) 取代草案 TIMESTAMPTZ 对齐所有现有表 - 冗余 idx_meeting_rooms_code 移除（UNIQUE 已自动建索引） - router/provider 目录扁平化（app/meeting/router.go / app/meeting/provider.go） - 延续项目 Go 侧"零 _test.go"风格，用代码审查 + psql 真库验证 + Playwright E2E 三层守护文档同步： - docs/progress/CURRENT_STATUS.md 新增 Task 3 / Task 4 交付段落 - .cursor/rules/project-context.mdc Phase 2e-2 进度推进至 Task 0-4 ✅ - docs/plans/2026-04-21-phase2e-2-implementation.plan.md Task 3 / Task 4 标记完成 + 实际产出 vs 计划差异 Made-with: Cursor	2026-04-21 16:04:57 +08:00

Author

SHA1

Message

Date

bujinyuan

ea2bf96c2f

fix(meeting): Task 16 修复 code-reviewer 审计 P1 八项

按 docs/reviews/2026-04-23-phase2e-2-code-review.md 清单落地 P1 批次：

- P1-1 主持人转让原子性：MeetingParticipantDAO.TransferHost 内部事务
  合并 meeting_rooms.host_id 更新，service/lifecycle 移除冗余 UpdateHost
- P1-2 ListChatMessages / ListMyMeetings 回归 DAO：新增
  MeetingChatDAO.ListByRoomBefore 反向游标，service 移除 s.db 直查
- P1-3 ListMyMeetings N+1 放大优化：新增
  MeetingParticipantDAO.ListJoinedRoomsByUser JOIN + DISTINCT 单次 SQL
- P1-4 EndRoom 行锁 + 事务：s.db.Transaction 包裹
  SELECT FOR UPDATE → 快照 → MarkEnded → LeaveAllActive；DAO 补
  WithTx(*gorm.DB) 和 GetByIDForUpdate
- P1-5 后台 goroutine trace_id 保留：新增 logs.DetachContext(ctx)；
  替换 meeting_service / meeting_signal_service 共 10 处 context.Background()；
  meeting_ws_handler 每条 WS 消息分配独立 trace_id
- P1-6 _broadcastSelfState 重试：最多 2 次指数退避（700ms→2100ms），
  检测 localAudioEnabled/localVideoEnabled 已被后续动作覆盖时放弃旧 patch
- P1-7 HandleHostGraceExpired / HandleEmptyRoomExpired 处理锁：
  独立 host_grace_handling:<code> / empty_ttl_handling:<code> SETNX + 60s TTL，
  消除 Redis 自然过期 + 本地 timer 到点 + DEL 返回 0 的盲区
- P1-8 pushExistingRoomState 同步化：OnRoomJoin 返回前完成补推，
  消除与 REST member.joined 并行造成的前端状态闪烁

并附带修复 .gitignore 中 logs/ 规则误伤 pkg/logs/ 代码目录的历史遗留问题，
将 logger.go / trace.go 正式纳入版本控制。

验证：backend go build + go vet 通过；frontend npm run build:h5 通过。

Made-with: Cursor

2026-04-23 16:53:30 +08:00

bujinyuan

d22f61fb9b

feat(phase2e-2): Go meeting 模块 DDL + Model + DAO + service/controller/router 骨架（Task 3+4）

Task 3 - 数据库与持久化层：
- init.sql 追加 3 张表（meeting_rooms / meeting_participants / meeting_chats）+ 9 索引 + 全量 COMMENT
- 新增 phase2e2_migration.sql 增量升级脚本（IF NOT EXISTS 幂等）
- app/constants/meeting.go 统一会议常量（类型/状态/角色/结束原因/离会原因/默认配置/WS 事件 8 组）
- app/meeting/model 3 个 model + GORM tag（TIMESTAMP(0) 对齐现有表风格）
- app/meeting/dao 3 个 DAO 共 24 个方法：
  * meeting_room_dao：Create/Get/Exists/MarkStarted/MarkEnded（乐观锁）/UpdateHost/UpdateSettings/ListByHost/ListExpiredForCleanup
  * meeting_participant_dao：JoinRoom（事务内复用旧记录）/LeaveRoom（EXTRACT(EPOCH) DB 时间）/LeaveAllActive/TransferHost（事务）/FindActiveByUser（JOIN 单点校验）/列表计数更新
  * meeting_chat_dao：Create/ListByRoom（游标）/DeleteByRoomIDs/CountByRoom
- psql 真库跑通 CRUD + UNIQUE + CASCADE + 主持人转让事务 + duration=10s 精确匹配 8 场景

Task 4 - Go 侧 service/controller/router/wire 骨架：
- app/meeting/service/interfaces.go：NotifyPusher / UserInfoResolver / OnlineChecker 三接口（解耦 notify/contact/ws）
- app/meeting/service/meeting_service.go：MeetingService + 8 sentinel error + 17 个空方法（返回 ErrNotImplemented）
- app/meeting/controller/meeting_controller.go：12 个 Gin handler + responseNotImplemented（501）
- app/meeting/router.go：12 条路由挂 /api/v1/meeting/* 并套 jwtAuth（扁平化 router.go，与 group/notify 一致）
- app/meeting/provider.go：MeetingSet = wire.NewSet(DAO×3, Service, Controller)
- 全局 wire.go 挂入 MeetingSet + 3 条 wire.Bind（Notify/UserInfo/Online）；App struct/NewApp 加字段；router/router.go 挂载
- 存量修复：admin/provider.go 补齐 MessageManage{DAO,Service,Controller} 解决旧版 wire 重生成报 no provider found
- 验证：go build ./... / go vet ./... / wire ./app/provider 零错误；GIN_MODE=debug 启动打印全部 12 条路由；无 token curl 3 条代表性路由均返回 401（JWT 中间件生效）

关键风格修正（偏离草案）：
- 常量归入 app/constants/meeting.go 单文件（与 group.go / notify.go 同构），非草案 app/meeting/constants/
- TIMESTAMP(0) 取代草案 TIMESTAMPTZ 对齐所有现有表
- 冗余 idx_meeting_rooms_code 移除（UNIQUE 已自动建索引）
- router/provider 目录扁平化（app/meeting/router.go / app/meeting/provider.go）
- 延续项目 Go 侧"零 _test.go"风格，用代码审查 + psql 真库验证 + Playwright E2E 三层守护

文档同步：
- docs/progress/CURRENT_STATUS.md 新增 Task 3 / Task 4 交付段落
- .cursor/rules/project-context.mdc Phase 2e-2 进度推进至 Task 0-4 ✅
- docs/plans/2026-04-21-phase2e-2-implementation.plan.md Task 3 / Task 4 标记完成 + 实际产出 vs 计划差异

Made-with: Cursor

2026-04-21 16:04:57 +08:00

2 Commits