feat: drive reporting/sync + contacts dedupe#116
feat: drive reporting/sync + contacts dedupe#116rohan-patnaik wants to merge 4 commits intoopenclaw:mainfrom
Conversation
|
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! |
7533518 to
ed0e926
Compare
|
Rebased onto the latest main and resolved conflicts. Branch is updated and ready for another pass. Thanks! |
|
@rohan-patnaik Looking good. Ci/test would like some lint cleanups: https://github.com/steipete/gogcli/actions/runs/22588136930/job/65478928149?pr=116 |
|
Addressed the CI lint cleanup items and pushed an update. Validation run locally:
|
9f54a38 to
b9127bc
Compare
|
Thanks @rohan-patnaik. I extracted and landed the read-only Drive reporting part of this PR in #554 / e9c496e:
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 |
|
Thanks again @rohan-patnaik. I split this up so we could land the safer pieces cleanly on current
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. |
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>
|
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. |
Summary
tree,du,inventory) with path-based output and size/modified metadata.pull/push) with dry-run, include/exclude filters, checksum support, delete safeguards, and a local state file.Testing
Notes
.gog-sync.json(no secrets).