Moved compiler parser to cocopa package for better testability and maintainability

Author: elektronikworkshop

BRANCHNOTES.md

@@ -36,8 +36,8 @@ Provide a configuration flag which allows the user to turn this feature off - th
 |                                       | :white_check_mark: Option to backup old configurations? |
 | **Configuration flags**               | :heavy_check_mark: Provide global disable flag for IntelliSense auto-config |
 |                                       | :white_check_mark: Provide project specific override for the global flag - most users will likely use the default setup and disable auto-generation for very specific projects |
-| **Unit tests**                        | :white_check_mark: Basic parser (known boards, match/no match)|
-|                                       | :white_check_mark: Querying of compiler built-in includes |
+| **Unit tests**                        | :heavy_check_mark: Basic parser (known boards, match/no match)|
+|                                       | :heavy_check_mark: Querying of compiler built-in includes (Note: to be changed to generic compiler such that Arduino is not necessary for unit testing) |
 |                                       | :white_check_mark: Throwing arbitrary data at parser engines |
 |                                       | :white_check_mark: JSON input |
 |                                       | :white_check_mark: JSON output |
@@ -47,7 +47,7 @@ Provide a configuration flag which allows the user to turn this feature off - th
 |                                       | :white_check_mark: Document configuration settings in [README.md](README.md) |
 |                                       | :white_check_mark: Document features in [README.md](README.md) |
 |                                       | :white_check_mark: How to handle compilation failure? Only set if more comprehensive |
-|                                       | :white_check_mark: Extract compiler command parser from vscode-arduino and [publish](https://itnext.io/step-by-step-building-and-publishing-an-npm-typescript-package-44fe7164964c) it as a separate package which will allow reusage and tests can be run without the heavy vscode-arduino rucksack |
+|                                       | :heavy_check_mark: Extract compiler command parser from vscode-arduino and [publish](https://itnext.io/step-by-step-building-and-publishing-an-npm-typescript-package-44fe7164964c) it as a separate package which will allow reusage and easy testing without heavy vscode-arduino rucksack. Done, see [cocopa](https://www.npmjs.com/package/cocopa) |
 |                                       | :white_check_mark: Finally: go through my code and look for TODOs |
 
 `*` not committed to branch yet

package.json

@@ -575,12 +575,13 @@
     "plugin-error": "^1.0.1",
     "tslint": "^5.20.1",
     "typemoq": "^1.6.0",
-    "typescript": "^2.2.1",
+    "typescript": "^3.7.5",
     "vscode-test": "^1.4.0",
     "webpack": "^4.44.1"
   },
   "dependencies": {
     "body-parser": "^1.16.1",
+    "cocopa": "0.0.3",
     "compare-versions": "^3.4.0",
     "eventemitter2": "^4.1.0",
     "express": "^4.14.1",

src/arduino/arduino.ts

@@ -6,6 +6,7 @@ import * as glob from "glob";
 import * as os from "os";
 import * as path from "path";
 import * as vscode from "vscode";
+import * as ccp from "cocopa";
 
 import * as constants from "../common/constants";
 import * as util from "../common/util";
@@ -23,7 +24,6 @@ import { ArduinoWorkspace } from "../common/workspace";
 import { SerialMonitor } from "../serialmonitor/serialMonitor";
 import { UsbDetector } from "../serialmonitor/usbDetector";
 import { ProgrammerManager } from "./programmerManager";
-import { CompilerCmdParserEngineGcc, CompilerCmdParser, CCppProperties } from "./intellisense";
 
 /**
  * Represent an Arduino application based on the official Arduino IDE.
@@ -353,8 +353,8 @@ export class ArduinoApp {
         if (!VscodeSettings.getInstance().disableIntelliSenseAutoGen) {
 
             // setup the parser with its engines
-            let gccParserEngine = new CompilerCmdParserEngineGcc(dc.sketch);
-            let compilerParser = new CompilerCmdParser([gccParserEngine]);
+            let gccParserEngine = new ccp.ParserGcc(dc.sketch);
+            let compilerParser = new ccp.Runner([gccParserEngine]);
 
             // set up the function to be called after parsing
             let _conclude = () => {
@@ -366,7 +366,7 @@ export class ArduinoApp {
                 }
             };
             return {
-                callback: compilerParser.callback,
+                callback: compilerParser.callback(),
                 conclude: _conclude,
             };
         }