@tailor-platform/sdk 0.16.0 → 0.16.1

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @tailor-platform/sdk
 
+## 0.16.1
+
+### Patch Changes
+
+- [#160](https://github.com/tailor-platform/sdk/pull/160) [`1406523`](https://github.com/tailor-platform/sdk/commit/14065237e5f0b05cf898c0fff196e1eb599fb96f) Thanks [@toiroakr](https://github.com/toiroakr)! - fix: correctly determine create/update for workflow job functions
+
+  Previously, the SDK used `hasExistingWorkflows` (based on workflow updates) to decide whether to use `createWorkflowJobFunction` or `updateWorkflowJobFunction`. This caused errors when renaming job functions, as renamed jobs were incorrectly sent to the update API which requires the job to already exist.
+
+  Now the SDK fetches the actual list of existing job function names via `listWorkflowJobFunctions` API and correctly uses:
+  - `createWorkflowJobFunction` for new job names (including renamed jobs)
+  - `updateWorkflowJobFunction` for existing job names
+
 ## 0.16.0
 
 ### Minor Changes

package/README.md

@@ -10,3 +10,4 @@ You can also view the documentation on [our website](https://docs.tailor.tech/re
 - [App Configuration](./docs/configuration.md)
 - [Core Concepts](./docs/core-concepts.md)
 - [CLI Reference](./docs/cli-reference.md)
+- [Testing Guide](./docs/testing.md)

package/dist/cli/api.mjs

@@ -1,4 +1,4 @@
-import { apply, generate, generateUserTypes, loadAccessToken, loadConfig, loadWorkspaceId, machineUserList, machineUserToken, oauth2ClientGet, oauth2ClientList, remove, show, workflowExecutionGet, workflowExecutionsList, workflowGet, workflowList, workflowResume, workflowStart, workspaceCreate, workspaceDelete, workspaceList } from "../resume-ar5nEitS.mjs";
+import { apply, generate, generateUserTypes, loadAccessToken, loadConfig, loadWorkspaceId, machineUserList, machineUserToken, oauth2ClientGet, oauth2ClientList, remove, show, workflowExecutionGet, workflowExecutionsList, workflowGet, workflowList, workflowResume, workflowStart, workspaceCreate, workspaceDelete, workspaceList } from "../resume-DvbIclRU.mjs";
 import "../job-CL8myeqs.mjs";
 import { register } from "node:module";
 

package/dist/cli/index.mjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { PATScope, applyCommand, commonArgs, createCommand, deleteCommand, executionsCommand, fetchAll, fetchLatestToken, fetchUserInfo, formatArgs, generateCommand, getCommand, getCommand$1, initOAuth2Client, initOperatorClient, listCommand as listCommand$1, listCommand$1 as listCommand$6, listCommand$2 as listCommand$7, listCommand$3 as listCommand, loadAccessToken, loadConfig, loadWorkspaceId, parseFormat, printWithFormat, readPackageJson, readPlatformConfig, removeCommand, resumeCommand, showCommand, startCommand, tokenCommand, withCommonArgs, writePlatformConfig } from "../resume-ar5nEitS.mjs";
+import { PATScope, applyCommand, commonArgs, createCommand, deleteCommand, executionsCommand, fetchAll, fetchLatestToken, fetchUserInfo, formatArgs, generateCommand, getCommand, getCommand$1, initOAuth2Client, initOperatorClient, listCommand as listCommand$1, listCommand$1 as listCommand$6, listCommand$2 as listCommand$7, listCommand$3 as listCommand, loadAccessToken, loadConfig, loadWorkspaceId, parseFormat, printWithFormat, readPackageJson, readPlatformConfig, removeCommand, resumeCommand, showCommand, startCommand, tokenCommand, withCommonArgs, writePlatformConfig } from "../resume-DvbIclRU.mjs";
 import "../job-CL8myeqs.mjs";
 import { register } from "node:module";
 import { defineCommand, runCommand, runMain } from "citty";

package/dist/{resume-ar5nEitS.mjs → resume-DvbIclRU.mjs}

@@ -147487,7 +147487,7 @@ function filterJobFunctionVersions(allVersions, usedJobNames) {
 /**
 * Register job functions used by any workflow.
 * Only registers jobs that are actually used (based on usedJobNames in changeSet).
-* Uses update for existing workflows, create for new workflows.
+* Uses create for new jobs and update for existing jobs.
 */
 async function registerJobFunctions(client, changeSet) {
 	const jobFunctionVersions = {};
@@ -147496,11 +147496,18 @@ async function registerJobFunctions(client, changeSet) {
 	const { workspaceId, scripts } = firstWorkflow;
 	const allUsedJobNames = /* @__PURE__ */ new Set();
 	for (const item of [...changeSet.creates, ...changeSet.updates]) for (const jobName of item.usedJobNames) allUsedJobNames.add(jobName);
-	const hasExistingWorkflows = changeSet.updates.length > 0;
+	const existingJobNames = await fetchAll(async (pageToken) => {
+		const response = await client.listWorkflowJobFunctions({
+			workspaceId,
+			pageToken
+		});
+		return [response.jobFunctions.map((j) => j.name), response.nextPageToken];
+	});
+	const existingJobNamesSet = new Set(existingJobNames);
 	const results = await Promise.all(Array.from(allUsedJobNames).map(async (jobName) => {
 		const script = scripts.get(jobName);
 		if (!script) throw new Error(`No bundled script found for job "${jobName}". Please run "generate" command before "apply".`);
-		const response = hasExistingWorkflows ? await client.updateWorkflowJobFunction({
+		const response = existingJobNamesSet.has(jobName) ? await client.updateWorkflowJobFunction({
 			workspaceId,
 			jobFunctionName: jobName,
 			script
@@ -149946,4 +149953,4 @@ const resumeCommand = defineCommand({
 
 //#endregion
 export { PATScope, apply, applyCommand, commonArgs, createCommand, deleteCommand, executionsCommand, fetchAll, fetchLatestToken, fetchUserInfo, formatArgs, generate, generateCommand, generateUserTypes, getCommand, getCommand$1, initOAuth2Client, initOperatorClient, listCommand, listCommand$1, listCommand$2, listCommand$3, loadAccessToken, loadConfig, loadWorkspaceId, machineUserList, machineUserToken, oauth2ClientGet, oauth2ClientList, parseFormat, printWithFormat, readPackageJson, readPlatformConfig, remove, removeCommand, resumeCommand, show, showCommand, startCommand, tokenCommand, withCommonArgs, workflowExecutionGet, workflowExecutionsList, workflowGet, workflowList, workflowResume, workflowStart, workspaceCreate, workspaceDelete, workspaceList, writePlatformConfig };
-//# sourceMappingURL=resume-ar5nEitS.mjs.map
+//# sourceMappingURL=resume-DvbIclRU.mjs.map