ci(release-local): stop macOS builds from picking up a broken system Python - #1810
Conversation
…Python actions/setup-python's macOS interpreter has SQLite extension loading compiled out, so uv could grab it instead of its own capable managed Python during `uv sync`. sqlite_vec needs enable_load_extension, so the frozen api/worker binaries crashed on startup on every Mac. Drop the redundant setup-python step and pin UV_PYTHON_PREFERENCE=only-managed so uv never considers a PATH-found interpreter.
|
@AnishSarkar22 is attempting to deploy a commit to the Rohan Verma's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review. 📝 WalkthroughWalkthroughThe release workflow now forces uv to use its managed Python and removes the separate Python 3.12 setup step. ChangesRelease workflow
Priority: ➖ Normal Estimated code review effort: 1 (Trivial) | ~5 minutes Change: Bug fix Suggested reviewers: Merge Risk: ⚪ Minimal · up to The release workflow no longer depends on the removed setup step and should use the managed Python required by the build. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
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. Comment |
actions/setup-python's macOS interpreter has SQLite extension loading compiled out, so uv could grab it instead of its own capable managed Python during
uv sync. sqlite_vec needs enable_load_extension, so the frozen api/worker binaries crashed on startup on every Mac. Drop the redundant setup-python step and pin UV_PYTHON_PREFERENCE=only-managed so uv never considers a PATH-found interpreter.What
Why
Fixes #
How to test
High-level PR Summary
This PR fixes a macOS build issue where the frozen API and worker binaries were crashing on startup. The problem occurred because
actions/setup-pythonprovides a Python interpreter with SQLite extension loading disabled, which is required bysqlite_vec. The fix removes the redundantsetup-pythonstep and setsUV_PYTHON_PREFERENCE=only-managedto ensureuvalways uses its own managed Python interpreter with SQLite extension support enabled.⏱️ Estimated Review Time: 5-15 minutes
💡 Review Order Suggestion
.github/workflows/release-local.ymlSummary by CodeRabbit