chore: Close IO objects (#2005)

* chore: close file handle

* chore: close package metadata IO objects

* chore: close file handle

src/pdm/cli/commands/publish/package.py

@@ -30,7 +30,11 @@
 
 
 def parse_metadata(fp: IO[bytes]) -> email.message.Message:
-    return email.message_from_file(io.TextIOWrapper(fp, encoding="utf-8", errors="surrogateescape"))
+    """
+    Note that this function will close fp. See https://github.com/python/cpython/issues/65562.
+    """
+    with io.TextIOWrapper(fp, encoding="utf-8", errors="surrogateescape") as file:
+        return email.message_from_file(file)
 
 
 @dataclass

src/pdm/models/working_set.py

@@ -19,7 +19,8 @@ def find_distributions(cls, context: im.DistributionFinder.Context = default_con
         meta_finder = im.MetadataPathFinder()
         for link in found_links:
             name = link.stem
-            link_pointer = Path(link.open("rb").readline().decode().strip())
+            with link.open("rb") as file_link:
+                link_pointer = Path(file_link.readline().decode().strip())
             dist = next(
                 iter(
                     meta_finder.find_distributions(im.DistributionFinder.Context(name=name, path=[str(link_pointer)]))

tests/models/test_setup_parsing.py

@@ -84,9 +84,10 @@ def test_parse_setup_cfg(content, result, tmp_path):
             Setup("foo", "0.1.0", ["click", "requests"], {"tui": ["rich"]}, ">=3.6"),
         ),
         (
-            """from setuptools import setup
+            """from pathlib import Path
+from setuptools import setup
 
-version = open('__version__.py').read().strip()
+version = Path('__version__.py').read_text().strip()
 
 setup(name="foo", version=version)
 """,