feature/Failed-run-reason · L-260925-3c79e2 - #2
Merged
Merged
Conversation
`pipelex-starter-python` takes `pipelex-sdk` from `pipelex-sdk-python`'s feature/Failed-run-report-python, written at [tool.uv.sources] of pyproject.toml with the lock regenerated in the same commit (P1). The collapse before this branch merges is `wt unpin _pipelex-starter-python--failed-run-reason pipelex-sdk-python --to <version>` (P2, P7).
…equires The pinned `pipelex-sdk` carries 0.12.0's `DownloadedArtifact.found_at`, a required field, so the stub download fixture and the artifact tests now build their fake verdict items with it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…its stored report A durable run that ended without a result used to print its status twice and hint at `status`, which printed it a third time. `widget/errors.py` now presents such a run from the error report the SDK carries on `RunFailedError`, `RunResultFailed` and `RunRead`: the reason (title and message), the next step and the retry advice, the same lines under `detached status`. A run with no stored report says that no reason was recorded, keeps the platform's sentence and hints at what is left. `print_error` prints every presentation and escapes the server's text so Rich never reads it as markup. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The entry claimed every server error message was printed as it came, while download errors and the usage assembly error still go through Rich markup; it now names the error presentation and the status read-out, which are what this branch escapes. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
`pipelex-starter-python` takes `pipelex-sdk` from the registry again, with the lock regenerated in the same commit. The pin stood at ffcb5c6e of `pipelex-sdk-python` (P7).
pipelex-sdk 0.13.0 pins mthds exactly at 0.16.0, and the manifest's comment holds the mthds floor at the version the SDK pins, so the floor follows it. The changelog names both floors, since the failed-run presentation needs the SDK's typed error report that first shipped in 0.13.0. Co-Authored-By: Claude Opus 5.5 <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 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.
A durable run that ended without a result used to print its status three times and never its reason. The failed-run presentation in
widget/errors.py, used bywidget attended,widget detached waitandwidget detached result, now reads out the stored error report the SDK carries (the reason from its title and message, the next step and the retry advice),widget detached statusprints the same lines under the status, and a run with no stored report says that no reason was recorded and what is left to do. It depends on the publishedpipelex-sdk0.13.0, which carries that typed report, and raises themthdsfloor to the 0.16.0 that release pins.Closes L-260925-3c79e2
🤖 Generated with Claude Code
Summary by cubic
A failed durable run used to print its status three times and never why it failed. The failed-run presentation now reads out the stored error report—the reason, next step, and retry advice—across
widget attended,widget detached wait, andwidget detached result;widget detached statusprints the same lines under the status, and a run with no stored report says no reason was recorded and what is left to do. Closes L-260925-3c79e2.Bug Fixes
Dependencies
pipelex-sdk0.13.0 andmthds0.16.0, the versions carrying the typed error report.Written for commit 911ac6c. Summary will update on new commits.