Handle RunTime exception thrown in TSCRuntime init.

openrewrite · Nov 30, 2023 · de89241 · de89241
1 parent b0b6362
commit de89241
Showing 1 changed file with 31 additions and 29 deletions.
diff --git a/src/main/java/org/openrewrite/javascript/JavaScriptParser.java b/src/main/java/org/openrewrite/javascript/JavaScriptParser.java
@@ -24,7 +24,6 @@
 import org.openrewrite.internal.lang.Nullable;
 import org.openrewrite.java.internal.JavaTypeCache;
 import org.openrewrite.javascript.internal.JavetNativeBridge;
-import org.openrewrite.javascript.internal.TsTreePrinter;
 import org.openrewrite.javascript.internal.TypeScriptParserVisitor;
 import org.openrewrite.javascript.internal.tsc.TSCRuntime;
 import org.openrewrite.javascript.tree.JS;
@@ -41,7 +40,6 @@
 import java.util.*;
 import java.util.stream.Stream;
 
-@SuppressWarnings("ConstantValue")
 public class JavaScriptParser implements Parser {
 
     @Nullable
@@ -118,34 +116,38 @@ public Stream<SourceFile> parseInputs(Iterable<Input> sources, @Nullable Path re
             sourceTextsForTSC.put(relativePath, sourceText.sourceText);
         });
 
-        //noinspection resource
-        runtime().parseSourceTexts(
-                sourceTextsForTSC,
-                (node, context) -> {
-                    SourceWrapper source = sourcesByRelativePath.get(context.getRelativeSourcePath());
-
-                    parsingListener.startedParsing(source.getInput());
-                    TypeScriptParserVisitor fileMapper = new TypeScriptParserVisitor(
-                            node,
-                            context,
-                            source.getSourcePath(),
-                            new JavaTypeCache(),
-                            source.getCharset().toString(),
-                            source.isCharsetBomMarked(),
-                            styles
-                    );
-                    SourceFile cu;
-                    try {
-                        cu = fileMapper.visitSourceFile();
-                        parsingListener.parsed(source.getInput(), cu);
-                    } catch (Throwable t) {
-                        ((ExecutionContext) pctx).getOnError().accept(t);
-                        cu = ParseError.build(JavaScriptParser.builder().build(), source.getInput(), relativeTo, pctx, t);
+        try {
+            //noinspection resource
+            runtime().parseSourceTexts(
+                    sourceTextsForTSC,
+                    (node, context) -> {
+                        SourceWrapper source = sourcesByRelativePath.get(context.getRelativeSourcePath());
+
+                        parsingListener.startedParsing(source.getInput());
+                        TypeScriptParserVisitor fileMapper = new TypeScriptParserVisitor(
+                                node,
+                                context,
+                                source.getSourcePath(),
+                                new JavaTypeCache(),
+                                source.getCharset().toString(),
+                                source.isCharsetBomMarked(),
+                                styles
+                        );
+                        SourceFile cu;
+                        try {
+                            cu = fileMapper.visitSourceFile();
+                            parsingListener.parsed(source.getInput(), cu);
+                        } catch (Throwable t) {
+                            ((ExecutionContext) pctx).getOnError().accept(t);
+                            cu = ParseError.build(JavaScriptParser.builder().build(), source.getInput(), relativeTo, pctx, t);
+                        }
+
+                        compilationUnits.add(cu);
                     }
-
-                    compilationUnits.add(cu);
-                }
-        );
+            );
+        } catch (Exception e) {
+            return acceptedInputs(sources).map(input -> ParseError.build(this, input, relativeTo, ctx, e));
+        }
         return compilationUnits.stream();
     }