microsoft / vscode-extension-telemetry

Node module to help VS Code extensions send telemetry using application insights
https://www.npmjs.com/package/@vscode/extension-telemetry
Other
125 stars 46 forks source link

Error trying to pack latest with web pack 4 #118

Closed brianignacio5 closed 1 year ago

brianignacio5 commented 2 years ago

Trying to use web pack 4 to pack 0.6.2 in vscode extension getting this error:

 ERROR in ./node_modules/@vscode/extension-telemetry/lib/telemetryReporter.node.min.js 1:36558
    Module parse failed: Unexpected token (1:36558)
    You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
    > var sS=Object.create;var Si=Object.defineProperty;var oS=Object.getOwnPropertyDescriptor;var uS=Object.getOwnPropertyNames,lp=Object.getOwnPropertySymbols,cS=Object.getPrototypeOf,pp=Object.prototype.hasOwnProperty,lS=Object.prototype.propertyIsEnumerable;var fp=(e,t,r)=>t in e?Si(e,t,{enumerable:!0,configurable:!0,writable:!0,value:r}):e[t]=r,At=(e,t)=>{for(var r in t||(t={}))pp.call(t,r)&&fp(e,r,t[r]);if(lp)for(var r of lp(t))lS.call(t,r)&&fp(e,r,t[r]);return e};var l=(e,t)=>()=>(t||e((t={exports:{}}).exports,t),t.exports),fS=(e,t)=>{for(var r in t)Si(e,r,{get:t[r],enumerable:!0})},dp=(e,t,r,n)=>{if(t&&typeof t=="object"||typeof t=="function")for(let i of uS(t))!pp.call(e,i)&&i!==r&&Si(e,i,{get:()=>t[i],enumerable:!(n=oS(t,i))||n.enumerable});return e};var Ai=(e,t,r)=>(r=e!=null?sS(cS(e)):{},dp(t||!e||!e.__esModule?Si(r,"default",{value:e,enumerable:!0}):r,e)),pS=e=>dp(Si({},"__esModule",{value:!0}),e);var ve=l((Kx,hp)=>{"use strict";var hS=function(){function e(){}return e.info=function(t){for(var r=[],n=1;n<arguments.length;n++)r[n-1]=arguments[n];e.enableDebug&&console.info(e.TAG+t,r)},e.warn=function(t){for(var r=[],n=1;n<arguments.length;n++)r[n-1]=arguments[n];e.disableWarnings||console.warn(e.TAG+t,r)},e.enableDebug=!1,e.disableWarnings=!1,e.disableErrors=!1,e.TAG="ApplicationInsights:",e}();hp.exports=hS});var gu=l(Ft=>{"use strict";var sn=Ft&&Ft.__assign||function(){return sn=Object.assign||function(e){for(var t,r=1,n=arguments.length;r<n;r++){t=arguments[r];for(var i in t)Object.prototype.hasOwnProperty.call(t,i)&&(e[i]=t[i])}return e},sn.apply(this,arguments)};Object.defineProperty(Ft,"__esModule",{value:!0});
......
license:"MIT"}});var om=l((m1,sm)=>{function RN(e){try{return Error.prototype.toString.call(e)}catch(t){try{return"<error: "+t+">"}catch{return"<error>"}}}sm.exports=function(t,r){var n=[];n.push(RN(t));for(var i=0;i<r.length;i++){var a=r[i],s;try{s=a.toString()}catch(o){try{s="<error: "+o+">"}catch{s="<error>"}}n.push("    at "+s)}return n.join(`
     @ ./src/telemetry/index.ts 18:0-60 28:37-54
     @ ./src/extension.ts

If I roll back to 0.4.8 the packing works.

lramos15 commented 2 years ago

Yeah I believe Webpack 5 is now required for this to work. That was the experience with our extensions. I'm not really super experienced in Webpack but if you have any ideas feel free to let me know

lramos15 commented 1 year ago

Once 0.7.x comes out of preview, we will no longer do bundling which will allow webpack 4 to work again.