Skip to content

Commit ee1d1fe

Browse files
fix(ci): use rye workspace instead of rye run pip (which doesn't exist)
Last commit tried `rye run pip install -e ./adk` in scripts/bootstrap, but rye's uv-backed venv doesn't include pip by default — `rye run pip` fails with "invalid or unknown script 'pip'". The rye-native way to install workspace members is `[tool.rye.workspace]`. The earlier suspicion that workspaces "didn't work" was wrong — the real problem was the agentex/ namespace conflict between two editable installs, which pkgutil.extend_path (added in src/agentex/__init__.py) now fixes. So: reinstate the workspace config, drop the broken pip install line. `rye sync --all-features` will install both root and adk/ editably and extend_path merges their namespaces at import time. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent 0796d55 commit ee1d1fe

2 files changed

Lines changed: 7 additions & 8 deletions

File tree

pyproject.toml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -55,6 +55,13 @@ dev = [
5555
# The `agentex` CLI entry point ships from the ADK package — see
5656
# `adk/pyproject.toml`. The slim client has no CLI surface.
5757

58+
[tool.rye.workspace]
59+
# `rye sync` from the root installs both this slim package and the ADK
60+
# overlay editably. The two contribute disjoint files to the same agentex.*
61+
# namespace — see the pkgutil.extend_path call in src/agentex/__init__.py
62+
# which merges their __path__ entries at import time.
63+
members = ["adk"]
64+
5865
[tool.rye]
5966
managed = true
6067
# version pins are in requirements-dev.lock

scripts/bootstrap

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,3 @@ echo "==> Installing Python dependencies…"
2525
rye config --set-bool behavior.use-uv=true
2626

2727
rye sync --all-features
28-
29-
# Install the sibling ADK package (agentex-sdk) editably so `agentex.lib.*`
30-
# imports resolve from the local source. The slim agentex-sdk-client (the
31-
# root package) has its agentex/__init__.py call pkgutil.extend_path so
32-
# the two source trees merge into the same `agentex.*` namespace at import
33-
# time.
34-
echo "==> Installing ADK overlay (agentex-sdk) editable…"
35-
rye run pip install -e ./adk

0 commit comments

Comments
 (0)