fix: Removing EcdsaSignature type

packages/daf-core/src/abstract/abstract-key-management-system.ts

@@ -1,10 +1,10 @@
-import { IKey, EcdsaSignature, TKeyType } from '../types'
+import { IKey, TKeyType } from '../types'
 
 export abstract class AbstractKeyManagementSystem {
   abstract createKey(args: { type: TKeyType; meta?: any }): Promise<Omit<IKey, 'kms'>>
   abstract deleteKey(args: { kid: string }): Promise<boolean>
   abstract encryptJWE(args: { key: IKey; to: Omit<IKey, 'kms'>; data: string }): Promise<string>
   abstract decryptJWE(args: { key: IKey; data: string }): Promise<string>
-  abstract signJWT(args: { key: IKey; data: string }): Promise<EcdsaSignature | string>
+  abstract signJWT(args: { key: IKey; data: string }): Promise<string>
   abstract signEthTX(args: { key: IKey; transaction: object }): Promise<string>
 }

packages/daf-core/src/key-manager.ts

@@ -1,6 +1,6 @@
 import { AbstractKeyStore } from './abstract/abstract-key-store'
 import { AbstractKeyManagementSystem } from './abstract/abstract-key-management-system'
-import { IKey, TKeyType, EcdsaSignature, IAgentPlugin, IPluginMethodMap } from './types'
+import { IKey, TKeyType, IAgentPlugin, IPluginMethodMap } from './types'
 
 export interface IKeyManagerCreateKeyArgs {
   type: TKeyType
@@ -38,7 +38,7 @@ export interface IKeyManager extends IPluginMethodMap {
   keyManagerImportKey(args: IKey): Promise<boolean>
   keyManagerEncryptJWE(args: IKeyManagerEncryptJWEArgs): Promise<string>
   keyManagerDecryptJWE(args: IKeyManagerDecryptJWEArgs): Promise<string>
-  keyManagerSignJWT(args: IKeyManagerSignJWTArgs): Promise<EcdsaSignature | string>
+  keyManagerSignJWT(args: IKeyManagerSignJWTArgs): Promise<string>
   keyManagerSignEthTX(args: IKeyManagerSignEthTXArgs): Promise<string>
 }
 export class KeyManager implements IAgentPlugin {
@@ -105,7 +105,7 @@ export class KeyManager implements IAgentPlugin {
     return kms.decryptJWE({ key, data })
   }
 
-  async keyManagerSignJWT({ kid, data }: IKeyManagerSignJWTArgs): Promise<EcdsaSignature | string> {
+  async keyManagerSignJWT({ kid, data }: IKeyManagerSignJWTArgs): Promise<string> {
     const key = await this.store.get({ kid })
     const kms = this.getKms(key.kms)
     return kms.signJWT({ key, data })

packages/daf-core/src/types.ts

@@ -32,13 +32,6 @@ export interface IIdentity {
   services: IService[]
 }
 
-// Todo make it a String
-export interface EcdsaSignature {
-  r: string
-  s: string
-  recoveryParam?: number
-}
-
 export interface IMetaData {
   type: string
   value?: string

packages/daf-libsodium/src/key-management-system.ts

@@ -1,4 +1,4 @@
-import { AbstractKeyManagementSystem, TKeyType, IKey, EcdsaSignature } from 'daf-core'
+import { AbstractKeyManagementSystem, TKeyType, IKey } from 'daf-core'
 import sodium from 'libsodium-wrappers'
 import { SimpleSigner } from 'did-jwt'
 const EC = require('elliptic').ec
@@ -71,9 +71,9 @@ export class KeyManagementSystem extends AbstractKeyManagementSystem {
     return sign(transaction, '0x' + key.privateKeyHex)
   }
 
-  async signJWT({ key, data }: { key: IKey; data: string }): Promise<EcdsaSignature | string> {
+  async signJWT({ key, data }: { key: IKey; data: string }): Promise<string> {
     if (!key.privateKeyHex) throw Error('No private key for kid: ' + key.kid)
     const signer = SimpleSigner(key.privateKeyHex)
-    return signer(data)
+    return signer(data) as any as string
   }
 }

scripts/generate-schemas.ts

@@ -49,10 +49,8 @@ function createSchema(generator: TJS.JsonSchemaGenerator, symbol: string) {
     return { components: { schemas: {} } }
   }
 
-  //hack
-  let fixedSymbol = symbol === 'EcdsaSignature | string' ? 'EcdsaSignature' : symbol
-  // TODO fix 'EcdsaSignature | string' in openApi responses
-  fixedSymbol = fixedSymbol.replace('Array<', '').replace('>', '')
+
+  let fixedSymbol = symbol.replace('Array<', '').replace('>', '')
 
   const schema = generator.getSchemaForSymbol(fixedSymbol)
 
@@ -104,7 +102,12 @@ function getResponseSchema(response: string): OpenAPIV3.ReferenceObject | OpenAP
 
 for (const packageName of Object.keys(agentPlugins)) {
   const program = TJS.getProgramFromFiles([resolve('packages/' + packageName + '/src/index.ts')])
-  const generator = TJS.buildGenerator(program, { required: true, topRef: true, excludePrivate: false })
+  const generator = TJS.buildGenerator(program, { 
+    required: true,
+    topRef: true,
+    excludePrivate: false,
+  })
+
 
   const apiModel: ApiModel = new ApiModel()
   const apiPackage = apiModel.loadPackage(