check dist dir

Author: alexrashed

plux/build/hatchling.py

@@ -117,9 +117,7 @@ def find_packages(self) -> t.Iterable[str]:
 
             # Package paths in hatchling are always relative to the project root, so we join
             # with the project root (not self.path, which is the sources root).
-            package_path = str(os.path.join(
-                self.builder_config.root, relative_package_path
-            ))
+            package_path = str(os.path.join(self.builder_config.root, relative_package_path))
             if not os.path.isdir(package_path):
                 continue
 
@@ -157,17 +155,24 @@ def path(self) -> str:
         """
         root = self.builder_config.root
 
+        # If no sources are configured, we assume the sources root is the project root
         if not self.builder_config.sources:
             return root
 
-        # No "" key.  The keys themselves are source directories (e.g. "localstack-core/").
-        # Filter out any empty-string key that slipped through, strip trailing separators.
-        source_dirs = [k.rstrip("/") for k in self.builder_config.sources.keys() if k]
+        # The keys themselves are source directories (e.g. "localstack-core/").
+        # Filter out any empty-string keys, strip trailing separators.
+        source_dirs = {k.rstrip("/"): v for k, v in self.builder_config.sources.items() if k}
         if len(source_dirs) == 1:
-            return os.path.join(root, str(source_dirs[0]))
+            source_dir, dest_dir = next(iter(source_dirs.items()))
+            if dest_dir:
+                LOG.warning(
+                    "plux doesn't know how to resolve sources with non-empty destination directories, using root dir."
+                )
+            else:
+                return os.path.join(root, str(source_dir))
 
         if source_dirs:
-            LOG.warning("plux doesn't know how to resolve multiple sources directories")
+            LOG.warning("plux doesn't know how to resolve multiple sources directories, using root dir.")
         return root
 
     def filter_packages(self, packages: t.Iterable[str]) -> t.Iterable[str]:

tests/build/test_hatchling.py

@@ -407,6 +407,7 @@ def test_path_with_packages_in_subdirectory(self, tmp_path):
         # Must not raise KeyError, and must return the sources root directory
         path = finder.path
         import os
+
         assert path == os.path.join(str(tmp_path), "localstack-core")
 
     def test_path_with_empty_sources(self, tmp_path):
@@ -419,4 +420,17 @@ def test_path_with_packages_in_root(self, tmp_path):
         """When packages are in the project root (sources = {'': ''}), path returns root."""
         finder = self._make_finder(sources={"": ""}, root=str(tmp_path))
 
-        assert finder.path == str(tmp_path)
+        assert finder.path == str(tmp_path)
+
+    def test_path_single_source_non_empty_dest_dir_falls_back_to_root(self, tmp_path):
+        """When a single source has a non-empty dest dir, fall back to project root.
+
+        A mapping like ``{"src/": "lib/"}`` means the wheel destination is remapped,
+        which plux cannot reason about — so it must return the project root and warn.
+        """
+
+        finder = self._make_finder(sources={"src/": "lib/"}, root=str(tmp_path))
+
+        path = finder.path
+
+        assert path == str(tmp_path)