feat: support both ESM and CJS and update to vitest

.changeset/commit.js

@@ -1,4 +1,4 @@
-const { execSync } = require('node:child_process')
+import { execSync } from 'node:child_process'
 
 const getSignedOffBy = () => {
   const gitUserName = execSync('git config user.name').toString('utf-8').trim()
@@ -7,18 +7,13 @@ const getSignedOffBy = () => {
   return `Signed-off-by: ${gitUserName} <${gitEmail}>`
 }
 
-const getAddMessage = async (changeset) => {
+export const getAddMessage = async (changeset) => {
   return `docs(changeset): ${changeset.summary}\n\n${getSignedOffBy()}\n`
 }
 
-const getVersionMessage = async (releasePlan) => {
+export const getVersionMessage = async (releasePlan) => {
   const publishableReleases = releasePlan.releases.filter((release) => release.type !== 'none')
   const releasedVersion = publishableReleases[0].newVersion
 
   return `chore(release): version ${releasedVersion}\n\n${getSignedOffBy()}\n`
 }
-
-module.exports = {
-  getAddMessage,
-  getVersionMessage,
-}

.changeset/tricky-readers-relax.md

@@ -0,0 +1,27 @@
+---
+"@credo-ts/askar-to-drizzle-storage-migration": minor
+"@credo-ts/indy-sdk-to-askar-migration": minor
+"@credo-ts/drizzle-storage": minor
+"@credo-ts/question-answer": minor
+"@credo-ts/react-native": minor
+"@credo-ts/action-menu": minor
+"@credo-ts/redis-cache": minor
+"@credo-ts/anoncreds": minor
+"@credo-ts/openid4vc": minor
+"@credo-ts/indy-vdr": minor
+"@credo-ts/didcomm": minor
+"@credo-ts/tenants": minor
+"@credo-ts/hedera": minor
+"@credo-ts/askar": minor
+"@credo-ts/cheqd": minor
+"@credo-ts/webvh": minor
+"@credo-ts/core": minor
+"@credo-ts/drpc": minor
+"@credo-ts/node": minor
+---
+
+Add support for both CJS and ESM module syntax.
+
+- Use `tsdown` to bundle for both CJS and ESM (bridge period) -> tsdown is based on rust, so it should help with performance
+- Update to `vitest` since jest doesn't work well with ESM -> this should also help with performance
+- Simplify type checking -> just a single type check script instead of one for all packages. This should help with performance.

.github/workflows/continuous-integration.yml

@@ -44,7 +44,7 @@ jobs:
       - name: Check Types
         run: pnpm types:check
 
-      - name: Compile
+      - name: Build
         run: pnpm build
 
   unit-tests:
@@ -59,7 +59,7 @@ jobs:
         # Each shard runs a set of the tests
         # Make sure to UPDATE THE TEST command with the total length of
         # the shards if you change this!!
-        shard: [1, 2, 3]
+        shard: [1, 2]
 
     steps:
       - uses: actions/checkout@v5
@@ -88,15 +88,8 @@ jobs:
       - name: Install dependencies
         run: pnpm install --frozen-lockfile
 
-      # Issue with node 22 starting 22.18
-      # https://github.com/nodejs/node/issues/59364
       - name: Run tests
-        run: NODE_OPTIONS="$NODE_OPTIONS --no-experimental-strip-types" pnpm test:unit --coverage --forceExit --shard=${{ matrix.shard }}/3
-        if: matrix.node-version == 22
-
-      - name: Run tests
-        run: pnpm test:unit --coverage --forceExit --shard=${{ matrix.shard }}/3
-        if: matrix.node-version != 22
+        run: pnpm test:unit --coverage --shard=${{ matrix.shard }}/2
 
       # Upload coverage for shard
       - run: mv coverage/coverage-final.json coverage/${{ matrix.shard }}.json
@@ -148,15 +141,8 @@ jobs:
       - name: Install dependencies
         run: pnpm install --frozen-lockfile
 
-      # Issue with node 22 starting 22.18
-      # https://github.com/nodejs/node/issues/59364
-      - name: Run tests
-        run: NODE_OPTIONS="$NODE_OPTIONS --no-experimental-strip-types" pnpm test:e2e --coverage --forceExit
-        if: matrix.node-version == 22
-
       - name: Run tests
-        run: pnpm test:e2e --coverage --forceExit
-        if: matrix.node-version != 22
+        run: pnpm test:e2e --coverage
 
       # Upload coverage for e2e
       - run: mv coverage/coverage-final.json coverage/e2e.json
@@ -212,15 +198,8 @@ jobs:
       - name: Build drizzle package
         run: pnpm --filter @credo-ts/drizzle-storage run build
 
-      # Issue with node 22 starting 22.18
-      # https://github.com/nodejs/node/issues/59364
-      - name: Run tests
-        run: NODE_OPTIONS="$NODE_OPTIONS --no-experimental-strip-types" pnpm test:drizzle --coverage --forceExit
-        if: matrix.node-version == 22
-
       - name: Run tests
-        run: pnpm test:drizzle --coverage --forceExit
-        if: matrix.node-version != 22
+        run: pnpm test:drizzle --coverage
 
       # Upload coverage for e2e
       - run: mv coverage/coverage-final.json coverage/drizzle.json

DEVREADME.md

@@ -18,7 +18,9 @@ GENESIS_TXN_PATH=/work/network/genesis/local-genesis.txn
 
 ## Running tests
 
-Test are executed using jest. E2E tests (ending in `.e2e.test.ts`) require the **indy ledger**, **cheqd ledger**, **postgres database**, or **redis** to be running.
+Test are executed using vitest. E2E tests (ending in `.e2e.test.ts`) require the **indy ledger**, **cheqd ledger**, **postgres database**, or **redis** to be running. Drizzle tests (ending in `drizzle.e2e.test.ts`) require the **postgres database** to be running.
+
+NOTE: when running the Drizzle tests, you must first run `pnpm build`, as the CLI requires the built packages.
 
 When running tests that require a connection to the indy ledger pool, you can set the `TEST_AGENT_PUBLIC_DID_SEED`, `ENDORSER_AGENT_PUBLIC_DID_SEED` and `GENESIS_TXN_PATH` environment variables.
 

biome.json

@@ -82,7 +82,7 @@
   },
   "overrides": [
     {
-      "include": ["jest.config.ts", "./scripts/**"],
+      "include": ["./scripts/**"],
       "linter": {
         "rules": {
           "correctness": {
@@ -105,7 +105,7 @@
       "include": [
         "*.test.ts",
         "**/__tests__/**",
-        "jest.*.ts",
+        "vitest.*.ts",
         "samples/**",
         "demo/**",
         "demo-openid/**",

demo-openid/package.json

@@ -9,10 +9,10 @@
   },
   "license": "Apache-2.0",
   "scripts": {
-    "issuer": "ts-node src/IssuerInquirer.ts",
+    "issuer": "vite-node src/IssuerInquirer.ts",
     "provider": "tsx src/Provider.js",
-    "holder": "ts-node src/HolderInquirer.ts",
-    "verifier": "ts-node src/VerifierInquirer.ts",
+    "holder": "vite-node src/HolderInquirer.ts",
+    "verifier": "vite-node src/VerifierInquirer.ts",
     "proxies": "ngrok --config ngrok.yml,ngrok.auth.yml start provider issuer verifier"
   },
   "dependencies": {
@@ -36,7 +36,6 @@
     "@types/oidc-provider": "^8.8.1",
     "clear": "^0.1.0",
     "figlet": "^1.5.2",
-    "ts-node": "catalog:",
     "tsx": "^4.19.4"
   }
 }

demo-openid/src/Holder.ts

@@ -8,8 +8,8 @@ import { AskarModule } from '@credo-ts/askar'
 import {
   DidJwk,
   DidKey,
-  JwkDidCreateOptions,
-  KeyDidCreateOptions,
+  type JwkDidCreateOptions,
+  type KeyDidCreateOptions,
   Kms,
   Mdoc,
   W3cJsonLdVerifiableCredential,
@@ -26,7 +26,7 @@ import {
 } from '@credo-ts/openid4vc'
 import { askar } from '@openwallet-foundation/askar-nodejs'
 
-import { AskarModuleConfigStoreOptions } from '@credo-ts/askar'
+import type { AskarModuleConfigStoreOptions } from '@credo-ts/askar'
 import { BaseAgent } from './BaseAgent'
 import { Output, greenText } from './OutputClass'
 

demo-openid/src/Issuer.ts

@@ -1,16 +1,16 @@
 import type { DidKey, X509Certificate } from '@credo-ts/core'
 import {
-  OpenId4VcIssuerModuleConfigOptions,
+  type OpenId4VcIssuerModuleConfigOptions,
   OpenId4VcIssuerRecord,
   OpenId4VcModule,
-  OpenId4VcVerifierModuleConfigOptions,
+  type OpenId4VcVerifierModuleConfigOptions,
   OpenId4VcVerifierRecord,
-  OpenId4VciCredentialConfigurationsSupportedWithFormats,
-  OpenId4VciCredentialRequestToCredentialMapper,
-  OpenId4VciSignMdocCredentials,
-  OpenId4VciSignSdJwtCredentials,
-  OpenId4VciSignW3cCredentials,
-  VerifiedOpenId4VcCredentialHolderBinding,
+  type OpenId4VciCredentialConfigurationsSupportedWithFormats,
+  type OpenId4VciCredentialRequestToCredentialMapper,
+  type OpenId4VciSignMdocCredentials,
+  type OpenId4VciSignSdJwtCredentials,
+  type OpenId4VciSignW3cCredentials,
+  type VerifiedOpenId4VcCredentialHolderBinding,
 } from '@credo-ts/openid4vc'
 
 import { AskarModule, transformSeedToPrivateJwk } from '@credo-ts/askar'

demo/package.json

@@ -9,8 +9,8 @@
   },
   "license": "Apache-2.0",
   "scripts": {
-    "alice": "ts-node src/AliceInquirer.ts",
-    "faber": "ts-node src/FaberInquirer.ts"
+    "alice": "vite-node src/AliceInquirer.ts",
+    "faber": "vite-node src/FaberInquirer.ts"
   },
   "dependencies": {
     "@hyperledger/anoncreds-nodejs": "catalog:",
@@ -31,7 +31,6 @@
     "@types/figlet": "^1.5.4",
     "@types/inquirer": "^8.2.6",
     "clear": "^0.1.0",
-    "figlet": "^1.5.2",
-    "ts-node": "catalog:"
+    "figlet": "^1.5.2"
   }
 }

demo/src/BaseAgent.ts

@@ -29,12 +29,12 @@ import {
 } from '@credo-ts/didcomm'
 import { IndyVdrAnonCredsRegistry, IndyVdrIndyDidResolver, IndyVdrModule } from '@credo-ts/indy-vdr'
 import { DidCommHttpInboundTransport, agentDependencies } from '@credo-ts/node'
-import { HederaNetwork } from '@hiero-did-sdk/client'
+import type { HederaNetwork } from '@hiero-did-sdk/client'
 import { anoncreds } from '@hyperledger/anoncreds-nodejs'
 import { indyVdr } from '@hyperledger/indy-vdr-nodejs'
 import { askar } from '@openwallet-foundation/askar-nodejs'
 
-import { AskarModuleConfigStoreOptions } from '@credo-ts/askar'
+import type { AskarModuleConfigStoreOptions } from '@credo-ts/askar'
 import { HederaAnonCredsRegistry, HederaDidRegistrar, HederaDidResolver, HederaModule } from '@credo-ts/hedera'
 import { greenText } from './OutputClass'