python-package-folder 4.1.0__py3-none-any.whl → 4.1.2__py3-none-any.whl

python_package_folder/python_package_folder.py

@@ -10,6 +10,8 @@ It can be invoked via:
 
 from __future__ import annotations
 
+import os
+import shutil
 import subprocess
 import sys
 from pathlib import Path
@@ -23,13 +25,23 @@ from .manager import BuildManager
 from .utils import find_project_root, find_source_directory
 
 
+def is_github_actions() -> bool:
+    """Check if running in GitHub Actions."""
+    return os.getenv("GITHUB_ACTIONS") == "true"
+
+
+def check_node_available() -> bool:
+    """Check if Node.js is available."""
+    return shutil.which("node") is not None
+
+
 def resolve_version_via_semantic_release(
     project_root: Path,
     subfolder_path: Path | None = None,
     package_name: str | None = None,
     repository: str | None = None,
     repository_url: str | None = None,
-) -> str | None:
+) -> tuple[str | None, str | None]:
     """
     Resolve the next version using semantic-release via Node.js script.
 
@@ -41,8 +53,34 @@ def resolve_version_via_semantic_release(
         repository_url: Optional repository URL (required for Azure Artifacts)
 
     Returns:
-        Version string if a release is determined, None if no release or error
+        Tuple of (version string if a release is determined, error message if any)
+        Returns (None, None) if no release or no error, (None, error_msg) on error
     """
+    # Note: Node.js availability should be checked before calling this function
+    # This check is a safety fallback
+    if not check_node_available():
+        if is_github_actions():
+            error_msg = """Node.js is not available in this GitHub Actions workflow.
+
+To fix this, add the following steps BEFORE running python-package-folder:
+
+- name: Setup Node.js
+  uses: actions/setup-node@v4
+  with:
+    node-version: '20'
+
+- name: Install semantic-release
+  run: |
+    npm install -g semantic-release semantic-release-commit-filter
+
+Alternatively, provide --version explicitly to skip automatic version resolution."""
+            print(f"Error: {error_msg}", file=sys.stderr)
+            return None, error_msg
+        else:
+            error_msg = "Node.js not found. Cannot resolve version via semantic-release."
+            print(f"Error: {error_msg}", file=sys.stderr)
+            return None, error_msg
+
     # Try to find the script in multiple locations:
     # 1. Project root / scripts (for development or when script is in repo)
     # 2. Package installation directory / scripts (for installed package)
@@ -60,37 +98,70 @@ def resolve_version_via_semantic_release(
         else:
             # Try to locate script in installed package using importlib.resources
             script_path = None
+            diagnostic_info = []
+            
+            # Try importlib.resources approach
             try:
                 package = resources.files("python_package_folder")
                 script_resource = package / "scripts" / "get-next-version.cjs"
+                
+                diagnostic_info.append(f"Checked importlib.resources: python_package_folder/scripts/get-next-version.cjs")
+                
                 if script_resource.is_file():
                     # Try direct path conversion first (normal file system install)
                     try:
                         script_path_candidate = Path(str(script_resource))
                         if script_path_candidate.exists():
                             script_path = script_path_candidate
-                    except (TypeError, ValueError):
-                        pass
+                            diagnostic_info.append(f"Found via direct path: {script_path}")
+                    except (TypeError, ValueError) as e:
+                        diagnostic_info.append(f"Direct path conversion failed: {e}")
 
                     # If direct path didn't work, try as_file() for zip/pex installs
                     if script_path is None:
                         try:
                             temp_script_context = resources.as_file(script_resource)
                             script_path = temp_script_context.__enter__()
-                        except (TypeError, ValueError, OSError):
-                            pass
-            except (ImportError, ModuleNotFoundError, TypeError, AttributeError, OSError):
-                pass
+                            diagnostic_info.append(f"Found via as_file() (temp): {script_path}")
+                        except (TypeError, ValueError, OSError) as e:
+                            diagnostic_info.append(f"as_file() extraction failed: {e}")
+                else:
+                    # Try to list what's actually in the scripts directory
+                    try:
+                        scripts_dir = package / "scripts"
+                        if scripts_dir.is_dir():
+                            available_files = list(scripts_dir.iterdir())
+                            diagnostic_info.append(f"Scripts directory exists. Available files: {[f.name for f in available_files]}")
+                        else:
+                            diagnostic_info.append("Scripts directory does not exist in package")
+                    except Exception as e:
+                        diagnostic_info.append(f"Could not list scripts directory: {e}")
+            except (ImportError, ModuleNotFoundError, TypeError, AttributeError, OSError) as e:
+                diagnostic_info.append(f"importlib.resources failed: {type(e).__name__}: {e}")
 
             # Fallback: try relative to package directory
             if script_path is None:
                 package_dir = Path(__file__).parent
                 fallback_script = package_dir / "scripts" / "get-next-version.cjs"
+                diagnostic_info.append(f"Checked fallback path: {fallback_script}")
                 if fallback_script.exists():
                     script_path = fallback_script
+                    diagnostic_info.append(f"Found via fallback: {script_path}")
+                else:
+                    diagnostic_info.append(f"Fallback path does not exist")
 
         if not script_path:
-            return None
+            error_msg = "Could not locate get-next-version.cjs script"
+            error_msg += "\n\nDiagnostic information:"
+            for info in diagnostic_info:
+                error_msg += f"\n  - {info}"
+            error_msg += (
+                "\n\nThis usually means the script was not included in the installed package."
+                "\nPlease ensure you're using the latest version of python-package-folder."
+                "\nIf the issue persists, report this as a bug."
+            )
+            print(f"Error: {error_msg}", file=sys.stderr)
+            return None, error_msg
 
         # Build command arguments
         cmd = ["node", str(script_path), str(project_root)]
@@ -123,38 +194,53 @@ def resolve_version_via_semantic_release(
         )
 
         if result.returncode != 0:
-            # Log error details for debugging
+            # Collect error details
+            error_details = []
             if result.stderr:
-                print(
-                    f"Warning: semantic-release version resolution failed: {result.stderr}",
-                    file=sys.stderr,
-                )
-            elif result.stdout:
-                print(
-                    f"Warning: semantic-release version resolution failed: {result.stdout}",
-                    file=sys.stderr,
-                )
-            return None
+                error_details.append(f"stderr: {result.stderr}")
+            if result.stdout:
+                error_details.append(f"stdout: {result.stdout}")
+            
+            error_msg = "semantic-release version resolution failed"
+            if error_details:
+                error_msg += f": {'; '.join(error_details)}"
+            
+            print(f"Error: {error_msg}", file=sys.stderr)
+            return None, error_msg
 
         version = result.stdout.strip()
         if version and version != "none":
-            return version
+            return version, None
 
-        return None
+        return None, None
     except FileNotFoundError:
-        # Node.js not found
-        print(
-            "Warning: Node.js not found. Cannot resolve version via semantic-release.",
-            file=sys.stderr,
-        )
-        return None
+        # Node.js not found (shouldn't happen if check_node_available() passed, but handle gracefully)
+        if is_github_actions():
+            error_msg = """Node.js is not available in this GitHub Actions workflow.
+
+To fix this, add the following steps BEFORE running python-package-folder:
+
+- name: Setup Node.js
+  uses: actions/setup-node@v4
+  with:
+    node-version: '20'
+
+- name: Install semantic-release
+  run: |
+    npm install -g semantic-release semantic-release-commit-filter
+
+Alternatively, provide --version explicitly to skip automatic version resolution."""
+            print(f"Error: {error_msg}", file=sys.stderr)
+            return None, error_msg
+        else:
+            error_msg = "Node.js not found. Cannot resolve version via semantic-release."
+            print(f"Error: {error_msg}", file=sys.stderr)
+            return None, error_msg
     except Exception as e:
         # Other errors (e.g., permission issues, script not found)
-        print(
-            f"Warning: Error resolving version via semantic-release: {e}",
-            file=sys.stderr,
-        )
-        return None
+        error_msg = f"Error resolving version via semantic-release: {e}"
+        print(f"Error: {error_msg}", file=sys.stderr)
+        return None, error_msg
     finally:
         # Clean up temporary file if we extracted from zip/pex
         # This must be at function level to ensure cleanup even on early return
@@ -311,6 +397,46 @@ def main() -> int:
             # Version is needed for subfolder builds or when publishing main package
             if is_subfolder or args.publish:
                 print("No --version provided, attempting to resolve via semantic-release...")
+                
+                # Check Node.js availability upfront
+                if not check_node_available():
+                    if is_github_actions():
+                        error_msg = """Node.js is not available in this GitHub Actions workflow.
+
+To fix this, add the following steps BEFORE running python-package-folder:
+
+- name: Setup Node.js
+  uses: actions/setup-node@v4
+  with:
+    node-version: '20'
+
+- name: Install semantic-release
+  run: |
+    npm install -g semantic-release semantic-release-commit-filter
+
+Alternatively, provide --version explicitly to skip automatic version resolution."""
+                        print(f"Error: {error_msg}", file=sys.stderr)
+                    else:
+                        print(
+                            "Error: Node.js is not available. Cannot resolve version via semantic-release.",
+                            file=sys.stderr,
+                        )
+                        print(
+                            "Please install Node.js or provide --version explicitly.",
+                            file=sys.stderr,
+                        )
+                    return 1
+                
+                # Log that Node.js is available (for debugging)
+                node_version = subprocess.run(
+                    ["node", "--version"],
+                    capture_output=True,
+                    text=True,
+                    check=False,
+                )
+                if node_version.returncode == 0:
+                    print(f"Node.js detected: {node_version.stdout.strip()}")
+                
                 # Get repository info if publishing
                 repository = args.publish if args.publish else None
                 repository_url = args.repository_url if args.publish else None
@@ -322,7 +448,7 @@ def main() -> int:
                         " ", "-"
                     ).lower().strip("-")
                     subfolder_rel_path = src_dir.relative_to(project_root)
-                    resolved_version = resolve_version_via_semantic_release(
+                    resolved_version, error_details = resolve_version_via_semantic_release(
                         project_root,
                         subfolder_rel_path,
                         package_name,
@@ -344,7 +470,7 @@ def main() -> int:
                         except Exception:
                             pass
                     
-                    resolved_version = resolve_version_via_semantic_release(
+                    resolved_version, error_details = resolve_version_via_semantic_release(
                         project_root,
                         subfolder_path=None,
                         package_name=package_name_for_registry,
@@ -360,8 +486,11 @@ def main() -> int:
                         "This could mean:\n"
                         "  - No release is needed (no relevant commits)\n"
                         "  - semantic-release is not installed or configured\n"
-                        "  - Node.js is not available\n\n"
-                        "Please either:\n"
+                    )
+                    if error_details:
+                        error_msg += f"\nDetails: {error_details}\n"
+                    error_msg += (
+                        "\nPlease either:\n"
                         "  - Install semantic-release: npm install -g semantic-release"
                     )
                     if is_subfolder:

{python_package_folder-4.1.0.dist-info → python_package_folder-4.1.2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-package-folder
-Version: 4.1.0
+Version: 4.1.2
 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-4.1.0.dist-info → python_package_folder-4.1.2.dist-info}/RECORD

@@ -6,13 +6,13 @@ python_package_folder/finder.py,sha256=RPidZ7LKCFuQ_KgCFIZdHWPXsZIDor3M4C0hKeYW7
 python_package_folder/manager.py,sha256=Z9RPg0ZQ7jZhmEXfCzX9OrD_oiA5p2Pnm5Y9tgW3ObQ,55970
 python_package_folder/publisher.py,sha256=TSjdOvxvnWLbJCnduTK_xZBRfvsrq9kpEH-sfebeWkU,13507
 python_package_folder/py.typed,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-python_package_folder/python_package_folder.py,sha256=xSMUD_uiCOVHDDT4WNso9AxAYNyqhKI103G2rJztaXw,17907
+python_package_folder/python_package_folder.py,sha256=cwc-o96qG3drhdhiTNomdR3jVOABLSP8T8rgtTxxXNE,23854
 python_package_folder/subfolder_build.py,sha256=oH_KKLJIMByUZCl8y3AyohUO6Om0OvsIQ7Xg1fkd3jE,38782
 python_package_folder/types.py,sha256=3yeSRR5p_3PDKEAaehW_RJ7NwJHexOIeA08bGaT1iSY,2368
 python_package_folder/utils.py,sha256=lIkWsFKeAYAJ9TDUM99T4pUBHJVbUvCdUgkWQN-LUho,3111
 python_package_folder/version.py,sha256=kIDP6S9trEfs9gj7lBYGxrWm4RPssRla24UtlO9Jkh4,9111
-python_package_folder-4.1.0.dist-info/METADATA,sha256=yk49H6KBuadlyqKiHeC589X6pBvUTBVd4YXs6idsnYE,37517
-python_package_folder-4.1.0.dist-info/WHEEL,sha256=QccIxa26bgl1E6uMy58deGWi-0aeIkkangHcxk2kWfw,87
-python_package_folder-4.1.0.dist-info/entry_points.txt,sha256=ttu4wAhoYSHGhWQNercLz9IVTTpXxhVlRA9vSTvaLe0,91
-python_package_folder-4.1.0.dist-info/licenses/LICENSE,sha256=vNgRJh8YiecqZoZld7TtwPI5I72HIymKD9g32fiJjCE,1073
-python_package_folder-4.1.0.dist-info/RECORD,,
+python_package_folder-4.1.2.dist-info/METADATA,sha256=VfbnD998xzZY883EYPQlKAgmWEocNeLl_FVm7YvBsGU,37517
+python_package_folder-4.1.2.dist-info/WHEEL,sha256=QccIxa26bgl1E6uMy58deGWi-0aeIkkangHcxk2kWfw,87
+python_package_folder-4.1.2.dist-info/entry_points.txt,sha256=ttu4wAhoYSHGhWQNercLz9IVTTpXxhVlRA9vSTvaLe0,91
+python_package_folder-4.1.2.dist-info/licenses/LICENSE,sha256=vNgRJh8YiecqZoZld7TtwPI5I72HIymKD9g32fiJjCE,1073
+python_package_folder-4.1.2.dist-info/RECORD,,