Skip to content

feature/Failed-run-reason-shown · L-260925-b0e3ca - #44

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

lchoquel merged 4 commits into
devfrom
feature/Failed-run-reason-shown

Conversation

@lchoquel

@lchoquel lchoquel commented Sep 26, 2026 •

Copy link
Copy Markdown
Member

A failed durable run in the web app template now shows why it failed, read from the run's stored error report rather than the fixed "no result available" sentence: the runtime's title and message with the model provider's raw text cut out, the user action's detail as the next step, a retry line only when the report gives a retryable verdict, and a support line with the run id, the error type and when the run ended. A run with no stored report keeps today's wording, and docs/errors.md plus the template's CLAUDE.md describe where the reason comes from. The @pipelex/sdk git pin is the sprint's and collapses onto the published release before merge.

Closes L-260925-b0e3ca

🤖 Generated with Claude Code


Summary by cubic

A failed durable run in the web-app template now explains why it failed, reading the run's stored error report instead of always showing "no result available": the runtime's title and message, the user action's detail as the next step, a retry line only when the report gives a retryable verdict, and a support line with the run id, error type, and when the run ended. Runs with no stored report keep the old wording.

Bug Fixes

  • Pulls the report from the result lookup, falling back to the status read, and passes the run's finished_at for the support line.
  • Strips the model provider's raw error text from everything a person can read, including the technical details.
  • Drops wait_and_retry advice from a failed run, since nothing retries it after it ends.
  • Adds docs/errors.md and updates the template's CLAUDE.md to describe where the reason comes from.

Dependencies

  • Bumps @pipelex/sdk to ^0.26.0, collapsing the sprint's git pin onto the published release.

Closes L-260925-b0e3ca

Written for commit 51f0b3b. Summary will update on new commits.

Review in cubic

lchoquel and others added 4 commits September 26, 2026 19:08
`pipelex-method-apps` takes `@pipelex/sdk` from `pipelex-sdk-js`'s feature/Failed-run-report-js, written at the dependency entry of webapp-js/package.json with the lock regenerated in the same commit (P1). The collapse before this branch merges is `wt unpin _pipelex-method-apps--failed-run-reason-shown pipelex-sdk-js --to <version>` (P2, P7).
…rom its stored report

`pollDurableRun` now puts the run's stored error report (the result lookup's,
else the status read's) on the `RunFailedError` it classifies, and passes the
status read's `finished_at`. `classifyRunFailed` reads the display from the
report: its title in the headline, its message with the provider's raw text cut
out, its user action's detail as the hint, its `retryable` verdict as a retry
line, and a support line with the run id, the error type and when the run
ended. A run with no report keeps the SDK's sentence. `<ErrorDisplay>` renders
the retry line and the support line, and `docs/errors.md` describes where a
failed run's reason comes from.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FigDssaJrvNcmbnBedi7oq
A `wait_and_retry` user action says the system will retry automatically,
which is untrue of a run that has ended: nothing retries it. The failed-run
display now drops that advice and lets the retry line say what to do.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FigDssaJrvNcmbnBedi7oq
`pipelex-method-apps` takes `@pipelex/sdk` from the registry again, with the lock regenerated in the same commit. The pin stood at 98260ff0 of `pipelex-sdk-js` (P7).
@lchoquel
lchoquel merged commit eb49d0c into dev Sep 27, 2026
3 checks passed
@lchoquel
lchoquel deleted the feature/Failed-run-reason-shown branch September 27, 2026 10:50
lchoquel added a commit that referenced this pull request Sep 27, 2026
…ady in this branch

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant