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

   Replaces the Dependabot UI bump for `v3-4-test`, keeping its commit and 
adding the two fixes it needs to pass CI.
   
   - **Dependency review:** `happy-dom` 20.14.6 is flagged by 
GHSA-834v-8h66-3h79 and GHSA-c8p4-j97h-3vg8 (both high). Both are only clear 
from 21.0.4, and Dependabot ignores majors on this branch, so it cannot get 
there on its own. `happy-dom` is a test-only dependency and `main` already runs 
21.x.
   - **React UI tests:** `@tanstack/virtual-core` now measures the scroll 
viewport from `clientHeight` rather than `offsetHeight`. `happy-dom` reports 0 
for it, so 28 log tests rendered no virtualized rows. They now mock 
`clientHeight` as well. This is a test fixture gap; there is no change to 
production code.
   
   Dependabot's #76395 on `main` fails the same 28 tests, so it will need the 
same mock.
   
   Checked locally:
   
   - `pnpm test` in `airflow-core/src/airflow/ui`: 2231 passed.
   - `pnpm test` in the simple auth manager UI: 1 passed on Node 22.
   - prek pre-commit hooks, including `ts-compile-lint-ui`: passed.
   
   The `Compile ui assets` manual hook could not run locally, so the UI build 
is left to CI.
   
   related: #76427
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Opus 5
   
   Generated-by: Claude Opus 5 following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


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