From 3caf64312ff81a682e0c0bf8795762efc832bbd1 Mon Sep 17 00:00:00 2001 From: Jammy2211 Date: Thu, 23 Jul 2026 22:35:34 +0100 Subject: [PATCH] feat: enable --strict-declarations in the reusable smoke gate (PyAutoHands#187) The env-declaration migration merged in every workspace (PyAutoHands#188 + 5 migration PRs); all seven target repos validate 0-error under the flag. From here, a new pattern-override expressible as an in-file '# ENV:' declaration fails the PR gate instead of accumulating. Co-Authored-By: Claude Fable 5 Claude-Session: https://claude.ai/code/session_01MRQH5HrmMPfpWkmfh3ysJb --- .github/workflows/smoke-tests.yml | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/.github/workflows/smoke-tests.yml b/.github/workflows/smoke-tests.yml index 9191ea8..3f39dce 100644 --- a/.github/workflows/smoke-tests.yml +++ b/.github/workflows/smoke-tests.yml @@ -94,12 +94,16 @@ jobs: run: | # Fire if the workspace carries a smoke profile: the canonical # profile_smoke.yaml (PyAutoHands#161/#181 step 6 rename). + # --strict-declarations (PyAutoHands#187): pattern-overrides that are + # expressible as in-file '# ENV:' declarations are errors — the + # migration landed in every workspace, so new ones must not creep in. if [ -f workspace/config/build/profile_smoke.yaml ]; then if [ -f workspace/config/build/profile_release.yaml ]; then python PyAutoHands/autohands/validate_env_profiles.py workspace \ - --strict-derivation --strict-markers + --strict-derivation --strict-markers --strict-declarations else - python PyAutoHands/autohands/validate_env_profiles.py workspace + python PyAutoHands/autohands/validate_env_profiles.py workspace \ + --strict-declarations fi else echo "No config/build/profile_smoke.yaml — profile validation skipped."