Skip to content

feat(contacts): add --relation flag to create and update#351

Merged
steipete merged 4 commits intoopenclaw:mainfrom
karbassi:feat/contacts-relation-flag
Mar 8, 2026
Merged

feat(contacts): add --relation flag to create and update#351
steipete merged 4 commits intoopenclaw:mainfrom
karbassi:feat/contacts-relation-flag

Conversation

@karbassi
Copy link
Copy Markdown
Contributor

Summary

  • Add --relation type=person flag to gog contacts create and gog contacts update for managing People API relations (spouse, child, friend, etc.)
  • Extract shared parseKeyValuePairs helper to reduce duplication between --custom and --relation parsing
  • Display relations in gog contacts get text output as relation:type
  • Fix pre-existing compile error in contactsApplyPersonName and contactsApplyPersonOrganization parameter signatures

Test plan

  • go test ./internal/cmd/... -run TestParseRelations — unit tests for parse function
  • go test ./internal/cmd/... -run TestContactsUpdate_Relation — integration tests for set/clear
  • go test ./internal/cmd/... — full suite passes

karbassi and others added 4 commits March 8, 2026 01:28
Add CLI support for the People API relations field (spouse, child,
friend, etc.) via --relation type=person. Extract shared parseKeyValuePairs
helper to reduce duplication with --custom parsing.

Also fix pre-existing compile error in contactsApplyPersonName and
contactsApplyPersonOrganization parameter signatures.
@steipete steipete force-pushed the feat/contacts-relation-flag branch from 50afb94 to 77d9943 Compare March 8, 2026 01:31
@steipete steipete merged commit efeef87 into openclaw:main Mar 8, 2026
1 check passed
@steipete
Copy link
Copy Markdown
Collaborator

steipete commented Mar 8, 2026

Landed via temp rebase onto main.

Thanks @karbassi!

klodr pushed a commit to klodr/gogcli that referenced this pull request Apr 22, 2026
klodr pushed a commit to klodr/gogcli that referenced this pull request Apr 22, 2026
klodr pushed a commit to klodr/gogcli that referenced this pull request Apr 22, 2026
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