Skip to content

test(quota): verify fallback wait outside the summary window - #7648

Merged
huangruiteng merged 1 commit into
mainfrom
codex/release220-wait-admission-regression-20261029
Oct 29, 2026
Merged

huangruiteng merged 1 commit into
mainfrom
codex/release220-wait-admission-regression-20261029

Conversation

@loopx-agent

Copy link
Copy Markdown
Collaborator

两个等待准入回归用例把后置 payload_compaction.omitted_lanes 的计数当作测试前提。deferred_items 已在上游摘要中限量时,该字段可以不存在,实际等待判断仍然正确。

改为检查完整 deferred 数量、摘要确实截断、声明的 fallback 不在可见窗口,再沿用实际 CLI 的等待断言。保留缺失 fallback、权威读取失败和依赖完成等反例;运行时没有变化。

验证:修正前 2 failed / 28 passed,修正后整模块 30 passed;Ruff 通过;标准 premerge 的 5 项直接检查通过,catalog 未选择额外 smoke。未重新运行全量发布资格,其他发布失败项仍待处理。

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)

动机 / Motivation

维护者运行完整等待准入回归时,会遇到两项与实际决策无关的失败。8 或 20 个填充任务把后备任务挤出摘要窗口;旧断言读取不存在的压缩计数而失败,新断言直接验证窗口之外的任务仍按真实依赖等待。相同实际 CLI 用例由 2 failed、28 passed 变为 30 passed,原有等待决策及反例断言保留。本 PR 不修改运行时、配额、调度规则或发布门槛,也不宣称整个版本已通过验收。不修改时,这两项误报会继续遮蔽真正的等待准入回归。

改动思路 / Approach

沿用现有真实 CLI 和 canonical SQLite fixture。摘要只负责展示,fallback 的依赖判断仍由同一权威源和 TypeScript owner 决定;新断言要求总数完整、可见集合有界,而且 fallback 确实不可见。只删除旧检查会丢掉这层语义压力,要求所有记录可见又会违反摘要边界,因此直接修原有断言最合适。

具体改动 / Changes

审阅精确 head 4cc1e45fd56d8277d6b535541e7d2aa502facb21,只改 tests/control_plane/test_fallback_wait_admission.py 的容量测试(+7/-6)。规范依据为 docs/development/good-smokes.md,spec_revision bcb1a8710844bda6b84c2a00cf5d86ba78e126d9,采用变更前已接受的规范。

  • Prefer Semantic Pressure:implemented。deferred_count、可见集合及 fallback 窗口位置共同证明容量压力,原有等待结论断言保留。
  • Consolidate Without Losing Coverage:implemented。仍在原 CLI 模块内验证,没有新增 wrapper 或重复 fixture;缺失、读取故障和依赖完成的反例均保留。
  • Public-Safe Fixtures:implemented。沿用隔离目录和合成 Todo,提交中不含凭证、私有链接或本机路径。

改动只使用现有字段,不新增状态词汇、协议或生产兼容分支。现有规则级测试不能替代这里的 CLI/SQLite 集成边界;近期同账号 8 个 PR 的有界检查未见相同容量修补的重复提交。

对主干的风险 / Risks

无阻断发现。最强反例是测试变绿但 fallback 实际仍在可见窗口;新不可见性断言会使这个情况失败。缺失 fallback 必须报 unresolved、读取失败必须报 lookup_uncertain、依赖完成必须恢复可执行,整模块均保留这些检查。合成测试不能证明线上长期效果或全部发布资格。

PASS:基线整模块 2 failed / 28 passed(两处 KeyError);本 head 30 passed;Ruff 通过。PASS:标准 premerge 5 项直接检查,catalog 额外 smoke 为 0。按当前策略未查询或等待 GitHub CI;没有将其他发布失败重标为通过。

我的整体评价 / Verdict

此处修正既有回归 oracle 的任务达到 goal_achieved,不代表整个版本已通过发布验收。long_horizon 为 preserved:实际等待和恢复路径未改;user_experience 为 not_applicable:仅测试文件变动,没有用户入口变化。视觉检查不适用,未验证的渲染范围为 none。

单文件修正可以独立验证和回退;在现有函数内替换检查已足够,相邻生产 refactor 不必要。残余风险是未来摘要窗口扩大时需要增加合成压力;应保持边界测试,不能为了通过而删断言。合并权限和发布资格仍是独立门槛。

English verdict: APPROVE — 4cc1e45; no blocking finding. The 30-case real CLI module, Ruff and five direct premerge checks pass; full release qualification remains separate.

@huangruiteng
huangruiteng merged commit ac56fb2 into main Oct 29, 2026
3 of 4 checks passed
@huangruiteng
huangruiteng deleted the codex/release220-wait-admission-regression-20261029 branch October 29, 2026 09:04
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.

2 participants