How do I get SST to build with proper sourcemaps? ...
# sstd
Danny
10/27/2021, 11:41 AMHow do I get SST to build with proper sourcemaps? My stacktrace in seed.run issues are minified.
Danny
10/27/2021, 11:47 AMindex.js and index.js.map are in the Lambda
Danny
10/27/2021, 11:48 AMvisible in AWS console
t
Thomas Ankcorn
10/27/2021, 11:48 AMHaven’t personally tried it but I think you need to add this env
Thomas Ankcorn
10/27/2021, 11:48 AM Copy code
NODE_OPTIONS=--enable-source-mapsd
Danny
10/27/2021, 11:49 AMthanks, so it’s not on by default?
Danny
10/27/2021, 11:49 AMI’ll try adding this to the deployed Lambda env vars
t
Thomas Ankcorn
10/27/2021, 11:49 AMI have no idea 🤷 lmk what you find!
t
thdxr
10/27/2021, 12:58 PMI haven't actually looked into sourceMaps yet
thdxr
10/27/2021, 12:58 PMI know some people were using a library
d
Danny
10/27/2021, 12:59 PMdo I have to add the npm package and that’s it?
Danny
10/27/2021, 12:59 PMwait already have
"source-map-support"t
thdxr
10/27/2021, 1:03 PMI think it needs to be required or something. I haven't used it yet
t
Thomas Ankcorn
10/27/2021, 1:22 PMI thought that was only for pre node 12
t
thdxr
10/27/2021, 1:34 PMI'm totally inexperienced so I'm probably wrong
t
Thomas Ankcorn
10/27/2021, 1:44 PMThomas Ankcorn
10/27/2021, 1:45 PMlooks like both would work, don’t know which is best though
d
Danny
10/28/2021, 3:53 PMI set
NODE_OPTIONS: '--enable-source-maps',environmentt
Thomas Ankcorn
10/28/2021, 3:55 PMNice! Any downsides of doing this? Like a performance hit. If not @thdxr would it be possible to make this an sst default?
t
thdxr
10/28/2021, 4:12 PMI've actually been somewhat confused by this
thdxr
10/28/2021, 4:12 PMWhat's the point of minifying if we ship the source map anyway
t
Thomas Ankcorn
10/28/2021, 4:13 PMthe source map helps because the code is bundled right?
d
Danny
10/28/2021, 4:13 PMbundling
Danny
10/28/2021, 4:13 PMand ts => js
t
thdxr
10/28/2021, 4:13 PMI guess it's also transpiled
thdxr
10/28/2021, 4:13 PMYeah
thdxr
10/28/2021, 4:14 PMOk that makes sense, it's more the minification I'm confused about
thdxr
10/28/2021, 4:14 PMIs it worth minifying if we're gonna ship the source map
d
Danny
10/28/2021, 4:14 PMstartup time?
t
thdxr
10/28/2021, 4:15 PMThe startup time is the function downloading the whole code bundle. So if you ship the sourcemap alongside you're losing the benefits
thdxr
10/28/2021, 4:15 PMThere may be some minor parsing benefits but can't imagine it's huge
d
Danny
10/28/2021, 4:15 PMand parsing and jitting
Danny
10/28/2021, 4:15 PMyea tiny but on lower ram lambdas maybe helps
t
thdxr
10/28/2021, 4:16 PMI guess we can leave it and as long as we're defaulting to enabling sourceMaps no one will have to be exposed to that detail
thdxr
10/28/2021, 4:18 PMApparently it does come with performance impacts, need to do some research
thdxr
10/28/2021, 4:19 PMSeeing some comments that startup time is slowed