From b0e0353c63b9b3b40f6204e04ba97ab18352bec8 Mon Sep 17 00:00:00 2001
From: Vinayak Kukreja <78971045+vinayak-kukreja@users.noreply.github.com>
Date: Thu, 11 Apr 2024 10:18:15 -0700
Subject: [PATCH] chore(step-function-tasks): add comment that `aws-sdk`
 integration pattern is not impacted by sdk upgrade (#29795)

### Reason for this change

For upgrading sdkv2 to sdkv3, I have confirmed that the [pattern](https://docs.aws.amazon.com/step-functions/latest/dg/supported-services-awssdk.html) used by step function task is not impacted by the migration and will remain the same.

Adding a comment to add historical context for anyone revisiting this package for another sdk upgrade.

### Description of changes

Just added comments. No code changes.

### Description of how you validated changes

No tests added for comments

### Checklist
- [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md)

----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
---
 .../aws-stepfunctions-tasks/lib/aws-sdk/call-aws-service.ts    | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/packages/aws-cdk-lib/aws-stepfunctions-tasks/lib/aws-sdk/call-aws-service.ts b/packages/aws-cdk-lib/aws-stepfunctions-tasks/lib/aws-sdk/call-aws-service.ts
index 4ce5714ce1181..f36f49fe501f7 100644
--- a/packages/aws-cdk-lib/aws-stepfunctions-tasks/lib/aws-sdk/call-aws-service.ts
+++ b/packages/aws-cdk-lib/aws-stepfunctions-tasks/lib/aws-sdk/call-aws-service.ts
@@ -118,6 +118,9 @@ export class CallAwsService extends sfn.TaskStateBase {
       service = service.toLowerCase();
     }
 
+    // The pattern here is: "arn:aws:states:::aws-sdk:serviceName:apiAction.[serviceIntegrationPattern]"
+    // See here: https://docs.aws.amazon.com/step-functions/latest/dg/supported-services-awssdk.html
+    // This does not change with sdk upgrades, TT:P125388388
     return {
       Resource: integrationResourceArn(
         'aws-sdk',