Repository navigation
fix(research): distinguish observations, interpretations and proposed interventions - #7343
Conversation
… view Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
…al-label-20261026 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)
动机
需要在手机或桌面阅读已保存研究汇报、并判断证据能支持什么结论的用户。
旧版深入阅读只有页首的两列标题,手机重排还会隐藏它;往下读第二条发现时,观察事实与 Agent 解释容易失去对应。现在每条发现都就地标明观察与依据、解释及已有竞争解释,引用仍跟随观察。
独立打包页面和真实 SQLite/HTTP 读取已验证新标签在普通展开与深入阅读中保留;同一保存结果的正文、竞争解释和引用与不可变主干逐项相同,关闭原文回到原引用。
本次只修复已保存汇报的阅读边界,不改变生成提示、科学判断或分析执行,不声称研究质量、理解速度、安装采用或完整成本已经改善。
不改动会保留已复现的阅读歧义;这里只修复可直接使用的阅读单元,没有把整个研究系统的校准、真实采用或用户理解当作本次已完成成果。
改动思路
复用既有 ResearchSynthesis 的 typed finding 和引用回调,用一个局部 findingAccount 渲染器服务普通与深入视图;投影不新增存储、推理调用或执行权限。
当前 PR 交付每条发现可辨识的观察与解释,以及原文、未知和返回行为的保留;模型推理质量与实际用户理解仍属于现有研究验收。
原有保存结果已经把事实、解释、竞争解释和引用分成不同字段,无需再让生成器改写正文,也无需加入文字分类器。旧做法依赖页面上方的位置关系,在手机改成上下排列后尤其脆弱;把同一组局部标题放在每条发现内,使读者滚动到任意发现时仍能辨认。普通页保持折叠摘要,用户自愿展开;详细页沿用既有进入方式和实验页签。引用只读取所选原文,界面不增加确认、必填表单或再次调用模型。
| 阅读状态 | 本提交的结果 |
|---|---|
| 普通汇报默认 | 保留摘要与两条折叠发现 |
| 展开发现或深入阅读 | 每条就地展示观察、依据引用、解释和已有竞争解释 |
| 手机上下重排 | 局部标题继续存在,正文自然换行 |
| 没有竞争解释或保留结果未知 | 不补造竞争解释,不把最佳数值当作最终采用 |
表格说明 1f805941a297ca28d53ff093e566fa7df5263160 的源码行为;实际验证来自独立构建、真实保存读取和浏览器操作,不把表格本身作为执行证据。
具体改动
规格依据为改动前的 docs/development/design.md,不可变修订 ce58c9141b4b805d4189f89959ade1be1d41ef40;同时阅读了既有 task-first frontend delivery 的授权、最短合法旅程和整页验收要求。逐项映射:Saved findings lead——原文观察与解释在每条发现内可读;Reading a citation——只打开请求原文,关闭准确回到引用;Reflow the composition——390px重排仍保留标签及完整正文;Opening or switching a view——阅读、切换和引用不生成分析或研究执行。四项均有当前生产读取路径和相应正反例。
关键代码讲解
findingAccount直接读取finding.observation、interpretation、alternative_explanations和原引用。观察标题、原文与来源在同一组内,解释独立成组;原有 alternatives helper 只在确有数组内容时展示竞争解释。ResearchSynthesis的两个分支 都调用这个局部渲染器。普通页仍用原生 details,深入页仍用既有 finding article 与实验页签;detail 只决定比较布局,不改变结果身份、正文或权限。.research-report-account h6使用现有正文与弱化标签层级;已有比较网格在窄容器/手机变为单列。删掉原先全局 aria-hidden 表头及其手机隐藏规则,避免滚过首条之后失去字段含义。
全差异为10条路径、74行增加/10行删除;生产两文件合计10/10,其余是24行现有真实读取测试、40行证据说明与六张公开合成页面图。没有新 schema、后端、生成提示、CLI、独立页面或配置所有者。已核对公开 before 的 renderer/CSS/合成夹具与实际基线相同,after 的同三项与最终头相同;图像本身不是安装态或理解收益的凭证。
对主干的风险
最强反例是标签改了,但引用被误放成对解释的证明,或公共渲染器丢掉第二条未知结果。我用同一冻结合成数据在不可变基线和本头的独立打包产物、真实 SQLite/store/HTTP 中比较:两条发现的完整段落、竞争解释数组和引用身份逐项相同,不归一化正文;仅新增标题有意排除在内容比较之外。数值7→3、无效候选、保留结果未知以及第二条空竞争解释均保留。原文实际打开 facts 的1–3行,Escape 后焦点回到同一引用;两个独立源的 mutation_attempts 均为0,原始数据字节未变。
本头完整 packaged research-history 场景运行180.68秒通过,覆盖中文/英文、普通/深入、390px、保存版本、缺失/失败/明确重试、来源变化恢复和相关片段返回;完整 source Chat 构建通过。原生 premerge 的4项直接检查与8条选中命令均通过,零失败命令,精确范围 CQR 有效且 PASS。整体 premerge 命令仍以1退出,因为六张二进制图要求人工检查,原生 gate 保留 manual_review_required、merge_gate_passed=false;我已逐张查看像素、元数据与合成来源,不把这一人工边界说成自动全绿,更不据此合并。
本轮一个初始私有比较使用了仅新头才有的 CSS 类,无法定位基线;改用两侧原本就有的 finding article 后,完整正文数组比较通过,原诊断保留。也检测到浏览器跨次调用会恢复视口,因此手机结论只引用已验证390×844的捕获与原生手机测试,不使用误标成手机的桌面图。没有查询或等待 CI,也没有运行真实模型。
我的整体评价
针对 1f805941a297ca28d53ff093e566fa7df5263160,APPROVE。这里的 goal_achieved 只指“保存发现的事实与解释在两种阅读模式中可辨识”这一完整修复。长程语义判断为 preserved:来源、未知、保存版本和明确执行边界未变;用户体验判断为 improved:后段发现与手机重排无需依赖页首位置,也没有新增确认轮次。没有以布局改进声称研究结论、理解速度或净效率已被测量改善。
整页视觉复核覆盖默认汇报、展开正文、深入阅读和390px窄屏;事实先于解释,证据贴近事实,长正文自然换行,未知结果与竞争解释保持可见,原文可按需打开并准确返回。
未验证已安装 App、辅助技术朗读体验、真实研究者理解收益或模型解释的科学有效性;合成页面与几何检查不能替代这些证据。
复用既有 ResearchSynthesis 的 typed finding 和引用回调,用一个局部 findingAccount 渲染器服务普通与深入视图;投影不新增存储、推理调用或执行权限。 当前 PR 交付每条发现可辨识的观察与解释,以及原文、未知和返回行为的保留;模型推理质量与实际用户理解仍属于现有研究验收。
局部抽取同时减少了普通/深入字段布局的重复,新增成本与问题相称;数据字段和共享语义词表都复用原所有者。当前没有复现本范围的新阻塞缺陷。保留 native 六项图像人工检查与维护者合并权限的独立边界;若头、实际保存字段或引用/返回行为改变,需要重新验证,不能沿用本结论。
English verdict: APPROVE — 1f805941a297ca28d53ff093e566fa7df5263160 keeps observation and interpretation locally identifiable in both saved-report readers, including390px reflow, while preserving original paragraphs, alternatives, citation identities and source-return focus. The complete synthetic SQLite/process/HTTP reader scenario and4+8 automated premerge checks pass. Native premerge still reports six manual image holds; installed use, comprehension and scientific validity remain unqualified. No merge authority is implied.
…15' into codex/research-proposal-label-20261026 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; reasoning_effort=max.
Approval conclusion (author-owned PR; GitHub blocks formal self-approval)
APPROVE — 22848c55bb3b67abc6144e7fa76cdbd1bac6b5b2,完整14条路径。 旧1f80头的批准不覆盖本提交;本轮重新核对完整差异、主干基线与实际打包入口。
动机
阅读已保存研究结果的人,需要分清观察事实、Agent 解释和下一步建议,随后按需检查原文。旧版普通展开没有字段标题,深入阅读又依赖一个在手机上隐藏的页首标题,后面的发现容易失去对应。旧“最小干预”还可能把建议写成已证明的最优实验。
本轮验证的结果是:每条发现保留自己的观察与依据、解释和已有竞争解释;实验采用“拟议干预 / Proposed intervention”。两条发现的原始正文、竞争解释、引用和四项实验内容与不可变主干逐项相同,未知的保留结果仍未知。没有把文案修复扩展成研究有效性、最小性、理解速度或净效率证明。
改动思路
最小有效边界是复用已有 ResearchSynthesis 和 typed finding,让普通与深入阅读共用局部 findingAccount;继续用原来的来源读取、实验字段和返回回调。一个共享页首说明无法覆盖后续发现或手机堆叠;新增协议、生成器或执行器则没有必要。重复局部标题增加一点高度,但实际展开仍可读,普通默认依然折叠,没有额外确认或设置步骤。
独立规格为目标主干 2b72d30fca7f452a71dffb723ebf6d1e2e878484 的 docs/development/design.md 和 task-first frontend delivery,读取先于实现。研究部分与先前完整读取的条款相同,本轮也核对了期间新增的无关 Forms 条款。验收映射覆盖可读的观察/解释、窄屏重排、仅打开所选原文、准确返回原引用以及阅读不触发分析执行;没有用本 PR 自己修改的证据说明自证。
规格映射:Saved findings lead 对应可读的观察/解释,Reading a citation 对应所选原文与准确返回,Reflow the composition 对应窄屏重排,Opening or switching a view 对应阅读不触发执行。
具体改动
完整差异为14条路径、+97/-11:两个既有生产文件11/-11,现有真实研究场景增加24行断言,说明62行及十张合成截图。相对上一评审头,新增提案标签及四张前后截图;整体验收仍重新执行。
findingAccount保留观察段落及其引用,再展示解释与存在时的竞争解释。普通展开与深入矩阵调用同一函数;第二条没有竞争解释时不凭空生成标题或内容。controls只修改两个本地化标签。intervention、controls、prediction、disconfirming_result、关联片段与引用保持原字段、原内容;建议本身不获得执行权限或科学资格。- 同组件的普通/深入分支、实验页签与 citations 回调沿用现有保存结果。样式移除全局标题与手机隐藏规则,局部 h6 随字段自然重排,正文仍16px。
- 现有 research-history 场景通过真实 SQLite、process/job、HTTP 和明确的合成保存输出检验两个阅读分支、语言切换、未知、版本、失败恢复及原文返回;没有真实模型调用。
对主干的风险
最强反例是标题正确而正文对错字段,或“提案”标签掩盖了内容、引用和未知结果的丢失。我在独立打包的精确主干与本头用同一冻结夹具比较完整 finding 段落、竞争解释、引用身份和四项实验值,不归一化正文;结果逐项相同。旧英文标签确为 Smallest intervention,本头为 Proposed intervention。实际中英文390px页面均无横向溢出,固定条件与预测仍可继续阅读;原文 facts1–3真实打开,Escape 后焦点回到同一证据按钮并保留实验页签。两个独立源均0次写入尝试,原数据字节未变。
完整当前 packaged research-history 场景193.458秒通过,当前完整 Chat 构建通过。原生 premerge 的4项直接检查、7条目录检查和1条公共边界检查均通过,零失败命令。整体仍退出1,保留 manual_review_required、merge_gate_passed=false 与十张二进制图的人工门;没有把人工复核说成自动全绿或合并授权。十张图的合成来源与元数据已检查:六张与先前已查看图像逐字节相同,四张新增提案桌面/手机前后图本轮逐张查看。当前默认报告、桌面深入、手机展开、提案及原文也独立查看。
没有查询、轮询或等待 CI。已安装 App、真实研究者理解收益、辅助技术朗读、科学有效性和完整成本仍未资格化;保存结果阅读的通过不替代这些边界。
我的整体评价
APPROVE。 本提交完成有用且可独立回退的保存报告展示修复:事实、解释与提案的含义更准确,原文与返回关系保留。long_horizon 为 preserved,限定于原始证据、未知和执行边界;user_experience 为 improved,限定于已观察的局部辨识与响应式阅读。没有声称真实研究质量或净收益已提升。
单一局部渲染器同时减少重复,既有 TypeScript 数据与 React 展示所有者足够,无需新协议或后续迁移。本次没有发现当前范围的新阻塞项;保留原生图像人工门和维护者合并权限。头、保存字段或来源返回行为变化后需要重新核验,不能沿用本结论。
当前默认报告、桌面深入、手机展开、提案及原文也独立查看。
已安装 App、真实研究者理解收益、辅助技术朗读、科学有效性和完整成本仍未资格化;保存结果阅读的通过不替代这些边界。
English verdict: APPROVE — 22848c55bb3b67abc6144e7fa76cdbd1bac6b5b2 keeps observations, interpretations and proposed interventions truthful in both saved-report readers. Exact base/head finding, experiment and citation contents match; the full193.458s synthetic SQLite/process/HTTP scenario, source return and zh/en390px checks pass. Native automated4+8 checks pass while ten image manual holds remain. Installed adoption, comprehension, scientific validity and merge authority are not established.
Ordinary expanded findings previously placed observation and interpretation in unlabeled paragraphs; deep reading hid its global column labels at narrow widths. Keep observation/evidence and interpretation/alternatives identifiable for each finding in both modes, with citations beside the observation. The next-experiment field now says Proposed intervention / 拟议干预: a saved proposal does not establish that its intervention is the smallest possible design.
The existing typed synthesis, tabs, saved text, controls, predictions, episode navigation and citation return remain unchanged. No prompt, model dispatch, execution permission or benchmark behavior changes. One shared finding renderer replaces duplicated JSX and the global column-label row.
Validation and limits
22848c55bb3b67abc6144e7fa76cdbd1bac6b5b2against targetmainat2b72d30fca7f452a71dffb723ebf6d1e2e878484; the normal update retains the original PR history. The diff contains only the 14 research-reader paths.research-historyscenario passed in 179,299ms using the disposable real SQLite/HTTP owner and saved synthetic provider output: ordinary/deep reading, Chinese/English, desktop/narrow, observation-bound citations, exact return, episode/history selection and failure/recovery.cqr_e5d4c4fac9e6ee47a1ebis valid. Standard premerge passed all 4 direct and 8 selected automatic checks; its exit remains nonpassing because 10 binary-image visual/privacy holds require the exact-head independent review.Visual evidence
Synthetic data only; reproduction and capture revisions are documented in the evidence account. Finding-label images retain the original source-build comparison; the proposal images use before UI
1f805941a2and the changed component now included in this head. Target synchronization did not change that component or its layout; the final complete packaged scenario was repeated after synchronization.