From 2a2136a028bbaaf97a33f0c6baaa28badac6d675 Mon Sep 17 00:00:00 2001 From: Andrew Chang Date: Fri, 17 Jul 2026 22:14:06 +0800 Subject: [PATCH] Skip ts-sdk supervisor schema check for doc-only changes ts-sdk/*.md edits (e.g. README) currently match TS_SDK_FILES and keep the check-ts-sdk-supervisor-schema prek hook from being skipped, so a documentation-only change needlessly regenerates and diffs the generated supervisor schema. Documentation does not affect it. --- .../src/airflow_breeze/utils/selective_checks.py | 3 ++- dev/breeze/tests/test_selective_checks.py | 10 ++++++++++ 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/dev/breeze/src/airflow_breeze/utils/selective_checks.py b/dev/breeze/src/airflow_breeze/utils/selective_checks.py index 17e3fca8e2abb..f2794f7c49db7 100644 --- a/dev/breeze/src/airflow_breeze/utils/selective_checks.py +++ b/dev/breeze/src/airflow_breeze/utils/selective_checks.py @@ -459,7 +459,8 @@ def __hash__(self): r"^java-sdk/(?!.*\.md$).*", ], FileGroupForCi.TS_SDK_FILES: [ - r"^ts-sdk/", + # `.md` excluded — doc-only edits do not affect the generated supervisor schema. + r"^ts-sdk/(?!.*\.md$).*", ], FileGroupForCi.ASSET_FILES: [ r"^airflow-core/src/airflow/assets/", diff --git a/dev/breeze/tests/test_selective_checks.py b/dev/breeze/tests/test_selective_checks.py index 60371125ade90..2624ba84b3dd0 100644 --- a/dev/breeze/tests/test_selective_checks.py +++ b/dev/breeze/tests/test_selective_checks.py @@ -1761,6 +1761,16 @@ def test_ktlint_hook_only_runs_for_java_sdk_changes(files: tuple[str, ...], ktli True, id="skipped when no ts-sdk files change", ), + pytest.param( + ("ts-sdk/README.md",), + True, + id="skipped when only ts-sdk docs change", + ), + pytest.param( + ("ts-sdk/example/README.md",), + True, + id="skipped when only nested ts-sdk docs change", + ), ], ) def test_check_ts_sdk_supervisor_schema_hook_only_runs_for_relevant_changes(