Repository navigation
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 8a79efe. Configure here.
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 11, 2026 18:11
8a79efe to
10c8f44
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 11, 2026 21:45
10c8f44 to
0bc847d
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 11, 2026 21:49
0bc847d to
bcd99d3
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 11, 2026 22:04
bcd99d3 to
0aedf45
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
2 times, most recently
from
October 12, 2026 01:29
2e09087 to
28fb9ac
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 12, 2026 01:34
28fb9ac to
17c3d4d
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 12, 2026 18:32
17c3d4d to
53b1a79
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 12, 2026 22:06
53b1a79 to
fa7578a
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 12, 2026 22:34
fa7578a to
bb25ff9
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
2 times, most recently
from
October 13, 2026 00:05
0143542 to
d58bc88
Compare
chruffins
added this pull request to stack #508
October 13, 2026 00:06
This was referenced Oct 13, 2026
chruffins
force-pushed
the
feat/macos-snapshots
branch
2 times, most recently
from
October 14, 2026 03:14
0156859 to
0783bf8
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 14, 2026 06:15
0783bf8 to
17ee175
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 14, 2026 06:31
17ee175 to
646d028
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
2 times, most recently
from
October 14, 2026 16:38
2fbc60c to
fe12f31
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 14, 2026 16:55
fe12f31 to
f19d7d9
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 14, 2026 17:08
f19d7d9 to
f482aaa
Compare
chruffins
removed this pull request from stack #508
October 14, 2026 17:08
chruffins
added this pull request to stack #515
October 14, 2026 17:10
chruffins
marked this pull request as ready for review
October 14, 2026 19:12
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 14, 2026 20:10
f482aaa to
9d2f21a
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 15, 2026 00:27
9d2f21a to
ef1d3e2
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
2 times, most recently
from
October 15, 2026 19:25
1865c45 to
da3595f
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 15, 2026 22:56
da3595f to
8229622
Compare
chruffins
force-pushed
the
feat/macos-snapshots
branch
from
October 19, 2026 15:49
8229622 to
e2e2ec4
Compare
…e, and stopped clones Stopped snapshots after a graceful stop, same-instance restore through a staged disk swap, experimental same-instance standby, and clones from a stopped source with a fresh machine identifier, MAC and SSH host keys.
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Summary
PR 4 of 6. Snapshots, standby and stopped clones for macOS guests. Base: #500.
POST /restoreresumes it and finishes the restore (records the shim, deletes the consumed checkpoint).Linux code touched
lib/instances: snapshot, standby, restore, fork, start and stop gain macOS hooks;manager.goresumes Paused instances through a macOS completion hook and recovers interrupted restore staging at startup.lib/hypervisor/vz/client.go: save/restore support probing;tracing.goforwards it.lib/network/allocate.go:NewUniqueMAC, so macOS clones share the allocator's uniqueness rule.Not supported
Memory-snapshot compression, warm memory forks, and snapshots carried to another host or user account (VZ saved state is tied to both). Clones change machine identifier, MAC and SSH host keys only; hostname and accounts are copied.
Testing
Review focus
lib/instances/snapshot_macos.go(staged swap, recovery,finishMacOSResume).lib/instances/macos_identity.go(clone identity, MAC assignment, rekey teardown).Stack
Each PR is one commit on the one below it. macOS logic lives in its own files (
macos*.go,*_darwin.go,exec_lifecycle.go,port_forward.go); shared Linux files get small hooks such asif stored.MacOS != nil.