Repository navigation
Conversation
…auto-save autocmd
da3a5f0 to
19befbb
Compare
d3903fa to
1be7231
Compare
There was a problem hiding this comment.
⚠️ Not ready to approve
A logic inversion in the session directory creation check will prevent sessions from being saved correctly in common cases.
Pull request overview
This PR refactors sessions.nvim’s session naming and initialization flow, adds some filesystem/path utilities, and refreshes the README to reflect new setup and installation guidance.
Changes:
- Introduces hashed, filename-safe session names derived from the current working directory.
- Moves the
:Sessionuser command (and a Neovim version gate) into aplugin/entrypoint and simplifiessetup()to primarily configure auto-save. - Updates health checks and README installation/configuration sections.
File summaries
| File | Description |
|---|---|
| README.md | Reworks documentation for installation (incl. vim.pack) and configuration usage examples. |
| plugin/sessions.lua | Adds Neovim version check and defines the :Session user command + completion. |
| lua/sessions/util.lua | Adds path→filename helper + hashed session naming and filesystem predicate helpers. |
| lua/sessions/session.lua | Switches to hashed session filenames and refactors save/load flow using new util helpers. |
| lua/sessions/notify.lua | Minor comment/doc formatting changes. |
| lua/sessions/init.lua | Simplifies setup: config + auto-save autocmd management; removes command creation. |
| lua/sessions/health.lua | Refactors directory/session checks and reuses new util helpers. |
| lua/sessions/config.lua | Adjusts defaults/docs and changes how defaults are merged into runtime options. |
Copilot's findings
- Files reviewed: 7/8 changed files
- Comments generated: 3
Note
Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…in :source - correct docstring: readable to writeable
There was a problem hiding this comment.
⚠️ Not ready to approve
The new hashed session filename scheme appears to break backward compatibility for existing saved sessions and the README’s lazy-loading/version messaging needs clarification.
Copilot's findings
- Files reviewed: 7/8 changed files
- Comments generated: 3
Note
Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.
No description provided.