From 85c2bd647523bba7139417af0f0cddc3d013fa82 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Pasternak?= Date: Thu, 6 Aug 2026 19:08:06 +0200 Subject: [PATCH] fix(ci): shellcheck SC2016 blokowal pre-commit od 381265d MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit CI na main jest czerwone od wczoraj 18:21 (ostatni zielony: a05f216). Pada wylacznie job "Pre-commit hooks" — pozostale piec (Testy Makefile na Linuksie/Windows/macOS, testy runtime WAF, walidacja compose) przechodzi. Przyczyna: asercja pozycji parsera-zaslepki dodana w 381265d uzywa wzorca grep-a z `\$__range`/`\$__interval`, ktore MAJA zostac literalne — dopasowuja tekst w JSON-ie Grafany. Pojedyncze cudzyslowy sa wiec celowe, a SC2016 ("Expressions don't expand in single quotes") to falszywy alarm. Rozwiniecie zepsuloby asercje. Wyciszenie w konwencji juz obecnej w tym pliku czterokrotnie (linie 351, 430, 455, 504): `# shellcheck disable=SC2016 # to WZORZEC grep-a: ... literalne`. Dlaczego nie zlapalo tego lokalnie ani na PR-ze: CI odpala `pre-commit run --all-files` (ci.yml:59), a lokalny hook chodzi tylko po plikach z commita. Cztery kolejne commity nie dotykaly test_makefile.sh, wiec dziedziczyly czerwone CI bez wlasnej winy. Zweryfikowane: symulacja CI (`shellcheck -e SC1091,SC2086`) czysta, asercja nadal DZIALA (PASS: parser-zaslepka w 11 z 11 zapytan + PASS na pozycji w potoku), pelny zestaw 225 passed / 0 failed. Co-Authored-By: Claude Opus 5 (1M context) Claude-Session: https://claude.ai/code/session_01J35fM3toviqNDsGxRg6P1c --- tests/test_makefile.sh | 1 + 1 file changed, 1 insertion(+) diff --git a/tests/test_makefile.sh b/tests/test_makefile.sh index f473b3f..8caf183 100755 --- a/tests/test_makefile.sh +++ b/tests/test_makefile.sh @@ -672,6 +672,7 @@ test_waf_crossfilter() { fi local zle_miejsce + # shellcheck disable=SC2016 # to WZORZEC grep-a: `$__range`/`$__interval` maja zostac literalne zle_miejsce="$(grep '"expr":' "$waf" \ | grep -cvE 'drop bpp_noop( \[\$__(range|interval)\])?[")]' || true)" if [ "$zle_miejsce" -eq 0 ]; then