From e935513735719eda7b96f4767985a06a015b6add Mon Sep 17 00:00:00 2001
From: Eric Jizba <erijiz@microsoft.com>
Date: Tue, 2 Apr 2019 13:27:29 -0700
Subject: [PATCH] Don't use webpack for local development (#1151)

But still provide that option
---
 .vscode/launch.json                           | 20 ++++++++++++++++++
 main.js                                       |  4 +++-
 ...CoreNodeModule.js => getCoreNodeModule.ts} | 21 +++++++------------
 webpack.config.js                             |  2 +-
 4 files changed, 31 insertions(+), 16 deletions(-)
 rename src/utils/{getCoreNodeModule.js => getCoreNodeModule.ts} (59%)

diff --git a/.vscode/launch.json b/.vscode/launch.json
index f0c78eb0..e3c386be 100644
--- a/.vscode/launch.json
+++ b/.vscode/launch.json
@@ -15,6 +15,26 @@
             "outFiles": [
                 "${workspaceFolder}/**/*.js"
             ],
+            "preLaunchTask": "npm: compile",
+            "env": {
+                "AZCODE_IGNORE_BUNDLE": "1",
+                "DEBUGTELEMETRY": "1",
+                "NODE_DEBUG": ""
+            }
+        },
+        {
+            "name": "Launch Extension (webpack)",
+            "type": "extensionHost",
+            "request": "launch",
+            "runtimeExecutable": "${execPath}",
+            "args": [
+                "--extensionDevelopmentPath=${workspaceFolder}"
+            ],
+            "stopOnEntry": false,
+            "sourceMaps": true,
+            "outFiles": [
+                "${workspaceFolder}/**/*.js"
+            ],
             "preLaunchTask": "npm: webpack",
             "env": {
                 "DEBUGTELEMETRY": "1",
diff --git a/main.js b/main.js
index 026db5ab..2df67451 100644
--- a/main.js
+++ b/main.js
@@ -16,7 +16,9 @@ let perfStats = {
 
 Object.defineProperty(exports, "__esModule", { value: true });
 
-const extension = require("./dist/extension.bundle");
+const ignoreBundle = !/^(false|0)?$/i.test(process.env.AZCODE_IGNORE_BUNDLE || '');
+const extensionPath = ignoreBundle ? "./out/src/extension" : "./dist/extension.bundle";
+const extension = require(extensionPath);
 
 async function activate(ctx) {
     return await extension.activateInternal(ctx, perfStats);
diff --git a/src/utils/getCoreNodeModule.js b/src/utils/getCoreNodeModule.ts
similarity index 59%
rename from src/utils/getCoreNodeModule.js
rename to src/utils/getCoreNodeModule.ts
index af993a35..38e869b4 100644
--- a/src/utils/getCoreNodeModule.js
+++ b/src/utils/getCoreNodeModule.ts
@@ -3,31 +3,24 @@
  *  Licensed under the MIT License. See LICENSE.md in the project root for license information.
  *--------------------------------------------------------------------------------------------*/
 
-"use strict";
-
-// Our util/getCoreNodeModule.js file uses a dynamic require that depends on the environment and so has to be excluded
-// from webpack'ing, and we'll just copy it to the distribution folder.
-//
-// Since webpack shouldn't have to depend on npm build to transpile .ts -> .js, we keep this file in .js.
-
-Object.defineProperty(exports, "__esModule", { value: true });
-const vscode = require("vscode");
+import * as vscode from 'vscode';
 
 /**
  * Returns a node module installed with VSCode, or undefined if it fails.
  */
-function getCoreNodeModule(moduleName) {
+export function getCoreNodeModule<T>(moduleName: string): T | undefined {
     try {
         // tslint:disable-next-line:non-literal-require no-unsafe-any
         return require(`${vscode.env.appRoot}/node_modules.asar/${moduleName}`);
+    } catch (err) {
+        // ignore
     }
-    catch (err) { }
+
     try {
         // tslint:disable-next-line:non-literal-require no-unsafe-any
         return require(`${vscode.env.appRoot}/node_modules/${moduleName}`);
+    } catch (err) {
+        // ignore
     }
-    catch (err) { }
     return undefined;
 }
-
-exports.getCoreNodeModule = getCoreNodeModule;
diff --git a/webpack.config.js b/webpack.config.js
index c68db6db..2bbcd1c7 100644
--- a/webpack.config.js
+++ b/webpack.config.js
@@ -35,7 +35,7 @@ let config = dev.getDefaultWebpackConfig({
         // Copy files to dist folder where the runtime can find them
         new CopyWebpackPlugin([
             // getCoreNodeModule.js -> dist/node_modules/getCoreNodeModule.js
-            { from: './src/utils/getCoreNodeModule.js', to: 'node_modules' }
+            { from: './out/src/utils/getCoreNodeModule.js', to: 'node_modules' }
         ])
     ]
 });