fix(checkpoints): /rollback no longer reports "No checkpoints found" while other projects have them (#10505) - #90459
Merged
Conversation
Contributor
૮ >ﻌ< ა ci reviewran on f0a2fa6 — fix(checkpoints): bare /rollback falls back to a labeled all
|
alt-glitch
added
type/bug
This was referenced Aug 20, 2026
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
2 participants
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
Bare
/rollbacknow falls back to a labeled cross-project checkpoint list instead of reporting "No checkpoints found" while checkpoints exist under another directory.Surgical reapply of PR #10633 by @nightq (fixes #10505) onto the v2 single-store checkpoint layout — the original targeted the pre-v2 per-shadow-dir scan and no longer applied.
Reproduced live in the round-2 QA sweep: the agent's writes checkpointed the session cwd (
/tmp/qa-repo, 2 fresh checkpoints), but bare/rollbacksearched onlyTERMINAL_CWD(/home/teknium) and printed "No checkpoints found for /home/teknium" — the user's undo path looked empty seconds after a write.Changes
tools/checkpoint_manager.py:list_all_checkpoints()— aggregates every registered project viaprojects/*.json, most-recent-first, each entry carryingworkdir;format_checkpoint_list()shows a[dirname]label in the all-directories viewhermes_cli/cli_commands_mixin.py: bare/rollbackfalls back to the cross-project view (with an explicit "showing all directories" notice) when the cwd project has nonetests/tools/test_rollback_all_directories.py: aggregation, empty-store, and label/no-label formatting contractsValidation
/rollbackfrom a cwd without checkpoints[qa-repo]-labeled list (live-verified in tmux)Original PR to close on merge: #10633 (credited author; commit reapplied rather than cherry-picked because the v2 store rewrite obsoleted the original patch base).
Infographic