GeezerVison

Closes #534.

Built on the #550 revert (#552), now on main.

The fix

entry_points() fed distributions() straight to _unique(), which keeps the first distribution seen for each normalized name. Where an unclean upgrade leaves an empty foo-1.0.dist-info beside the valid foo-1.1.dist-info, the empty one could be listed first, and with it the valid distribution's entry points disappeared entirely.

_prefer_valid already existed for exactly this preference, used by from_name. Applying it ahead of _unique in entry_points() is the whole of the fix.

Detecting metadata rather than loading it

_prefer_valid decided validity by consulting metadata, which runs the whole METADATA file through the email parser and then discards the result. Applied to every distribution rather than to one, that is the cost @jaraco anticipated on the issue — measured, entry_points() went from 1.53 msec to 13.4 msec, 8.7x.

So PathDistribution grows a _has_metadata that detects the metadata file without opening it, and that is the preference's predicate. The remaining 29% on entry_points() buys a large improvement in retrieval by name, which no longer parses the metadata of the distribution it returns twice:

Distributions that aren't ours

localize.dist passes a provider's own Distribution subclass through unadapted (#486), so it has no _has_metadata and test_conforming_provider_is_not_warned_about caught that directly.

Nothing on IDistribution is as cheap for those: locate_file returns a SimplePath, which does have exists(), but it resolves against the package root rather than the metadata, so locate_file('METADATA') names site-packages/METADATA. Every other member loads the metadata or parses it. So Distribution._has_metadata() reads, through a _metadata_text that metadata shares so neither names the files twice, and PathDistribution overrides it with the detection above. The predicate is _safe_has_metadata, which falls back for a distribution supplying neither helper.

That turns out to fix something pre-existing. Asking such a distribution for metadata raises importlib.metadata.MetadataNotFound, a different class from this package's, so it escaped ExceptionTrap(MetadataNotFound) on main and propagated out of from_name instead of marking the distribution invalid. test_dist_without_metadata_is_deferred covers it.

Tests

• test_entry_points_from_valid_dist — fails before the fix with assert set() == {'ep'}. It brackets the valid distribution between two invalid ones, as test_valid_dists_preferred does, so that no file system lists it first.
• test_invalid_dists_deferred — the invalid distributions are moved to the end, not dropped.
• test_dist_without_metadata_is_deferred — the foreign-MetadataNotFound case above.

tox passes on each commit separately: 260 passed, 1 skipped, 41 subtests; mypy and ruff clean.

🤖 Generated with Claude Code