Consider expression before statement when determining binding kind

[charliermarsh]

Summary

I believe these should always bind more tightly -- e.g., in:

for _ in bar(baz for foo in [1]):
    pass

The inner baz and foo should be considered comprehension variables, not for loop bindings.

We need to revisit this more holistically. In some of these cases, BindingKind should probably be a flag, not an enum, since the values aren't mutually exclusive. Separately, we should probably be more precise in how we set it (e.g., by passing down from the parent rather than sniffing in handle_node_store).

Closes #12339

[github-actions]

ruff-ecosystem results

Linter (stable)

ℹ️ ecosystem check detected linter changes. (+0 -1 violations, +0 -0 fixes in 1 projects; 53 projects unchanged)

apache/airflow (+0 -1 violations, +0 -0 fixes)

ruff check --no-cache --exit-zero --ignore RUF9 --output-format concise --no-preview --select ALL

- dev/breeze/src/airflow_breeze/commands/kubernetes_commands.py:689:51: PLR1704 Redefining argument with the local name `python`

Changes by rule (1 rules affected)

code	total	+ violation	- violation	+ fix	- fix
PLR1704	1	0	1	0	0

Linter (preview)

ℹ️ ecosystem check detected linter changes. (+0 -1 violations, +0 -0 fixes in 1 projects; 53 projects unchanged)

apache/airflow (+0 -1 violations, +0 -0 fixes)

ruff check --no-cache --exit-zero --ignore RUF9 --output-format concise --preview --select ALL

- dev/breeze/src/airflow_breeze/commands/kubernetes_commands.py:689:51: PLR1704 Redefining argument with the local name `python`

Changes by rule (1 rules affected)

code	total	+ violation	- violation	+ fix	- fix
PLR1704	1	0	1	0	0

[charliermarsh]

Ecosystem change is correct.