Skip to content

feat(DidDocument): keyType and Purpose-aware Verification Method helpers - #2934

Open
rmlearney-digicatapult wants to merge 5 commits into
openwallet-foundation:mainfrom
rmlearney-digicatapult:fix/relationship-aware-verification-method-lookup
Open

feat(DidDocument): keyType and Purpose-aware Verification Method helpers#2934
rmlearney-digicatapult wants to merge 5 commits into
openwallet-foundation:mainfrom
rmlearney-digicatapult:fix/relationship-aware-verification-method-lookup

Conversation

@rmlearney-digicatapult

@rmlearney-digicatapult rmlearney-digicatapult commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Multiple points in the codebase use the same logic to resolve keys within a DID document

Created 2 self-describing helpers inside didDocument.ts for:

  • findVerificationMethodsByPurpose()
  • findVerificationMethodsByTypeAndPurpose()

Wired these helpers into call sites throughout the codebase to simplify reasoning:

  • DQCL
  • DidComm
  • PEX
  • Cheqd
  • Webvh
  • W3cJwt

Helpers necessary for fixing currently-broken linked-data-proofs when using Ed25519-2020 keys in subsequent PR

Signed-off-by: rmlearney-digicatapult <robert.learney@digicatapult.org.uk>
Signed-off-by: rmlearney-digicatapult <robert.learney@digicatapult.org.uk>
Signed-off-by: rmlearney-digicatapult <robert.learney@digicatapult.org.uk>
Signed-off-by: rmlearney-digicatapult <robert.learney@digicatapult.org.uk>
Signed-off-by: rmlearney-digicatapult <robert.learney@digicatapult.org.uk>
@rmlearney-digicatapult
rmlearney-digicatapult requested a review from a team as a code owner September 11, 2026 14:04
@changeset-bot

changeset-bot Bot commented Sep 11, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: dd93927

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 18 packages
Name Type
@credo-ts/core Minor
@credo-ts/anoncreds Minor
@credo-ts/cheqd Minor
@credo-ts/didcomm Minor
@credo-ts/webvh Minor
@credo-ts/action-menu Minor
@credo-ts/askar-to-drizzle-storage-migration Minor
@credo-ts/askar Minor
@credo-ts/drizzle-storage Minor
@credo-ts/drpc Minor
@credo-ts/hedera Minor
@credo-ts/indy-vdr Minor
@credo-ts/node Minor
@credo-ts/openid4vc Minor
@credo-ts/question-answer Minor
@credo-ts/react-native Minor
@credo-ts/redis-cache Minor
@credo-ts/tenants Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

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