Skip to content

fix: bump deepagents floor to >=0.5.3 to unblock CI - #137

Closed
chrispatil wants to merge 1 commit into
tkellogg:mainfrom
chrispatil:fix/deepagents-version-pin
Closed

fix: bump deepagents floor to >=0.5.3 to unblock CI#137
chrispatil wants to merge 1 commit into
tkellogg:mainfrom
chrispatil:fix/deepagents-version-pin

Conversation

@chrispatil

Copy link
Copy Markdown
Contributor

What

CI on main is currently red (has been since commit 11fede7, 2026-08-05).
pyproject.toml pins deepagents>=0.4.1, but open_strix/app.py calls
CompositeBackend(..., artifacts_root=...) -- a kwarg that doesn't exist
until deepagents 0.5.x. A clean uv sync --dev resolves to 0.4.1 and every
test that constructs an OpenStrixApp fails at agent-build time with:

TypeError: CompositeBackend.__init__() got an unexpected keyword argument 'artifacts_root'

Confirmed independently via the GitHub Actions API: the Prerelease run for
main's current HEAD has conclusion: failure, with no follow-up fix commit
since. Reproduced locally on a pristine clone using CI's literal
uv sync --dev && pytest tests/ ... invocation: 86 failed, 252 passed.

The actual runtime this was caught in already has deepagents==0.5.3
installed (hand-patched outside of git at some point) -- the repo's own lock
file was never updated to match, so a fresh clone/CI run reproduces the
original break.

Fix

Bump the floor to deepagents>=0.5.3,<0.6 and regenerate the lockfile
(resolves to deepagents==0.5.9, matching what's already proven-good in
production). Also verified an unconstrained >=0.5.3 (resolves to current
latest 0.7.8) is fully green too, if a floating latest is preferred instead
of a pinned minor -- happy to switch.

Testing

Pristine clone, CI's exact pytest tests/ invocation (current 9-file ignore
list): 86 failed -> 0 failed, 252 passed -> 338 passed.

Why this matters for the other PRs in this batch

Every other branch needed this fix synced locally before its own tests could
even run -- OpenStrixApp() can't construct an agent without it. Recommend
merging this one first; the others should rebase onto it.

…ith CompositeBackend usage)

open_strix/app.py calls CompositeBackend(..., artifacts_root=...), a
keyword argument that does not exist on deepagents 0.4.1. The pyproject
floor (deepagents>=0.4.1) is loose enough that a clean 'uv sync' resolves
to 0.4.1, which cannot construct the backend at all -- OpenStrixApp
fails to build an agent, and 86 tests across 20+ files fail with
'CompositeBackend.__init__() got an unexpected keyword argument
artifacts_root'.

Confirmed via GitHub Actions API that the current main HEAD (11fede7)
has a failing 'Prerelease' run for exactly this reason, and that it has
been red since 2026-08-05 with no follow-up fix commit. The production
runtime already has deepagents==0.5.3 installed (apparently patched by
hand outside of git), which is why this hasn't been noticed operationally
-- but a fresh clone + uv sync (i.e. any new contributor, or CI on a
cache-bust) reproduces the failure immediately.

Verified: full test suite (minus the 9 permanently-ignored files) goes
from 86 failed / 252 passed to 0 failed / 338 passed after this bump.
@chrispatil

Copy link
Copy Markdown
Contributor Author

Closing as duplicate — #136 (filed 2026-08-05, still open) already fixes this exact break with a nearly identical diff (unbounded >=0.5.1 vs. this PR's >=0.5.3,<0.6; both verified green). Should have checked open PRs before filing. Redirecting the 3 dependent PRs in this batch (#138, #139, #140) to reference #136 instead.

@chrispatil chrispatil closed this Aug 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant