Skip to content

Commit

Permalink
refactor: use require.resolve instead of require-relative and resolve…
Browse files Browse the repository at this point in the history
…-from
  • Loading branch information
merceyz committed Jan 9, 2021
1 parent 7e740f0 commit 2ff541e
Show file tree
Hide file tree
Showing 6 changed files with 14 additions and 19 deletions.
7 changes: 4 additions & 3 deletions packages/cli/lib/lib/webpack/prerender.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ const { readFileSync } = require('fs');
const stackTrace = require('stack-trace');
const URL = require('url');
const { SourceMapConsumer } = require('source-map');
const requireRelative = require('require-relative');

module.exports = function (env, params) {
params = params || {};
Expand All @@ -27,8 +26,10 @@ module.exports = function (env, params) {
return '';
}
const { cwd } = env;
const preact = require(requireRelative.resolve('preact', cwd));
const renderToString = require(requireRelative.resolve('preact-render-to-string', cwd));
const preact = require(require.resolve('preact', { paths: [cwd] }));
const renderToString = require(require.resolve('preact-render-to-string', {
paths: [cwd],
}));
return renderToString(preact.h(app, { ...params, url }));
} catch (err) {
let stack = stackTrace.parse(err).filter(s => s.getFileName() === entry)[0];
Expand Down
5 changes: 3 additions & 2 deletions packages/cli/lib/lib/webpack/proxy-loader.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,4 @@
var utils = require('loader-utils');
var requireRelative = require('require-relative');

function proxyLoader(source, map) {
var options = utils.getOptions(this);
Expand All @@ -17,7 +16,9 @@ function proxyLoader(source, map) {

var loader;
try {
loader = requireRelative(proxyOptions.loader, proxyOptions.cwd);
loader = require(require.resolve(proxyOptions.loader, {
paths: [proxyOptions.cwd],
}));
} catch (e) {
loader = require(proxyOptions.loader);
}
Expand Down
9 changes: 5 additions & 4 deletions packages/cli/lib/lib/webpack/utils.js
Original file line number Diff line number Diff line change
@@ -1,10 +1,11 @@
const resolveFrom = require('resolve-from');

function isInstalledVersionPreactXOrAbove(cwd) {
try {
return (
parseInt(require(resolveFrom(cwd, 'preact/package.json')).version, 10) >=
10
parseInt(
require(require.resolve('preact/package.json', { paths: [cwd] }))
.version,
10
) >= 10
);
} catch (e) {}
return false;
Expand Down
5 changes: 2 additions & 3 deletions packages/cli/lib/lib/webpack/webpack-base-config.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@ const { readFileSync, existsSync } = require('fs');
const { isInstalledVersionPreactXOrAbove } = require('./utils');
const autoprefixer = require('autoprefixer');
const browserslist = require('browserslist');
const requireRelative = require('require-relative');
const MiniCssExtractPlugin = require('mini-css-extract-plugin');
const FixStyleOnlyEntriesPlugin = require('webpack-fix-style-only-entries');
const ProgressBarPlugin = require('progress-bar-webpack-plugin');
Expand Down Expand Up @@ -97,12 +96,12 @@ module.exports = function (env) {
let compat = 'preact-compat';
try {
compat = dirname(
requireRelative.resolve('preact/compat/package.json', cwd)
require.resolve('preact/compat/package.json', { paths: [cwd] })
);
} catch (e) {
try {
compat = dirname(
requireRelative.resolve('preact-compat/package.json', cwd)
require.resolve('preact-compat/package.json', { paths: [cwd] })
);
} catch (e) {}
}
Expand Down
2 changes: 0 additions & 2 deletions packages/cli/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -134,8 +134,6 @@
"prompts": "^2.2.1",
"raw-loader": "^4.0.0",
"react-refresh": "0.8.3",
"require-relative": "^0.8.7",
"resolve-from": "^5.0.0",
"rimraf": "^3.0.2",
"sade": "^1.4.1",
"size-plugin": "^2.0.1",
Expand Down
5 changes: 0 additions & 5 deletions yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -12366,11 +12366,6 @@ require-main-filename@^2.0.0:
resolved "https://registry.yarnpkg.com/require-main-filename/-/require-main-filename-2.0.0.tgz#d0b329ecc7cc0f61649f62215be69af54aa8989b"
integrity sha512-NKN5kMDylKuldxYLSUfrbo5Tuzh4hd+2E8NPPX02mZtn1VuREQToYe/ZdlJy+J3uCpfaiGF05e7B8W0iXbQHmg==

require-relative@^0.8.7:
version "0.8.7"
resolved "https://registry.yarnpkg.com/require-relative/-/require-relative-0.8.7.tgz#7999539fc9e047a37928fa196f8e1563dabd36de"
integrity sha1-eZlTn8ngR6N5KPoZb44VY9q9Nt4=

requires-port@^1.0.0:
version "1.0.0"
resolved "https://registry.yarnpkg.com/requires-port/-/requires-port-1.0.0.tgz#925d2601d39ac485e091cf0da5c6e694dc3dcaff"
Expand Down

0 comments on commit 2ff541e

Please sign in to comment.