Skip to content

Conversation

nikgraf
Copy link
Collaborator

@nikgraf nikgraf commented Jun 26, 2025

No description provided.

Copy link
Contributor

@Copilot 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 refactors identity verification by requiring explicit network configuration parameters (chain and rpcUrl) in multiple functions.

  • Added chain and rpcUrl parameters to validateSpaceInboxMessage, validateAccountInboxMessage, getVerifiedIdentity, and verifyIdentityOwnership.
  • Adjusted import orders and updated the HypergraphAppContext to provide CHAIN and RPC_URL constants for consistency.

Reviewed Changes

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

File Description
packages/hypergraph/src/inboxes/message-validation.ts Added chain and rpcUrl parameters to message validation functions and updated getVerifiedIdentity calls.
packages/hypergraph/src/identity/prove-ownership.ts Removed default parameters for chain and rpcUrl in verifyIdentityOwnership and reordered imports.
packages/hypergraph/src/identity/get-verified-identity.ts Extended getVerifiedIdentity with chain and rpcUrl parameters, passing them to verifyIdentityOwnership.
packages/hypergraph-react/src/HypergraphAppContext.tsx Introduced CHAIN and RPC_URL constants and updated identity verification calls accordingly.

@nikgraf nikgraf merged commit a31d2ba into main Jun 26, 2025
6 checks passed
@nikgraf nikgraf deleted the ng/fix-verifyIdentityOwnership branch June 26, 2025 13:33
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.

1 participant