Skip Redundant Metadata Writes on Paginated Ingestion

[juliusvaart]

Summary

The non-paginated ingestion path guards its writes with isInSameDatabaseStoreableRemoteState. The paginated one — which every enumeration takes on servers from Nextcloud 31 — wrote every row of every listing unconditionally, one transaction and one evictLogicalDuplicates query each. One working set scan rewrote 1,998 rows to surface 7 actual changes, against 5,858 evaluations that found no difference at all.

Applying the same guard to the paginated path suppressed 99.7% of the writes on the next run: 11 written, 3,655 skipped.

visitedDirectory is compared explicitly because it is local-only and so absent from the remote-state comparison. A caller passing preserveVisitedDirectory: false is recording a visit and must always be written, or folders would fall out of the working set.

Tests

Full package suite green, 373 XCTest (1 skipped) and 78 swift-testing.

Assisted-by: Claude Code:claude-opus-5

Checklist

• Sign-off message is added to all commits.
• The commit messages are following The Conventional Commits specification.
• The commit history is clean with no merge commits.
• Uploaded screenshots from before and after for UI changes.
• Test(s) added to branch, with before/after results included in PR description, for performance improvements where applicable.
• Documentation has been updated or is not required.
• Backports requested where applicable (critical bugfixes).
• Labels added where applicable (bug/enhancement).
• Milestone added for target version.

AI (if applicable)

• The content of this PR was partly or fully generated using AI
  • I have read the guidelines for AI-assisted contributions.
  • I have read Nextcloud's AI Contribution Policy.