@njdamstra/appwrite-utils-cli 1.11.6 → 1.11.8

package/dist/migrations/migrateStrings.js

@@ -250,6 +250,31 @@ export async function executeMigrationPlan(adapter, options) {
             skipped += entries.length;
             continue;
         }
+        // Temporarily set ALL required attributes in this collection to optional.
+        // Appwrite rejects partial updateRow calls if a required attribute is missing
+        // from the payload — even if the field exists in the document. This affects
+        // every data-copy step during migration.
+        const schemaRes = await tryAwaitWithRetry(() => adapter.getTable({ databaseId: first.databaseId, tableId: first.collectionId }));
+        const allAttrs = schemaRes?.data?.attributes || schemaRes?.data?.columns || [];
+        const originallyRequired = allAttrs
+            .filter((a) => a.required === true && a.status === "available")
+            .map((a) => a.key);
+        if (originallyRequired.length > 0) {
+            MessageFormatter.info(`  Temporarily setting ${originallyRequired.length} required attribute(s) to optional...`, { prefix: "Execute" });
+            for (const key of originallyRequired) {
+                try {
+                    await tryAwaitWithRetry(() => adapter.updateAttribute({
+                        databaseId: first.databaseId,
+                        tableId: first.collectionId,
+                        key,
+                        required: false,
+                    }));
+                }
+                catch {
+                    // Non-fatal — attribute might not support updating required
+                }
+            }
+        }
         // Migrate each attribute in this collection
         for (const entry of entries) {
             const cpEntry = getOrCreateCheckpointEntry(checkpoint, entry);
@@ -275,6 +300,25 @@ export async function executeMigrationPlan(adapter, options) {
                 MessageFormatter.error(`  ${entry.attributeKey}: FAILED — ${cpEntry.error}`, undefined, { prefix: "Execute" });
             }
         }
+        // Restore required flags for ALL originally-required attributes.
+        // This covers both migrated attributes (recreated as optional) and
+        // non-migrated attributes (temporarily set to optional above).
+        if (originallyRequired.length > 0) {
+            MessageFormatter.info(`  Restoring ${originallyRequired.length} required attribute(s)...`, { prefix: "Execute" });
+            for (const key of originallyRequired) {
+                try {
+                    await tryAwaitWithRetry(() => adapter.updateAttribute({
+                        databaseId: first.databaseId,
+                        tableId: first.collectionId,
+                        key,
+                        required: true,
+                    }));
+                }
+                catch {
+                    MessageFormatter.info(`  Warning: could not restore required flag for ${key}`, { prefix: "Execute" });
+                }
+            }
+        }
         // After collection completes, offer to update local YAML
         const successInGroup = entries.filter((e) => {
             const cp = findCheckpointEntry(checkpoint, e);
@@ -426,21 +470,8 @@ async function migrateOneAttribute(adapter, entry, cpEntry, checkpoint, checkpoi
         advance("backup_deleted");
     }
     // Step 9: Mark completed
-    // If the original attribute was required, update it now (after data is in place)
-    if (entry.isRequired) {
-        try {
-            await tryAwaitWithRetry(() => adapter.updateAttribute({
-                databaseId,
-                tableId: collectionId,
-                key: attributeKey,
-                required: true,
-            }));
-        }
-        catch {
-            // Non-fatal — attribute is migrated, just not set back to required
-            MessageFormatter.info(`    Warning: could not set ${attributeKey} back to required`, { prefix: "Migrate" });
-        }
-    }
+    // NOTE: required flag is restored AFTER all attributes in the collection
+    // are migrated, to avoid partial-update validation errors on other attributes.
     advance("completed");
 }
 // ────────────────────────────────────────────────────────

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@njdamstra/appwrite-utils-cli",
   "description": "Appwrite Utility Functions to help with database management, data conversion, data import, migrations, and much more. Meant to be used as a CLI tool, I do not recommend installing this in frontend environments.",
-  "version": "1.11.6",
+  "version": "1.11.8",
   "main": "dist/main.js",
   "type": "module",
   "repository": {

package/src/migrations/migrateStrings.ts

@@ -338,6 +338,40 @@ export async function executeMigrationPlan(
       continue;
     }
 
+    // Temporarily set ALL required attributes in this collection to optional.
+    // Appwrite rejects partial updateRow calls if a required attribute is missing
+    // from the payload — even if the field exists in the document. This affects
+    // every data-copy step during migration.
+    const schemaRes = await tryAwaitWithRetry(() =>
+      adapter.getTable({ databaseId: first.databaseId, tableId: first.collectionId })
+    );
+    const allAttrs: any[] =
+      schemaRes?.data?.attributes || schemaRes?.data?.columns || [];
+    const originallyRequired = allAttrs
+      .filter((a: any) => a.required === true && a.status === "available")
+      .map((a: any) => a.key as string);
+
+    if (originallyRequired.length > 0) {
+      MessageFormatter.info(
+        `  Temporarily setting ${originallyRequired.length} required attribute(s) to optional...`,
+        { prefix: "Execute" }
+      );
+      for (const key of originallyRequired) {
+        try {
+          await tryAwaitWithRetry(() =>
+            adapter.updateAttribute({
+              databaseId: first.databaseId,
+              tableId: first.collectionId,
+              key,
+              required: false,
+            } as any)
+          );
+        } catch {
+          // Non-fatal — attribute might not support updating required
+        }
+      }
+    }
+
     // Migrate each attribute in this collection
     for (const entry of entries) {
       const cpEntry = getOrCreateCheckpointEntry(checkpoint, entry);
@@ -382,6 +416,33 @@ export async function executeMigrationPlan(
       }
     }
 
+    // Restore required flags for ALL originally-required attributes.
+    // This covers both migrated attributes (recreated as optional) and
+    // non-migrated attributes (temporarily set to optional above).
+    if (originallyRequired.length > 0) {
+      MessageFormatter.info(
+        `  Restoring ${originallyRequired.length} required attribute(s)...`,
+        { prefix: "Execute" }
+      );
+      for (const key of originallyRequired) {
+        try {
+          await tryAwaitWithRetry(() =>
+            adapter.updateAttribute({
+              databaseId: first.databaseId,
+              tableId: first.collectionId,
+              key,
+              required: true,
+            } as any)
+          );
+        } catch {
+          MessageFormatter.info(
+            `  Warning: could not restore required flag for ${key}`,
+            { prefix: "Execute" }
+          );
+        }
+      }
+    }
+
     // After collection completes, offer to update local YAML
     const successInGroup = entries.filter((e) => {
       const cp = findCheckpointEntry(checkpoint, e);
@@ -630,25 +691,8 @@ async function migrateOneAttribute(
   }
 
   // Step 9: Mark completed
-  // If the original attribute was required, update it now (after data is in place)
-  if (entry.isRequired) {
-    try {
-      await tryAwaitWithRetry(() =>
-        adapter.updateAttribute({
-          databaseId,
-          tableId: collectionId,
-          key: attributeKey,
-          required: true,
-        } as any)
-      );
-    } catch {
-      // Non-fatal — attribute is migrated, just not set back to required
-      MessageFormatter.info(
-        `    Warning: could not set ${attributeKey} back to required`,
-        { prefix: "Migrate" }
-      );
-    }
-  }
+  // NOTE: required flag is restored AFTER all attributes in the collection
+  // are migrated, to avoid partial-update validation errors on other attributes.
   advance("completed");
 }