Skip to content

feat: allow clients to skip legacy replay - #50

Merged
lsaether merged 1 commit into
mainfrom
fix/legacy-replay-skip
May 26, 2026
Merged

lsaether merged 1 commit into
mainfrom
fix/legacy-replay-skip

Conversation

@lsaether

Copy link
Copy Markdown
Owner

Summary

  • add /acp?...&replay=skip to suppress the legacy WebSocket auto-replay for a single connection
  • keep default late-join replay unchanged for existing clients
  • preserve direct amux/session_context and post-connect live frames in skip mode
  • document the migration split: legacy clients use WS replay; attach-aware clients use replay skip plus session/attach history

Fixes #48.

Test plan

  • cargo test replay_skip_suppresses_legacy_history_but_keeps_context_and_live_frames -- --nocapture
  • cargo test replay_log_delivers_history_to_late_joiner -- --nocapture
  • cargo test validate_requires_session_and_peer_id -- --nocapture
  • git diff --check
  • cargo fmt -- --check
  • cargo clippy --all-targets -- -D warnings
  • cargo test

@lsaether
lsaether merged commit f592353 into main May 26, 2026
1 check passed
@lsaether
lsaether deleted the fix/legacy-replay-skip branch May 26, 2026 23:13
lsaether added a commit that referenced this pull request May 27, 2026
Cuts v0.1.3 — rooms + RFD #533 attach polish.

Highlights since v0.1.2:
- Rooms abstraction with compaction-aware transcripts (#58) and
  cross-segment turn-bookend carry in historyPolicy: full (#59).
  Closes #56.
- RFD #533-inspired session/attach + session/detach foundation (#46)
  with replay ordering (#47), streamed history delivery (#53), and
  ?replay=skip opt-out for attach-aware clients (#50).
- amux/* active-turn controls (steer #41, queue #41, cancel #30)
  plus replay-safe agent request lifecycle openings (#33).
- Safe-by-default client-tool blocking (fs/* + terminal/*) with
  --unsafe-debug-client-tool-broadcast escape hatch (#36).
- amux/session_context surfaces the mux/agent cwd to subscribers (#35).

Validation:
- cargo fmt --check
- cargo clippy --all-targets -- -D warnings
- cargo test (63 unit + 78 integration = 141 tests)

Co-authored-by: lsaether <25539605+lsaether@users.noreply.github.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.

Allow attach-aware clients to suppress legacy WS replay

1 participant