python-package-folder 5.1.3__tar.gz → 5.1.5__tar.gz

{python_package_folder-5.1.3 → python_package_folder-5.1.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-package-folder
-Version: 5.1.3
+Version: 5.1.5
 Summary: Python package to automatically package and build a folder, fetching all relevant dependencies.
 Project-URL: Repository, https://github.com/alelom/python-package-folder
 Author-email: Alessio Lombardi <work@alelom.com>

{python_package_folder-5.1.3 → python_package_folder-5.1.5}/coverage.svg

@@ -14,7 +14,7 @@
   <g fill="#fff" text-anchor="middle" font-family="DejaVu Sans,Verdana,Geneva,sans-serif" font-size="11">
     <text x="31.5" y="15" fill="#010101" fill-opacity=".3">coverage</text>
     <text x="31.5" y="14">coverage</text>
-    <text x="81" y="15" fill="#010101" fill-opacity=".3">66%</text>
-    <text x="81" y="14">66%</text>
+    <text x="81" y="15" fill="#010101" fill-opacity=".3">68%</text>
+    <text x="81" y="14">68%</text>
   </g>
 </svg>

{python_package_folder-5.1.3 → python_package_folder-5.1.5}/pyproject.toml

@@ -43,7 +43,7 @@ dependencies = [
 
 # ---- Dev dependencies ----
 
-version = "5.1.3"
+version = "5.1.5"
 [dependency-groups]
 dev = [
     "pytest>=8.3.5",

{python_package_folder-5.1.3 → python_package_folder-5.1.5}/src/python_package_folder/subfolder_build.py

@@ -357,6 +357,13 @@ class SubfolderBuildConfig:
 
         original_content = original_pyproject.read_text(encoding="utf-8")
 
+        # Read exclude patterns from root pyproject.toml BEFORE moving the file
+        exclude_patterns = read_exclude_patterns(original_pyproject)
+        print(
+            f"INFO: Read exclude patterns from {original_pyproject}: {exclude_patterns}",
+            file=sys.stderr,
+        )
+
         # Store reference to original
         self.original_pyproject_path = original_pyproject
 
@@ -391,13 +398,6 @@ class SubfolderBuildConfig:
                     file=sys.stderr,
                 )
 
-        # Read exclude patterns from root pyproject.toml
-        exclude_patterns = read_exclude_patterns(original_pyproject)
-        print(
-            f"INFO: Read exclude patterns from {original_pyproject}: {exclude_patterns}",
-            file=sys.stderr,
-        )
-
         if data:
             # Modify using parsed data
             if "project" in data:
@@ -694,44 +694,55 @@ class SubfolderBuildConfig:
             # Usually after [dependency-groups] or at the end
             insert_index = len(result)
             tool_section_exists = False
+            tool_section_start = -1
+            tool_section_end = -1
+            
+            # First, search specifically for [tool.python-package-folder]
             for i, line in enumerate(result):
                 if line.strip() == "[tool.python-package-folder]":
                     tool_section_exists = True
-                    insert_index = i
-                    break
-                elif line.strip().startswith("[tool.") and i > 0:
-                    # Insert before other tool sections
-                    insert_index = i
+                    tool_section_start = i
+                    # Find end of section (next [section] or end of file)
+                    for j in range(i + 1, len(result)):
+                        if result[j].strip().startswith("["):
+                            tool_section_end = j
+                            break
+                    if tool_section_end == -1:
+                        tool_section_end = len(result)
                     break
+            
+            # If not found, find a good insertion point before other [tool.*] sections
+            if not tool_section_exists:
+                for i, line in enumerate(result):
+                    if line.strip().startswith("[tool.") and i > 0:
+                        # Insert before other tool sections
+                        insert_index = i
+                        break
 
             # Format exclude patterns
             patterns_str = ", ".join(f'"{p}"' for p in exclude_patterns)
-            exclude_lines = [
-                "",
-                "[tool.python-package-folder]",
-                f'exclude-patterns = [{patterns_str}]',
-            ]
 
             if tool_section_exists:
-                # Replace or update existing section
-                end_index = insert_index + 1
-                while end_index < len(result) and not result[end_index].strip().startswith("["):
-                    end_index += 1
-                # Check if exclude-patterns already exists
-                has_exclude_patterns = any(
-                    "exclude-patterns" in result[i] for i in range(insert_index, end_index)
-                )
-                if has_exclude_patterns:
-                    # Update existing exclude-patterns line
-                    for i in range(insert_index, end_index):
-                        if "exclude-patterns" in result[i]:
-                            result[i] = f'exclude-patterns = [{patterns_str}]'
-                            break
-                else:
-                    # Add exclude-patterns to existing section
-                    result.insert(end_index - 1, f'exclude-patterns = [{patterns_str}]')
+                # Update existing section
+                # Check if exclude-patterns already exists in the section
+                has_exclude_patterns = False
+                for i in range(tool_section_start + 1, tool_section_end):
+                    if "exclude-patterns" in result[i]:
+                        has_exclude_patterns = True
+                        # Update the existing line
+                        result[i] = f'exclude-patterns = [{patterns_str}]'
+                        break
+                
+                if not has_exclude_patterns:
+                    # Add exclude-patterns to existing section (before the next section)
+                    result.insert(tool_section_end, f'exclude-patterns = [{patterns_str}]')
             else:
                 # Insert new section
+                exclude_lines = [
+                    "",
+                    "[tool.python-package-folder]",
+                    f'exclude-patterns = [{patterns_str}]',
+                ]
                 result[insert_index:insert_index] = exclude_lines
 
         return "\n".join(result)

{python_package_folder-5.1.3 → python_package_folder-5.1.5}/tests/test_exclude_patterns.py

@@ -184,3 +184,87 @@ class TestExcludePatternsInBuild:
         assert ".*_test.*" in content
 
         config.restore()
+
+    def test_exclude_patterns_no_subfolder_toml(self, test_project_with_exclude_patterns: Path) -> None:
+        """Test that exclude patterns are read correctly when there's no subfolder pyproject.toml."""
+        project_root = test_project_with_exclude_patterns
+        src_dir = project_root / "src" / "test_package"
+
+        # This simulates the scenario where there's no subfolder pyproject.toml
+        # and we need to read exclude patterns from the root before it's moved
+        config = SubfolderBuildConfig(
+            project_root=project_root,
+            src_dir=src_dir,
+            package_name="test-package",
+            version="1.0.0",
+        )
+
+        # Create temporary pyproject.toml (this will read exclude patterns before moving root toml)
+        temp_pyproject = config.create_temp_pyproject()
+        assert temp_pyproject is not None
+
+        # Check that exclude patterns are in the temporary pyproject.toml
+        content = temp_pyproject.read_text()
+        assert "[tool.python-package-folder]" in content
+        assert "exclude-patterns" in content
+        assert "_SS" in content
+        assert ".*_test.*" in content
+        assert "sandbox" in content
+
+        # Verify there's only ONE [tool.python-package-folder] section (no duplicates)
+        sections = [line for line in content.split("\n") if line.strip() == "[tool.python-package-folder]"]
+        assert len(sections) == 1, f"Found {len(sections)} duplicate [tool.python-package-folder] sections"
+
+        config.restore()
+
+    def test_exclude_patterns_no_duplicate_section(self, tmp_path: Path) -> None:
+        """Test that exclude patterns don't create duplicate sections when original already has it."""
+        project_root = tmp_path / "test_project"
+        project_root.mkdir()
+
+        # Create pyproject.toml with existing [tool.python-package-folder] section
+        pyproject_content = """[project]
+name = "test-package"
+version = "0.1.0"
+
+[build-system]
+requires = ["hatchling"]
+build-backend = "hatchling.build"
+
+[tool.hatch.build.targets.wheel]
+packages = ["src/test_package"]
+
+[tool.python-package-folder]
+exclude-patterns = ["_SS", ".*_test.*"]
+
+[tool.pylint.'TYPECHECK']
+generated-members = ["networkx.*"]
+"""
+        (project_root / "pyproject.toml").write_text(pyproject_content)
+
+        # Create source directory
+        src_dir = project_root / "src" / "test_package"
+        src_dir.mkdir(parents=True)
+        (src_dir / "__init__.py").write_text("")
+
+        config = SubfolderBuildConfig(
+            project_root=project_root,
+            src_dir=src_dir,
+            package_name="test-package",
+            version="1.0.0",
+        )
+
+        # Create temporary pyproject.toml
+        temp_pyproject = config.create_temp_pyproject()
+        assert temp_pyproject is not None
+
+        # Check that there's only ONE [tool.python-package-folder] section
+        content = temp_pyproject.read_text()
+        sections = [line for line in content.split("\n") if line.strip() == "[tool.python-package-folder]"]
+        assert len(sections) == 1, f"Found {len(sections)} duplicate [tool.python-package-folder] sections: {sections}"
+
+        # Verify exclude-patterns is present
+        assert "exclude-patterns" in content
+        assert "_SS" in content
+
+        config.restore()