chore: track pipelex keyword-only refactor + migrate to PipelexMTHDSProtocol#75
Open
lchoquel wants to merge 3 commits into
Open
chore: track pipelex keyword-only refactor + migrate to PipelexMTHDSProtocol#75lchoquel wants to merge 3 commits into
lchoquel wants to merge 3 commits into
Conversation
…rotocol Pin pipelex to the keyword-only-arguments refactor branch (git rev) and migrate the SWE runner off the removed PipelexRunner class. - pyproject/uv.lock: pipelex ==0.32.1 -> ==0.33.0 via git rev 529b9082 - swe_cmd.py + integration test: PipelexRunner -> PipelexMTHDSProtocol, execute_pipeline() -> execute() (response.pipe_output unchanged) make agent-check and agent-test pass; uv lock --check is consistent. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Greptile SummaryThis PR updates cocode for the newer Pipelex runner API and temporary dependency pin. The main changes are:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (1): Last reviewed commit: "chore: track pipelex keyword-only refact..." | Re-trigger Greptile |
…ngs) Tracks the latest pipelex keyword-only refactor commit, which reorders several function signatures to keyword-only. No call-site changes needed here — every call site already passes the affected args by keyword. agent-check + agent-test green. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Bumps the pinned pipelex git rev 755b8211 -> 0e32c8c0. That commit makes all remaining non-subject params keyword-only (no reordering); all call sites here already pass by keyword, so this is a no-op bump. agent-check + agent-test green. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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
Pins
cocodeto the pipelex keyword-only-arguments refactor branch (refactor/Function-calling-4) to validate it in lockstep, and migrates the SWE runner off the removedPipelexRunnerclass.Changes
pyproject.toml/uv.lock:pipelex ==0.32.1→==0.33.0via a temporary[tool.uv.sources]git pin at rev529b9082.cocode/swe/swe_cmd.py(production) +tests/integration/test_hello_world.py:PipelexRunner→PipelexMTHDSProtocol,execute_pipeline()→execute().response.pipe_outputis unchanged.CHANGELOG.md:[Unreleased]entry.Why the runner rename is here
The pinned PyPI
pipelex(0.32.1) still hadPipelexRunner. Pinning to the branch (which tracks pipelexdev) surfaces the already-landed runner rename — the runner is now the MTHDS Protocol implementation (PipelexRunnerremoved,execute_pipeline→execute) — in addition to the keyword-only work, so cocode needs the rename to keep building.Verification
Local, all green:
make agent-check,make agent-test,uv lock --check.Note
The git-rev pin is temporary — swap it back to a released
pipelexversion once the keyword-only refactor lands on PyPI.🤖 Generated with Claude Code
Summary by cubic
Pins
pipelexto the latest keyword-only refactor (git rev0e32c8c0) and migrates the SWE pipeline toPipelexMTHDSProtocolto stay compatible.Refactors
PipelexRunnerwithPipelexMTHDSProtocolandexecute_pipeline()withexecute()incocode/swe/swe_cmd.pyand the hello-world test;response.pipe_outputunchanged.Dependencies
pipelex0.33.0via a temporary git source at rev0e32c8c0(all remaining non-subject params are now keyword-only). No call-site changes needed; switch back to a released version once available.Written for commit b82d26c. Summary will update on new commits.