[BUG] Investigate changing the sideEffects: false to list only the fi…

…les that include the shims module from the all AI modules so that webpack can evaluate correctly #1509

Investigate removing the globals __extends() and __assign() populated by applicationinsights-shims #1280

.gitignore

@@ -249,6 +249,7 @@ temp
 # Rush files
 common/temp/**
 package-deps.json
+package-lock.json
 
 # test output
 aicore.tests.js*

AISKU/package.json

@@ -46,6 +46,7 @@
         "grunt-ts": "^6.0.0-beta.22",
         "grunt-tslint": "^5.0.2",
         "globby": "^11.0.0",
+        "magic-string": "^0.25.7",
         "@rollup/plugin-commonjs": "^15.1.0",
         "@rollup/plugin-node-resolve": "^9.0.0",
         "@rollup/plugin-replace": "^2.3.3",
@@ -60,7 +61,7 @@
     },
     "dependencies": {
         "@microsoft/dynamicproto-js": "^1.1.1",
-        "@microsoft/applicationinsights-shims" : "1.0.3",
+        "@microsoft/applicationinsights-shims" : "2.0.0",
         "@microsoft/applicationinsights-analytics-js": "2.6.1",
         "@microsoft/applicationinsights-channel-js": "2.6.1",
         "@microsoft/applicationinsights-common": "2.6.1",

AISKU/src/applicationinsights-web.ts

@@ -1,5 +1,3 @@
-import '@microsoft/applicationinsights-shims';
-
 export {
     IApplicationInsights,
     Snippet,

AISKU/tsconfig.json

@@ -9,8 +9,9 @@
     "alwaysStrict": true,
     "suppressImplicitAnyIndexErrors": true,
     "allowSyntheticDefaultImports": true,
-    "importHelpers": false,
+    "importHelpers": true,
     "noEmitHelpers": true,
+    "skipLibCheck": true,
     "forceConsistentCasingInFileNames": true,
     "declaration": true,
     "declarationDir": "AISKU/types",

AISKULight/index.ts

@@ -1,7 +1,6 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT License.
 
-import '@microsoft/applicationinsights-shims';
 import {
     IConfiguration,
     AppInsightsCore,

AISKULight/package.json

@@ -25,6 +25,7 @@
         "grunt-run": "^0.8.1",
         "grunt-ts": "^6.0.0-beta.22",
         "globby": "^11.0.0",
+        "magic-string": "^0.25.7",
         "@rollup/plugin-commonjs": "^15.1.0",
         "@rollup/plugin-node-resolve": "^9.0.0",
         "@rollup/plugin-replace": "^2.3.3",
@@ -37,7 +38,7 @@
     },
     "dependencies": {
         "@microsoft/dynamicproto-js": "^1.1.1",
-        "@microsoft/applicationinsights-shims" : "1.0.3",
+        "@microsoft/applicationinsights-shims" : "2.0.0",
         "@microsoft/applicationinsights-common": "2.6.1",
         "@microsoft/applicationinsights-channel-js": "2.6.1",
         "@microsoft/applicationinsights-core-js": "2.6.1"

AISKULight/tsconfig.json

@@ -9,8 +9,9 @@
     "alwaysStrict": true,
     "suppressImplicitAnyIndexErrors": true,
     "allowSyntheticDefaultImports": true,
-    "importHelpers": false,
+    "importHelpers": true,
     "noEmitHelpers": true,
+    "skipLibCheck": true,
     "forceConsistentCasingInFileNames": true,
     "declaration": true,
     "declarationDir": "AISKULight/types",

channels/applicationinsights-channel-js/package.json

@@ -29,6 +29,7 @@
         "grunt-run": "^0.8.1",
         "grunt-ts": "^6.0.0-beta.22",
         "globby": "^11.0.0",
+        "magic-string": "^0.25.7",
         "@rollup/plugin-commonjs": "^15.1.0",
         "@rollup/plugin-node-resolve": "^9.0.0",
         "@rollup/plugin-replace": "^2.3.3",
@@ -42,7 +43,7 @@
     },
     "dependencies": {
         "@microsoft/dynamicproto-js": "^1.1.1",
-        "@microsoft/applicationinsights-shims" : "1.0.3",
+        "@microsoft/applicationinsights-shims" : "2.0.0",
         "@microsoft/applicationinsights-core-js": "2.6.1",
         "@microsoft/applicationinsights-common": "2.6.1"
     },

channels/applicationinsights-channel-js/src/applicationinsights-channel-js.ts

@@ -1,3 +1 @@
-import '@microsoft/applicationinsights-shims';
-
 export { Sender } from "./Sender";

channels/applicationinsights-channel-js/tsconfig.json

@@ -10,8 +10,9 @@
     "suppressImplicitAnyIndexErrors": true,
     "allowSyntheticDefaultImports": true,
     "forceConsistentCasingInFileNames": true,
-    "importHelpers": false,
+    "importHelpers": true,
     "noEmitHelpers": true,
+    "skipLibCheck": true,
     "declaration": true,
     "declarationDir": "channels/applicationinsights-channel-js/types",
     "outDir": "dist-esm",

common/Tests/Framework/package.json

@@ -42,9 +42,12 @@
         "rollup": "^2.32.0",
         "typescript": "2.5.3",
         "qunit": "^2.9.1",
-        "sinon": "^7.3.1"
+        "sinon": "^7.3.1",
+        "globby": "^11.0.0",
+        "magic-string": "^0.25.7"
     },
     "dependencies": {
+        "@microsoft/applicationinsights-shims" : "2.0.0",
         "@microsoft/dynamicproto-js": "^1.1.1"
     }
 }

common/Tests/Framework/rollup.config.js

@@ -1,4 +1,5 @@
 import nodeResolve from "@rollup/plugin-node-resolve";
+import { updateDistEsmFiles } from "../../../tools/updateDistEsm/updateDistEsm";
 
 const version = require("./package.json").version;
 const outputName = "ai-test-framework";
@@ -29,6 +30,8 @@ const nodeUmdRollupConfigFactory = () => {
   return nodeRollupConfig;
 }
 
+updateDistEsmFiles({}, banner);
+
 export default [
   nodeUmdRollupConfigFactory(),
 ];

common/Tests/Framework/tsconfig.json

@@ -7,8 +7,9 @@
       "moduleResolution": "node",
       "target": "es5",
       "forceConsistentCasingInFileNames": true,
-      "importHelpers": false,
+      "importHelpers": true,
       "noEmitHelpers": true,
+      "skipLibCheck": true,
       "alwaysStrict": true,
       "declaration": true,
       "outDir": "dist-esm/",