jason810496 opened a new pull request, #70285:
URL: https://github.com/apache/airflow/pull/70285

   ## Why
   
   Language-SDK work in the repo needs a Node.js/pnpm runtime for the `ts-sdk` 
(running integration/system tests, building the example bundle, etc.), but 
unlike Go and the JDK — which every breeze CI image already ships 
(`install_golang` / `install_jdk` in 
`scripts/docker/install_os_dependencies.sh`) — Node.js is absent, so there is 
no runtime available inside `breeze shell` today. This bakes Node.js + pnpm 
into the CI image the same unconditional way, so it is always present for 
lang-SDK integration/system testing.
   
   ## Verification
   
   - Built the CI image end to end (`breeze ci-image build --python 3.10`) and 
confirmed inside the built image: `node --version` → `v22.23.1`, `npm 
--version` → `10.9.8`, `pnpm --version` → `11.16.0`, all resolving on the 
default `PATH`; Go 1.26.5 and OpenJDK 11 remain present (no regression to the 
existing baked-in toolchains).
   - Ran the `install_nodejs` logic standalone in a bare `debian:bookworm-slim` 
container (the image's `BASE_IMAGE`) with a live checksum match against 
`nodejs.org`'s published `SHASUMS256.txt`.
   - `prek run` passes on all changed files (Dockerfile lint, shell syntax, 
license headers, inlined-scripts sync).
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [x] Yes, with help of Claude Code Opus 4.8 following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   


-- 
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]

Reply via email to