Don’t crash if we can’t create an analysis context

Author: kevmoo

bin/main.dart

@@ -18,7 +18,11 @@ main(List<String> arguments) async {
   log.Logger.root.onRecord.listen((record) {
     var wroteHeader = false;
 
-    var msg = LineSplitter.split(record.message).map((l) {
+    var msg = LineSplitter
+        .split([record.message, record.error, record.stackTrace]
+            .where((e) => e != null)
+            .join('\n'))
+        .map((l) {
       String prefix;
       if (wroteHeader) {
         prefix = '';

lib/pana.dart

@@ -71,22 +71,31 @@ class PackageAnalyzer {
         upgrade.exitCode, upgrade.stdout, upgrade.stderr, pkgDir);
     log.info("Package version: ${summary.pkgVersion}");
 
-    LibraryScanner libraryScanner =
-        new LibraryScanner(package, pkgDir, isFlutter);
     Map<String, List<String>> directLibs;
-    try {
-      directLibs = await libraryScanner.scanDirectLibs();
-    } catch (e, st) {
-      log.severe('Error scanning direct librariers', e, st);
-    }
     Map<String, List<String>> transitiveLibs;
+
+    LibraryScanner libraryScanner;
+
     try {
-      transitiveLibs = await libraryScanner.scanTransitiveLibs();
-      // TODO: add platform classification based on transitive libs
-    } catch (e, st) {
-      log.severe('Error scanning transitive librariers', e, st);
+      libraryScanner = new LibraryScanner(package, pkgDir, isFlutter);
+    } on StateError catch (e, stack) {
+      log.severe("Could not create LibraryScanner", e, stack);
+    }
+
+    if (libraryScanner != null) {
+      try {
+        directLibs = await libraryScanner.scanDirectLibs();
+      } catch (e, st) {
+        log.severe('Error scanning direct librariers', e, st);
+      }
+      try {
+        transitiveLibs = await libraryScanner.scanTransitiveLibs();
+        // TODO: add platform classification based on transitive libs
+      } catch (e, st) {
+        log.severe('Error scanning transitive librariers', e, st);
+      }
+      if (!keepTransitiveLibs) transitiveLibs = null;
     }
-    if (!keepTransitiveLibs) transitiveLibs = null;
 
     Set<AnalyzerOutput> analyzerItems;
     try {

lib/src/library_analyzer.dart

@@ -45,7 +45,7 @@ class LibraryScanner {
     var dotPackagesPath = p.join(projectPath, '.packages');
     if (!FileSystemEntity.isFileSync(dotPackagesPath)) {
       throw new StateError('A package configuration file was not found at the '
-          'expectetd location. $dotPackagesPath');
+          'expectetd location.\n$dotPackagesPath');
     }
 
     RunPubList runPubList;