Skip to content

fix(commands,persist): Improve session listing UX when persist is disabled#7

Merged
mhiro2 merged 2 commits intomainfrom
fix/list-sessions-notify-when-disabled
Feb 15, 2026
Merged

fix(commands,persist): Improve session listing UX when persist is disabled#7
mhiro2 merged 2 commits intomainfrom
fix/list-sessions-notify-when-disabled

Conversation

@mhiro2
Copy link
Copy Markdown
Owner

@mhiro2 mhiro2 commented Feb 15, 2026

Summary

  • Added explicit user feedback when :PeekstackListSessions is executed while persist is disabled.
  • Switched session timestamp formatting from os.date to vim.fn.strftime to align with Neovim APIs.

Changes

  • d77dde2 — fix(persist): notify when list sessions is disabled
    • Added disabled-state guard to persist.list_sessions.
    • Added silent handling to control notification behavior.
    • Ensured early return with {} when persist is disabled.
    • Added/updated tests to verify:
      • notification is shown for :PeekstackListSessions when disabled
      • disabled list_sessions returns empty data and does not invoke async callback
  • a8f4da7 — fix(commands): use strftime for session updated_at
    • Replaced os.date with vim.fn.strftime for session updated_at display.
    • Added test coverage to assert vim.fn.strftime is actually used.

@mhiro2 mhiro2 self-assigned this Feb 15, 2026
@mhiro2 mhiro2 added the enhancement New feature or request label Feb 15, 2026
@mhiro2 mhiro2 merged commit f3df59e into main Feb 15, 2026
3 checks passed
@mhiro2 mhiro2 deleted the fix/list-sessions-notify-when-disabled branch February 15, 2026 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant