Repository navigation
fix(research): reserve 0% and 100% for exact input shares - #7479
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 · effort=max
Approval conclusion (author-owned PR; GitHub blocks formal self-approval)
精确 head 82a92767badcce9f2f213fd50a12b18517ed78c8 未发现阻塞问题;同意这个有边界的显示修复进入维护者评审。这是作者自评,不是独立验收或合并授权。
动机 / Motivation
研究页读者在查看结构输入开销时,会把微小但非零的占比看成没有开销。同一合成输入中,20 个 LoopX token 除以 1,000,000 个请求输入 token,旧组件显示 0%,候选显示小于 0.1%;接近全部输入时也不再显示成恰好 100%。
相同最终夹具在不可变基线组件失败、在候选完整打包场景通过;中文、英文、零和全部输入的读回都保持可区分。本次不增加归因采集、模型调用、译文修订流程或新的操作步骤。真实 provider 归因、完整成本、模型语言质量和正式 App 采用仍待各自原任务验证。做精确计量的人仍可打开原明细,不必为这次修复配置新功能或重跑分析。
改动思路 / Approach
比例格式由现有研究页投影组件负责;精确计数及归因事实仍归原 TypeScript owner,不增加第二份计算或状态。本 PR 的边界是有完整读回和反例的占比显示修复;原始数据、生成质量与安装采用没有因此获得验收。
最小修复放在原 formatter:小于现有一位小数精度的正值显示上界,接近全部但仍有非 LoopX 输入的值显示下界,精确零和全部保留。增加固定小数位只会把误报推到更小数量级;另建格式模块、设置或修订流程没有收益。原 helper 同时服务主比例、未缓存比例与缺失归因范围,不形成第二份规则。
| 输入 | 旧读数 | 候选读数 |
|---|---|---|
| 20 / 1,000,000 | 0% | <0.1% |
| 0 / 1,000,000 | 0% | 0% |
| 999,999 / 1,000,000 | 100% | >99.9% |
| 1,000,000 / 1,000,000 | 100% | 100% |
这张表说明显示差异;实际执行证据来自打包读者和最终同夹具对照。原有导航、可选明细和故障恢复保留,新增必需交互为零。
具体改动 / Changes
规范为 docs/development/design.md,修订 a1875be6057bcfdfac9e8fa0d38e20c25e5f6542。按修改前 Explanations On Demand 与 Populated Page Review 的条款核对:necessary-value-qualifier 保留邻近数值限定、未知与分母;explanations-on-demand 复用已有帮助和明细、键盘入口;populated-page-review 实看首屏、混合状态与窄屏展开,和功能断言分列。前两个标识引用规范无编号条款,没有新增规范义务。
关键代码讲解
LoopXInputShare用严格数值范围决定显示界限;0、1不进入新分支。原null、不可用和部分归因守卫、精确计数、缓存未知及方法说明原样保留。Intl formatter 在同一次渲染内复用,不导出新的公共 API。ResearchProgress仍将同一选中用量传入原区域。来源与用量默认收起,研究结果与六项判读仍先呈现;没有用显示修复改写归因或发起分析。researchHistoryScenario.run在原场景加入中英四组边界值和准确分子/分母读回。合成请求的会话总量、缓存与 request coverage 一致,结束后撤销 override,继续原有全历史、引用、失败恢复与竞态流程。
完整差分为两个文本文件和八张 PNG。远端完整 patch 在隔离 Git index 中重建的树与本地 head 完全一致,包括所有图片;没有对当前工作区应用远端 patch。图片均来自合成场景,逐张看过像素,PNG 只有 IHDR/IDAT/IEND,未含文本或 EXIF。原截图资产保持不变。
对主干的风险 / Risks
最强反例是展示夹具自行提供了比例,因此不能把它当作 provider 真实归因或净效率损失证据。本次测试只认证原读者的显示;十项原 typed-owner 测试分别覆盖缺失、零、部分、重复、缓存及错误输入。没有新增持久化协议、状态分类、授权或数据计算 owner,失败仍沿原来源核对与恢复处理。估计专属界面没有另跑,复用 formatter 及既有估计边界检查已经核对;实际模型与安装采用没有验收。
PASS:最终相同夹具的不可变基线在实际 0% 断言失败,候选完整打包场景通过,耗时 233352ms;中英端点、精确计数、键盘、桌面/390 及原故障恢复保持。前后构建与生产/夹具哈希已核对。静态构建和 CQR 有效,标准预检 4 项直接与 16 项选中检查通过,无当前自动失败或跳过;八张图保留原生人工审阅 hold,未宣称 merge-ready。没有查询或等待 CI。
早期测试的中文 selector、英文 summary 帮助命中及不一致合成覆盖已在最终对照前纠正,没有删去数值断言或放宽门槛;早期 CQR 长度/引用错误与旧回执状态保留为失败历史。最终对照使用同一份完整新夹具,覆盖此前暴露的显示条件。预览中旧的基线 advisory 不代表本次失败:当前 ratchet 与最后预检实际通过。构建原有大 chunk 警告保留。
语义与 CI 对齐
改动复用现有类型、method 和 status,显示精度留在本地;未新增语义词表或机器义务,没有 prose 分类或自动执行默认变化。普通比例和真零/全量采用同一原读者;本次没有 opt-in/default-off 主张,不能把缺 feature 字段当成隔离证据。没有成对 RFC 变更。合并与安装资格、独立 review 和实际归因保持各自边界。
我的整体评价 / Verdict
long_horizon=preserved:没有改变执行、结算、状态或后续工作 owner;完整读回及恢复流程保持,不能据此宣称长期净收益。user_experience=improved 仅指这个原有读者中错误端点已被可辨的界限替代,精确计数仍在同一入口,没有新必填项或确认。
首屏研究结果保持主次,占比仍在原用量入口;展开后精确计数分行可读,没有增加解释段落或缩小字号。实际用户理解、真实 provider 计量与正式 App 采用未验证;本次画面和新增比例边界行使用合成数据。另外实看了本次完整场景的长重复问题、混合未知/重复判读与刷新失败;数值、正文和方法仍有层级,恢复入口可达。折叠、无溢出和字段存在的自动断言没有代替这些观察,也没有将合成画面升级成整站或真实用户验收。
复用现有 helper 和场景与这个确定性缺陷的成本相称;没有兼容迁移或更小且等价的必要修复。规范、来源与不确定性未被重写来匹配补丁。原生图片 hold 已由本次像素/元数据/来源核对回应,但自动 gate 仍保留该类别,维护者的精确版本评审和合并授权仍独立。
English verdict: APPROVE — 82a92767badcce9f2f213fd50a12b18517ed78c8. No blocking finding for the scoped input-share display fix; matched baseline failure, packaged locale/count controls and local checks cover it. Actual provider attribution and installed adoption remain unverified; author review is not merge authority.
Goal And Delivered Outcome
研究页目前把
20 / 1,000,000显示成0%,把999,999 / 1,000,000显示成100%,会误导读者判断结构输入开销。本提案复用现有比例读数和归因明细,将这两种情况显示为
<0.1%和>99.9%;实际为零和全部输入时仍显示0%、100%。精确 token 数沿原明细读取。目标分支为main。Author Declaration
Written by: model_agent · GPT-6 family · OpenAI Codex.
Implemented against:
docs/development/design.md与docs/development/frontend-delivery.md,修订a1875be6057bcfdfac9e8fa0d38e20c25e5f6542;上述可复现数值缺陷是本次精度修复的直接依据。LoopXInputShare保留端点的真实含义;同一最终夹具在旧组件失败、候选通过Scope And Continuation
此显示修复在本范围内完成:一处既有 formatter、一份既有浏览器场景和八张合成截图。formatter 也供未缓存占比及缺失归因范围使用,复用一个 locale formatter,没有新增配置或状态。
实际归因数据、完整成本、真实模型语言质量和正式 App 采用仍由各自原任务验收。本提案不授予合并或安装权限,也不增加译文修订产品。
Validation
82a92767badcce9f2f213fd50a12b18517ed78c8,构建/浏览器检查使用与此 commit 相同的生产源码及最终夹具。0%断言失败,候选通过;基线失败保留,未补标通过LOOPX_PERSONAL_WORKSPACE_SCENARIO=research-history LOOPX_PERSONAL_WORKSPACE_PACKAGED=1 node examples/personal-workspace-browser-smoke.mjs,完整场景 233352ms,中英四种端点、精确计数、键盘、1280/390 和原场景故障恢复通过trajectory_input_usage.test.ts十项通过,覆盖缺失、零、部分、重复、缓存、错误归因和估计范围;owner 未改动build:chat、build:desktop、TypeScript 检查、项目范围公共扫描;CQRcqr_5e359c5f5020bb54bb0b已精确核验早期新增测试曾使用固定中文 selector,并误点击英文 summary 内的帮助按钮;修正为对应语言和键盘开合后,使用完整最终夹具重做基线/候选对照。首次检查记录格式也曾被 CQR 拒绝,最终精确回执有效。已保留这些失败;没有查询或等待 CI。
未运行真实模型、benchmark、正式 App 安装或使用验收;没有修改归因 owner、持久化协议或 authority,未新增 PostgreSQL 路径。构建保留既有大 chunk 警告。
Frontend / Visual Evidence
UI impact: changed. Source data: synthetic. Packaged chat bundle;未安装到正式 App。以下图像均来自上述测试,精度改动之外的布局保持。首屏主问题仍由研究结果、原生读数和六项判读回答;用量放在既有次级入口。
相同
20 / 1,000,000数据、相同键盘焦点状态,1280×900:390×844:
明细沿原入口打开,精确分子 20 与分母 1,000,000 分行可读,缓存归属未知仍为破折号。正文未缩小;方法标识留在明细底部。键盘焦点可见,数值和方法层级分开。
Attention / craft review: 默认研究结果保持主次,用量按需进入;此次只让非零和接近全部的读数可辨,不声称整站改版或实际用户理解已验证。原字体、间距、状态图标和帮助入口保持,没有新增卡片或动画。完整长内容、未知和失败读回仍由既有整页场景覆盖;截图不能证明真实归因准确或长期净效率收益。可用预览为本地可复现的打包测试;无公开部署链接。
Type And Boundary