Skip to content

fix: ignore Emacs artifact files when refreshing the note cache#11

Open
guitmz wants to merge 3 commits into
jonathanchu:mainfrom
guitmz:ignore-emacs-locks
Open

fix: ignore Emacs artifact files when refreshing the note cache#11
guitmz wants to merge 3 commits into
jonathanchu:mainfrom
guitmz:ignore-emacs-locks

Conversation

@guitmz

@guitmz guitmz commented May 10, 2026

Copy link
Copy Markdown
Contributor

Prevent grove--refresh-cache from trying to parse editor artifact files as real notes.

  • add grove--cacheable-note-p to exclude:
    • Lockfiles like .#note.org
    • Autosaves like #note.org#
    • Backup files like note.org~
  • filter recursive note scans through that predicate before parsing
  • add regression tests covering:
    • a stale Emacs lock symlink whose target no longer exists
    • autosave and backup files alongside a real note

I think this should do it, maybe even too defensive even, but shouldn't cause any issues.

Fixes #10.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

grove--refresh-cache fails on Emacs lock files when inserting links

1 participant