diff --git a/changelog/pending/20221017--sdk-nodejs--fix-ts-all-types.yaml b/changelog/pending/20221017--sdk-nodejs--fix-ts-all-types.yaml new file mode 100644 index 000000000000..b1ddaab2614e --- /dev/null +++ b/changelog/pending/20221017--sdk-nodejs--fix-ts-all-types.yaml @@ -0,0 +1,4 @@ +changes: +- type: fix + scope: sdk/nodejs + description: Fixes loss of undefined type case in `all()` diff --git a/sdk/nodejs/output.ts b/sdk/nodejs/output.ts index 87908852b7a3..c73540fb5162 100644 --- a/sdk/nodejs/output.ts +++ b/sdk/nodejs/output.ts @@ -577,14 +577,14 @@ function createSimpleOutput(val: any) { */ /* eslint-disable max-len */ export function all(val: Record>): Output>>; -export function all(values: [Input | undefined, Input | undefined, Input | undefined, Input | undefined, Input | undefined, Input | undefined, Input | undefined, Input | undefined]): Output<[Unwrap, Unwrap, Unwrap, Unwrap, Unwrap, Unwrap, Unwrap, Unwrap]>; -export function all(values: [Input | undefined, Input | undefined, Input | undefined, Input | undefined, Input | undefined, Input | undefined, Input | undefined]): Output<[Unwrap, Unwrap, Unwrap, Unwrap, Unwrap, Unwrap, Unwrap]>; -export function all(values: [Input | undefined, Input | undefined, Input | undefined, Input | undefined, Input | undefined, Input | undefined]): Output<[Unwrap, Unwrap, Unwrap, Unwrap, Unwrap, Unwrap]>; -export function all(values: [Input | undefined, Input | undefined, Input | undefined, Input | undefined, Input | undefined]): Output<[Unwrap, Unwrap, Unwrap, Unwrap, Unwrap]>; -export function all(values: [Input | undefined, Input | undefined, Input | undefined, Input | undefined]): Output<[Unwrap, Unwrap, Unwrap, Unwrap]>; -export function all(values: [Input | undefined, Input | undefined, Input | undefined]): Output<[Unwrap, Unwrap, Unwrap]>; -export function all(values: [Input | undefined, Input | undefined]): Output<[Unwrap, Unwrap]>; -export function all(ds: (Input | undefined)[]): Output[]>; +export function all(values: [Input, Input, Input, Input, Input, Input, Input, Input]): Output<[Unwrap, Unwrap, Unwrap, Unwrap, Unwrap, Unwrap, Unwrap, Unwrap]>; +export function all(values: [Input, Input, Input, Input, Input, Input, Input]): Output<[Unwrap, Unwrap, Unwrap, Unwrap, Unwrap, Unwrap, Unwrap]>; +export function all(values: [Input, Input, Input, Input, Input, Input]): Output<[Unwrap, Unwrap, Unwrap, Unwrap, Unwrap, Unwrap]>; +export function all(values: [Input, Input, Input, Input, Input]): Output<[Unwrap, Unwrap, Unwrap, Unwrap, Unwrap]>; +export function all(values: [Input, Input, Input, Input]): Output<[Unwrap, Unwrap, Unwrap, Unwrap]>; +export function all(values: [Input, Input, Input]): Output<[Unwrap, Unwrap, Unwrap]>; +export function all(values: [Input, Input]): Output<[Unwrap, Unwrap]>; +export function all(ds: (Input)[]): Output[]>; export function all(val: Input[] | Record>): Output { // Our recursive `output` helper already does exactly what `all` needs to do in terms of the // implementation. Why have both `output` and `all` then? Currently, to the best of our