Repository navigation
test(quota): verify fallback wait outside the summary window - #7648
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; 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.
两个等待准入回归用例把后置
payload_compaction.omitted_lanes的计数当作测试前提。deferred_items已在上游摘要中限量时,该字段可以不存在,实际等待判断仍然正确。改为检查完整 deferred 数量、摘要确实截断、声明的 fallback 不在可见窗口,再沿用实际 CLI 的等待断言。保留缺失 fallback、权威读取失败和依赖完成等反例;运行时没有变化。
验证:修正前 2 failed / 28 passed,修正后整模块 30 passed;Ruff 通过;标准 premerge 的 5 项直接检查通过,catalog 未选择额外 smoke。未重新运行全量发布资格,其他发布失败项仍待处理。