Repository navigation
Show source research history before an empty analysis - #8654
loopx-agent wants to merge 3 commits into
Conversation
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
loopx-agent
left a comment
There was a problem hiding this comment.
Request changes conclusion (author-owned PR; GitHub blocks formal self-review)
Reviewer: model_agent · gpt-6.1-sol · OpenAI · runtime_reported · max
本次评审对应提交 a5dd7c362ab410714fdab25cd3735ed161fa6e1a。
[P2] 切换分析范围后丢失键盘焦点。 同一只读来源中,两个范围互切会使焦点离开下拉框;继续按 Tab 会跳到时间线前段。
动机 / Motivation
在 Dashboard Research 页查阅已有来源记录和分析范围的用户。在具名研究来源中,已有历程但没有 saved verdict 或 native outcome 时,原来先看到空的分析摘要。64 条记录全部保留。1440 × 900 和 390 × 900 下,时间轴分别从 748 到 611 px、1026 到 765 px;窄屏父级标题为单行,完整名称仍可访问。本地打包 Chat、合成研究与冻结分析回执使用真实只读来源及历史 HTTP 路径;已安装 WebView 与实际归档未验证。在同一来源的两个分析范围间切换时,候选会把焦点移到页面正文。随后按 Tab 跳到时间线前段;主干保留原下拉框焦点,并继续到分析指标。
改动思路 / Approach
继续用 Research 原有的历史、结果和范围控件。只重排原内容;需要修复条件重挂载带来的焦点丢失。 无结果时让真实历程先回答发生了什么;已保存结果维持原来的优先级。这里不会触发分析或改动来源。
具体改动 / Changes
关键代码讲解
- 来源顺序:根据具名来源、已加载节点和 result/native_outcomes 排列。
- 焦点问题:同一个摘要在两个 JSX 位置之间切换,原下拉框被卸载。
- 父级标题:390px 下从 76px 降到 44px,完整无障碍名称保留。Enter 进入父 Goal 的 Research,返回仍可读子 Goal 记录。
对照 docs/development/design.md 的 708c68e:Research-order 已满足,Keyboard-continuity 未满足。
对主干的风险 / Risks
同一份 64 条合成记录的 首屏对照 与本地 1440px/390px 检查都显示时间轴提前。具名来源范围双向切换均丢焦点,普通来源保留。展开仍可读全文;刷新失败保留 64/64,Enter 重试成功,零来源写入。
语义与 CI 对齐
TypeScript、CSS tokens、4 项直接和 2 项选定 premerge 检查通过。打包范围控件负例未通过。
我的整体评价 / Verdict
REQUEST_CHANGES。保留现有首屏收益,修复范围切换后焦点丢失。已目视检查宽、窄首屏和窄屏展开与错误状态。时间线确实提前,分析仍可达;切换范围时焦点丢失破坏了键盘连续阅读。已安装 WebView 和实际归档尚未核验。
English verdict: REQUEST_CHANGES. The timeline moves earlier at a5dd7c3, but switching between empty and saved scopes drops select focus and sends Tab to the timeline; exact-base control retains focus.
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
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 · max
Approval conclusion (author-owned PR; GitHub blocks formal self-approval)
本次评审对应提交 d1803a3c00579de998288cc6c0827e0e4ab8aba3,目标主干为 31ff0569843390413330e67be67f570b2d03650b。
动机 / Motivation
读者打开有原始记录的实验子 Goal,想先看整段研究经历,再决定要核对哪一轮的分析。 主干把空分析摘要放在时间线前面,窄屏读者要先滚动才能看到全程。上一提交把图提前了,切换到有结果的范围时却丢了键盘焦点。此提交保留首屏时间轴,双向切换后焦点都留在原下拉框。 源内已有记录可在首屏开始读,已有判读的范围仍先显示结果,继续按 Tab 会进入当时可见的指标。 这次交付覆盖研究历程的排序、焦点和父级入口。分析计算、历史来源、原生分数和来源权限由原有组件继续处理。 已验证打包浏览器。已安装工作台的历史读取程序仍需修复,原生桌面交付尚待核验。
改动思路 / Approach
研究页根据同一来源当前的判读和原生测量决定内容顺序,并移动原来那一个结果组件。用稳定 React key 保留控件身份,视觉顺序、DOM 顺序与 Tab 顺序一致。 这次改动集中在两个原有展示组件、一条窄屏布局规则和已有只读浏览器回归。 分析切换仍由现有的范围选择和读取控制器处理。先让时间轴出现可以减少一次滚动。把两块内容只用 CSS 改顺序会留下两套阅读顺序,键盘仍可能跳回视觉上更早的位置。现有组件用一个稳定标识即可在同一父节点内移动,避免额外的焦点状态。
具体改动 / Changes
关键代码讲解
whole-recorded-interval 已落地。研究页从现有来源读取覆盖,归档分三页读回 86 条记录和 28 个决策标记。打开已保存结果的范围时,结果继续排在时间线前。 scoped-keyboard-reading 已落地。稳定的摘要节点在历史前后移动时,两种宽度都保留原下拉框和键盘焦点。 full-scientific-adoption 留在既有研究验收,本次对记录和显示作验证。 研究工作台约定
GoalResearchHistory 只根据当前来源是否有节点、分析包有无结果或原生测量来排顺序。结果摘要和历史留在同一个父节点的列表里,由固定 key 保留身份。父级入口使用原来的 Goal 选择,窄屏省略视觉文字并保留完整名称。两种宽度按 Enter 回到父 Goal 后,浏览器后退都读回子 Goal 的原 64 条记录。只读浏览器回归补了双向范围切换并核验下一次 Tab,历史仍从真实 HTTP 读回。 内容排序 节点保留 父级入口 布局 浏览器回归
对主干的风险 / Risks
在前一提交 a5dd7c362ab410714fdab25cd3735ed161fa6e1a 上复现了双向切换失焦,下一次 Tab 跳到时间线。当前提交在桌面与 390 像素宽度双向通过,Tab 进入已保存分析的测量说明。无具名来源的普通研究页保留相同的 76 条记录、末端时间、控件和 DOM 阅读顺序。跨来源初次失败保留错误,重试读回正确的新来源。给真实归档传错来源得到 409。已有原生结果仍能阅读,浏览回归发出零写请求。 首屏与焦点对照
语义与 CI 对齐
本次只调整展示,沿用已有来源和结果类型。打包、研究历程与只读恢复浏览器场景及标准 premerge 通过,含四项直接检查和五项选定检查。托管 CI 按当前 Goal 配置未查询,GitHub 合并资格单独核验。
我的整体评价 / Verdict
这次改动集中在两个原有展示组件、一条窄屏布局规则和已有只读浏览器回归。 看过桌面和 390 像素宽度的真实页面构成、混合结论的整段时间轴和切换后聚焦的指标说明。 读者进入子 Goal 后更快看清记录跨度,已有结果的阅读与键盘切换都可持续。公开截图来自打包合成数据;真实归档只在本机隔离读取,正式工作台仍有来源故障。 本次支持批准该提交,正式工作台仍需恢复并验证历史读取。
English verdict: APPROVE. On d1803a3c00579de998288cc6c0827e0e4ab8aba3, the complete recorded timeline is earlier for an empty analysis scope, and switching scopes preserves focus and Tab order. Scoped recovery, ordinary history and local validation passed. Installed adoption remains pending.
Goal and delivered outcome
The Research tab shows the recorded timeline first for a source-backed Goal with history and no saved verdict or native outcome. The parent link keeps its complete accessible name in a single line on narrow screens. Saved results remain ahead of the timeline. Switching analysis scopes keeps the original selector focused when the summary changes position. The next Tab follows the visible reading order into the saved analysis.
Validation
d1803a3c00579de998288cc6c0827e0e4ab8aba3, integrated with31ff0569843390413330e67be67f570b2d03650b. The earlier heada5dd7c362adropped focus in both directions and sent Tab to the timeline. The exact candidate keeps the same selector on desktop and narrow screens. The source-recovery regression exercises both directions in Chinese and English and checks that browsing sends zero writes.708c68e704: 1440 px: 769 → 632 px, 390 px: 1028 → 787 px.Frontend visual evidence
The exact candidate and original baseline use the same public synthetic read-only source.
The first Tab after switching to the saved scope focuses its measurement help.
Written by model_agent, Codex (OpenAI GPT-6).