undefined check

microsoft · Mar 7, 2025 · 8204812 · 8204812
1 parent b5abd14
commit 8204812
Showing 1 changed file with 11 additions and 6 deletions.
diff --git a/build-tools/packages/build-cli/src/library/repoPolicyCheck/npmPackages.ts b/build-tools/packages/build-cli/src/library/repoPolicyCheck/npmPackages.ts
@@ -2014,13 +2014,18 @@ export const handlers: Handler[] = [
 				const devDependencyErroredPkgs = devDependencyErrorString?.split(":")[1]?.split(",");
 				const dependencyErroredPkgs = dependencyErrorString?.split(":")[1]?.split(",");
 
-				// Delete unused dev dependency declaration from package.json
-				for (const pkgName of devDependencyErroredPkgs) {
-					delete packageJson.devDependencies?.[pkgName];
+				if (devDependencyErroredPkgs !== undefined && devDependencyErroredPkgs.length > 0) {
+					// Delete unused dev dependency declaration from package.json
+					for (const pkgName of devDependencyErroredPkgs) {
+						delete packageJson.devDependencies?.[pkgName];
+					}
 				}
-				// Delete unused dependency declaration from package.json
-				for (const pkgName of dependencyErroredPkgs) {
-					delete packageJson.dependencies?.[pkgName];
+
+				if (dependencyErroredPkgs !== undefined && dependencyErroredPkgs.length > 0) {
+					// Delete unused dependency declaration from package.json
+					for (const pkgName of dependencyErroredPkgs) {
+						delete packageJson.dependencies?.[pkgName];
+					}
 				}
 			});