Repository navigation
fix(research): simplify long and short timeline reading - #7556
Conversation
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
|
This pull request has merge conflicts with Choose the remote for the base repository, not an out-of-date fork. git fetch upstream
git rebase upstream/main
# Resolve each conflict, git add the resolved files, then git rebase --continue.
git push --force-with-lease origin HEADFor a same-repository clone whose Keep the DCO |
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
loopx-agent
left a comment
There was a problem hiding this comment.
Reviewer: model_agent — gpt-6.1-sol / OpenAI; runtime_reported; effort=max
Approval conclusion (author-owned PR; GitHub blocks formal self-approval)
动机 / Motivation
阅读研究历史、寻找原始问题和转向依据的用户,在窄屏上需要先越过多行重复说明才能到达研究内容。在同一份 76 条公开记录的英文 390px 页面,旧布局把问题日期入口挤到首屏之外;新布局让该入口出现在首屏,并继续提供完整帮助和原文。独立同输入对照测得英文窄屏问题区提前约 116px,阶段计数一致;桌面、窄屏的完整阅读、帮助和返回也已核对。本 PR 不改变研究事实、分析生成、评分或执行权限,也不交付整个研究工作台的最终验收。
正式 App 安装采用、真实用户理解效果和整体研究能力仍由原产品工作承接,本次源码与打包证据不替代这些结果。本次有用增量是更早触达既有问题和完整依据,并保留实际状态;不以布局变化或通过数量推断研究质量和用户理解收益。
改动思路 / Approach
现有时间线和 ContextHelp 已覆盖数据及交互责任;本次只重组展示层,减少重复说明,并用真实打包阅读路径验证,机制与问题相称。当前 PR 完成时间线默认构图的独立可回退切片;正式安装采用和整个研究工作台的余项继续归原产品工作。原 Goal 的研究入口、来源加载、typed timeline、阶段筛选和原文阅读器继续使用同一套实现,没有新增配置步骤、数据填写、确认轮次或第二个解释弹层。
默认先看前/中/后三段真实读数,再按需打开一个帮助。原“暂定不等于排队检查”“来源确认不等于独立验证或 Goal 完成”“空档不代表研究停止”等限定均完整保留。真实无记录、继续和转向仍在图中显示。只删一个标题会遗留两处帮助和柱高解释占用的多行;缩小字体则损害原文阅读,因此当前 grouping 是相称的局部修复。
具体改动 / Changes
完整审阅不可变 base 19c213324fbb5990288bd374c1b79096f22bb152 到 head 68103c282a26e85bfae67154ff105632aa64362c 的12个路径,文本 +35/-18:两个生产展示文件 +15/-10,两个既有 browser smoke +20/-8,另有8张公开 PNG。没有后端、分析生成、数据契约或持久化状态改动。
关键代码讲解
- GoalResearchTimeline 使用原 timeline props 和阶段回调;summary/key 区 把 phase cards 提到前面,把 legend、原 ContextHelp 和条件性的全部时段按钮放在同一组。findingCopy.meaning 原文移入已有帮助,未删除状态解释。
- research-timeline-summary / research-timeline-key 用 grid 间距与可换行 flex 组织;legend 允许收缩,重置按钮至少44px高。没有缩小正文或用固定位置掩盖布局问题。
- ContextHelp 保持现有悬浮、键盘焦点、点按、滚动和 Escape owner;此次复用它承载完整读图内容,不另建弹层状态。
- GoalResearchHistory 的调用 仍交付原来源、选择、日期组和阶段状态;完整原文和准确入口返回继续沿现有 reader,分页及错误恢复也未换 owner。
独立规范依据为 docs/development/design.md,版本 19c213324fbb5990288bd374c1b79096f22bb152,在读实现前核对其既有 Research Workspace、Explanations On Demand 和 Populated Page Review。三项在该 head 均有对应实现与验证:原问题/负面历史及来源完整,非必要定义按需打开,长内容与混合状态的默认/展开桌面及390px整页实际检查。docs/development/frontend-delivery.md 的真实打包后端与失败恢复要求也经下述两条原场景验证;没有用新说明反向改写验收。
对主干的风险 / Risks
主要风险是帮助合并后限定被隐藏、窄屏控制挤压,或字段虽在但原问题仍不可触达。独立检查了完整解释、可滚动窄屏帮助、44px触控区、阶段切换/全部时段、长问题原文及 Escape 返回;桌面原文16px,未重写英文证据。空态清楚显示0条历史,稀疏态保留真实零结论阶段及暂定标记。两处 source 终读均 source_unchanged=true、mutation_attempts=0,浏览未执行模型或写来源。
| 同一公开76条记录,独立 Ego 对照 | base 问题区 top | head 问题区 top | 观察 |
|---|---|---|---|
| 英文1440×900 | 614.1px | 550.1px | 约64px提前 |
| 英文390×900 | 901.2px | 785.2px | 约116px提前,日期入口移入首屏 |
另核中文桌面/390px,共8个 base/head 组合,阶段文本相等且无横向溢出。base 与 head 各用其准确构建的 assets,对照共用相同真实 HTTP/SQLite 后端和固定来源,以隔离展示变化;这不是两个后端版本的全量兼容实验。作者桌面76px来自其浏览器,我的独立读数约64px,未据此宣称同一测量。几何正确与实际像素检查分开,也不直接证明理解效果。
本地 PASS:准确 source 的 chat build 与 bundle verify;完整 LOOPX_PERSONAL_WORKSPACE_PACKAGED=1 LOOPX_PERSONAL_WORKSPACE_SCENARIO=research-history node examples/personal-workspace-browser-smoke.mjs(241.846秒);research-route-reading 同入口(13.956秒)。原场景覆盖完整正文、旧决策、引用/返回、分页、Goal/source/language隔离及断线/重试;不等于真实付费模型分析。standard premerge 的4个直接与13个选中自动检查通过,但命令自身 ok=false / exit1,原因是8项 PNG 人工 review hold;我已逐张看图并核对其公开来源及仅图像数据的 PNG metadata,没有改写原生 hold 或声称整个 merge gate 通过。
私有验证中的 root npm 路径、稀疏夹具/选择器及 CORS 读取错误均保留并用已有正确入口处理,未将其报为产品缺陷或放宽浏览器权限。base 临时服务有一次客户端离开后的 BrokenPipe,最终同源来源读回成功;两自有服务现已停止。远端 CI 未查询、未轮询、未等待。最新原生队列显示此 PR CONFLICTING / DIRTY,这是作者重基与新准确 head 验证的合并前提,当前源版本批准不会沿用到修改后的 head。
我的整体评价 / Verdict
已查看八张 PR 公开图片,并在真实 HTTP/SQLite 页面检查默认阶段读数、长问题原文和可滚动帮助的桌面及 390px 展开视图;空态与稀疏态也可辨识,原负面记录和来源仍可读取。未验证正式 App 安装采用、真实用户理解效果、整站可读性及模型或科研收益。上述视觉证据支持本切片的 UX 改善;长程状态、调度和执行权限保持,未从布局推断模型、科研或完整成本收益。
删除重复帮助和标题是已经应用的局部简化,两个展示文件即可独立回退,没有应在此引入的框架或兼容分支。当前准确 head 未发现需修改的代码问题,结论为 APPROVE;native 图片人工 hold、分支冲突和独立 merge 授权仍须分别处理,未合并、安装或替作者重基。正式产品的剩余验收继续沿原工作推进。
English verdict: APPROVE at 68103c2 for the verified timeline reading slice; branch conflicts and native merge holds remain separate.
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Goal And Delivered Outcome
研究时间线把重复的记录时间、状态帮助和柱高说明排成多行,挤走研究内容。现在先呈现阶段读数,图例和完整说明共用一个按需帮助入口;空档和负面状态仍可见。
同时接好已合入的短窗口规则:一个阅读分组占满可用行,帮助与辅助名称都准确描述“同一阅读分组”。主干一小时样本的单卡曾只占桌面348.66/1070px、390屏105.33/328px,并仍向辅助技术宣布三等分;本提案分别恢复1070/1070和328/328px。
同一76条、约17小时公开样本中,英文桌面记录密度及后续问题区提前76px,390px提前116px。依据为相同数据/视口的实际整页前后读回;这是布局观察。
main的0cfbf11dcea4ba7bc8fb13f06f1c915b63ce8fad。Author Declaration
AGENTS.mdUI Design Standard、docs/development/design.mdwhole-viewport / populated-page review、docs/development/frontend-delivery.md,按上述主干版本。ContextHelp;桌面/390前后整页Self-check: 阅读现行规范,核对完整 diff 和全部12张公开图;复用既有组件、状态色和测试。
phaseNote同时供帮助与辅助名称使用,单组仅增加局部布局规则。Scope And Continuation
这是时间线默认/帮助/分组筛选的完整可回退切片。已合入的 #7478 短窗语义、#7522 汇报布局和 #7534 同组导航均在当前组合中验证;#7551 计数文案仍是独立提案。正式 App 采用、默认生成语义及完整研究验收继续沿既有工作推进。
Validation
532e4b7937b023bb2518f9b0575e9cb7d7ba5f3c;完整旅程测于相同最终源的0cfbf11dc组合,最终提交构建和来源校验再次通过。research-history238712ms、research-route-reading20086ms;实际隔离 HTTP/SQLite,分页/断线恢复/旧锚点/完整原文/范围隔离/双语键盘及返回cqr_e48b356e02b069f77259,16路径、0 blockers;safe-fix未使用同题早期组合的238.361秒/14.799秒检查作为历史证据保留;本表采用当前组合。初次短夹具直接改时间被原事件校验拒绝,改用原始合成事件子集后通过。共享主干前移曾使质量指纹失效,已按确切基线重新准备;这些失败没有改为通过。无额外分析模型或 benchmark 运行。
Frontend / Visual Evidence
以下12图直接展示实际打包界面:76条/约17小时仓库公开样本,以及既有合成夹具的两条/一小时子集。来源为公开/合成材料,无活动上下文;可通过现有 packaged scenarios 本地复现,无共享预览服务。
Attention review: 阶段读数先读,空档图例仍可见;完整时间/状态/密度含义在原帮助内。长窗研究问题更早进入首屏,短窗单组不再留下空列。展开帮助可滚动,完整原文保持可读;没有缩字号或增加装饰卡。
Type of Change / Boundary
Signed-off-by;待精确版本独立评审及维护者合并。