-
Notifications
You must be signed in to change notification settings - Fork 916
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Remove examples and other unwanted artifacts from installed dependenc…
…ies (#4896) (#4905) Signed-off-by: Miki <miki@amazon.com> (cherry picked from commit cf07424) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> # Conflicts: # CHANGELOG.md Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
- Loading branch information
1 parent
5910a95
commit fe367eb
Showing
2 changed files
with
42 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
/* | ||
* Copyright OpenSearch Contributors | ||
* SPDX-License-Identifier: Apache-2.0 | ||
*/ | ||
|
||
/* eslint no-restricted-syntax: 0 */ | ||
|
||
const fs = require('fs/promises'); | ||
|
||
/** | ||
* Some libraries pack their demos and examples into their release artifacts. | ||
* This unwanted content makes our release artifacts larger but more importantly, | ||
* some contain in-browser references to outdated and vulnerable versions of | ||
* libraries that are not even mentioned in the dependency tree. This is a | ||
* problem when vulnerability scanners point them out, and we have no way to fix | ||
* them. This function looks for folders that are unwanted and deletes them. | ||
*/ | ||
const removeUnwantedFolders = async (root, unwantedNames) => { | ||
const items = await fs.readdir(root, { withFileTypes: true }); | ||
const promises = []; | ||
for (const item of items) { | ||
if (!item.isDirectory()) continue; | ||
|
||
if (unwantedNames.includes(item.name)) { | ||
promises.push(fs.rm(`${root}/${item.name}`, { recursive: true, force: true })); | ||
} else { | ||
promises.push(...(await removeUnwantedFolders(`${root}/${item.name}`, unwantedNames))); | ||
} | ||
} | ||
|
||
return promises; | ||
}; | ||
const run = async () => { | ||
const promises = await removeUnwantedFolders('node_modules', ['demo', 'example', 'examples']); | ||
await Promise.all(promises); | ||
}; | ||
|
||
run().catch((err) => { | ||
console.error(err); | ||
process.exit(1); | ||
}); |