Skip to content

Commit 4f50ca3

Browse files
author
Daniel Del Core
committed
adds necessary null checks to new dependency whitelisting
1 parent dc41e54 commit 4f50ca3

File tree

1 file changed

+12
-4
lines changed

1 file changed

+12
-4
lines changed

packages/fetcher/src/index.ts

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -148,14 +148,22 @@ export async function fetchRemotePackage(
148148
const configExport = resolveConfigExport(pkg);
149149

150150
// Install whitelisted deps
151-
// @ts-expect-error legacy module loader doesn't know about these properties
152-
if (info.pkgJson) {
151+
if (
152+
// @ts-expect-error legacy module loader doesn't know about these properties
153+
info.pkgJson &&
154+
// @ts-expect-error legacy module loader doesn't know about these properties
155+
info.pkgJson.devDependencies &&
156+
configExport.dependencies
157+
) {
153158
await Promise.all(
154-
configExport.dependencies?.map(dep => {
159+
configExport.dependencies.map(dep => {
155160
// @ts-expect-error legacy module loader doesn't know about these properties
156161
const version = info.pkgJson.devDependencies[dep];
162+
163+
if (!version) return;
164+
157165
return packageManager.install(`${dep}@${version}`);
158-
}) ?? [],
166+
}),
159167
);
160168
}
161169

0 commit comments

Comments
 (0)