fix uncaught parse errors

Author: bmeck

src/data/report.ts

@@ -174,7 +174,12 @@ export function activate(context: vscode.ExtensionContext, disposables?: Array<v
             let needRun = false
             for (const file of files) {
                 let existing = knownPkgFiles.get(file.fsPath)
-                const cacheKey = pkgJSONSrcToStableStringKey(file.str)
+                let cacheKey;
+                try {
+                    cacheKey = pkgJSONSrcToStableStringKey(file.str)
+                } catch {
+                    continue
+                }
                 if (!existing) {
                     needRun = true
                     existing = {

src/extension.ts

@@ -85,6 +85,9 @@ export async function activate(context: ExtensionContext) {
     )
     function normalizeReportAndLocations(report: SocketReport, doc: Parameters<typeof parseExternals>[0]) {
         const externals = parseExternals(doc)
+        if (!externals) {
+            return
+        }
         const issuesForSource = radixMergeReportIssues(report)
         let ranges: Record<string, Array<vscode.Range>> = Object.create(null);
         let prioritizedRanges: Record<string, { range: vscode.Range, prioritize: boolean }> = Object.create(null)

src/ui/javascript-file.ts

@@ -143,6 +143,9 @@ export function activate(
             if (!socketReport) return
             let hovers: Array<vscode.Hover> = []
             const externals = parseExternals(doc)
+            if (!externals) {
+                return
+            }
             const issues = radixMergeReportIssues(socketReport)
             for (const {name, range} of externals) {
                 const pkgIssues = issues.get(name)
@@ -269,7 +272,11 @@ ${issues.sort((a, b) => sortIssues({
         e.setDecorations(errorDecoration, errorDecorations);
         e.setDecorations(warningDecoration, warningDecorations);
 
-        for (const {name, range} of parseExternals(e.document)) {
+        const externals = parseExternals(e.document)
+        if (!externals) {
+            return
+        }
+        for (const {name, range} of externals) {
             if (isBuiltin(name)) {
                 const deco: vscode.DecorationOptions = {
                     range,

src/ui/parse-externals.ts

@@ -31,24 +31,29 @@ function getPackageNameFromVersionRange(name: string): string {
         name.split('@', 2)
     ).join('@');
 }
-export function parseExternals(doc: Pick<vscode.TextDocument, 'getText' | 'languageId' | 'fileName'>): Iterable<ExternalRef> {
+export function parseExternals(doc: Pick<vscode.TextDocument, 'getText' | 'languageId' | 'fileName'>): Iterable<ExternalRef> | null {
     const src = doc.getText();
     const results: Array<ExternalRef> = []
     if (SUPPORTED_LANGUAGE_IDS.includes(doc.languageId)) {
-        const ast = parser.parse(
-            src,
-            {
-                allowAwaitOutsideFunction: true,
-                allowImportExportEverywhere: true,
-                allowReturnOutsideFunction: true,
-                errorRecovery: true,
-                plugins: [
-                    'jsx',
-                    'typescript',
-                    'decorators'
-                ],
-            }
-        )
+        let ast
+        try {
+            ast = parser.parse(
+                src,
+                {
+                    allowAwaitOutsideFunction: true,
+                    allowImportExportEverywhere: true,
+                    allowReturnOutsideFunction: true,
+                    errorRecovery: true,
+                    plugins: [
+                        'jsx',
+                        'typescript',
+                        'decorators'
+                    ],
+                }
+            )
+        } catch {
+            return null
+        }
         function addResult(node: astTypes.namedTypes.Node, specifier: string) {
             if (/^[\.\/]/u.test(specifier)) {
                 return