Reuse ExecutionContext with onError for Omniparser (#368)

To avoid swallowing throwables from parsers
openrewrite · Feb 6, 2025 · dcfbaf9 · dcfbaf9
1 parent b59913e
commit dcfbaf9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/plugin/src/main/java/org/openrewrite/gradle/isolated/AndroidProjectParser.java b/plugin/src/main/java/org/openrewrite/gradle/isolated/AndroidProjectParser.java
@@ -172,7 +172,7 @@ SourceFileStream parseProjectSourceSets(Project project,
                                         .collect(Collectors.toSet());
                         sourceSetSourceFiles = Stream.concat(
                                 sourceSetSourceFiles,
-                                omniParser.parse(accepted, baseDir, new InMemoryExecutionContext())
+                                omniParser.parse(accepted, baseDir, ctx)
                                         .map(it -> it.withMarkers(it.getMarkers().add(javaVersion))));
                         alreadyParsed.addAll(accepted);
                         sourceSetSize += accepted.size();

diff --git a/plugin/src/main/java/org/openrewrite/gradle/isolated/DefaultProjectParser.java b/plugin/src/main/java/org/openrewrite/gradle/isolated/DefaultProjectParser.java
@@ -881,7 +881,7 @@ private SourceFileStream parseGradleProjectSourceSets(Project subproject,
                     List<Path> accepted = omniParser.acceptedPaths(baseDir, resourcesDir.toPath());
                     sourceSetSourceFiles = Stream.concat(
                             sourceSetSourceFiles,
-                            omniParser.parse(accepted, baseDir, new InMemoryExecutionContext())
+                            omniParser.parse(accepted, baseDir, ctx)
                                     .map(it -> it.withMarkers(it.getMarkers().add(javaVersion))));
                     alreadyParsed.addAll(accepted);
                     sourceSetSize += accepted.size();