@tailor-platform/sdk 1.44.0 → 1.44.2

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # @tailor-platform/sdk
 
+## 1.44.2
+
+### Patch Changes
+
+- [#1096](https://github.com/tailor-platform/sdk/pull/1096) [`53dbbaf`](https://github.com/tailor-platform/sdk/commit/53dbbaf4a6de353f227c3ef57f39580b7e1d9379) Thanks [@k1LoW](https://github.com/k1LoW)! - Fix workflow bundle build failure caused by removing default exports from dependency files during cross-file workflow imports
+
+## 1.44.1
+
+### Patch Changes
+
+- [#1088](https://github.com/tailor-platform/sdk/pull/1088) [`6dc5318`](https://github.com/tailor-platform/sdk/commit/6dc53185418c117c65cebd92c8ee38ae406e8c9a) Thanks [@k1LoW](https://github.com/k1LoW)! - Fix workflow bundle build failure caused by dead default imports after cross-file trigger transformation
+
+- [#1085](https://github.com/tailor-platform/sdk/pull/1085) [`0947e14`](https://github.com/tailor-platform/sdk/commit/0947e14daf39bf35b84f496984c32acb4c7bc24b) Thanks [@k1LoW](https://github.com/k1LoW)! - Add `concurrencyPolicy` option to `createWorkflow` for limiting concurrent workflow executions
+
 ## 1.44.0
 
 ### Minor Changes

package/dist/{application-DkVNbIuh.mjs → application-7wtQzifE.mjs}

@@ -1,5 +1,5 @@
 
-import { u as initOAuth2Client } from "./client-DQl5NPG9.mjs";
+import { u as initOAuth2Client } from "./client-DcuQRqSd.mjs";
 import { n as isSdkBranded } from "./brand-Ll48SMXe.mjs";
 import { a as parseBoolean, n as logger, r as styles } from "./logger-DTNAMYGy.mjs";
 import { n as enumConstantsPlugin, t as EnumConstantsGeneratorID } from "./enum-constants-C3KSpsYj.mjs";
@@ -1781,6 +1781,121 @@ function extractAuthInvokerInfo(configArg, sourceText) {
 	}
 }
 /**
+* Check if an AST binding pattern (parameter, catch clause, etc.) contains an Identifier with the given name.
+* @param node - AST node to inspect
+* @param name - Identifier name to look for
+* @returns True if the binding pattern contains the name
+*/
+function containsBindingName(node, name) {
+	if (!node || typeof node !== "object") return false;
+	if (node.type === "Identifier" && node.name === name) return true;
+	for (const key of Object.keys(node)) {
+		const child = node[key];
+		if (Array.isArray(child)) {
+			if (child.some((c) => containsBindingName(c, name))) return true;
+		} else if (child && typeof child === "object") {
+			if (containsBindingName(child, name)) return true;
+		}
+	}
+	return false;
+}
+/**
+* Build a map of reference counts for multiple identifiers in a single AST pass.
+* Scope-aware: references inside functions or catch clauses that shadow the name
+* via parameters are excluded, so only references to the original import binding
+* are counted.
+* Excludes import declarations and property names in non-computed member expressions.
+* @param program - The parsed AST program
+* @param names - Set of identifier names to count
+* @returns Map from identifier name to reference count
+*/
+function buildReferenceCountMap(program, names) {
+	if (names.size === 0) return /* @__PURE__ */ new Map();
+	const counts = /* @__PURE__ */ new Map();
+	const shadowDepth = /* @__PURE__ */ new Map();
+	for (const name of names) {
+		counts.set(name, 0);
+		shadowDepth.set(name, 0);
+	}
+	function walk(node, parentNode, parentKey) {
+		if (!node || typeof node !== "object") return;
+		const nodeType = node.type;
+		if (nodeType === "ImportDeclaration") return;
+		const shadowedHere = [];
+		if (nodeType === "FunctionDeclaration" || nodeType === "FunctionExpression" || nodeType === "ArrowFunctionExpression") {
+			const params = node.params;
+			if (params) {
+				for (const name of names) if (params.some((p) => containsBindingName(p, name))) {
+					shadowDepth.set(name, (shadowDepth.get(name) ?? 0) + 1);
+					shadowedHere.push(name);
+				}
+			}
+		}
+		if (nodeType === "CatchClause") {
+			const param = node.param;
+			if (param) {
+				for (const name of names) if (containsBindingName(param, name)) {
+					shadowDepth.set(name, (shadowDepth.get(name) ?? 0) + 1);
+					shadowedHere.push(name);
+				}
+			}
+		}
+		if (nodeType === "Identifier") {
+			const identName = node.name;
+			if (identName && names.has(identName) && (shadowDepth.get(identName) ?? 0) === 0) {
+				const isMemberProperty = parentNode && parentNode.type === "MemberExpression" && parentKey === "property" && !parentNode.computed;
+				const isObjectPropertyKey = parentNode && parentNode.type === "Property" && parentKey === "key" && !parentNode.shorthand && !parentNode.computed;
+				if (!isMemberProperty && !isObjectPropertyKey) counts.set(identName, (counts.get(identName) ?? 0) + 1);
+			}
+		}
+		for (const key of Object.keys(node)) {
+			const child = node[key];
+			if (Array.isArray(child)) child.forEach((c) => walk(c, node, key));
+			else if (child && typeof child === "object") walk(child, node, key);
+		}
+		for (const name of shadowedHere) shadowDepth.set(name, (shadowDepth.get(name) ?? 0) - 1);
+	}
+	walk(program);
+	return counts;
+}
+/**
+* Find the text range to remove for a dead default import.
+*
+* - Default-only import (`import wf from "..."`): returns the full declaration range.
+* - Mixed import (`import wf, { helper } from "..."`): returns the range covering
+*   the default specifier and trailing comma/whitespace so the result becomes
+*   `import { helper } from "..."`.
+* @param program - The parsed AST program
+* @param localName - The local name of the default import
+* @param source - The source code text (used to locate the `{` in mixed imports)
+* @returns Range to remove, or null if the import was not found
+*/
+function findDefaultImportRemovalRange(program, localName, source) {
+	for (const statement of program.body) {
+		if (statement.type !== "ImportDeclaration") continue;
+		const importDecl = statement;
+		const specifiers = importDecl.specifiers || [];
+		for (const spec of specifiers) {
+			if (spec.type !== "ImportDefaultSpecifier") continue;
+			const defaultSpec = spec;
+			if (defaultSpec.local?.name !== localName) continue;
+			if (specifiers.length === 1) return {
+				start: importDecl.start,
+				end: importDecl.end,
+				isFullDeclaration: true
+			};
+			const braceIndex = source.indexOf("{", defaultSpec.end);
+			if (braceIndex !== -1) return {
+				start: defaultSpec.start,
+				end: braceIndex,
+				isFullDeclaration: false
+			};
+			return null;
+		}
+	}
+	return null;
+}
+/**
 * Detect .trigger() calls for known workflows and jobs
 * Only detects calls where the identifier is in workflowNames or jobNames
 * @param program - The parsed AST program
@@ -1866,6 +1981,7 @@ function detectExtendedTriggerCalls(program, sourceText, workflowNames, jobNames
 function transformFunctionTriggers(source, workflowNameMap, jobNameMap, workflowFileMap, currentFilePath, authNamespace) {
 	const { program } = parseSync("input.ts", source);
 	const localWorkflowNameMap = new Map(workflowNameMap);
+	const workflowDefaultImportNames = /* @__PURE__ */ new Set();
 	if (workflowFileMap && currentFilePath) {
 		const defaultImports = detectDefaultImports(program);
 		const currentDir = currentFilePath.replace(/[/\\][^/\\]+$/, "");
@@ -1873,12 +1989,16 @@ function transformFunctionTriggers(source, workflowNameMap, jobNameMap, workflow
 			if (!importSource.startsWith(".")) continue;
 			const resolvedPath = resolvePath(currentDir, importSource);
 			const workflowName = workflowFileMap.get(resolvedPath);
-			if (workflowName) localWorkflowNameMap.set(localName, workflowName);
+			if (workflowName) {
+				localWorkflowNameMap.set(localName, workflowName);
+				workflowDefaultImportNames.add(localName);
+			}
 		}
 	}
 	const triggerCalls = detectExtendedTriggerCalls(program, source, new Set(localWorkflowNameMap.keys()), new Set(jobNameMap.keys()));
 	const replacements = [];
 	let needsNormalizerHelper = false;
+	const transformedCallsPerIdentifier = /* @__PURE__ */ new Map();
 	for (const call of triggerCalls) if (call.kind === "workflow" && call.authInvoker) {
 		const workflowName = localWorkflowNameMap.get(call.identifierName);
 		if (workflowName) {
@@ -1894,6 +2014,7 @@ function transformFunctionTriggers(source, workflowNameMap, jobNameMap, workflow
 				end: call.callRange.end,
 				text: transformedCall
 			});
+			transformedCallsPerIdentifier.set(call.identifierName, (transformedCallsPerIdentifier.get(call.identifierName) ?? 0) + 1);
 		}
 	} else if (call.kind === "job") {
 		const jobName = jobNameMap.get(call.identifierName);
@@ -1905,6 +2026,20 @@ function transformFunctionTriggers(source, workflowNameMap, jobNameMap, workflow
 				end: range.end,
 				text: transformedCall
 			});
+			transformedCallsPerIdentifier.set(call.identifierName, (transformedCallsPerIdentifier.get(call.identifierName) ?? 0) + 1);
+		}
+	}
+	const refCounts = buildReferenceCountMap(program, workflowDefaultImportNames);
+	for (const localName of workflowDefaultImportNames) {
+		const transformedCount = transformedCallsPerIdentifier.get(localName) ?? 0;
+		const refCount = refCounts.get(localName) ?? 0;
+		if (refCount === 0 || transformedCount >= refCount) {
+			const removal = findDefaultImportRemovalRange(program, localName, source);
+			if (removal) replacements.push({
+				start: removal.start,
+				end: removal.isFullDeclaration ? findStatementEnd(source, removal.end) : removal.end,
+				text: ""
+			});
 		}
 	}
 	const transformed = applyReplacements(source, replacements);
@@ -4225,6 +4360,7 @@ function findWorkflowDefaultExport(program) {
 function transformWorkflowSource(source, targetJobName, targetJobExportName, otherJobExportNames, allJobsMap) {
 	const { program } = parseSync("input.ts", source);
 	const detectedJobs = findAllJobs(program, source);
+	if (!detectedJobs.some((j) => j.name === targetJobName)) return source;
 	const jobNameMap = allJobsMap ?? buildJobNameMap(detectedJobs);
 	const allDeclarations = findVariableDeclarationsByName(program);
 	const triggerCalls = detectTriggerCalls(program, source);
@@ -4301,6 +4437,15 @@ function transformWorkflowSource(source, targetJobName, targetJobExportName, oth
 
 //#endregion
 //#region src/cli/services/workflow/bundler.ts
+function safeRealpath(p) {
+	const resolved = path.resolve(p);
+	try {
+		return fs$1.realpathSync(resolved);
+	} catch (e) {
+		logger.debug(`realpathSync failed for ${resolved}: ${e instanceof Error ? e.message : e}`);
+		return resolved;
+	}
+}
 /**
 * Bundle workflow jobs
 *
@@ -4456,14 +4601,17 @@ async function bundleSingleJob(job, allJobs, outputDir, tsconfig, env, triggerCo
 			const otherJobExportNames = allJobs.filter((j) => j.name !== job.name).map((j) => j.exportName);
 			const allJobsMap = /* @__PURE__ */ new Map();
 			for (const j of allJobs) allJobsMap.set(j.exportName, j.name);
+			const resolvedSourceFile = safeRealpath(job.sourceFile);
 			const plugins = [{
 				name: "workflow-transform",
 				transform: {
 					filter: { id: { include: [/\.ts$/, /\.js$/] } },
 					handler(code, id) {
 						if (!code.includes("createWorkflowJob") && !code.includes("createWorkflow") && !code.includes(".trigger(")) return null;
-						let transformed = transformWorkflowSource(code, job.name, job.exportName, otherJobExportNames, allJobsMap);
+						let transformed = code;
+						if (safeRealpath(id) === resolvedSourceFile) transformed = transformWorkflowSource(code, job.name, job.exportName, otherJobExportNames, allJobsMap);
 						if (triggerContext && transformed.includes(".trigger(")) transformed = transformFunctionTriggers(transformed, triggerContext.workflowNameMap, triggerContext.jobNameMap, triggerContext.workflowFileMap, id, triggerContext.authNamespace);
+						if (transformed === code) return null;
 						return { code: transformed };
 					}
 				}
@@ -4527,10 +4675,12 @@ const RetryPolicySchema = z.object({
 	message: "initialBackoff must be greater than 0",
 	path: ["initialBackoff"]
 });
+const ConcurrencyPolicySchema = z.object({ maxConcurrentExecutions: z.number().int().min(1).max(1e3).describe("Maximum number of concurrent executions (1-1000)") });
 const WorkflowSchema = z.object({
 	name: z.string().describe("Workflow name"),
 	mainJob: WorkflowJobSchema.describe("Main job that starts the workflow"),
-	retryPolicy: RetryPolicySchema.optional().describe("Retry policy for the workflow")
+	retryPolicy: RetryPolicySchema.optional().describe("Retry policy for the workflow"),
+	concurrencyPolicy: ConcurrencyPolicySchema.optional().describe("Concurrency policy for the workflow")
 });
 
 //#endregion
@@ -5167,4 +5317,4 @@ async function loadApplication(params) {
 
 //#endregion
 export { loadWorkspaceId as C, writePlatformConfig as D, saveUserTokens as E, loadAccessToken as S, resolveTokens as T, getDistDir as _, WorkflowJobSchema as a, deleteUserTokens as b, ExecutorSchema as c, buildResolverOperationHookExpr as d, OAuth2ClientSchema as f, createBundleCache as g, loadFilesWithIgnores as h, resolveInlineSourcemap as i, INVOKER_EXPR as l, stringifyFunction as m, generatePluginFilesIfNeeded as n, ResolverSchema as o, TailorDBTypeSchema as p, loadApplication as r, createExecutorService as s, defineApplication as t, buildExecutorArgsExpr as u, hashFile as v, readPlatformConfig as w, fetchLatestToken as x, loadConfig as y };
-//# sourceMappingURL=application-DkVNbIuh.mjs.map
+//# sourceMappingURL=application-7wtQzifE.mjs.map