potiuk commented on PR #66017: URL: https://github.com/apache/airflow/pull/66017#issuecomment-4658687320
@gdivya6028 — good news first: this isn't a problem with your PR. The `compile-ui-assets` / `compile-common-ai-provider-assets` steps fail at the local **asset-build** stage, which is almost always a local Node/pnpm environment mismatch rather than anything in your change. A few things to check: 1. **Node + pnpm versions.** The UI (`airflow-core/src/airflow/ui`) builds with Vite via pnpm and needs **node ≥ 20 (22 recommended)** and **pnpm ≥ 9** — mismatched versions are the most common cause. See [contributing-docs/15_node_environment_setup.rst](https://github.com/apache/airflow/blob/main/contributing-docs/15_node_environment_setup.rst); `nvm` helps manage the node version. 2. **Build the assets explicitly to see the real error:** `breeze compile-ui-assets` (or `pre-commit run compile-ui-assets --all-files`). The hook output you pasted is truncated — it shows the hook failed but not *why*. The explicit command prints the actual Vite/pnpm error. 3. If it still fails, please paste the **full** output from step 2 (everything after "Failed") — the first real error line is what we need to diagnose. For interactive setup help, **#development** in the [ASF Slack](https://airflow.apache.org/community/) is usually the fastest place — local-env issues are easier to debug there than in a PR thread. Your PR itself isn't blocked by this — once the assets compile locally you'll be able to push and let CI run. 👍 --- Drafted-by: Claude Code (Opus 4.8 1M context); reviewed by @potiuk before posting -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
