fix(finders): filter find_wheel glob to .whl files only

find_wheel() used glob("*") which could match non-wheel files like
.tar.gz or .metadata, causing InvalidWheelFilename crashes downstream
in extract_info_from_wheel_file. Narrow the glob to "*.whl" so only
wheel files are considered.

Closes: #1067
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Justin Larkin <jlarkin@redhat.com>
Made-with: Cursor

Author: 2 people

src/fromager/finders.py

@@ -138,7 +138,7 @@ def find_wheel(
     # comparison.
     for base in candidate_bases:
         logger.debug('looking for wheel as "%s"', base)
-        for filename in downloads_dir.glob("*"):
+        for filename in downloads_dir.glob("*.whl"):
             if str(filename.name).lower().startswith(base.lower()):
                 return filename
 

tests/test_finders.py

@@ -58,6 +58,31 @@ def test_find_wheel(
     assert str(wheel) == str(actual)
 
 
+def test_find_wheel_ignores_non_wheel_files(tmp_path: pathlib.Path) -> None:
+    downloads = tmp_path / "downloads"
+    downloads.mkdir()
+    wheel = downloads / "mypkg-1.2-py3-none-any.whl"
+    wheel.write_text("not-empty")
+    (downloads / "mypkg-1.2-py3-none-any.tar.gz").write_text("not-a-wheel")
+    (downloads / "mypkg-1.2.metadata").write_text("not-a-wheel")
+
+    req = Requirement("mypkg")
+    actual = finders.find_wheel(downloads, req, "1.2", ())
+    assert str(wheel) == str(actual)
+
+
+def test_find_wheel_returns_none_when_only_non_wheel_files(
+    tmp_path: pathlib.Path,
+) -> None:
+    downloads = tmp_path / "downloads"
+    downloads.mkdir()
+    (downloads / "mypkg-1.2-py3-none-any.tar.gz").write_text("not-a-wheel")
+    (downloads / "mypkg-1.2.metadata").write_text("not-a-wheel")
+
+    req = Requirement("mypkg")
+    assert finders.find_wheel(downloads, req, "1.2", ()) is None
+
+
 @pytest.mark.parametrize(
     "dist_name,version_string,unpack_base,source_base",
     [