Make a nicer error handler for the case where users don't have the cl…

…i installed (dataform-co#1130)

* Make a nicer error handler for the case where users don't have the cli installed

* Better messages

* No no-console

* Apparently the previous version was wrong

vscode/README.md

@@ -1,6 +1,6 @@
 ## This extension is in Alpha
 
-To run this extension you will need the dataform cli installed globally: `npm i -g @dataform/cli`.
+**To run this extension you will need the dataform cli installed globally: `npm i -g @dataform/cli`.**
 
 Includes:
 

vscode/package.json

@@ -6,7 +6,7 @@
   "description": "Syntax highlighting, compilation, and intellisense for Dataform and SQLX projects.",
   "displayName": "Dataform",
   "publisher": "dataform",
-  "version": "0.0.9",
+  "version": "0.0.11",
   "icon": "dataform_logo.png",
   "repository": {
     "url": "https://github.com/dataform-co/dataform/tree/master/vscode"

vscode/server.ts

@@ -45,6 +45,14 @@ documents.onDidSave(change => {
 
 async function compileAndValidate() {
   const spawnedProcess = spawn("dataform", ["compile", "--json"]);
+  spawnedProcess.on("error", err => {
+    // tslint:disable-next-line: no-console
+    console.error("Error running 'dataform compile':", err);
+    connection.sendNotification(
+      "error",
+      "Errors encountered when running 'dataform' CLI. Please ensure that the CLI is installed and up-to-date: 'npm i -g @dataform/cli'."
+    );
+  });
   const compileResult = await getProcessResult(spawnedProcess);
 
   const parsedResult: dataform.ICompiledGraph = JSON.parse(compileResult.stdout);