Removed batchOptions from the trigger options and the docs

Author: matt-aitken

.changeset/smart-needles-move.md

@@ -0,0 +1,5 @@
+---
+"@trigger.dev/sdk": patch
+---
+
+Remove unimplemented batchOptions

docs/v3/triggering.mdx

@@ -168,15 +168,15 @@ This is where it gets interesting. You can trigger a task and then wait for the
 <Accordion title="Don't use this in parallel, e.g. with `Promise.all()`">
   Instead, use `batchTriggerAndWait()` if you can, or a for loop if you can't.
 
+  To control concurrency using batch triggers, you can set `queue.concurrencyLimit` on the child task.
+
 <CodeGroup>
 ```ts /trigger/batch.ts
 export const batchTask = task({
   id: "batch-task",
   run: async (payload: string) => {
     const results = await childTask.batchTriggerAndWait({
       items: [{ payload: "item1" }, { payload: "item2" }],
-      // optional: how many to run in parallel
-      batchOptions: { maxConcurrency: 1 },
     });
     console.log("Results", results);
 
@@ -189,6 +189,8 @@ export const batchTask = task({
 export const loopTask = task({
   id: "loop-task",
   run: async (payload: string) => {
+    //this will be slower than the batch version
+    //as we have to resume the parent after each iteration
     for (let i = 0; i < 2; i++) {
       const result = await childTask.triggerAndWait({ payload: `item${i}` });
       console.log("Result", result);
@@ -221,15 +223,15 @@ You can batch trigger a task and wait for all the results. This is useful for th
 <Accordion title="Don't use this in parallel, e.g. with `Promise.all()`">
   Instead, pass in all items at once and set an appropriate `maxConcurrency`. Alternatively, use sequentially with a for loop.
 
+  To control concurrency, you can set `queue.concurrencyLimit` on the child task.
+
 <CodeGroup>
 ```ts /trigger/batch.ts
 export const batchTask = task({
   id: "batch-task",
   run: async (payload: string) => {
     const results = await childTask.batchTriggerAndWait({
       items: [{ payload: "item1" }, { payload: "item2" }],
-      // optional: how many tasks to run in parallel
-      batchOptions: { maxConcurrency: 1 },
     });
     console.log("Results", results);
 
@@ -242,11 +244,11 @@ export const batchTask = task({
 export const loopTask = task({
   id: "loop-task",
   run: async (payload: string) => {
+    //this will be slower than a single batchTriggerAndWait()
+    //as we have to resume the parent after each iteration
     for (let i = 0; i < 2; i++) {
       const result = await childTask.batchTriggerAndWait({
         items: [{ payload: `itemA${i}` }, { payload: `itemB${i}` }],
-        // optional: how many tasks to run in parallel
-        batchOptions: { maxConcurrency: 1 },
       });
       console.log("Result", result);
 

packages/trigger-sdk/src/v3/shared.ts

@@ -179,12 +179,12 @@ export interface Task<TInput, TOutput = any> {
   trigger: (params: { payload: TInput; options?: TaskRunOptions }) => Promise<InvokeHandle>;
   batchTrigger: (params: {
     items: { payload: TInput; options?: TaskRunOptions }[];
-    batchOptions?: BatchRunOptions;
+    // batchOptions?: BatchRunOptions;
   }) => Promise<InvokeBatchHandle>;
   triggerAndWait: (params: { payload: TInput; options?: TaskRunOptions }) => Promise<TOutput>;
   batchTriggerAndWait: (params: {
     items: { payload: TInput; options?: TaskRunOptions }[];
-    batchOptions?: BatchRunOptions;
+    // batchOptions?: BatchRunOptions;
   }) => Promise<BatchResult<TOutput>>;
 }