Merge pull request #2770 from lpsinger/sourcemaps

Enable sourcemaps for legible stack traces in lambdas

app/email-incoming/circulars/index.ts

@@ -6,6 +6,7 @@
  * SPDX-License-Identifier: Apache-2.0
  */
 import { tables } from '@architect/functions'
+import 'source-map-support/register'
 
 import {
   bodyIsValid,

app/email-incoming/support/index.ts

@@ -11,6 +11,7 @@ import {
   transformRecipients,
   /* @ts-expect-error: aws-lambda-ses-forwarder does not have type definitions */
 } from 'aws-lambda-ses-forwarder'
+import 'source-map-support/register'
 
 import { createEmailIncomingMessageHandler } from '../handler'
 import { getEnvOrDie, hostname } from '~/lib/env.server'

app/scheduled/ads/index.ts

@@ -7,6 +7,7 @@
  */
 import { tables } from '@architect/functions'
 import { ConditionalCheckFailedException } from '@aws-sdk/client-dynamodb'
+import 'source-map-support/register'
 
 import { getEnvOrDie } from '~/lib/env.server'
 

app/scheduled/circulars/index.ts

@@ -5,6 +5,8 @@
  *
  * SPDX-License-Identifier: Apache-2.0
  */
+import 'source-map-support/register'
+
 import { forAllCirculars } from './actions'
 import { sitemapAction } from './actions/sitemap'
 import { statsAction } from './actions/stats'

app/table-streams/circulars/index.ts

@@ -8,6 +8,7 @@
 import { search as getSearchClient } from '@nasa-gcn/architect-functions-search'
 import { errors } from '@opensearch-project/opensearch'
 import type { DynamoDBRecord } from 'aws-lambda'
+import 'source-map-support/register'
 
 import { unmarshallTrigger } from '../utils'
 import { send as sendKafka } from '~/lib/kafka.server'

app/table-streams/synonyms/index.ts

@@ -8,6 +8,7 @@
 import { search as getSearchClient } from '@nasa-gcn/architect-functions-search'
 import { errors } from '@opensearch-project/opensearch'
 import type { DynamoDBRecord } from 'aws-lambda'
+import 'source-map-support/register'
 
 import { unmarshallTrigger } from '../utils'
 import { createTriggerHandler } from '~/lib/lambdaTrigger.server'

esbuild.js

@@ -24,7 +24,8 @@ const options = {
   platform: 'node',
   target: ['node20'],
   minify: !dev,
-  sourcemap: dev,
+  sourcemap: true,
+  sourcesContent: false,
   metafile: true,
   loader: { '.node': 'empty' },
   plugins: [

package.json

@@ -9,7 +9,7 @@
   },
   "license": "Apache-2.0",
   "scripts": {
-    "build:remix": "remix build && mv build/server/metafile.* build/",
+    "build:remix": "remix build --sourcemap && mv build/server/metafile.* build/",
     "build:sass": "sass -Inode_modules/nasawds/src/theme -Inode_modules/@uswds -Inode_modules/@uswds/uswds/packages app:app",
     "build:esbuild": "node esbuild.js",
     "build:website": "run-s build:sass build:remix",

server.ts

@@ -2,9 +2,8 @@ import { type RequestHandler, createRequestHandler } from '@remix-run/architect'
 import * as build from '@remix-run/dev/server-build'
 import { installGlobals } from '@remix-run/node'
 import { type APIGatewayProxyStructuredResultV2 } from 'aws-lambda'
-import sourceMapSupport from 'source-map-support'
+import 'source-map-support/register'
 
-sourceMapSupport.install()
 installGlobals({ nativeFetch: true })
 
 const remixHandler = createRequestHandler({