Skip to content

fix(providers): reject unsupported rewind before restoring files - #269

Merged
aaditagrawal merged 5 commits into
mainfrom
fix/alignment-provider-rewind
Sep 5, 2026
Merged

fix(providers): reject unsupported rewind before restoring files#269
aaditagrawal merged 5 commits into
mainfrom
fix/alignment-provider-rewind

Conversation

@aaditagrawal

@aaditagrawal aaditagrawal commented Sep 5, 2026

Copy link
Copy Markdown
Owner

Adapters that cannot rewind native conversations previously failed only after checkpoint restoration could change workspace files. Explicitly declares unsupported conversation rollback for Amp, Copilot, Cursor, Droid, Gemini and standard ACP adapters. Cursor now rejects rewind instead of only trimming local history while reporting success.

Validation: six real adapter suites passed (48 tests, two existing skips at this stack layer); checkpoint/provider service suites passed (97 tests). Workspace check/typecheck passed at the integrated tip.

@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.6 KiB 15.1 KiB
Codex Thread snapshot wire 7.0 KiB 7.3 KiB
Codex Live turn WebSocket wire 6.5 KiB 7.8 KiB
Codex Live turn WebSocket decoded 57.0 KiB 66.4 KiB
Codex Live turn messages 9 21
Claude Total thread wire 13.4 KiB 15.1 KiB
Claude Thread snapshot wire 7.0 KiB 7.3 KiB
Claude Live turn WebSocket wire 6.4 KiB 7.8 KiB
Claude Live turn WebSocket decoded 56.9 KiB 66.4 KiB
Claude Live turn messages 7 21

Baseline: unavailable · PR result: bb8837f · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 113.9 KiB
  • Claude decoded thread snapshot: 114.6 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

cursoragent and others added 2 commits September 5, 2026 06:41
… into fix/alignment-provider-rewind

Co-authored-by: aaditagrawal <aaditagrawal@users.noreply.github.com>
cursoragent and others added 2 commits September 5, 2026 06:49
… into fix/alignment-provider-rewind

Co-authored-by: aaditagrawal <aaditagrawal@users.noreply.github.com>
… into fix/alignment-provider-rewind

Co-authored-by: aaditagrawal <aaditagrawal@users.noreply.github.com>

aaditagrawal commented Sep 5, 2026

Copy link
Copy Markdown
Owner Author

Merge activity

@aaditagrawal
aaditagrawal changed the base branch from sync/alignment-20260905-current to main September 5, 2026 07:48
@aaditagrawal
aaditagrawal merged commit 82d320d into main Sep 5, 2026
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 effective changed lines (test files excluded in mixed PRs). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants