Skip to content

feat: Add address publish actions across clients#392

Merged
macterra merged 6 commits intomainfrom
feat/publish-address-actions
Apr 10, 2026
Merged

feat: Add address publish actions across clients#392
macterra merged 6 commits intomainfrom
feat/publish-address-actions

Conversation

@macterra
Copy link
Copy Markdown
Collaborator

@macterra macterra commented Apr 9, 2026

Summary

  • add per-address Publish/Unpublish actions to the identity Addresses tab in the gatekeeper client, keymaster client, react wallet, and browser extension
  • store the published address in the agent's address property and ensure only one address is published at a time
  • keep published-address UI state in sync on startup, refresh, and address removal, and document that the two KeymasterUI.jsx copies must stay identical

Testing

  • npm run build --prefix apps/gatekeeper-client
  • npm run build --prefix apps/keymaster-client
  • npm run build --prefix apps/react-wallet
  • npm run build --prefix apps/browser-extension

@macterra macterra changed the title Add address publish actions across clients feat: Add address publish actions across clients Apr 9, 2026
@macterra macterra marked this pull request as ready for review April 9, 2026 21:21
@macterra macterra requested a review from Copilot April 9, 2026 21:21
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds per-address Publish/Unpublish controls across the various clients’ identity address UIs, persisting the “published” address into the identity’s didDocumentData.address field (via mergeData) and keeping the UI state synchronized across refresh/startup/removal flows.

Changes:

  • Add Publish/Unpublish actions to address lists (React wallet, browser extension, Gatekeeper client, Keymaster client) and visually highlight the published address.
  • Sync published-address state by reading didDocumentData.address on refresh/resolve, and clear it when the published address is removed.
  • Document that apps/gatekeeper-client/src/KeymasterUI.jsx and apps/keymaster-client/src/KeymasterUI.jsx must remain identical.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
apps/react-wallet/src/components/IdentitiesTab.tsx Adds published-address state, publish/unpublish actions, and UI updates in the Addresses table.
apps/browser-extension/src/components/IdentitiesTab.tsx Mirrors the React wallet changes for publish/unpublish actions and published state syncing.
apps/keymaster-client/src/KeymasterUI.jsx Adds publish/unpublish controls in the identity addresses table and syncs published state during DID resolution.
apps/gatekeeper-client/src/KeymasterUI.jsx Keeps parity with keymaster-client KeymasterUI changes (publish/unpublish + syncing).
AGENTS.md Adds repo guidance to keep the two KeymasterUI.jsx copies identical.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread apps/keymaster-client/src/KeymasterUI.jsx
Comment thread apps/gatekeeper-client/src/KeymasterUI.jsx
Comment thread apps/react-wallet/src/components/IdentitiesTab.tsx
Comment thread apps/browser-extension/src/components/IdentitiesTab.tsx
@macterra macterra merged commit c88e5ac into main Apr 10, 2026
24 checks passed
@macterra macterra deleted the feat/publish-address-actions branch April 10, 2026 00:55
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