@tailor-platform/sdk-codemod 0.3.0-next.6 → 0.3.0-next.8

package/CHANGELOG.md

@@ -1,5 +1,60 @@
 # @tailor-platform/sdk-codemod
 
+## 0.3.0-next.8
+
+### Patch Changes
+
+- [#1811](https://github.com/tailor-platform/sdk/pull/1811) [`b2fc104`](https://github.com/tailor-platform/sdk/commit/b2fc104d9cdfc52e98c97bc18d80a9e2e9d5f4c2) Thanks [@toiroakr](https://github.com/toiroakr)! - Move the TailorDB `erdSite` setting out of the core config schema into the ERD plugin's own configuration. `db.<namespace>.erdSite` is no longer accepted in `tailor.config.ts`; configure the ERD deploy target on the plugin instead:
+  
+  ```ts
+  import { definePlugins } from "@tailor-platform/sdk";
+  import { tailordbErdPlugin } from "@tailor-platform/sdk-plugin-tailordb-erd";
+  
+  export const plugins = definePlugins(
+    // TailorDB namespace name → static website name
+    tailordbErdPlugin({ sites: { tailordb: "my-erd-site" } }),
+  );
+  ```
+  
+  The `tailor tailordb erd` commands resolve deploy targets from `tailordbErdPlugin({ sites })` and now validate each namespace against `config.db` and each site name against `staticWebsites`, so typos surface when the config is loaded instead of at deploy time. The `v2/erd-site-to-plugin` codemod migrates existing configs automatically. For programmatic users, `loadTailorDBNamespaces()` additionally returns the config module's registered `plugins`, and namespace selector callbacks receive them as a second argument.
+
+- [#1807](https://github.com/tailor-platform/sdk/pull/1807) [`817454f`](https://github.com/tailor-platform/sdk/commit/817454fff35e4093bce5fdcb9e1fcda8bbd1d7ef) Thanks [@dqn](https://github.com/dqn)! - `seedPlugin` no longer generates the `exec.mjs` seed runner. Seeding and validation move to the `tailor seed` commands provided by the `@tailor-platform/sdk-plugin-seed` CLI plugin: install it with `npm install -D @tailor-platform/sdk-plugin-seed`, replace `node <distPath>/exec.mjs` with `tailor seed apply` and `node <distPath>/exec.mjs validate` with `tailor seed validate`, and delete the stale generated `exec.mjs`. Seed data and schema generation (`data/*.jsonl`, `data/*.schema.ts`) is unchanged. Because the plugin reads the config at run time, `machineUserName` changes in seedPlugin options now take effect without regenerating. `@tailor-platform/sdk/cli` gains `loadSeedContext` (and `SeedContext` types) for this, `SeedData` is now JSON-typed, and `executeScript` accepts a plain object `invoker` (`ScriptInvoker`).
+
+## 0.3.0-next.7
+
+### Patch Changes
+
+- [#1787](https://github.com/tailor-platform/sdk/pull/1787) [`898d0b0`](https://github.com/tailor-platform/sdk/commit/898d0b0f809d15ea883a32a78a247eda4ca7caa7) Thanks [@toiroakr](https://github.com/toiroakr)! - Fix `tailor upgrade` reporting zero codemods across several v2 prerelease boundaries. `v2/db-type-to-table` and `v2/runtime-subpath-namespace` now trigger at `2.0.0-next.4` (where they actually shipped) instead of `2.0.0-next.3`, and `v2/forward-relation-name`, `v2/tailordb-validate-simplify`, and `v2/tailordb-hook-redesign` now trigger at `2.0.0-next.5` instead of `2.0.0-next.4`.
+
+- [#1787](https://github.com/tailor-platform/sdk/pull/1787) [`6653afd`](https://github.com/tailor-platform/sdk/commit/6653afd5a0be52f8c9a1dc6fa50e445f0f678dc0) Thanks [@toiroakr](https://github.com/toiroakr)! - Add a `V2_NEXT_PENDING` placeholder `prereleaseUntil` for codemods whose exact `2.0.0-next.N` release boundary isn't known yet at implementation time, plus a `pnpm codemod:resolve-pending` step (wired into the release workflow) that resolves it to the real version constant once the release PR bumps `@tailor-platform/sdk`'s version. Prevents codemod boundaries from drifting out of sync with the version they actually ship in, which previously required a manual follow-up fix after each release.
+
+- [#1782](https://github.com/tailor-platform/sdk/pull/1782) [`c971797`](https://github.com/tailor-platform/sdk/commit/c971797c9bfa035a43771c46f2b1c3bd93f989a9) Thanks [@toiroakr](https://github.com/toiroakr)! - Rename `Workflow.trigger()` (returned by `createWorkflow()`) and `WorkflowJob.trigger()` (returned by `createWorkflowJob()`) to `.start()`, aligning the SDK's ergonomic verb with the platform's `start*` RPC vocabulary:
+  
+  ```diff
+   const inventory = checkInventory.trigger({ orderId: input.orderId });
+  +const inventory = checkInventory.start({ orderId: input.orderId });
+  
+  -const workflowRunId = await orderProcessingWorkflow.trigger(args, { invoker: "manager" });
+  +const workflowRunId = await orderProcessingWorkflow.start(args, { invoker: "manager" });
+  ```
+  
+  `mockWorkflow()`'s `wf.job(definition)` / `wf.workflow(definition)` now return a mock of the `.start` method, and `wf.setTriggerHandler` / `wf.triggeredJobs` are renamed to `wf.setStartHandler` / `wf.startedJobs`. No codemod ships for the `.trigger()` → `.start()` call-site rename itself — see the `v2/workflow-start-rename` migration guide entry for manual migration steps.
+
+- [#1782](https://github.com/tailor-platform/sdk/pull/1782) [`c971797`](https://github.com/tailor-platform/sdk/commit/c971797c9bfa035a43771c46f2b1c3bd93f989a9) Thanks [@toiroakr](https://github.com/toiroakr)! - Remove the pre-alignment `tailor.workflow` names `triggerWorkflow`, `triggerJobFunction`, and `resumeWorkflow` (and their `TriggerWorkflowOptions` / `TriggerJobFunctionOptions` option types) from `@tailor-platform/sdk/runtime`, the ambient `@tailor-platform/sdk/runtime/globals` types, and the `mockWorkflow()` test facade. Use the canonical names instead:
+  
+  ```diff
+   import { workflow } from "@tailor-platform/sdk/runtime";
+  
+  -await workflow.triggerWorkflow("myWorkflow", { data: "value" });
+  +await workflow.startWorkflow("myWorkflow", { data: "value" });
+  -workflow.triggerJobFunction("myJob", { data: "value" });
+  +workflow.startJobFunction("myJob", { data: "value" });
+  -await workflow.resumeWorkflow("execution-id");
+  +await workflow.resumeWorkflowExecution("execution-id");
+  ```
+  
+  Run the `v2/workflow-trigger-rename` codemod to migrate call sites automatically.
+
 ## 0.3.0-next.6
 
 ### Patch Changes

package/dist/codemods/v2/erd-site-to-plugin/scripts/transform.js

@@ -0,0 +1,195 @@
+import { Lang, parse } from "@ast-grep/napi";
+//#region codemods/v2/erd-site-to-plugin/scripts/transform.ts
+const PLUGIN_IMPORT = "import { tailordbErdPlugin } from \"@tailor-platform/sdk-plugin-tailordb-erd\";";
+const DEFINE_PLUGINS_IMPORT = "import { definePlugins } from \"@tailor-platform/sdk\";";
+const SDK_VALUE_IMPORT_REGEX = /(^|\n)import\s*\{[^}\n]*\}\s*from\s*["']@tailor-platform\/sdk["'];?/;
+const FUNCTION_KINDS = /* @__PURE__ */ new Set([
+	"arrow_function",
+	"function_declaration",
+	"function_expression",
+	"generator_function",
+	"generator_function_declaration",
+	"method_definition"
+]);
+function unquote(text) {
+	return text.replace(/^["']|["']$/g, "");
+}
+function propertyName(pair) {
+	const key = pair.field("key");
+	if (!key || key.kind() === "computed_property_name") return null;
+	return unquote(key.text());
+}
+function insideFunction(node) {
+	for (let current = node.parent(); current; current = current.parent()) if (FUNCTION_KINDS.has(current.kind())) return true;
+	return false;
+}
+/**
+* Resolve the local binding name of `definePlugins` imported from
+* `@tailor-platform/sdk`, honoring `import { definePlugins as alias }`.
+* @param tree - Parsed source tree root.
+* @returns Local binding name, or null when it is not imported.
+*/
+function definePluginsLocalName(tree) {
+	const importStatements = tree.findAll({ rule: {
+		kind: "import_statement",
+		has: {
+			kind: "string",
+			regex: "^[\"']@tailor-platform/sdk[\"']$"
+		}
+	} });
+	for (const statement of importStatements) for (const specifier of statement.findAll({ rule: { kind: "import_specifier" } })) {
+		const name = specifier.field("name");
+		if (name?.text() !== "definePlugins") continue;
+		return (specifier.field("alias") ?? name).text();
+	}
+	return null;
+}
+/**
+* Build an edit that removes a property pair from an object literal, cleaning
+* up the separating comma, an inline line comment that documented the removed
+* property, and the removed line's indentation.
+* @param objectNode - Object literal containing the pair.
+* @param pairNode - Property pair to remove.
+* @returns Edit replacing the object literal with the pair removed.
+*/
+function removePairEdit(objectNode, pairNode) {
+	const objText = objectNode.text();
+	const objStart = objectNode.range().start.index;
+	const start = pairNode.range().start.index - objStart;
+	const end = pairNode.range().end.index - objStart;
+	const before = objText.slice(0, start);
+	const after = objText.slice(end);
+	let removeFrom = start;
+	let removeTo = end;
+	const trailing = after.match(/^[ \t]*,[ \t]*(?:\/\/[^\n]*)?\n?/);
+	if (trailing) {
+		removeTo = end + trailing[0].length;
+		const indent = before.match(/\n[ \t]*$/);
+		if (indent) removeFrom = start - (indent[0].length - 1);
+	} else {
+		const leading = before.match(/,\s*$/);
+		if (leading) removeFrom = start - leading[0].length;
+		const inlineComment = after.match(/^[ \t]*\/\/[^\n]*/);
+		if (inlineComment) removeTo = end + inlineComment[0].length;
+	}
+	return objectNode.replace(objText.slice(0, removeFrom) + objText.slice(removeTo));
+}
+/**
+* Build an edit that appends an argument to a call expression, preserving the
+* call's single-line or multi-line formatting. Insertion points are derived
+* from argument-list AST nodes so a trailing line comment after the last
+* argument cannot swallow the separating comma.
+* @param callNode - Call expression to extend.
+* @param arg - Argument expression to append.
+* @returns Edit replacing the call expression with the argument appended.
+*/
+function appendArgEdit(callNode, arg) {
+	const callText = callNode.text();
+	const base = callNode.range().start.index;
+	const children = callNode.field("arguments").children();
+	const args = children.filter((child) => child.isNamed() && child.kind() !== "comment");
+	const closeOffset = children.at(-1).range().start.index - base;
+	const multiline = callText.includes("\n");
+	const closeIndent = callText.slice(0, closeOffset).match(/\n([ \t]*)$/)?.[1] ?? "";
+	const argIndent = `${closeIndent}  `;
+	if (args.length === 0) {
+		const head = callText.slice(0, closeOffset).replace(/[ \t]*$/, "");
+		const rewritten = multiline ? `${head}${argIndent}${arg},\n${closeIndent})` : `${head}${arg})`;
+		return callNode.replace(rewritten);
+	}
+	const lastArg = args.at(-1);
+	const followers = children.slice(children.indexOf(lastArg) + 1);
+	const trailingComma = followers.find((child) => !child.isNamed() && child.text() === ",");
+	const anchor = trailingComma ?? lastArg;
+	if (!multiline) {
+		const insertAt = anchor.range().end.index - base;
+		const insertion = `${trailingComma ? "" : ","} ${arg}`;
+		return callNode.replace(callText.slice(0, insertAt) + insertion + callText.slice(insertAt));
+	}
+	const argInsertAt = (followers.findLast((child) => child.kind() === "comment" && child.range().start.index >= anchor.range().end.index && child.range().start.line === anchor.range().end.line) ?? anchor).range().end.index - base;
+	let rewritten = callText.slice(0, argInsertAt) + `\n${argIndent}${arg},` + callText.slice(argInsertAt);
+	if (!trailingComma) {
+		const commaAt = lastArg.range().end.index - base;
+		rewritten = rewritten.slice(0, commaAt) + "," + rewritten.slice(commaAt);
+	}
+	return callNode.replace(rewritten);
+}
+/**
+* Add `definePlugins` to an existing single-line value import from
+* `@tailor-platform/sdk`, or return null when no such import exists.
+* @param source - Source code to modify.
+* @returns Modified source, or null when a separate import line is needed.
+*/
+function addDefinePluginsSpecifier(source) {
+	const match = source.match(SDK_VALUE_IMPORT_REGEX);
+	if (!match) return null;
+	const updated = match[0].replace(/,?\s*\}/, ", definePlugins }");
+	return source.replace(match[0], updated);
+}
+function insertImports(source, importLines) {
+	const sdkImportRegex = /^import\s+.*from\s+["']@tailor-platform\/sdk[^"']*["'];?$/gm;
+	let lastMatch = null;
+	for (let match = sdkImportRegex.exec(source); match; match = sdkImportRegex.exec(source)) lastMatch = match;
+	const block = importLines.join("\n");
+	if (lastMatch) {
+		const insertPos = lastMatch.index + lastMatch[0].length;
+		return source.slice(0, insertPos) + "\n" + block + source.slice(insertPos);
+	}
+	return block + "\n" + source;
+}
+/**
+* Move `db.<namespace>.erdSite` entries in defineConfig() into a
+* `tailordbErdPlugin({ sites })` argument of definePlugins():
+*
+* 1. Remove each `erdSite` property from `db.<namespace>` objects of
+*    top-level defineConfig() calls (factory-wrapped configs are left for
+*    manual review, since their erdSite values may reference local bindings)
+* 2. Append `tailordbErdPlugin({ sites: { <namespace>: <value> } })` to the
+*    existing definePlugins() call (honoring an import alias), or add a
+*    `plugins` export when none exists
+* 3. Add the plugin import (and a definePlugins import when newly needed)
+* @param source - Source code to transform
+* @returns Transformed source or null if no changes needed
+*/
+function transform(source) {
+	if (!source.includes("erdSite") || !source.includes("@tailor-platform/sdk")) return null;
+	if (source.includes("tailordbErdPlugin")) return null;
+	const tree = parse(Lang.TypeScript, source).root();
+	const edits = [];
+	const siteEntries = [];
+	for (const call of tree.findAll({ rule: { pattern: "defineConfig($CONFIG)" } })) {
+		if (insideFunction(call)) continue;
+		const config = call.getMatch("CONFIG");
+		if (!config || config.kind() !== "object") continue;
+		const dbObject = config.children().find((child) => child.kind() === "pair" && propertyName(child) === "db")?.field("value");
+		if (!dbObject || dbObject.kind() !== "object") continue;
+		for (const nsPair of dbObject.children().filter((child) => child.kind() === "pair")) {
+			const nsKey = nsPair.field("key");
+			const nsObject = nsPair.field("value");
+			if (!nsKey || nsKey.kind() === "computed_property_name") continue;
+			if (!nsObject || nsObject.kind() !== "object") continue;
+			const erdPair = nsObject.children().find((child) => child.kind() === "pair" && propertyName(child) === "erdSite");
+			const valueNode = erdPair?.field("value");
+			if (!erdPair || !valueNode) continue;
+			siteEntries.push(`${nsKey.text()}: ${valueNode.text()}`);
+			edits.push(removePairEdit(nsObject, erdPair));
+		}
+	}
+	if (siteEntries.length === 0) return null;
+	const pluginExpr = `tailordbErdPlugin({ sites: { ${siteEntries.join(", ")} } })`;
+	const localDefinePlugins = definePluginsLocalName(tree);
+	const pluginsCall = tree.find({ rule: { pattern: `${localDefinePlugins ?? "definePlugins"}($$$ARGS)` } });
+	if (pluginsCall) edits.push(appendArgEdit(pluginsCall, pluginExpr));
+	let result = tree.commitEdits(edits);
+	const importLines = [PLUGIN_IMPORT];
+	if (!pluginsCall && !localDefinePlugins) {
+		const merged = addDefinePluginsSpecifier(result);
+		if (merged !== null) result = merged;
+		else importLines.push(DEFINE_PLUGINS_IMPORT);
+	}
+	result = insertImports(result, importLines);
+	if (!pluginsCall) result = result.replace(/\s*$/, "\n\n") + `export const plugins = ${localDefinePlugins ?? "definePlugins"}(\n  ${pluginExpr},\n);\n`;
+	return result;
+}
+//#endregion
+export { transform as default };

package/dist/codemods/v2/workflow-trigger-rename/scripts/transform.js

@@ -0,0 +1,122 @@
+import { c as localDeclarationNames, i as importBindings, r as findImportStatements } from "../../../ast-grep-helpers-CXtWn3RB.js";
+import { Lang, parse } from "@ast-grep/napi";
+//#region codemods/v2/workflow-trigger-rename/scripts/transform.ts
+const RENAMES = {
+	triggerWorkflow: "startWorkflow",
+	triggerJobFunction: "startJobFunction",
+	resumeWorkflow: "resumeWorkflowExecution"
+};
+const WORKFLOW_MODULE_SOURCES = /* @__PURE__ */ new Set(["@tailor-platform/sdk/runtime", "@tailor-platform/sdk/runtime/workflow"]);
+function quickFilter(source) {
+	return Object.keys(RENAMES).some((name) => source.includes(name));
+}
+function sourceLang(filePath, source) {
+	const lower = filePath.toLowerCase();
+	if (lower.endsWith(".tsx") || lower.endsWith(".jsx")) return Lang.Tsx;
+	return source.includes("</") || source.includes("/>") ? Lang.Tsx : Lang.TypeScript;
+}
+function collectWorkflowLocals(root, imports) {
+	const locals = /* @__PURE__ */ new Set();
+	for (const importStmt of imports) for (const binding of importBindings(importStmt)) if (binding.importedName === "workflow" && WORKFLOW_MODULE_SOURCES.has(binding.source)) locals.add(binding.localName);
+	const declaredNames = localDeclarationNames(root);
+	for (const name of locals) if (declaredNames.has(name)) locals.delete(name);
+	return locals;
+}
+function tailorIsAmbientGlobal(root, imports) {
+	if (localDeclarationNames(root).has("tailor")) return false;
+	return !imports.some((stmt) => importBindings(stmt).some((b) => b.localName === "tailor"));
+}
+function isAmbientTailorWorkflow(object) {
+	if (!object || object.kind() !== "member_expression") return false;
+	const base = object.field("object");
+	const property = object.field("property");
+	return base?.kind() === "identifier" && base.text() === "tailor" && property?.text() === "workflow";
+}
+function expressionArguments(args) {
+	return args.children().filter((child) => ![
+		"(",
+		")",
+		","
+	].includes(child.kind()));
+}
+function thirdArgument(member) {
+	const call = member.parent();
+	if (call?.kind() !== "call_expression") return null;
+	const args = call.field("arguments");
+	if (!args) return null;
+	return expressionArguments(args)[2] ?? null;
+}
+/**
+* A `triggerWorkflow(name, args, options)` call whose third argument is not a
+* literal object (e.g. a variable), or is an object literal that spreads one
+* (`{ ...rest }`), cannot be safely renamed: we cannot tell whether it carries
+* an `invoker` key that also needs renaming to `authInvoker`, and renaming
+* just the method name would erase the `triggerWorkflow` token that flags the
+* call for manual review, while the option silently stops working at
+* runtime. Such calls are left entirely unrenamed.
+* @param member - The `.triggerWorkflow` member-expression node being considered
+* @returns Whether the call must be skipped
+*/
+function hasUnsafeThirdArgument(member) {
+	const optionsArg = thirdArgument(member);
+	if (optionsArg === null) return false;
+	if (optionsArg.kind() !== "object") return true;
+	return optionsArg.children().some((child) => child.kind() === "spread_element");
+}
+/**
+* `triggerWorkflow`'s removed SDK wrapper converted an `invoker` option to
+* the platform's `authInvoker` shape before calling through; `startWorkflow`
+* expects `authInvoker` directly. Build an edit renaming a literal `invoker`
+* key (or shorthand) in the third argument of a `triggerWorkflow(...)` call
+* being renamed to `startWorkflow`, so the option keeps working.
+* @param member - The `.triggerWorkflow` member-expression node being renamed
+* @returns An edit renaming the `invoker` key, or null when not applicable
+*/
+function findInvokerOptionEdit(member) {
+	const optionsArg = thirdArgument(member);
+	if (!optionsArg || optionsArg.kind() !== "object") return null;
+	for (const child of optionsArg.children()) if (child.kind() === "pair") {
+		const key = child.field("key");
+		if (key?.kind() === "property_identifier" && key.text() === "invoker") return key.replace("authInvoker");
+	} else if (child.kind() === "shorthand_property_identifier" && child.text() === "invoker") return child.replace("authInvoker: invoker");
+	return null;
+}
+/**
+* Rewrite `.triggerWorkflow(`, `.triggerJobFunction(`, and `.resumeWorkflow(`
+* member accesses to their canonical `start*`/`resumeWorkflowExecution` names,
+* either on the ambient `tailor.workflow` global or on a `workflow` value
+* imported from `@tailor-platform/sdk/runtime(/workflow)`.
+* @param source - File contents
+* @param filePath - Absolute path to the file
+* @returns Transformed source or null when nothing matched.
+*/
+function transform(source, filePath) {
+	if (!quickFilter(source)) return null;
+	const root = parse(sourceLang(filePath ?? "", source), source).root();
+	const imports = findImportStatements(root);
+	const workflowLocals = collectWorkflowLocals(root, imports);
+	const tailorIsAmbient = tailorIsAmbientGlobal(root, imports);
+	const edits = [];
+	for (const member of root.findAll({ rule: { kind: "member_expression" } })) {
+		const property = member.field("property");
+		if (!property || property.kind() !== "property_identifier") continue;
+		const newName = RENAMES[property.text()];
+		if (!newName) continue;
+		const object = member.field("object");
+		if (!object) continue;
+		const isWorkflowImportReceiver = object.kind() === "identifier" && workflowLocals.has(object.text());
+		const isAmbientReceiver = tailorIsAmbient && isAmbientTailorWorkflow(object);
+		if (!isWorkflowImportReceiver && !isAmbientReceiver) continue;
+		if (newName === "startWorkflow") {
+			if (hasUnsafeThirdArgument(member)) continue;
+			edits.push(property.replace(newName));
+			const invokerEdit = findInvokerOptionEdit(member);
+			if (invokerEdit) edits.push(invokerEdit);
+		} else edits.push(property.replace(newName));
+	}
+	if (edits.length === 0) return null;
+	const result = root.commitEdits(edits);
+	return result === source ? null : result;
+}
+//#endregion
+export { transform as default };

package/dist/index.js

@@ -116,8 +116,11 @@ const RENAME_BIN_QUOTED_LEGACY_COMMAND_PATTERN = new RegExp([
 ].join(""));
 const V2_NEXT_1 = "2.0.0-next.1";
 const V2_NEXT_2 = "2.0.0-next.2";
-const V2_NEXT_3 = "2.0.0-next.3";
 const V2_NEXT_4 = "2.0.0-next.4";
+const V2_NEXT_5 = "2.0.0-next.5";
+const V2_NEXT_6 = "2.0.0-next.6";
+const V2_NEXT_7 = "2.0.0-next.7";
+const V2_NEXT_9 = "2.0.0-next.9";
 /** All registered codemods, in registration order. */
 const allCodemods = [
 	{
@@ -417,7 +420,7 @@ const allCodemods = [
 	{
 		id: "v2/auth-invoker-unwrap",
 		name: "auth.invoker(\"name\") → invoker: \"name\"",
-		description: "Rename statically identified SDK `authInvoker` options to `invoker`, replace `auth.invoker(\"name\")` there with the bare `\"name\"` string, and drop the `auth` import when no other reference remains. Ambiguous workflow `.trigger()` calls are left for manual review. The `auth.invoker()` helper is removed in v2 because importing `auth` from `tailor.config.ts` into runtime files pulls Node-only modules into the bundle.",
+		description: "Rename statically identified SDK `authInvoker` options to `invoker`, replace `auth.invoker(\"name\")` there with the bare `\"name\"` string, and drop the `auth` import when no other reference remains. Ambiguous workflow `.start()` calls are left for manual review. The `auth.invoker()` helper is removed in v2 because importing `auth` from `tailor.config.ts` into runtime files pulls Node-only modules into the bundle.",
 		since: "1.0.0",
 		until: "2.0.0",
 		prereleaseUntil: V2_NEXT_2,
@@ -450,8 +453,8 @@ const allCodemods = [
 			"   machine user name string; platform/runtime authInvoker payloads still expect",
 			"   the object form.",
 			"2. Rename remaining authInvoker option keys to invoker only for SDK resolver,",
-			"   executor, workflow.trigger(), or startWorkflow() options. Keep platform/runtime",
-			"   payload keys such as tailor.workflow.triggerWorkflow(..., { authInvoker: ... }).",
+			"   executor, workflow.start(), or startWorkflow() options. Keep platform/runtime",
+			"   payload keys such as tailor.workflow.startWorkflow(..., { authInvoker: ... }).",
 			"3. After removing every auth.invoker usage in a file, delete the now-unused auth",
 			"   import (keeping it pulls Node-only config modules into runtime bundles); leave",
 			"   the import if auth is still referenced elsewhere.",
@@ -502,7 +505,7 @@ const allCodemods = [
 		description: "Rewrite `@tailor-platform/sdk/runtime/*` namespace-star and flat value imports to self-named namespace imports, and aggregate `file.deleteFile` calls to `file.delete`. `TailorContextAPI` and `TailorWorkflowAPI` now describe SDK wrappers; direct platform globals use `PlatformContextAPI` and `PlatformWorkflowAPI`.",
 		since: "1.0.0",
 		until: "2.0.0",
-		prereleaseUntil: V2_NEXT_3,
+		prereleaseUntil: V2_NEXT_4,
 		scriptPath: "v2/runtime-subpath-namespace/scripts/transform.js",
 		filePatterns: ["**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}"],
 		legacyPatterns: [
@@ -574,7 +577,7 @@ const allCodemods = [
 		description: "Rename TailorDB schema builder calls from `db.type()` to `db.table()`. TailorDB schema definitions now use table terminology in SDK projects.",
 		since: "1.0.0",
 		until: "2.0.0",
-		prereleaseUntil: V2_NEXT_3,
+		prereleaseUntil: V2_NEXT_4,
 		scriptPath: "v2/db-type-to-table/scripts/transform.js",
 		filePatterns: ["**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}"],
 		legacyPatterns: ["db.type"],
@@ -600,7 +603,7 @@ const allCodemods = [
 		description: "Review TailorDB relations that omit `toward.as`. Their forward GraphQL field names now derive from the relation field name with a trailing `ID`, `Id`, or `id` removed, instead of from the target table name.",
 		since: "1.0.0",
 		until: "2.0.0",
-		prereleaseUntil: V2_NEXT_4,
+		prereleaseUntil: V2_NEXT_5,
 		scriptPath: "v2/forward-relation-name/scripts/transform.js",
 		filePatterns: ["**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}"],
 		suspiciousPatterns: [
@@ -686,6 +689,54 @@ const allCodemods = [
 			after: "import { createWaitPoints } from \"@tailor-platform/sdk\";\n\nexport const { approval } = createWaitPoints((define) => ({\n  approval: define<{ message: string }, { approved: boolean }>(),\n}));"
 		}]
 	},
+	{
+		id: "v2/workflow-trigger-rename",
+		name: "workflow.triggerWorkflow/triggerJobFunction/resumeWorkflow → startWorkflow/startJobFunction/resumeWorkflowExecution",
+		description: "Rename tailor.workflow call sites from the pre-alignment triggerWorkflow/triggerJobFunction/resumeWorkflow names to the canonical startWorkflow/startJobFunction/resumeWorkflowExecution names, on both the ambient tailor.workflow global and a workflow value imported from @tailor-platform/sdk/runtime(/workflow). For a renamed triggerWorkflow call, also renames a literal `invoker` option key to `authInvoker` — startWorkflow's options expect the platform shape directly, unlike the removed triggerWorkflow wrapper, which converted invoker to authInvoker internally.",
+		since: "1.0.0",
+		until: "2.0.0",
+		prereleaseUntil: V2_NEXT_6,
+		scriptPath: "v2/workflow-trigger-rename/scripts/transform.js",
+		filePatterns: ["**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}"],
+		legacyPatterns: [
+			"triggerWorkflow",
+			"triggerJobFunction",
+			"resumeWorkflow"
+		],
+		examples: [{
+			before: "import { workflow } from \"@tailor-platform/sdk/runtime\";\n\nawait workflow.triggerWorkflow(\"myWorkflow\", { data: \"value\" });",
+			after: "import { workflow } from \"@tailor-platform/sdk/runtime\";\n\nawait workflow.startWorkflow(\"myWorkflow\", { data: \"value\" });"
+		}, {
+			caption: "A literal invoker option is renamed to authInvoker:",
+			before: "await workflow.triggerWorkflow(\"myWorkflow\", { data: \"value\" }, { invoker: myInvoker });",
+			after: "await workflow.startWorkflow(\"myWorkflow\", { data: \"value\" }, { authInvoker: myInvoker });"
+		}],
+		prompt: [
+			"The pre-alignment tailor.workflow names triggerWorkflow, triggerJobFunction, and",
+			"resumeWorkflow are removed from the SDK's type surface in v2; use the canonical",
+			"startWorkflow, startJobFunction, and resumeWorkflowExecution names instead. The",
+			"codemod rewrites direct member-access call sites on the ambient tailor.workflow",
+			"global and on a workflow value imported from @tailor-platform/sdk/runtime or",
+			"@tailor-platform/sdk/runtime/workflow (including aliased imports). It skips a",
+			"file entirely when a local declaration shadows the workflow import or the",
+			"ambient tailor name, to avoid rewriting an unrelated same-named value — review",
+			"those manually.",
+			"",
+			"For a renamed triggerWorkflow call, the codemod also renames a literal invoker",
+			"option key (including shorthand { invoker }) to authInvoker, since startWorkflow",
+			"expects the platform's authInvoker shape directly while triggerWorkflow's removed",
+			"wrapper converted invoker to authInvoker internally.",
+			"",
+			"Also review, and migrate by hand:",
+			"- Destructured references (e.g. const { triggerWorkflow } = workflow) — the",
+			"  codemod only rewrites direct member-access calls.",
+			"- Imported TriggerWorkflowOptions / TriggerJobFunctionOptions types — rename",
+			"  them to StartWorkflowOptions / StartJobFunctionOptions.",
+			"- An invoker option passed via a variable or spread (not a literal object) —",
+			"  the codemod only inspects literal object arguments; rename the invoker key",
+			"  to authInvoker in the options object's own definition."
+		].join("\n")
+	},
 	{
 		id: "v2/open-download-stream",
 		name: "openDownloadStream → downloadStream",
@@ -781,23 +832,57 @@ const allCodemods = [
 	},
 	{
 		id: "v2/workflow-trigger-dispatch",
-		name: "Workflow .trigger() and trigger tests",
-		description: "Workflow job `.trigger()` now aligns with the platform runtime: it returns the job result directly instead of a Promise wrapper, and tests no longer run job bodies locally. Mock trigger responses with `mockWorkflow()` (`setJobHandler` / `enqueueResult`, assert via `triggeredJobs`), or use `runWorkflowLocally()` for a full-chain local run.",
+		name: "Workflow job start() and start tests",
+		description: "Workflow job `.start()` (previously `.trigger()`) now aligns with the platform runtime: it returns the job result directly instead of a Promise wrapper, and tests no longer run job bodies locally. Mock start responses with `mockWorkflow()` (`setJobHandler` / `enqueueResult`, assert via `startedJobs`), or use `runWorkflowLocally()` for a full-chain local run.",
 		since: "1.0.0",
 		until: "2.0.0",
 		prereleaseUntil: V2_NEXT_1,
 		suspiciousPatterns: [".trigger("],
 		examples: [{
 			caption: "Tests must mock the workflow runtime instead of running bodies locally:",
-			before: "const result = await orderJob.trigger({ id });\nexpect(result.status).toBe(\"done\");",
-			after: "using wf = mockWorkflow();\nwf.setJobHandler((jobName) => (jobName === \"order-job\" ? { status: \"done\" } : null));\nconst result = await orderJob.trigger({ id });\nexpect(result.status).toBe(\"done\");"
+			before: "const result = await orderJob.start({ id });\nexpect(result.status).toBe(\"done\");",
+			after: "using wf = mockWorkflow();\nwf.setJobHandler((jobName) => (jobName === \"order-job\" ? { status: \"done\" } : null));\nconst result = await orderJob.start({ id });\nexpect(result.status).toBe(\"done\");"
 		}],
 		prompt: [
-			"Workflow job .trigger() now uses the platform workflow runtime instead of running",
+			"Workflow job .start() now uses the platform workflow runtime instead of running",
 			"the job body locally. In tests, acquire `using wf = mockWorkflow()` and provide",
-			"trigger responses (setJobHandler / enqueueResult), or use runWorkflowLocally() for a",
-			"full-chain local run; an unmocked trigger now throws. Outside tests, treat the",
-			"trigger result as the job output directly (no Promise wrapper to unwrap)."
+			"start responses (setJobHandler / enqueueResult), or use runWorkflowLocally() for a",
+			"full-chain local run; an unmocked start now throws. Outside tests, treat the",
+			"start result as the job output directly (no Promise wrapper to unwrap)."
+		].join("\n")
+	},
+	{
+		id: "v2/workflow-start-rename",
+		name: "Workflow.trigger()/WorkflowJob.trigger() → .start()",
+		description: "Rename `Workflow.trigger()` (returned by `createWorkflow()`) and `WorkflowJob.trigger()` (returned by `createWorkflowJob()`) to `.start()`, aligning the SDK's ergonomic verb with the platform's `start*` RPC vocabulary. No codemod ships for this rename: distinguishing a workflow/job `.trigger()` call from an unrelated object's own `.trigger()` method requires resolving the receiver back to a `createWorkflow`/`createWorkflowJob` result across files, which the SDK's own CLI bundler already does for build-time rewriting. Reusing that logic in a standalone script is a nontrivial lift, and — unlike the bundler, which fails loudly when it cannot rewrite a call — a codemod false positive would silently rewrite an unrelated `.trigger()` call with no error. For the call-site volume this rename typically involves, manual review guided by the prompt below is the safer trade-off.",
+		since: "1.0.0",
+		until: "2.0.0",
+		prereleaseUntil: V2_NEXT_7,
+		filePatterns: ["**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}"],
+		suspiciousPatterns: [".trigger("],
+		examples: [{
+			before: ["const inventory = checkInventory.trigger({ orderId: input.orderId });", "const workflowRunId = await orderProcessingWorkflow.trigger(args, { invoker: \"manager\" });"].join("\n"),
+			after: ["const inventory = checkInventory.start({ orderId: input.orderId });", "const workflowRunId = await orderProcessingWorkflow.start(args, { invoker: \"manager\" });"].join("\n")
+		}],
+		prompt: [
+			"In Tailor SDK v2, the ergonomic .trigger() method on a createWorkflow() or",
+			"createWorkflowJob() result is renamed to .start(). This is unrelated to the",
+			"separate tailor.workflow.triggerWorkflow/triggerJobFunction/resumeWorkflow removal",
+			"(see the workflow-trigger-rename codemod) — this rename targets the SDK's own",
+			"ergonomic wrapper, not the low-level platform call.",
+			"",
+			"For each flagged `.trigger(` call in these files:",
+			"1. Confirm the receiver is a workflow or job object — typically a local const",
+			"   assigned from createWorkflow(...)/createWorkflowJob(...), a named import of one,",
+			"   or the default import of a workflow module. Skip receivers that are unrelated",
+			"   objects with their own .trigger() method (state machines, event emitters, etc.).",
+			"2. Rename the call from .trigger(...) to .start(...); the argument list is unchanged.",
+			"3. Update any mock/test code that reads WorkflowJob['trigger'] / Workflow['trigger']",
+			"   as a type, or that mocks the ergonomic method via a wrapper — for example,",
+			"   `wf.job(definition)` / `wf.workflow(definition)` from mockWorkflow() now return a",
+			"   mock of the `.start` method.",
+			"4. Update prose/docs/comments that say \"trigger the workflow/job\" to \"start\" only",
+			"   where they describe this SDK verb specifically, not unrelated event terminology."
 		].join("\n")
 	},
 	{
@@ -898,7 +983,7 @@ const allCodemods = [
 		description: "Field-level `ValidateFn` is simplified from `(args: { value, data, invoker }) => boolean` to `(args: { value }) => string | void` — the function now returns the error message directly instead of a separate `[fn, message]` tuple. The `ValidateConfig` tuple form and `Validators<F>` record syntax on `db.type().validate()` are removed. Type-level validation uses `db.type().validate((args, issues) => void)` with `{ newRecord, oldRecord, invoker }` args and an `issues(field, message)` callback for cross-field rules.",
 		since: "1.0.0",
 		until: "2.0.0",
-		prereleaseUntil: V2_NEXT_4,
+		prereleaseUntil: V2_NEXT_5,
 		suspiciousPatterns: [
 			"ValidateConfig",
 			"Validators<",
@@ -944,7 +1029,7 @@ const allCodemods = [
 		description: "Field-level `HookFn` args change from `{ value, data, invoker }` to create `{ input, invoker, now }` / update `{ input, oldValue, invoker, now }` — `value` is renamed to `input`, matching the `input` arg on type-level hooks (same pre-hook data, narrowed to one field); `data` (the full record) is removed; `oldValue` (previous field value) is added for update hooks only; `now` (operation timestamp) is shared across all hooks. Type-level hooks on `db.type().hooks()` change from per-field mapping `{ fieldName: { create, update } }` (`Hooks<F>`) to a single `{ create, update }` object (`TypeHook<F>`) — create hooks take `{ input, invoker, now }`, update hooks take `{ input, oldRecord, invoker, now }` (oldRecord is always non-null). Both return partial field overrides.",
 		since: "1.0.0",
 		until: "2.0.0",
-		prereleaseUntil: V2_NEXT_4,
+		prereleaseUntil: V2_NEXT_5,
 		suspiciousPatterns: [
 			"Hooks<",
 			"HookFn<",
@@ -988,13 +1073,80 @@ const allCodemods = [
 			"3. Remove unused `Hooks<F>` / `HookFn<>` type imports"
 		].join("\n")
 	},
+	{
+		id: "v2/erd-site-to-plugin",
+		name: "`db.<namespace>.erdSite` → `tailordbErdPlugin({ sites })`",
+		description: "Move the TailorDB `erdSite` setting from `db.<namespace>` in tailor.config.ts into `tailordbErdPlugin({ sites })` from `@tailor-platform/sdk-plugin-tailordb-erd`, registered via definePlugins(). The core config schema no longer accepts `erdSite`; the `tailor tailordb erd` commands read the target static website from the plugin configuration and validate each site name against `staticWebsites`. Install `@tailor-platform/sdk-plugin-tailordb-erd` as a dev dependency: the migrated config imports it, so config loading fails with a module-not-found error until it is installed.",
+		since: "1.0.0",
+		until: "2.0.0",
+		prereleaseUntil: V2_NEXT_9,
+		scriptPath: "v2/erd-site-to-plugin/scripts/transform.js",
+		legacyPatterns: ["erdSite:"],
+		sourceStringLegacyPatterns: ["erdSite"],
+		suspiciousPatterns: [
+			"erdSite:",
+			/\berdSite\s*[,}]/,
+			"tailordbErdPlugin"
+		],
+		sourceStringSuspiciousPatterns: ["erdSite"],
+		examples: [{
+			before: [
+				"export default defineConfig({",
+				"  db: {",
+				"    tailordb: {",
+				"      files: [\"./tailordb/*.ts\"],",
+				"      erdSite: \"my-erd-site\",",
+				"    },",
+				"  },",
+				"});"
+			].join("\n"),
+			after: [
+				"import { tailordbErdPlugin } from \"@tailor-platform/sdk-plugin-tailordb-erd\";",
+				"",
+				"export default defineConfig({",
+				"  db: {",
+				"    tailordb: {",
+				"      files: [\"./tailordb/*.ts\"],",
+				"    },",
+				"  },",
+				"});",
+				"",
+				"export const plugins = definePlugins(",
+				"  tailordbErdPlugin({ sites: { tailordb: \"my-erd-site\" } }),",
+				");"
+			].join("\n")
+		}],
+		prompt: [
+			"In Tailor SDK v2 the TailorDB `erdSite` setting is removed from the core config",
+			"schema; the ERD deploy target is configured on the ERD CLI plugin instead. The",
+			"codemod rewrites literal `db.<namespace>.erdSite` entries inside top-level",
+			"defineConfig() calls into a `tailordbErdPlugin({ sites: { <namespace>: <value> } })`",
+			"argument of definePlugins(), importing it from @tailor-platform/sdk-plugin-tailordb-erd.",
+			"",
+			"First, for every config that now registers tailordbErdPlugin, make sure",
+			"@tailor-platform/sdk-plugin-tailordb-erd is installed as a dev dependency — the",
+			"migrated config imports it, so config loading fails with ERR_MODULE_NOT_FOUND",
+			"until it is installed.",
+			"",
+			"For any remaining `erdSite` config keys the codemod did not rewrite — a db config",
+			"built dynamically or passed via a variable, quoted or computed keys, spread",
+			"properties, a defineConfig() call inside a factory function, or a file that",
+			"already registers tailordbErdPlugin — move the namespace → static-website-name",
+			"mapping into tailordbErdPlugin({ sites }) and delete the `erdSite` key. For",
+			"factory-built configs, keep any referenced parameters or locals in scope when",
+			"moving the value to the module-level definePlugins() export. Each site name",
+			"must match a static website defined in staticWebsites. Leave unrelated",
+			"identifiers that merely contain the name (e.g. a defineStaticWebSite variable",
+			"named erdSite) unchanged."
+		].join("\n")
+	},
 	{
 		id: "v2/generate-watch-flag",
 		name: "generate --watch flag removed",
 		description: "Review and remove `tailor generate --watch` / `-W` invocations and the `watch` option on `GenerateOptions`. The flag, its dependency watcher, and the self-restart-on-change logic are removed; `generate` now always performs a single generation pass.",
 		since: "1.0.0",
 		until: "2.0.0",
-		prereleaseUntil: "2.0.0-next.6",
+		prereleaseUntil: V2_NEXT_6,
 		filePatterns: [
 			"**/package.json",
 			"**/*.{sh,bash,zsh,yml,yaml}",
@@ -1024,6 +1176,36 @@ const allCodemods = [
 			"single generation pass and resolves once it completes."
 		].join("\n")
 	},
+	{
+		id: "v2/seed-exec-to-cli-plugin",
+		name: "Generated seed exec.mjs → tailor seed CLI plugin",
+		description: "`seedPlugin` no longer generates the `exec.mjs` seed runner. Seeding and validation move to the `tailor seed` commands provided by the `@tailor-platform/sdk-plugin-seed` CLI plugin: install it as a devDependency, replace `node <distPath>/exec.mjs` invocations with `tailor seed apply` and `node <distPath>/exec.mjs validate` with `tailor seed validate`, and delete the stale generated `<distPath>/exec.mjs` file. Seed data and schema generation (`data/*.jsonl`, `data/*.schema.ts`) is unchanged, and the `tailor seed apply` options mirror the old script (`--machine-user`, `--namespace`, `--skip-idp`, `--truncate`, `--yes`, type-name arguments).",
+		since: "1.0.0",
+		until: "2.0.0",
+		prereleaseUntil: V2_NEXT_9,
+		filePatterns: ["**/package.json", "**/*.{sh,yml,yaml,md,mjs,ts}"],
+		suspiciousPatterns: ["exec.mjs"],
+		sourceStringSuspiciousPatterns: ["exec.mjs"],
+		examples: [{
+			before: "\"seed\": \"node ./seed/exec.mjs\",\n\"seed:validate\": \"node ./seed/exec.mjs validate\"",
+			after: "\"seed\": \"tailor seed apply\",\n\"seed:validate\": \"tailor seed validate\"",
+			lang: "jsonc"
+		}],
+		prompt: [
+			"seedPlugin no longer generates the exec.mjs seed runner in v2. The tailor seed",
+			"CLI plugin (@tailor-platform/sdk-plugin-seed) replaces it:",
+			"",
+			"- Install @tailor-platform/sdk-plugin-seed as a devDependency next to",
+			"  @tailor-platform/sdk.",
+			"- Replace `node <distPath>/exec.mjs [options] [types...]` invocations with",
+			"  `tailor seed apply [options] [types...]` (same options: --machine-user/-m,",
+			"  --namespace/-n, --skip-idp, --truncate, --yes, and type-name arguments).",
+			"- Replace `node <distPath>/exec.mjs validate [path]` with",
+			"  `tailor seed validate [path]`.",
+			"- Delete the stale generated `<distPath>/exec.mjs` file; keep the data/",
+			"  directory (JSONL data and generated schemas) as-is."
+		].join("\n")
+	},
 	{
 		id: "v2/node-minimum-22-15-0",
 		name: "Node.js minimum version raised to 22.15.0",
@@ -1031,6 +1213,14 @@ const allCodemods = [
 		since: "1.0.0",
 		until: "2.0.0",
 		notice: true
+	},
+	{
+		id: "v2/remove-legacy-bundle-cleanup",
+		name: "Legacy bundle artifact cleanup removed from deploy",
+		description: "`tailor deploy` no longer deletes on-disk bundle artifacts (`.entry.js` files, workflow-job bundles, and the `hooks-validate-scripts/` directory) left in the SDK output directory (`.tailor` by default) by SDK versions that predate the current in-memory bundling approach. Current bundlers no longer write these files. No source change is required; if such stale files remain from a very old SDK version, delete only those specific files/directories manually — do not delete the output directory itself, since it also holds deploy state (e.g. `secrets-state/`, `*.context.json`) that existing secrets and Auth Connections depend on.",
+		since: "1.0.0",
+		until: "2.0.0",
+		notice: true
 	}
 ];
 /**
@@ -1043,12 +1233,13 @@ function resolveCodemodScript(scriptPath) {
 }
 function reachesCodemodBoundary(toVersion, codemod) {
 	if (gte(toVersion, codemod.until)) return true;
-	if (codemod.prereleaseUntil === void 0 || !gte(toVersion, codemod.prereleaseUntil)) return false;
+	if (codemod.prereleaseUntil === void 0 || codemod.prereleaseUntil === "pending" || !gte(toVersion, codemod.prereleaseUntil)) return false;
 	const target = parse(toVersion);
 	const boundary = parse(codemod.until);
 	return target.prerelease.length > 0 && target.major === boundary.major && target.minor === boundary.minor && target.patch === boundary.patch;
 }
 function effectiveCodemodBoundary(codemod) {
+	if (codemod.prereleaseUntil === "pending") return codemod.until;
 	return codemod.prereleaseUntil ?? codemod.until;
 }
 function assertCodemodBoundaries(codemods) {
@@ -1056,7 +1247,7 @@ function assertCodemodBoundaries(codemods) {
 		const boundary = parse(codemod.until);
 		if (boundary === null) throw new Error(`Codemod ${codemod.id} until must be a valid semver version: ${codemod.until}`);
 		if (boundary.prerelease.length > 0) throw new Error(`Codemod ${codemod.id} until must be a stable version: ${codemod.until}`);
-		if (codemod.prereleaseUntil === void 0) continue;
+		if (codemod.prereleaseUntil === void 0 || codemod.prereleaseUntil === "pending") continue;
 		const prereleaseBoundary = parse(codemod.prereleaseUntil);
 		if (prereleaseBoundary === null) throw new Error(`Codemod ${codemod.id} prereleaseUntil must be a valid semver version: ${codemod.prereleaseUntil}`);
 		if (prereleaseBoundary.prerelease.length === 0) throw new Error(`Codemod ${codemod.id} prereleaseUntil must be a prerelease version: ${codemod.prereleaseUntil}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tailor-platform/sdk-codemod",
-  "version": "0.3.0-next.6",
+  "version": "0.3.0-next.8",
   "description": "Codemod runner for Tailor Platform SDK upgrades",
   "license": "MIT",
   "repository": {
@@ -32,7 +32,7 @@
     "@types/semver": "7.7.1",
     "eslint-plugin-zod": "4.7.0",
     "oxlint": "1.73.0",
-    "tsdown": "0.22.5",
+    "tsdown": "0.22.7",
     "typescript": "6.0.3",
     "vitest": "4.1.10"
   },