chore: track pipelex keyword-only refactor + migrate to PipelexMTHDSProtocol#75
Merged
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>
- Removed experimental feature flags from pipelex.toml. - Updated VSCode settings to enable format on save and organize imports. - Cleaned up ai_instruction_update.mthds by removing combined_output. - Adjusted changelog.mthds template formatting for consistency. - Removed combined_output from swe_docs.mthds. - Renamed TestingStrategyDoc to TestStrategyDoc in swe_docs.py and updated references. - Updated pipelex.toml to remove pipeline tracking and reporting flags. - Specified exact version for ruff in pyproject.toml and updated dependencies. - Updated uv.lock to reflect changes in package versions and sources.
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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
Tracks the
pipelexkeyword-only-arguments refactor and migrates the SWE runner toPipelexMTHDSProtocol. Moves from a temporary git pin to the releasedpipelex0.37.0 and cleans up configs/docs to match the new protocol.Refactors
PipelexRunnerwithPipelexMTHDSProtocolandexecute_pipeline()withexecute()incocode/swe/swe_cmd.pyand the hello‑world test;response.pipe_outputunchanged.pipelexconfigs; drop deprecatedcombined_outputin*.mthds, normalize the changelog template, and renameTestingStrategyDoc→TestStrategyDoc(fieldtesting_strategy→test_strategy) in SWE docs.Dependencies
pipelex0.37.0from PyPI (keyword‑only non‑subject params); removed the temporary local/git pin. Call sites already pass affected args by keyword.ruffto0.14.13; lockfile updated.Written for commit 0a52a82. Summary will update on new commits.