[ty] Fix 'too many cycle iterations' for unions of literals by sharkdp · Pull Request #20137 · astral-sh/ruff (original) (raw)

@sharkdp

@sharkdp sharkdp added bug

Something isn't working

ty

Multi-file analysis & type inference

labels

Aug 28, 2025

AlexWaygood

dcreager added a commit that referenced this pull request

Aug 28, 2025

@dcreager

second-ed pushed a commit to second-ed/ruff that referenced this pull request

Sep 9, 2025

@sharkdp @second-ed

…h#20137)

Summary

Decrease the maximum number of literals in a union before we collapse to the supertype. The better fix for this will be astral-sh/ty#957, but it is very tempting to solve this for now by simply decreasing the limit by one, to get below the salsa limit of 200.

closes astral-sh/ty#660

Test Plan

Added a regression test that would previously lead to a "too many cycle iterations" panic.

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})