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

   ## Result (TL;DR)
   
   `breeze verify` turns the working-tree diff into local verification 
commands. It lists directly relevant checks by default, with `--full` for the 
expanded default CI cell and `--json` for machine-readable output. 
   
   This PR only lists out the suggested commands; a follow-up can add `--run` 
to actually execute all tests in one command.
   
   <img width="1276" height="282" alt="image" 
src="https://github.com/user-attachments/assets/32dbe7ec-605a-459c-ac80-85247633704b";
 />
   
   ## Why
   
   Before opening a PR, AGENTS.md step 7 asks an agent to find the tests CI 
would run for the change and run the relevant ones locally. The tool for that 
is `breeze ci selective-check`, which was built for the build-info job in CI: 
it reads one commit and prints outputs like `run-unit-tests=true` for the 
workflows to consume. It does not say which command each output maps to, so a 
contributor (agent) still has to read `ci-amd.yml`, `run-unit-tests.yml` and 
`run_unit_tests.sh` to get a `breeze testing` line. 
   
   This PR makes the mapping from selective-check outputs to local commands 
deterministic.  For a human it is a preview of what CI will do. For an agent 
following AGENTS.md step 7, and for the `--run` flag that comes next, it is the 
list to execute. It could help save a lot of tokens and prevent the agent from 
selecting the wrong checks.
   
   ## What
   
   Add `breeze verify`. It reads committed and untracked changes against 
`--base-ref`, using the same rename behavior as CI. Nothing is executed.
   
   - To prevent drift between this tool and CI, two tests lock the mapping: 
every `run_*` flag on `SelectiveChecks` must be classified, and the printed 
core/providers flags must match a line in 
`scripts/ci/testing/run_unit_tests.sh`.
   - I considered a flag on `breeze ci selective-check` instead, but that group 
is CI-facing and its inputs assume a CI context. The plan builder is a 
standalone function, so moving it there is a small change if you prefer.
   - A follow-up can add `--run`. This PR keeps the command list-only so the 
selection and mapping can be reviewed separately.
   
   #### `--json` mode
   Better for user copying or agent reading.
   <img width="1268" height="633" alt="image" 
src="https://github.com/user-attachments/assets/5c04d144-c176-4ce9-84d7-98c30f775d79";
 />
   
   #### `--full` mode
   Lists the expanded plan for CI's default Python and sqlite cell, including 
checks that run on every PR.
   <img width="1272" height="469" alt="image" 
src="https://github.com/user-attachments/assets/26be8d86-dbd7-4361-b0d3-d3d2da410951";
 />
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Fable 5.1); reviewed with Codex (gpt-5.6-sol)
   
   Generated-by: Claude Code (Fable 5.1) 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