Fix SDK bug in the deployTrigger method (#16934)

jverce · web-flow · commit 2799a704c528 · 2025-06-02T14:41:26.000-07:00
The `deployTrigger` method was ignoring the `workflowId` argument
when calling the API, which prevented the new trigger from emitting
events to the target workflow.
diff --git a/packages/sdk/CHANGELOG.md b/packages/sdk/CHANGELOG.md
@@ -2,6 +2,12 @@
 
 # Changelog
 
+## [1.6.5] - 2025-06-02
+
+### Changed
+
+- Fix the `deployTrigger` method so that it sends the workflow ID to the API
+
 ## [1.6.4] - 2025-05-30
 
 ### Added
diff --git a/packages/sdk/package.json b/packages/sdk/package.json
@@ -1,7 +1,7 @@
 {
   "name": "@pipedream/sdk",
   "type": "module",
-  "version": "1.6.4",
+  "version": "1.6.5",
   "description": "Pipedream SDK",
   "main": "./dist/server.js",
   "module": "./dist/server.js",
diff --git a/packages/sdk/src/shared/index.ts b/packages/sdk/src/shared/index.ts
@@ -1481,6 +1481,7 @@ export abstract class BaseClient {
       id,
       configured_props: opts.configuredProps,
       dynamic_props_id: opts.dynamicPropsId,
+      workflow_id: opts.workflowId,
       webhook_url: opts.webhookUrl,
     };
     return this.makeConnectRequest<DeployTriggerResponse>("/triggers/deploy", {
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml

Original file line number	Diff line number	Diff line change
`@@ -1,7 +1,7 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "@pipedream/sdk",`
`3`	`3`	`"type": "module",`
`4`		`- "version": "1.6.4",`
	`4`	`+ "version": "1.6.5",`
`5`	`5`	`"description": "Pipedream SDK",`
`6`	`6`	`"main": "./dist/server.js",`
`7`	`7`	`"module": "./dist/server.js",`