@njdamstra/appwrite-utils-cli 1.11.6 → 1.11.7

package/dist/migrations/migrateStrings.js

@@ -275,6 +275,27 @@ export async function executeMigrationPlan(adapter, options) {
                 MessageFormatter.error(`  ${entry.attributeKey}: FAILED — ${cpEntry.error}`, undefined, { prefix: "Execute" });
             }
         }
+        // Restore required flags after all attributes in this collection are done.
+        // This must happen AFTER all migrations to avoid partial-update validation
+        // errors (Appwrite rejects updateRow if a required attribute is missing
+        // from the payload, even for partial updates).
+        const completedRequired = entries.filter((e) => {
+            const cp = findCheckpointEntry(checkpoint, e);
+            return cp?.phase === "completed" && e.isRequired;
+        });
+        for (const entry of completedRequired) {
+            try {
+                await tryAwaitWithRetry(() => adapter.updateAttribute({
+                    databaseId: entry.databaseId,
+                    tableId: entry.collectionId,
+                    key: entry.attributeKey,
+                    required: true,
+                }));
+            }
+            catch {
+                MessageFormatter.info(`  Warning: could not set ${entry.attributeKey} back to required`, { prefix: "Execute" });
+            }
+        }
         // After collection completes, offer to update local YAML
         const successInGroup = entries.filter((e) => {
             const cp = findCheckpointEntry(checkpoint, e);
@@ -426,21 +447,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.7",
   "main": "dist/main.js",
   "type": "module",
   "repository": {

package/src/migrations/migrateStrings.ts

@@ -382,6 +382,32 @@ export async function executeMigrationPlan(
       }
     }
 
+    // Restore required flags after all attributes in this collection are done.
+    // This must happen AFTER all migrations to avoid partial-update validation
+    // errors (Appwrite rejects updateRow if a required attribute is missing
+    // from the payload, even for partial updates).
+    const completedRequired = entries.filter((e) => {
+      const cp = findCheckpointEntry(checkpoint, e);
+      return cp?.phase === "completed" && e.isRequired;
+    });
+    for (const entry of completedRequired) {
+      try {
+        await tryAwaitWithRetry(() =>
+          adapter.updateAttribute({
+            databaseId: entry.databaseId,
+            tableId: entry.collectionId,
+            key: entry.attributeKey,
+            required: true,
+          } as any)
+        );
+      } catch {
+        MessageFormatter.info(
+          `  Warning: could not set ${entry.attributeKey} back to required`,
+          { prefix: "Execute" }
+        );
+      }
+    }
+
     // After collection completes, offer to update local YAML
     const successInGroup = entries.filter((e) => {
       const cp = findCheckpointEntry(checkpoint, e);
@@ -630,25 +656,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");
 }