Function: Don't overwrite Lambda description for Live Lambda developm…

…ent, just suffix label (#3674) * fix: don't overwrite Function description if provided, but always suffix Live Lambda label * update comment * add trim * add changeset * Sync --------- Co-authored-by: Frank <frank@sst.dev>
sst · Mar 1, 2024 · 602ddbf · 602ddbf
1 parent d3f0a3a
commit 602ddbf
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/.changeset/swift-chicken-float.md b/.changeset/swift-chicken-float.md
@@ -0,0 +1,5 @@
+---
+"sst": patch
+---
+
+Function: don't overwrite description, but add Live Lambda label
diff --git a/packages/sst/src/constructs/Function.ts b/packages/sst/src/constructs/Function.ts
@@ -947,11 +947,16 @@ export class Function extends CDKFunction implements SSTConstruct {
         };
       }
 
+      // Ensure descriptions fits the 256 chars limit
+      const description = props.description
+        ? `${props.description.substring(0, 240)} (live)`
+        : `live`;
+
       super(scope, id, {
         ...props,
         ...(props.runtime === "container"
           ? {
-              description: "SST Live Lambda handler",
+              description,
               code: Code.fromAssetImage(
                 path.resolve(__dirname, "../support/bridge"),
                 {
@@ -965,7 +970,7 @@ export class Function extends CDKFunction implements SSTConstruct {
               layers: undefined,
             }
           : {
-              description: "SST Live Lambda handler",
+              description,
               runtime: CDKRuntime.NODEJS_18_X,
               code: Code.fromAsset(
                 path.resolve(__dirname, "../support/bridge")