fix: store recipient keys by default

packages/core/src/modules/connections/__tests__/InMemoryDidRegistry.ts

@@ -40,11 +40,6 @@ export class InMemoryDidRegistry implements DidRegistrar, DidResolver {
       did: didDocument.id,
       role: DidDocumentRole.Created,
       didDocument,
-      tags: {
-        // We need to save the recipientKeys, so we can find the associated did
-        // of a key when we receive a message from another connection.
-        recipientKeyFingerprints: didDocument.recipientKeys.map((key) => key.fingerprint),
-      },
     })
     const didRepository = agentContext.dependencyManager.resolve(DidRepository)
     await didRepository.save(agentContext, didRecord)

packages/core/src/modules/connections/services/ConnectionService.ts

@@ -797,11 +797,6 @@ export class ConnectionService {
       did: peerDid,
       role,
       didDocument,
-      tags: {
-        // We need to save the recipientKeys, so we can find the associated did
-        // of a key when we receive a message from another connection.
-        recipientKeyFingerprints: didDocument.recipientKeys.map((key) => key.fingerprint),
-      },
     })
 
     // Store the unqualified did with the legacy did document in the metadata

packages/core/src/modules/connections/services/DidRotateService.ts

@@ -194,8 +194,6 @@ export class DidRotateService {
       did: didDocument.id,
       didDocument,
       tags: {
-        recipientKeyFingerprints: didDocument.recipientKeys.map((key) => key.fingerprint),
-
         // For did:peer, store any alternative dids (like short form did:peer:4),
         // it may have in order to relate any message referencing it
         alternativeDids: isValidPeerDid(didDocument.id) ? getAlternativeDidsForPeerDid(didDocument.id) : undefined,

packages/core/src/modules/dids/DidsApi.ts

@@ -157,7 +157,6 @@ export class DidsApi {
     if (existingDidRecord) {
       existingDidRecord.didDocument = didDocument
       existingDidRecord.setTags({
-        recipientKeyFingerprints: didDocument.recipientKeys.map((key) => key.fingerprint),
         alternativeDids: isValidPeerDid(didDocument.id) ? getAlternativeDidsForPeerDid(did) : undefined,
       })
 
@@ -170,7 +169,6 @@ export class DidsApi {
       did,
       didDocument,
       tags: {
-        recipientKeyFingerprints: didDocument.recipientKeys.map((key) => key.fingerprint),
         alternativeDids: isValidPeerDid(didDocument.id) ? getAlternativeDidsForPeerDid(did) : undefined,
       },
     })

packages/core/src/modules/dids/__tests__/peer-did.test.ts

@@ -141,11 +141,6 @@ describe('peer dids', () => {
       // It is important to take the did document from the PeerDid class
       // as it will have the id property
       didDocument: didDocument,
-      tags: {
-        // We need to save the recipientKeys, so we can find the associated did
-        // of a key when we receive a message from another connection.
-        recipientKeyFingerprints: didDocument.recipientKeys.map((key) => key.fingerprint),
-      },
     })
 
     await didRepository.save(agentContext, didDocumentRecord)

packages/core/src/modules/dids/repository/DidRecord.ts

@@ -30,6 +30,11 @@ export interface CustomDidTags extends TagsBase {
 }
 
 type DefaultDidTags = {
+  // We set the recipientKeyFingeprints as a default tag, if the did record has a did document
+  // If the did record does not have a did document, we can't calculate it, and it needs to be
+  // handled by the creator of the did record
+  recipientKeyFingerprints?: string[]
+
   role: DidDocumentRole
   method: string
   legacyUnqualifiedDid?: string
@@ -75,6 +80,11 @@ export class DidRecord extends BaseRecord<DefaultDidTags, CustomDidTags, DidReco
       legacyUnqualifiedDid: legacyDid?.unqualifiedDid,
       did: this.did,
       methodSpecificIdentifier: did.id,
+
+      // Calucate if we have a did document, otherwise use the already present recipient keys
+      recipientKeyFingerprints: this.didDocument
+        ? this.didDocument.recipientKeys.map((recipientKey) => recipientKey.fingerprint)
+        : this._tags.recipientKeyFingerprints,
     }
   }
 }

packages/core/src/storage/migration/updates/0.1-0.2/connection.ts

@@ -179,9 +179,6 @@ export async function extractDidDocument<Agent extends BaseAgent>(agent: Agent,
         role: DidDocumentRole.Created,
         didDocument: newOurDidDocument,
         createdAt: connectionRecord.createdAt,
-        tags: {
-          recipientKeyFingerprints: newOurDidDocument.recipientKeys.map((key) => key.fingerprint),
-        },
       })
 
       ourDidRecord.metadata.set(DidRecordMetadataKeys.LegacyDid, {
@@ -231,9 +228,6 @@ export async function extractDidDocument<Agent extends BaseAgent>(agent: Agent,
         role: DidDocumentRole.Received,
         didDocument: newTheirDidDocument,
         createdAt: connectionRecord.createdAt,
-        tags: {
-          recipientKeyFingerprints: newTheirDidDocument.recipientKeys.map((key) => key.fingerprint),
-        },
       })
 
       theirDidRecord.metadata.set(DidRecordMetadataKeys.LegacyDid, {

packages/indy-vdr/src/dids/IndyVdrIndyDidRegistrar.ts

@@ -196,9 +196,7 @@ export class IndyVdrIndyDidRegistrar implements DidRegistrar {
     const didRecord = new DidRecord({
       did,
       role: DidDocumentRole.Created,
-      tags: {
-        recipientKeyFingerprints: didDocument.recipientKeys.map((key: Key) => key.fingerprint),
-      },
+      didDocument,
     })
 
     const didRepository = agentContext.dependencyManager.resolve(DidRepository)