Skip to content

Commit

Permalink
fix: fix npm alias present in requires of dependencies
Browse files Browse the repository at this point in the history
Alias (npm:*) from requires of a npm project are removed with the method NodePackageAnalyzer.shouldSkipDependency but not from the dependencies of dependencies. This modification is used to update this.
  • Loading branch information
phoenix741 committed May 5, 2023
1 parent 7211c5f commit 4510c8c
Showing 1 changed file with 9 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -244,7 +244,15 @@ private static JsonObject buildDependencies(JsonObject dep, MultiValuedMap<Strin
depBuilder.add("integrity", dep.getString("integrity"));
}
if (dep.containsKey("requires")) {
depBuilder.add("requires", dep.getJsonObject("requires"));
final JsonObjectBuilder requiresBuilder = Json.createObjectBuilder();
dep.getJsonObject("requires").forEach((key, value) -> {
if (NodePackageAnalyzer.shouldSkipDependency(key, ((JsonString) value).getString())) {
return;
}

requiresBuilder.add(key, value);
});
depBuilder.add("requires", requiresBuilder.build());
}
if (dep.containsKey("dependencies")) {
final JsonObjectBuilder dependeciesBuilder = Json.createObjectBuilder();
Expand Down

0 comments on commit 4510c8c

Please sign in to comment.