Skip to content

Some improvements to the way we allocate and track snapshot ids - #784

Merged
Dmitry Berkovich (dberkov) merged 1 commit into
agent-substrate:mainfrom
juli4n:snapshot_ids
Aug 8, 2026
Merged

Some improvements to the way we allocate and track snapshot ids#784
Dmitry Berkovich (dberkov) merged 1 commit into
agent-substrate:mainfrom
juli4n:snapshot_ids

Conversation

@juli4n

@juli4n Julian Gutierrez Oschmann (juli4n) commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

Fixes #773

Some improvements to the way we create, track and propagate snapshot ids.

  • Consolidate snapshot id and name into a single concept.
  • Use consistent terminology across the whole stack (i.e. remove confusion about prefix vs id vs name).
  • Get rid of internal only storage to hide the snapshot URI.

@juli4n Julian Gutierrez Oschmann (juli4n) changed the title WIP Some improvements to the way we allocate and track snapshot ids Aug 6, 2026
@juli4n
Julian Gutierrez Oschmann (juli4n) force-pushed the snapshot_ids branch 4 times, most recently from 3030f69 to 1ff1491 Compare August 7, 2026 16:00
@juli4n
Julian Gutierrez Oschmann (juli4n) marked this pull request as ready for review August 7, 2026 16:03
Comment thread internal/resources/snapshot.go
Comment thread cmd/ateapi/internal/store/ateredis/ateredis_test.go
…ids.

* Consolidate snapshot id and name into a single concept.
* Use consistent terminology across the whole stack (i.e. remove confusion about prefix vs id vs name).
* Get rid of internal only storage to hide the snapshot URI.
@dberkov
Dmitry Berkovich (dberkov) merged commit ddeee6c into agent-substrate:main Aug 8, 2026
11 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.

Some improvements to actor snapshots

3 participants