fix(schedule): expose other fields when id optional (#451)

This PR appears to follow-up on
#276 which made it so `id`
was optional ✅ but did so in a way that removed the remainder of the
required fields from the type definition ❌, e.g. `actorTaskId` was no
longer recognized as a valid property as of v2.8.2

Because `ScheduleAction` is defined as a union of different types, we
must use [distributive conditional
types](https://www.typescriptlang.org/docs/handbook/2/conditional-types.html#distributive-conditional-types)
to capture the specific properties of the individual subtypes correctly.

src/resource_clients/schedule.ts

@@ -9,7 +9,7 @@ import {
     parseDateFields,
     catchNotFoundOrThrow,
     cast,
-    MakeOptional,
+    DistributiveOptional,
 } from '../utils';
 
 export class ScheduleClient extends ResourceClient {
@@ -93,7 +93,7 @@ export type ScheduleCreateOrUpdateData = Partial<
         | 'isExclusive'
         | 'description'
     > & {
-        actions: MakeOptional<ScheduleAction, 'id'>[]
+        actions: DistributiveOptional<ScheduleAction, 'id'>[]
     }
 >;
 

src/utils.ts

@@ -201,4 +201,4 @@ export function cast<T>(input: unknown): T {
 
 export type Dictionary<T = unknown> = Record<PropertyKey, T>;
 
-export type MakeOptional<T, U extends keyof T> = Pick<T, Exclude<keyof T, U>> & Partial<Pick<T, U>>;
+export type DistributiveOptional<T, K extends keyof T> = T extends any ? Omit<T, K> & Partial<Pick<T, K>> : never;