Skip to content

fix: 隔离 Hook 性能门槛以稳定 CI - #5

Merged
z2z23n0 merged 1 commit into
mainfrom
fix/isolate-hook-performance-checks
Jul 17, 2026
Merged

fix: 隔离 Hook 性能门槛以稳定 CI#5
z2z23n0 merged 1 commit into
mainfrom
fix/isolate-hook-performance-checks

Conversation

@z2z23n0

@z2z23n0 z2z23n0 commented Jul 17, 2026

Copy link
Copy Markdown
Owner

Mainline Intent

Intent: int_4b75cf69
Status: proposed
Title: 隔离 Hook 性能门槛以稳定 CI

What changed

在共享包测试阶段显式跳过两条 Hook CLI 墙钟性能测试,并在其他包完成后由 scripts/ci.sh 单独、禁用测试缓存地执行它们;测试自身新增只供共享阶段使用的跳过开关。

Why

GitHub Ubuntu runner 在 go test ./... 的包级并行负载下把 Hook CLI + SQLite 的 p95 放大到 184.8ms,造成与业务改动无关的 CI 失败。隔离执行可以消除跨包资源争抢,同时继续执行原有 50ms 性能契约。

Decisions

  • 如何消除共享 runner 对 Hook 性能门槛的干扰: 把两条 Hook CLI 性能测试从并行全包阶段隔离出来,在共享 CI 入口中串行单独执行 (保留真实 50ms 门槛和功能测试并行度,同时避免其他重测试造成墙钟抖动)
    • Rejected: 直接放宽 50ms 阈值
    • Rejected: 删除或只在本地运行性能测试

Subsystems: cli, scripts

@z2z23n0
z2z23n0 marked this pull request as ready for review July 17, 2026 18:12
@z2z23n0
z2z23n0 merged commit 532b577 into main Jul 17, 2026
2 checks passed
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