Skip to content

Commit 57e6fff

Browse files
authoredApr 11, 2022
fix: make resolve node-only (#962)
Resolve requires use of node features so add browser overrides to ensure it's not pulled in during bundling.
1 parent 86695bb commit 57e6fff

File tree

3 files changed

+12
-2
lines changed

3 files changed

+12
-2
lines changed
 

‎package.json

+3-1
Original file line numberDiff line numberDiff line change
@@ -80,6 +80,7 @@
8080
"import": "./utils/get-port.js"
8181
},
8282
"./resolve": {
83+
"browser": "./utils/resolve.browser.js",
8384
"import": "./utils/resolve.js"
8485
}
8586
},
@@ -287,6 +288,7 @@
287288
},
288289
"browser": {
289290
"fs": false,
290-
"./utils/fixtures.js": "./utils/fixtures.browser.js"
291+
"./utils/fixtures.js": "./utils/fixtures.browser.js",
292+
"./utils/resolve.js": "./utils/resolve.browser.js"
291293
}
292294
}

‎tsconfig-check.aegir.json

-1
This file was deleted.

‎utils/resolve.browser.js

+9
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,9 @@
1+
/**
2+
* Returns the full path to the requested resource, if available
3+
*
4+
* @param {string} filePath
5+
* @param {string} [module]
6+
*/
7+
export default function resolve (filePath, module = '') {
8+
throw new Error('Not supported in the browser')
9+
}

0 commit comments

Comments
 (0)
Please sign in to comment.