-
Notifications
You must be signed in to change notification settings - Fork 656
feat(contacts): add an apply/merge step to dedupe (currently preview-only) #815
Copy link
Copy link
Closed
Labels
P3Low-risk cleanup, docs, polish, ergonomics, or speculative feature.Low-risk cleanup, docs, polish, ergonomics, or speculative feature.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.impact:data-lossThis issue is about lost, corrupted, or silently dropped user/session/config data.This issue is about lost, corrupted, or silently dropped user/session/config data.issue-rating: 🌊 off-meta tidepoolIssue quality rating does not apply to this item.Issue quality rating does not apply to this item.
Description
Metadata
Metadata
Assignees
Labels
P3Low-risk cleanup, docs, polish, ergonomics, or speculative feature.Low-risk cleanup, docs, polish, ergonomics, or speculative feature.clawsweeper:needs-maintainer-reviewClawSweeper marked this issue as needing maintainer review before automation.ClawSweeper marked this issue as needing maintainer review before automation.clawsweeper:needs-product-decisionClawSweeper marked this issue as needing a product or behavior decision.ClawSweeper marked this issue as needing a product or behavior decision.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.impact:data-lossThis issue is about lost, corrupted, or silently dropped user/session/config data.This issue is about lost, corrupted, or silently dropped user/session/config data.issue-rating: 🌊 off-meta tidepoolIssue quality rating does not apply to this item.Issue quality rating does not apply to this item.
Type
Fields
Priority
None yet
Summary
gog contacts dedupefinds likely duplicate contacts but is preview-only ("Find likelyduplicate contacts (preview only)"), so there is no way to act on the results from gog. Users must
open Google Contacts and merge by hand. An opt-in, guarded apply/merge step would close the loop.
Current (main, v0.27.0)
docs/commands/gog-contacts-dedupe.md: "Find likely duplicate contacts (preview only)". There is no--apply/--mergeflag; only the global flags (--dry-run, etc.).Suggested
Add an explicit merge step, e.g.
gog contacts dedupe --apply(or a separatecontacts merge),gated behind a confirmation (
--force/-yto skip) and honoring--dry-run, consistent with otherdestructive gog operations. Merging is destructive, so preview-by-default is the right default; this
just adds the ability to apply the merge without leaving the CLI.
Environment
gog v0.27.0 (main).