Skip to content

fix: use correct excalidraw-plugin frontmatter key#84

Merged
rogerdigital merged 1 commit into
mainfrom
fix/excalidraw-frontmatter-key
Jun 23, 2026
Merged

fix: use correct excalidraw-plugin frontmatter key#84
rogerdigital merged 1 commit into
mainfrom
fix/excalidraw-frontmatter-key

Conversation

@rogerdigital

Copy link
Copy Markdown
Owner

Summary

Fixes #83.

The default ignoredLargeMarkdownFrontmatterKeys was ["excalidraw"], but the Excalidraw plugin writes the excalidraw-plugin frontmatter key, so the default never matched and Excalidraw drawings kept showing up in large-file results.

Changes

  • Default valueDEFAULT_SETTINGS.ignoredLargeMarkdownFrontmatterKeys now ["excalidraw-plugin"]
  • MigrationloadSettings auto-corrects persisted settings that still contain the legacy excalidraw key (replace, dedup, preserve other custom keys) and rewrites data.json once. Fresh installs and users who never touched this setting are left untouched.
  • Settings UI — placeholder now shows the real key (E.g. excalidraw-plugin)
  • Docs — README default value + CLI config note (defaults already cover Excalidraw; update legacy config if present)
  • Tests — scanner, settings, CLI, and migration tests updated to the correct key

Verification

lint ✅  obsidian-warnings ✅  build ✅  test 152/152 ✅

Excalidraw writes the excalidraw-plugin frontmatter key, not excalidraw.
Update the default ignoredLargeMarkdownFrontmatterKeys value and migrate
persisted settings on load so existing users who set the legacy key are
auto-corrected. Updates docs, settings placeholder, and tests.
@rogerdigital
rogerdigital merged commit 588c821 into main Jun 23, 2026
1 check passed
@rogerdigital
rogerdigital deleted the fix/excalidraw-frontmatter-key branch June 23, 2026 04:00
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.

Excalidraw ignoring implemented wrong

1 participant