Skip to content

fix(sync): scanner default ignore syncthing metadata (D-1)#1

Merged
juice094 merged 1 commit into
mainfrom
fix/scanner-default-ignore
May 16, 2026
Merged

fix(sync): scanner default ignore syncthing metadata (D-1)#1
juice094 merged 1 commit into
mainfrom
fix/scanner-default-ignore

Conversation

@juice094
Copy link
Copy Markdown
Owner

Fix D-1: Scanner does not auto-exclude syncthing metadata files.

  • Add DEFAULT_IGNORED_NAMES and DEFAULT_IGNORED_SUFFIXES constants
  • Apply default ignore before .stignore in scan_directory
  • Fix pre-existing clippy only_used_in_recursion warning
  • Include cargo fmt cleanup

Verified: cargo check, clippy (-D warnings), test (309 passed), fmt all pass.

@juice094 juice094 force-pushed the fix/scanner-default-ignore branch from 5ffa617 to 1c6b650 Compare May 16, 2026 12:58
修复 D-1:Scanner 不自动排除元数据文件。

- 新增 DEFAULT_IGNORED_NAMES:.stfolder, .stversions, .stignore,
  config.json, cert.pem, key.pem, db, logs
- 新增 DEFAULT_IGNORED_SUFFIXES:.syncthing.tmp, ~syncthing~
- 在 scan_directory 中先于 .stignore 检查应用默认排除
- 修复 pre-existing clippy error:将未在方法体内直接使用的 base_path
  重命名为 _base_path(clippy::only_used_in_recursion)

验证:
- cargo check --workspace ✅
- cargo clippy --workspace --all-targets -- -D warnings ✅
- cargo test --workspace ✅ (309 passed, 0 failed)

追踪:DUAL_NODE_TEST_2026-05-15.md D-1 / KNOWN_ISSUES.md §11
@juice094 juice094 force-pushed the fix/scanner-default-ignore branch from 1c6b650 to 5ceb661 Compare May 16, 2026 13:04
@juice094 juice094 merged commit d6d8c01 into main May 16, 2026
12 checks passed
@juice094 juice094 deleted the fix/scanner-default-ignore branch May 16, 2026 13:11
juice094 pushed a commit that referenced this pull request May 16, 2026
- Register §11 in KNOWN_ISSUES.md
- Update roadmap impact table for v0.3.0
- Cross-reference PR #1 / commit d6d8c01
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.

1 participant