Add per-session save files/session manager script#557
Open
philippossfrn wants to merge 1 commit into
tmux-plugins:mastertmux-plugins/tmux-resurrect:masterfrom
philippossfrn:feature/per-session-saves-with-managerphilippossfrn/tmux-resurrect:feature/per-session-saves-with-managerCopy head branch name to clipboard
Open
Add per-session save files/session manager script#557philippossfrn wants to merge 1 commit intotmux-plugins:mastertmux-plugins/tmux-resurrect:masterfrom philippossfrn:feature/per-session-saves-with-managerphilippossfrn/tmux-resurrect:feature/per-session-saves-with-managerCopy head branch name to clipboard
philippossfrn wants to merge 1 commit into
tmux-plugins:mastertmux-plugins/tmux-resurrect:masterfrom
philippossfrn:feature/per-session-saves-with-managerphilippossfrn/tmux-resurrect:feature/per-session-saves-with-managerCopy head branch name to clipboard
Conversation
This was referenced Jan 29, 2026
laur89
reviewed
Feb 1, 2026
| local attach_script="$bin_dir/attach" | ||
|
|
||
| # Only create if attach script doesn't already exist | ||
| if [ -f "$attach_script" ]; then |
There was a problem hiding this comment.
Wouldn't mix posix [] with bash's [[]]. If we're writing bash, let's stay in bash syntax.
This applies not only to this line, but throughout this patch.
Author
There was a problem hiding this comment.
Good point, fixed. Replaced all [ ] with [[ ]] for consistent bash syntax throughout the script.
|
|
||
| # List saved sessions | ||
| echo "Available saved tmux sessions:" | ||
| SAVED_SESSIONS=($(ls "$RESURRECT_DIR" | grep -vE 'last|readme')) |
There was a problem hiding this comment.
ls is generally frowned upon in script usage. Best collect directory contents using find. That way there's also no need for the filtering grep, as find can provide that functionality.
| cp "$RESURRECT_DIR/$SELECTED_SESSION" "$RESURRECT_DIR/last" | ||
|
|
||
| # Ensure the tmux server is running by creating a detached session | ||
| tmux new-session -d -s temp_session 2>/dev/null |
There was a problem hiding this comment.
Maybe I'm paranoid, but would randomize the temp session name just in case. E.g. temp_session="temp-session-$RANDOM"
Author
There was a problem hiding this comment.
Good catch, also added a timestamp suffix to avoid creating more than one session with the same name
philippossfrn
force-pushed
the
feature/per-session-saves-with-manager
branch
3 times, most recently
from
February 2, 2026 21:13
84cb3b9 to
11c2757
Compare
- Modified resurrect_file_path() to save sessions using session name
- Prompts user to name session if using default numeric name
- Added session manager script to $HOME/bin/ on install
- Interactive menu to attach, delete, or rename saved sessions
- Allows multiple distinct session saves instead of single global file
- Randomize temp session name with PID and $RANDOM to avoid collisions
- Only kill temp session if we successfully created it
- Use dynamic path for restore.sh based on plugin install location
- Add Unix timestamp suffix to saved session filenames
- Add custom keybinding example to README
- Use find cmd to list saved sessions
philippossfrn
force-pushed
the
feature/per-session-saves-with-manager
branch
from
March 19, 2026 08:38
11c2757 to
dc1dd01
Compare
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.
What the change does
This modification enhances tmux-resurrect to save session state on a per-session basis rather than using a single global save file. Additionally, it includes a session manager script that
provides an interactive menu for managing saved sessions.
Core changes:
Why it's useful
Breaking changes
naming scheme.