This is an automated email from the ASF dual-hosted git repository.
pierrejeambrun pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 0022f42bcf9 Fix ts-compile-lint-simple-auth-manager-ui prek hook
(#64310)
0022f42bcf9 is described below
commit 0022f42bcf9487f4d66c8e9ad40c5acc26ac2c2b
Author: Pierre Jeambrun <[email protected]>
AuthorDate: Mon Mar 30 18:27:36 2026 +0200
Fix ts-compile-lint-simple-auth-manager-ui prek hook (#64310)
* Fix ts-compile-lint-simple-auth-manager-ui prek hook
The hook was silently failing due to two issues:
- Used AIRFLOW_ROOT_PATH instead of AIRFLOW_CORE_ROOT_PATH for relative
path computation, causing file matching to fail silently.
- The minimatch pnpm override forced all minimatch to v10+, breaking
eslint-plugin-react which requires minimatch v3 (default export as
function). Aligned overrides with the main UI's version-range-aware
approach.
* Fix static checks
---
.../auth/managers/simple/ui/package.json | 4 +-
.../auth/managers/simple/ui/pnpm-lock.yaml | 67 +++++++++++++++-------
.../managers/simple/ui/src/alert/ErrorAlert.tsx | 1 +
.../prek/ts_compile_lint_simple_auth_manager_ui.py | 4 +-
4 files changed, 52 insertions(+), 24 deletions(-)
diff --git
a/airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/package.json
b/airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/package.json
index e1746be7daa..d7e59083611 100644
--- a/airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/package.json
+++ b/airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/package.json
@@ -72,7 +72,9 @@
"overrides": {
"tar@<7.5.11": ">=7.5.11",
"lodash-es@>=4.0.0 <=4.17.22": ">=4.17.23",
- "minimatch@<10.2.3": ">=10.2.3",
+ "minimatch@>=3.0.0 <3.1.4": ">=3.1.4 <4.0.0",
+ "minimatch@>=9.0.0 <9.0.7": ">=9.0.7 <10.0.0",
+ "minimatch@>=10.0.0 <10.2.3": ">=10.2.3",
"ajv@<6.14.0": ">=6.14.0",
"rollup@>=4.0.0 <4.59.0": ">=4.59.0",
"flatted@<=3.4.1": ">=3.4.2",
diff --git
a/airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/pnpm-lock.yaml
b/airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/pnpm-lock.yaml
index 2a5fdd24f58..14af0395ca9 100644
---
a/airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/pnpm-lock.yaml
+++
b/airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/pnpm-lock.yaml
@@ -7,7 +7,9 @@ settings:
overrides:
tar@<7.5.11: '>=7.5.11'
lodash-es@>=4.0.0 <=4.17.22: '>=4.17.23'
- minimatch@<10.2.3: '>=10.2.3'
+ minimatch@>=3.0.0 <3.1.4: '>=3.1.4 <4.0.0'
+ minimatch@>=9.0.0 <9.0.7: '>=9.0.7 <10.0.0'
+ minimatch@>=10.0.0 <10.2.3: '>=10.2.3'
ajv@<6.14.0: '>=6.14.0'
rollup@>=4.0.0 <4.59.0: '>=4.59.0'
flatted@<=3.4.1: '>=3.4.2'
@@ -695,79 +697,66 @@ packages:
resolution: {integrity:
sha512-t4ONHboXi/3E0rT6OZl1pKbl2Vgxf9vJfWgmUoCEVQVxhW6Cw/c8I6hbbu7DAvgp82RKiH7TpLwxnJeKv2pbsw==}
cpu: [arm]
os: [linux]
- libc: [glibc]
'@rollup/[email protected]':
resolution: {integrity:
sha512-CikFT7aYPA2ufMD086cVORBYGHffBo4K8MQ4uPS/ZnY54GKj36i196u8U+aDVT2LX4eSMbyHtyOh7D7Zvk2VvA==}
cpu: [arm]
os: [linux]
- libc: [musl]
'@rollup/[email protected]':
resolution: {integrity:
sha512-jYgUGk5aLd1nUb1CtQ8E+t5JhLc9x5WdBKew9ZgAXg7DBk0ZHErLHdXM24rfX+bKrFe+Xp5YuJo54I5HFjGDAA==}
cpu: [arm64]
os: [linux]
- libc: [glibc]
'@rollup/[email protected]':
resolution: {integrity:
sha512-peZRVEdnFWZ5Bh2KeumKG9ty7aCXzzEsHShOZEFiCQlDEepP1dpUl/SrUNXNg13UmZl+gzVDPsiCwnV1uI0RUA==}
cpu: [arm64]
os: [linux]
- libc: [musl]
'@rollup/[email protected]':
resolution: {integrity:
sha512-gbUSW/97f7+r4gHy3Jlup8zDG190AuodsWnNiXErp9mT90iCy9NKKU0Xwx5k8VlRAIV2uU9CsMnEFg/xXaOfXg==}
cpu: [loong64]
os: [linux]
- libc: [glibc]
'@rollup/[email protected]':
resolution: {integrity:
sha512-yTRONe79E+o0FWFijasoTjtzG9EBedFXJMl888NBEDCDV9I2wGbFFfJQQe63OijbFCUZqxpHz1GzpbtSFikJ4Q==}
cpu: [loong64]
os: [linux]
- libc: [musl]
'@rollup/[email protected]':
resolution: {integrity:
sha512-sw1o3tfyk12k3OEpRddF68a1unZ5VCN7zoTNtSn2KndUE+ea3m3ROOKRCZxEpmT9nsGnogpFP9x6mnLTCaoLkA==}
cpu: [ppc64]
os: [linux]
- libc: [glibc]
'@rollup/[email protected]':
resolution: {integrity:
sha512-+2kLtQ4xT3AiIxkzFVFXfsmlZiG5FXYW7ZyIIvGA7Bdeuh9Z0aN4hVyXS/G1E9bTP/vqszNIN/pUKCk/BTHsKA==}
cpu: [ppc64]
os: [linux]
- libc: [musl]
'@rollup/[email protected]':
resolution: {integrity:
sha512-NDYMpsXYJJaj+I7UdwIuHHNxXZ/b/N2hR15NyH3m2qAtb/hHPA4g4SuuvrdxetTdndfj9b1WOmy73kcPRoERUg==}
cpu: [riscv64]
os: [linux]
- libc: [glibc]
'@rollup/[email protected]':
resolution: {integrity:
sha512-nLckB8WOqHIf1bhymk+oHxvM9D3tyPndZH8i8+35p/1YiVoVswPid2yLzgX7ZJP0KQvnkhM4H6QZ5m0LzbyIAg==}
cpu: [riscv64]
os: [linux]
- libc: [musl]
'@rollup/[email protected]':
resolution: {integrity:
sha512-oF87Ie3uAIvORFBpwnCvUzdeYUqi2wY6jRFWJAy1qus/udHFYIkplYRW+wo+GRUP4sKzYdmE1Y3+rY5Gc4ZO+w==}
cpu: [s390x]
os: [linux]
- libc: [glibc]
'@rollup/[email protected]':
resolution: {integrity:
sha512-3AHmtQq/ppNuUspKAlvA8HtLybkDflkMuLK4DPo77DfthRb71V84/c4MlWJXixZz4uruIH4uaa07IqoAkG64fg==}
cpu: [x64]
os: [linux]
- libc: [glibc]
'@rollup/[email protected]':
resolution: {integrity:
sha512-2UdiwS/9cTAx7qIUZB/fWtToJwvt0Vbo0zmnYt7ED35KPg13Q0ym1g442THLC7VyI6JfYTP4PiSOWyoMdV2/xg==}
cpu: [x64]
os: [linux]
- libc: [musl]
'@rollup/[email protected]':
resolution: {integrity:
sha512-M3bLRAVk6GOwFlPTIxVBSYKUaqfLrn8l0psKinkCFxl4lQvOSz8ZrKDz2gxcBwHFpci0B6rttydI4IpS4IS/jQ==}
@@ -831,28 +820,24 @@ packages:
engines: {node: '>=10'}
cpu: [arm64]
os: [linux]
- libc: [glibc]
'@swc/[email protected]':
resolution: {integrity:
sha512-0a+Lix+FSSHBSBOA0XznCcHo5/1nA6oLLjcnocvzXeqtdjnPb+SvchItHI+lfeiuj1sClYPDvPMLSLyXFaiIKw==}
engines: {node: '>=10'}
cpu: [arm64]
os: [linux]
- libc: [musl]
'@swc/[email protected]':
resolution: {integrity:
sha512-wG9J8vReUlpaHz4KOD/5UE1AUgirimU4UFT9oZmupUDEofxJKYb1mTA/DrMj0s78bkBiNI+7Fo2EgPuvOJfuAA==}
engines: {node: '>=10'}
cpu: [x64]
os: [linux]
- libc: [glibc]
'@swc/[email protected]':
resolution: {integrity:
sha512-4nwbVvCphKzicwNWRmvD5iBaZj8JYsRGa4xOxJmOyHlMDpsvvJ2OR2cODlvWyGFH6BYL1MfIAK3qph3hp0Az6g==}
engines: {node: '>=10'}
cpu: [x64]
os: [linux]
- libc: [musl]
'@swc/[email protected]':
resolution: {integrity:
sha512-zk0RYO+LjiBCat2RTMHzAWaMky0cra9loH4oRrLKLLNuL+jarxKLFDA8xTZWEkCPLjUTwlRN7d28eDLLMgtUcQ==}
@@ -1423,6 +1408,9 @@ packages:
resolution: {integrity:
sha512-Cg7TFGpIr01vOQNODXOOaGz2NpCU5gl8x1qJFbb6hbZxR7XrcE2vtbAsTAbJ7/xwJtUuJEw8K8Zr/AE0LHlesg==}
engines: {node: '>=10', npm: '>=6'}
+ [email protected]:
+ resolution: {integrity:
sha512-3oSeUO0TMV67hN1AmbXsK4yaqU7tjiHlbxRDZOpH0KW9+CeX4bRAaX0Anxt0tx2MrpRpWwQaPwIlISEJhYU5Pw==}
+
[email protected]:
resolution: {integrity:
sha512-BLrgEcRTwX2o6gGxGOCNyMvGSp35YofuYzw9h1IMTRmKqttAZZVU67bdb9Pr2vUHA8+j3i2tJfjO6C6+4myGTA==}
engines: {node: 18 || 20 || >=22}
@@ -1431,6 +1419,12 @@ packages:
resolution: {integrity:
sha512-ipDqC8FrAl/76p2SSWKSI+H9tFwm7vYqXQrItCuiVPt26Km0jS+NzSsBWAaBusvSbQcfJG+JitdMm+wZAgTYqg==}
hasBin: true
+ [email protected]:
+ resolution: {integrity:
sha512-9ZLprWS6EENmhEOpjCYW2c8VkmOvckIJZfkr7rBW6dObmfgJ/L1GpSYW5Hpo9lDz4D1+n0Ckz8rU7FwHDQiG/w==}
+
+ [email protected]:
+ resolution: {integrity:
sha512-MCV/fYJEbqx68aE58kv2cA/kiky1G8vux3OR6/jbS+jIMe/6fJWa0DTzJU7dqijOWYwHi1t29FlfYI9uytqlpA==}
+
[email protected]:
resolution: {integrity:
sha512-VZznLgtwhn+Mact9tfiwx64fA9erHH/MCXEUfB/0bX/6Fz6ny5EGTXYltMocqg4xFAQZtnO3DHWWXi8RiuN7cQ==}
engines: {node: 18 || 20 || >=22}
@@ -1526,6 +1520,9 @@ packages:
resolution: {integrity:
sha512-H+y0Jo/T1RZ9qPP4Eh1pkcQcLRglraJaSLoyOtHxu6AapkjWVCy2Sit1QQ4x3Dng8qDlSsZEet7g5Pq06MvTgw==}
engines: {node: '>=20'}
+ [email protected]:
+ resolution: {integrity:
sha512-/Srv4dswyQNBfohGpz9o6Yb3Gz3SrUDqBH5rTuhGR7ahtlbYKnVxw2bCFMRljaA7EXHaXZ8wsHdodFvbkhKmqg==}
+
[email protected]:
resolution: {integrity:
sha512-ysOGlgTFbN2/Y6Cg3Iye8YKulHw+R2fNXHrgSmXISQdMnomY6eNDprVdW9R5xBguEqI954+S6709UyiO7B+6OQ==}
@@ -2258,6 +2255,13 @@ packages:
resolution: {integrity:
sha512-oRjTw/97aTBN0RHbYCdtF1MQfvusSIBQM0IZEgzl6426+8jSC0nF1a/GmnVLpfB9yyr6g6FTqWqiZVbxrtaCIg==}
engines: {node: 18 || 20 || >=22}
+ [email protected]:
+ resolution: {integrity:
sha512-VgjWUsnnT6n+NUk6eZq77zeFdpW2LWDzP6zFGrCbHXiYNul5Dzqk2HHQ5uFH2DNW5Xbp8+jVzaeNt94ssEEl4w==}
+
+ [email protected]:
+ resolution: {integrity:
sha512-OBwBN9AL4dqmETlpS2zasx+vTeWclWzkblfZk7KTA5j3jeOONz/tRCnZomUyvNg83wL5Zv9Ss6HMJXAgL8R2Yg==}
+ engines: {node: '>=16 || 14 >=14.17'}
+
[email protected]:
resolution: {integrity:
sha512-6FlzubTLZG3J2a/NVCAleEhjzq5oxgHyaCU9yYXvcLsvoVaHJq/s5xXI6/XXP6tz7R9xAOtHnSO/tXtF3WRTlA==}
@@ -3693,7 +3697,7 @@ snapshots:
'@babel/types': 7.28.6
javascript-natural-sort: 0.7.1
lodash-es: 4.17.23
- minimatch: 10.2.4
+ minimatch: 9.0.9
parse-imports-exports: 0.2.4
prettier: 3.8.1
transitivePeerDependencies:
@@ -4579,10 +4583,21 @@ snapshots:
cosmiconfig: 7.1.0
resolve: 1.22.11
+ [email protected]: {}
+
[email protected]: {}
[email protected]: {}
+ [email protected]:
+ dependencies:
+ balanced-match: 1.0.2
+ concat-map: 0.0.1
+
+ [email protected]:
+ dependencies:
+ balanced-match: 1.0.2
+
[email protected]:
dependencies:
balanced-match: 4.0.4
@@ -4680,6 +4695,8 @@ snapshots:
[email protected]: {}
+ [email protected]: {}
+
[email protected]: {}
[email protected]: {}
@@ -4948,7 +4965,7 @@ snapshots:
hasown: 2.0.2
jsx-ast-utils: 3.3.5
language-tags: 1.0.9
- minimatch: 10.2.4
+ minimatch: 3.1.5
object.fromentries: 2.0.8
safe-regex-test: 1.1.0
string.prototype.includes: 2.0.1
@@ -4998,7 +5015,7 @@ snapshots:
estraverse: 5.3.0
hasown: 2.0.2
jsx-ast-utils: 3.3.5
- minimatch: 10.2.4
+ minimatch: 3.1.5
object.entries: 1.1.9
object.fromentries: 2.0.8
object.values: 1.2.1
@@ -5532,6 +5549,14 @@ snapshots:
dependencies:
brace-expansion: 5.0.5
+ [email protected]:
+ dependencies:
+ brace-expansion: 1.1.13
+
+ [email protected]:
+ dependencies:
+ brace-expansion: 2.0.3
+
[email protected]: {}
[email protected]: {}
diff --git
a/airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/src/alert/ErrorAlert.tsx
b/airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/src/alert/ErrorAlert.tsx
index f1214fc05f8..12c29dab3f9 100644
---
a/airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/src/alert/ErrorAlert.tsx
+++
b/airflow-core/src/airflow/api_fastapi/auth/managers/simple/ui/src/alert/ErrorAlert.tsx
@@ -18,6 +18,7 @@
*/
import { HStack } from "@chakra-ui/react";
import type { AxiosError } from "axios";
+
import type { HttpExceptionResponse, HttpValidationError } from
"openapi/requests/types.gen";
import { Alert } from "./Alert";
diff --git a/scripts/ci/prek/ts_compile_lint_simple_auth_manager_ui.py
b/scripts/ci/prek/ts_compile_lint_simple_auth_manager_ui.py
index 1a3727bc910..d6a178bafae 100755
--- a/scripts/ci/prek/ts_compile_lint_simple_auth_manager_ui.py
+++ b/scripts/ci/prek/ts_compile_lint_simple_auth_manager_ui.py
@@ -21,8 +21,8 @@ import sys
from pathlib import Path
from common_prek_utils import (
+ AIRFLOW_CORE_ROOT_PATH,
AIRFLOW_CORE_SOURCES_PATH,
- AIRFLOW_ROOT_PATH,
run_command,
temporary_tsc_project,
)
@@ -37,7 +37,7 @@ if __name__ == "__main__":
original_files = sys.argv[1:]
print("Original files:", original_files)
dir = AIRFLOW_CORE_SOURCES_PATH / "airflow" / "api_fastapi" / "auth" /
"managers" / "simple" / "ui"
- relative_dir = Path(dir).relative_to(AIRFLOW_ROOT_PATH)
+ relative_dir = Path(dir).relative_to(AIRFLOW_CORE_ROOT_PATH)
files = [
file[len(relative_dir.as_posix()) + 1 :]
for file in original_files