[flake8-type-checking] Fixes quote_type_expression

[Daverball]

Supersedes: #14614
Fixes: #14538
Fixes: #14554

Summary

This achieves a more feature complete version of wrapping annotations in quotes while removing nested forward references and handling nested quotes correctly.

It's not currently making use of the parsed annotations cache. I tried to pass around Checker and ParsedAnnotationsCache, either approach caused a cascade of borrow checker errors related to lifetimes I wasn't able to resolve.

This is not yet trying to force the outermost quotes of the quoted type expression to match our preference, so there are some scenarios where Generator will pick the opposite style.

Test Plan

cargo nextest run

[github-actions]

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

[Daverball]

@MichaReiser This is looking pretty good. Do you want to handle the changes to Generator to allow forcing the quote preference in a separate PR or should I do it as part of this one?

[MichaReiser]

@MichaReiser This is looking pretty good. Do you want to handle the changes to Generator to allow forcing the quote preference in a separate PR or should I do it as part of this one?

Glad to hear that! I would prefer a separate PR, unless it requires extra work from your side

[MichaReiser]

This is great