Skip to content

feat: restore the approved Omo knowledge library - #37

Open
MaxHan7 wants to merge 7 commits into
codex/continue-from-latest-mainfrom
codex/knowledge-library-search
Open

MaxHan7 wants to merge 7 commits into
codex/continue-from-latest-mainfrom
codex/knowledge-library-search

Conversation

@MaxHan7

@MaxHan7 MaxHan7 commented Aug 2, 2026

Copy link
Copy Markdown
Collaborator

结果

在团队现有主动回忆版本之上重建 Figma 884:324 知识库页面;保留现有上传、卡片详情、OmoStore、首页抽卡/刮刮乐/滑动自评和后端行为,没有回退或删除团队已有交互。

本次新增

  • 完整知识卡两列错落浏览、真实 SwiftUI 高度测量和分页页点
  • 文字搜索、清除、加载、有结果、无结果、失败和空库恢复状态
  • Apple Speech 语音输入边界,以及只在 Debug 使用的可替换合成场景
  • KnowledgeLibrarySearching 协议;Release 不使用 mock,也不伪装生产向量检索
  • Figma 返回/麦克风 SVG、权限文案、可访问性标签、Dynamic Type 与 Reduce Motion 适配
  • PRD、前端架构/布局/数据合同、素材哈希和 Simulator 截图证据

验证

  • Debug iOS Simulator build:通过
  • Release iOS Simulator build:通过
  • build-for-testing:通过
  • npm --prefix backend run docs:check:通过
  • git diff --check:通过
  • iPhone 17 Pro 与 iPhone SE 3:全部卡片、搜索结果、语音模拟、无结果、失败、空库、详情、上传、键盘与 Accessibility 字号已人工验收

截图与完整路径见 docs/knowledge-library-validation.md。

明确未验证

  • 生产 embedding/向量索引尚未接入;本 PR 只冻结可替换客户端合同
  • 真机麦克风与真实语音识别质量待真机验证
  • iOS 26.5 XCTest runner 在本机持续停在 waiting for workers to materialize;测试源码已编译,但不声明实际执行通过
  • serve-sim 鼠标拖动未生成 iOS TabView 横滑,完整 VoiceOver 首启停在系统教学弹窗;这些工具限制已如实记录

临时计划历史

  • plans/codex-knowledge-library-search.md
  • 开工:07d5047
  • 完成证据:9237eae
  • 退役:43c79e1

请使用 merge commit,不要 squash,以保留计划生命周期历史。

MaxHan7 added 7 commits August 3, 2026 00:31
Preserve the existing upload, detail, store, and active-recall flows while replacing the former system list with the approved Figma knowledge-library UI. Add complete-card browsing, adaptive measured pagination, text and voice search contracts, explicit Debug fixtures, production-safe unavailable adapter, recovery states, accessibility behavior, tests, PRD, and Simulator evidence.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants