feat(core): IEnvironmentAware interface to retrieve a construct's environment

[rix0rrr]

Previously, construct.env was declared in the IResource interface.

Downside is that IResource also:

• requires providing stack: Stack (which is a bit pointless to begin with since we can always call Stack.of(x)); and
• applyRemovalPolicy() which doesn't belong on this interface but now is required to implement.

Therefore, we extract IResource#env out to its own interface, IEnvironmentAware; make IXxxRef extend that interface, simplify IResourceWithPolicy to only be IEnvironmentAware and not an entire IResource.

Lessening the requirements on IResourceWithPolicy also lessens its guarantees, which makes that a breaking change. In order to make this code change, we do the following:

• Introduce IResourceWithPolicyV2 to be the version of IResourceWithPolicy we should have had.
• Accept IResourceWithPolicyV2 everywhere
• Make IResourceWithPolicy extend IResourcWithPolicyV2

                   ┌──────────────────────────┐                     
                   │    IEnvironmentAware     │                     
                   └──────────────────────────┘                     
                                 ▲                                  
             ┌───────────────────┴────────────────────┐             
             │                                        │             
┌──────────────────────────┐            ┌──────────────────────────┐
│  IResourceWithPolicyV2   │            │        IResource         │
└──────────────────────────┘            └──────────────────────────┘
              ▲                                       ▲             
              └──────────────────┬────────────────────┘             
                                 │                                  
                   ┌──────────────────────────┐                     
                   │   IResourceWithPolicy    │                     
                   └──────────────────────────┘                     

---

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)

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

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

[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]

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.