gh-121190: A better error message from importlib.resources.files() when module spec is None

Lib/importlib/resources/_adapters.py

@@ -9,7 +9,7 @@ class SpecLoaderAdapter:
     Adapt a package spec to adapt the underlying loader.
     """
 
-    def __init__(self, spec, adapter=lambda spec: spec.loader):
+    def __init__(self, spec, adapter):
         self.spec = spec
         self.loader = adapter(spec)
 
@@ -160,9 +160,9 @@ def files(self):
         return CompatibilityFiles.SpecPath(self.spec, self._reader)
 
 
-def wrap_spec(package):
+def wrap_spec(spec):
     """
     Construct a package spec with traversable compatibility
     on the spec/loader/reader.
     """
-    return SpecLoaderAdapter(package.__spec__, TraversableResourcesLoader)
+    return SpecLoaderAdapter(spec, TraversableResourcesLoader)

Lib/importlib/resources/_common.py

@@ -113,7 +113,13 @@ def from_package(package: types.ModuleType):
     # deferred for performance (python/cpython#109829)
     from ._adapters import wrap_spec
 
-    spec = wrap_spec(package)
+    if package.__spec__ is None:
+        raise TypeError(
+            f"Cannot access resources because the code used to populate '{package.__name__}' "
+            "does not correspond directly with an importable module."
+        )
+
+    spec = wrap_spec(package.__spec__)
     reader = spec.loader.get_resource_reader(spec.name)
     return reader.files()
 

Lib/test/test_importlib/resources/test_resource.py

@@ -1,4 +1,5 @@
 import unittest
+import types
 
 from . import util
 from importlib import resources, import_module
@@ -232,5 +233,17 @@ class ResourceFromNamespaceZipTests(
     MODULE = 'namespacedata01'
 
 
+class ResourceFromMainModuleWithNoneSpecTests(unittest.TestCase):
+    # `__main__.__spec__` can be `None` depending on how it is populated.
+    # https://docs.python.org/3/reference/import.html#main-spec
+    def test_main_module_with_none_spec(self):
+        mainmodule = types.ModuleType("__main__")
+
+        self.assertIsNone(mainmodule.__spec__)
+
+        with self.assertRaises(TypeError):
+            resources.files(mainmodule)
+
+
 if __name__ == '__main__':
     unittest.main()

Misc/NEWS.d/next/Library/2025-09-05-08-26-11.gh-issue-121190.nBBcWu.rst

@@ -0,0 +1 @@
+``importlib.resources.files(package)`` raises an error with a clearer message when ``package.__spec__`` is ``None``