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

   The `update-breeze-cmd-output` prek hook invokes `breeze` from PATH, which 
is normally the uvx shim (ADR 0017) running from a **cached** build. `uvx` does 
not rebuild a local path dependency on `--refresh`/`--reinstall` — only 
`--no-cache` does — so after editing `dev/breeze` the hook computes command 
hashes and option groups from **stale code**: it misses a needed image 
regeneration, or worse, **reverts** a correctly-regenerated image back to the 
stale version during commit.
   
   This prepends the local breeze sources to `PYTHONPATH` for the hook's 
`breeze` subprocesses, so the in-process computation (and the help rendering it 
spawns) always reflects the current worktree source, while third-party 
dependencies (rich-click, click, ...) still resolve from the breeze 
environment. CI is unaffected — it installs breeze fresh from the same checkout.
   
   Also wraps the hook body in `main()` so the new 
`breeze_env_with_local_sources()` helper is unit-testable, and adds tests for 
it.
   
   Found while fixing the `Static checks` failure on #67960, where the stale 
uvx cache silently reverted a regenerated command image during commit.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 4.8)
   
   Generated-by: 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