fix(stepfunctions): allow JSONata expressions for Map maxConcurrency by mu7889yoon · Pull Request #36462 · aws/aws-cdk

mu7889yoon · 2025-12-19T15:47:03Z

Issue # (if applicable)

Reason for this change

In the current implementation, the Map state's maxConcurrency property only accepts numeric values.
However, according to the Amazon States Language specification, maxConcurrency can also be defined using a JSONata expression.

This limitation prevents users from dynamically controlling concurrency based on input data and causes a mismatch between CDK and the underlying Step Functions / CloudFormation capabilities.

This change updates the implementation to allow JSONata expressions for Map maxConcurrency, aligning CDK behavior with the Step Functions specification.

Description of changes

This PR updates the Map state to support JSONata expressions for the maxConcurrency property.

A new option is introduced to allow maxConcurrency to be specified as a JSONata expression, in addition to a static numeric value.
This option is mutually exclusive with the existing numeric maxConcurrency configuration.

During synthesis, when a JSONata expression is provided, it is rendered directly into the CloudFormation template as the Map state's maxConcurrency value.

An alternative approach, such as introducing a new class or restructuring the existing API, was considered.
However, this was avoided to prevent breaking changes for existing users.

Describe any new or updated permissions being added

None

Description of how you validated changes

Added unit test and an integration test

Checklist

My code adheres to the CONTRIBUTING GUIDE and DESIGN GUIDELINES

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

aws-cdk-automation

(This review is outdated)

badmintoncryer · 2025-12-19T16:21:45Z

@mu7889yoon Thank you for your contirbution. If you have any problem to execute integ test, please read this document (in Japanese!).

https://jaws-ug-cdk.github.io/cdk-conf-2024-contribute-workshop/

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

badmintoncryer

Thank you for your contribution! I've added some minor comments.

badmintoncryer

Thanks! I've added a tiny comment. After your modification, I'll approve this PR.

Co-authored-by: Kazuho Cryer-Shinozuka <malaysia.cryer@gmail.com>

badmintoncryer

Thanks!

mu7889yoon · 2025-12-20T12:22:08Z

Thanks!

Thank you for your review !!

badmintoncryer

Could you please merge the latest main branch? This resolves some kind of CI problems. It is better to merge it once a week or more.

github-actions · 2025-12-21T15:31:20Z

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
Please try merge from main to avoid findings unrelated to the PR.

	Tests	Passed ✅	Skipped	Failed
Security Guardian Results	48 ran	48 passed

Test	Result
No test annotations available

View Security Guardian Results

leonmk-aws · 2026-01-12T15:10:29Z

@Mergifyio refresh

mergify · 2026-01-12T15:11:01Z

refresh

✅ Pull request refreshed

mergify · 2026-01-12T15:11:10Z

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

mergify · 2026-01-12T15:11:53Z

mergify · 2026-01-12T22:33:13Z

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

github-actions · 2026-01-12T22:33:25Z

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

mu7889yoon added 3 commits December 20, 2025 00:44

add JSONata expression support for MaxConcurrency

dedd847

add README

743af02

add Testcase

163def4

aws-cdk-automation requested a review from a team December 19, 2025 15:47

github-actions Bot added beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK bug This issue is a bug. effort/medium Medium work item – several days of effort p2 labels Dec 19, 2025

aws-cdk-automation previously requested changes Dec 19, 2025

View reviewed changes

pahud mentioned this pull request Dec 19, 2025

CI Pending Approval - 43 PR(s) pahud/aws-cdk#11

Open

add integ test

6d774f2

aws-cdk-automation added the pr/reviewer-clarification-requested The contributor has requested clarification on feedback, a failing build, or a failing PR Linter run label Dec 19, 2025

aws-cdk-automation removed the pr/reviewer-clarification-requested The contributor has requested clarification on feedback, a failing build, or a failing PR Linter run label Dec 19, 2025

mu7889yoon marked this pull request as draft December 19, 2025 17:39

mu7889yoon marked this pull request as ready for review December 19, 2025 17:43

badmintoncryer suggested changes Dec 20, 2025

View reviewed changes

Comment thread ...-cdk-testing/framework-integ/test/aws-stepfunctions/test/integ.map-jsonata-maxconcurrency.ts Outdated

Comment thread packages/aws-cdk-lib/aws-stepfunctions/lib/states/map-base.ts Outdated

mu7889yoon added 3 commits December 20, 2025 15:30

fix integ test

3e08281

impl renderMaxConcurrency

c11745f

fix

63808e0

mu7889yoon requested a review from badmintoncryer December 20, 2025 07:23

badmintoncryer suggested changes Dec 20, 2025

View reviewed changes

Comment thread packages/aws-cdk-lib/aws-stepfunctions/lib/states/map-base.ts Outdated

Update packages/aws-cdk-lib/aws-stepfunctions/lib/states/map-base.ts

ad4dee7

Co-authored-by: Kazuho Cryer-Shinozuka <malaysia.cryer@gmail.com>

mu7889yoon requested review from badmintoncryer December 20, 2025 12:17

badmintoncryer approved these changes Dec 20, 2025

View reviewed changes

badmintoncryer approved these changes Dec 21, 2025

View reviewed changes

aws-cdk-automation added the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Dec 21, 2025

mu7889yoon added 2 commits December 24, 2025 13:08

Merge branch 'aws:main' into map-maxconcurrency-jsonata

e5aae55

Merge branch 'main' into map-maxconcurrency-jsonata

ae15d8b

pahud mentioned this pull request Dec 29, 2025

Failed CodeBuild CI - 16 PR(s) pahud/aws-cdk#17

Open

Merge branch 'main' into map-maxconcurrency-jsonata

5904ed6

github-actions Bot mentioned this pull request Jan 1, 2026

Monthly PR metrics report #36544

Open

mu7889yoon and others added 3 commits January 5, 2026 21:09

Merge branch 'main' into map-maxconcurrency-jsonata

5a9f4e4

Merge branch 'main' into map-maxconcurrency-jsonata

7b7938e

Merge branch 'main' into map-maxconcurrency-jsonata

c0b3613

gasolima added the pr/needs-integration-tests-deployment Requires the PR to deploy the integration test snapshots. label Jan 9, 2026

gasolima temporarily deployed to deployment-integ-test January 9, 2026 15:29 — with GitHub Actions Inactive

gasolima approved these changes Jan 9, 2026

View reviewed changes

aws-cdk-automation removed the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Jan 9, 2026

Merge branch 'main' into map-maxconcurrency-jsonata

8bf21fa

mu7889yoon temporarily deployed to deployment-integ-test January 10, 2026 08:15 — with GitHub Actions Inactive

gasolima self-requested a review January 12, 2026 12:49

gasolima approved these changes Jan 12, 2026

View reviewed changes

gasolima removed the pr/needs-integration-tests-deployment Requires the PR to deploy the integration test snapshots. label Jan 12, 2026

leonmk-aws approved these changes Jan 12, 2026

View reviewed changes

rix0rrr approved these changes Jan 12, 2026

View reviewed changes

gasolima removed the beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK label Jan 12, 2026

Merge branch 'main' into map-maxconcurrency-jsonata

505f842

mergify Bot merged commit 2230c87 into aws:main Jan 12, 2026
22 of 23 checks passed

github-actions Bot locked as resolved and limited conversation to collaborators Jan 12, 2026

Conversation

mu7889yoon commented Dec 19, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Issue # (if applicable)

Reason for this change

Description of changes

Describe any new or updated permissions being added

Description of how you validated changes

Checklist

Uh oh!

aws-cdk-automation left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

badmintoncryer commented Dec 19, 2025

Uh oh!

badmintoncryer left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

badmintoncryer left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

badmintoncryer left a comment

Choose a reason for hiding this comment

Uh oh!

mu7889yoon commented Dec 20, 2025

Uh oh!

badmintoncryer left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Dec 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

leonmk-aws commented Jan 12, 2026

Uh oh!

mergify Bot commented Jan 12, 2026

✅ Pull request refreshed

Uh oh!

mergify Bot commented Jan 12, 2026

Uh oh!

mergify Bot commented Jan 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Merge Queue Status

Uh oh!

mergify Bot commented Jan 12, 2026

Uh oh!

Uh oh!

github-actions Bot commented Jan 12, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

mu7889yoon commented Dec 19, 2025 •

edited

Loading

aws-cdk-automation left a comment •

edited

Loading

badmintoncryer left a comment •

edited

Loading

github-actions Bot commented Dec 21, 2025 •

edited

Loading

mergify Bot commented Jan 12, 2026 •

edited

Loading