fix: serverless deployment imageTag and force deployment, using extraHash.

[zMynxx]

PR Type

Bug fix, Enhancement

---

Description

• Add extraHash parameter to force CDK deployment when imageTag changes

• Ensures Docker image rebuild triggers on imageTag updates

• Prevents stale image deployments in serverless Lambda stack

---

Diagram Walkthrough

flowchart LR
  imageTag["imageTag property"] -- "triggers rebuild via" --> extraHash["extraHash parameter"]
  extraHash -- "forces" --> dockerAsset["DockerImageAsset rebuild"]
  dockerAsset -- "ensures" --> deployment["Fresh Lambda deployment"]

Loading

File Walkthrough

	Relevant files
Bug fix	cdk-app-stack.ts Add extraHash to force Docker image rebuild                            serverless/cdk-app/lib/cdk-app-stack.ts Added extraHash property to DockerImageAsset configuration Set extraHash value to props?.imageTag || "latest-lambda" to match buildArgs Includes comment explaining the force rebuild behavior on imageTag changes +1/-0

---

[Copilot]

Pull Request Overview

This PR fixes serverless deployment issues by ensuring Lambda functions are properly redeployed when the image tag changes. The solution adds an extraHash parameter that forces AWS CDK to recognize changes in the image tag and trigger redeployment.

• Adds extraHash parameter to force Lambda function redeployment when image tag changes

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[codiumai-pr-agent-free]

PR Compliance Guide 🔍

Below is a summary of compliance checks for this PR:

Security Compliance
🟢	No security concerns identified No security vulnerabilities detected by AI analysis. Human verification advised for critical code.
Ticket Compliance
⚪	🎫 No ticket provided - [ ] Create ticket/issue <!-- /create_ticket --create_ticket=true --> </details></td></tr>
Codebase Duplication Compliance
⚪	Codebase context is not defined Follow the guide to enable codebase context checks.
Custom Compliance
⚪	No custom compliance provided Follow the guide to enable custom compliance check.

Compliance status legend

🟢 - Fully Compliant
🟡 - Partial Compliant
🔴 - Not Compliant
⚪ - Requires Further Human Verification
🏷️ - Compliance label

[github-actions]

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines	Covered	Coverage	Threshold	Status
170	154	91%	0%	🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: b6b42a9 by action🐍

[github-actions]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
aws-lambda-calculator/src/aws_lambda_calculator
calculator.py
aws-lambda-calculator/tests
test_calculator.py
test_pytest_generate_tests_sample_code.py
Project Total

This report was generated by python-coverage-comment-action

[codiumai-pr-agent-free]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Use nullish coalescing for safer defaults Replace the logical OR operator (||) with the nullish coalescing operator (??) for setting the extraHash default value to handle potential falsy values for imageTag correctly. serverless/cdk-app/lib/cdk-app-stack.ts [27] -extraHash: props?.imageTag || "latest-lambda", // Force rebuild when imageTag changes +extraHash: props?.imageTag ?? "latest-lambda", // Force rebuild when imageTag changes Apply / Chat Suggestion importance[1-10]: 4 __ Why: The suggestion correctly identifies that using the nullish coalescing operator (??) is more robust than the logical OR (||) for providing a default, as it avoids incorrectly overriding falsy but potentially valid values like an empty string.	Low
More

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud