@tailor-platform/sdk 1.46.0 → 1.47.1

package/CHANGELOG.md

@@ -1,5 +1,53 @@
 # @tailor-platform/sdk
 
+## 1.47.1
+
+### Patch Changes
+
+- [#1176](https://github.com/tailor-platform/sdk/pull/1176) [`5abed20`](https://github.com/tailor-platform/sdk/commit/5abed20dff48e24f23675989143e33c8afb23845) Thanks [@toiroakr](https://github.com/toiroakr)! - Declare `undici` as a direct dependency. The SDK CLI imports `getGlobalDispatcher` from `undici`, but the package was previously available only through accidental hoisting of a transitive dependency. Strict node_modules layouts (e.g. pnpm 11 with stricter hoisting) would fail to resolve the import; declaring it directly fixes that.
+
+- [#1172](https://github.com/tailor-platform/sdk/pull/1172) [`7f37a07`](https://github.com/tailor-platform/sdk/commit/7f37a076a281fdca03d2301e1dba95668b2f7222) Thanks [@toiroakr](https://github.com/toiroakr)! - Widen `TailorEnv` fallback to `Record<string, string | number | boolean>` so it matches the values the type generator emits (string literal / number / boolean). Previously the fallback was `Record<string, string>`, which rejected number and boolean env values until `tailor.d.ts` was generated.
+
+- [#1161](https://github.com/tailor-platform/sdk/pull/1161) [`3e835c5`](https://github.com/tailor-platform/sdk/commit/3e835c55e27fbfedff94a169d1c2fb3c4f50e0a3) Thanks [@renovate](https://github.com/apps/renovate)! - fix(deps): update rolldown
+
+- [#1165](https://github.com/tailor-platform/sdk/pull/1165) [`f4ff7bd`](https://github.com/tailor-platform/sdk/commit/f4ff7bd5318be0a1142a557c76a17896d9df193a) Thanks [@toiroakr](https://github.com/toiroakr)! - Fix a type/runtime mismatch where calling `workflowJob.trigger()` without `await` returned a raw value at runtime even though the static type is `Promise<Awaited<Output>>`. The bundler now wraps `.trigger()` in an async IIFE (`(async () => tailor.workflow.triggerJobFunction("...", args))()`), so the returned value is always a `Promise` (including for `.then()` chains), synchronous throws from the platform surface as Promise rejections, and the platform's synchronous suspend semantics are preserved (the call site runs to completion before subsequent statements).
+
+## 1.47.0
+
+### Minor Changes
+
+- [#1115](https://github.com/tailor-platform/sdk/pull/1115) [`8dd619e`](https://github.com/tailor-platform/sdk/commit/8dd619e9c58f4662b117bbd968ecf9528d688fe4) Thanks [@toiroakr](https://github.com/toiroakr)! - Add `@tailor-platform/sdk/vitest` (beta) — a Vitest plugin and environment that emulates the Tailor Platform function runtime locally. Catches `node:*` imports and Node.js globals usage that would fail at deploy time, and provides mock control objects (`tailordbMock`, `workflowMock`, `secretmanagerMock`, `authconnectionMock`, `idpMock`, `fileMock`, `iconvMock`) for all platform APIs with response configuration and call recording.
+
+  Revamp `packages/sdk/docs/testing.md` into a 2-layer model (Unit Tests / E2E Tests). The previous structure split Unit, Bundled, and Workflow tests across overlapping sections and contained broken vitest imports and references to a non-existent `--template testing`. The new docs cover testing resolvers (simple, with TailorDB mocks, with DI, and with wait points) and workflow jobs (simple, with `triggerJobFunction` mocks, with wait-point mocks, and full-workflow integration), all anchored on the actual `resolver` and `workflow` templates.
+
+  Mark `createImportMain` and `setupInvokerMock` from `@tailor-platform/sdk/test` as `@deprecated`. `createImportMain` is an SDK-internal helper for verifying bundled output; applications should test their TypeScript source directly (unit) and verify deployed behavior via E2E. `setupInvokerMock` is superseded by the `tailor-runtime` Vitest environment, where bundled tests can drive the invoker via `vi.spyOn(globalThis.tailor.context, "getInvoker").mockReturnValue(...)` and unit tests can pass `invoker` directly to `.body()`. Both exports remain in place for now to avoid a breaking change and will be removed in a future release.
+
+  Remove the broken `tests/bundled.test.ts` from the `resolver` and `workflow` templates along with the related `bundled` vitest project and `test:bundled` / `test:bundled:prepare` scripts. These tests were not exercised by CI and had drifted out of sync with the SDK, producing failures on a fresh scaffold.
+
+  Fix a broken anchor in `docs/services/workflow.md` that pointed at the removed `#testing-wait-points` heading; it now links to `../testing.md#jobs-that-wait-on-approval` to match the new testing docs structure.
+
+### Patch Changes
+
+- [#1155](https://github.com/tailor-platform/sdk/pull/1155) [`fd70f8c`](https://github.com/tailor-platform/sdk/commit/fd70f8c75aa9a315eaf23ee25f36f43324b2eb54) Thanks [@toiroakr](https://github.com/toiroakr)! - Fix false-positive remote schema drift on `decimal` fields without an explicit
+  `scale`. The platform stores decimal fields with a default scale of 6, but the
+  snapshot generated by `tailor-sdk tailordb migration generate --init` did not
+  record this default, so `tailor-sdk deploy` (formerly `apply`) reported drift as
+  `scale: remote=6, expected=undefined` and blocked the deploy.
+
+  Migration drift detection now treats an unset `scale` on a `decimal` field as
+  equivalent to the platform default (6). No regeneration of existing snapshots
+  is required; users no longer need `--no-schema-check` to work around this case.
+
+- [#1164](https://github.com/tailor-platform/sdk/pull/1164) [`b1e8f5a`](https://github.com/tailor-platform/sdk/commit/b1e8f5a7e971527ff04cdb5867cacda2415e01bf) Thanks [@toiroakr](https://github.com/toiroakr)! - Eliminate the parser-layer exception that allowed `parser/service/tailordb/runtime.ts` to re-export runtime helpers from the configure module. Plugin attachment processing for a TailorDB type now lives in `PluginManager.processAttachmentsForType` and returns plain data (extended type, generated types, render events), so the cli layer applies state and renders progress without depending on `TailorAnyDBType`. Internal refactor with no public API change.
+
+- [#1134](https://github.com/tailor-platform/sdk/pull/1134) [`02e12f6`](https://github.com/tailor-platform/sdk/commit/02e12f6b4a0a5986bc925defcac44717ae293c88) Thanks [@renovate](https://github.com/apps/renovate)! - fix(deps): update @opentelemetry
+
+- [#1135](https://github.com/tailor-platform/sdk/pull/1135) [`8f53196`](https://github.com/tailor-platform/sdk/commit/8f53196bd66e4ecba702074399979d577e289a07) Thanks [@renovate](https://github.com/apps/renovate)! - fix(deps): update dependency @clack/prompts to v1.3.0
+
+- [#1137](https://github.com/tailor-platform/sdk/pull/1137) [`aef3653`](https://github.com/tailor-platform/sdk/commit/aef365380900c65420237cb47f75ff56f31c939a) Thanks [@renovate](https://github.com/apps/renovate)! - fix(deps): update dependency @napi-rs/keyring to v1.3.0
+
+- [#1047](https://github.com/tailor-platform/sdk/pull/1047) [`aa6ec4a`](https://github.com/tailor-platform/sdk/commit/aa6ec4a562b771526ffb553aa03f7061ed483504) Thanks [@toiroakr](https://github.com/toiroakr)! - chore(setup github): bump bundled `tailor-platform/actions` ref to v1.1.0 and let Renovate keep it up to date going forward.
+
 ## 1.46.0
 
 ### Minor Changes

package/dist/{actor-BmxQeMFP.d.mts → actor-jk4-f0yp.d.mts}

@@ -1,10 +1,10 @@
 /// <reference types="@tailor-platform/function-types" />
-import { Bt as InferredAttributeMap, zt as InferredAttributeList } from "./tailor-db-field-Hx9OqPWY.mjs";
+import { Bt as InferredAttributeMap, zt as InferredAttributeList } from "./tailor-db-field-Bn8ZC5lK.mjs";
 
 //#region src/types/env.d.ts
 interface Env {}
 /** Represents environment variables in the Tailor platform. */
-type TailorEnv = keyof Env extends never ? Record<string, string> : Env;
+type TailorEnv = keyof Env extends never ? Record<string, string | number | boolean> : Env;
 //#endregion
 //#region src/types/actor.d.ts
 /** User type enum values from the Tailor Platform server. */
@@ -27,4 +27,4 @@ type TailorActor = {
 };
 //#endregion
 export { Env as n, TailorEnv as r, TailorActor as t };
-//# sourceMappingURL=actor-BmxQeMFP.d.mts.map
+//# sourceMappingURL=actor-jk4-f0yp.d.mts.map

package/dist/{application-B4zVVNRS.mjs → application-C7H7y0hS.mjs}

@@ -1,6 +1,6 @@
 
-import { u as initOAuth2Client } from "./client-BwXkoiMq.mjs";
-import { n as isSdkBranded } from "./brand-Ll48SMXe.mjs";
+import { n as isSdkBranded } from "./brand-D-d15jx3.mjs";
+import { u as initOAuth2Client } from "./client-DbyKSN1F.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";
 import { t as multiline } from "./multiline-e3IpANmS.mjs";
@@ -20,8 +20,8 @@ import { parseTOML, parseYAML, stringifyYAML } from "confbox";
 import { findUpSync } from "find-up-simple";
 import { lt } from "semver";
 import { xdgConfig } from "xdg-basedir";
-import * as crypto from "node:crypto";
 import { resolveTSConfig } from "pkg-types";
+import * as crypto from "node:crypto";
 import * as rolldown from "rolldown";
 import * as fs from "node:fs/promises";
 import { parseSync } from "oxc-parser";
@@ -1601,7 +1601,7 @@ function buildJobNameMap(jobs) {
 */
 function detectTriggerCalls(program, sourceText) {
 	const calls = [];
-	function walk(node, parent = null) {
+	function walk(node) {
 		if (!node || typeof node !== "object") return;
 		if (node.type === "CallExpression") {
 			const callExpr = node;
@@ -1613,33 +1613,23 @@ function detectTriggerCalls(program, sourceText) {
 					let argsText = "";
 					if (callExpr.arguments.length > 0) {
 						const firstArg = callExpr.arguments[0];
-						const lastArg = callExpr.arguments[callExpr.arguments.length - 1];
-						if (firstArg && lastArg && "start" in firstArg && "end" in lastArg) argsText = sourceText.slice(firstArg.start, lastArg.end);
+						if (firstArg && "start" in firstArg && "end" in firstArg) argsText = sourceText.slice(firstArg.start, firstArg.end);
 					}
-					const hasAwait = parent?.type === "AwaitExpression";
-					const awaitExpr = hasAwait ? parent : null;
-					const callRange = {
-						start: callExpr.start,
-						end: callExpr.end
-					};
-					const fullRange = awaitExpr ? {
-						start: awaitExpr.start,
-						end: awaitExpr.end
-					} : callRange;
 					calls.push({
 						identifierName,
-						callRange,
-						argsText,
-						hasAwait,
-						fullRange
+						callRange: {
+							start: callExpr.start,
+							end: callExpr.end
+						},
+						argsText
 					});
 				}
 			}
 		}
 		for (const key of Object.keys(node)) {
 			const child = node[key];
-			if (Array.isArray(child)) child.forEach((c) => walk(c, node));
-			else if (child && typeof child === "object") walk(child, node);
+			if (Array.isArray(child)) child.forEach((c) => walk(c));
+			else if (child && typeof child === "object") walk(child);
 		}
 	}
 	walk(program);
@@ -1906,7 +1896,7 @@ function findDefaultImportRemovalRange(program, localName, source) {
 */
 function detectExtendedTriggerCalls(program, sourceText, workflowNames, jobNames) {
 	const calls = [];
-	function walk(node, parent = null) {
+	function walk(node) {
 		if (!node || typeof node !== "object") return;
 		if (node.type === "CallExpression") {
 			const callExpr = node;
@@ -1918,50 +1908,43 @@ function detectExtendedTriggerCalls(program, sourceText, workflowNames, jobNames
 				if (identifierName && propertyName === "trigger") {
 					const isWorkflow = workflowNames.has(identifierName);
 					const isJob = jobNames.has(identifierName);
-					if (!isWorkflow && !isJob) return;
-					const argCount = callExpr.arguments.length;
-					let argsText = "";
-					if (argCount > 0) {
-						const firstArg = callExpr.arguments[0];
-						if (firstArg && "start" in firstArg && "end" in firstArg) argsText = sourceText.slice(firstArg.start, firstArg.end);
-					}
-					const hasAwait = parent?.type === "AwaitExpression";
-					const awaitExpr = hasAwait ? parent : null;
-					if (isWorkflow && argCount >= 2) {
-						const secondArg = callExpr.arguments[1];
-						const authInvoker = extractAuthInvokerInfo(secondArg, sourceText);
-						if (authInvoker) calls.push({
-							kind: "workflow",
+					if (isWorkflow || isJob) {
+						const argCount = callExpr.arguments.length;
+						let argsText = "";
+						if (argCount > 0) {
+							const firstArg = callExpr.arguments[0];
+							if (firstArg && "start" in firstArg && "end" in firstArg) argsText = sourceText.slice(firstArg.start, firstArg.end);
+						}
+						if (isWorkflow && argCount >= 2) {
+							const secondArg = callExpr.arguments[1];
+							const authInvoker = extractAuthInvokerInfo(secondArg, sourceText);
+							if (authInvoker) calls.push({
+								kind: "workflow",
+								identifierName,
+								callRange: {
+									start: callExpr.start,
+									end: callExpr.end
+								},
+								argsText,
+								authInvoker
+							});
+						} else if (isJob) calls.push({
+							kind: "job",
 							identifierName,
 							callRange: {
 								start: callExpr.start,
 								end: callExpr.end
 							},
-							argsText,
-							authInvoker,
-							hasAwait: false
+							argsText
 						});
-					} else if (isJob) calls.push({
-						kind: "job",
-						identifierName,
-						callRange: {
-							start: callExpr.start,
-							end: callExpr.end
-						},
-						argsText,
-						hasAwait,
-						fullRange: awaitExpr ? {
-							start: awaitExpr.start,
-							end: awaitExpr.end
-						} : void 0
-					});
+					}
 				}
 			}
 		}
 		for (const key of Object.keys(node)) {
 			const child = node[key];
-			if (Array.isArray(child)) child.forEach((c) => walk(c, node));
-			else if (child && typeof child === "object") walk(child, node);
+			if (Array.isArray(child)) child.forEach((c) => walk(c));
+			else if (child && typeof child === "object") walk(child);
 		}
 	}
 	walk(program);
@@ -2019,11 +2002,10 @@ function transformFunctionTriggers(source, workflowNameMap, jobNameMap, workflow
 	} else if (call.kind === "job") {
 		const jobName = jobNameMap.get(call.identifierName);
 		if (jobName) {
-			const transformedCall = `tailor.workflow.triggerJobFunction("${jobName}", ${call.argsText || "undefined"})`;
-			const range = call.hasAwait && call.fullRange ? call.fullRange : call.callRange;
+			const transformedCall = `(async () => tailor.workflow.triggerJobFunction("${jobName}", ${call.argsText || "undefined"}))()`;
 			replacements.push({
-				start: range.start,
-				end: range.end,
+				start: call.callRange.start,
+				end: call.callRange.end,
 				text: transformedCall
 			});
 			transformedCallsPerIdentifier.set(call.identifierName, (transformedCallsPerIdentifier.get(call.identifierName) ?? 0) + 1);
@@ -3287,46 +3269,27 @@ function createTailorDBService(params) {
 	*/
 	const processPluginsForType = async (rawType, attachments, sourceFilePath) => {
 		if (!pluginManager) return;
-		let currentType = rawType;
-		for (const attachment of attachments) {
-			const result = await pluginManager.processAttachment({
-				type: currentType,
-				typeConfig: attachment.config,
-				namespace,
-				pluginId: attachment.pluginId
-			});
-			if (!result.success) {
-				logger.error(result.error);
-				throw new Error(result.error);
-			}
-			const output = result.output;
-			const extendFields = output.extends?.fields;
-			if (extendFields && Object.keys(extendFields).length > 0) {
-				const extendedType = pluginManager.extendType({
-					originalType: currentType,
-					extendFields,
-					pluginId: attachment.pluginId
-				});
-				rawTypes[sourceFilePath][currentType.name] = extendedType;
-				currentType = extendedType;
-				logger.log(`  Extended: ${styles.success(currentType.name)} with ${styles.highlight(Object.keys(extendFields).length.toString())} fields by plugin ${styles.info(attachment.pluginId)}`);
-			}
-			const plugin = pluginManager.getPlugin(attachment.pluginId);
-			for (const [kind, generatedType] of Object.entries(output.types ?? {})) {
-				rawTypes[sourceFilePath][generatedType.name] = generatedType;
-				typeSourceInfo[generatedType.name] = {
-					exportName: generatedType.name,
-					pluginId: attachment.pluginId,
-					pluginImportPath: pluginManager.getPluginImportPath(attachment.pluginId) ?? "",
-					originalFilePath: sourceFilePath,
-					originalExportName: typeSourceInfo[rawType.name]?.exportName || rawType.name,
-					generatedTypeKind: kind,
-					pluginConfig: plugin?.pluginConfig,
-					namespace
-				};
-				logger.log(`  Generated: ${styles.success(generatedType.name)} by plugin ${styles.info(attachment.pluginId)}`);
-			}
+		const { extendedType, generatedTypes, events } = await pluginManager.processAttachmentsForType({
+			rawType,
+			attachments,
+			namespace
+		});
+		if (extendedType) rawTypes[sourceFilePath][rawType.name] = extendedType;
+		for (const gen of generatedTypes) {
+			rawTypes[sourceFilePath][gen.typeName] = gen.type;
+			typeSourceInfo[gen.typeName] = {
+				exportName: gen.typeName,
+				pluginId: gen.pluginId,
+				pluginImportPath: gen.pluginImportPath,
+				originalFilePath: sourceFilePath,
+				originalExportName: typeSourceInfo[rawType.name]?.exportName || rawType.name,
+				generatedTypeKind: gen.kind,
+				pluginConfig: gen.pluginConfig,
+				namespace
+			};
 		}
+		for (const ev of events) if (ev.kind === "extended") logger.log(`  Extended: ${styles.success(ev.typeName)} with ${styles.highlight(ev.fieldCount.toString())} fields by plugin ${styles.info(ev.pluginId)}`);
+		else logger.log(`  Generated: ${styles.success(ev.typeName)} by plugin ${styles.info(ev.pluginId)}`);
 	};
 	const loadTypeFile = async (typeFile, tsconfig) => {
 		rawTypes[typeFile] = {};
@@ -4425,10 +4388,10 @@ function transformWorkflowSource(source, targetJobName, targetJobExportName, oth
 		if (isInsideRemovedRange(call.callRange.start)) continue;
 		const jobName = jobNameMap.get(call.identifierName);
 		if (jobName) {
-			const transformedCall = `tailor.workflow.triggerJobFunction("${jobName}", ${call.argsText || "undefined"})`;
+			const transformedCall = `(async () => tailor.workflow.triggerJobFunction("${jobName}", ${call.argsText || "undefined"}))()`;
 			replacements.push({
-				start: call.fullRange.start,
-				end: call.fullRange.end,
+				start: call.callRange.start,
+				end: call.callRange.end,
 				text: transformedCall
 			});
 		}
@@ -5318,4 +5281,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-B4zVVNRS.mjs.map
+//# sourceMappingURL=application-C7H7y0hS.mjs.map