Skip to content

feat: drive reporting/sync + contacts dedupe#116

Closed
rohan-patnaik wants to merge 4 commits intoopenclaw:mainfrom
rohan-patnaik:feature/drive-sync
Closed

feat: drive reporting/sync + contacts dedupe#116
rohan-patnaik wants to merge 4 commits intoopenclaw:mainfrom
rohan-patnaik:feature/drive-sync

Conversation

@rohan-patnaik
Copy link
Copy Markdown
Contributor

@rohan-patnaik rohan-patnaik commented Jan 26, 2026

Summary

  • Add Drive reporting commands (tree, du, inventory) with path-based output and size/modified metadata.
  • Add Drive sync (pull/push) with dry-run, include/exclude filters, checksum support, delete safeguards, and a local state file.
  • Add contacts dedupe with preview/apply workflow and merge rules.
  • Add tests and README examples for the new commands.

Testing

  • go test ./...

Notes

  • Sync state is stored locally in .gog-sync.json (no secrets).
  • Google Docs are exported on pull and skipped on push.
  • I’m new to OSS contributions — happy to adjust or split this if you prefer.

@rohan-patnaik rohan-patnaik changed the title feat: add drive sync + contacts dedupe feat: drive reporting/sync + contacts dedupe Jan 26, 2026
@visionik
Copy link
Copy Markdown
Collaborator

visionik commented Mar 2, 2026

hi @rohan-patnaik - I like what you've done here, thank you. I'm helping peter get gogcli PR's and issues addressed quickly. Since it's been a while, the merge for this --> main is complex. could you rebase from main and resubmit the PR? Thank you!

@rohan-patnaik
Copy link
Copy Markdown
Contributor Author

Rebased onto the latest main and resolved conflicts. Branch is updated and ready for another pass. Thanks!

@visionik
Copy link
Copy Markdown
Collaborator

visionik commented Mar 2, 2026

@rohan-patnaik Looking good. Ci/test would like some lint cleanups:

https://github.com/steipete/gogcli/actions/runs/22588136930/job/65478928149?pr=116

@rohan-patnaik
Copy link
Copy Markdown
Contributor Author

rohan-patnaik commented Mar 3, 2026

Addressed the CI lint cleanup items and pushed an update.

Validation run locally:

  • make lint (0 issues)
  • go test ./... (all packages passing)

@steipete
Copy link
Copy Markdown
Collaborator

steipete commented May 5, 2026

Thanks @rohan-patnaik. I extracted and landed the read-only Drive reporting part of this PR in #554 / e9c496e:

  • gog drive tree
  • gog drive du
  • gog drive inventory

I kept Drive sync and contacts dedupe out of that landing because they have a larger behavioral/risk surface and deserve separate review/live proof. Your authorship is preserved via Co-authored-by and the changelog credits #116. Leaving this PR open for now as the source thread for the remaining sync/dedupe pieces.

@steipete
Copy link
Copy Markdown
Collaborator

steipete commented May 5, 2026

Thanks again @rohan-patnaik. I split this up so we could land the safer pieces cleanly on current main:

I intentionally did not carry over Drive sync from this PR: bidirectional sync/delete/state behavior needs a separate design and live proof path. Closing this broad/stale PR as superseded by the split PRs rather than leaving the old bundle open.

@steipete
Copy link
Copy Markdown
Collaborator

steipete commented May 5, 2026

Superseded by the split landing path: Drive reporting in #554 and contacts dedupe preview in #555. Drive sync remains separate future design work.

@steipete steipete closed this May 5, 2026
steipete added a commit that referenced this pull request May 5, 2026
Adds a read-only contacts dedupe preview command with JSON/table output, generated command docs, README/spec docs, and changelog credit for the extracted idea from #116.\n\nCo-authored-by: Rohan Patnaik <rohan-patnaik@users.noreply.github.com>
@rohan-patnaik
Copy link
Copy Markdown
Contributor Author

Thanks @steipete Peter, I appreciate you splitting this up and landing the parts that were ready and thanks for the co-author credit on #554 and #555.

The Drive sync feedback makes sense. I’ll take another pass at that after I have a cleaner live-proof path and a better proposal, instead of trying to keep it bundled into this PR.

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.

3 participants