Repository navigation
fix(context): apply native reader pages with their output budgets - #7421
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)
当前精确 head 6c01362d49f14616b9795084e0245c444f553089 没有阻塞 finding,结论 APPROVE。这是本轮独立执行的代码及原生路径核验;共享 GitHub 作者账户,按规则用 COMMENTED 承载结论。
动机
已明确启用原生差分上下文、需要完整读取任务要求并在压缩后继续工作的维护者和执行 Agent。
以前共享读取器虽支持较大输出,自动生成的 worker 仍沿用约 8 KB 的续读;现在同一原生配置同时提供分页、历史预算和逐次输出指引,首屏保持较小。
本轮真实原生源码与非 editable 安装包均完成完整读取、差分、压缩恢复和进程恢复;独立的 77,018 字节对照从 13 页降至 3 页,原始内容和停止条件完整保留。
本次只确认这条可选配置的源码与打包执行路径;不认证已安装 App 的启用、任意宿主版本、真实模型任务收益或完整费用,也不授予合并权限。
实际 App 配置应用、其他宿主版本和持续任务效果仍须各自验收;读取完整性的工程假设与既有恢复边界继续保留。
改动思路
复用原生配置生成器、现有分页器和摘要/生命周期校验;Python 只运输输出容量,worker 不再维护另一份配置,准入、身份与结算仍由原 owner 决定。
本 PR 把同一可选输出配置贯通设置结果、普通 guard 的续读、原捕获恢复和既有 worker;没有新增持久状态、用户开关或宿主安装器。
仅改分页大小仍可能丢失后续原生输出,因此本轮重点核验配置是否真正进入执行进程,以及恢复后的模型输入是否包含每份完整原文。给既有 worker 再加独立参数或设置 owner 没有必要;当前实现直接消费共享生成结果。输出容量声明只决定传输大小,不启用能力、不证明保留,也不能给出新执行权限。
| 条件 | 当前结果 |
|---|---|
| 未应用配置、未知或不支持的容量值 | 保持原小页及完整恢复指引 |
| 已应用生成的原生配置 | 续读上限 64,000 字节,原生历史上限 20,000 token;首屏仍不超过 7,900 字节 |
| 历史失效或原文摘要不符 | 恢复同一完整原捕获,或拒绝读取;不另行准入 |
这张表概括源码与实际验证的分支;它本身不是执行证据。
具体改动
完整差异 11 文件 +167/-41:4 个生产文件 +49/-16、4 个测试文件 +69/-14、3 个现有文档 +49/-11,没有新模块或持久状态。独立先读的接受依据是 docs/reference/required-work-context.md,spec_revision 99921829f2c5fa2ad35a024c72abb88717003ef3,尤其原生配置、完整读取、未知预算和同捕获恢复约定。未把本 PR 新改写的文字当作独立规格。
关键代码讲解
- native_context_page_budget 只接受精确受支持的容量声明;缺省、未知和其他数字仍回到旧上限。它是现有 Python 传输边界内的小 helper,不新增业务状态判断。
- codex_context_configuration 在原有 hooks 与恢复文字之外返回
config_overrides,复用原生设置名称;逐次读取建议 22,000 token,Code Mode 发射建议 24,000。配置数据仍须由原宿主路径应用和信任。 - native_context_capture_view 让首次输出和后续读取分别采用原小上限与已声明上限,原摘要、UTF-8 游标和恢复引用不变。普通 guard 与已保存投影继续消费同一实现。
- prepare_context_delivery 将共享配置追加为原生覆盖项;既有精确 hook 多重集合、哈希信任和 provider 保留检查未被替换。
条款映射(本次评审标签):complete-capture、native-setup、off-parity、same-capture-recovery 均 implemented。证据分别是完整字节对照、实际生成配置进入原生进程、关闭/未知值反事实,以及两种正文的压缩/进程恢复。实际 App 应用和持续模型收益仍是上文明确保留的范围。
对主干的风险
最强反例是“JSON 和配置都正确,原生工具输出或历史仍少了要求”。本轮在真实 Codex 0.163.0-alpha.11.1 上执行生成配置的两项源码 Code Mode 案例,以及非 editable 物理包的六项 worker 案例:覆盖普通 quota/TurnEnvelope、完整到差分、压缩恢复、跨进程恢复和被替换/不合资格历史的全量回退。推理只使用确定性 loopback;原生宿主、CLI 和隔离后端实际执行,逐字节模型输入 oracle、原摘要、两次原 guard 与原身份结算断言保持。物理包的 1,748 个 Python/TypeScript 成员与 Git、wheel、安装文件逐字相同。
独立配对将同一 77,018 字节的混合 UTF-8、JSON 转义和末尾停止条件交给不可变基线与当前版本。四种缺省/未知/不支持声明的完整输出相同;五种声明下的关闭态完整响应、错误摘要拒绝和默认恢复文字也相同。只替换已验证的实验目录前缀,未删除身份、诊断、命令、正文或控制字段。已支持配置从 13 页变为 3 页且原文完整;这是分页观察,不是费用或模型任务收益。基线 Python 3.14、当前 Python 3.12 的环境差异保留披露。
完整相关源码套件 351 通过/64 条件跳过,worker adapter 套件 110 通过/43 条件跳过;实际必需原生案例另行执行,未把 skip 当通过。首次单独运行 adapter 套件暴露 5 个共享测试 fixture 导入错误;只补齐测试导入路径后重跑整个模块,原断言未改,初次失败保留。首次 wheel 构建拒绝上一工作树版本的旧 Chat bundle,重建当前源码 bundle 后构建通过;未改产品源码来绕过校验。
仓库要求的 Ruff、变更文件 Ruff、配置内 mypy 17 文件、TypeScript typecheck、模块预算 6 项、双语/空白检查与 11 路径公开边界扫描通过。标准 premerge 的 6 个直接检查和 19 个选择检查通过;当前精确 diff 的质量收据已合格。初始总体门的旧收据失败保留,benchmark_sensitive 维护者评审门仍未解除;没有查询、轮询或等待远端 CI。
语义与 CI 对齐
复用既有 native 配置键、捕获页与生命周期词汇;新增容量声明不等于能力启用或可信保留。差异 advisory 未发现支持范围内的新词汇 carrier,这不是穷尽证明。新增文字逐条保留完整原文、固定摘要、未知历史、小页回退、失效事件、原身份和不得重新准入的义务;默认恢复文字与关闭态已配对,而非仅比较几个决策字段。回滚须移除生成的覆盖项并恢复宿主旧值;不需要新状态迁移或兼容解码分支。
我的整体评价
APPROVE,justified_increment。 long_horizon 为 preserved:完整任务与恢复承诺保持;user_experience 为 improved:既有设置结果与自动 worker 一致,不要求重复填身份或新增确认。共享 helper 和配置消费者的整理与问题相称,当前无需另建框架或追踪任务。完整差异没有渲染页面变化,视觉评审不适用;App 仍只输出配置数据,不能据此声称已安装启用。
没有阻塞 finding。剩余最强未验证范围为其他原生版本/平台、实际 App 应用、真实模型自主行为和持续完整成本。保留原生生命周期的工程假设;这些实测路径不构成任意模型输入保留证明。维护者门、精确 head readiness 与合并权限继续分别成立。
English verdict: APPROVE — 6c01362d49f14616b9795084e0245c444f553089. The existing opt-in setup now carries matching reader, history and emission budgets into the real native consumer. Two source and six physical worker cases pass full/delta/compaction/resume and invalid-history fallback; paired complete outputs preserve off/unknown defaults and every original byte. Required local checks pass. Native retention assumptions, installed App adoption, broader versions/platforms, full-cost benefit and maintainer merge gates remain separate qualifications.
|
This pull request has merge conflicts with Choose the remote for the base repository, not an out-of-date fork. 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 HEADFor a same-repository clone whose Keep the DCO |
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)
本次复审的精确 head 为 7c3d03eb86a0517e7ee3fbad89b72de01538b5a1。旧 6c01362d 的评审只属于原提交;本轮已合入当前相关主线,重新检查完整差异和执行路径,没有继承旧批准。
动机
已开启原生上下文续读、需要完整读到任务要求并在压缩或进程恢复后继续工作的维护者与执行 Agent。
以前共享读取器已支持较大输出,但自动 worker 仍按约 8 KB 续读;现在同一配置同时提供分页、原生历史和每次输出预算,首次提示仍保持较小。
本轮真实原生进程完成完整读取、差分、压缩恢复和进程恢复;相同 180,054 字节正文的配对验证从 35 页降到 5 页,原始内容与停止条件完整保留。
本次只确认可选配置的源码与原生 worker 路径,不认证已安装 App 的启用、任意宿主版本、真实模型任务收益或完整费用,也不修改信任和权限。
实际 App 配置应用、其他原生版本或平台、真实模型自主采用和持续完整成本仍须各自验收。
改动思路
共享配置和分页器已经是合适的 owner;本改动只把原生输出容量沿实际调用链贯通,不新增状态、安装器或第二份 worker 配置。
当前 PR 交付可选原生配置及已有隔离 worker 的完整读取/恢复路径;实际 App 应用、其他宿主版本与持续效果验收保留原有归属。
仅增大一页仍可能在原生历史或 Code Mode 输出处截断,所以同时检查配置是否进入原生进程、原始字节是否出现在后续模型输入。Python 保留已有 provider IO,通用准入、会话范围、完整读取与结算仍由原 typed owner 决定。没有新的持久事实、用户开关、安装器或第二份权限来源;也没有新增用户填参或确认轮次。
| 条件 | 验证结果 |
|---|---|
| 显式关闭,即使存在容量声明 | 五种声明的完整 CLI 响应与基线相同 |
| 缺省、未知、不支持容量或 7900 | 完整分页、错误与默认恢复文本与基线相同 |
| 已应用共享 64000 配置 | 同一原文从 35 页到 5 页,首屏仍不超过 7900 字节 |
| 原摘要错误、历史被替换或不合资格 | 拒绝复用或恢复原始全量;不重新准入 |
表格概括源码和实际验证分支,本身不是执行证据。相邻重构检查保留已有共享 helper 与配置消费者,进一步抽模块会增加无必要的维护面。
具体改动
依据先读的 docs/reference/required-work-context.md,spec_revision bb150595664fe0e910db9587cff22ea3b471f5ee,逐项检查 complete-capture、native-setup、off-parity、same-capture-recovery:完整原文与摘要、原生配置的信任和生命周期边界、关闭态隔离、同一原捕获恢复。上述标签是本次条款映射,不把新改写的文档当独立 oracle。可选配置下的大输出是明确的行为增量,原小页默认和硬性完整读取义务保留。
完整差异仍为 11 文件 +167/-41:4 个生产文件 +49/-16,4 个测试文件 +69/-14,3 个现有文档 +49/-11。两次主线整合保留了延迟 hook 发现并不等于 SessionStart/实际采用,以及恢复时不得假设工具缓存仍在的约束;没有把新主线修复抹掉。
- native_context_page_budget 只识别精确支持的容量声明,缺省和其他值回到原有小页。它不启用能力、不证明模型保留,也不是控制面状态分类。
- codex_context_configuration 在现有 hooks 和恢复指引之外返回
config_overrides,复用原生设置名:20,000 token 历史与 64,000 字节续读;逐次读取/Code Mode 分别说明 22,000/24,000 token 预算。宿主仍负责应用和信任。 - native_context_capture_view 把首次提示与后续读取分开限额,普通 guard、已保存投影和原文恢复继续共用原摘要、UTF-8 游标和读取器。
- prepare_context_delivery 只把同一设置追加到已有隔离 worker;精确 hook 多重集合、原生哈希信任、provider 和认证边界不变。
文档明确如何应用、验证与移除覆盖项,并保留加载中线程不可凭发现结果推断采用的警告。没有改产品页面、首屏或展示布局;App 仍只是返回配置,不能据此声称已安装启用。
对主干的风险
最强反例是配置和 JSON 正确,但原生工具输出或历史仍丢掉最后一条要求。本轮在实际 Codex 0.163.0-alpha.11.1 执行 8 项案例:两种正文的生成配置 → 完整/差分 → 压缩恢复 → 进程恢复,四种历史失效回退控制,以及两项已加载线程的晚发现 hook 边界。它们使用确定性 loopback 推理,实际 CLI、宿主和隔离后端执行;完整模型输入字节、原摘要、停止条件、原身份与无重复准入断言未删。
与不可变基线配对时,同一 180,054 字节混合 UTF-8/JSON 转义正文使用完全相同的物理捕获路径,未做输出归一化。五种声明下的完整关闭响应、错误摘要拒绝和默认恢复文本相同;四种未支持大输出的完整分页响应也相同。支持配置的 35 → 5 页逐字节重建原文;这是输出分页观察,不是费用、延迟或真实任务收益证明。
最终 head 的完整相关套件 322 通过/76 条件跳过,完整 worker adapter 模块 110 通过/48 条件跳过。必需的 8 项原生验证另行实跑,不把宽套件的 skip 当作通过。最初独立运行 adapter 缺根测试 fixture 的导入路径,补齐测试路径后重跑完整模块通过;断言和产品源码未为此修改。旧 head 的安装包与配对回执保持旧来源,本轮不冒充最终安装包或发布资格。
仓库规定范围的 Ruff、全部变更 Python 的 Ruff、配置内 mypy 17 文件与控制面 TypeScript 类型检查通过;双语、空白、模块预算、完整语义检查和 11 路径公开边界检查通过。标准 premerge 的 6 个直接检查 +19 个选择检查均通过,零自动失败或警告;精确最终差异的严格 CQR 有效。
额外扩大的全 loopx Ruff 仍有 3 项失败:content_ops/cli.py 两项 E731、native_chat/codex_auth.py 一项 F401。它们不在本 diff 与仓库规定 lint 根内;不可变基线和当前 head 的完整诊断、相关源码逐字相同。CQR 保留 failed 并记录独立基线归因,未把它们改成通过或缩窄规定检查。
原始 premerge 的 benchmark_sensitive 人工分类继续保留,不宣称机器自动放行。本批已有维护者针对 #7421 的明确自合并授权;按仓库既有 helper/runtime 例外逐项核对,本 diff 只运输可选原生输出配置,不改评分、任务语义、leaderboard/submission 或权限,也不启动 benchmark 作业。当前公开 exact-head 评审和对应验证完成后,才能进入即时 readiness 和授权 admin 合并。没有查询、轮询或等待 PR CI。
语义上复用既有 native 设置名、捕获页与生命周期;diff advisory 未发现支持范围的新词汇 carrier,这不是穷尽证明。逐条比较生成文字,保留原捕获优先、完整字节、固定摘要、未知历史、小页回退、失效事件、原身份与不得重新准入;增加的只是显式配置及读取预算。回滚同时移除生成覆盖项并恢复宿主旧值。没有把容量声明、可用性或发现结果当权限、启用或保留证明。
我的整体评价
APPROVE,justified_increment。 共享配置和分页器已经是合适的 owner;本改动只把原生输出容量沿实际调用链贯通,不新增状态、安装器或第二份 worker 配置。 当前 PR 交付可选原生配置及已有隔离 worker 的完整读取/恢复路径;实际 App 应用、其他宿主版本与持续效果验收保留原有归属。
本轮独立的当前源码和原生执行证据支持这个有用切片,关闭态及同捕获恢复没有退化。真实模型自主行为、其他版本/平台、安装中 App 的应用与完整成本仍保持未验证;过程完整和页数下降不等于科学效果。视觉评审:完整差异仅改变原生配置、CLI 与既有文档文字,未修改 apps/**、产品首屏或渲染布局,因此本切片无需界面截图。 本切片没有渲染变更;实际 App 配置应用仍未验证。 当前没有阻塞 finding。原始人工分类、维护者授权、公开评审与即时精确提交 readiness 是分别核对的依据,不能相互替代。
English verdict: APPROVE — 7c3d03eb86a0517e7ee3fbad89b72de01538b5a1. Shared opt-in native setup now carries matching reader/history/emission budgets into the existing worker. The current whole diff is reviewed after integrating main; 322 focused and110 adapter tests pass, with76/48 conditional skips retained. Eight real native cases pass full/delta/compaction/resume, invalid-history and late-hook controls. Complete paired180054-byte outputs preserve off/unknown defaults; supported continuations fall35→5 pages. Required local checks and exact-scope CQR pass. Three unrelated expanded-lint baseline failures stay failed; benchmark-sensitive classification and final authorized merge readiness remain separate. Installed App, broader hosts, autonomous model adoption and full-cost benefit remain unqualified.
The existing opt-in native worker used small continuations even though the shared reader supports larger bounded outputs. Apply one shared profile for 64,000-byte continuations, 20,000-token native history and matching per-read instructions. Page zero stays within 7,900 bytes; unconfigured hosts and explicit off retain their original paths.
The worker consumes
config_overridesfrom the existingquota context-configrenderer. Immutable captures, complete byte coverage, UTF-8 cursors, digests, exact native hook trust and scoped recovery retain their current owners. Setup does not apply configuration to an already loaded App thread or grant provider, credential or execution authority. Rollback removes generated output overrides and restores previous host values.Validated head:
7c3d03eb86a0517e7ee3fbad89b72de01538b5a1, integrated with main atbb150595664fe0e910db9587cff22ea3b471f5ee.content_ops/cli.py, one F401 innative_chat/codex_auth.py). Full base/head diagnostics and causal source bytes match. Required roots and changed paths pass; the optional observation remains failed with independent baseline attribution.benchmark_sensitivemanual classification is retained. This batch is explicitly authorized for self-merge and meets the repository's public helper/runtime exception: no scoring, task-semantics, leaderboard/submission, permission or benchmark-job launch change. Published exact-head review, approval closeout and immediate merge readiness remain separate gates. PR CI is not consulted under the configured policy.The bounded refactoring keeps one provider sizing helper and shared native configuration consumer. No new state owner, module, user setting or protocol version is introduced. Actual installed App activation, other native versions/platforms, autonomous model adoption and complete task/cost benefit remain unqualified. Earlier wheel and native evidence stays bound to its original commit; this head does not inherit that packaged qualification.
已有原生 worker 在开启上下文续读后仍使用小页。本改动把 64,000 字节续读、20,000 token 原生历史及逐次输出指引放进同一生成配置;首屏仍不超过 7,900 字节,未应用配置和关闭态保留原路径。原捕获、摘要、完整字节、精确 hook 信任、会话范围和同捕获恢复不变;已加载 App 线程并不会因配置已发现而自动采用。
最终 head
7c3d03eb86a0517e7ee3fbad89b72de01538b5a1:完整相关模块 322 通过/76 条件跳过,完整 adapter 模块 110 通过/48 条件跳过,8 项真实原生进程验证另行通过。相同 180,054 字节正文在支持配置下从 35 页变为 5 页,原文完整,关闭与未知容量的完整输出和基线相同,未做输出归一化。规定 lint、类型、双语、语义、预算、公开边界和 25 项 premerge 自动检查通过,严格 CQR 有效。额外扩大的 lint 仍保留 3 项无关基线失败及独立归因;原
benchmark_sensitive人工分类保留,按维护者对本批的明确自合并授权与仓库 helper/runtime 例外分别处理。未查询 PR CI,不宣称自动门全面放行。实际 App 应用、其他宿主、真实模型自主行为和完整效果/成本仍须各自验收;历史 wheel 证据继续只绑定原提交。