From 4c5623e068d2e6373fb8068f1ee566edd77e4df2 Mon Sep 17 00:00:00 2001
From: Hassan Azhar Khan <hassanazharkhan@gmail.com>
Date: Thu, 17 Dec 2020 21:21:33 +0500
Subject: [PATCH] Update README.md

---
 packages/@aws-cdk/aws-lambda-nodejs/README.md            | 4 ++++
 packages/@aws-cdk/aws-lambda-nodejs/lib/bundling.ts      | 9 ++++++---
 packages/@aws-cdk/aws-lambda-nodejs/lib/types.ts         | 2 +-
 .../@aws-cdk/aws-lambda-nodejs/test/bundling.test.ts     | 2 +-
 4 files changed, 12 insertions(+), 5 deletions(-)

diff --git a/packages/@aws-cdk/aws-lambda-nodejs/README.md b/packages/@aws-cdk/aws-lambda-nodejs/README.md
index aceb450596bb5..723ce157c6fbb 100644
--- a/packages/@aws-cdk/aws-lambda-nodejs/README.md
+++ b/packages/@aws-cdk/aws-lambda-nodejs/README.md
@@ -145,6 +145,10 @@ new lambda.NodejsFunction(this, 'my-handler', {
     },
     logLevel: LogLevel.SILENT, // defaults to LogLevel.WARNING
     keepNames: true, // defaults to false
+    tsconfig: 'custom-tsconfig.json' // use custom-tsconfig.json instead of default, 
+    metafile: true, // include meta file, defaults to false
+    banner : '/* comments */', // by default no comments are passed
+    footer : '/* comments */', // by default no comments are passed
   },
 });
 ```
diff --git a/packages/@aws-cdk/aws-lambda-nodejs/lib/bundling.ts b/packages/@aws-cdk/aws-lambda-nodejs/lib/bundling.ts
index 381fef9c304ad..137f641eb439a 100644
--- a/packages/@aws-cdk/aws-lambda-nodejs/lib/bundling.ts
+++ b/packages/@aws-cdk/aws-lambda-nodejs/lib/bundling.ts
@@ -64,7 +64,10 @@ export class Bundling implements cdk.BundlingOptions {
 
     const projectRoot = path.dirname(props.depsLockFilePath);
     this.relativeEntryPath = path.relative(projectRoot, path.resolve(props.entry));
-    this.relativeTsconfigPath = path.relative(projectRoot, path.resolve(props.tsconfig ?? '')); // setting empty string if tsconfig prop is not passed
+
+    if (props.tsconfig) {
+      this.relativeTsconfigPath = path.relative(projectRoot, path.resolve(props.tsconfig));
+    }
 
     this.externals = [
       ...props.externalModules ?? ['aws-sdk'], // Mark aws-sdk as external by default (available in the runtime)
@@ -142,8 +145,8 @@ export class Bundling implements cdk.BundlingOptions {
       ...loaders.map(([ext, name]) => `--loader:${ext}=${name}`),
       ...this.props.logLevel ? [`--log-level=${this.props.logLevel}`] : [],
       ...this.props.keepNames ? ['--keep-names'] : [],
-      ...this.props.tsconfig ? [`--tsconfig=${pathJoin(inputDir, this.relativeTsconfigPath!)}`] : [],
-      ...this.props.metaFile ? [`--metafile=${pathJoin(outputDir, 'index.meta.json')}`] : [],
+      ...this.relativeTsconfigPath ? [`--tsconfig=${pathJoin(inputDir, this.relativeTsconfigPath)}`] : [],
+      ...this.props.metafile ? [`--metafile=${pathJoin(outputDir, 'index.meta.json')}`] : [],
       ...this.props.banner ? [`--banner='${this.props.banner}'`] : [],
       ...this.props.footer ? [`--footer='${this.props.footer}'`] : [],
     ].join(' ');
diff --git a/packages/@aws-cdk/aws-lambda-nodejs/lib/types.ts b/packages/@aws-cdk/aws-lambda-nodejs/lib/types.ts
index 922e7ddb0934c..f1008dc7b0368 100644
--- a/packages/@aws-cdk/aws-lambda-nodejs/lib/types.ts
+++ b/packages/@aws-cdk/aws-lambda-nodejs/lib/types.ts
@@ -107,7 +107,7 @@ export interface BundlingOptions {
    * @see https://esbuild.github.io/api/#metafile
    * @default - false
    */
-  readonly metaFile?: boolean
+  readonly metafile?: boolean
 
   /**
    * Use this to insert an arbitrary string at the beginning of generated JavaScript files.
diff --git a/packages/@aws-cdk/aws-lambda-nodejs/test/bundling.test.ts b/packages/@aws-cdk/aws-lambda-nodejs/test/bundling.test.ts
index 78807b8f44e97..e3f6a07144b88 100644
--- a/packages/@aws-cdk/aws-lambda-nodejs/test/bundling.test.ts
+++ b/packages/@aws-cdk/aws-lambda-nodejs/test/bundling.test.ts
@@ -163,7 +163,7 @@ test('esbuild bundling with esbuild options', () => {
     logLevel: LogLevel.SILENT,
     keepNames: true,
     tsconfig,
-    metaFile: true,
+    metafile: true,
     banner: '/* comments */',
     footer: '/* comments */',
     forceDockerBundling: true,