Revert "fix(client): Make action.workflowId optional on Schedule update"

This reverts commit 1993731.

Author: mjameswh

packages/client/src/schedule-client.ts

@@ -149,6 +149,9 @@ function assertRequiredScheduleOptions(
       if (!opts.action.taskQueue) {
         throw new TypeError(`Missing ${structureName}.action.taskQueue for 'startWorkflow' action`);
       }
+      if (!opts.action.workflowId && action === 'UPDATE') {
+        throw new TypeError(`Missing ${structureName}.action.workflowId for 'startWorkflow' action`);
+      }
       if (!opts.action.workflowType) {
         throw new TypeError(`Missing ${structureName}.action.workflowType for 'startWorkflow' action`);
       }
@@ -456,11 +459,7 @@ export class ScheduleClient extends BaseClient {
         const currentHeader: Headers = current.raw.schedule?.action?.startWorkflow?.header?.fields ?? {};
         const updated = updateFn(current);
         assertRequiredScheduleOptions(updated, 'UPDATE');
-        await this.client._updateSchedule(
-          scheduleId,
-          compileUpdatedScheduleOptions(scheduleId, updated),
-          currentHeader
-        );
+        await this.client._updateSchedule(scheduleId, compileUpdatedScheduleOptions(updated), currentHeader);
       },
 
       async delete(): Promise<void> {

packages/client/src/schedule-helpers.ts

@@ -221,17 +221,13 @@ export function compileScheduleOptions(options: ScheduleOptions): CompiledSchedu
   };
 }
 
-export function compileUpdatedScheduleOptions(
-  scheduleId: string,
-  options: ScheduleUpdateOptions
-): CompiledScheduleUpdateOptions {
+export function compileUpdatedScheduleOptions(options: ScheduleUpdateOptions): CompiledScheduleUpdateOptions {
   const workflowTypeOrFunc = options.action.workflowType;
   const workflowType = extractWorkflowType(workflowTypeOrFunc);
   return {
     ...options,
     action: {
       ...options.action,
-      workflowId: options.action.workflowId ?? `${scheduleId}-workflow`,
       workflowType,
       args: (options.action.args ?? []) as unknown[],
     },

packages/client/src/schedule-types.ts

@@ -135,7 +135,13 @@ export type CompiledScheduleOptions = Replace<
 export type ScheduleUpdateOptions<A extends ScheduleOptionsAction = ScheduleOptionsAction> = Replace<
   Omit<ScheduleOptions, 'scheduleId' | 'memo' | 'searchAttributes'>,
   {
-    action: A;
+    action: Replace<
+      A,
+      {
+        // No default value on update
+        workflowId: string;
+      }
+    >;
     state: Omit<ScheduleOptions['state'], 'triggerImmediately' | 'backfill'>;
   }
 >;

packages/test/src/test-schedules.ts

@@ -396,6 +396,7 @@ if (RUN_INTEGRATION_TESTS) {
         ...x,
         action: {
           type: 'startWorkflow',
+          workflowId: `${scheduleId}-workflow-2`,
           workflowType: dummyWorkflowWith2Args,
           args: [3, 4],
           taskQueue,