Last level fallback to install uv into venv (penv)

[Jason2866]

Description:

Related issue (if applicable): fixes #

Checklist:

• The pull request is done against the latest develop branch
• Only relevant files were touched
• Only one feature/fix was added per PR, more changes are allowed when changing boards.json
• I accept the CLA

Summary by CodeRabbit

• Chores
  • Strengthened the dependency installation process by adding fallback mechanisms for the uv package manager to improve reliability across different system environments.
  • Enhanced error handling with better timeout management and comprehensive status messages for installation outcomes.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

📝 Walkthrough

Walkthrough

Introduces a platform-specific fallback installer function for uv that handles Windows PowerShell and Unix curl/wget flows, with comprehensive post-install verification. Enhanced install_python_deps to attempt this fallback when primary uv installation methods fail, including additional status messaging and exception handling.

Changes

Cohort / File(s)	Summary
UV Fallback Installation Logic builder/penv_setup.py	Added install_uv_fallback() function implementing platform-specific installation (Windows PowerShell script vs. Unix curl/wget), post-install verification, and timeout/exception handling. Enhanced install_python_deps() to invoke fallback when initial uv installation fails, with additional warning messages and control-flow adjustments. Imported shutil module for utility location.

Sequence Diagram

sequenceDiagram
    participant Client as Build Process
    participant Installer as install_python_deps()
    participant Fallback as install_uv_fallback()
    participant System as System Package Manager
    participant FS as File System

    Client->>Installer: Attempt uv installation
    Installer->>Installer: Primary installation method
    alt Primary method fails
        Installer->>Fallback: Invoke fallback installer
        alt Windows Platform
            Fallback->>System: Execute PowerShell script
        else Unix Platform
            Fallback->>System: Execute curl/wget download
        end
        System->>FS: Download & install uv
        Fallback->>FS: Verify uv executable
        alt Verification successful
            Fallback->>Installer: Return uv path
            Installer->>Client: Continue with system uv
        else Verification fails
            Fallback->>Installer: Return None
            Installer->>Client: Abort & report failure
        end
    else Primary method succeeds
        Installer->>Client: Return success
    end

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• Fall back to pip when external uv fails to install uv into penv #387 — Adds fallback behavior to uv installation flow in the same file when external installation fails
• Ensure packages are checked from penv #253 — Modifies install_python_deps and uv executable resolution, complementing the fallback installer addition
• Remove use of pip in espidf.py to create the venv #265 — Supports UV-based venv/dependency installation flow that relies on the system-wide UV fallback mechanism

Poem

🐰 A hop, skip, and jump through the build,
When uv won't install as we'd willed,
Now PowerShell scripts and curl do conspire,
Cross platforms they fly, ever higher,
With fallbacks in place, we'll never retire! 🚀

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically describes the main change: adding a fallback mechanism to install uv into a virtual environment (penv), which aligns with the primary objective of the pull request.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Merge Conflict Detection	✅ Passed	✅ No merge conflicts detected when merging into develop

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch uv_install

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Jason2866]

@coderabbitai full review

[coderabbitai]

✅ Actions performed

Full review triggered.