Repository navigation
Fix Dashboard and Windows release qualification - #8296
Conversation
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
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)
动机 / Motivation
在 Goal 的成果页审阅报告的人会先看标题和处理入口,再阅读正文。维护者用打包浏览器场景核对哪些任务需要用户决定。 打开同一份长标题报告,旧包的标题与正文没有预期边线。旧打包检查也会把一份完整队列的 1 项阻塞显示判成错误。 独立安装的 head 恢复标题分隔线与报告边框。修正后的浏览器场景把三行预览和完整队列分开,分别验证未知和实际的阻塞数。 两处改动分别落在报告阅读样式和既有测试输入。 这次资格覆盖隔离的打包轮次与合成报告,完整浏览器场景矩阵和用户机器上的发布版留待各自验证。
改动思路 / Approach
报告组件使用现有主题的边线和卡片底色。测试沿用原生 fixture,从顶层列表驱动计数,另保留部分预览对照。 两条 CSS 声明归主题所有,三种队列边界归现有 smoke 和投影适配器。复用混合 fixture 消除一次重复生成。 样式消费既有主题。任务总数仍由原生投影提供,报告读权限由现有读取路径处理。
具体改动 / Changes
基线为 32167518b3c2646fd9b6a3a6e0aaf74b258a7b70。基线的前端交付规范要求从真实入口完成阅读并核对打包页面。这里用同一合成报告和原生任务投影测前后差异。 基线规范
- Theme boundary。 同一报告在实际打包 Chat 可打开,默认主题的标题分隔和正文框在桌面及窄屏均可见。
- Qualified counts。 同一修正场景在 main wheel 与 head wheel 均通过。三条预览保留总数且显示未知,完整混合队列为 1 个阻塞、20 项可选,全可选队列为 0 个阻塞、21 项可选。
关键代码讲解
report-review-masthead 和 iframe 使用现有主题边线与卡片底色。
goal-masthead 给实际消费的列表使用三行部分预览,完整混合输入保留 21 行,并断言 20 项可选。
对主干的风险 / Risks
修正后的绿色本身无法区分产品和测试问题。我在同一基线 wheel 重跑修正后的场景,同样通过。旧场景失败归于输入,样式修复则由真实报告页的边框与底色证明。页面版继续使用 sandboxed iframe,验收和反馈由原组件分别处理。
PASS。主题审计 39 个样式表、3695 个裸引用通过。两个报告 HTTP 用例通过,打包场景及桌面和窄屏阅读对照通过。 Quick premerge 的 4 项直接检查和 4 项选中检查通过。 与实际目标 main 的合成树干净。 历史对照保留 3 处 CSS 未定义引用和旧场景的 1 vs — 失败。 远端 CI 未取样。
我的整体评价 / Verdict
赞同提交这两个修复。报告页恢复可见边界,打包检查也能分辨预览行数与完整队列分类。 这次资格覆盖隔离的打包轮次与合成报告,完整浏览器场景矩阵和用户机器上的发布版留待各自验证。
我在独立安装的打包 Chat 中核对了 1440 和 390 像素的报告阅读。细线分开标题区、操作区与正文,窄屏的长标题换行,验收与反馈仍能在正文前找到。 范围限于合成报告的默认主题与这两个视口,其他主题与线上数据留待验证。
English verdict: APPROVE. 538814d restores the report reader’s theme borders and background and fixes the masthead fixture. Matched packaged browser and HTTP checks passed. The old fixture failed on the same baseline wheel that passed with the corrected fixture.
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
Signed-off-by: LoopX Agent <337587101+loopx-agent@users.noreply.github.com>
…validation 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)
动机 / Motivation
打开成果页报告的用户会遇到预览分隔和底色丢失;Windows 上用 Python 3.11 安装 LoopX 的用户也可能在运行 doctor 时看不到实际安装器。 此前预览边框计算为 none,文件与操作区缺少分界;含有盘符的 RECORD 条目也可能被当成相对位置,导致已安装的包失去正确的升级提示。修复后预览使用工作台现有的边线和卡片色,doctor 用系统路径读取绝对记录,并仍然只认实际加载的模块。
同一隔离安装包通过了对话活动和 Goal 注意事项的浏览器验证;修复同时恢复了报告主题与已知安装来源的正确识别。 不改报告读取权限、对话或 Todo 的生产规则、更新执行方式,也不迁移历史数据。 原生研究历程仍由 #8287 和 #8291 交付;这个 PR 单独修复两个可复现的前置问题。
改动思路 / Approach
三处 Dashboard 修复都在既有边界内:样式跟随主题,两个浏览器用例补齐输入;安装来源判断留在现有 doctor 适配器。 保留原有报告读取、聊天事件、Todo 分类和升级权限;两类修复各有可单独回退的提交。 报告数据仍从原有 Files 读取接口进入预览;CSS 只决定外框怎么画。两个用例继续走真实 Dashboard 消费路径:对话路由替身补上浏览器时钟生成的消息时间,Goal 则从状态生产者取得完整列表,再只在部分场景里裁剪行数并保留总数。doctor 从发行包的 RECORD(安装文件清单)选择候选路径,只有解析后等于导入模块的路径才查询安装器。复用这些边界比放宽浏览器断言或新增第二套安装器推断更直接。
具体改动 / Changes
关键代码讲解
report-review-masthead与预览 frame:把未定义的边线和底色替换为主题已有的--pw-line与--pw-card。从成果页打开报告的路线、表单与 iframe 设置保持一致。conversationActivityScenario:中断完成后追加的助手消息,以及普通发送先进入队列时的用户消息,都从浏览器时钟取得created_at。中断仍应收起活动状态并保留未发送草稿;排队后切换标签也不会重发。goalMastheadScenario:混合样本的完整 21 条给出 1 条阻塞、20 条可选;同一来源只显示 3 条时保留总数,阻塞总数继续显示未知。空样本不凭空制造决策卡片。python_distribution_install:绝对路径保留盘符或 UNC 根,发行包相对路径仍用locate()。同名的外部模块和缺失 RECORD 都不构成安装归属,原有安装器适配器继续决定升级提示。
依据 docs/development/frontend-delivery.md,文档版本 94b91e48aa4a2cdafe9d4d31d918fe84d0112b41。FE-3(按风险验证部分输入或恢复):验证了不完整与完整 Todo、会话中断和报告 503 后重试。FE-4(在实际安装包走通产品入口):从成果页打开报告,在桌面与窄屏检查,并用真实的本地 HTTP 报告读取验证恢复;对话 HTTP 替身不作为原生模型或服务端并发的证据。Windows 子问题沿用 #8299 对实际安装来源的行为边界,没有引入新的更新动作。
对主干的风险 / Risks
当前提交 edf32c71931f3773943638ebba26cb71744e7806 的标准预合并 PASS:6 个直接检查与 5 个选定检查全部通过,之前两次超时的词汇检查也在本次完整跑完。隔离 Python 3.11 wheel 的 Dashboard 两个场景、CSS 检查和 CLI 安装来源读回均 PASS。已留存的原生 Windows 生命周期测试 PASS 对应本提交内字节一致的 doctor 提交及测试;本机跨平台反例另用同一份 wheel 代码重现了旧盘符失根、验证 UNC 与相对路径,并确认外部和缺失记录仍被拒绝。这组反例模拟文件系统解析,不等同于一次新的原生 Windows 运行。
最重要的回归是把旧发行包错误认成活跃安装,给出错误升级入口。实际模块的解析路径比较仍是最终判定,拒绝时回到原有 unavailable 分支。样式不改变 iframe 边界,浏览器测试修改不进入产品运行时,新增字段也只在测试样本中补齐现有契约。原有持久状态、调度、权限以及两套生产端入口均无改动。远端 CI 结果 UNVERIFIED,本次按评审规则使用本地验证。
语义与 CI 对齐
复用现有主题属性、消息时间和 Todo 投影,以及安装器现有字段。没有新增词汇或扩大适用权限;预合并的词汇和边界检查已执行。
我的整体评价 / Verdict
长程行为无变化:报告读取、Todo 分类和会话继续沿用原有状态及入口。使用体验改善在于报告阅读层级和安装来源提示恢复,用户无须额外配置或确认。查看了完整的桌面与窄屏成果页首屏:报告标题和操作在边线之上,正文区域从卡片边线开始;390 px 上长标题和按钮自然换行。 截图来自独立安装包与合成报告;它们不证明真实 Goal 的研究数据或当前本机 App 已更新。 四个文件合计 34 行新增、19 行删除;没有平行版本或新的控制层。单独重做主题或 doctor 都能回退,但在同一安装包验证两者可以避免继承另一个已知故障。当前没有需要阻断的发现。
English verdict: APPROVE edf32c71931f3773943638ebba26cb71744e7806. Local premerge and packaged changed-invariant checks passed; no blocking finding.
Goal and delivered outcome
Two shared failures block validation of the research workbench. The Dashboard report reader refers to undefined theme tokens, and the installed browser checks receive two incomplete fixtures. On Python 3.11 for Windows,
loopx doctorcan also miss the active installer's metadata when RECORD uses an absolute path.On this PR’s previous revision, the native Windows job failed all three installer-owner cases. The standalone Windows fix passed them, but its Dashboard shards stopped on the undefined theme tokens. The required Python Tests workflow runs both sets of checks. Windows failure · Dashboard failure on the standalone Windows fix
This PR qualifies both fixes on a single candidate. The Windows change is the existing signed commit from #8299. The research changes are tracked in #8287 and #8291.
Author declaration
Written by Codex (OpenAI). The Dashboard reuses its existing theme tokens and fixture producers. The installation check stays in the existing distribution metadata adapter. The changes are independently reviewable as their original commits.
Validation
Candidate
edf32c71931f3773943638ebba26cb71744e7806against94b91e48aa4a2cdafe9d4d31d918fe84d0112b41.Visual evidence
Packaged Chat at
8667bdb3ff3d3e0c139a710b1276b3704d3b0d18with synthetic reports and the production HTTP reader. The combined candidate retains the same stylesheet and reader behavior. The matched desktop first view shows the restored division between review metadata, controls and content. The long title wraps on the narrow viewport and the preview fits its column. Loading and retry were checked in the same browser journey.Desktop before, 1440 × 900
Desktop after, 1440 × 900
Narrow after, 390 × 900