wangdl/ios-projects
2
0
Fork 0
Code Issues 14 Pull Requests Actions Packages Projects Releases Wiki Activity

IOS-INFO-004：V1 ReadingEventCollector 断链修复与兼容层 #112

New Issue
Closed
opened 2026-06-10 21:28:09 +08:00 by wangdl · 1 comment
wangdl commented 2026-06-10 21:28:09 +08:00
Owner
Copy Link

目标

恢复当前被注释的 ReadingEventCollector.open/close 链路，使 V1 兼容层在 V2 完成前不继续断裂。

需要处理

  1. 找到 MaterialReaderView 或相关 Reader 页面中被注释的 collector.open。
  2. 找到 close / onDisappear / deinit 中被注释或缺失的 collector.close。
  3. 恢复基础 open / close / position 采集。
  4. 避免重复打开 session。
  5. 避免页面销毁时未 close。
  6. 标记 V1 collector 为 deprecated compatibility。

验收标准

  1. 进入阅读页会触发 open。
  2. 退出阅读页会触发 close。
  3. 不会重复 open。
  4. 不会出现 close 未打开 session 崩溃。
  5. lastPosition 能被更新。
  6. V1 逻辑通过兼容层保留。
  7. 有基础生命周期测试。
## 目标 恢复当前被注释的 ReadingEventCollector.open/close 链路，使 V1 兼容层在 V2 完成前不继续断裂。 ## 需要处理 1. 找到 MaterialReaderView 或相关 Reader 页面中被注释的 collector.open。 2. 找到 close / onDisappear / deinit 中被注释或缺失的 collector.close。 3. 恢复基础 open / close / position 采集。 4. 避免重复打开 session。 5. 避免页面销毁时未 close。 6. 标记 V1 collector 为 deprecated compatibility。 ## 验收标准 1. 进入阅读页会触发 open。 2. 退出阅读页会触发 close。 3. 不会重复 open。 4. 不会出现 close 未打开 session 崩溃。 5. lastPosition 能被更新。 6. V1 逻辑通过兼容层保留。 7. 有基础生命周期测试。
wangdl added this to the M-IOS-INFO：学习信息采集、上传、继续学习与基础分析闭环 milestone 2026-06-10 21:28:09 +08:00
wangdl commented 2026-06-12 21:07:21 +08:00
Author
Owner
Copy Link

开发完成评论

完成内容

MaterialReaderView 断链修复：

  • 新增 isV2Active 属性，统一判断 V2 session 状态（.active 或 .paused）
  • openReadingSession() 添加重复 open 保护：guard !isV2Active
  • markAsRead() 改为 V2/V1 分支：V2 活跃时只调 sessionManager.markAsRead()，否则 fallback collector.markAsRead()
  • closeReadingSession() 逻辑已正确（V2 活跃 → 关 V2 return；否则关 V1）
  • updatePosition 已有正确 V2/V1 分支（line 410-414），无需修改

测试文件：

  • 新建 AIStudyAppTests/ReadingEventCollectorTests.swift（6 个测试）：
    • test_open_sets_activeMaterialId — open 产生 materialOpened 事件
    • test_close_returns_active_seconds — close 返回 activeSeconds
    • test_close_on_unopened_material_does_not_crash — 关闭未打开资料不崩溃
    • test_mark_as_read_pushes_event — markAsRead 产生 markedAsRead 事件
    • test_duplicate_open_is_safe — 重复 open 不崩溃
    • test_lastPosition_updated — updatePosition 更新 lastPosition

修改文件

  • Features/MaterialReader/MaterialReaderView.swift：+6 行（isV2Active + guard + markAsRead 分支）
  • AIStudyAppTests/ReadingEventCollectorTests.swift：新建

代码证据

  • MaterialReaderView.swift:116-118：isV2Active 属性
  • MaterialReaderView.swift:297-300：guard !isV2Active 防重复 open
  • MaterialReaderView.swift:347-353：markAsRead V2/V1 分支

验收标准逐项

  1. 进入阅读页会触发 open — ✅ onAppear → openReadingSession()
  2. 退出阅读页会触发 close — ✅ onDisappear → closeReadingSession()
  3. 不会重复 open — ✅ guard !isV2Active
  4. 不会出现 close 未打开 session 崩溃 — ✅ V1 close 安全返回 0
  5. lastPosition 能被更新 — ✅ test_lastPosition_updated
  6. V1 逻辑通过兼容层保留 — ✅ fallback 分支完整
  7. 有基础生命周期测试 — ✅ 6 个测试

是否建议进入 Review

  • 是（需 Xcode 编译验证）
## 开发完成评论 ### 完成内容 **MaterialReaderView 断链修复：** - 新增 `isV2Active` 属性，统一判断 V2 session 状态（`.active` 或 `.paused`） - `openReadingSession()` 添加重复 open 保护：`guard !isV2Active` - `markAsRead()` 改为 V2/V1 分支：V2 活跃时只调 `sessionManager.markAsRead()`，否则 fallback `collector.markAsRead()` - `closeReadingSession()` 逻辑已正确（V2 活跃 → 关 V2 return；否则关 V1） - `updatePosition` 已有正确 V2/V1 分支（line 410-414），无需修改 **测试文件：** - 新建 `AIStudyAppTests/ReadingEventCollectorTests.swift`（6 个测试）： - `test_open_sets_activeMaterialId` — open 产生 materialOpened 事件 - `test_close_returns_active_seconds` — close 返回 activeSeconds - `test_close_on_unopened_material_does_not_crash` — 关闭未打开资料不崩溃 - `test_mark_as_read_pushes_event` — markAsRead 产生 markedAsRead 事件 - `test_duplicate_open_is_safe` — 重复 open 不崩溃 - `test_lastPosition_updated` — updatePosition 更新 lastPosition ### 修改文件 - `Features/MaterialReader/MaterialReaderView.swift`：+6 行（`isV2Active` + guard + markAsRead 分支） - `AIStudyAppTests/ReadingEventCollectorTests.swift`：新建 ### 代码证据 - `MaterialReaderView.swift:116-118`：`isV2Active` 属性 - `MaterialReaderView.swift:297-300`：`guard !isV2Active` 防重复 open - `MaterialReaderView.swift:347-353`：`markAsRead` V2/V1 分支 ### 验收标准逐项 1. 进入阅读页会触发 open — ✅ `onAppear → openReadingSession()` 2. 退出阅读页会触发 close — ✅ `onDisappear → closeReadingSession()` 3. 不会重复 open — ✅ `guard !isV2Active` 4. 不会出现 close 未打开 session 崩溃 — ✅ V1 close 安全返回 0 5. lastPosition 能被更新 — ✅ `test_lastPosition_updated` 6. V1 逻辑通过兼容层保留 — ✅ fallback 分支完整 7. 有基础生命周期测试 — ✅ 6 个测试 ### 是否建议进入 Review - 是（需 Xcode 编译验证）
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area:ai
AI对话
area:analysis
分析页
area:auth
认证/Apple登录
area:import
文件导入
area:ios-info
iOS 学习信息采集与上传
area:library
知识库
area:notification
通知推送
area:profile
我的页面
area:quiz
测验/自测
area:review
复习卡片
area:search
搜索
area:study
学习首页
audit:planned
已完成宏观规划
blocked-by:api
等待 API 接口
blocked-by:api-info-core
等待 API 学习信息体系核心
blocked-by:doc-runtime
等待 Rust document runtime
priority:p0
最高优先级
priority:p1
高优先级
priority:p2
中优先级
repo:ios
iOS App 仓库
status:blocked
被阻塞
status:done
已完成
status:partial
部分存在，需补全
status:todo
待执行
type:bug
缺陷修复
type:docs
文档
type:feature
新功能
type:ui
UI/UX
work:adapter
适配器
work:analysis
分析页
work:api-client
API Client
work:api-contract
API 契约
work:architecture
架构
work:auth
认证
work:background
后台
work:bg-task
BGTask
work:compatibility
兼容性
work:context
阅读上下文
work:continue-learning
继续学习
work:debug
调试
work:design
架构设计
work:developer-tool
开发者工具
work:diagnostics
诊断
work:docs
文档
work:dto
DTO/数据传输
work:e2e
端到端
work:error-handling
错误处理
work:event
事件
work:event-chain
事件链路
work:event-export
事件导出
work:ffi
FFI 绑定
work:flush
刷新推送
work:heartbeat
心跳计时
work:heatmap
热力图
work:history
历史
work:home
首页
work:integration
接入
work:lifecycle
生命周期
work:local-cache
本地缓存
work:local-queue
本地队列
work:logging
日志
work:mapping
字段映射
work:marked-as-read
标记已读
work:material-detail
资料详情
work:model
数据模型
work:network
网络
work:offline
离线
work:permission
权限
work:persistence
持久化
work:position
阅读位置
work:queue
队列
work:reader-view
阅读器视图
work:reading-progress
阅读进度
work:records
学习记录
work:recovery
恢复
work:restore
位置恢复
work:retry
重试
work:routing
路由
work:runtime
Runtime 接入
work:rust-runtime
Rust document runtime
work:scheduler
调度器
work:session-manager
会话管理
work:state
状态管理
work:summary
Summary
work:test
测试
work:time-tracking
时长追踪
work:trend
趋势
work:ui
UI
work:upload
上传
No Label
area:ios-info
audit:planned
priority:p0
repo:ios
status:partial
status:todo
work:compatibility
work:event-chain
Milestone
No items
No Milestone M-IOS-INFO：学习信息采集、上传、继续学习与基础分析闭环
Projects
Clear projects
No project
Assignees
Clear assignees
suyun-Claude wangdl
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wangdl/ios-projects#112
No description provided.
Delete Branch "%!s()"

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?