chore(spec2cdk): refactor to use hand-written helpers for arrays

[leonmk-aws]

Reason for this change

The previously generated code for getting the values of the IxxRef in arrays was not readable, as a for loop to mutate the array in place and comma syntax were used to return the mutated array in a single line.
Before:

cdk.withResolved(props.${name}, (props.xx?.forEach((item: T | string, i: number, arr: Array<T | string>) => { arr[i] = (item as T)?.xxxRef?.xx ?? cdk.ensureStringOrUndefined(item, "xxx", "lambda.T | string"); }), props.xxx as Array<string>));

After:

cdk.mapArrayInPlace(props.layers, (item: IxxxRef | string) => (item as IxxxRef)?.xxxRef?.xxxArn ?? cdk.ensureStringOrUndefined(item, "xxx", "lambda.IxxxRef | string"))

Description of changes

Added a handwritten function to do the in place in mapping in a generic way, preserving type safety.

Description of how you validated changes

• This is a refactoring, unit tests for this are already in:

  aws-cdk/packages/aws-cdk-lib/aws-lambda/test/function.test.ts

  Line 5246 in 3e048c1

  it('array reference should be valid', () => {

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

[rix0rrr]

Oof. This could do with a helper couldn't it?