Skip to content

feat: sync summary and streamlined conflicts#13

Merged
michvllni merged 2 commits into
masterfrom
feat/summary
May 29, 2026
Merged

feat: sync summary and streamlined conflicts#13
michvllni merged 2 commits into
masterfrom
feat/summary

Conversation

@michvllni

Copy link
Copy Markdown
Collaborator
  • add SyncSummary struct for tracking sync outcomes and report generation
  • streamline conflict detection by treating first sync with both files existing as a simple conflict

- add SyncSummary struct for tracking sync outcomes and report generation
- streamline conflict detection by treating first sync with both files existing as a simple conflict

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds end-of-run reporting for Google Drive bidirectional syncs by introducing a SyncSummary accumulator, and simplifies “first sync” handling by treating “both sides exist with no manifest baseline” as a regular conflict.

Changes:

  • Added a SyncSummary struct to count uploads/downloads/unchanged files and list per-file actions.
  • Streamlined conflict detection by treating “no manifest + both sides exist” as a standard conflict case.
  • Updated performSync to record outcomes into the shared summary and print the summary after the sync pass.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread source/main.cpp
@michvllni michvllni merged commit 1d38b65 into master May 29, 2026
4 checks passed
@michvllni michvllni deleted the feat/summary branch May 29, 2026 14:13
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.

2 participants