feature/Starter-sdk-0140-typed · L-260927-23e105 - #4
Merged
Merged
Conversation
… error pipelex-sdk 0.14.0 raises the typed ApiResponseError on every route, the protocol ones included, so the raw httpx.HTTPStatusError branch is gone: each mode's _run() catches PipelineRequestError alone, and httpx is no longer a runtime dependency. A refused request is read out by problem_lines the way a failed run's report is: the reason, the pipe or concept each validation item names, the next step and the retry advice, with a multi-line reason hanging under its label. The codegen explain helper drops its httpx arm too, prints the server's next step, and no longer reads a 404 carrying the runner's error class as a missing route. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Xcr1G5gkEa4Aeoo1a7ATyh
… or start The `_run()` docstrings, the architecture doc, the error module's docstring, the changelog and the pyproject comment said or implied that every error the SDK raises descends from `PipelineRequestError`. A connection that fails on `execute` or `start` does not: pipelex-sdk 0.14.0 sends those routes through the base client's unwrapped transport, so a mistyped PIPELEX_BASE_URL still crashes with httpx's own ConnectError, as it did on 0.13.0. The sentences now say so; the fix belongs in the SDK. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Xcr1G5gkEa4Aeoo1a7ATyh
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.
The starter moves onto pipelex-sdk 0.14.0 and mthds 0.17.0, where every route raises the typed
ApiResponseErroron a non-2xx answer, so the rawhttpx.HTTPStatusErrorbranch is gone andhttpxleaves the runtime dependencies. A refused run now prints its reason, the pipe or concept each validation error names, the server's next step and the retry advice, in every mode, with theStartRequiresAsyncOrchestrationhint moved onto the typed branch;make codegenandmake add-methodalso stop reading a 404 the route answered as a missing route.Closes L-260927-23e105
🤖 Generated with Claude Code
https://claude.ai/code/session_01Xcr1G5gkEa4Aeoo1a7ATyh
Summary by cubic
Moves the starter onto
pipelex-sdk0.14.0 andmthds0.17.0, where every route raises the typedApiResponseErroron a non-2xx answer. The rawhttpx.HTTPStatusErrorbranch is gone from all three modes, andhttpxleaves the runtime dependencies (it stays dev-only for the test transport fakes).Refused requests now say why, where, and what to do
StartRequiresAsyncOrchestrationhint now rides on the typed error.Codegen scripts stop misreading 404s
explainsends you toPIPELEX_BASE_URLonly for a bare 404; a 404 carrying the runner's error class (e.g. a missingmethod_refpackage) prints the server's reason instead.Closes L-260927-23e105.
Written for commit 89c05f6. Summary will update on new commits.