I actually have a few issues with this section:

• the sentence should mention how, i.e. by using the fromCfnMonitorsProperty method.
• however, I am not a fan of this approach at all. am I missing some context about why this method was added in the first place? CDK doesn't typically encourage this type of L1 method on L2 so I want to dig a bit deeper on why its necessary at all
• finally, looking at the code, it seems like the issue with fromCfnMonitorsProperty is that it will fail entirely if a role is not provided, likely in a confusing manner (because errors due to ! don't tend to be very descriptive). Looking at this code below:

  public static fromCfnMonitorsProperty(monitorsProperty: CfnEnvironment.MonitorsProperty): Monitor {
    return {
      alarmArn: monitorsProperty.alarmArn!,
      alarmRoleArn: monitorsProperty.alarmRoleArn,
      monitorType: MonitorType.CFN_MONITORS_PROPERTY,
    };
  }

I think, if we have a reason to keep this method, it should be updated to (to avoid ! entirely):

  public static fromCfnMonitorsProperty(monitorsProperty: CfnEnvironment.MonitorsProperty): Monitor {
    if (monitorsProperty.alarmArn === undefined) { throw new Error('need this prop'); }
    return {
      alarmArn: monitorsProperty.alarmArn,
      alarmRoleArn: monitorsProperty.alarmRoleArn,
      monitorType: MonitorType.CFN_MONITORS_PROPERTY,
    };
  }

updated with suggested changes

sorry! we need a better error message than this. This was a placeholder :)

Oops! Updated