run402-mcp 3.8.3 → 4.0.1

package/sdk/dist/namespaces/deploy.js

@@ -1960,7 +1960,7 @@ const DEPLOYABLE_SPEC_FIELDS = [
 ];
 const BASE_SPEC_FIELDS = new Set(["release", "release_id"]);
 const DATABASE_SPEC_FIELDS = new Set(["migrations", "expose", "zero_downtime"]);
-const MIGRATION_SPEC_FIELDS = new Set(["id", "checksum", "sql", "sql_ref", "transaction"]);
+const MIGRATION_SPEC_FIELDS = new Set(["id", "name", "checksum", "sql", "sql_ref", "transaction"]);
 const FUNCTIONS_SPEC_FIELDS = new Set(["replace", "patch"]);
 const FUNCTIONS_PATCH_FIELDS = new Set(["set", "delete"]);
 const FUNCTION_SPEC_FIELDS = new Set([
@@ -1995,6 +1995,7 @@ const I18N_LOCALE_TAG_REGEX = /^[A-Za-z0-9][A-Za-z0-9._-]{0,63}$/;
 const I18N_COOKIE_NAME_REGEX = /^[!#$%&'*+\-.^_`|~0-9A-Za-z]+$/;
 const I18N_MAX_LOCALES = 50;
 const I18N_MAX_DETECT_SOURCES = 10;
+const MIGRATION_NAME_RE = /^[a-zA-Z0-9][a-zA-Z0-9_-]{0,63}$/;
 function validateSpec(spec) {
     if (!spec || typeof spec !== "object") {
         throw new Run402DeployError("ReleaseSpec must be an object", {
@@ -2072,15 +2073,38 @@ function validateDatabaseSpec(database) {
         if (!Array.isArray(obj.migrations)) {
             throw invalidSpec("ReleaseSpec.database.migrations must be an array", "database.migrations");
         }
+        const seenNames = new Map();
         for (const [index, migration] of obj.migrations.entries()) {
             const m = requireObject(migration, `database.migrations.${index}`);
             validateKnownFields(m, `database.migrations.${index}`, MIGRATION_SPEC_FIELDS);
+            validateMigrationIdentity(m, index, seenNames);
         }
     }
     if (obj.expose !== undefined) {
         requireObject(obj.expose, "database.expose");
     }
 }
+function validateMigrationIdentity(migration, index, seenNames) {
+    const resource = `database.migrations.${index}`;
+    const hasId = hasOwn(migration, "id") && migration.id !== undefined;
+    const hasName = hasOwn(migration, "name") && migration.name !== undefined;
+    if (hasId && hasName) {
+        throw invalidSpec(`ReleaseSpec.${resource} declares both id and name; use exactly one`, resource);
+    }
+    if (!hasId && !hasName) {
+        throw invalidSpec(`ReleaseSpec.${resource} must declare exactly one of id or name`, resource);
+    }
+    if (hasName) {
+        if (typeof migration.name !== "string" || !MIGRATION_NAME_RE.test(migration.name)) {
+            throw invalidSpec(`ReleaseSpec.${resource}.name must match /^[a-zA-Z0-9][a-zA-Z0-9_-]{0,63}$/`, `${resource}.name`);
+        }
+        const firstIndex = seenNames.get(migration.name);
+        if (firstIndex !== undefined) {
+            throw invalidSpec(`ReleaseSpec.${resource}.name duplicates database.migrations.${firstIndex}.name ${JSON.stringify(migration.name)}`, `${resource}.name`);
+        }
+        seenNames.set(migration.name, index);
+    }
+}
 function validateFunctionsSpec(functions) {
     if (functions === undefined)
         return;
@@ -3021,6 +3045,7 @@ async function normalizeReleaseSpec(client, spec, opts = {}) {
         }
         if (spec.database.migrations && spec.database.migrations.length > 0) {
             db.migrations = await Promise.all(spec.database.migrations.map(async (m) => normalizeMigration(client, spec.project, m, rememberRelease, opts)));
+            assertUniqueMigrationIds(db.migrations);
         }
         normalized.database = db;
     }
@@ -3438,8 +3463,10 @@ async function normalizeAssetSlice(slice, remember) {
     return out;
 }
 async function normalizeMigration(client, projectId, m, remember, opts = {}) {
-    if (!m.id) {
-        throw new Run402DeployError("MigrationSpec.id is required", {
+    const originalId = "id" in m ? m.id : undefined;
+    const name = "name" in m ? m.name : undefined;
+    if (!originalId && !name) {
+        throw new Run402DeployError("MigrationSpec.id or MigrationSpec.name is required", {
             code: "INVALID_SPEC",
             phase: "validate",
             resource: "database.migrations",
@@ -3451,37 +3478,42 @@ async function normalizeMigration(client, projectId, m, remember, opts = {}) {
     let sql_ref;
     let sql;
     let checksum;
+    let contentHash;
+    const identityLabel = originalId ?? name ?? "<unknown>";
     if (m.sql_ref) {
         sql_ref = m.sql_ref;
+        contentHash = m.sql_ref.sha256;
         checksum = m.checksum ?? m.sql_ref.sha256;
     }
     else if (m.sql !== undefined) {
         const bytes = new TextEncoder().encode(m.sql);
         const sha256 = await sha256Hex(bytes);
+        contentHash = sha256;
         if (opts.inlineMigrationSql) {
             sql = m.sql;
         }
         else {
             const ref = { sha256, size: bytes.byteLength, contentType: "application/sql" };
-            remember({ ref, reader: makeBytesReader(bytes, `migration:${m.id}`) });
+            remember({ ref, reader: makeBytesReader(bytes, `migration:${identityLabel}`) });
             sql_ref = ref;
         }
         checksum = m.checksum ?? sha256;
     }
     else {
-        throw new Run402DeployError(`MigrationSpec ${m.id} must include sql or sql_ref`, {
+        throw new Run402DeployError(`MigrationSpec ${identityLabel} must include sql or sql_ref`, {
             code: "INVALID_SPEC",
             phase: "validate",
-            resource: `database.migrations.${m.id}`,
+            resource: `database.migrations.${identityLabel}`,
             retryable: false,
             fix: {
                 action: "set_field",
-                path: `database.migrations.${m.id}.sql`,
+                path: `database.migrations.${identityLabel}.sql`,
             },
             context: "validating spec",
         });
     }
-    const out = { id: m.id, checksum };
+    const id = originalId ?? `${name}_${contentHash.slice(0, 16)}`;
+    const out = { id, checksum };
     if (sql_ref)
         out.sql_ref = sql_ref;
     if (sql !== undefined)
@@ -3493,6 +3525,23 @@ async function normalizeMigration(client, projectId, m, remember, opts = {}) {
     void client;
     void projectId;
 }
+function assertUniqueMigrationIds(migrations) {
+    const seen = new Map();
+    for (const [index, migration] of migrations.entries()) {
+        const firstIndex = seen.get(migration.id);
+        if (firstIndex !== undefined) {
+            throw new Run402DeployError(`ReleaseSpec.database.migrations.${index}.id duplicates database.migrations.${firstIndex}.id ${JSON.stringify(migration.id)}`, {
+                code: "INVALID_SPEC",
+                phase: "validate",
+                resource: `database.migrations.${index}.id`,
+                retryable: false,
+                fix: { action: "set_field", path: `database.migrations.${index}.id` },
+                context: "validating spec",
+            });
+        }
+        seen.set(migration.id, index);
+    }
+}
 // ─── Content source resolution ───────────────────────────────────────────────
 async function resolveContent(source, label) {
     // Pre-resolved ContentRef — pass through, no reader needed (caller is