From 4dce953de05f89b4b4ce8b6a3aaa24a5f3dbff3d Mon Sep 17 00:00:00 2001 From: thekevinbot <248217931+thekevinbot@users.noreply.github.com> Date: Fri, 4 Sep 2026 17:21:40 -0400 Subject: [PATCH] ci: ratchet one-function-per-file to max_lines = 24 24 is the lowest uniform threshold green across every opted-in scan root today; internals/checks binds it, where preflight/gate.py's `run` is 24 lines. At 23 that file fails, so this is the floor, not a round number. Re-verified at 24 on this base with testing-conventions@0.0.111: all five roots exit 0, and a 23 probe fails as expected. --- internals/checks/testing-conventions.toml | 2 +- packages/python/testing-conventions.toml | 2 +- packages/ts/testing-conventions.toml | 2 +- plugins/dirsql-plugin-embeddings/testing-conventions.toml | 2 +- testing-conventions.toml | 4 ++-- 5 files changed, 6 insertions(+), 6 deletions(-) diff --git a/internals/checks/testing-conventions.toml b/internals/checks/testing-conventions.toml index 62a130eb8..783fe2d2b 100644 --- a/internals/checks/testing-conventions.toml +++ b/internals/checks/testing-conventions.toml @@ -15,7 +15,7 @@ # its own config drops that phantom root; a change to decide.py's path list still # demands a receipt through the normal --scope internals/checks/src walk. [python] -one_function_per_file = { max_lines = 100 } +one_function_per_file = { max_lines = 24 } [python.coverage] fail_under = 100 diff --git a/packages/python/testing-conventions.toml b/packages/python/testing-conventions.toml index 8cd029a15..af5dc0049 100644 --- a/packages/python/testing-conventions.toml +++ b/packages/python/testing-conventions.toml @@ -10,7 +10,7 @@ extra_scope = ["packages/rust/src", "packages/python/src"] [python] -one_function_per_file = { max_lines = 100 } +one_function_per_file = { max_lines = 24 } # Same floor as the root file, which this lane no longer reads. [python.coverage] diff --git a/packages/ts/testing-conventions.toml b/packages/ts/testing-conventions.toml index a353a5191..9fb60b33f 100644 --- a/packages/ts/testing-conventions.toml +++ b/packages/ts/testing-conventions.toml @@ -11,7 +11,7 @@ extra_scope = ["packages/rust/src", "packages/ts/napi"] [typescript] -one_function_per_file = { max_lines = 100 } +one_function_per_file = { max_lines = 24 } # Same floors as the root file, which this lane no longer reads. [typescript.coverage] diff --git a/plugins/dirsql-plugin-embeddings/testing-conventions.toml b/plugins/dirsql-plugin-embeddings/testing-conventions.toml index fdf5dd969..62de08320 100644 --- a/plugins/dirsql-plugin-embeddings/testing-conventions.toml +++ b/plugins/dirsql-plugin-embeddings/testing-conventions.toml @@ -3,7 +3,7 @@ # exemptions, and deliberately no `[e2e] extra_scope` -- this package compiles # none of packages/rust/src, so a core change never stales its attestation. [python] -one_function_per_file = { max_lines = 100 } +one_function_per_file = { max_lines = 24 } [python.coverage] fail_under = 100 diff --git a/testing-conventions.toml b/testing-conventions.toml index 7c4cea1a1..84defd620 100644 --- a/testing-conventions.toml +++ b/testing-conventions.toml @@ -80,7 +80,7 @@ extra_scope = ["packages/rust/src"] # --- Python (scanned: packages/python/dirsql) ------------------------------ [python] -one_function_per_file = { max_lines = 100 } +one_function_per_file = { max_lines = 24 } # Unit-only coverage floor (branch coverage on, matching the prior --cov-branch). # The default for a config-less python caller (internals/distcheck); the @@ -99,7 +99,7 @@ branch = true # build_command field), with `rust_toolchain: true` on the caller supplying cargo. [typescript] -one_function_per_file = { max_lines = 100 } +one_function_per_file = { max_lines = 24 } # Unit-only coverage floor, per vitest metric. [typescript.coverage]