Commit Graph

6 Commits

Author SHA1 Message Date
lucaronin
68bc72a830 fix: cargo fmt on migration.rs tests 2026-02-24 00:01:16 +01:00
lucaronin
f2268a4aa5 test: add unit tests for vault/migration.rs to restore Rust coverage to ≥85%
Tests cover: has_legacy_is_a, extract_is_a_value, migrate_file_is_a_to_type,
and migrate_is_a_to_type (public function). migration.rs was at 0% coverage.
2026-02-23 23:22:42 +01:00
lucaronin
7a66fd4149 fix: add migration module with migrate_is_a_to_type — was missing after vault.rs refactor 2026-02-23 23:05:34 +01:00
lucaronin
2582250286 fix: use inner doc comment to fix clippy::empty_line_after_doc_comments 2026-02-23 22:37:31 +01:00
lucaronin
6500e721c1 fix: restore save_note_content and missing pub fns from main merge 2026-02-23 22:37:31 +01:00
lucaronin
0eb0cfdf47 refactor: split vault.rs into focused submodules
Break the 2338-line vault.rs into vault/ directory with 6 focused modules:
- mod.rs: core types (VaultEntry, Frontmatter), parse_md_file, scan_vault
- parsing.rs: text processing (snippet extraction, markdown stripping, date parsing)
- cache.rs: git-based incremental vault caching
- trash.rs: purge_trash with flat iterator chain
- rename.rs: note renaming with wikilink updates
- image.rs: attachment saving with filename sanitization

Also moves frontmatter update/delete wrappers to frontmatter.rs where they
belong, and converts public functions to accept &Path instead of &str.

CodeScene scores: mod.rs 10.0, parsing.rs 9.68, cache.rs 9.68,
trash.rs 9.38, rename.rs 9.68, image.rs 10.0 (all above 9.0 target).
All 169 Rust tests pass, 86.28% line coverage.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-23 22:37:31 +01:00