This is an automated email from the ASF dual-hosted git repository.
jason810496 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 568aef293bf Stop UI and ts-sdk lint hooks from rewriting the global
pnpm config (#73538)
568aef293bf is described below
commit 568aef293bf54a821699a5ee2b5b0f4015129399
Author: Andrew Chang <[email protected]>
AuthorDate: Wed Sep 23 00:41:04 2026 +0800
Stop UI and ts-sdk lint hooks from rewriting the global pnpm config (#73538)
Eight prek hooks (ts-compile-lint-ui and its copies) run
`pnpm config set store-dir .pnpm-store` before `pnpm install`. Without
`--location project` that line lands in the user's global pnpm config, so
every worktree on the machine downloads the UI dependencies again instead
of sharing the global store.
---
scripts/ci/prek/ts_compile_lint_common_ai.py | 1 -
scripts/ci/prek/ts_compile_lint_edge.py | 1 -
scripts/ci/prek/ts_compile_lint_simple_auth_manager_ui.py | 1 -
scripts/ci/prek/ts_compile_lint_ui.py | 1 -
ts-sdk/scripts/ci/prek/check_supervisor_schema.py | 1 -
ts-sdk/scripts/ci/prek/compile_ts_sdk.py | 1 -
ts-sdk/scripts/ci/prek/ts_sdk_lint.py | 1 -
ts-sdk/scripts/ci/prek/verify_ts_sdk_package.py | 1 -
8 files changed, 8 deletions(-)
diff --git a/scripts/ci/prek/ts_compile_lint_common_ai.py
b/scripts/ci/prek/ts_compile_lint_common_ai.py
index ae0032feb31..9aeb9ebf2ae 100755
--- a/scripts/ci/prek/ts_compile_lint_common_ai.py
+++ b/scripts/ci/prek/ts_compile_lint_common_ai.py
@@ -59,7 +59,6 @@ if __name__ == "__main__":
all_ts_files = [file for file in files if file.endswith(".ts") or
file.endswith(".tsx")]
print("All TypeScript files:", all_ts_files)
- run_command(["pnpm", "config", "set", "store-dir", ".pnpm-store"], cwd=dir)
run_command(["pnpm", "install", "--frozen-lockfile",
"--config.confirmModulesPurge=false"], cwd=dir)
if any("/openapi/" in file for file in original_files):
run_command(["pnpm", "codegen"], cwd=dir)
diff --git a/scripts/ci/prek/ts_compile_lint_edge.py
b/scripts/ci/prek/ts_compile_lint_edge.py
index 229b890da18..9126c5e9068 100755
--- a/scripts/ci/prek/ts_compile_lint_edge.py
+++ b/scripts/ci/prek/ts_compile_lint_edge.py
@@ -50,7 +50,6 @@ if __name__ == "__main__":
all_ts_files = [file for file in files if file.endswith(".ts") or
file.endswith(".tsx")]
print("All TypeScript files:", all_ts_files)
- run_command(["pnpm", "config", "set", "store-dir", ".pnpm-store"], cwd=dir)
run_command(["pnpm", "install", "--frozen-lockfile",
"--config.confirmModulesPurge=false"], cwd=dir)
if any("/openapi/" in file for file in original_files):
run_command(["pnpm", "codegen"], cwd=dir)
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 d6a178bafae..0e6912331a6 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
@@ -48,7 +48,6 @@ if __name__ == "__main__":
all_ts_files = [file for file in files if file.endswith(".ts") or
file.endswith(".tsx")]
print("All TypeScript files:", all_ts_files)
- run_command(["pnpm", "config", "set", "store-dir", ".pnpm-store"], cwd=dir)
run_command(["pnpm", "install", "--frozen-lockfile",
"--config.confirmModulesPurge=false"], cwd=dir)
if any("/openapi/" in file for file in original_files):
run_command(["pnpm", "codegen"], cwd=dir)
diff --git a/scripts/ci/prek/ts_compile_lint_ui.py
b/scripts/ci/prek/ts_compile_lint_ui.py
index 58d7929d47f..06bd3349306 100755
--- a/scripts/ci/prek/ts_compile_lint_ui.py
+++ b/scripts/ci/prek/ts_compile_lint_ui.py
@@ -72,7 +72,6 @@ if __name__ == "__main__":
if cache_valid:
print("pnpm deps unchanged — skipping install.")
else:
- run_command(["pnpm", "config", "set", "store-dir", ".pnpm-store"],
cwd=dir)
run_command(["pnpm", "install", "--frozen-lockfile",
"--config.confirmModulesPurge=false"], cwd=dir)
hash_file.parent.mkdir(parents=True, exist_ok=True)
tmp_hash_file = hash_file.with_suffix(".tmp")
diff --git a/ts-sdk/scripts/ci/prek/check_supervisor_schema.py
b/ts-sdk/scripts/ci/prek/check_supervisor_schema.py
index 81e6fe1ca5e..7ff1be538df 100755
--- a/ts-sdk/scripts/ci/prek/check_supervisor_schema.py
+++ b/ts-sdk/scripts/ci/prek/check_supervisor_schema.py
@@ -36,7 +36,6 @@ GENERATED = "ts-sdk/src/generated/supervisor.ts"
if __name__ == "__main__":
directory = AIRFLOW_ROOT_PATH / "ts-sdk"
- run_command(["pnpm", "config", "set", "store-dir", ".pnpm-store"],
cwd=directory)
run_command(["pnpm", "install", "--frozen-lockfile",
"--config.confirmModulesPurge=false"], cwd=directory)
# Regenerate, then format exactly as `pnpm run format` would, so the diff
# reflects a stale schema and not raw-vs-prettier formatting noise.
diff --git a/ts-sdk/scripts/ci/prek/compile_ts_sdk.py
b/ts-sdk/scripts/ci/prek/compile_ts_sdk.py
index b0e43e52510..bcd8023b6d9 100755
--- a/ts-sdk/scripts/ci/prek/compile_ts_sdk.py
+++ b/ts-sdk/scripts/ci/prek/compile_ts_sdk.py
@@ -32,6 +32,5 @@ if __name__ not in ("__main__", "__mp_main__"):
if __name__ == "__main__":
directory = AIRFLOW_ROOT_PATH / "ts-sdk"
- run_command(["pnpm", "config", "set", "store-dir", ".pnpm-store"],
cwd=directory)
run_command(["pnpm", "install", "--frozen-lockfile",
"--config.confirmModulesPurge=false"], cwd=directory)
run_command(["pnpm", "run", "build"], cwd=directory)
diff --git a/ts-sdk/scripts/ci/prek/ts_sdk_lint.py
b/ts-sdk/scripts/ci/prek/ts_sdk_lint.py
index c46ccc81d85..06240766fa7 100755
--- a/ts-sdk/scripts/ci/prek/ts_sdk_lint.py
+++ b/ts-sdk/scripts/ci/prek/ts_sdk_lint.py
@@ -32,7 +32,6 @@ if __name__ not in ("__main__", "__mp_main__"):
if __name__ == "__main__":
directory = AIRFLOW_ROOT_PATH / "ts-sdk"
- run_command(["pnpm", "config", "set", "store-dir", ".pnpm-store"],
cwd=directory)
run_command(["pnpm", "install", "--frozen-lockfile",
"--config.confirmModulesPurge=false"], cwd=directory)
run_command(["pnpm", "run", "lint:fix"], cwd=directory)
run_command(["pnpm", "run", "format"], cwd=directory)
diff --git a/ts-sdk/scripts/ci/prek/verify_ts_sdk_package.py
b/ts-sdk/scripts/ci/prek/verify_ts_sdk_package.py
index a1aba68d1c0..dc09fac60fd 100755
--- a/ts-sdk/scripts/ci/prek/verify_ts_sdk_package.py
+++ b/ts-sdk/scripts/ci/prek/verify_ts_sdk_package.py
@@ -32,6 +32,5 @@ if __name__ not in ("__main__", "__mp_main__"):
if __name__ == "__main__":
directory = AIRFLOW_ROOT_PATH / "ts-sdk"
- run_command(["pnpm", "config", "set", "store-dir", ".pnpm-store"],
cwd=directory)
run_command(["pnpm", "install", "--frozen-lockfile",
"--config.confirmModulesPurge=false"], cwd=directory)
run_command(["pnpm", "run", "verify:package"], cwd=directory)
