* Run IntelliSense analysis for every build

* Added global enum for log level

Author: elektronikworkshop

src/arduino/arduino.ts

@@ -439,7 +439,8 @@ Please make sure the folder is not occupied by other procedures .`);
         const dc = DeviceContext.getInstance();
         const args: string[] = [];
         let restoreSerialMonitor: boolean = false;
-        let cocopa: ICoCoPaContext;
+        const cocopa = makeCompilerParserContext(dc);
+        const verbose = VscodeSettings.getInstance().logLevel === constants.LogLevel.Verbose;
 
         if (!this.boardManager.currentBoard) {
             if (mode !== BuildMode.Analyze) {
@@ -499,18 +500,12 @@ Please make sure the folder is not occupied by other procedures .`);
                       "--port", dc.port,
                       "--useprogrammer",
                       "--pref", "programmer=" + programmer);
-
-        } else if (mode === BuildMode.Analyze) {
-            cocopa = makeCompilerParserContext(dc);
-            args.push("--verify", "--verbose");
         } else {
             args.push("--verify");
         }
 
-        const verbose = VscodeSettings.getInstance().logLevel === "verbose";
-        if (mode !== BuildMode.Analyze && verbose) {
-            args.push("--verbose");
-        }
+        // We always build verbosely but filter the output based on the settings
+        args.push("--verbose");
 
         await vscode.workspace.saveAll(false);
 
@@ -530,7 +525,6 @@ Please make sure the folder is not occupied by other procedures .`);
                 logger.notifyUserError("InvalidOutPutPath", new Error(constants.messages.INVALID_OUTPUT_PATH + outputPath));
                 return false;
             }
-
             args.push("--pref", `build.path=${outputPath}`);
             arduinoChannel.info(`Please see the build logs in output path: ${outputPath}`);
         } else {
@@ -549,33 +543,31 @@ Please make sure the folder is not occupied by other procedures .`);
         args.push(path.join(ArduinoWorkspace.rootPath, dc.sketch));
 
         const cleanup = async () => {
-            if (cocopa) {
-                await cocopa.conclude();
-            }
+            await cocopa.conclude();
             if (mode === BuildMode.Upload || mode === BuildMode.UploadProgrammer) {
                 UsbDetector.getInstance().resumeListening();
                 if (restoreSerialMonitor) {
                     await SerialMonitor.getInstance().openSerialMonitor();
                 }
             }
         }
+        const stdoutcb = (line: string) => {
+            cocopa.callback(line);
+            if (verbose) {
+                arduinoChannel.channel.append(line);
+            }
+        }
+        const stderrcb = (line: string) => {
+            arduinoChannel.channel.append(line);
+        }
 
         return await util.spawn(
             this._settings.commandPath,
             args,
             undefined,
-            {
-                channel: !cocopa || cocopa && verbose ? arduinoChannel.channel : undefined,
-                stdout: cocopa ? cocopa.callback : undefined,
-            },
+            { stdout: stdoutcb, stderr: stderrcb },
         ).then(async () => {
             await cleanup();
-            if (mode !== BuildMode.Analyze) {
-                const cmd = os.platform() === "darwin"
-                    ? "Cmd + Alt + I"
-                    : "Ctrl + Alt + I";
-                arduinoChannel.info(`To rebuild your IntelliSense configuration run "${cmd}"`);
-            }
             arduinoChannel.end(`${mode} sketch '${dc.sketch}${os.EOL}`);
             return true;
         }, async (reason) => {

src/arduino/intellisense.ts

@@ -3,14 +3,14 @@
 
 import * as ccp from "cocopa";
 import * as fs from "fs";
+import * as os from "os";
 import * as path from "path";
 import * as tp from "typed-promisify";
 
 import * as constants from "../common/constants";
 import { arduinoChannel } from "../common/outputChannel";
 import { ArduinoWorkspace } from "../common/workspace";
 import { DeviceContext } from "../deviceContext";
-
 import { VscodeSettings } from "./vscodeSettings";
 
 export interface ICoCoPaContext {
@@ -103,6 +103,10 @@ export function makeCompilerParserContext(dc: DeviceContext): ICoCoPaContext {
             const estr = JSON.stringify(e);
             arduinoChannel.error(`Failed to read or write IntelliSense configuration: ${estr}`);
         }
+        const cmd = os.platform() === "darwin"
+            ? "Cmd + Alt + I"
+            : "Ctrl + Alt + I";
+        arduinoChannel.info(`To manually rebuild your IntelliSense configuration run "${cmd}"`);
     };
     return {
         callback: runner.callback(),

src/common/constants.ts

@@ -10,6 +10,11 @@ export const CPP_CONFIG_FILE = path.join(".vscode", "c_cpp_properties.json");
 /** The name of the intellisense configuration managed by vscode-arduino. */
 export const C_CPP_PROPERTIES_CONFIG_NAME = "Arduino";
 
+export enum LogLevel {
+    Info = "info",
+    Verbose = "verbose",
+};
+
 export const ARDUINO_MODE: vscode.DocumentSelector = [
     { language: "cpp", scheme: "file" },
     { language: "arduino", scheme: "file" },

src/debug/configurationProvider.ts

@@ -9,6 +9,7 @@ import ArduinoActivator from "../arduinoActivator";
 import ArduinoContext from "../arduinoContext";
 
 import { VscodeSettings } from "../arduino/vscodeSettings";
+import * as constants from "../common/constants";
 import * as platform from "../common/platform";
 import * as util from "../common/util";
 import { ArduinoWorkspace } from "../common/workspace";
@@ -77,7 +78,7 @@ export class ArduinoDebugConfigurationProvider implements vscode.DebugConfigurat
             await ArduinoActivator.activate();
         }
 
-        if (VscodeSettings.getInstance().logLevel === "verbose" && !config.logging) {
+        if (VscodeSettings.getInstance().logLevel === constants.LogLevel.Verbose && !config.logging) {
             config = {
                 ...config, logging: {
                     engineLogging: true,