Skip to content

feat(sandbox): teleport snapshots and GC leftover tars - #677

Closed
justrach wants to merge 1 commit into
mainfrom
cursor/sandbox-teleport-gc-4ffc
Closed

feat(sandbox): teleport snapshots and GC leftover tars#677
justrach wants to merge 1 commit into
mainfrom
cursor/sandbox-teleport-gc-4ffc

Conversation

@justrach

Copy link
Copy Markdown
Owner

Closes the leftover half of #554.

What changed

  • /teleport <id> [docker|container] restores a captured docker_image_tar onto a different CLI backend. Apple Container (container) is the second backend — same wire as Docker, different bin_name. Dest defaults to container when the live backend is docker or nothing is attached.
  • /snapshot gc [n] keeps the newest n snapshot trees (default 1) and deletes the rest. Missing dest CLIs stay explanations, not crashes.
  • ADR 0042 records the leftover decision. Experiment-pool Docker snapshots still point here (ADR 0037).

Why

#554 shipped the seam, Docker CLI backend, /snapshot, and /rewind <id>. Teleport and snapshot GC were the two exo misses left open. Conversation rewind is still never implied — every restore reprints the filesystem-only note.

Verification

  • scripts/eval-tier1.sh via pre-push: fmt, 600-line ceiling, reachability, build, 1759 unit tests (was 1753 on main; +6), TUI suite, PTY probes, invariants, SDK sync.
  • Focused #554 filter: 61/61 including the new gc / teleport / missing-CLI cases.
Open in Web Open in Cursor 

Close the leftover #554 gaps: /teleport restores a docker_image_tar on
the Apple Container CLI (same wire, different binary), and /snapshot gc
keeps the newest n snapshots. Missing dest CLIs stay explanations.
@justrach
justrach marked this pull request as ready for review August 29, 2026 17:41
cursor Bot pushed a commit that referenced this pull request Aug 31, 2026
The parked Smolify and idle-localhost PRs are closed as abandoned vs tip;
issues stay inventory. Live leftovers remain #675 and #677.
cursor Bot pushed a commit that referenced this pull request Aug 31, 2026
#677 leftover of #554: /teleport restores a snapshot tar onto another
CLI backend; /snapshot gc keeps the newest n trees. ADR 0051 (their
0042 collided with TUI claims).

#675: lean -p bounces prose-only first "done"; short Codegraff flakes
retry twice; DeepSeek thinking off at default low; lean bash
auto-backgrounds at 15s. ADRs remapped 0048-0051 -> 0052-0055.

Suite floor 1861. Keep-alive-only bodies share the short-flake retry
so the request-scratch split stays.

Co-authored-by: yxlyx <85774423+yxlyx@users.noreply.github.com>
@justrach

Copy link
Copy Markdown
Owner Author

Folded into #696 (release/v0.0.282). Leftover of #554: /teleport + /snapshot gc. ADR remapped 0042 → 0051 (0042 on 282 is TUI claims screen).

Unit tests (fake docker, no daemon) pass, including /snapshot gc, /teleport onto container, and missing dest CLI. Closing as superseded.

@justrach justrach closed this Aug 31, 2026
@justrach
justrach deleted the cursor/sandbox-teleport-gc-4ffc branch August 31, 2026 07:37
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