refactor(webvh): remove WebvhDidRegistrar and clean up exports

- Removed WebvhDidRegistrar class and its related exports from the dids module.
- Updated index.ts and WebvhModule.ts to reflect the removal of the registrar.
- Adjusted DIDWebvhCrypto class to ensure agentContext is always defined.
- Cleaned up unused functions and imports in didWebvhUtil.ts.

Author: brianorwhatever

packages/webvh/src/WebvhModule.ts

@@ -1,7 +1,6 @@
 import type { DependencyManager, Module } from '@credo-ts/core'
 
 import { WebvhModuleConfig } from './WebvhModuleConfig'
-import { WebvhDidRegistrar } from './dids/WebvhDidRegistrar'
 import { WebvhDidResolver } from './dids/WebvhDidResolver'
 
 export class WebvhModule implements Module {
@@ -15,11 +14,7 @@ export class WebvhModule implements Module {
    * Registers the dependencies of the WebVH module on the dependency manager.
    */
   public register(dependencyManager: DependencyManager) {
-    // Register config
     dependencyManager.registerInstance(WebvhModuleConfig, this.config)
-
-    // Register did registrar and resolver
-    dependencyManager.registerSingleton(WebvhDidRegistrar)
     dependencyManager.registerSingleton(WebvhDidResolver)
   }
 }

packages/webvh/src/dids/WebvhDidRegistrar.ts

@@ -1,10 +1,10 @@
-import { type AgentContext, Buffer, DidDocument, JsonTransformer, KeyType, Key } from '@credo-ts/core'
-import { createDID, AbstractCrypto, type SigningOutput, type SigningInput } from 'didwebvh-ts'
+import { type AgentContext, Buffer, KeyType, Key } from '@credo-ts/core'
+import { AbstractCrypto, type SigningOutput } from 'didwebvh-ts'
 
 export class DIDWebvhCrypto extends AbstractCrypto {
-  private agentContext?: AgentContext
+  private agentContext: AgentContext
 
-  public constructor(agentContext?: AgentContext) {
+  public constructor(agentContext: AgentContext) {
     super({
       verificationMethod: {
         id: 'did:webvh:123',
@@ -17,8 +17,7 @@ export class DIDWebvhCrypto extends AbstractCrypto {
     this.agentContext = agentContext
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  public async sign(input: SigningInput): Promise<SigningOutput> {
+  public async sign(): Promise<SigningOutput> {
     throw new Error('Not implemented')
   }
 
@@ -38,52 +37,13 @@ export class DIDWebvhCrypto extends AbstractCrypto {
     } catch (error) {
       // Log error in a non-production environment
       if (process.env.NODE_ENV !== 'production') {
-        // eslint-disable-next-line no-console
-        console.error('Error verifying signature:', error)
+        this.agentContext.config.logger.error('Error verifying signature:', error)
       }
       return false
     }
   }
 }
 
-export function validateSpecCompliantPayload(didDocument: DidDocument): SpecValidationResult {
-  // id is required, validated on both compile and runtime
-  if (!didDocument.id || !didDocument.id.startsWith('did:webvh:')) return { valid: false, error: 'id is required' }
-
-  // verificationMethod is required
-  if (!didDocument.verificationMethod) return { valid: false, error: 'verificationMethod is required' }
-
-  // verificationMethod must be an array
-  if (!Array.isArray(didDocument.verificationMethod))
-    return { valid: false, error: 'verificationMethod must be an array' }
-
-  // verificationMethod must be not be empty
-  if (!didDocument.verificationMethod.length) return { valid: false, error: 'verificationMethod must be not be empty' }
-
-  return { valid: true }
-}
-
-export interface SpecValidationResult {
-  valid: boolean
-  error?: string
-}
-
-export async function generateDidDoc(options: IDidDocOptions, agentContext?: AgentContext) {
-  const { verificationMethods, baseUrl, updateKeys } = options
-
-  const { doc } = await createDID({
-    domain: baseUrl.replace(/^https?:\/\//, '').replace('/', ':'),
-    updateKeys,
-    signer: new DIDWebvhCrypto(agentContext),
-    verificationMethods: verificationMethods.map((verificationMethod) => ({
-      type: 'Multikey',
-      publicKeyMultibase: verificationMethod.publicKeyMultibase,
-    })),
-  })
-
-  return JsonTransformer.fromJSON(doc, DidDocument)
-}
-
 export interface IDidDocOptions {
   verificationMethods: {
     publicKeyMultibase: string

packages/webvh/src/dids/didWebvhUtil.ts

@@ -1,7 +1 @@
-export {
-  WebvhDidRegistrar,
-  WebvhDidCreateOptions,
-  WebvhDidDeactivateOptions,
-  WebvhDidUpdateOptions,
-} from './WebvhDidRegistrar'
 export { WebvhDidResolver } from './WebvhDidResolver'

packages/webvh/src/dids/index.ts

@@ -1,13 +1,5 @@
-// Dids
-export {
-  WebvhDidRegistrar,
-  WebvhDidCreateOptions,
-  WebvhDidDeactivateOptions,
-  WebvhDidUpdateOptions,
-  WebvhDidResolver,
-} from './dids'
+export { WebvhDidResolver } from './dids'
 
-// Module
 export { WebvhModule } from './WebvhModule'
 export { WebvhModuleConfig, WebvhModuleConfigOptions } from './WebvhModuleConfig'
 export { WebvhApi } from './WebvhApi'

packages/webvh/src/index.ts

@@ -1,6 +1,6 @@
 import { DidsModule } from '@credo-ts/core'
 
-import { WebvhModule, WebvhDidRegistrar, WebvhDidResolver, WebvhModuleConfig } from '../src'
+import { WebvhModule, WebvhDidResolver, WebvhModuleConfig } from '../src'
 
 export const getWebvhModules = (baseUrl?: string) => ({
   webvhSdk: new WebvhModule(
@@ -9,7 +9,6 @@ export const getWebvhModules = (baseUrl?: string) => ({
     })
   ),
   dids: new DidsModule({
-    registrars: [new WebvhDidRegistrar()],
     resolvers: [new WebvhDidResolver()],
   }),
 })