Skip to content

fix(session): hide persisted reminders in resume history - #333

Merged
Brian Krabach (bkrabach) merged 1 commit into
mainfrom
fix/resume-hide-persisted-reminders
Sep 8, 2026
Merged

Brian Krabach (bkrabach) merged 1 commit into
mainfrom
fix/resume-hide-persisted-reminders

Conversation

@bkrabach

Copy link
Copy Markdown
Collaborator

Summary

  • Add one shared UI-only classifier for persisted reminder envelopes.
  • Apply it to session history, replay timing, and interrupted replay remainder handling.
  • Keep transcript storage and resume context unchanged; document the display-only contract.

Why

Persisted system reminders are needed for resumed context but should not appear as conversation history or consume the visible history window. Human-authored, malformed, mixed, and unflagged content remains visible.

Verification

  • uv run --with truststore==0.10.4 pytest -q — 2113 passed, 1 skipped, 13 deselected, 1 xfailed.
  • Deterministic DTU verification passed: real PTY amplifier session resume and --replay exited 0; fixture-backed checks covered hidden identified reminders, visible human/unflagged quotes, default-window behavior, unchanged transcript bytes, and retained resume context. This validates the fixture-shim scope; it does not claim live vendor or full default-bundle coverage.

Compatibility

No breaking changes. No provider requests, version changes, global CLI behavior, Foundation changes, or workspace-root changes.

Generated with Amplifier

Co-Authored-By: Amplifier <240397093+microsoft-amplifier@users.noreply.github.com>
@bkrabach
Brian Krabach (bkrabach) merged commit dfa56a7 into main Sep 8, 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