Bump @web5/dids to 0.3.0 (#679)

This PR will:
- Bump the `@web5/dids` dependency from 0.2.4 to 0.3.0.
- Removed the `@types/dns-packet` dev dependency since it is no longer
needed.
- Removes `src/did/did.ts` and the associated tests since there are no
longer any references to these methods in DWN SDK. Equivalent
functionality is available in
[`DidUri`](https://github.com/TBD54566975/web5-js/blob/main/packages/dids/src/did-uri.ts)
should it ever be needed.
- Removes `TestDataGenerator.getKeyId()` since a DID method agnostic
method `getSigningMethod()` is now available across all DID method
implementations in `@web5/dids`.
- Bump release version to `0.2.16`

---------

Signed-off-by: Frank Hinek <[email protected]>
Co-authored-by: Liran Cohen <[email protected]>

Author: frankhinek

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@tbd54566975/dwn-sdk-js",
-  "version": "0.2.15",
+  "version": "0.2.16",
   "description": "A reference implementation of https://identity.foundation/decentralized-web-node/spec/",
   "repository": {
     "type": "git",
@@ -65,7 +65,7 @@
     "@js-temporal/polyfill": "0.4.4",
     "@noble/ed25519": "2.0.0",
     "@noble/secp256k1": "2.0.0",
-    "@web5/dids": "^0.2.4",
+    "@web5/dids": "0.3.0",
     "abstract-level": "1.0.3",
     "ajv": "8.12.0",
     "blockstore-core": "4.2.0",
@@ -89,7 +89,6 @@
   "devDependencies": {
     "@types/chai": "4.3.0",
     "@types/chai-as-promised": "7.1.5",
-    "@types/dns-packet": "^5.6.4",
     "@types/flat": "^5.0.2",
     "@types/karma": "^6.3.3",
     "@types/lodash": "4.14.179",
@@ -159,4 +158,4 @@
     "license-check": "license-report --only=prod > license-report.json && node ./build/license-check.cjs",
     "publish:unstable": "./build/publish-unstable.sh"
   }
-}
+}

package.json

@@ -30,7 +30,7 @@ import { RecordsQueryHandler } from './handlers/records-query.js';
 import { RecordsReadHandler } from './handlers/records-read.js';
 import { RecordsSubscribeHandler } from './handlers/records-subscribe.js';
 import { RecordsWriteHandler } from './handlers/records-write.js';
-import { DidDhtMethod, DidIonMethod, DidKeyMethod, DidResolver } from '@web5/dids';
+import { DidDht, DidIon, DidKey, DidResolver } from '@web5/dids';
 import { DwnInterfaceName, DwnMethodName } from './enums/dwn-interface-method.js';
 
 export class Dwn {
@@ -134,7 +134,7 @@ export class Dwn {
    * Creates an instance of the DWN.
    */
   public static async create(config: DwnConfig): Promise<Dwn> {
-    config.didResolver ??= new DidResolver({ didResolvers: [DidKeyMethod, DidIonMethod, DidDhtMethod] });
+    config.didResolver ??= new DidResolver({ didResolvers: [DidKey, DidIon, DidDht] });
     config.tenantGate ??= new AllowAllTenantGate();
 
     const dwn = new Dwn(config);

src/did/did.ts

@@ -1,7 +1,7 @@
 import type { Cache } from '../../../types/cache.js';
 import type { GeneralJws } from '../../../types/jws-types.js';
 import type { PublicJwk } from '../../../types/jose-types.js';
-import type { DidResolver, VerificationMethod } from '@web5/dids';
+import type { DidResolver, DidVerificationMethod } from '@web5/dids';
 
 import { Jws } from '../../../utils/jws.js';
 import { MemoryCache } from '../../../utils/memory-cache.js';
@@ -87,7 +87,7 @@ export class GeneralJwsVerifier {
     const { didDocument } = await didResolver.resolve(did);
     const { verificationMethod: verificationMethods = [] } = didDocument || {};
 
-    let verificationMethod: VerificationMethod | undefined;
+    let verificationMethod: DidVerificationMethod | undefined;
 
     for (const method of verificationMethods) {
       // consider optimizing using a set for O(1) lookups if needed

src/dwn.ts

@@ -1,13 +1,13 @@
 import { authenticate } from '../../src/core/auth.js';
 import { DwnErrorCode } from '../../src/core/dwn-error.js';
 import { expect } from 'chai';
-import { DidDhtMethod, DidResolver } from '@web5/dids';
+import { DidDht, DidResolver } from '@web5/dids';
 
 describe('Auth', () => {
   describe('authenticate()', () => {
     it('should throw if given JWS is `undefined`', async () => {
       const jws = undefined;
-      await expect(authenticate(jws, new DidResolver({ didResolvers: [DidDhtMethod] }))).to.be.rejectedWith(DwnErrorCode.AuthenticateJwsMissing);
+      await expect(authenticate(jws, new DidResolver({ didResolvers: [DidDht] }))).to.be.rejectedWith(DwnErrorCode.AuthenticateJwsMissing);
     });
   });
 });

src/jose/jws/general/verifier.ts

@@ -14,7 +14,7 @@ import { TestDataGenerator } from '../utils/test-data-generator.js';
 import { Message } from '../../src/core/message.js';
 import { TestEventStream } from '../test-event-stream.js';
 import { TestStores } from '../test-stores.js';
-import { DidKeyMethod, DidResolver } from '@web5/dids';
+import { DidKey, DidResolver } from '@web5/dids';
 
 export function testEventsGetHandler(): void {
   describe('EventsGetHandler.handle()', () => {
@@ -28,7 +28,7 @@ export function testEventsGetHandler(): void {
     // important to follow the `before` and `after` pattern to initialize and clean the stores in tests
     // so that different test suites can reuse the same backend store for testing
     before(async () => {
-      didResolver = new DidResolver({ didResolvers: [DidKeyMethod] });
+      didResolver = new DidResolver({ didResolvers: [DidKey] });
 
       const stores = TestStores.get();
       messageStore = stores.messageStore;

tests/core/auth.spec.ts

@@ -11,7 +11,7 @@ import { expect } from 'chai';
 import { TestDataGenerator } from '../utils/test-data-generator.js';
 import { TestEventStream } from '../test-event-stream.js';
 import { TestStores } from '../test-stores.js';
-import { DidKeyMethod, DidResolver } from '@web5/dids';
+import { DidKey, DidResolver } from '@web5/dids';
 
 
 export function testEventsQueryHandler(): void {
@@ -26,7 +26,7 @@ export function testEventsQueryHandler(): void {
     // important to follow the `before` and `after` pattern to initialize and clean the stores in tests
     // so that different test suites can reuse the same backend store for testing
     before(async () => {
-      didResolver = new DidResolver({ didResolvers: [DidKeyMethod] });
+      didResolver = new DidResolver({ didResolvers: [DidKey] });
 
       const stores = TestStores.get();
       messageStore = stores.messageStore;

tests/did/did.spec.ts

@@ -9,7 +9,7 @@ import { Message } from '../../src/core/message.js';
 import { TestDataGenerator } from '../utils/test-data-generator.js';
 import { TestEventStream } from '../test-event-stream.js';
 import { TestStores } from '../test-stores.js';
-import { DidKeyMethod, DidResolver } from '@web5/dids';
+import { DidKey, DidResolver } from '@web5/dids';
 
 import sinon from 'sinon';
 import chai, { expect } from 'chai';
@@ -31,7 +31,7 @@ export function testEventsSubscribeHandler(): void {
       // important to follow the `before` and `after` pattern to initialize and clean the stores in tests
       // so that different test suites can reuse the same backend store for testing
       before(async () => {
-        didResolver = new DidResolver({ didResolvers: [DidKeyMethod] });
+        didResolver = new DidResolver({ didResolvers: [DidKey] });
 
         const stores = TestStores.get();
         messageStore = stores.messageStore;
@@ -85,7 +85,7 @@ export function testEventsSubscribeHandler(): void {
       // important to follow the `before` and `after` pattern to initialize and clean the stores in tests
       // so that different test suites can reuse the same backend store for testing
       before(async () => {
-        didResolver = new DidResolver({ didResolvers: [DidKeyMethod] });
+        didResolver = new DidResolver({ didResolvers: [DidKey] });
 
         const stores = TestStores.get();
         messageStore = stores.messageStore;