@tailor-platform/sdk 1.49.0 → 1.50.0

package/CHANGELOG.md

@@ -1,5 +1,33 @@
 # @tailor-platform/sdk
 
+## 1.50.0
+
+### Minor Changes
+
+- [#1223](https://github.com/tailor-platform/sdk/pull/1223) [`c2f078d`](https://github.com/tailor-platform/sdk/commit/c2f078d6fb82d42bcc8c040e020b34072a2d2eb6) Thanks [@toiroakr](https://github.com/toiroakr)! - Improve `tailordb migration` handling of data-loss-possible changes:
+
+  - Removed fields (`field_removed`) and removed types (`type_removed`) are now reported as **warnings** during `tailordb migration generate`, not silent changes. They are no longer dropped before `migrate.ts` runs: the field/type stays available during the Pre-migration phase so that scripts can read it (e.g. `innerJoin` through a foreign key that is being dropped in the same migration), then the physical drop happens in the Post-migration phase.
+  - Add `tailordb migration script <number>` subcommand to add a `migrate.ts` (and `db.ts`) template to an existing migration directory. Useful for warning-tier changes where you want a custom data migration even though one was not generated automatically.
+  - `migrate.ts` is now executed whenever the file exists on disk for a pending migration, regardless of whether the diff originally required a script. Breaking changes still hard-require a script as before.
+
+### Patch Changes
+
+- [#1194](https://github.com/tailor-platform/sdk/pull/1194) [`4c227cc`](https://github.com/tailor-platform/sdk/commit/4c227cc404e51331c0514b0aaa07d96d8940c347) Thanks [@toiroakr](https://github.com/toiroakr)! - Colocate `src/vitest/` tests next to their sources (drop the
+  `src/vitest/__tests__/` directory). Vitest discovers the test files
+  via the existing `**/?(*.)+(spec|test).ts` include pattern, so the
+  `**/__tests__/**/*.ts` entry has been removed from `vitest.config.ts`.
+  The nested integration runner moves from
+  `src/vitest/__tests__/integration/` to `src/vitest/integration/`. Pure
+  refactor: no public API or behavior changes.
+
+- [#1207](https://github.com/tailor-platform/sdk/pull/1207) [`c0b392d`](https://github.com/tailor-platform/sdk/commit/c0b392d95f81bcb2a4bfd9e856183222da87ef06) Thanks [@toiroakr](https://github.com/toiroakr)! - Fix `tailor deploy` so an intermediate migration's data script can still read fields that a later migration removes. Each migration's pre/post phase now submits the schema state reconstructed up to that migration (initial baseline + diffs through N), instead of the FINAL post-all-migrations schema. Previously, removals declared in later migrations leaked into earlier migrations' pre-phase and caused `field 'X' not found` failures at script execution time.
+
+- [#1208](https://github.com/tailor-platform/sdk/pull/1208) [`59d7e0e`](https://github.com/tailor-platform/sdk/commit/59d7e0e7c1b42dcd1571365780114d596e44350c) Thanks [@toiroakr](https://github.com/toiroakr)! - fix(skills): rename bundled skill directory to `agent-skills/` so `gh skill install tailor-platform/sdk` resolves a single skill. `gh skill` uses a recursive `**/skills/*/SKILL.md` match and was picking up both `skills/tailor-sdk/` (the public mirror) and `packages/sdk/skills/tailor-sdk/` (the npm bundle), erroring with conflicting names. The bundled copy is now under `packages/sdk/agent-skills/` (excluded from gh's match), while the repo-root `skills/` mirror remains the single discovery target for `gh skill install` and `npx skills add <repo>`. `npx tailor-sdk skills install` continues to work via the bundled `agent-skills/` path inside the published package.
+
+- [#1217](https://github.com/tailor-platform/sdk/pull/1217) [`b827cf9`](https://github.com/tailor-platform/sdk/commit/b827cf95c0c048959c269f7ed9a31fdc81a5eb13) Thanks [@renovate](https://github.com/apps/renovate)! - fix(deps): update dependency @toiroakr/read-multiline to v0.4.1
+
+- [#1184](https://github.com/tailor-platform/sdk/pull/1184) [`dee2ad7`](https://github.com/tailor-platform/sdk/commit/dee2ad7199b2da238b7e9edea2d9bb9605033456) Thanks [@toiroakr](https://github.com/toiroakr)! - Fix `tailor deploy` so decimal fields without an explicit `scale` no longer show spurious drift against the platform (which materializes the default `6`). Deploy now plans and applies through the same snapshot pipeline as `tailordb migrate`.
+
 ## 1.49.0
 
 ### Minor Changes

package/README.md

@@ -60,7 +60,7 @@ npx tailor-sdk skills install -a codex -y
 ```
 
 This uses the `skills` CLI under the hood, sourcing the skill from
-`node_modules/@tailor-platform/sdk/skills` so the skill version always matches
+`node_modules/@tailor-platform/sdk/agent-skills` so the skill version always matches
 the installed SDK version. Files are copied (not symlinked) so they survive
 `pnpm install` wiping `node_modules`.
 

package/dist/cli/index.mjs

@@ -2,7 +2,7 @@
 
 import { G as PATScope, R as AuthInvokerSchema, c as fetchUserInfo, d as initOperatorClient, h as userAgent, i as fetchAll, j as FunctionExecution_Type, n as closeConnectionPool, o as fetchPaged, s as fetchPlatformMachineUserToken, u as initOAuth2Client } from "../client-_kHh0Pip.mjs";
 import { n as logger, r as styles } from "../logger-DTNAMYGy.mjs";
-import { $ as deleteCommand$3, At as executionsCommand, C as listCommand$13, Cn as paginationArgs, Ct as jobsCommand, E as resumeCommand, Et as startCommand, F as showCommand, Ft as getCommand$1, G as getCommand$5, H as treeCommand, I as logBetaWarning, Lt as deploy, N as generateCommand$1, Nt as functionExecutionStatusToString, O as listCommand$12, Ot as getCommand$6, Pt as formatKeyValueTable, R as removeCommand$1, Rt as executeScript, Sn as pagedLogArgs, T as healthCommand, Tn as workspaceArgs, U as listCommand$11, Vt as parseMigrationLabelNumber, Y as listCommand$10, Z as getCommand$4, Zt as formatMigrationNumber, _n as defineAppCommand, _t as webhookCommand, at as getCommand$3, b as createCommand$4, bn as deploymentArgs, bt as listCommand$6, c as listCommand$14, ct as tokenCommand, dn as trnPrefix, dt as generate, en as getMigrationFilePath, f as restoreCommand, ft as listCommand$7, g as getCommand$7, gn as assertWritable, i as updateCommand$4, in as loadDiff, j as truncateCommand, ln as getNamespacesWithMigrations, lt as listCommand$8, m as listCommand$15, mn as apiCommand, mt as getCommand$2, o as removeCommand, pn as prompt, q as updateCommand$2, r as queryCommand, rn as isValidMigrationNumber, rt as listCommand$9, t as isNativeTypeScriptRuntime, tn as getMigrationFiles, tt as createCommand$3, u as inviteCommand, un as sdkNameLabelKey, v as deleteCommand$4, vn as commonArgs, vt as triggerCommand, wn as toPageDirection, xn as isVerbose, yn as confirmationArgs, z as updateCommand$3 } from "../runtime-oZgK353r.mjs";
+import { $ as listCommand$10, An as workspaceArgs, At as startCommand, B as logBetaWarning, C as listCommand$13, Cn as configArg, Dn as pagedLogArgs, Dt as jobsCommand, E as resumeCommand, En as isVerbose, F as writeDbTypesFile, Gt as parseMigrationLabelNumber, H as removeCommand$1, Ht as executeScript, I as getConfiguredEditorCommand, K as treeCommand, L as openInConfiguredEditor, Lt as functionExecutionStatusToString, Mt as getCommand$6, N as generateCommand$1, O as listCommand$12, On as paginationArgs, P as generateMigrationScript, Pt as executionsCommand, Rt as formatKeyValueTable, Sn as commonArgs, St as triggerCommand, T as healthCommand, Tn as deploymentArgs, U as updateCommand$3, Vt as deploy, Y as getCommand$5, Yt as INITIAL_SCHEMA_NUMBER, Z as updateCommand$2, _n as prompt, an as getMigrationFiles, at as createCommand$3, b as createCommand$4, bn as assertWritable, c as listCommand$14, cn as loadDiff, f as restoreCommand, ft as tokenCommand, g as getCommand$7, gt as listCommand$7, hn as trnPrefix, ht as generate, i as updateCommand$4, in as getMigrationFilePath, j as truncateCommand, kn as toPageDirection, ln as reconstructSnapshotFromMigrations, lt as getCommand$3, m as listCommand$15, mn as sdkNameLabelKey, o as removeCommand, pn as getNamespacesWithMigrations, pt as listCommand$8, q as listCommand$11, r as queryCommand, rt as deleteCommand$3, sn as isValidMigrationNumber, st as listCommand$9, t as isNativeTypeScriptRuntime, tn as formatMigrationNumber, tt as getCommand$4, u as inviteCommand, v as deleteCommand$4, vn as apiCommand, vt as getCommand$2, wn as confirmationArgs, wt as listCommand$6, xn as defineAppCommand, xt as webhookCommand, z as showCommand, zt as getCommand$1 } from "../runtime-B2K6JW7V.mjs";
 import { D as saveUserTokens, E as resolveTokens, O as writePlatformConfig, S as loadAccessToken, T as readPlatformConfig, _ as getDistDir, a as WorkflowJobSchema, b as deleteUserTokens, c as ExecutorSchema, i as resolveInlineSourcemap, l as INVOKER_EXPR, o as ResolverSchema, w as loadWorkspaceId, x as fetchLatestToken, y as loadConfig } from "../application-CZMzt9jL.mjs";
 import { t as multiline } from "../multiline-e3IpANmS.mjs";
 import { t as readPackageJson } from "../package-json-6Px8bDpG.mjs";
@@ -25,6 +25,7 @@ import * as crypto from "node:crypto";
 import * as http from "node:http";
 import open from "open";
 import * as rolldown from "rolldown";
+import * as fsPromises from "node:fs/promises";
 import { TraceMap, generatedPositionFor, originalPositionFor } from "@jridgewell/trace-mapping";
 import { spawn, spawnSync } from "node:child_process";
 import * as fs from "fs";
@@ -2716,7 +2717,7 @@ async function runSkillsInstaller(options) {
 //#endregion
 //#region src/cli/commands/skills/install.ts
 async function resolveBundledSkillsDir() {
-	return resolve(dirname(await resolvePackageJSON(import.meta.url)), "skills");
+	return resolve(dirname(await resolvePackageJSON(import.meta.url)), "agent-skills");
 }
 const DEFAULT_AGENT = "claude-code";
 const installCommand = defineAppCommand({
@@ -3571,6 +3572,91 @@ const erdCommand = defineCommand({
 	}
 });
 
+//#endregion
+//#region src/cli/commands/tailordb/migrate/script.ts
+/**
+* Script command for TailorDB migrations
+*
+* Adds a `migrate.ts` (and supporting `db.ts`) template to an existing
+* migration directory. Useful for warning-tier changes where users may
+* want to write a custom data migration even though the change does not
+* automatically require one.
+*/
+/**
+* Add a migrate.ts template to an existing migration directory.
+* @param {ScriptOptions} options - Command options
+*/
+async function script(options) {
+	logBetaWarning("tailordb migration");
+	let migrationNumber;
+	if (isValidMigrationNumber(options.number)) migrationNumber = parseInt(options.number, 10);
+	else if (/^[1-9]\d*$/.test(options.number)) {
+		migrationNumber = parseInt(options.number, 10);
+		if (migrationNumber > 9999) throw new Error(`Migration number ${options.number} is out of range. Expected 1-9999.`);
+	} else throw new Error(`Invalid migration number format: ${options.number}. Expected 4-digit format (e.g., 0001) or integer 1-9999 (e.g., 1).`);
+	if (migrationNumber === 0) throw new Error(`Migration ${options.number} is the initial schema snapshot and cannot have a migration script.`);
+	const { config } = await loadConfig(options.configPath);
+	const namespacesWithMigrations = getNamespacesWithMigrations(config, path.dirname(config.path));
+	if (namespacesWithMigrations.length === 0) throw new Error("No TailorDB services with migrations configuration found");
+	const targetNamespace = resolveTargetNamespace(namespacesWithMigrations, options.namespace);
+	const { migrationsDir } = namespacesWithMigrations.find((ns) => ns.namespace === targetNamespace);
+	const diffPath = getMigrationFilePath(migrationsDir, migrationNumber, "diff");
+	if (!fs$1.existsSync(diffPath)) throw new Error(`Migration ${options.number} not found in ${migrationsDir}. Expected ${diffPath}.`);
+	const migratePath = getMigrationFilePath(migrationsDir, migrationNumber, "migrate");
+	if (fs$1.existsSync(migratePath)) throw new Error(`Migration script already exists at ${migratePath}.`);
+	const diff = loadDiff(diffPath);
+	const previousSnapshot = reconstructSnapshotFromMigrations(migrationsDir, migrationNumber - 1);
+	if (!previousSnapshot) throw new Error(`Could not reconstruct previous schema for migration ${options.number}. Make sure migration ${0} exists.`);
+	const scriptContent = generateMigrationScript(diff);
+	await fsPromises.writeFile(migratePath, scriptContent);
+	await writeDbTypesFile(previousSnapshot, migrationsDir, migrationNumber, diff);
+	logger.success(`Added migration script for migration ${styles.bold(options.number)} in namespace ${styles.bold(targetNamespace)}`);
+	logger.info(`  Migration script: ${migratePath}`);
+	logger.info(`  DB types: ${getMigrationFilePath(migrationsDir, migrationNumber, "db")}`);
+	logger.newline();
+	logger.log("Edit the script to implement your data migration logic.");
+	logger.log("It will be executed by 'tailor-sdk deploy' between Pre and Post phases.");
+	const editor = getConfiguredEditorCommand();
+	if (!editor) return;
+	logger.newline();
+	logger.info(`Opening ${path.basename(migratePath)} in ${editor}...`);
+	try {
+		await openInConfiguredEditor(migratePath);
+	} catch {
+		return;
+	}
+}
+function resolveTargetNamespace(namespacesWithMigrations, requested) {
+	if (requested) {
+		if (!namespacesWithMigrations.some((ns) => ns.namespace === requested)) throw new Error(`Namespace "${requested}" not found or does not have migrations configured`);
+		return requested;
+	}
+	if (namespacesWithMigrations.length === 1) return namespacesWithMigrations[0].namespace;
+	throw new Error(`Multiple TailorDB services found. Please specify namespace with --namespace flag: ${namespacesWithMigrations.map((ns) => ns.namespace).join(", ")}`);
+}
+const scriptCommand = defineAppCommand({
+	name: "script",
+	description: "Add a migration script (migrate.ts) template to an existing migration directory.",
+	args: z.object({
+		...configArg,
+		number: arg(z.string(), {
+			positional: true,
+			description: "Migration number to add a script to (e.g., 0001 or 1)"
+		}),
+		namespace: arg(z.string().optional(), {
+			alias: "n",
+			description: "Target TailorDB namespace (required if multiple namespaces exist)"
+		})
+	}).strict(),
+	run: async (args) => {
+		await script({
+			configPath: args.config,
+			number: args.number,
+			namespace: args.namespace
+		});
+	}
+});
+
 //#endregion
 //#region src/cli/commands/tailordb/migrate/set.ts
 /**
@@ -3750,14 +3836,16 @@ const statusCommand = defineAppCommand({
 *
 * Subcommands:
 * - generate: Generate migration files from schema differences
-* - set: Set migration checkpoint to a specific number
-* - status: Show migration status for TailorDB namespaces
+* - script:   Add a migrate.ts template to an existing migration
+* - set:      Set migration checkpoint to a specific number
+* - status:   Show migration status for TailorDB namespaces
 */
 const migrationCommand = defineCommand({
 	name: "migration",
 	description: "Manage TailorDB schema migrations.",
 	subCommands: {
 		generate: generateCommand$1,
+		script: scriptCommand,
 		set: setCommand,
 		status: statusCommand
 	}