Bump to v0.4.4 (#1331)

* change to dev branch (#1323)

* Change node serial to new serial cli (#1322)

* add args to build

* initial integration of serial cli

* Add gulp task for serial-monitor-cli insertion

* Temporarily disable serial monitor insertion task

* Uncomment task

* change platforms to match os.platform

* fix linting

* add relative path for serial

Co-authored-by: Alan Leung <[email protected]>

* Add fixes from code review (#1328)

* remove native serial dependencies

* fixes from code review

* Revert "remove native serial dependencies"

This reverts commit 6586a66.

* add vid pid support

* code review fixes

* revert stop to return bool

* pre release v0.4.4-rc1

* bump to 0.4.4-rc2

* bump to v0.4.4

Co-authored-by: Alan Leung <[email protected]>

Author: adiazulay

.github/workflows/build.yml

@@ -3,13 +3,13 @@ on:
   push:
     branches:
       - master
-      - develop
+      - dev
     tags:
       - v*
   pull_request:
     branches:
       - master
-      - develop
+      - dev
 
 
 jobs:

.vscode/tasks.json

@@ -15,7 +15,7 @@
     "tasks": [
         {
             "label": "build",
-            "args": [],
+            "args": ["build"],
             "type": "shell",
             "group": "build",
             "isBackground": false,
@@ -25,7 +25,7 @@
         },
         {
             "label": "build_without_view",
-            "args": [],
+            "args": ["build_without_view"],
             "group": "build",
             "isBackground": false,
             "problemMatcher": [

CHANGELOG.md

@@ -1,6 +1,11 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
+## Version 0.4.4
+
+### Changed
+- Serial monitor uses a new backend which doesn't break with updates of VSCode. [#1322](https://github.com/microsoft/vscode-arduino/pull/1322)
+
 ## Version 0.4.3
 
 ### Fixed

gulpfile.js

@@ -1,16 +1,18 @@
 const gulp = require("gulp");
-const eslint = require('gulp-eslint');
+const eslint = require("gulp-eslint");
 const tslint = require("gulp-tslint");
-const PluginError = require('plugin-error');
-const log = require('fancy-log');
+const PluginError = require("plugin-error");
+const log = require("fancy-log");
 const ts = require("gulp-typescript");
 const sourcemaps = require("gulp-sourcemaps");
 const webpack = require("webpack");
-const del = require('del');
+const del = require("del");
+const download = require("download");
+const extract = require("extract-zip");
 const fs = require("fs");
 const path = require("path");
 const childProcess = require("child_process");
-const argv = require('minimist')(process.argv.slice(2));
+const argv = require("minimist")(process.argv.slice(2));
 
 gulp.task("tslint", () => {
     return gulp.src(["**/*.ts", "**/*.tsx", "!**/*.d.ts", "!./vendor/**", "!node_modules/**", "!./src/views/node_modules/**", "!out/**"])
@@ -28,39 +30,61 @@ gulp.task("eslint", () => {
 gulp.task("html-webpack", (done) => {
     const config = require("./src/views/webpack.config.js");
     config.context = `${__dirname}/src/views`;
-    config.mode = argv.mode ? argv.mode : 'production';
+    config.mode = argv.mode ? argv.mode : "production";
     return webpack(config, (err, stats) => {
         const statsJson = stats.toJson();
         if (err || (statsJson.errors && statsJson.errors.length)) {
             statsJson.errors.forEach(webpackError => {
                 log.error(`Error (webpack): ${webpackError}`);
             });
 
-            throw new PluginError('webpack', JSON.stringify(err || statsJson.errors));
+            throw new PluginError("webpack", JSON.stringify(err || statsJson.errors));
         }
-        log('[webpack]', stats.toString());
+        log("[webpack]", stats.toString());
         done();
     });
 });
 
 gulp.task("node_modules-webpack", (done) => {
     const config = require("./webpack.config.js");
     config.context = `${__dirname}`;
-    config.mode = argv.mode ? argv.mode : 'production';
+    config.mode = argv.mode ? argv.mode : "production";
     return webpack(config, (err, stats) => {
         const statsJson = stats.toJson();
         if (err || (statsJson.errors && statsJson.errors.length)) {
             statsJson.errors.forEach(webpackError => {
                 log.error(`Error (webpack): ${webpackError}`);
             });
 
-            throw new PluginError('webpack', JSON.stringify(err || statsJson.errors));
+            throw new PluginError("webpack", JSON.stringify(err || statsJson.errors));
         }
-        log('[webpack]', stats.toString());
+        log("[webpack]", stats.toString());
         done();
     });
 });
 
+gulp.task("insert-serial-monitor-cli", async (done) => {
+    const platforms = [
+        "linux",
+        "darwin",
+        "win32",
+    ];
+    const release = "latest";
+    const destDir = path.resolve("out", "serial-monitor-cli");
+
+    async function downloadAndUnzip(platform) {
+        const fileName = `${platform}.zip`;
+        const zipPath = path.join(destDir, fileName);
+        await download(`https://github.com/microsoft/serial-monitor-cli/releases/${release}/download/${fileName}`,
+                       destDir,
+                       );
+        await extract(zipPath, { dir: path.join(destDir, platform) });
+        fs.rmSync(zipPath);
+    }
+
+    Promise.all(platforms.map(downloadAndUnzip)).then(done);
+});
+
 gulp.task("ts-compile", () => {
     const tsProject = ts.createProject("./tsconfig.json");
     return tsProject.src()
@@ -71,13 +95,13 @@ gulp.task("ts-compile", () => {
                 // Correct source map path.
                 const relativeSourcePath = path.relative(path.dirname(file.path), path.join(file.base, sourcePath));
                 return relativeSourcePath;
-            }
+            },
         }))
         .pipe(gulp.dest("out"));
 });
 
 gulp.task("clean", (done) => {
-    return del('out', done);
+    return del("out", done);
 });
 
 gulp.task("genAikey", (done) => {
@@ -100,8 +124,8 @@ gulp.task("test", (done) => {
     }
 
     // When using cli command "npm test" to exec test, the depended extensions (cpptools) are not available so that
-    // the extension cannot be activated. As a workaround, remove extensionDependencies from package.json before running test
-    // and restore extensionDependencies after test exited.
+    // the extension cannot be activated. As a workaround, remove extensionDependencies from package.json before
+    // running test and restore extensionDependencies after test exited.
     removeExtensionDependencies();
 
     const child = childProcess.spawn("node", ["./out/test/runTest"], {
@@ -132,11 +156,11 @@ gulp.task("test", (done) => {
     });
 });
 
-gulp.task("build", gulp.series("clean", "ts-compile", "html-webpack", "node_modules-webpack"));
+gulp.task("build", gulp.series("clean", "ts-compile", "html-webpack", "node_modules-webpack", "insert-serial-monitor-cli"));
 
 gulp.task("build_without_view", gulp.series("clean", "ts-compile"));
 
 gulp.task("watch", () => {
     gulp.watch(["./src/**/*", "./test/**/*", "!./src/views/**/*"], ["ts-compile"]);
     gulp.watch(["./src/views/**/*", "!./src/views/node_modules/**"], ["html-webpack"]);
-});
+});