fix: change manifest priority for uv workflow

Author: reedham-aws

aws_lambda_builders/workflows/python_uv/utils.py

@@ -37,20 +37,17 @@ def detect_uv_manifest(source_dir: str) -> Optional[str]:
 
     Note: uv.lock is NOT a manifest - it's a lock file that accompanies pyproject.toml.
     UV workflows support these manifest types:
-    1. pyproject.toml (preferred) - may have accompanying uv.lock
-    2. requirements.txt and variants - traditional pip-style manifests
+    1. requirements.txt and variants - traditional pip-style manifests
+    2. pyproject.toml (preferred) - may have accompanying uv.lock
+
+    We favor requirements.txt because it is possible to have both, but the pyproject could just be project metadata.
 
     Args:
         source_dir: Directory to search for manifest files
 
     Returns:
         Path to the detected manifest file, or None if not found
     """
-    # Check for pyproject.toml first (preferred manifest)
-    pyproject_path = os.path.join(source_dir, "pyproject.toml")
-    if os.path.isfile(pyproject_path):
-        return pyproject_path
-
     # Check for requirements.txt variants (in order of preference)
     requirements_variants = [
         "requirements.txt",
@@ -64,6 +61,11 @@ def detect_uv_manifest(source_dir: str) -> Optional[str]:
         if os.path.isfile(requirements_path):
             return requirements_path
 
+    # Check for pyproject.toml first (preferred manifest)
+    pyproject_path = os.path.join(source_dir, "pyproject.toml")
+    if os.path.isfile(pyproject_path):
+        return pyproject_path
+
     return None
 
 

tests/unit/workflows/python_uv/test_utils.py

@@ -73,7 +73,7 @@ def test_detect_uv_manifest_pyproject_priority(self):
 
             result = detect_uv_manifest(temp_dir)
             # pyproject.toml should have priority over requirements.txt
-            self.assertEqual(result, pyproject_path)
+            self.assertEqual(result, req_path)
 
     def test_detect_uv_manifest_requirements_variants(self):
         with tempfile.TemporaryDirectory() as temp_dir: