Prevent saving an "empty" session about persisted.nvim HOT 8 CLOSED

@loqusion - really good suggestion! Much cleaner and will cover anyone's edge case. I've added this in the latest commit

from persisted.nvim.

I've marked this as an enhancement and will try and pick this up in a couple of weeks

from persisted.nvim.

Another option is to use the return value of a callback (e.g. true or false) to determine whether a session should be auto-saved — that way, any condition can be implemented. We could either use the existing before_save or introduce a new callback like should_save or should_autosave, or perhaps even allow the user to pass a callback to the existing autosave as an alternative to a boolean. It shouldn't affect manual SessionSaves, of course.

from persisted.nvim.

Are you lazy loading the plugin? I use this for my config:

use({
  "olimorris/persisted.nvim",
  module = "persisted", -- For lazy loading
  config = function()
    require("persisted").setup()
    require("telescope").load_extension("persisted") -- To load the telescope extension
  end,
})

and my "Find Session" command calls require("persisted").load().

I take your point though that there could be times when an empty session is saved to disk. Defo a risk if you don't lazy load.

from persisted.nvim.

Hmm, how would lazyloading work with autoloading sessions? My use case is that I want to load alpha dashboard if session autoload fails, my reasoning being that manually loading an existing session for the current workdir is an unnecessary extra step.

from persisted.nvim.

It wouldn't; misread that first part of the issue 😄. Really good use case btw.

I think there's two components we'd need:

• A global variable to inform other plugins (or your config) that no session has been autoloaded (a global variable feels like the cleanest way of doing this)
• The ability to ignore the alpha filetype, ensuring that a blank session is not saved to disk if you quit Neovim from the dashboard

Do you agree?

from persisted.nvim.

That should do it, or at least I can't think of a reason why that wouldn't work.

from persisted.nvim.

That's really nice, works like a charm and makes it easy to update if there's more conditions in the future! Takkâ ennuv!

from persisted.nvim.