Skip to content

feature/Failed-run-reason · L-260925-3c79e2 - #2

Merged
lchoquel merged 6 commits into
devfrom
feature/Failed-run-reason
Sep 27, 2026
Merged

lchoquel merged 6 commits into
devfrom
feature/Failed-run-reason

Conversation

@lchoquel

@lchoquel lchoquel commented Sep 26, 2026 •

Copy link
Copy Markdown
Member

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 by widget attended, widget detached wait and widget 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 status prints 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 published pipelex-sdk 0.13.0, which carries that typed report, and raises the mthds floor 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, and widget detached result; widget detached status prints 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

  • The reason is the report's title and message (its error type when it carries neither), the next step its advice, and the retry line its retryable flag, left unsaid when unknown.
  • Server error text is escaped before Rich renders it, so bracketed spans in provider messages no longer disappear or crash the print.

Dependencies

  • Raises the floors to pipelex-sdk 0.13.0 and mthds 0.16.0, the versions carrying the typed error report.

Written for commit 911ac6c. Summary will update on new commits.

Review in cubic

lchoquel and others added 6 commits September 26, 2026 19:33
`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>
@lchoquel
lchoquel merged commit 705701e into dev Sep 27, 2026
18 checks passed
@lchoquel
lchoquel deleted the feature/Failed-run-reason branch September 27, 2026 10:45
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant