Remove is_valid_module change, continue linking to pyx for now

Author: vyasr

src/scikit_build_core/build/_pathutil.py

@@ -60,5 +60,4 @@ def is_valid_module(path: Path) -> bool:
     return (
         all(p.isidentifier() for p in parts[:-1])
         and parts[-1].split(".", 1)[0].isidentifier()
-        and path.suffix == ".py"
     )

src/scikit_build_core/build/wheel.py

@@ -30,6 +30,7 @@
 
 if TYPE_CHECKING:
     from collections.abc import Sequence
+    from typing import Iterable
 
     from ..settings.skbuild_model import ScikitBuildSettings
 
@@ -50,7 +51,7 @@ def _make_editable(
     reload_dir: Path | None,
     settings: ScikitBuildSettings,
     wheel: WheelWriter,
-    packages: Sequence[str],
+    packages: Iterable[str],
 ) -> None:
     modules = mapping_to_modules(mapping, libdir)
     installed = libdir_to_installed(libdir)
@@ -73,9 +74,7 @@ def _make_editable(
     # This is necessary because Cython does not support sys.meta_path for
     # cimports (as of 3.0.5).
     pth_import_paths = (
-        f"import _{name}_editable\n"
-        + "\n".join(os.fspath(Path(pkg).parent.resolve()) for pkg in packages)
-        + "\n"
+        f"import _{name}_editable\n" + "\n".join(pkg for pkg in packages) + "\n"
     )
     wheel.writestr(
         f"_{name}_editable.pth",
@@ -350,6 +349,7 @@ def _build_wheel_impl(
         ) as wheel:
             wheel.build(wheel_dirs)
 
+            str_pkgs = (str(Path.cwd().joinpath(p).parent.resolve()) for p in packages)
             if editable and settings.editable.mode == "redirect":
                 reload_dir = build_dir.resolve() if settings.build_dir else None
 
@@ -362,16 +362,13 @@ def _build_wheel_impl(
                     settings=settings,
                     wheel=wheel,
                     name=normalized_name,
-                    packages=packages,
+                    packages=str_pkgs,
                 )
             elif editable and settings.editable.mode == "inplace":
                 if not packages:
                     msg = "Editable inplace mode requires at least one package"
                     raise AssertionError(msg)
 
-                str_pkgs = (
-                    str(Path.cwd().joinpath(p).parent.resolve()) for p in packages
-                )
                 wheel.writestr(
                     f"_{normalized_name}_editable.pth",
                     "\n".join(str_pkgs).encode(),