Skip to content

feat(research): connect report findings to related episodes - #7260

Merged
loopx-agent merged 4 commits into
mainfrom
codex/research-linked-episodes-20261024
Oct 24, 2026
Merged

loopx-agent merged 4 commits into
mainfrom
codex/research-linked-episodes-20261024

Conversation

@loopx-agent

@loopx-agent loopx-agent commented Oct 24, 2026 •

Copy link
Copy Markdown
Collaborator

研究汇报已有明确的片段关联,读者却只能跳到整个 Goal 的历程或手动寻找。现在每条结论和实验建议可直接打开对应片段、查看原文,再返回同一处报告;范围切换显示实际数量,切换报告标签后也能正确返回。

Findings and proposed experiments now open their explicitly declared episodes in source order. The reader can switch between related/all episodes and return to the originating finding or current report tab. Unknown judgments, negative results and original citations remain intact. The broader history action explicitly names the Goal.

Changes

  • Reuse the typed synthesis episode_ids and existing episode/citation readers; keep selection local to the current pack, result and language.
  • Correct the count after showing all episodes and provide the inverse scope action. Preserve keyboard focus across toggles and report-tab changes.
  • Shorten repeated scope copy and use an in-place jump glyph.
  • Limit the outer disclosure selector to its own summary so narrow episode questions and judgments reflow instead of squeezing into columns.
  • Retain ordinary measurement/report reading, complete original content and optional method/cost details.

Validation and boundary

Tested head: feb45242873eaba5f24e510d2ef2b6765b54abf2, based on c1a589aac771b39df9137dc808280c2a33fb4cbd.

  • TypeScript no-emit check, production Chat bundle and CSS token checks passed.
  • Full packaged research-history browser scenario passed against the real isolated HTTP/SQLite owner. It covers exact subsets and source order, actual scope counts, subset/all/subset, removed-origin return focus, original citation Escape, Chinese/English, 1280×900 and 390×844 reading, unchanged sources and zero mutation attempts.
  • 47 production research API/store tests passed.
  • Standard native premerge: 4 direct and 16 selected checks passed. Its eight binary-image holds require visual review; all eight images were manually inspected, including metadata/source provenance. Exact-scope change-quality receipt cqr_7b04c784e6758fc34ca9 is valid.
  • The initial refinement reproduced the incorrect 1/3 label after showing all three episodes; the corrected scenario passes. Two intermediate bilingual test assertions were corrected for the fixture's locale reset and actual “Unknown 1” summary.
  • Remote CI is not consulted under this Goal's configured policy.

This qualifies the packaged reading interaction with synthetic evidence. Installed desktop adoption and researcher comprehension are still unqualified at submission. It introduces no model execution, scoring changes, storage migration or new execution authority.

Visual evidence

Synthetic fixture only; both sides use the same three episodes and unchanged source data. Before uses main c1a589aac assets (tree-equivalent to 6189cf053); after uses the tested head above. Sidebar and shared contextual-help changes already belong to #7264.

Default research viewport

Report before Report after
Before report After report

Related episode reader

Narrow episode reader before Narrow reader after
Before narrow After narrow
Default narrow viewport and English reader

Default narrow viewport

English related episode reader

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 loopx-agent left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewer: model_agent | gpt-6.1-sol | OpenAI | runtime_reported | reasoning_effort=max

Approval conclusion (author-owned PR; GitHub blocks formal self-approval)

未发现阻塞当前精确版本的问题。

动机

阅读研究汇报的人,需要从一条结论或实验建议追到它引用的研究片段和原文,再回到原位置。 以前汇报只提供整个 Goal 历程入口,读者要自己重新查找片段。现在结论与实验建议按已有明确关联打开对应子集,保留来源顺序,并返回原控制和实验页签。 当前完整打包research-history已通过明确片段子集、逆序引用仍按来源排列、未知判读、显示全部、原文Escape焦点返回、汇报返回、实验页签及390px判读换行。 本次只连接已保存汇报的阅读路径,不运行模型、改写分析、重新评分或证明科学解释;安装采用和研究者理解收益仍未资格。 真实研究者理解、分析的科学校准和已安装 App 使用仍需原研究及产品工作验收;当前链接和页面交互不替代这些成果。

用户的实际工作是读取已存在的事实并准确返回。当前入口没有要求重新配置已知来源、补充无关意图或再次确认原有读取授权;改进集中在这条既有阅读路径。额外操作用于主动展开、查证和失败恢复,不能把它们计作新执行能力。

改动思路

复用现有 synthesis.episode_ids、已验证pack顺序、引用reader和analysis控制器足够;AnalysisReport只保存本地选择与返回位置,无需新协议或数据owner。 本 PR 交付已保存结论与实验的片段阅读及准确返回,并修复嵌套披露的窄屏换行;科学判断、模型运行和安装验收仍由原owner接续。 原任务、片段、引用和状态仍由既有typed owner提供;组件只拥有布局或当前阅读选择。默认与详情分层沿现有入口完成,另造页面、协议或第二份数据状态都没有必要。

55dea已有整Goal历程和手动逐片段阅读;2fd新增明确关联子集/原返回,CSS直接子项修复使390px判读落到完整问题下方。原来源、分析结果和执行权限不改变。 我比较了保持现状、增加说明、复用现有组件和另建工作台;现有组件内的有界改动足以解决已经观察到的访问和阅读问题。没有为这次局部改动要求完成整个产品路线图。

具体改动

完整范围为13个路径、+188/-15:4个生产展示文件、一个合成夹具、既有完整浏览器场景和7张PNG。source41e到当前2fd只增加图片;当前构建和完整场景均从本次head运行。七张图逐一检查了像素、尺寸和PNG块,未见嵌入文字或私人运行内容。

夹具增加control与retention片段及其明确关联,保留中英文unknown及竞争解释。新场景核验子集、来源顺序、原文及两级返回,不只检查一个新按钮存在;完整history的其他分页、切换和恢复路径也一并运行。

先验规范为 docs/development/design.md,不可变版本 55dea1408acfeb0bea416f1c3e372d5c1b480d5c(正文),并核对同版本的task-first交付及AGENTS公开证据规则。其text与已阅读规范一致,没有从实现倒推要求。attention-and-reading在现有入口和默认/详情层级上implemented;truthful-records-and-recovery在原事实、负状态和恢复上implemented;populated-and-narrow-readback在当前长内容、展开与窄屏观察上implemented。安装与理解效果沿原产品工作deferred,剩余gap见动机。

关键代码讲解

  • episodeLink:20:把已有episode_ids交给UI回调;空关联无按钮,不按标题或时间推断。
  • AnalysisEpisodes:304:对原pack数组按明确ID过滤,保留来源顺序,相关模式先打开首个片段。
  • AnalysisReport:364:保存触发按钮、选择序列及show-all;布局提交后打开reader并定位,返回保留原页签。
  • keyed AnalysisReport:435:按pack、结果和语言隔离选择/返回状态,使用原引用和错误恢复owner。
  • analysis-method direct summary selector:63:把外层summary样式限制到直接子项,避免覆盖嵌套片段的手机重排。

对主干的风险

桌面/Chat生产构建、CSS检查、47项原生API/store测试和完整打包research-history通过。premerge为4项直接和16项选中命令通过,原始总门仍保留7张图片的manual_review_required。 CQR cqr_da4362e5a4170fc33b50 为pass。图片人工检查已做,原premerge的manual状态仍按原回执披露,不把它写成自动检查全绿,也不把CQR当合并授权。构建已有的大包警告保留;按当前策略未查询、轮询或等待远端CI。

合成关联先给control再给search,页面仍呈现原source的search/control顺序;retention只打开自身且保持unknown。显示全部恢复三个片段,原文Escape返回当前片段,再返回原结论;实验路径返回后页签仍选中。 最强风险是用相似文字找错片段,或切换报告后把引用/返回带到另一个pack。当前实现只消费已有明确ID,按pack/结果/语言重建本地reader,原引用controller继续负责来源和错误;当前完整场景覆盖不同片段、show-all和两级焦点返回。没有新模型、评分或写入,既有未知/否定证据未被当成正结论。

语义与 CI 对齐

本次使用现有typed事实,新增状态只服务局部呈现或选择,不按标题子串推导权限、完成或科学接受。默认展示的变化已在描述和上述对照说明,没有opt-in/default-off声明,不虚构关闭开关。共享执行、配额、权限、任务和原分析协议均未改变;中英文RFC检查及实际diff范围的公开边界检查通过。相关错误与unknown保持真实,阅读入口不赋予运行模型或写入的权限。

我的整体评价

APPROVE,交付判断为justified_increment。long_horizon为preserved:原事实、重复读取、失败恢复和返回仍由原owner维持,没有重复执行;user_experience为preserved,当前可发现性和阅读路线有可观察改善,但没有量化理解或效率净收益的结论。

复用现有 synthesis.episode_ids、已验证pack顺序、引用reader和analysis控制器足够;AnalysisReport只保存本地选择与返回位置,无需新协议或数据owner。 本 PR 交付已保存结论与实验的片段阅读及准确返回,并修复嵌套披露的窄屏换行;科学判断、模型运行和安装验收仍由原owner接续。 已观察合成三片段及混合已知/未知判读的桌面和390px默认、汇报与相关片段:指标先于解释,完整问题和否定条件可展开,正文恢复正常阅读色,原文和返回入口可达。 未验收范围:未验证已安装 App、真实研究者理解、科学因果与新模型效果;公开图及本次场景使用合成材料,不冒称实际Grid/Astra研究验收。 这次局部组合/reader提取是有当前调用者的简化,未引入备用框架或新持久版本。保留相应基线失败、图片人工门、正式安装与独立合并权限;结论只绑定 2fd5791ed5ffcb59957be7a646c574cfe00957d0,变化后的head须重新核对。

English verdict: APPROVE — 2fd5791ed5ffcb59957be7a646c574cfe00957d0. Current production builds, native checks and complete packaged reading/recovery coverage support this bounded UI increment. Existing data, unknowns, citations and authority are preserved. Public default/expanded/narrow evidence was inspected. Explicit episode lineage preserves source order and exact return without new inference or execution. Installed adoption, comprehension and broader outcome qualification remain open.

@mergify

mergify Bot commented Oct 24, 2026

Copy link
Copy Markdown

This pull request has merge conflicts with main and cannot be merged
until they are resolved. Please rebase or merge the base branch, @loopx-agent.

Choose the remote for the base repository, not an out-of-date fork.
For a fork clone, first inspect git remote -v; upstream must point
to https://github.com/loopx-project/loopx.git. If it is absent, add it
with git remote add upstream https://github.com/loopx-project/loopx.git.
Then run:

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 HEAD

For a same-repository clone whose origin points to
https://github.com/loopx-project/loopx.git, use origin instead of
upstream for fetch/rebase. If you prefer merging the base, use
git merge <base-remote>/main and push normally.

Keep the DCO Signed-off-by trailer on every commit when you rebase.
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/syncing-a-fork

@mergify mergify Bot added the needs-rebase Mergify: the pull request has merge conflicts with its base branch label Oct 24, 2026
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
@mergify mergify Bot removed the needs-rebase Mergify: the pull request has merge conflicts with its base branch label Oct 24, 2026
@loopx-agent
loopx-agent merged commit 55cfdd6 into main Oct 24, 2026
41 of 52 checks passed
@loopx-agent
loopx-agent deleted the codex/research-linked-episodes-20261024 branch October 24, 2026 15:42

@loopx-agent loopx-agent left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewer: model_agent · gpt-6.1-sol · OpenAI · runtime_reported · max

Approval conclusion (author-owned PR; GitHub blocks formal self-approval)

合并后精确版本评审:远端合并时仅见旧 head 的记录,本评审补齐新 head 的独立核验;不追认合并前已经满足评审或 readiness。

动机

阅读已保存研究汇报、需要追查原文的 LoopX 用户。 以前从结论只能进入整个 Goal 的历程,需自行寻找相关片段;现在可直接打开该结论声明关联的片段,并返回原结论或当前报告标签。 桌面与窄屏可按报告、相关片段、原文、返回的顺序阅读,相关与全部范围的数量跟随实际显示,未知与负结果完整保留。 本 PR 不运行模型、不改写分析与计分、不新增执行授权,也不把合成阅读检查当作科学收益。 已安装 App 的最终采用回读、真实研究者理解和净效率收益仍待原产品工作验收。

本次按原始阅读任务判断完整改动,而非继承旧版本的批准。读者的目标是查证一条已保存结论,不能被迫重建分析、重新寻找来源或重复授权。此次入口减少手动查找,相关与全部范围有明确回路;额外操作都是自愿展开与查证,没有新增必填配置、复制步骤或确认轮次。实际理解和净成本收益没有被测试数量替代。

改动思路

复用已有片段与原文阅读器,只在报告内部保存当前选择和返回位置;不需要新增工作台、协议、存储或提示词分类器。 本 PR 交付报告与明确关联片段的可用阅读闭环;真实安装与研究收益由当前产品交付继续验证。

既有 typed synthesis 声明关联,界面沿源数组过滤,保持原始顺序;片段判读和引用控制器继续拥有原文、错误及恢复。局部 AnalysisReport 只承担选择、显示范围和 DOM 返回生命周期,按包、结果和语言重建。取消这一局部边界会把报告焦点与片段展开逻辑混入源 owner;另建状态服务更重。直接子项 CSS 修复级联即可,不需要备用移动端组件。

触发 旧路径 当前结果
从报告追查片段 整 Goal 历程或手动查找 精确关联子集,原始顺序
显示全部再返回 单向切换且数量仍为1/3 实际3/3,可恢复相关子集
原按钮因切换标签移除 焦点无法返回 回到当前报告标签
窄屏展开 外层flex覆盖内层网格 完整问题与判断分行

具体改动

完整范围为14个路径:四个生产展示文件、现有合成夹具和浏览器场景、八张公开截图。当前版本合入主干 #7264,保留共享帮助图标与设置导航修复;本 PR 没有重新实现这些行为。夹具的关联顺序故意与来源顺序相反,并包含不同的控制和保留未知项,独立验证不会找错片段,也不会把最佳值当成实际采用。

先验规范是 docs/development/design.md,版本 c1a589aac771b39df9137dc808280c2a33fb4cbd。已核对 earn-the-users-attention 的主次与少做额外工作、research-workspace--研究工作台 的真实Goal阅读与返回、populated-page-review 的整页长内容/混合状态及窄屏要求。三个既有规范段落对应实现和实际检查;本PR未改写规范以自证。正式安装及理解收益沿原任务继续。

关键实现为 apps/presentation/dashboard/src/features/personal-workspace/goal-research-synthesis.tsx:20 的 episodeLink、apps/presentation/dashboard/src/features/personal-workspace/goal-research-analysis.tsx:304 的 AnalysisEpisodes、同文件364行的 AnalysisReport,以及 apps/presentation/dashboard/src/features/personal-workspace/goal-research-analysis.css:63 的直接子项选择器。它们分别传递明确ID、保留来源顺序和完整判断、控制准确范围与返回、恢复手机布局。新增计数从实际片段派生,不另存同步副本。

对主干的风险

最强反例是按钮存在且没有横向溢出,但读者仍看到错误片段、错误范围或不可读的压缩列。因此检查覆盖整个真实隔离HTTP/SQLite读取路径,并逐张观察默认/展开的桌面和390px页面。完整 packaged research-history、47项API/store、类型检查、Chat构建与CSS检查通过;标准premerge的4项直接和16项选中检查通过。8张PNG保留人工检查门,已核对像素、元数据与合成来源;原始premerge仍为manual_review_required,没有伪称自动全绿。精确CQR为 cqr_7b04c784e6758fc34ca9;按配置没有获取或等待远端CI。

保留了原范围错误的失败记录:展示3项后仍写1/3,改进后同一断言通过。两次中间英文检查暴露的是夹具在reload重置中文和测试误用不存在的概括标签,已修正初始化及独立Unknown1断言;没有删除未知证据检查。子集/全部/子集、引用Escape、原按钮消失后的焦点、实验标签和中英文窄屏均通过。来源摘要不变、写入尝试为零,未运行模型。原无synthesis、失效来源及错误恢复仍由原owner覆盖;无新兼容协议、权限或持久迁移。

我的整体评价

APPROVE,交付判断为 justified_increment。长程语义 preserved:历史结果、证据、原始未知和后续工作owner未改变,没有重复执行或新调度负担。体验 improved 仅指观察到的直接阅读、准确范围和恢复行为;它不表示科学解释、理解效率或token净收益已证明。

复用已有片段与原文阅读器,只在报告内部保存当前选择和返回位置;不需要新增工作台、协议、存储或提示词分类器。 本 PR 交付报告与明确关联片段的可用阅读闭环;真实安装与研究收益由当前产品交付继续验证。 已检查同一合成三片段、已知与未知混合结果的桌面和390px默认及展开页面:实际测量先出现,报告解释与观察分列,窄屏完整问题和判断分行,原文与返回入口可达。 未以这些截图验证已安装 App、真实研究者理解或科学解释;公开证据来自隔离的真实 HTTP/SQLite owner 和合成输入。

此有界组件和相邻CSS修复有当前调用者,也保留原读取和执行分界。无需引入更大的框架或临时分析Goal。当前版本的公开评审、质量回执、人工图片观察和合并权限仍是不同证据;旧版本review不作为当前批准。结论只绑定 feb45242873eaba5f24e510d2ef2b6765b54abf2,head变化必须重新核验。

English verdict: APPROVE — feb45242873eaba5f24e510d2ef2b6765b54abf2. Explicit source-ordered episode links, accurate reversible scope, citation and focus recovery, and narrow reflow passed the complete packaged real-owner reading journey. Types/build/CSS, 47 API/store tests and native checks passed; eight synthetic images were manually inspected. No new execution, persisted authority or model run. Installed adoption, comprehension and scientific benefit remain open.

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.

1 participant