Skip to content

fix: persist failed headless transcripts - #354

Merged
Brian Krabach (bkrabach) merged 1 commit into
mainfrom
fix/headless-failure-transcript
Sep 20, 2026
Merged

Brian Krabach (bkrabach) merged 1 commit into
mainfrom
fix/headless-failure-transcript

Conversation

@bkrabach

Copy link
Copy Markdown
Collaborator

Summary

  • persist the accumulated headless transcript when execution fails before the normal save path
  • preserve existing session metadata and emit the normal store lifecycle events
  • keep persistence failures from masking the original execution error

Non-goals

  • no changes to interactive-session persistence
  • no changes to durable checkpoints, session ownership, or takeover behavior
  • no provider behavior or context-limit changes

Validation

  • Local suite: 2,422 passed, 1 skipped, 13 deselected, 1 expected xfail
  • DTU supplemental validation: 2,419 passed, 4 skipped, 13 deselected, 1 expected xfail
  • PTY validation: 13 passed
  • New regression coverage: 25 passed; base comparison had 12 expected failures
  • Real Context/Loop/root-writer path verified that a tool-result failure persists before cleanup and restores in a fresh process
  • Synthetic offline provider only; no live provider validation
  • Independent final review approved the reviewed diff

PR CI and post-merge CI remain required. No host installations.

Generated with Amplifier

Co-Authored-By: Amplifier <240397093+microsoft-amplifier@users.noreply.github.com>
@bkrabach
Brian Krabach (bkrabach) merged commit 95f9d2a into main Sep 20, 2026
9 checks passed
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.

2 participants