Skip to content

Commit 8867757

Browse files
authored
fix: splitting compile into separate esm and cjs dists (#111)
* fix: splitting compile into separate esm and cjs dists * chore: testing ci * chore: testing ci * refactor: addressing pr comments * chore: addressing pr comments * chore: minor tweaks * refactor: addressing pr comments * chore: minor tweaks
1 parent b24d298 commit 8867757

File tree

7 files changed

+61
-9
lines changed

7 files changed

+61
-9
lines changed
File renamed without changes.
File renamed without changes.

package.json

Lines changed: 39 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,16 +8,51 @@
88
"engines": {
99
"node": ">=16.0"
1010
},
11-
"main": "./index.js",
11+
"type": "module",
12+
"main": "./cjs/index.js",
13+
"module": "./esm/index.js",
14+
"types": "./types/index.d.ts",
15+
"typesVersions": {
16+
"*": {
17+
".": [
18+
"types/index.d.ts"
19+
],
20+
"types/app": [
21+
"types/types/app.d.ts"
22+
],
23+
"types/app-client": [
24+
"types/types/app-client.d.ts"
25+
],
26+
"types/app-spec": [
27+
"types/types/app-spec.d.ts"
28+
],
29+
"types/transaction": [
30+
"types/types/transaction.d.ts"
31+
]
32+
}
33+
},
1234
"files": [
1335
"**/*"
1436
],
37+
"exports": {
38+
".": {
39+
"import": "./esm/index.js",
40+
"require": "./cjs/index.js"
41+
},
42+
"./testing": {
43+
"import": "./esm/testing/index.js",
44+
"require": "./cjs/testing/index.js"
45+
},
46+
"./index.d.ts": "./types/index.d.ts",
47+
"./package.json": "./package.json"
48+
},
1549
"scripts": {
1650
"build": "run-s build:*",
1751
"build:0-clean": "rimraf dist coverage",
18-
"build:1-compile": "tsc -p tsconfig.build.json",
19-
"build:2-copy-pkg-json": "npx --yes @makerx/ts-toolkit@latest copy-package-json",
20-
"build:3-copy-readme": "copyfiles README.md dist",
52+
"build:1-compile-cjs": "tsc -p tsconfig.cjs.json",
53+
"build:2-compile-esm": "tsc -p tsconfig.esm.json",
54+
"build:3-copy-pkg-json": "npx --yes @makerx/ts-toolkit@latest copy-package-json --main ./cjs/index.js --types ./types/index.d.ts --custom-sections exports typesVersions module",
55+
"build:4-copy-readme": "copyfiles README.md dist",
2156
"test": "dotenv -e .env -- jest --coverage --passWithNoTests",
2257
"lint": "eslint ./src/ --ext .ts",
2358
"lint:fix": "eslint ./src/ --ext .ts --fix",

src/types/app-client.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -534,8 +534,8 @@ describe('application-client', () => {
534534
invariant(result.confirmations)
535535
invariant(result.confirmations[0])
536536
expect(result.transactions.length).toBe(1)
537-
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
538537
const returnValue = algokit.getABIReturn(
538+
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
539539
{ method: client.getABIMethod('call_abi_foreign_refs')!, methodArgs: [] },
540540
result.confirmations[0],
541541
)

tsconfig.build.json

Lines changed: 0 additions & 4 deletions
This file was deleted.

tsconfig.cjs.json

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
{
2+
"extends": "./tsconfig.json",
3+
"compilerOptions": {
4+
"outDir": "./dist/cjs",
5+
"target": "esnext",
6+
"module": "commonjs",
7+
"moduleResolution": "node"
8+
},
9+
"exclude": ["src/**/*.spec.ts", "tests/**/*.*"]
10+
}

tsconfig.esm.json

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
{
2+
"extends": "./tsconfig.json",
3+
"compilerOptions": {
4+
"outDir": "./dist/esm",
5+
"target": "esnext",
6+
"module": "esnext",
7+
"moduleResolution": "node",
8+
"declarationDir": "./dist/types"
9+
},
10+
"exclude": ["src/**/*.spec.ts", "tests/**/*.*"]
11+
}

0 commit comments

Comments
 (0)