safe-global / safe-client-gateway / 25036580642

90%

Build Type

push

github

Committed by web-flow

Commit Message

feat: private address book contacts and request-to-add workflow (#3026)

* feat: add private address book contacts and request-to-add workflow

Allow non-admin Space members to create private contacts visible only
to them, and request to add those contacts to the shared space address
book. Admins can approve or reject pending requests.

New tables:
- user_address_book_items: per-user private contacts scoped to a space
- address_book_requests: pending approval requests with status enum

New endpoints:
- GET/PUT/DELETE /v1/spaces/:spaceId/address-book/private
- GET/POST /v1/spaces/:spaceId/address-book/requests
- PUT /v1/spaces/:spaceId/address-book/requests/:id/approve
- PUT /v1/spaces/:spaceId/address-book/requests/:id/reject

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: preserve original author when approving address book requests

Add createdByOverride to upsertMany so approved contacts keep the
requester's wallet address as createdBy instead of the admin's.
The approve flow looks up the private contact to get the original
author's wallet address.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: load requestedBy.wallets relation for Requested by column

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* fix: resolve requestedBy wallet via IWalletsRepository instead of nested relation

The requestedBy.wallets nested relation caused a TypeORM joinColumns error
because the Wallet entity wasn't registered in SpacesModule. Instead,
resolve the wallet address via IWalletsRepository.findByUser() in the
service layer.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore: fix CI lint, prettier and license-headers

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* fix: persist requester wallet on address-book requests

Stores the requester's wallet on `address_book_requests` at creation, so
attribution is stable across private-cont... (continued)

Coverage Stats

3312 of 4157 branches covered (79.67%)

Branch coverage included in aggregate %.

169 of 304 new or added lines in 17 files covered. (55.59%)

1 existing line in 1 file now uncovered.

15835 of 17275 relevant lines covered (91.66%)

589.79 hits per line