From 22f4adbb4514dc1a03c69c7b63329bb75ccafe12 Mon Sep 17 00:00:00 2001 From: "J. Nick Koston" Date: Fri, 28 Aug 2026 20:44:53 -0500 Subject: [PATCH] Keep espidf-only infra out of the esp8266 native trigger set The dev merge moved the ESP-IDF trigger constants into clang_tidy_hash; the esp8266 set keeps its explicit shared-module list with a subset tripwire against the imported set instead of unioning it wholesale. --- script/determine-jobs.py | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/script/determine-jobs.py b/script/determine-jobs.py index 8438b168b9..6de6c91961 100755 --- a/script/determine-jobs.py +++ b/script/determine-jobs.py @@ -654,10 +654,20 @@ ESP8266_NATIVE_TRIGGER_PATH_PREFIXES = ( "esphome/arduino/", "esphome/build_helpers/", ) -# Unions the ESP-IDF infra set (one source of truth in clang_tidy_hash) so -# a shared native-build module change never skips this smoke test either +# Shared library-conversion modules every native build imports; espidf-only +# infra (build_gen/espidf.py) deliberately stays out of the esp8266 set. +_NATIVE_SHARED_TRIGGER_FILES = frozenset( + { + "esphome/framework_helpers.py", + "esphome/platformio/library.py", + "esphome/platformio/extra_script.py", + } +) +# Tripwire: the shared modules must stay in the ESP-IDF trigger set too +# (now defined in clang_tidy_hash), or its smoke test silently skips them +assert _NATIVE_SHARED_TRIGGER_FILES <= ESP_IDF_INFRA_TRIGGER_FILES ESP8266_NATIVE_TRIGGER_FILES = ( - ESP_IDF_INFRA_TRIGGER_FILES + _NATIVE_SHARED_TRIGGER_FILES | _SMOKE_HARNESS_TRIGGER_FILES | { "esphome/build_gen/arduino8266.py",