From c309c95f5afb481cc2d1f2fd229ed9eb3b6ed331 Mon Sep 17 00:00:00 2001 From: Krzysztof kuhy Rudnicki Date: Sat, 22 Aug 2026 18:40:05 +0200 Subject: [PATCH] Clear bt_audio.sh and arch_perf_report.sh: allowlist 95 -> 93 Both were already tested to their reachable ceiling earlier in this session (a5b6663c, ffcf99f1) and held below 100% only by the closing-brace artifact that d989cd7e removed. With the instrument corrected they measure: bt_audio.sh 127/127 = 100.00% arch_perf_report.sh 59/59 = 100.00% both at --min 100 with exit 0, and check_shell_coverage.sh returns "every checked shell library meets the coverage bar" with their allowlist lines already gone, i.e. with no exemption. bt_report.sh and bt_pairing.sh were gated in the same run to confirm the two libs cleared earlier did not regress. Note that the closing-brace exclusion is now load-bearing: reverting d989cd7e without also restoring these two lines would turn every subsequent commit red. Co-Authored-By: Claude Opus 5 (1M context) --- ...-bt-audio-arch-perf-report-2026-08-22.json | 43 +++++++++++++++++++ meta/shell-coverage-allowlist.txt | 2 - 2 files changed, 43 insertions(+), 2 deletions(-) create mode 100644 docs/superpowers/evidence/clear-bt-audio-arch-perf-report-2026-08-22.json diff --git a/docs/superpowers/evidence/clear-bt-audio-arch-perf-report-2026-08-22.json b/docs/superpowers/evidence/clear-bt-audio-arch-perf-report-2026-08-22.json new file mode 100644 index 00000000..ec623230 --- /dev/null +++ b/docs/superpowers/evidence/clear-bt-audio-arch-perf-report-2026-08-22.json @@ -0,0 +1,43 @@ +{ + "intent": "Take bt_audio.sh and arch_perf_report.sh off the shell-coverage allowlist. Both were already tested to their reachable ceiling; the closing-brace fix in d989cd7e removed the instrument artifact that was holding them below 100%, so they now meet the bar with no exemption.", + "scope": [ + "meta/shell-coverage-allowlist.txt (two lines removed)", + "Non-goal: any test or lib change -- the tests landed in a5b6663c and ffcf99f1, the instrument fix in d989cd7e, and this commit only records the consequence", + "Non-goal: bt_adapter.sh, still exempt at 98.55% for the backslash-continued assignment flavour" + ], + "changes": [ + "Removed linux_configuration/scripts/single_use/fixes/lib/bt_audio.sh from meta/shell-coverage-allowlist.txt.", + "Removed linux_configuration/scripts/single_use/fixes/lib/arch_perf_report.sh from meta/shell-coverage-allowlist.txt.", + "Confirmed the diff removes exactly those two lines and nothing else." + ], + "verification": [ + { + "command": "bash meta/scripts/shell_coverage_jail.sh --subject linux_configuration/scripts/single_use/fixes/lib/tests/run_all.sh --measure bt_audio.sh --min 100 --fail-on-case-error -- \"\"", + "result": "pass", + "evidence": "bt_audio.sh: 127/127 lines = 100.00%, exit code 0. Measured at the --min 100 bar so a shortfall would be a non-zero exit rather than a number to misread." + }, + { + "command": "bash meta/scripts/shell_coverage_jail.sh --subject linux_configuration/scripts/single_use/fixes/lib/tests/run_all.sh --measure arch_perf_report.sh --min 100 --fail-on-case-error -- \"\"", + "result": "pass", + "evidence": "arch_perf_report.sh: 59/59 lines = 100.00%, exit code 0." + }, + { + "command": "bash meta/scripts/check_shell_coverage.sh on bt_audio.sh, arch_perf_report.sh, bt_report.sh and bt_pairing.sh", + "result": "pass", + "evidence": "'check_shell_coverage.sh: every checked shell library meets the coverage bar', exit 0, run with both allowlist lines already removed so neither lib had an exemption. bt_report.sh and bt_pairing.sh are included to confirm the two libs cleared earlier in the session did not regress." + }, + { + "command": "git diff meta/shell-coverage-allowlist.txt", + "result": "pass", + "evidence": "Exactly two removed lines, both the intended ones, and no additions. The allowlist drops from 95 to 93 fixes/lib-relevant entries." + } + ], + "risks": [ + "Both libs are now gated at 100% with no exemption, so any future change to them, to their tests, or to continuation_lines() that lowers their coverage will fail the commit. That is the point of clearing them, but it means the closing-brace exclusion is now load-bearing: reverting d989cd7e without also restoring these two allowlist lines would turn every subsequent commit red.", + "The measurements were made in a detached worktree under the scratchpad, because the jail aborts on any working-tree delta during a run. The libs and tests were byte-identical to the commits named above." + ], + "rollback": [ + "git revert the commit; that restores both allowlist lines and returns the two libs to exempt status.", + "After rollback, run bash meta/scripts/check_shell_coverage.sh on both libs and confirm it exits 0 with them exempt." + ] +} diff --git a/meta/shell-coverage-allowlist.txt b/meta/shell-coverage-allowlist.txt index 10308ea8..0271bc73 100644 --- a/meta/shell-coverage-allowlist.txt +++ b/meta/shell-coverage-allowlist.txt @@ -77,10 +77,8 @@ linux_configuration/scripts/single_use/fixes/lib/arch_cpu.sh linux_configuration/scripts/single_use/fixes/lib/arch_hardware.sh linux_configuration/scripts/single_use/fixes/lib/arch_perf_fixes.sh linux_configuration/scripts/single_use/fixes/lib/arch_perf_probes.sh -linux_configuration/scripts/single_use/fixes/lib/arch_perf_report.sh linux_configuration/scripts/single_use/fixes/lib/arch_sysctl.sh linux_configuration/scripts/single_use/fixes/lib/bt_adapter.sh -linux_configuration/scripts/single_use/fixes/lib/bt_audio.sh linux_configuration/scripts/single_use/fixes/lib/hosts_guard_migrate.sh linux_configuration/scripts/single_use/fixes/lib/hosts_guard_rollback.sh linux_configuration/scripts/single_use/fixes/lib/nvidia_config.sh