@tailor-platform/sdk-codemod 0.4.2 → 0.6.0

package/CHANGELOG.md

@@ -1,5 +1,29 @@
 # @tailor-platform/sdk-codemod
 
+## 0.6.0
+
+### Minor Changes
+
+- [#2075](https://github.com/tailor-platform/sdk/pull/2075) [`bd0e397`](https://github.com/tailor-platform/sdk/commit/bd0e39720015248e6ebb2c31efca49f9238b7060) Thanks [@dqn](https://github.com/dqn)! - `tailor function test-run` is renamed to `tailor function run`. The old name keeps working as a deprecated alias until v3 and prints a deprecation warning when used; `tailor upgrade` offers the `v3/function-test-run-rename` codemod to rewrite `function test-run` invocations across package.json scripts, shell and Windows scripts, YAML, Markdown, and JavaScript/TypeScript sources.
+
+### Patch Changes
+
+- [#2031](https://github.com/tailor-platform/sdk/pull/2031) [`ecc18ee`](https://github.com/tailor-platform/sdk/commit/ecc18eebc4e5c03c78e362a9b17bacc31bc88a31) Thanks [@renovate](https://github.com/apps/renovate)! - fix(deps): update dependency @ast-grep/napi to v0.45.1
+
+- [#2065](https://github.com/tailor-platform/sdk/pull/2065) [`5a3a0e1`](https://github.com/tailor-platform/sdk/commit/5a3a0e1ce8bfadb769dc4540ef257e944f0c077e) Thanks [@toiroakr](https://github.com/toiroakr)! - Raise the minimum supported Node.js version to 22.18.0 (from 22.15.0).
+  
+  `tailor seed validate` crashed on Node 22.15.0–22.17.x with `Expected a string, an ArrayBuffer, or a TypedArray to be returned for the "source" from the "load" hook but got null`. This is a Node.js bug ([nodejs/node#58607](https://github.com/nodejs/node/issues/58607)): requiring a `node:`-scheme-only builtin (`node:sqlite`, used internally by the seed validator) while both a synchronous `resolve` and `load` hook are registered via `module.registerHooks()` crashes the loader on those versions. The SDK always registers both hooks, so any project on Node 22.15.0–22.17.x hit this. Node fixed it upstream in 22.18.0 ([nodejs/node#58612](https://github.com/nodejs/node/pull/58612)); this release raises `engines.node` to match, since Node 22.15.0–22.17.x never actually supported `tailor seed validate`.
+
+## 0.5.0
+
+### Minor Changes
+
+- [#2023](https://github.com/tailor-platform/sdk/pull/2023) [`9dc826f`](https://github.com/tailor-platform/sdk/commit/9dc826fc0a83aa926de5b07245513f67c1ace877) Thanks [@dqn](https://github.com/dqn)! - Guide users through the v2 type-only import requirement. The CLI loads TypeScript by stripping types from each file in isolation, so a plain import of a type-only export fails at load time with `SyntaxError: ... does not provide an export named '<name>'` and no indication that the import form is the cause.
+  
+  - The CLI now appends a suggestion to that error: import the name with `import type` and set `"verbatimModuleSyntax": true` in tsconfig.json to catch violations at typecheck.
+  - Projects scaffolded by `tailor init` now enable `verbatimModuleSyntax` in their tsconfig.json, so new projects catch violations at typecheck instead of at load time.
+  - The v2 migration guide gains a `v2/type-only-imports` entry documenting the requirement, the failure mode, and the migration steps, offered by `tailor upgrade` when crossing the v2 boundary.
+
 ## 0.4.2
 
 ### Patch Changes

package/dist/codemods/v2/auth-attributes-rename/scripts/transform.js

@@ -197,7 +197,8 @@ function rewriteImportTypeReferences(root, edits, editedRanges) {
 function transform(source, _filePath) {
 	if (!quickFilter(source)) return null;
 	const filePath = _filePath?.toLowerCase();
-	const root = parse(filePath?.endsWith(".tsx") || filePath?.endsWith(".jsx") ? Lang.Tsx : filePath ? Lang.TypeScript : source.includes("</") || source.includes("/>") ? Lang.Tsx : Lang.TypeScript, source).root();
+	const lang = filePath?.endsWith(".tsx") || filePath?.endsWith(".jsx") ? Lang.Tsx : filePath ? Lang.TypeScript : source.includes("</") || source.includes("/>") ? Lang.Tsx : Lang.TypeScript;
+	const root = parse(lang, source).root();
 	const edits = [];
 	const editedRanges = /* @__PURE__ */ new Set();
 	const { localTypeRenames, namespaceNames } = collectSdkImports(root, edits, editedRanges);

package/dist/codemods/v2/auth-invoker-unwrap/scripts/transform.js

@@ -224,7 +224,8 @@ function renameQuotedKey(node) {
 function transformAuthInvoker(source, _filePath, options = {}) {
 	if (!quickFilter(source)) return null;
 	const renameOptionKeys = options.renameOptionKeys ?? true;
-	const root = parse(source.includes("</") || source.includes("/>") ? Lang.Tsx : Lang.TypeScript, source).root();
+	const lang = source.includes("</") || source.includes("/>") ? Lang.Tsx : Lang.TypeScript;
+	const root = parse(lang, source).root();
 	const calls = findInvokerCalls(root).filter((c) => isSupportedInvokerValueCall(c.callNode));
 	const edits = calls.map((c) => c.callNode.replace(c.argText));
 	if (renameOptionKeys) {

package/dist/codemods/v2/execute-script-arg/scripts/transform.js

@@ -44,7 +44,8 @@ function isExecuteScriptArg(stringifyCall) {
 */
 function transform(source, _filePath) {
 	if (!quickFilter(source)) return null;
-	const root = parse(source.includes("</") || source.includes("/>") ? Lang.Tsx : Lang.TypeScript, source).root();
+	const lang = source.includes("</") || source.includes("/>") ? Lang.Tsx : Lang.TypeScript;
+	const root = parse(lang, source).root();
 	const edits = [];
 	for (const match of root.findAll({ rule: { pattern: "JSON.stringify($X)" } })) {
 		if (!isExecuteScriptArg(match)) continue;

package/dist/codemods/v2/plugin-cli-import/scripts/transform.js

@@ -43,7 +43,8 @@ function hasNonNamedBinding(importStmt) {
 */
 function transform(source, _filePath) {
 	if (!source.includes(CLI_MODULE)) return null;
-	const root = parse(source.includes("</") || source.includes("/>") ? Lang.Tsx : Lang.TypeScript, source).root();
+	const lang = source.includes("</") || source.includes("/>") ? Lang.Tsx : Lang.TypeScript;
+	const root = parse(lang, source).root();
 	const edits = [];
 	const importStmts = root.findAll({ rule: {
 		kind: "import_statement",

package/dist/codemods/v2/principal-unify/scripts/transform.js

@@ -699,18 +699,19 @@ function transformResolverBody(arrowNode, edits, parseContext) {
 		const principalAliasBindings = [];
 		for (const child of pattern.children()) {
 			const kind = child.kind();
-			if (kind === "shorthand_property_identifier_pattern" && child.text() === "user") if (aliasRenamedUser) {
-				edits.push(child.replace("caller: user"));
-				aliasedShorthandUser = true;
-				principalAliasBindings.push({
-					name: "user",
-					bindingStart: child.range().start.index
-				});
-			} else {
-				edits.push(child.replace("caller"));
-				renamedShorthandUser = true;
-			}
-			else if (kind === "pair_pattern") {
+			if (kind === "shorthand_property_identifier_pattern" && child.text() === "user") {
+				if (aliasRenamedUser) {
+					edits.push(child.replace("caller: user"));
+					aliasedShorthandUser = true;
+					principalAliasBindings.push({
+						name: "user",
+						bindingStart: child.range().start.index
+					});
+				} else {
+					edits.push(child.replace("caller"));
+					renamedShorthandUser = true;
+				}
+			} else if (kind === "pair_pattern") {
 				const key = child.field("key");
 				if (key && key.text() === "user") {
 					edits.push(key.replace("caller"));
@@ -722,16 +723,18 @@ function transformResolverBody(arrowNode, edits, parseContext) {
 				}
 			} else if (kind === "object_assignment_pattern") {
 				const inner = child.children().find((c) => c.kind() === "shorthand_property_identifier_pattern");
-				if (inner && inner.text() === "user") if (aliasRenamedUser) {
-					edits.push(inner.replace("caller: user"));
-					aliasedShorthandUser = true;
-					principalAliasBindings.push({
-						name: "user",
-						bindingStart: inner.range().start.index
-					});
-				} else {
-					edits.push(inner.replace("caller"));
-					renamedShorthandUser = true;
+				if (inner && inner.text() === "user") {
+					if (aliasRenamedUser) {
+						edits.push(inner.replace("caller: user"));
+						aliasedShorthandUser = true;
+						principalAliasBindings.push({
+							name: "user",
+							bindingStart: inner.range().start.index
+						});
+					} else {
+						edits.push(inner.replace("caller"));
+						renamedShorthandUser = true;
+					}
 				}
 			}
 		}
@@ -1061,33 +1064,36 @@ function transformPrincipalCallbackParam(fn, edits, typeContext) {
 	let renamedShorthandUser = false;
 	for (const child of pattern.children()) {
 		const kind = child.kind();
-		if (kind === "shorthand_property_identifier_pattern" && child.text() === "user") if (aliasRenamedUser) {
-			edits.push(child.replace("invoker: user"));
-			aliasedShorthandUser = true;
-			principalAliasBindings.push({
-				name: "user",
-				bindingStart: child.range().start.index
-			});
-		} else {
-			edits.push(child.replace("invoker"));
-			renamedShorthandUser = true;
-		}
-		else if (kind === "pair_pattern") {
-			const key = child.field("key");
-			if (key?.text() === "user") edits.push(key.replace("invoker"));
-		} else if (kind === "object_assignment_pattern") {
-			const inner = child.children().find((c) => c.kind() === "shorthand_property_identifier_pattern");
-			if (inner?.text() === "user") if (aliasRenamedUser) {
-				edits.push(inner.replace("invoker: user"));
+		if (kind === "shorthand_property_identifier_pattern" && child.text() === "user") {
+			if (aliasRenamedUser) {
+				edits.push(child.replace("invoker: user"));
 				aliasedShorthandUser = true;
 				principalAliasBindings.push({
 					name: "user",
-					bindingStart: inner.range().start.index
+					bindingStart: child.range().start.index
 				});
 			} else {
-				edits.push(inner.replace("invoker"));
+				edits.push(child.replace("invoker"));
 				renamedShorthandUser = true;
 			}
+		} else if (kind === "pair_pattern") {
+			const key = child.field("key");
+			if (key?.text() === "user") edits.push(key.replace("invoker"));
+		} else if (kind === "object_assignment_pattern") {
+			const inner = child.children().find((c) => c.kind() === "shorthand_property_identifier_pattern");
+			if (inner?.text() === "user") {
+				if (aliasRenamedUser) {
+					edits.push(inner.replace("invoker: user"));
+					aliasedShorthandUser = true;
+					principalAliasBindings.push({
+						name: "user",
+						bindingStart: inner.range().start.index
+					});
+				} else {
+					edits.push(inner.replace("invoker"));
+					renamedShorthandUser = true;
+				}
+			}
 		}
 	}
 	if (!renamedShorthandUser) {

package/dist/codemods/v2/wait-point-rename/scripts/transform.js

@@ -24,7 +24,8 @@ function isInsideImportStatement(node) {
 function transform(source, _filePath) {
 	if (!Object.keys(RENAMES).some((name) => source.includes(name))) return null;
 	if (!source.includes(SDK_MODULE)) return null;
-	const root = parse(source.includes("</") || source.includes("/>") ? Lang.Tsx : Lang.TypeScript, source).root();
+	const lang = source.includes("</") || source.includes("/>") ? Lang.Tsx : Lang.TypeScript;
+	const root = parse(lang, source).root();
 	const edits = [];
 	const needsBodyRename = /* @__PURE__ */ new Set();
 	const importStmts = root.findAll({ rule: {

package/dist/codemods/v3/function-test-run-rename/scripts/transform.js

@@ -0,0 +1,48 @@
+import * as path from "pathe";
+//#region codemods/v3/function-test-run-rename/scripts/transform.ts
+const TOKEN_SEPARATOR = "(?:[ \\t]|\\\\\\r?\\n)+";
+const COMMAND_PATTERN = new RegExp(`(?<![\\w.-])(tailor(?:-sdk)?(?:\\.(?:cmd|ps1|exe))?${TOKEN_SEPARATOR}function${TOKEN_SEPARATOR})test-run(?![\\w.-])`, "g");
+function replaceCommand(value) {
+	return value.replace(COMMAND_PATTERN, "$1run");
+}
+function transformText(source) {
+	const updated = replaceCommand(source);
+	return updated === source ? null : updated;
+}
+function transformPackageJson(source) {
+	let parsed;
+	try {
+		parsed = JSON.parse(source);
+	} catch {
+		return null;
+	}
+	let modified = false;
+	const scripts = parsed.scripts;
+	if (typeof scripts === "object" && scripts != null && !Array.isArray(scripts)) for (const [name, value] of Object.entries(scripts)) {
+		if (typeof value !== "string") continue;
+		const updated = replaceCommand(value);
+		if (updated !== value) {
+			scripts[name] = updated;
+			modified = true;
+		}
+	}
+	if (!modified) return null;
+	const trailing = source.endsWith("\n") ? "\n" : "";
+	return JSON.stringify(parsed, null, 2) + trailing;
+}
+/**
+* Rename `tailor function test-run` invocations to `tailor function run`.
+*
+* `test-run` remains as a deprecated alias until v3, where only `run` is
+* recognized.
+* @param source - File contents
+* @param filePath - Absolute path to the file (used to dispatch package.json vs text)
+* @returns Transformed source or null when nothing matched.
+*/
+function transform(source, filePath) {
+	if (!source.includes("test-run")) return null;
+	if (path.extname(filePath).toLowerCase() === ".json") return transformPackageJson(source);
+	return transformText(source);
+}
+//#endregion
+export { transform as default };

package/dist/index.js

@@ -116,6 +116,7 @@ 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";
@@ -124,6 +125,50 @@ const V2_NEXT_9 = "2.0.0-next.9";
 const V2_NEXT_11 = "2.0.0-next.11";
 /** All registered codemods, in registration order. */
 const allCodemods = [
+	{
+		id: "v2/type-only-imports",
+		name: "Type-only imports → `import type`",
+		description: [
+			"The v2 CLI runs TypeScript by stripping types from each file in isolation,",
+			"with no cross-file type information. A plain (non-`type`) import of a",
+			"type-only export therefore survives stripping and fails when the module",
+			"loads:",
+			"",
+			"```",
+			"SyntaxError: The requested module './types.ts' does not provide an export named 'Row'",
+			"```",
+			"",
+			"`tailor generate` / `tailor deploy` stop at this error before doing any work.",
+			"Import types with `import type` (or the inline `type` modifier), and",
+			"re-export them with `export type`, in every module the CLI loads. Generated",
+			"Kysely types (`DB`, `Insertable`, `Selectable`, table row types) are almost",
+			"entirely type-only, so v1 projects typically hit this in many files at once.",
+			"Set `\"verbatimModuleSyntax\": true` in tsconfig.json to catch every violation",
+			"at typecheck; projects scaffolded by v2 `tailor init` enable it by default."
+		].join("\n"),
+		since: "1.0.0",
+		until: "2.0.0",
+		prereleaseUntil: V2_NEXT_3,
+		examples: [{
+			before: "import { DB, getDB } from \"./generated/db\";",
+			after: "import { type DB, getDB } from \"./generated/db\";"
+		}],
+		prompt: [
+			"In Tailor SDK v2 the CLI loads TypeScript by stripping types from each file",
+			"in isolation, so type-only exports do not exist at runtime and plain imports",
+			"of them fail to load with \"does not provide an export named '<name>'\".",
+			"Migrate the project so every type-only import and re-export is marked:",
+			"",
+			"1. Add `\"verbatimModuleSyntax\": true` to compilerOptions in tsconfig.json.",
+			"2. Run `tsc --noEmit` and fix every reported violation: add the `type`",
+			"   modifier to type-only named imports (`import type { Row }` or",
+			"   `import { type Row, marker }`) and change type-only re-exports to",
+			"   `export type { ... }`.",
+			"",
+			"Only add `type` modifiers; do not reorder, remove, or otherwise change",
+			"imports that are used as values."
+		].join("\n")
+	},
 	{
 		id: "v2/define-generators-to-plugins",
 		name: "defineGenerators → definePlugins",
@@ -1258,7 +1303,7 @@ const allCodemods = [
 	{
 		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), plus a new `--upsert` flag to update existing rows instead of failing on duplicate ids.",
+		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`, entity-name arguments), plus a new `--upsert` flag to update existing rows instead of failing on duplicate ids.",
 		since: "1.0.0",
 		until: "2.0.0",
 		prereleaseUntil: V2_NEXT_9,
@@ -1283,8 +1328,8 @@ const allCodemods = [
 			"- 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,",
+			"  `tailor seed apply [options] [entities...]` (same options: --machine-user/-m,",
+			"  --namespace/-n, --skip-idp, --truncate, --yes, and entity-name arguments,",
 			"  plus a new --upsert flag to update existing rows instead of failing on",
 			"  duplicate ids).",
 			"- Replace `node <distPath>/exec.mjs validate [path]` with",
@@ -1405,7 +1450,7 @@ const allCodemods = [
 	{
 		id: "v2/node-minimum-22-15-0",
 		name: "Node.js minimum version raised to 22.15.0",
-		description: "v2 requires Node.js **22.15.0** or later. This is the first version that includes `module.registerHooks()`, which the SDK uses to register its TypeScript loader hook synchronously in the main thread. No source change is required; ensure your environment runs Node.js 22.15.0+.",
+		description: "v2 requires Node.js **22.15.0** or later. This is the first version that includes `module.registerHooks()`, which the SDK uses to register its TypeScript loader hook synchronously in the main thread. The actual floor is now **22.18.0**: Node 22.15.0–22.17.x has a bug ([nodejs/node#58607](https://github.com/nodejs/node/issues/58607)) that crashes `tailor seed validate` when requiring `node:`-scheme-only builtins such as `node:sqlite`, fixed upstream in 22.18.0. No source change is required; ensure your environment runs Node.js 22.18.0+.",
 		since: "1.0.0",
 		until: "2.0.0",
 		notice: true
@@ -1451,6 +1496,35 @@ const allCodemods = [
 			"editing `tailor.d.ts`: it is generated and will be overwritten, and embedding",
 			"env values there is what leaked configured secrets into version control."
 		].join("\n")
+	},
+	{
+		id: "v3/function-test-run-rename",
+		name: "function test-run → function run",
+		description: "Rename `tailor function test-run` invocations to `tailor function run`. `test-run` remains as a deprecated alias until it is removed in v3.",
+		since: "1.22.0",
+		until: "3.0.0",
+		scriptPath: "v3/function-test-run-rename/scripts/transform.js",
+		filePatterns: [
+			"**/package.json",
+			"**/*.{sh,bash,zsh,ps1,cmd,bat,yml,yaml}",
+			"**/*.md",
+			"**/*.{ts,tsx,mts,cts,js,jsx,mjs,cjs}"
+		],
+		legacyPatterns: [/\bfunction[\s\\]{1,16}test-run(?![\w.-])/],
+		sourceStringLegacyPatterns: [/\bfunction[\s\\]{1,16}test-run(?![\w.-])/],
+		examples: [{
+			lang: "sh",
+			before: "tailor function test-run resolvers/add.ts --arg '{\"a\":1,\"b\":2}'",
+			after: "tailor function run resolvers/add.ts --arg '{\"a\":1,\"b\":2}'"
+		}],
+		prompt: [
+			"The `tailor function test-run` subcommand is renamed to `tailor function run`;",
+			"the old name is removed in v3. Replace any remaining `function test-run`",
+			"invocations the codemod did not rewrite (e.g. wrapped across lines or invoked",
+			"through a package runner such as `npx @tailor-platform/sdk`) with",
+			"`function run`. Leave prose that merely mentions the old subcommand name",
+			"unchanged unless it documents a command to type."
+		].join("\n")
 	}
 ];
 /**
@@ -2174,7 +2248,7 @@ function printLlmReview(review) {
 	}
 	process.stderr.write(`\nPrompt for an LLM:\n${review.prompt.trim()}\n`);
 }
-runMain(defineCommand({
+const main = defineCommand({
 	name: packageName,
 	description: packageJson.description ?? "Codemod runner for Tailor Platform SDK upgrades",
 	subCommands: { list: listCommand },
@@ -2256,6 +2330,7 @@ human-readable form, so \`stdout\` stays pure JSON for piping.`,
 		process.stdout.write(JSON.stringify(output) + "\n");
 		if (output.errors.length > 0) process.exit(1);
 	}
-}), { version: packageVersion });
+});
+runMain(main, { version: packageVersion });
 //#endregion
 export {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tailor-platform/sdk-codemod",
-  "version": "0.4.2",
+  "version": "0.6.0",
   "description": "Codemod runner for Tailor Platform SDK upgrades",
   "license": "MIT",
   "repository": {
@@ -16,7 +16,7 @@
   ],
   "type": "module",
   "dependencies": {
-    "@ast-grep/napi": "0.45.0",
+    "@ast-grep/napi": "0.45.1",
     "diff": "9.0.0",
     "pathe": "2.0.3",
     "picomatch": "4.0.5",
@@ -29,7 +29,7 @@
     "@types/node": "24.13.3",
     "@types/picomatch": "4.0.3",
     "@types/semver": "7.8.0",
-    "eslint-plugin-zod": "4.9.0",
+    "eslint-plugin-zod": "4.9.1",
     "oxlint": "1.76.0",
     "tsdown": "0.22.14",
     "typescript": "6.0.3",
@@ -38,7 +38,7 @@
   },
   "engines": {
     "bun": ">=1.2.0",
-    "node": ">=22.15.0"
+    "node": ">=22.18.0"
   },
   "scripts": {
     "build": "tsdown",