Repository navigation
Make deep analysis readable by episode - #6968
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.
Reviewer: model_agent · gpt-6.1-sol · OpenAI · runtime_reported · reasoning_effort=max
Approval conclusion (author-owned PR; GitHub blocks formal self-approval)
动机
在工作台查看研究分析的人需要先比较各片段的判断,再按需查证。
原先点击查看分析会铺开每个长问题、两段理由和多组引用,重复文字遮住比较;本版先列片段身份与实际判断,选择某一条后可读取完整问题、理由与引用。
默认没有展开的证据正文;键盘或点击展开后,原始问题、全部六项判断及精确引用仍可读取,关闭引用返回原来的按钮。
不改分析数据、统计分母、原生收益、模型生成、权限、作业执行或缓存协议。
本次完成阅读层修复;正式安装后的原结果回看在合并升级后核验,分析方法校准及完整研究工作台验收不由此关闭。
改动思路
两个入口消费同一份判读,差别在阅读轴:指标页按指标比较,深度阅读按片段看完整论证;应共享现有判读语义并各自采用按需详情,无需再造分析数据或状态。
当前PR只调整原深度阅读器、对应响应式样式、现有浏览器回归及设计合同,并提供合成整页截图。
保留当前页面会继续让重复段落随片段数增长。截断或从句子猜短标题会损失原意,也不能作为清理界面的办法。本版直接复用浏览器原生 details/summary 和现有指标标签、判读格式与引用读取,没有新增设置、状态库或框架。查看已有结果仍沿一个既有入口,零次追加确认;完整证据的展开是自愿阅读。收益与采用两段在片段内保留一次,其余四项直接按需展开,去掉了原来再包一层的全部判断入口。
具体改动
按修改前的 docs/development/design.md、固定版本 46cdf1971c4753151b521c10493203ed425f0f58 评审:Earn The User’s Attention 已实现,默认整页先呈现可比较条目;Explanations On Demand 已实现,声明身份、真实判断在前,完整问题、原文理由、否/未知与引用可继续读取。新增设计文字只澄清该规则同样用于深度阅读器,不能拿修改后的规范给自己作独立验收证明。
关键代码讲解
ResearchAnalysisPanel从原来的 section 改为原生 details,summary 使用声明的 episode.id 与两项已有判断,不从问题正文猜标签。内层完整生成问题、原始问题和逐字理由不变;列表只过滤已在关键上下文展示过的两项,因此六项判断全部仍在,收益检验的“是”不会被改成收益为正。- 原有
verdictLabel继续是两个阅读入口共用的格式化点:未知仍为证据不足,收益检验只表示做过检验,无益重复保持负面含义。没有另加六项综合分、原生收益或新的语义分类规则。 analysis-episode的 key 包含 pack、analysis key 和片段身份,避免换版本/语言后沿用同名片段的展开状态。对应 CSS 在宽屏并排显示身份与判断,窄屏自然换行;原引用加载与 Escape 的准确焦点返回仍由既有控制器负责。浏览不改变作业、缓存或来源权限。
7条路径64增14删及3张合成截图。生产部分为既有 TSX 与 CSS 的局部变化;现有研究历史场景继续承担集成回归,没有新造一份夹具仓库、独立模块或新的持久化格式。截图是打包浏览器的公开合成示例,PR正文明确区分它与原生安装及真实研究数据。
对主干的风险
最强反例是界面变短却丢掉理由、未知或引用,或换语言后展示上一版正文。不可变基线和本版用相同的现有场景及合成输入运行:旧阅读器没有比较行,独立行数断言得到0而应为2;本版完整旅程通过。再主动恢复“全部默认展开”的错误,检查得到2个已展开而应为0;恢复的代码与合格源文件一致。不是仅检查工具返回成功或组件存在。
打包回归同时检查完整原始问题、六项判断、理由只出现一次、中英文切换、桌面/390px、内部宽度、键盘展开、引用返回、历史/版本切换和迟到响应;模型写入保持零。Chat TypeScript/打包和 CSS token 检查通过,标准 premerge 的4项直接检查与4项目录检查通过,无人工 hold;自动选择侧重文档,所以由上述打包旅程补充实际 UI 风险覆盖。首次未重建 bundle 的基线尝试被来源校验正确拒绝,未当作任何基线通过;实际对照另用不可变源码构建。基线与本版的后端、模型输入及存储格式未改,未把渲染回归当作科学校准。CI 按 wait_for_ci=false 未查询。
我的整体评价
APPROVE,精确 head 276ad0c5e036b7b08b21f0eacae17a066d55b8ac,无阻断发现。long_horizon 为 preserved:原版本、来源、完整判断、失败恢复和执行边界不变;user_experience 为 improved:主要比较信息先呈现,完整论证仍可一次展开,减少重复滚动而不要求新确认、手填标识或配置准备。可选证据层级是有意的展示变化,已在正文和设计规范明确。
两个入口消费同一份判读,差别在阅读轴:指标页按指标比较,深度阅读按片段看完整论证;应共享现有判读语义并各自采用按需详情,无需再造分析数据或状态。
当前PR只调整原深度阅读器、对应响应式样式、现有浏览器回归及设计合同,并提供合成整页截图。
这是 justified_increment,架构判断 retain、规模相称;已删除重复理由,没有为了兼容另添 decoder 或第二个判断 owner。CQR cqr_e2c0d31c5db163113c92 对应本精确范围。正式 App 的安装后原结果回看仍要独立验证;更广研究工作台、独立科学结论及签名桌面壳升级不因本次批准关闭。当前 head 和主仓 main 目标已核对,最终合并还须新鲜 readiness 与已有维护者授权。
English verdict: APPROVE — 276ad0c5e036b7b08b21f0eacae17a066d55b8ac. The existing reader now compares declared episodes before opening full localized/original evidence. Identical-fixture baseline and expand-all mutation expose the old defect; packaged desktop/mobile, language, keyboard/citation and repository checks pass. No analysis data, execution, authority or scientific claim changes. Native installed readback remains a separate post-merge check.
Opening deep analysis currently expands every long question, utility/adoption rationale and citation group. The reader now starts with one comparable row per declared episode and its actual utility/adoption judgments. Expanding a row preserves the complete localized question, original question, all six judgments and verbatim citations. Duplicate utility/adoption evidence is removed.
The existing analysis producer, counts, denominators, language cache, source permissions and explicit execution/retry paths remain unchanged. Version/language changes reset optional episode details. The existing design contract now applies the same on-demand hierarchy to both metric and deep-analysis readers.
Validation: Chat TypeScript/package build, CSS token inventory, packaged research-history journey (desktop/390px, Chinese/English, keyboard, citation return, version/history/error paths and zero model dispatch) and risk-selected premerge. A deliberate expand-all mutation is rejected by the reader regression. No scientific or model-quality claim is made.
Visual evidence
Public synthetic fixtures in the packaged Goal workspace; tested presentation source
276ad0c5e036b7b08b21f0eacae17a066d55b8ac(rebase added only an unrelated monitor test). These are rendered browser captures, not an installed native App or real research result. Normal navigation reaches this reader through the existing “View analysis” action; there is no additional required setup or confirmation.Earlier reader, synthetic fixture at
b9e60c59f3c367a63755d97f4595d9c2eef91947:Comparable episode rows, desktop:
390px reader and optional complete question: