Skip to content

Commit 23910e3

Browse files
authored
chore: move dev deps to [dependency-groups] so uv sync gives a working test env (#2217)
* chore: move dev deps to [dependency-groups] so uv sync gives a working test env [project.optional-dependencies] dev is now [dependency-groups] dev (PEP 735), which uv installs by default. This eliminates the four places that had to remember --extra dev (ci.yml shards + lint jobs, CONTRIBUTING.md, README.md) and prevents a fresh clone from silently running zero tests. worker, proxy, torrent, and e2e remain optional extras. uv.lock regenerated. * chore: regenerate uv.lock for the dev dependency-group (uv sync --frozen now installs test deps)
1 parent 3155fd0 commit 23910e3

5 files changed

Lines changed: 35 additions & 27 deletions

File tree

‎.github/workflows/ci.yml‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ jobs:
9595
# not from a fresh unpinned resolve that can pull in a regressed
9696
# release (see the fastapi 0.137 incident, #903).
9797
- name: Install dependencies
98-
run: uv sync --frozen --extra dev --python ${{ matrix.python-version }}
98+
run: uv sync --frozen --python ${{ matrix.python-version }}
9999

100100
# SPA bundle is stubbed by tests/conftest.py — see pytest_configure.
101101
# The real build is exercised in the spa-build job below.
@@ -168,7 +168,7 @@ jobs:
168168
enable-cache: true
169169

170170
- name: Install dependencies
171-
run: uv sync --frozen --extra dev --python 3.12
171+
run: uv sync --frozen --python 3.12
172172

173173
- name: Check for syntax errors
174174
run: uv run --no-sync python -m compileall tinyagentos/ -q

‎CONTRIBUTING.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ To keep this sustainable, **all contributors must agree to the Contributor Licen
1919
```bash
2020
git clone https://github.com/jaylfc/taOS.git
2121
cd tinyagentos
22-
uv sync --extra dev
22+
uv sync
2323
# Build the desktop SPA - static/desktop/ is gitignored (generated artifact)
2424
cd desktop && npm install && npm run build && cd ..
2525
uv run pytest tests/ --ignore=tests/e2e -n auto

‎README.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -686,7 +686,7 @@ uv run exo
686686
## Development
687687

688688
```bash
689-
uv sync --extra dev
689+
uv sync
690690
uv run pytest tests/ --ignore=tests/e2e -n auto # ~7,400 tests
691691
cd desktop && npx vitest run # ~1,900 desktop tests
692692
```

‎pyproject.toml‎

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,12 @@ proxy = ["litellm[proxy]>=1.93.0", "prisma>=0.11.0"]
6262
# dependency or fresh installs abort. torrent_downloader guards the import and
6363
# falls back to a direct download when it is absent.
6464
torrent = ["libtorrent>=2.0.9"]
65+
e2e = [
66+
"pytest>=9.1.1",
67+
"pytest-playwright>=0.5.0",
68+
]
69+
70+
[dependency-groups]
6571
dev = [
6672
"pytest>=9.1.1",
6773
"pytest-asyncio>=0.23.0",
@@ -75,10 +81,6 @@ dev = [
7581
"respx>=0.21.0",
7682
"websockets>=12.0",
7783
]
78-
e2e = [
79-
"pytest>=9.1.1",
80-
"pytest-playwright>=0.5.0",
81-
]
8284

8385
[tool.setuptools.packages.find]
8486
include = ["tinyagentos*"]

‎uv.lock‎

Lines changed: 25 additions & 19 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)