Skip to content

Commit 8eff730

Browse files
authored
Merge pull request #4175 from aryaemami59/use-tsconfigPaths
Use vite-tsconfig-paths to make path aliasing easier
2 parents a89db66 + 773d636 commit 8eff730

File tree

4 files changed

+40
-24
lines changed

4 files changed

+40
-24
lines changed

packages/toolkit/package.json

+1
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,7 @@
9191
"tsup": "^7.2.0",
9292
"tsx": "^3.12.2",
9393
"typescript": "^5.3.3",
94+
"vite-tsconfig-paths": "^4.3.1",
9495
"vitest": "^1.1.3",
9596
"yargs": "^15.3.1"
9697
},

packages/toolkit/tsconfig.base.json

+5-11
Original file line numberDiff line numberDiff line change
@@ -31,20 +31,14 @@
3131
"downlevelIteration": false,
3232
"allowSyntheticDefaultImports": true,
3333
"emitDeclarationOnly": true,
34-
"baseUrl": ".",
3534
"types": ["vitest/globals", "vitest/importMeta"],
3635
"paths": {
37-
"@reduxjs/toolkit": ["src/index.ts"], // @remap-prod-remove-line
38-
"@reduxjs/toolkit/react": ["src/react/index.ts"], // @remap-prod-remove-line
39-
"@reduxjs/toolkit/query": ["src/query/index.ts"], // @remap-prod-remove-line
40-
"@reduxjs/toolkit/query/react": ["src/query/react/index.ts"], // @remap-prod-remove-line
41-
// for type imports in tests only
42-
"@reduxjs/toolkit/dist/*": ["src/*"], // @remap-prod-remove-line
43-
// for type imports in tests only
44-
"@reduxjs/toolkit/dist/query/*": ["src/query/*"], // @remap-prod-remove-line
36+
"@reduxjs/toolkit": ["./src/index.ts"], // @remap-prod-remove-line
37+
"@reduxjs/toolkit/react": ["./src/react/index.ts"], // @remap-prod-remove-line
38+
"@reduxjs/toolkit/query": ["./src/query/index.ts"], // @remap-prod-remove-line
39+
"@reduxjs/toolkit/query/react": ["./src/query/react/index.ts"], // @remap-prod-remove-line
4540
// internal imports in tests only
46-
"@internal/*": ["src/*"],
47-
"react": ["node_modules/react"]
41+
"@internal/*": ["./src/*"]
4842
}
4943
}
5044
}

packages/toolkit/vitest.config.mts

+3-13
Original file line numberDiff line numberDiff line change
@@ -1,29 +1,19 @@
1-
import { defineConfig } from 'vitest/config'
2-
31
import path from 'node:path'
42
import { fileURLToPath } from 'node:url'
3+
import tsconfigPaths from 'vite-tsconfig-paths'
4+
import { defineConfig } from 'vitest/config'
55

66
// No __dirname under Node ESM
77
const __filename = fileURLToPath(import.meta.url)
88
const __dirname = path.dirname(__filename)
99

1010
export default defineConfig({
11+
plugins: [tsconfigPaths({ root: __dirname })],
1112
test: {
1213
globals: true,
1314
environment: 'jsdom',
1415
setupFiles: ['./vitest.setup.ts'],
1516
include: ['./src/**/*.(spec|test).[jt]s?(x)'],
16-
alias: {
17-
// prettier-ignore
18-
'@reduxjs/toolkit/query/react': path.join(__dirname,'./src/query/react/index.ts'), // @remap-prod-remove-line
19-
'@reduxjs/toolkit/query': path.join(__dirname, './src/query/index.ts'), // @remap-prod-remove-line
20-
'@reduxjs/toolkit/react': path.join(__dirname, './src/index.ts'), // @remap-prod-remove-line
21-
'@reduxjs/toolkit': path.join(__dirname, './src/index.ts'), // @remap-prod-remove-line
22-
23-
// this mapping is disabled as we want `dist` imports in the tests only to be used for "type-only" imports which don't play a role for jest
24-
//'^@reduxjs/toolkit/dist/(.*)$': '<rootDir>/src/*',
25-
'@internal': path.join(__dirname, './src'),
26-
},
2717
server: { deps: { inline: ['redux', '@reduxjs/toolkit'] } },
2818
},
2919
})

yarn.lock

+31
Original file line numberDiff line numberDiff line change
@@ -7320,6 +7320,7 @@ __metadata:
73207320
tsup: ^7.2.0
73217321
tsx: ^3.12.2
73227322
typescript: ^5.3.3
7323+
vite-tsconfig-paths: ^4.3.1
73237324
vitest: ^1.1.3
73247325
yargs: ^15.3.1
73257326
peerDependencies:
@@ -27800,6 +27801,20 @@ __metadata:
2780027801
languageName: node
2780127802
linkType: hard
2780227803

27804+
"tsconfck@npm:^3.0.1":
27805+
version: 3.0.2
27806+
resolution: "tsconfck@npm:3.0.2"
27807+
peerDependencies:
27808+
typescript: ^5.0.0
27809+
peerDependenciesMeta:
27810+
typescript:
27811+
optional: true
27812+
bin:
27813+
tsconfck: bin/tsconfck.js
27814+
checksum: bf40e547a01610571a55ff33beb74751942561963f9c5f1152ad415ad4bd27eb90d65e9fb2edbb60ecb9407b115d1d52a633fc1f83cf4ea0424041a10e7eeea3
27815+
languageName: node
27816+
linkType: hard
27817+
2780327818
"tsconfig-paths@npm:^3.14.1":
2780427819
version: 3.14.1
2780527820
resolution: "tsconfig-paths@npm:3.14.1"
@@ -28818,6 +28833,22 @@ __metadata:
2881828833
languageName: node
2881928834
linkType: hard
2882028835

28836+
"vite-tsconfig-paths@npm:^4.3.1":
28837+
version: 4.3.1
28838+
resolution: "vite-tsconfig-paths@npm:4.3.1"
28839+
dependencies:
28840+
debug: ^4.1.1
28841+
globrex: ^0.1.2
28842+
tsconfck: ^3.0.1
28843+
peerDependencies:
28844+
vite: "*"
28845+
peerDependenciesMeta:
28846+
vite:
28847+
optional: true
28848+
checksum: 2934d5d674e0b6ab8d435d4262ff30143c30512072d71ece5a3859a70a188cbdbcc2b0370277049ab551f7bbb6ffb2be94a0bf30154e5c58d38b32adce9740bd
28849+
languageName: node
28850+
linkType: hard
28851+
2882128852
"vite@npm:^5.0.0":
2882228853
version: 5.0.11
2882328854
resolution: "vite@npm:5.0.11"

0 commit comments

Comments
 (0)