Skip to content

fix(mise): initialize Python dependencies in fresh worktrees #2429

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
matthewgrossman merged 3 commits into from
Jul 22, 2026
+7 −6
Merged

fix(mise): initialize Python dependencies in fresh worktrees #2429

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
dc0a686
fix(mise): Properly initialize venv for fresh worktrees
matthewgrossman Jul 22, 2026
d298792
fix(mise): use uv sync for protobuf generation
matthewgrossman Jul 22, 2026
647a1a2
docs: remove redundant worktree setup note
matthewgrossman Jul 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions architecture/build.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,9 +133,11 @@ do not infer from kube context.
## Python Wheel Packaging

The generated protobuf/gRPC stubs under `python/openshell/_proto/` are gitignored
build outputs of `mise run python:proto`. maturin honors `.gitignore` when
collecting `python-source` files, so native builds (Linux CI, local
`pip install .`) would drop them and ship an unimportable wheel. `pyproject.toml`
build outputs of `mise run python:proto`. The task uses `uv run --frozen` to
synchronize the current worktree's `.venv` from `uv.lock` before generation.
maturin honors `.gitignore` when collecting `python-source` files, so native
builds (Linux CI, local `pip install .`) would drop them and ship an unimportable
wheel. `pyproject.toml`
pins them back in with `[tool.maturin].include` globs. The release workflows
install each Linux wheel in a clean image and import `openshell.sandbox` as a
smoke check.
Expand Down
5 changes: 2 additions & 3 deletions tasks/python.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -218,11 +218,10 @@ hide = true

["python:proto"]
description = "Generate Python protobuf stubs from .proto files"
env = { UV_NO_SYNC = "1" }
run = """
#!/usr/bin/env bash
set -euo pipefail
uv run python -m grpc_tools.protoc \
uv run --frozen python -m grpc_tools.protoc \
-Iproto \
--python_out=python/openshell/_proto \
--pyi_out=python/openshell/_proto \
Expand All @@ -233,7 +232,7 @@ uv run python -m grpc_tools.protoc \
proto/options.proto \
proto/sandbox.proto
# Fix absolute imports in generated stubs to use package-relative imports
uv run python - <<'PY'
uv run --frozen python - <<'PY'
from pathlib import Path
import re

Expand Down
Loading