Improve ruff_python_semantic::all::extract_all_names()

[AlexWaygood]

Summary

Following 22639c5, this function now lives in the ruff_python_semantic crate. That means we can simplify the function's signature: where previously it required a closure to be passed in, we can now simply pass in a reference to a SemanticModel instance. As well as simplifying the signature of the function, this also means we can make the function more accurate: it now recognises __all__ = builtins.list(["foo", "bar"]) as a valid __all__ definition, as well as __all__ = list(["foo", "bar"]).

Test Plan

I added a new pyflakes fixture. cargo test / cargo insta review.

[github-actions]

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

[AlexWaygood]

MichaReiser added internal and removed linter labels

FWIW this does have a small user-visible impact (__all__ = builtins.list(["foo"]) is now recognised as a valid __all__ assignment, when it previously was not). But I'm also fine with skipping the changelog.