feat: property injection for 122 constructs

[pcheungamz]

Issue # (if applicable)

This Implements the Property Injectors RFC. https://github.com/aws/aws-cdk-rfcs/blob/main/text/0693-property-injection.md

Reason for this change

Implement the Property Injectors feature so orgs can set up default Construct props values.

Description of changes

Key changes:

• propertyInjectionDecorator to make a Construct Property Injectable.
• applyInjectors is called from Construct's constructor to inject property defaults.
• IPropertyInjector defines an Property Injector.
• PropertyInjectors class that stores the map of Constructs to Injectors.
• App, Stage, and Stack now has a new propertyInjectors property.

Describe any new or updated permissions being added

None

Description of how you validated changes

• Unit test in core/test/prop-injectors.test.ts
• Use yarn package --target js to create a library and write Injectors for Constructs.

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)

[pcheungamz]

This is a preliminary PR to get early review on core/lib/prop-injectors.ts and core/test/prop-injectors.test.ts.

TODOs:

• remove debug logging.
• enable property injection on more Constructs

[aws-cdk-automation]

This PR cannot be merged because it has conflicts. Please resolve them. The PR will be considered stale and closed if it remains in an unmergeable state.

[aws-cdk-automation]

This PR cannot be merged because it has conflicts. Please resolve them. The PR will be considered stale and closed if it remains in an unmergeable state.

[aws-cdk-automation]

This PR cannot be merged because it has conflicts. Please resolve them. The PR will be considered stale and closed if it remains in an unmergeable state.

[rix0rrr]

Exciting!

A lot of work in here applying these changes to all of those constructs, thanks!

[moelasmar]

thanks @pcheungamz for implementing this feature .. I have some questions/comments about the L2 updates you did
1- Did you update all L2 constructs, and if yes, I assume you did not do it manually, if also yes, can I ask to push this script to be used as a GH action, so for future L2s, this change will be automatically applied.
2- Can you update the aws-cdk-lib/Readme file to add a new section explaining how this feature should be used. I believe we will also need some documentation updates to the CDK Docs
3- Can you add some integration test cases to https://github.com/aws/aws-cdk/tree/main/packages/%40aws-cdk-testing/framework-integ/test/core/test

[aws-cdk-automation]

This PR cannot be merged because it has conflicts. Please resolve them. The PR will be considered stale and closed if it remains in an unmergeable state.

[aws-cdk-automation]

This PR cannot be merged because it has conflicts. Please resolve them. The PR will be considered stale and closed if it remains in an unmergeable state.

[aws-cdk-automation]

This PR cannot be merged because it has conflicts. Please resolve them. The PR will be considered stale and closed if it remains in an unmergeable state.

[pcheungamz]

This PR will make all L2 Constructs property injectable in the future: #34328. But it shouldn't block this PR.

[rix0rrr]

Shipped then!

[mergify]

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]

This pull request has been removed from the queue for the following reason: checks failed.

The merge conditions cannot be satisfied due to failing checks:

You may have to fix your CI before adding the pull request to the queue again.
If you update this pull request, to fix the CI, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio requeue comment.

Pull request has been modified.

[mergify]

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).

[aws-cdk-automation]

AWS CodeBuild CI Report

• CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
• Commit ID: fa0d021
• Result: SUCCEEDED
• Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

[mergify]

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]

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.