Skip to content

embedder: propagate producer failures to pending reads - #333

Merged
lannbot merged 1 commit into
mainfrom
fix/325-producer-failure-propagation
Sep 11, 2026
Merged

embedder: propagate producer failures to pending reads#333
lannbot merged 1 commit into
mainfrom
fix/325-producer-failure-propagation

Conversation

@lannbot

@lannbot lannbot commented Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

Fixes #325.

  • Surface the recorded site-named producer failure on pending stream reads, iterator/readable consumers, and unfinished direct-read sessions.
  • Retire failed futures through the existing abandonment path, waking readers and releasing host activity while preserving the producer cause.
  • Preserve completed payloads and direct sessions, and keep unrelated read failures primary.

Regression coverage includes failure before/after CABI binding, parked host operations, guest future abandonment, error identity, activity cleanup, and completed-result races.

Verification

  • just gates < /dev/null passed, including browser and consumer smokes.
  • Focused embedder suites: 71 passed, no skips.
  • Independent review approved after error-precedence and race-test corrections.

Automerge will be armed with a merge commit after opening; required core checks gate the merge.

@lannbot
lannbot enabled auto-merge September 11, 2026 01:42
@lannbot
lannbot merged commit ec87116 into main Sep 11, 2026
3 checks passed
@lannbot
lannbot deleted the fix/325-producer-failure-propagation branch September 11, 2026 01:46
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.

embedder: producer failures become false EOS or strand pending future awaits

2 participants