fix: bump deepagents floor to >=0.5.3 to unblock CI - #137
Closed
chrispatil wants to merge 1 commit into
Closed
Conversation
…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.
Contributor
Author
|
Closing as duplicate — #136 (filed 2026-08-05, still open) already fixes this exact break with a nearly identical diff (unbounded |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
CI on
mainis currently red (has been since commit11fede7, 2026-08-05).pyproject.tomlpinsdeepagents>=0.4.1, butopen_strix/app.pycallsCompositeBackend(..., artifacts_root=...)-- a kwarg that doesn't existuntil deepagents 0.5.x. A clean
uv sync --devresolves to0.4.1and everytest that constructs an
OpenStrixAppfails at agent-build time with:Confirmed independently via the GitHub Actions API: the
Prereleaserun formain's current HEAD hasconclusion: failure, with no follow-up fix commitsince. 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.3installed (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.6and regenerate the lockfile(resolves to
deepagents==0.5.9, matching what's already proven-good inproduction). Also verified an unconstrained
>=0.5.3(resolves to currentlatest
0.7.8) is fully green too, if a floating latest is preferred insteadof a pinned minor -- happy to switch.
Testing
Pristine clone, CI's exact
pytest tests/invocation (current 9-file ignorelist):
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. Recommendmerging this one first; the others should rebase onto it.