chore: addressed code review feedback

Author: rubenfonseca

layer/Python/Dockerfile

@@ -10,7 +10,7 @@ WORKDIR /tmp
 # PACKAGE_SUFFIX = '=='2.0.0'
 # PACKAGE_SUFFIX = ''
 
-RUN yum update -y && yum install -y zip unzip wget tar gzip
+RUN yum update -y && yum install -y zip unzip wget tar gzip binutils
 
 RUN pip install -t /asset/python aws-lambda-powertools$PACKAGE_SUFFIX
 
@@ -23,8 +23,4 @@ RUN cd /asset && \
   # remove tests
   find python -wholename "*/tests/*" -type f -delete && \
   # remove python bytecode
-  find python -regex '^.*\(__pycache__\|\.py[co]\)$' -delete && \
-  # zip layer
-  zip -r9 aws-lambda-powertools.zip ./python && \
-  # delete original
-  rm -rf ./python
+  find python -regex '^.*\(__pycache__\|\.py[co]\)$' -delete

src/lambda-powertools-layer.ts

@@ -95,8 +95,8 @@ export class LambdaPowertoolsLayer extends lambda.LayerVersion {
       license: 'MIT-0',
       compatibleRuntimes: getRuntimesFromRuntimeFamily(runtimeFamily),
       compatibleArchitectures,
-      description: `Lambda Powertools for ${languageName}${
-        props?.includeExtras ? ' with Extras' : ''
+      description: `Lambda Powertools for ${languageName} [${compatibleArchitectures.map(x => x.name).join(', ')}]${
+        props?.includeExtras ? ' with extra dependencies' : ''
       } ${props?.version ? `version ${props?.version}` : 'latest version'}`.trim(),
     });
   }

test/lambda-powertools-python-layer.test.ts

@@ -10,7 +10,7 @@ describe('with no configuration the construct', () => {
   const template = Template.fromStack(stack);
   test('synthesizes successfully', () => {
     template.hasResourceProperties('AWS::Lambda::LayerVersion', {
-      Description: 'Lambda Powertools for Python latest version',
+      Description: 'Lambda Powertools for Python [x86_64] latest version',
     });
   });
 
@@ -40,7 +40,7 @@ describe('with arm64 architecture', () => {
   const template = Template.fromStack(stack);
   test('synthesizes successfully', () => {
     template.hasResourceProperties('AWS::Lambda::LayerVersion', {
-      Description: 'Lambda Powertools for Python latest version',
+      Description: 'Lambda Powertools for Python [arm64] latest version',
       CompatibleArchitectures: ['arm64'],
     });
   });
@@ -68,7 +68,7 @@ describe('with version configuration the construct', () => {
 
 
     Template.fromStack(stack).hasResourceProperties('AWS::Lambda::LayerVersion', {
-      Description: 'Lambda Powertools for Python version 1.21.0',
+      Description: 'Lambda Powertools for Python [x86_64] version 1.21.0',
     });
   });
 
@@ -87,7 +87,7 @@ describe('with version configuration the construct', () => {
     });
 
     Template.fromStack(stack).hasResourceProperties('AWS::Lambda::LayerVersion', {
-      Description: 'Lambda Powertools for Python with Extras version 1.22.0',
+      Description: 'Lambda Powertools for Python [x86_64] with extra dependencies version 1.22.0',
     });
 
   });
@@ -99,7 +99,7 @@ describe('with version configuration the construct', () => {
     });
 
     Template.fromStack(stack).hasResourceProperties('AWS::Lambda::LayerVersion', {
-      Description: 'Lambda Powertools for Python with Extras latest version',
+      Description: 'Lambda Powertools for Python [x86_64] with extra dependencies latest version',
     });
   });
 });

test/lambda-powertools-typescript-layer.test.ts

@@ -12,7 +12,7 @@ describe('with minimal configuration the construct', () => {
   const template = Template.fromStack(stack);
   test('synthesizes successfully', () => {
     template.hasResourceProperties('AWS::Lambda::LayerVersion', {
-      Description: 'Lambda Powertools for TypeScript latest version',
+      Description: 'Lambda Powertools for TypeScript [x86_64] latest version',
     });
   });
 
@@ -57,7 +57,7 @@ describe('with version configuration the construct', () => {
 
 
     Template.fromStack(stack).hasResourceProperties('AWS::Lambda::LayerVersion', {
-      Description: `Lambda Powertools for TypeScript version ${version}`,
+      Description: `Lambda Powertools for TypeScript [x86_64] version ${version}`,
     });
   });