Skip to content

Keep known N+1 fixes on the fast path - #2

Merged
CCDawn merged 1 commit into
mainfrom
agent/keep-known-n-plus-one-in-check
Jul 17, 2026
Merged

Keep known N+1 fixes on the fast path#2
CCDawn merged 1 commit into
mainfrom
agent/keep-known-n-plus-one-in-check

Conversation

@CCDawn

@CCDawn CCDawn commented Jul 17, 2026

Copy link
Copy Markdown
Owner

修改内容

  • 将“已定位 N+1 + 明确批量修复 + 查询次数断言”固定为当前 owner 的 CHECK,避免性能 skill 抢占。
  • 收紧 ccdawn-performance-engineering 的 frontmatter 与 CHECK 边界,只在仍需 profiling 时触发。
  • 将 live routing case 改成更接近真实开发命令的表达。
  • 为 Windows Codex Desktop 增加一次性 Python runtime fallback,避免循环尝试失效 WindowsApps 别名。

实际问题

只读路由测试通过后,真实写入测试仍加载了性能 skill,并进行了多轮环境探测。首轮完成任务但累计输入约 434k tokens。第二轮修复路由后不再加载 skill,但因重复查找 Python 解释器而未完成验证。

前向测试

同一个临时 N+1 fixture、同一条开发提示词:

  • 首轮:读取性能 skill,约 20 次命令,约 434k 输入 tokens。
  • 最终轮:0 次 skill 读取,8 次命令,约 137k 输入 tokens。
  • 降幅约 68%,并实际完成批量查询修改、查询次数断言和单元测试。
  • Codex runtime Python 在首次 WindowsApps alias 失败后直接命中。

验证

  • py -3 scripts\validate_ccdawn_skills.py --warnings-as-errors
  • py -3 -m unittest discover -s tests -p "test_*.py" -q
  • .\install.ps1 -Agent codex-grok -VerifyOnly
  • 独立 Codex workspace-write fixture:1 test, OKgit diff --check 通过

@CCDawn
CCDawn marked this pull request as ready for review July 17, 2026 12:07
@CCDawn
CCDawn merged commit e93b0d5 into main Jul 17, 2026
1 check passed
@CCDawn
CCDawn deleted the agent/keep-known-n-plus-one-in-check branch July 17, 2026 12:07
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.

1 participant