Skip to content

Commit 0ae81b6

Browse files
Trecekclaude
andcommitted
fix(review): expand works_as_is_stems to cover all scripts per HookDef
works_as_is_stems used hd.scripts[0] only, missing non-first scripts of multi-script works-as-is hooks (e.g. mcp_run_skill has 5 scripts, dispatch_food_truck has 4). Probe rows for those scripts were recorded in the matrix but excluded from the coverage check, allowing gaps to pass silently. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 10c41b5 commit 0ae81b6

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

tests/execution/backends/test_hook_strength_matrix.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,10 @@ def test_works_as_is_hooks_have_soft_or_better() -> None:
5454
rows = matrix["combinations"]
5555

5656
works_as_is_stems: set[str] = {
57-
Path(hd.scripts[0]).stem for hd in HOOK_REGISTRY if hd.codex_status == "works-as-is"
57+
Path(script).stem
58+
for hd in HOOK_REGISTRY
59+
if hd.codex_status == "works-as-is"
60+
for script in hd.scripts
5861
}
5962
seen_stems: set[str] = set()
6063
for row in rows:

0 commit comments

Comments
 (0)